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

consider making pipeline runtime image image_name reference sha256 digest format #259

Closed
shalberd opened this issue Oct 16, 2023 · 3 comments
Assignees
Labels
feature/elyra kind/feature New feature or request priority/normal An issue with the product; fix when possible

Comments

@shalberd
Copy link
Contributor

shalberd commented Oct 16, 2023

related to

opendatahub-io-contrib/workbench-images#44

docker image tag can be put in tags array of json.
the image_name should be in digest format

for all runtime image jsons at notebooks/jupyter/datascience/

so as to be compatible with restricted network environments and ImageContentSourcePolicy. Important for restricted network environments / airgapped. Yes, users can add additional runtimes via GUI per user notebook / PVC, but the default runtimes as delived via ODH in the docker jupyter datascience images should be in the correct format as well.

image_name is used for pulling images, tags are shown in Elyra GUI, just like display_name.

i.e.

{
    "display_name": "CUDA Datascience with Python 3.11 (CentOS Stream 9)",
    "metadata": {
        "tags": ["cuda-runtime-datascience-c9s-py39_RELEASE_latest"],
        "display_name": "CUDA Datascience with Python 3.11 (CentOS Stream 9)",
        "image_name": "quay.io/opendatahub-contrib/workbench-images@sha256:bd565c0e8b4e71d7ae3fd556a2f874e6251d80db1c4c36bfe1d490f04df2bdb1"
    },
    "schema_name": "runtime-image"
}
@github-project-automation github-project-automation bot moved this to 📋 Backlog in ODH IDE Planning Oct 16, 2023
@shalberd shalberd changed the title consider making pipeline runtime image reference sha256 digest format consider making pipeline runtime image image_name reference sha256 digest format Oct 16, 2023
@harshad16 harshad16 added the kind/feature New feature or request label Oct 17, 2023
@harshad16
Copy link
Member

Acceptance criteria

  • Change the from tag to sha256 format. ex: link

@harshad16 harshad16 moved this from 📋 Backlog to 🔖Todo in ODH IDE Planning Oct 17, 2023
@harshad16 harshad16 added the priority/normal An issue with the product; fix when possible label Oct 17, 2023
@shalberd
Copy link
Contributor Author

shalberd commented Oct 17, 2023

@harshad16 yes, and for readability in Elyra GUI, you can put the tag that was originally part of the docker image image_name in the tags array instead, as an element of that. https://github.com/opendatahub-io/notebooks/blob/main/jupyter/datascience/ubi9-python-3.9/runtime-images/datascience-ubi9-py39.json#L4

Bildschirmfoto 2023-10-17 um 19 01 30

I wonder if image pull policy is also a part of that json. If yes, since we would now be at unique hashes, we could use IfNotPresent, saving a lot of time during pipeline task container startup potentially. i.e. add pull_policy to json in metadata

https://github.com/elyra-ai/elyra/blob/main/docs/source/user_guide/runtime-image-conf.md?plain=1#L213

"pull_policy": "IfNotPresent"

full example:

{
    "display_name": "CUDA Datascience with Python 3.11 (CentOS Stream 9)",
    "metadata": {
        "tags": ["cuda-runtime-datascience-c9s-py39_2023c_latest"],
        "display_name": "CUDA Datascience with Python 3.11 (CentOS Stream 9)",
        "image_name": "quay.io/opendatahub-contrib/workbench-images@sha256:bd565c0e8b4e71d7ae3fd556a2f874e6251d80db1c4c36bfe1d490f04df2bdb1",
        "pull_policy": "IfNotPresent"
    },
    "schema_name": "runtime-image"
}

@harshad16
Copy link
Member

This feature is completed with the PR: #286
Marking this as complete, please check the next opendatahub release to see this in effect.
Closing the issue, thanks all for the work 💯

@github-project-automation github-project-automation bot moved this from 🔖Todo to ✅Done in ODH IDE Planning Nov 20, 2023
jstourac pushed a commit to jstourac/notebooks that referenced this issue Jun 11, 2024
…wnload location (opendatahub-io#259)

* RHOAIENG-8255: Fix spawn-fcgi-1.6.3-23.fc37.x86_64.rpm download location

Previously used location is now unavailable and returns 404 error.

This new location comes from https://koji.fedoraproject.org/koji/buildinfo?buildID=2028619

(cherry picked from commit e9c3f9a)

* fixup from review, use a little bit more proper rpm link

(cherry picked from commit ab0f785)

* also fix the rhel9 downstream-only build

---------

Co-authored-by: Jiri Daněk <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/elyra kind/feature New feature or request priority/normal An issue with the product; fix when possible
Projects
Status: Done
Status: No status
Archived in project
Development

No branches or pull requests

3 participants