Skip to content

fix: oci upload behavior#1806

Merged
matthiasbruns merged 13 commits into
open-component-model:mainfrom
fabianburth:oci-upload
Feb 20, 2026
Merged

fix: oci upload behavior#1806
matthiasbruns merged 13 commits into
open-component-model:mainfrom
fabianburth:oci-upload

Conversation

@fabianburth

@fabianburth fabianburth commented Feb 19, 2026

Copy link
Copy Markdown
Contributor

What this PR does / why we need it

The behaviour for
localblobs:

  • oci-upload set AND target ocm repo is oci AND localblob is oci artifact (media type) --> upload as oci
  • otherwise -> upload as localblob

ociArtifacts:

  • oci-upload set AND target ocm repo is oci --> upload as oci
  • otherwise -> upload as localblob

Which issue(s) this PR fixes

Contributes: open-component-model/ocm-project#796

…r oci

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>
Signed-off-by: Fabian Burth <fabian.burth@sap.com>
# Conflicts:
#	cli/cmd/transfer/component-version/internal/oci.go
Signed-off-by: Fabian Burth <fabian.burth@sap.com>
Signed-off-by: Fabian Burth <fabian.burth@sap.com>
@github-actions github-actions Bot added the size/l Large label Feb 20, 2026
@fabianburth fabianburth marked this pull request as ready for review February 20, 2026 14:02
@fabianburth fabianburth requested a review from a team as a code owner February 20, 2026 14:02
@matthiasbruns

Copy link
Copy Markdown
Contributor

I feel that we start getting into an if-else condition hell and it's worth exploring open-component-model/ocm-project#903 in the next sprints.

@piotrjanik

Copy link
Copy Markdown
Contributor

i just tested it locally, lgtm

@matthiasbruns matthiasbruns merged commit e578a97 into open-component-model:main Feb 20, 2026
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants