feat: add / get local blob transformers#1654
Merged
jakobmoellerdev merged 9 commits intoJan 27, 2026
Merged
Conversation
Signed-off-by: Jakob Möller <contact@jakob-moeller.com>
02f46f2 to
f436e49
Compare
Signed-off-by: Jakob Möller <jakob.moeller@sap.com>
makes sure that the generators we run dont clash by being sequential. a bit slower, but shouldnt hurt too much Signed-off-by: Jakob Möller <jakob.moeller@sap.com>
Signed-off-by: Jakob Möller <jakob.moeller@sap.com>
### What was done - Updated the error message during artifact access conversion to provide clearer context about the failure with local blob access. - Adjusted a related test case to match the refined error message. ### Why Improves error clarity for developers by providing actionable insights into invalid conversions. Signed-off-by: Jakob Möller <jakob.moeller@sap.com>
piotrjanik
reviewed
Jan 26, 2026
fabianburth
reviewed
Jan 27, 2026
…dependency ### What was done - Replaced `v0.0.11-0.20260123164636-4d10ccc182e4` with the stable `v0.0.11` version of `ocm.software/open-component-model/bindings/go/blob`. - Updated dependency references in `bindings/go/oci/integration` and `bindings/go/oci`. ### Why Ensures stability and consistency by locking the dependency to a stable release version. Signed-off-by: Jakob Möller <jakob.moeller@sap.com>
…tions ### What was done - Updated `ArtifactBlob` function signature to improve type clarity. - Refined error message for missing file URI to give more descriptive guidance. - Enforced `ID` field as required in multiple transformation specs (`CTFAddLocalResource`, `CTFGetLocalResource`, `OCIAddLocalResource`, `OCIGetLocalResource`). - Removed unused methods in `mockRepository` for cleaner and more maintainable tests. ### Why - Enhances developer experience with clearer error messages and stricter validation. - Simplifies codebase by eliminating unused mocks, ensuring the focus is on relevant functionality. - Enforces consistency across transformation specifications, aligning with intended schema design. Co-authored-by: Fabian Burth <fabian.burth@sap.com> Signed-off-by: Jakob Möller <jakob.moeller@sap.com>
fabianburth
approved these changes
Jan 27, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What this PR does / why we need it
creates AddLocalBlob / GetLocalBlob transformation variants for OCI / CTF
Which issue(s) this PR fixes
creates the transformers in OCI required for local blob transformations.
add / get for now based on file buffers. copy op coming up separately
NOTE: blocked until blob module is released0.0.11 releasedpart of open-component-model/ocm-project#795