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

fix: Add scipy to the UI jupyter-images list #291

Merged
merged 1 commit into from
Sep 4, 2023

Conversation

phoevos
Copy link
Contributor

@phoevos phoevos commented Aug 29, 2023

Add kubeflownotebookswg/jupyter-scipy:v1.7.0 to the list of jupyter-images exposed as a configurable option via the jupyter-ui charm. Adding it to the top of the list restores it as the default option.

Closes #290

@i-chvets
Copy link
Contributor

LGTM, but CI is acting up.

@phoevos
Copy link
Contributor Author

phoevos commented Aug 31, 2023

Looks like Prometheus and Grafana were having issues (looks unrelated to the changes introduced here). Retriggering the failed run.

@phoevos
Copy link
Contributor Author

phoevos commented Aug 31, 2023

So, I reran the tests which didn't help. I saw this error which seemed curious.

juju.errors.JujuError: ['Charm feature requirements cannot be met:\n  - charm requires feature "juju" (version >= 3.0.3) but model currently supports version 2.9.44\n\nFeature descriptions:\n  - "juju": the version of Juju used by the model\n\nFor additional information please see: https://juju.is/docs/olm/supported-features']

It turns out, the error is caused by Prometheus, and more specifically this change introduced just yesterday, which assumes juju >= 3.0.3, which I'd say is unfortunate. It means we can't deploy the latest/edge version of Prometheus.

Add `kubeflownotebookswg/jupyter-scipy:v1.7.0` to the list of
jupyter-images exposed as a configurable option via the jupyter-ui
charm. Adding it to the top of the list restores it as the default
option.

Signed-off-by: Phoevos Kalemkeris <[email protected]>
Copy link
Contributor

@NohaIhab NohaIhab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @phoevos

@phoevos phoevos merged commit eaca4e7 into main Sep 4, 2023
14 checks passed
@phoevos phoevos deleted the fix-phoevos-ui-config-scipy branch September 4, 2023 07:16
NohaIhab pushed a commit that referenced this pull request Sep 26, 2023
Add `kubeflownotebookswg/jupyter-scipy:v1.7.0` to the list of
jupyter-images exposed as a configurable option via the jupyter-ui
charm. Adding it to the top of the list restores it as the default
option.

Signed-off-by: Phoevos Kalemkeris <[email protected]>
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.

Image kubeflownotebookswg/jupyter-scipy missing from config
3 participants