Skip to content

Pin jupyterlab package to version 4.2.6 #810

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

Merged
merged 1 commit into from
Dec 21, 2024

Conversation

caponetto
Copy link
Contributor

@caponetto caponetto commented Dec 16, 2024

Related to https://issues.redhat.com/browse/RHOAIENG-11156

Description

Pin jupyterlab package to version 4.2.6 to avoid changes made on the base images being overridden due to JL upgrades. No need to update the Pipfile.lock files since they are already using the target version.

How Has This Been Tested?

Merge criteria:

  • The commits are squashed in a cohesive manner and have meaningful messages.
  • Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
  • The developer has manually tested the changes and verified that the changes work

@openshift-ci openshift-ci bot requested review from dibryant and paulovmr December 16, 2024 16:12
@jiridanek
Copy link
Member

/lgtm

Copy link
Contributor

@paulovmr paulovmr left a comment

Choose a reason for hiding this comment

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

/lgtm

@caponetto
Copy link
Contributor Author

/retest

@atheo89
Copy link
Member

atheo89 commented Dec 17, 2024

Hi Guilherme, I know this change is small and straightforward, but I think we're not ready to commit only the Pipfiles without their corresponding Piplock files. I understand it’s a bit of a tedious process, but I have a solution that won’t require you to strain your PC. You just need to follow the comment I wrote on Diamond's PR link

@jiridanek
Copy link
Member

commit only the Pipfiles without their corresponding Piplock files

Wouldn't the automatic Tuesday piplock renewal github action take care of that?

@caponetto
Copy link
Contributor Author

caponetto commented Dec 17, 2024

commit only the Pipfiles without their corresponding Piplock files

Wouldn't the automatic Tuesday piplock renewal github action take care of that?

That's what I thought. Also, the Pipfile.lock files are already on the correct version (4.2.6) on main, so there is nothing to be changed. Am I missing something @atheo89?

@atheo89
Copy link
Member

atheo89 commented Dec 17, 2024

commit only the Pipfiles without their corresponding Piplock files

Wouldn't the automatic Tuesday piplock renewal github action take care of that?

That's what I thought. Also, the Pipfile.lock files are already on the correct version (4.2.6) on main, so there is nothing to be changed. Am I missing something @atheo89?

Right, sorry I didn't notice that the pipfile.locks point already to 4.2.6!

About the piplock renewal i just saw that it failed today while the cron firing, need to check. Seems that it can not ingest the py version. need to check

@atheo89
Copy link
Member

atheo89 commented Dec 17, 2024

I think i know what is happening i will open a pr to fix that, I don't have specify what to pick in case of no inputs

@caponetto
Copy link
Contributor Author

/retest

@atheo89
Copy link
Member

atheo89 commented Dec 17, 2024

This is the fix for the cron firing: #811
I changed the cron from Monday to Wednesday, for two reasons one, to be closer to the release day, and secondly to generate the fresh piplocks from your PR.

So to unblock this /lgtm

@caponetto
Copy link
Contributor Author

caponetto commented Dec 17, 2024

/override ci/prow/notebooks-ubi9-e2e-tests
/override ci/prow/rocm-notebooks-e2e-tests
/approve

Copy link
Contributor

openshift-ci bot commented Dec 17, 2024

@caponetto: /override requires failed status contexts, check run or a prowjob name to operate on.
The following unknown contexts/checkruns were given:

  • ci/prow/intel-notebooks-e2e-tests

Only the following failed contexts/checkruns were expected:

  • build (cuda-jupyter-tensorflow-ubi9-python-3.11) / build
  • build (rocm-jupyter-tensorflow-ubi9-python-3.11) / build
  • ci/prow/images
  • ci/prow/notebook-cuda-jupyter-ds-ubi9-python-3-11-pr-image-mirror
  • ci/prow/notebook-cuda-jupyter-min-ubi9-python-3-11-pr-image-mirror
  • ci/prow/notebook-cuda-jupyter-tf-ubi9-python-3-11-pr-image-mirror
  • ci/prow/notebook-jupyter-datascience-ubi9-python-3-11-pr-image-mirror
  • ci/prow/notebook-jupyter-minimal-ubi9-python-3-11-pr-image-mirror
  • ci/prow/notebook-jupyter-pytorch-ubi9-python-3-11-pr-image-mirror
  • ci/prow/notebook-jupyter-trustyai-ubi9-python-3-11-pr-image-mirror
  • ci/prow/notebook-rocm-jupyter-pyt-ubi9-python-3-11-pr-image-mirror
  • ci/prow/notebook-rocm-jupyter-tf-ubi9-python-3-11-pr-image-mirror
  • ci/prow/notebooks-ubi9-e2e-tests
  • ci/prow/rocm-notebooks-e2e-tests
  • pull-ci-opendatahub-io-notebooks-2023a-images
  • pull-ci-opendatahub-io-notebooks-main-notebook-cuda-jupyter-ds-ubi9-python-3-11-pr-image-mirror
  • pull-ci-opendatahub-io-notebooks-main-notebook-cuda-jupyter-min-ubi9-python-3-11-pr-image-mirror
  • pull-ci-opendatahub-io-notebooks-main-notebook-cuda-jupyter-tf-ubi9-python-3-11-pr-image-mirror
  • pull-ci-opendatahub-io-notebooks-main-notebook-jupyter-datascience-ubi9-python-3-11-pr-image-mirror
  • pull-ci-opendatahub-io-notebooks-main-notebook-jupyter-minimal-ubi9-python-3-11-pr-image-mirror
  • pull-ci-opendatahub-io-notebooks-main-notebook-jupyter-pytorch-ubi9-python-3-11-pr-image-mirror
  • pull-ci-opendatahub-io-notebooks-main-notebook-jupyter-trustyai-ubi9-python-3-11-pr-image-mirror
  • pull-ci-opendatahub-io-notebooks-main-notebook-rocm-jupyter-pyt-ubi9-python-3-11-pr-image-mirror
  • pull-ci-opendatahub-io-notebooks-main-notebook-rocm-jupyter-tf-ubi9-python-3-11-pr-image-mirror
  • pull-ci-opendatahub-io-notebooks-main-notebooks-ubi9-e2e-tests
  • pull-ci-opendatahub-io-notebooks-main-rocm-notebooks-e2e-tests
  • tide

If you are trying to override a checkrun that has a space in it, you must put a double quote on the context.

In response to this:

/override ci/prow/notebooks-ubi9-e2e-tests
/override ci/prow/intel-notebooks-e2e-tests
/approve

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-sigs/prow repository.

Copy link
Contributor

openshift-ci bot commented Dec 17, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: caponetto

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

@caponetto
Copy link
Contributor Author

/override ci/prow/rocm-notebooks-e2e-tests

Copy link
Contributor

openshift-ci bot commented Dec 17, 2024

@caponetto: Overrode contexts on behalf of caponetto: ci/prow/rocm-notebooks-e2e-tests

In response to this:

/override ci/prow/rocm-notebooks-e2e-tests

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-sigs/prow repository.

@caponetto
Copy link
Contributor Author

/override ci/prow/notebooks-ubi9-e2e-tests

Copy link
Contributor

openshift-ci bot commented Dec 17, 2024

@caponetto: Overrode contexts on behalf of caponetto: ci/prow/notebooks-ubi9-e2e-tests

In response to this:

/override ci/prow/notebooks-ubi9-e2e-tests

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-sigs/prow repository.

@caponetto
Copy link
Contributor Author

/override ci/prow/images

Copy link
Contributor

openshift-ci bot commented Dec 17, 2024

@caponetto: Overrode contexts on behalf of caponetto: ci/prow/images

In response to this:

/override ci/prow/images

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-sigs/prow repository.

@caponetto
Copy link
Contributor Author

/override "build (cuda-jupyter-tensorflow-ubi9-python-3.11) / build"
/override "build (rocm-jupyter-tensorflow-ubi9-python-3.11) / build"

Copy link
Contributor

openshift-ci bot commented Dec 17, 2024

@caponetto: Overrode contexts on behalf of caponetto: build (cuda-jupyter-tensorflow-ubi9-python-3.11) / build, build (rocm-jupyter-tensorflow-ubi9-python-3.11) / build

In response to this:

/override "build (cuda-jupyter-tensorflow-ubi9-python-3.11) / build"
/override "build (rocm-jupyter-tensorflow-ubi9-python-3.11) / build"

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-sigs/prow repository.

@caponetto
Copy link
Contributor Author

/retest

1 similar comment
@caponetto
Copy link
Contributor Author

/retest

@caponetto
Copy link
Contributor Author

/retest-required

@shalberd
Copy link
Contributor

@caponetto related info FYI: Jupyterlab 4 support is coming for upstream Elyra as well.
I think @paulovmr reviewed it and it is merged into a fork branch of Luciano's.
lresende/elyra#2
This will make its way to https://github.com/elyra-ai/elyra soon ...

@jiridanek
Copy link
Member

@caponetto is a co-author on the PR, so I wouldn't worry he's not aware ;p

image

@caponetto
Copy link
Contributor Author

/override ci/prow/notebooks-ubi9-e2e-tests
/override ci/prow/rocm-notebooks-e2e-tests

Copy link
Contributor

openshift-ci bot commented Dec 18, 2024

@caponetto: Overrode contexts on behalf of caponetto: ci/prow/notebooks-ubi9-e2e-tests, ci/prow/rocm-notebooks-e2e-tests

In response to this:

/override ci/prow/notebooks-ubi9-e2e-tests
/override ci/prow/rocm-notebooks-e2e-tests

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-sigs/prow repository.

@caponetto
Copy link
Contributor Author

/retest

@jiridanek
Copy link
Member

/retest-required

1 similar comment
@caponetto
Copy link
Contributor Author

/retest-required

@caponetto
Copy link
Contributor Author

/override ci/prow/images

Copy link
Contributor

openshift-ci bot commented Dec 19, 2024

@caponetto: Overrode contexts on behalf of caponetto: ci/prow/images

In response to this:

/override ci/prow/images

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-sigs/prow repository.

@caponetto
Copy link
Contributor Author

/retest-required

@jiridanek
Copy link
Member

/retest
Prow might be in good form today, let's see

@jiridanek
Copy link
Member

no joy, we'll have to override, me thinks

/retest-required

@caponetto
Copy link
Contributor Author

/override ci/prow/codeserver-notebook-e2e-tests
/override ci/prow/intel-notebooks-e2e-tests
/override ci/prow/notebooks-ubi9-e2e-tests
/override ci/prow/rocm-notebooks-e2e-tests
/override ci/prow/rocm-runtimes-ubi9-e2e-tests
/override ci/prow/rstudio-notebook-e2e-tests
/override ci/prow/runtimes-ubi9-e2e-tests

Copy link
Contributor

openshift-ci bot commented Dec 20, 2024

@caponetto: Overrode contexts on behalf of caponetto: ci/prow/codeserver-notebook-e2e-tests, ci/prow/intel-notebooks-e2e-tests, ci/prow/notebooks-ubi9-e2e-tests, ci/prow/rocm-notebooks-e2e-tests, ci/prow/rocm-runtimes-ubi9-e2e-tests, ci/prow/rstudio-notebook-e2e-tests, ci/prow/runtimes-ubi9-e2e-tests

In response to this:

/override ci/prow/codeserver-notebook-e2e-tests
/override ci/prow/intel-notebooks-e2e-tests
/override ci/prow/notebooks-ubi9-e2e-tests
/override ci/prow/rocm-notebooks-e2e-tests
/override ci/prow/rocm-runtimes-ubi9-e2e-tests
/override ci/prow/rstudio-notebook-e2e-tests
/override ci/prow/runtimes-ubi9-e2e-tests

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-sigs/prow repository.

@caponetto
Copy link
Contributor Author

/retest-required

2 similar comments
@caponetto
Copy link
Contributor Author

/retest-required

@caponetto
Copy link
Contributor Author

/retest-required

@openshift-merge-bot openshift-merge-bot bot merged commit 532f554 into opendatahub-io:main Dec 21, 2024
27 of 29 checks passed
@caponetto
Copy link
Contributor Author

🥳

jesuino pushed a commit to jesuino/notebooks that referenced this pull request Jun 17, 2025
…lux/component-updates/component-update-odh-workbench-jupyter-minimal-cuda-py311-ubi9-n-v2-22

Update odh-workbench-jupyter-minimal-cuda-py311-ubi9-n-v2-22 to fb10bc3
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.

5 participants