Skip to content

feat: adjust cli to updated constructor (and plugins and credentials)#889

Merged
jakobmoellerdev merged 10 commits into
open-component-model:mainfrom
fabianburth:feat/add-cv
Sep 17, 2025
Merged

feat: adjust cli to updated constructor (and plugins and credentials)#889
jakobmoellerdev merged 10 commits into
open-component-model:mainfrom
fabianburth:feat/add-cv

Conversation

@fabianburth

Copy link
Copy Markdown
Contributor

What this PR does / why we need it

Follow up to #772.

  • Pass an external component repository to the construction. This allows to resolve components that are not part of the constructor. This is required to calculate the digest of reference.
  • Minor adjustments due to breaking API changes to dependent features in the plugins and credential module

Which issue(s) this PR fixes

Contributes to open-component-model/ocm-project#564

@fabianburth fabianburth requested a review from a team as a code owner September 16, 2025 08:18
@github-actions github-actions Bot added kind/feature new feature, enhancement, improvement, extension size/m Medium labels Sep 16, 2025
@fabianburth fabianburth force-pushed the feat/add-cv branch 3 times, most recently from e740218 to 4f045c9 Compare September 16, 2025 15:26
Comment thread cli/cmd/add/component-version/cmd.go Outdated
Comment thread cli/cmd/add/component-version/cmd.go Outdated
Comment thread cli/cmd/add/component-version/cmd.go Outdated
Comment thread cli/internal/repository/ocm/resolver.go Outdated
Comment thread cli/internal/repository/ocm/resolver.go Outdated
Comment thread cli/cmd/add/component-version/cmd.go Outdated
…stry provider instead of the oci provider

Signed-off-by: Fabian Burth <fabian.burth@sap.com>
…mConfig function

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

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

Signed-off-by: Fabian Burth <fabian.burth@sap.com>
…ion for internal plugins

Signed-off-by: Fabian Burth <fabian.burth@sap.com>
Signed-off-by: Fabian Burth <fabian.burth@sap.com>
Skarlso
Skarlso previously approved these changes Sep 17, 2025
Signed-off-by: Fabian Burth <fabian.burth@sap.com>
@jakobmoellerdev jakobmoellerdev merged commit 286d482 into open-component-model:main Sep 17, 2025
19 checks passed
@fabianburth fabianburth deleted the feat/add-cv branch September 17, 2025 13:15
matthiasbruns pushed a commit to matthiasbruns/open-component-model that referenced this pull request Sep 18, 2025
…open-component-model#889)

<!-- markdownlint-disable MD041 -->
#### What this PR does / why we need it
Follow up to
open-component-model#772.

- Pass an external component repository to the construction. This allows
to resolve components that are not part of the constructor. This is
required to calculate the digest of reference.
- Minor adjustments due to breaking API changes to dependent features in
the plugins and credential module

#### Which issue(s) this PR fixes
<!--
Usage: `Fixes #<issue number>`, or `Fixes (paste link of issue)`.
-->
Contributes to
open-component-model/ocm-project#564

---------

Signed-off-by: Fabian Burth <fabian.burth@sap.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/feature new feature, enhancement, improvement, extension size/m Medium

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants