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

Notebooks WG and Kubeflow 1.5 release #2109

Closed
kimwnasptd opened this issue Jan 24, 2022 · 8 comments
Closed

Notebooks WG and Kubeflow 1.5 release #2109

kimwnasptd opened this issue Jan 24, 2022 · 8 comments

Comments

@kimwnasptd
Copy link
Member

Will use this issue to coordinate alongside the rest @kubeflow/wg-notebooks-leads around the KF 1.5 release. I'm using a slightly different format than #2106 and #2105 since I'm already a member of both the release team and the Notebook leads.

For the necessary information around the release:

@kimwnasptd
Copy link
Member Author

Let's also try out our new
cc @kubeflow/release-team

@kimwnasptd
Copy link
Member Author

For Notebooks we've merged our two big PRs we were targeting for the feature freeze

I'm preparing the next RC1 right now. We'll keep our eyes open for any problems, this last week of manifests testing

@kimwnasptd
Copy link
Member Author

I've identified two more issues for the Notebooks Controller, one of which is quite important:

I have a PR for the second one and another one for the first tomorrow. Hopefully both fixes will be included in the next RC
cc @kubeflow/release-team

@juliusvonkohout
Copy link
Member

kubeflow/kubeflow#6382 seems to be a severe regression.

@imiller445
Copy link

Hi @kimwnasptd , am working through testing the RCs. I apologize if this is just my incomplete knowledge, but I am wondering if there are any notes on why sidecar.istio.io/inject was set to true for the central dashboard deployment in #2123 and thus injecting a sidecar into the central dashboard pod. When I have this on, requests don't get properly routed to central dashboard for me. I'm wondering if there is a missing istio resource in the manifests? When I set that back to false, everything works as before.

@juliusvonkohout
Copy link
Member

juliusvonkohout commented Mar 3, 2022

Hi @kimwnasptd , am working through testing the RCs. I apologize if this is just my incomplete knowledge, but I am wondering if there are any notes on why sidecar.istio.io/inject was set to true for the central dashboard deployment in #2123 and thus injecting a sidecar into the central dashboard pod. When I have this on, requests don't get properly routed to central dashboard for me. I'm wondering if there is a missing istio resource in the manifests? When I set that back to false, everything works as before.

Are you using a clean install of 1.5rc1? because it does work fine on our cluster. We need it because otherwise the user management is completely broken from a security perspective. There is a long discussions in the issues and pull requests about that.

@juliusvonkohout
Copy link
Member

/close

@google-oss-prow
Copy link

@juliusvonkohout: Closing this issue.

In response to this:

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants