-
Notifications
You must be signed in to change notification settings - Fork 53
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/Unify naming of the variables used for the images #524
Conversation
This should also fix the commit ID update in the commit.env file.
@jstourac: The following tests failed, say
Full PR test history. Your PR dashboard. 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. I understand the commands that are listed here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/approve
thanks 💯
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: harshad16 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 |
Inconsistency for codeserver notebook parameters. There was upstream change recently that probably not got properly backported to downstream, see [1,2]. * [1] opendatahub-io#524 * [2] red-hat-data-services@ceb3dc8
Inconsistency for codeserver notebook parameters. There was upstream change recently that probably not got properly backported to downstream, see [1,2]. * [1] opendatahub-io#524 * [2] red-hat-data-services@ceb3dc8
These changes shouldn't have any functional impact. [fix] CI for the images checks based on recent updates [fix] This fixes an inconsistency with the kustomize params Inconsistency for codeserver notebook parameters. There was upstream change recently that probably not got properly backported to downstream, see [1,2]. * [1] opendatahub-io#524 * [2] red-hat-data-services@ceb3dc8 Set the rstudio builds with the branch rhoai-2.10 Signed-off-by: Harshad Reddy Nalla <hnalla@redhat.com> Update image commits for release N via digest-updater-9215094498 GitHub action Update images for release N via digest-updater-9215094498 GitHub action Update file via digest-updater-9213110410 GitHub action Allow runtime script to cp the package from bin to Rpackage default path Update codeflare-sdk version on imagestreams annotations (opendatahub-io#235) * Update codeflare-sdk version on imagestreams annotations * fix kfp version in the annotation for tensorflow Co-authored-by: Jan Stourac <jstourac@redhat.com> Update images for release N and N-1 with 2024a commit db8bd76 Update file via digest-updater-8806399693 GitHub action Update annotations for kfp (opendatahub-io#229) Update image commits for release N via digest-updater-8665769109 GitHub action Update images for release N via digest-updater-8665769109 GitHub action Update manifest for code-freeze 2.9 Signed-off-by: Harshad Reddy Nalla <hnalla@redhat.com> Update image commits for release N-1 via digest-updater-8581586298 GitHub action Update images for release N-1 via digest-updater-8581586298 GitHub action Update image commits for release N via digest-updater-8581586298 GitHub action Update images for release N via digest-updater-8581586298 GitHub action Update file via digest-updater-8577545330 GitHub action Fix the runtime updater github action branch 2024a Signed-off-by: Harshad Reddy Nalla <hnalla@redhat.com> Fix the runtime updater github action Signed-off-by: Harshad Reddy Nalla <hnalla@redhat.com> Remove the intel based image from the overlay as its ODH only - Fix the typo in the datascience notebook Signed-off-by: Harshad Reddy Nalla <hnalla@redhat.com> Revert nginx version to 1.22 since 1.24 is not available on rhel yet update cuda layer for RHEL to 12.1 Add runtimes workflow updater Update digest updater workflow Fix check-params-env test with the new changes (opendatahub-io#196) Update Imagesteam for habana 1.13 Update runtime images with e1aee40 build commit Update the manifests to retain old image in shadow state Signed-off-by: Harshad Reddy Nalla <hnalla@redhat.com> Update image commits for release N via digest-updater-8319475892 GitHub action Update images for release N via digest-updater-8319475892 GitHub action Update Codeserver ImageStream for the 2024a release inclusion (opendatahub-io#173) * Update Codeserver imagestream with the 2024a release Co-authored-by: Harshad Reddy Nalla <harshadreddy16@gmail.com> Fix test file for the trustyai image I don't really understand how and why this file was broken by this commit aac0662 . Our CI check notifies that something is broken in the file. Update Imagestreams with in favor of the new release 2024.1 (opendatahub-io#175) Co-authored-by: Harshad Reddy Nalla <hnalla@redhat.com> Update digest updater workflow in favor 2024a release Remove opendatahub.io/dashboard: 'true' label from rstudio ImageSteams Create sync workflow for the release-2024a Format yaml and json files to statisfy code-quality - Fix validation of the params-env Signed-off-by: Harshad Reddy Nalla <hnalla@redhat.com> Update RStudio-server Dockefile for RHEL version Fix library path version on rsession.conf file hot fix: bump cuda resources HotFix: Remove the annotation notebook-images=true from RStudio imagestreams Signed-off-by: Harshad Reddy Nalla <hnalla@redhat.com> Fix user R library path version Update image commits for release N via digest-updater-7846262944 GitHub action Update images for release N via digest-updater-7846262944 GitHub action Remove the R-package install from workbench Co-authored-by: Diamond Bryant <dibryant@redhat.com> Signed-off-by: Harshad Reddy Nalla <hnalla@redhat.com> Fix naming for RStudio Server on rhel flavor Increase build resources for R Studio buildconfigs Mount the secret on the buildConfig instead of using ENVs to avoid their expose on the logs Adjust the imagestream annotation for codeflare-sdk upgrade Signed-off-by: Harshad Reddy Nalla <hnalla@redhat.com> Update image commits for release N via digest-updater-7761501425 GitHub action Update images for release N via digest-updater-7761501425 GitHub action Add optional: true option for the base and server url envs Add BuildConfiguration objects to build RStudio and CUDA RStudio images on OCP cluster Fixes on the CUDA Dockerfile setup r-studio based with rhel9 base image (opendatahub-io#125) * Content of R Studio switched to the rhel based image. Add rhel9 base image [Fix] typo in logging of the `notebook-digest-updater.yaml` Update image commits for release N via digest-updater-7533330854 GitHub action Update images for release N-1 via digest-updater-7533330854 GitHub action Update images for release N via digest-updater-7533330854 GitHub action Fix: update the code-server and annotation Signed-off-by: Harshad Reddy Nalla <hnalla@redhat.com> Co-authored-by: aTheo <atheodorak@outlook.com> Incorporate VSCode on Downstream (opendatahub-io#105) Co-authored-by: Harshad Reddy Nalla <hnalla@redhat.com> hot-fix: Fix the tensorflow imagestream by removing the trailing space Signed-off-by: Harshad Reddy Nalla <hnalla@redhat.com> hot-fix: Fix the imagestream minimal-cuda sha Signed-off-by: Harshad Reddy Nalla <hnalla@redhat.com> hot-fix: Fixed imagestream with CVE 44487 changes Signed-off-by: Harshad Reddy Nalla <hnalla@redhat.com> hot-fix: update the base ubi9 images for cve 44487 fix Signed-off-by: Harshad Reddy Nalla <hnalla@redhat.com> hot-fix: CVE 44487 fix with libnghttp2 Signed-off-by: Harshad Reddy Nalla <hnalla@redhat.com> Update the pipfile.lock via the weekly workflow action chores: Update the runtime image with the commit: 8bda2fa Signed-off-by: Harshad Reddy Nalla <hnalla@redhat.com> Patch the imagestream by removing habana 1.11.0 Signed-off-by: Harshad Reddy Nalla <hnalla@redhat.com> Update the runtime image with the commit: 8bda2fa on main Update images for release N via digest-updater-6655629712 GitHub action Fix the annotation and additional recommended-true Signed-off-by: Harshad Reddy Nalla <hnalla@redhat.com> Patch the imagestream to have same name as in odh-manifests Signed-off-by: Harshad Reddy Nalla <hnalla@redhat.com> Fix digest updater from failing if there are no updates on the image streams Fix the path to the params.env file Several fixes Upgrade the notebook images with 2023b and 2023a images Signed-off-by: Harshad Reddy Nalla <hnalla@redhat.com> Include only sync github workflow on the main branch Signed-off-by: Harshad Reddy Nalla <hnalla@redhat.com>
Inconsistency for codeserver notebook parameters. There was upstream change recently that probably not got properly backported to downstream, see [1,2]. * [1] opendatahub-io#524 * [2] red-hat-data-services@ceb3dc8 (cherry picked from commit 8bb8def)
Inconsistency for codeserver notebook parameters. There was upstream change recently that probably not got properly backported to downstream, see [1,2]. * [1] opendatahub-io#524 * [2] red-hat-data-services@ceb3dc8 (cherry picked from commit 8bb8def)
This should also fix the commit ID update in the commit.env file.
Description
https://issues.redhat.com/browse/RHOAIENG-7884
How Has This Been Tested?
Not yet very thoroughly. Will have to check what is the best approach, today is too late for me to think more 🙂
Note: the CI failure is because of this #523
Merge criteria: