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

feat: Expand 'helm search' #1199

Closed
technosophos opened this issue Sep 14, 2016 · 3 comments · Fixed by #1284
Closed

feat: Expand 'helm search' #1199

technosophos opened this issue Sep 14, 2016 · 3 comments · Fixed by #1284
Milestone

Comments

@technosophos
Copy link
Member

It has been suggested that we expand the scope of helm search, and add new similar tools, in an effort to improve findability of charts.

New output for Helm search

No longer show versions as a default in search output:

$ helm search nginx
stable/nginx
stable/nginx-reverse-proxy

Add a command: helm charts list

$ helm charts list
stable/alpine
stable/nginx
incubator/zookeeper

If helm search is called with no arguments, it returns the results of helm charts list

List all charts in a specific repository with helm charts list --repo stable

$ helm charts list --repo stable
stable/alpine
stable/nginx

Alias helm search --repo stable to the above.

List versions only when requested

$ helm search --versions nginx
stable/nginx      1.1.2
stable/nginx      1.2.3
@technosophos technosophos added this to the 2.0.0-Alpha.5 milestone Sep 14, 2016
@technosophos
Copy link
Member Author

technosophos added a commit to technosophos/k8s-helm that referenced this issue Sep 24, 2016
Switch 'helm search' from file crawling to using the indices. Also
add scorable indexing, forward porting the search code I originally
wrote for Helm Classic.

Closes helm#1226
Partially addresses helm#1199
technosophos added a commit to technosophos/k8s-helm that referenced this issue Sep 26, 2016
Switch 'helm search' from file crawling to using the indices. Also
add scorable indexing, forward porting the search code I originally
wrote for Helm Classic.

Closes helm#1226
Partially addresses helm#1199
technosophos added a commit to technosophos/k8s-helm that referenced this issue Sep 27, 2016
Switch 'helm search' from file crawling to using the indices. Also
add scorable indexing, forward porting the search code I originally
wrote for Helm Classic.

Closes helm#1226
Partially addresses helm#1199
technosophos added a commit to technosophos/k8s-helm that referenced this issue Sep 27, 2016
Switch 'helm search' from file crawling to using the indices. Also
add scorable indexing, forward porting the search code I originally
wrote for Helm Classic.

Closes helm#1226
Partially addresses helm#1199
@technosophos
Copy link
Member Author

As of #1229:

  • The command helm search (no args) returns a list of all charts
  • helm search stable/ (note the slash) returns all charts in the repo stable
  • BUT charts are still listed by version number

technosophos added a commit to technosophos/k8s-helm that referenced this issue Oct 5, 2016
This causes search to index by name/version instead of just name, which
means you can get a list of versions of a chart. The '--versions' flag
enables this behavior.

Partially fixes helm#1199
technosophos added a commit to technosophos/k8s-helm that referenced this issue Oct 5, 2016
This causes search to index by name/version instead of just name, which
means you can get a list of versions of a chart. The '--versions' flag
enables this behavior.

Partially fixes helm#1199
@technosophos
Copy link
Member Author

It has been requested that the helm charts * subcommands be moved to v2.1.0 so that we don't have to rush them right before Alpha.5. I think that all of the features listed above are now present (as of #1284). So when that gets merged, I will move the milestone on this to 2.1.0-Alpha.1 unless somebody objects.

vaikas pushed a commit to vaikas/deployment-manager that referenced this issue Oct 17, 2016
Switch 'helm search' from file crawling to using the indices. Also
add scorable indexing, forward porting the search code I originally
wrote for Helm Classic.

Closes helm#1226
Partially addresses helm#1199
vaikas pushed a commit to vaikas/deployment-manager that referenced this issue Oct 17, 2016
This causes search to index by name/version instead of just name, which
means you can get a list of versions of a chart. The '--versions' flag
enables this behavior.

Partially fixes helm#1199
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 a pull request may close this issue.

1 participant