Skip to content

fix: no longer require a repository context in component descriptors - #305

Open
piotrjanik wants to merge 3 commits into
openmcp-project:mainfrom
piotrjanik:fix/1235-repository-context
Open

fix: no longer require a repository context in component descriptors#305
piotrjanik wants to merge 3 commits into
openmcp-project:mainfrom
piotrjanik:fix/1235-repository-context

Conversation

@piotrjanik

@piotrjanik piotrjanik commented Aug 21, 2026

Copy link
Copy Markdown

What this PR does / why we need it:

The legacy OCM spec mandated at least one repository context, but OCM v2 component descriptors no longer carry one. Drop the checks so component versions from OCM v2 repositories can be resolved.

Which issue(s) this PR fixes:
Fixes open-component-model/ocm-project#1235

Special notes for your reviewer:

Release note:

Drop mandatory check for repository context to support OCM v2 component descriptors

The legacy OCM spec mandated at least one repository context, but OCM v2
component descriptors no longer carry one. Drop the checks so component
versions from OCM v2 repositories can be resolved.

Signed-off-by: Piotr Janik <piotr.janik@sap.com>
@piotrjanik
piotrjanik force-pushed the fix/1235-repository-context branch from f700006 to 2f09fc8 Compare September 1, 2026 13:23
@piotrjanik piotrjanik changed the title DRAFT: LocalBlob Experiment fix: no longer require a repository context in component descriptors Sep 1, 2026
@piotrjanik
piotrjanik marked this pull request as ready for review September 1, 2026 13:25

@Skarlso Skarlso left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This looks good from our side. :) 🎉

Diaphteiros
Diaphteiros previously approved these changes Sep 3, 2026

@Diaphteiros Diaphteiros left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

@Diaphteiros

Copy link
Copy Markdown
Contributor

The failed check is because the PR description is missing a Release note section (see https://github.com/openmcp-project/build/blob/main/.github/pull_request_template.md)

The legacy OCM spec mandated at least one repository context, but OCM v2
component descriptors no longer carry one. Drop the checks so component
versions from OCM v2 repositories can be resolved.

Signed-off-by: Piotr Janik <piotr.janik@sap.com>
@Diaphteiros

Copy link
Copy Markdown
Contributor

@piotrjanik Seems there is an unused variable now.

Signed-off-by: Piotr Janik <piotr.janik@sap.com>
@piotrjanik

Copy link
Copy Markdown
Author

@Diaphteiros, how about now?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Analyze landscaper with OCM v2

4 participants