Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix checksum issue in image resource #481

Merged
merged 2 commits into from
Aug 4, 2022
Merged

Fix checksum issue in image resource #481

merged 2 commits into from
Aug 4, 2022

Conversation

bhati-pradeep
Copy link
Collaborator

  1. The resource nutanix_image was not considering checksum incase of local image upload. So have added logic to send checksum in headers while uploading image chunks.
  2. Have added logic in polling to check the image upload tasks (triggered by PC) status after upload from terraform finishes so that any failures in checks done by PC can be collected and displayed.
  3. Restrict user to update checksum, cluster references, source path and source uri as these are not allowed from api side

@bhati-pradeep bhati-pradeep linked an issue Aug 1, 2022 that may be closed by this pull request
Checksum is not considered while uploading image from local using nutanix_image resource #469
Closed
@bhati-pradeep bhati-pradeep changed the base branch from master to feat/1.7.0 August 2, 2022 06:23
@bhati-pradeep bhati-pradeep merged commit a284406 into feat/1.7.0 Aug 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@siddharth-kulshrestha siddharth-kulshrestha siddharth-kulshrestha approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Checksum is not considered while uploading image from local using nutanix_image resource
2 participants
@bhati-pradeep @siddharth-kulshrestha