Skip to content

feat: add support for OCI repository pattern for add component version #628

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 15 commits into from
Sep 4, 2025
Merged

feat: add support for OCI repository pattern for add component version #628

merged 15 commits into from
Sep 4, 2025
+593 −43

Conversation

Skarlso
Copy link
Contributor

@Skarlso Skarlso commented Aug 26, 2025 •
edited
Loading

What this PR does / why we need it

Fixes open-component-model/ocm-project#578

Which issue(s) this PR fixes

@github-actions github-actions bot added kind/feature new feature, enhancement, improvement, extension size/s Small size/m Medium labels Aug 26, 2025
@Skarlso Skarlso marked this pull request as ready for review August 26, 2025 07:28
@Skarlso Skarlso requested a review from a team as a code owner August 26, 2025 07:28
jakobmoellerdev
jakobmoellerdev reviewed Aug 26, 2025
View reviewed changes
@Skarlso Skarlso marked this pull request as draft August 26, 2025 08:04
@Skarlso Skarlso force-pushed the support-oci-repositories branch from c1fbd25 to 817f238 Compare August 26, 2025 10:16
Skarlso
Skarlso commented Aug 26, 2025
View reviewed changes
@Skarlso Skarlso force-pushed the support-oci-repositories branch from e29110b to c28c8f5 Compare August 26, 2025 13:50
@Skarlso Skarlso mentioned this pull request Aug 27, 2025
Merged
@Skarlso Skarlso force-pushed the support-oci-repositories branch from c28c8f5 to 106aaed Compare August 27, 2025 07:30
@github-actions github-actions bot added the size/l Large label Aug 27, 2025
@Skarlso Skarlso force-pushed the support-oci-repositories branch from 4b1523f to e5b249f Compare August 28, 2025 07:44
@Skarlso
Copy link
Contributor Author

Skarlso commented Aug 28, 2025

Ugh, the tests together are not working of course. I need a way to isolate them.

@Skarlso
Copy link
Contributor Author

Skarlso commented Aug 28, 2025

Yay, the tests are passing. I'm going to overhaul the suite_test a bit.

@Skarlso Skarlso force-pushed the support-oci-repositories branch from 3db798e to 476c932 Compare August 29, 2025 06:03
fabianburth
fabianburth reviewed Aug 29, 2025
View reviewed changes
@Skarlso Skarlso removed size/s Small size/m Medium labels Sep 1, 2025
@Skarlso
feat: add support for OCI repository pattern for add component version
2128be5 
On-behalf-of: Gergely Brautigam <gergely.brautigam@sap.com>

Signed-off-by: Gergely Brautigam <182850+Skarlso@users.noreply.github.com>
@Skarlso
use the actual type after repository scheme
5c01907 
On-behalf-of: Gergely Brautigam <gergely.brautigam@sap.com>

Signed-off-by: Gergely Brautigam <182850+Skarlso@users.noreply.github.com>
@Skarlso
add test coverage for GetRepositorySpec
8b16b27 
On-behalf-of: Gergely Brautigam <gergely.brautigam@sap.com>

Signed-off-by: Gergely Brautigam <182850+Skarlso@users.noreply.github.com>
@Skarlso
the beginnings of a refactor to not duplicate logic over reference pa…
0ed87b3 
…rsing

On-behalf-of: Gergely Brautigam <gergely.brautigam@sap.com>

Signed-off-by: Gergely Brautigam <182850+Skarlso@users.noreply.github.com>
@Skarlso
shifted the tests into compdesc
317cdab 
On-behalf-of: Gergely Brautigam <gergely.brautigam@sap.com>

Signed-off-by: Gergely Brautigam <182850+Skarlso@users.noreply.github.com>
@Skarlso
add integration test for add component version
9409127 
On-behalf-of: Gergely Brautigam <gergely.brautigam@sap.com>

Signed-off-by: Gergely Brautigam <182850+Skarlso@users.noreply.github.com>
@Skarlso Skarlso force-pushed the support-oci-repositories branch from a7c9ab2 to f61ceaa Compare September 1, 2025 08:20
@Skarlso Skarlso marked this pull request as ready for review September 1, 2025 08:32
jakobmoellerdev
jakobmoellerdev reviewed Sep 1, 2025
View reviewed changes
@Skarlso
Copy link
Contributor Author

Skarlso commented Sep 2, 2025

We talked about this, I'm going to try to remedy the integration test situation. For now, add component version is failing with

        	            	error constructing components: error checking for existing component version in target repository: failed to get manifest: failed to resolve reference "localhost:53806/component-descriptors/ocm.software/explicit-add-oci-component:v2.0.0": HEAD "http://localhost:53806/v2/component-descriptors/ocm.software/explicit-add-oci-component/manifests/v2.0.0": response status code 401: Unauthorized

Unauthorized if run together with others. And doesn't fail alone. Even after modifying the container name, random ports, and new component names.

@Skarlso
Copy link
Contributor Author

Skarlso commented Sep 2, 2025

ah I bet it's the config 🤦

@Skarlso
Copy link
Contributor Author

Skarlso commented Sep 2, 2025

Narrator: But it wasn't the config.

fabianburth
fabianburth reviewed Sep 2, 2025
View reviewed changes
@Skarlso
add scheme to the repository spec for credentials to work
e6a5603 
On-behalf-of: Gergely Brautigam <gergely.brautigam@sap.com>

Signed-off-by: Gergely Brautigam <182850+Skarlso@users.noreply.github.com>
@Skarlso Skarlso force-pushed the support-oci-repositories branch from 31468ed to e6a5603 Compare September 2, 2025 13:29
@Skarlso
Copy link
Contributor Author

Skarlso commented Sep 2, 2025

The integration test madness is over. Now focusing on the actual coding. :D

Skarlso and others added 2 commits September 2, 2025 16:37
@Skarlso @jakobmoellerdev
Apply suggestions from code review
c938ed2 
Co-authored-by: Jakob Möller <contact@jakob-moeller.com>
Signed-off-by: Gergely Brautigam <skarlso777@gmail.com>
@Skarlso
adjusted the renamed function usage
73fb386 
On-behalf-of: Gergely Brautigam <gergely.brautigam@sap.com>

Signed-off-by: Gergely Brautigam <182850+Skarlso@users.noreply.github.com>
@Skarlso
Copy link
Contributor Author

Skarlso commented Sep 2, 2025

this is just a repository reference now (new concept, didnt exist before). You will need to document in detail how this works!

Where should I document it?

@jakobmoellerdev
Copy link
Contributor

this is just a repository reference now (new concept, didnt exist before). You will need to document in detail how this works!

Where should I document it?

It needs a documentation in the reference docs for the command. See get cv on examples on ref documentation

@Skarlso
renamed spec and added logger
8b8e7c7 
On-behalf-of: Gergely Brautigam <gergely.brautigam@sap.com>

Signed-off-by: Gergely Brautigam <182850+Skarlso@users.noreply.github.com>
@Skarlso
renamed in test
4f91d48 
On-behalf-of: Gergely Brautigam <gergely.brautigam@sap.com>

Signed-off-by: Gergely Brautigam <182850+Skarlso@users.noreply.github.com>
@Skarlso Skarlso force-pushed the support-oci-repositories branch from 9798c5a to 4f91d48 Compare September 3, 2025 10:55
Skarlso
Skarlso commented Sep 3, 2025
View reviewed changes
@Skarlso Skarlso force-pushed the support-oci-repositories branch 3 times, most recently from b4bd444 to 53e3f22 Compare September 3, 2025 12:54
@Skarlso
adding documentation of the ref parsing behaviour for add component a…
2d215ee 
…nd examples

On-behalf-of: Gergely Brautigam <gergely.brautigam@sap.com>

Signed-off-by: Gergely Brautigam <182850+Skarlso@users.noreply.github.com>
@Skarlso Skarlso force-pushed the support-oci-repositories branch from 53e3f22 to 2d215ee Compare September 3, 2025 12:56
jakobmoellerdev
jakobmoellerdev reviewed Sep 3, 2025
View reviewed changes
Copy link
Contributor

@jakobmoellerdev jakobmoellerdev left a comment

Choose a reason for hiding this comment

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

I still think the tests are very weird, once that is done Im good for a merge 🥳

Skarlso and others added 2 commits September 4, 2025 08:53
@Skarlso
changed the test to use proper types instead of strings
4296667 
On-behalf-of: Gergely Brautigam <gergely.brautigam@sap.com>

Signed-off-by: Gergely Brautigam <182850+Skarlso@users.noreply.github.com>
@jakobmoellerdev
Merge branch 'main' into support-oci-repositories
4d0ef8d
@jakobmoellerdev jakobmoellerdev enabled auto-merge (squash) September 4, 2025 07:11
@jakobmoellerdev jakobmoellerdev merged commit ee02d71 into open-component-model:main Sep 4, 2025
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fabianburth fabianburth fabianburth left review comments

@jakobmoellerdev jakobmoellerdev jakobmoellerdev approved these changes

Assignees
No one assigned
Labels
kind/feature new feature, enhancement, improvement, extension size/l Large
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

make ocm add cv --repository to support OCI registries
3 participants
@Skarlso @jakobmoellerdev @fabianburth