Skip to content

Speeding Up Pulling Container Images/CRI-O Additional Storage Support#110809

Open
mburke5678 wants to merge 9 commits intoopenshift:mainfrom
mburke5678:nodes-crio-addt-storage
Open

Speeding Up Pulling Container Images/CRI-O Additional Storage Support#110809
mburke5678 wants to merge 9 commits intoopenshift:mainfrom
mburke5678:nodes-crio-addt-storage

Conversation

@mburke5678
Copy link
Copy Markdown
Contributor

@mburke5678 mburke5678 commented Apr 24, 2026

OSDOCS-19406
OSDOCS-19402

Link to docs preview:

QE review:

  • QE has approved this change.

@mburke5678 mburke5678 added this to the Planned for 4.22 GA milestone Apr 24, 2026
@openshift-ci openshift-ci Bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Apr 24, 2026
Comment thread modules/nodes-nodes-additional-crio-storage-about.adoc Outdated
Comment thread modules/nodes-nodes-additional-crio-storage-configuring.adoc Outdated
Comment thread modules/nodes-nodes-additional-crio-storage-configuring.adoc Outdated
Comment thread modules/nodes-nodes-additional-crio-storage-configuring.adoc
Comment thread modules/nodes-nodes-additional-crio-storage-configuring.adoc
Comment thread modules/nodes-nodes-additional-crio-storage-configuring.adoc
@mburke5678
Copy link
Copy Markdown
Contributor Author

/retest

Copy link
Copy Markdown
Member

@saschagrunert saschagrunert left a comment

Choose a reason for hiding this comment

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

Reviewed against the openshift/api types, MCO implementation (helpers.go), CRI-O PR cri-o/cri-o#9702, the Red Hat developer blog post, and the Jira epics (OCPSTRAT-2623, OCPSTRAT-1285, OCPNODE-4051). Main issues: copy-paste error in the definition list terms, wrong assembly path/context, missing CRI-O artifacts/ subdirectory convention, and the :ref suffix description should be scoped to layer stores only (not image stores). The :ref auto-append by MCO is the agreed approach but depends on the openshift/machine-config-operator#5858 rework landing.

Comment thread nodes/nodes/nodes-nodes-additional-crio-storage.adoc
Comment thread modules/nodes-nodes-additional-crio-storage-about.adoc Outdated
Comment thread modules/nodes-nodes-additional-crio-storage-about.adoc Outdated

For example, large OCI artifacts, such as machine learning models, are stored on the default location, consuming space and preventing the use of faster dedicated storage. By configuring the `additionalArtifactStores` parameter, you can store large AI/ML models on high-performance storage (SSD) separate from the root file system space storage. As a result, your workloads can experience faster start times and your clusters can use storage more efficiently.

Also, you could use the `additionalImageStores` parameter to mount an NFS share with pre-populated images across all worker nodes. Nodes read from the shared cache instead of pulling from an external registry. This is useful in a disconnected environments or when many nodes run the same workloads.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Grammar: "in a disconnected environments"

Suggested change
Also, you could use the `additionalImageStores` parameter to mount an NFS share with pre-populated images across all worker nodes. Nodes read from the shared cache instead of pulling from an external registry. This is useful in a disconnected environments or when many nodes run the same workloads.
Also, you could use the `additionalImageStores` parameter to mount an NFS share with pre-populated images across all worker nodes. Nodes read from the shared cache instead of pulling from an external registry. This is useful in disconnected environments or when many nodes run the same workloads.

Comment thread modules/nodes-nodes-additional-crio-storage-about.adoc Outdated
Comment thread modules/nodes-nodes-additional-crio-storage-configuring.adoc Outdated
Comment thread modules/nodes-nodes-additional-crio-storage-configuring.adoc Outdated
sh-5.1# chroot /host
----

.. For an artifact store, look at the contents of the `etc/crio/crio.conf.d/01-ctrcfg-additionalArtifactStores` directory:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is a file, not a directory. Also missing leading / in the path.

Suggested change
.. For an artifact store, look at the contents of the `etc/crio/crio.conf.d/01-ctrcfg-additionalArtifactStores` directory:
.. For an artifact store, look at the contents of the `/etc/crio/crio.conf.d/01-ctrcfg-additionalArtifactStores` file:

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ugh. I saw that, meant to circle back and fix it, but forgot apparently!

Comment thread modules/nodes-nodes-additional-crio-storage-configuring.adoc Outdated
Comment thread modules/nodes-nodes-additional-crio-storage-configuring.adoc Outdated
@mburke5678
Copy link
Copy Markdown
Contributor Author

@BhargaviGudi PTAL. Thank you!

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Apr 28, 2026

@mburke5678: all tests passed!

Full PR test history. Your PR dashboard.

Details

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch/enterprise-4.22 size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants