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

helm search hub should optionally return the repo URL as part of the results #7419

Closed
omkensey opened this issue Jan 16, 2020 · 7 comments
Closed

Comments

@omkensey
Copy link

It would be nice if helm search hub could give the URL of the repo (not necessarily by default, but as an optional column/JSON key), so as to avoid a roundtrip to the Hub description page for the actual repo URL if I can tell from the search listings which chart repo I want.

@SimonHeimberg
Copy link

SimonHeimberg commented Apr 6, 2020

The hub server returns the information, only cli needs adaption.

So a possible work around (when searching "rancher server"):
curl https://hub.helm.sh/api/chartsvc/v1/charts/search?q=rancher+server | json_pp
The repo url is in key url in repo (data[...].attributes.repo.url).
(json_pp is optional. It pretty prints json, which helps me to read it.)

@bacongobbler
Copy link
Member

bacongobbler commented Apr 6, 2020

This is a great suggestion, @omkensey. I think adding another field to the table output displaying the repository URL would be a welcome change.

A little bit of context here: The idea with helm search hub returning that URL was that the chart repository maintainers were looking to add support to use that URL as input to helm install, like so:

><> helm search hub wordpress --max-col-width 0 | tail -n 1
https://hub.helm.sh/charts/bitnami/wordpress           	9.1.1        	5.3.2      	Web publishing platform for building blogs and websites.

then:

><> helm install wordpress https://hub.helm.sh/charts/bitnami/wordpress

cc'ing @mattfarina as he may have more context on where we are with that particular feature.

Adding the repository URL to the table output would be good as well. More options are better :)

@SimonHeimberg
Copy link

SimonHeimberg commented Apr 7, 2020

A little bit of context here: ...

Thank you. I did not see this in the doc. Now I found it in docs/intro/using_helm since I knew what to look for.

@tharun208
Copy link

I like to work on this

@mattfarina
Copy link
Collaborator

I like the idea of adding the column. I would also like to see ><> helm install wordpress https://hub.helm.sh/charts/bitnami/wordpress work. This might be something to add into the Artifact Hub as well.

@scottrigby
Copy link
Member

I like both of these ideas::

  1. optional flag (--list-repo or something)
  2. updating Artifact Hub to allow installing directly from paths like https://artifacthub.io/packages/helm/bitnami/etcd (curl https://hub.helm.sh/api/chartsvc/v1/charts/search?q=etcd)

How would idea 2 ideally work though? Maybe under the hood something like helm install option 5 (By chart reference and repo url: helm install --repo https://example.com/charts/mynginx nginx), grabbing chart name by splitting second part of monocular search API r.id (etcd from bitnami/etcd)?

@github-actions
Copy link

This issue has been marked as stale because it has been open for 90 days with no activity. This thread will be automatically closed in 30 days if no further activity occurs.

@github-actions github-actions bot added the Stale label Nov 20, 2020
ErmakovDmitriy added a commit to ErmakovDmitriy/helm that referenced this issue Dec 12, 2020
The flag makes helm print the chart repo URL to use with helm repo add.
Issue: helm#7419

Signed-off-by: Dmitriy Ermakov <demonihin@gmail.com>
zak905 pushed a commit to zak905/helm that referenced this issue Jan 19, 2023
The flag makes helm print the chart repo URL to use with helm repo add.
Issue: helm#7419

Signed-off-by: Dmitriy Ermakov <demonihin@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants