Skip to content

feat: support oci protocol. fix #2#4

Open
denis-ikomia wants to merge 1 commit intoazimuth-cloud:mainfrom
denis-ikomia:fix/#2/oci_support
Open

feat: support oci protocol. fix #2#4
denis-ikomia wants to merge 1 commit intoazimuth-cloud:mainfrom
denis-ikomia:fix/#2/oci_support

Conversation

@denis-ikomia
Copy link

This add oci support on chart ref.
Quickly tested on my k8s, it works properly for me.

Note: this doesn't support helm < 3.10.1 as it print on stdout something like

Pulled: xxxxx
Digest: xxxxx

even if output is to json ( and break json response parsing ).

@voidlily
Copy link

voidlily commented Dec 9, 2024

This has been working well for me too. One thing I noticed was that I needed to pass the full OCI url to the chart argument instead of using the repo argument. Seems like helm pull --repo ... was deprecated with OCI urls maybe?

@AimbotParce
Copy link

Hi, why is this not merged yet?

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.

3 participants

Comments