Skip to content
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.

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

[RFE] Support OCI registries for charts #231

Closed
clausa opened this issue Apr 18, 2024 · 2 comments
Closed

[RFE] Support OCI registries for charts #231

clausa opened this issue Apr 18, 2024 · 2 comments
Labels
enhancement New feature or request size/M Denotes an issue/PR requiring a relatively moderate amount of work

Comments

@clausa
Copy link
Contributor

clausa commented Apr 18, 2024

Is this RFE related to an Existing Problem? If so, please describe:

It seems that hauler does not support fetching Helm charts from OCI registries:

$ hauler version
 __    __       ___       __    __   __       _______ .______
|  |  |  |     /   \     |  |  |  | |  |     |   ____||   _  \
|  |__|  |    /  ^  \    |  |  |  | |  |     |  |__   |  |_)  |
|   __   |   /  /_\  \   |  |  |  | |  |     |   __|  |      /
|  |  |  |  /  _____  \  |  `--'  | |  `----.|  |____ |  |\  \----.
|__|  |__| /__/     \__\  \______/  |_______||_______|| _| `._____|
hauler: Airgap Swiss Army Knife

GitVersion:    1.0.2
GitCommit:     0674e0a
GitTreeState:  clean
BuildDate:     2024-04-15T16:37:17Z
GoVersion:     go1.21.9
Compiler:      gc
Platform:      darwin/arm64

$ hauler store add chart memcached --repo "oci://registry-1.docker.io/bitnamicharts/memcached"  -l debug 2> /dev/null
2024-04-18 11:25:56 DBG running cli command [hauler store add chart]
2024-04-18 11:25:56 DBG using store at /path/to/store
2024-04-18 11:25:56 INF adding 'chart' [memcached] to the store
2024-04-18 11:25:56 ERR looks like "oci://registry-1.docker.io/bitnamicharts/memcached" is not a valid chart repository or cannot be reached: object required

$ helm pull oci://registry-1.docker.io/bitnamicharts/memcached
Pulled: registry-1.docker.io/bitnamicharts/memcached:7.0.3
Digest: sha256:40a21252594826385a398101e1840a1f6c39379066812ef5a651a5517310e9b3

Describe Proposed Solution(s):

  • Add OCI registry support for Helm charts

Describe Possible Alternatives:

  • Use helm to fetch charts from OCI registries to a local folder first, but it kind of defeats the purpose of hauler.

Additional Context:

  • N/A
@clausa clausa added the enhancement New feature or request label Apr 18, 2024
@github-project-automation github-project-automation bot moved this to Pending Review in Hauler Apr 18, 2024
@clausa
Copy link
Contributor Author

clausa commented Apr 18, 2024

Perhaps same issue as in #204 ?

@zackbradys zackbradys moved this from Pending Review to Work In Progress in Hauler Apr 18, 2024
@zackbradys zackbradys added the size/M Denotes an issue/PR requiring a relatively moderate amount of work label Apr 18, 2024
@zackbradys
Copy link
Member

Closing as resolved per PR #245!

@github-project-automation github-project-automation bot moved this from Work In Progress to Resolved in Hauler Jun 4, 2024
@github-project-automation github-project-automation bot moved this to Pending Review in Hauler Jul 17, 2024
@zackbradys zackbradys moved this from Pending Review to Resolved in Hauler Jul 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request size/M Denotes an issue/PR requiring a relatively moderate amount of work
Projects
Status: Resolved
Development

No branches or pull requests

2 participants