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.

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

Update ImagePullPolicy to Always #186

Conversation

black-dragon74
Copy link
Collaborator

This PR updates the ImagePullPolicy for the must-gather pod to Always.

This adds value to the development phase where one can just delete the pod and have an updated one with the latest image.

Prior to this one had to use different tags for each test which was not ideal.

Note: This would not increase pod deploy timinigs in prod env as we do not update the prod image that often and ImagePullPolicy of always first checks the SHA sum from registry against local cache.

If the image is cached locally, it is not pulled again.

This PR updates the ImagePullPolicy for the must-gather
pod to Always.

This adds value to the development phase where one can just
delete the pod and have an updated one with the latest image.

Prior to this one had to use different tags for each test
which was not ideal.

Note: This would not increase pod deploy timinigs in prod env
as we do not update the prod image that often and ImagePullPolicy
of always first checks the SHA sum from registry against local cache.

If the image is cached locally, it is not pulled again.

Signed-off-by: Niraj Yadav <[email protected]>
@yati1998
Copy link
Member

yati1998 commented Aug 8, 2024

can you please check the CI failure. @black-dragon74

@black-dragon74
Copy link
Collaborator Author

can you please check the CI failure. @black-dragon74

It is unrelated to must-gather. The CI is unable to pull images from registry.

@black-dragon74
Copy link
Collaborator Author

/retest

1 similar comment
@black-dragon74
Copy link
Collaborator Author

/retest

Copy link
Contributor

openshift-ci bot commented Aug 16, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: black-dragon74, yati1998

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:
  • OWNERS [black-dragon74,yati1998]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot openshift-merge-bot bot merged commit f4cff39 into red-hat-storage:main Aug 16, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants