-
Notifications
You must be signed in to change notification settings - Fork 852
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 image tags #1645
update image tags #1645
Conversation
@thesuperzapper Thanks taking care of this Is there a change list for these images? |
I will cut a new rc release once they are merged into 1.2 /lgtm |
Can we also include this change? https://github.com/kubeflow/kubeflow/pull/5404/files Metadata has already removed UI service. We need to have this change in central dashboard |
@Jeffwan I need someone to approvve adding me to OWNERs (see kubeflow/kubeflow#5408) so I can approve your PR. |
Also @Jeffwan, some stacks have hard coded these image tags in their manifests, we should get them to remove these:
|
/cc @animeshsingh @Tomcli @pvaneck @berndverst @PatrickXYS Please have a check on the hard coded images. some components images needs to be updated |
@thesuperzapper Do you have the permission to approve changes in these control plane components? Let me know when it's ready to merge |
@Bobgy, I need someone in the root OWNERS file to approve this |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: Bobgy, thesuperzapper 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:
Approvers can indicate their approval by writing |
(cherry picked from commit fdd3204)
@pvaneck will address the hardcoded image tags for IBM images |
@thesuperzapper @Jeffwan there were no hardcoded image tags in the Azure stack or kfdefs. Where did you see that there was something hard coded? The Azure stack refers to the shared component bases in the root of the repo and does not override images. |
@Jeffwan I also see that openshift stack has some hard coded as well. PS: the trick to finding them is searching for |
@nakfour Can you follow @thesuperzapper 's suggestion and update master image to your v1.2 manifest? |
@Jeffwan we use a custom JH image which we will submit a PR to update soon. |
Which issue is resolved by this Pull Request:
This resolves #1553 (for the components that belong to wg-notebook)
Description of your changes:
Updates the image tags for the following applications:
Checklist:
cd manifests/tests
make generate-changed-only
make test