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 image tags #1645

Merged
merged 1 commit into from
Nov 16, 2020
Merged

Conversation

thesuperzapper
Copy link
Member

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:

  • admission-webhook
  • centraldashboard
  • jupyter-web-app
  • notebook-controller
  • kfam
  • profile-controller

Checklist:

  • Unit tests have been rebuilt:
    1. cd manifests/tests
    2. make generate-changed-only
    3. make test

@thesuperzapper
Copy link
Member Author

@Bobgy @Jeffwan can you review? (this needs to get into Kubeflow 1.2)

@Jeffwan
Copy link
Member

Jeffwan commented Nov 16, 2020

@thesuperzapper Thanks taking care of this

Is there a change list for these images?

@Jeffwan
Copy link
Member

Jeffwan commented Nov 16, 2020

I will cut a new rc release once they are merged into 1.2

/lgtm

@Jeffwan
Copy link
Member

Jeffwan commented Nov 16, 2020

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

@thesuperzapper
Copy link
Member Author

@Jeffwan I need someone to approvve adding me to OWNERs (see kubeflow/kubeflow#5408) so I can approve your PR.

@thesuperzapper
Copy link
Member Author

Also @Jeffwan, some stacks have hard coded these image tags in their manifests, we should get them to remove these:

  • IBM
  • Azure

@Jeffwan
Copy link
Member

Jeffwan commented Nov 16, 2020

/cc @animeshsingh @Tomcli @pvaneck @berndverst @PatrickXYS

Please have a check on the hard coded images. some components images needs to be updated

@Jeffwan
Copy link
Member

Jeffwan commented Nov 16, 2020

@thesuperzapper Do you have the permission to approve changes in these control plane components? Let me know when it's ready to merge

@thesuperzapper
Copy link
Member Author

@Bobgy, I need someone in the root OWNERS file to approve this

@Bobgy
Copy link
Contributor

Bobgy commented Nov 16, 2020

/lgtm
/approve
Sure

@k8s-ci-robot
Copy link
Contributor

[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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

k8s-ci-robot pushed a commit that referenced this pull request Nov 16, 2020
* update jupyter config with tolerations/selectors (#1644)

(cherry picked from commit 051f31a)

* update image tags (#1645)

(cherry picked from commit fdd3204)

* fix application selectors (#1646)

(cherry picked from commit 3e88eab)
@animeshsingh
Copy link
Contributor

@pvaneck will address the hardcoded image tags for IBM images

@berndverst
Copy link
Member

@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.

@thesuperzapper
Copy link
Member Author

@berndverst here is a link:
https://github.com/kubeflow/manifests/blob/master/stacks/azure/application/jupyter-web-app/base/kustomization.yaml#L10

@thesuperzapper
Copy link
Member Author

@Jeffwan I also see that openshift stack has some hard coded as well.

PS: the trick to finding them is searching for newTag:

@Jeffwan
Copy link
Member

Jeffwan commented Nov 17, 2020

@nakfour Can you follow @thesuperzapper 's suggestion and update master image to your v1.2 manifest?

@nakfour
Copy link
Member

nakfour commented Nov 19, 2020

@Jeffwan we use a custom JH image which we will submit a PR to update soon.

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.

all distributions of Kubeflow 1.1 have incorrect container images
7 participants