Skip to content
This repository was archived by the owner on Apr 13, 2026. It is now read-only.

docs: podinfo image reference#794

Merged
morri-son merged 3 commits into
open-component-model:mainfrom
fabianburth:fix/image-reference
Apr 10, 2026
Merged

docs: podinfo image reference#794
morri-son merged 3 commits into
open-component-model:mainfrom
fabianburth:fix/image-reference

Conversation

@fabianburth

Copy link
Copy Markdown
Contributor

What this PR does / why we need it

The image reference in the constructor is not aligned with the one we show in the resulting descriptor

Which issue(s) this PR is related to

Type of content

  • Tutorial (getting-started/ or tutorials/)
  • How-to Guide (how-to/)
  • Explanation / Concept (concepts/)
  • Reference (reference/)
  • Other (infrastructure, config, fixes)

Checklist

  • I have read and followed the Contributing Guide
  • All commands/code snippets are tested and can be copy-pasted

…lready updated

Signed-off-by: Fabian Burth <fabian.burth@sap.com>
@fabianburth fabianburth requested a review from a team as a code owner April 1, 2026 18:10
@netlify

netlify Bot commented Apr 1, 2026

Copy link
Copy Markdown

Deploy Preview for open-component-model ready!

Name Link
🔨 Latest commit eea85fa
🔍 Latest deploy log https://app.netlify.com/projects/open-component-model/deploys/69d8b293aa4a900008ee5e51
😎 Deploy Preview https://deploy-preview-794--open-component-model.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai

coderabbitai Bot commented Apr 1, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: be9f5b9b-3e74-4b44-b628-f66f8d0a9aa4

📥 Commits

Reviewing files that changed from the base of the PR and between 1b1c530 and 71c2d0a.

📒 Files selected for processing (3)
  • content/docs/concepts/signing-and-verification-concept.md
  • content/docs/how-to/model-products.md
  • content/docs/tutorials/advanced-component-constructor.md
✅ Files skipped from review due to trivial changes (3)
  • content/docs/how-to/model-products.md
  • content/docs/concepts/signing-and-verification-concept.md
  • content/docs/tutorials/advanced-component-constructor.md

📝 Walkthrough

Walkthrough

Updated documentation examples to reference a newer podinfo OCI image version and adjusted one OCI resource digest in examples; changes are limited to documentation examples across multiple guides.

Changes

Cohort / File(s) Summary
Getting Started
content/docs/getting-started/create-component-version.md
Bumped example podinfo imageReference from ghcr.io/stefanprodan/podinfo:6.9.1 to ghcr.io/stefanprodan/podinfo:6.11.1.
Concepts — Signing & Verification
content/docs/concepts/signing-and-verification-concept.md
Updated example OCI resource access.ociArtifact.imageReference digest fragment to a different digest prefix (doc-only change).
How-to / Model Products
content/docs/how-to/model-products.md
Updated two occurrences of podinfo imageReference from :6.9.1 to :6.11.1 in constructor examples.
Tutorials — Advanced Component Constructor
content/docs/tutorials/advanced-component-constructor.md
Replaced podinfo imageReference :6.9.1 with :6.11.1 across constructor YAML and expected CLI output snippets.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

kind/chore

Suggested reviewers

  • morri-son
  • jakobmoellerdev

Poem

🐰 A tiny hop, a tidy tweak,
I changed the tag, then stopped to peek.
Podinfo hops from old to new,
In docs it shines, refreshed and true.
Yum carrots, code, and version stew! 🥕

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The PR title accurately summarizes the main change: updating the podinfo image reference across documentation examples.
Description check ✅ Passed The description clearly explains the PR's purpose: aligning the image reference in the constructor with the one shown in the resulting descriptor.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Signed-off-by: Fabian Burth <fabian.burth@sap.com>
@fabianburth

Copy link
Copy Markdown
Contributor Author

To be clean here, I should also update corresponding digest fields (if present in the example). Potentially should just omit the ocm digest, so we can eventually run renovate on our docs to keep the image references up to date.

@morri-son morri-son enabled auto-merge (squash) April 10, 2026 08:19
@morri-son morri-son merged commit 765b918 into open-component-model:main Apr 10, 2026
7 checks passed
ocmbot Bot pushed a commit that referenced this pull request Apr 10, 2026
<!-- markdownlint-disable MD041 -->
#### What this PR does / why we need it
The image reference in the constructor is not aligned with the one we
show in the resulting descriptor

#### Which issue(s) this PR is related to
<!--
Usage: `Related to #<issue number>`, or `Related to (paste link of
issue)`.
-->

#### Type of content
<!--
Which section does this PR target? See CONTRIBUTING.md for guidance.
-->
- [ ] Tutorial (`getting-started/` or `tutorials/`)
- [ ] How-to Guide (`how-to/`)
- [ ] Explanation / Concept (`concepts/`)
- [ ] Reference (`reference/`)
- [ ] Other (infrastructure, config, fixes)

#### Checklist

- [ ] I have read and followed the [Contributing
Guide](https://github.com/open-component-model/ocm-website/blob/main/CONTRIBUTING.md)
- [ ] All commands/code snippets are tested and can be copy-pasted

---------

Signed-off-by: Fabian Burth <fabian.burth@sap.com>
Co-authored-by: Gergely Bräutigam <gergely.brautigam@sap.com> 765b918
morri-son pushed a commit to jakobmoellerdev/ocm-website that referenced this pull request Apr 10, 2026
<!-- markdownlint-disable MD041 -->
#### What this PR does / why we need it
The image reference in the constructor is not aligned with the one we
show in the resulting descriptor

#### Which issue(s) this PR is related to
<!--
Usage: `Related to #<issue number>`, or `Related to (paste link of
issue)`.
-->

#### Type of content
<!--
Which section does this PR target? See CONTRIBUTING.md for guidance.
-->
- [ ] Tutorial (`getting-started/` or `tutorials/`)
- [ ] How-to Guide (`how-to/`)
- [ ] Explanation / Concept (`concepts/`)
- [ ] Reference (`reference/`)
- [ ] Other (infrastructure, config, fixes)

#### Checklist

- [ ] I have read and followed the [Contributing
Guide](https://github.com/open-component-model/ocm-website/blob/main/CONTRIBUTING.md)
- [ ] All commands/code snippets are tested and can be copy-pasted

---------

Signed-off-by: Fabian Burth <fabian.burth@sap.com>
Co-authored-by: Gergely Bräutigam <gergely.brautigam@sap.com>
Signed-off-by: Gerald Morrison (SAP) <gerald.morrison@sap.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants