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.

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

feat: Version not required for install, update commands #1198

Closed
technosophos opened this issue Sep 14, 2016 · 3 comments
Closed

feat: Version not required for install, update commands #1198

technosophos opened this issue Sep 14, 2016 · 3 comments
Assignees
@technosophos
Milestone
2.0.0-Alpha.5

Comments

@technosophos
Copy link
Member

Currently, to install a chart, a user must supply the chart and version:

$ helm install charts/nginx-1.2.3

Likewise with other commands like helm upgrade

To better follow the conventions of other package managers, version should not be required.

$ helm install charts/nginx
# installs latest nginx: nginx-1.2.3

Specifying version should be an explicit operation:

$ helm install charts/nginx --version 1.2.2
# installs older nginx 1.2.2

Additionally, it has been suggested that we bring back the idea of a default repository:

$ helm repo default charts
Repo 'charts' is now your default
$ helm install nginx
# installs charts/nginx 1.2.3

Analogous changes would be made of helm upgrade, too.
@technosophos technosophos added this to the 2.0.0-Alpha.5 milestone Sep 14, 2016
@technosophos
Copy link
Member Author

/cc @prydonius @michelleN @viglesiasce @migmartri @jackfrancis @adamreese

@technosophos technosophos mentioned this issue Sep 14, 2016
Closed
@jackfrancis
Copy link

@technosophos speaking to a default chart repo, do we have analogous commands that resemble the below?:

$ helm repo list
stable       https://github.com/kubernetes/charts (default)
incubator    https://github.com/kubernetes/charts/incubator
$ helm repo default
'stable' is the default repo
stable       https://github.com/kubernetes/charts

@technosophos
Copy link
Member Author

We have a helm repo list, but no notion of a default

@technosophos technosophos self-assigned this Oct 5, 2016
technosophos added a commit to technosophos/k8s-helm that referenced this issue Oct 7, 2016
@technosophos
feat(helm): remove the requirement that fetch/install need version
4faa2ea 
This removes the requirement that a fetch or install command must
explicitly state the version number to install. Instead, this goes to
the strategy used by OS package managers: Install the latest until told
to do otherwise.

Closes helm#1198
technosophos added a commit to technosophos/k8s-helm that referenced this issue Oct 7, 2016
@technosophos
feat(helm): remove the requirement that fetch/install need version
040a206 
This removes the requirement that a fetch or install command must
explicitly state the version number to install. Instead, this goes to
the strategy used by OS package managers: Install the latest until told
to do otherwise.

Closes helm#1198
@technosophos technosophos mentioned this issue Oct 7, 2016
Merged
technosophos added a commit to technosophos/k8s-helm that referenced this issue Oct 7, 2016
@technosophos
feat(helm): remove the requirement that fetch/install need version
4eda5e0 
This removes the requirement that a fetch or install command must
explicitly state the version number to install. Instead, this goes to
the strategy used by OS package managers: Install the latest until told
to do otherwise.

Closes helm#1198
technosophos added a commit to technosophos/k8s-helm that referenced this issue Oct 7, 2016
@technosophos
feat(helm): remove the requirement that fetch/install need version
193315e 
This removes the requirement that a fetch or install command must
explicitly state the version number to install. Instead, this goes to
the strategy used by OS package managers: Install the latest until told
to do otherwise.

Closes helm#1198
@technosophos technosophos mentioned this issue Oct 7, 2016
Closed
technosophos added a commit to technosophos/k8s-helm that referenced this issue Oct 7, 2016
@technosophos
feat(helm): remove the requirement that fetch/install need version
1c6fc9c 
This removes the requirement that a fetch or install command must
explicitly state the version number to install. Instead, this goes to
the strategy used by OS package managers: Install the latest until told
to do otherwise.

Closes helm#1198
@technosophos technosophos mentioned this issue Oct 7, 2016
Closed
vaikas pushed a commit to vaikas/deployment-manager that referenced this issue Oct 17, 2016
@technosophos
feat(helm): remove the requirement that fetch/install need version
e429aaf 
This removes the requirement that a fetch or install command must
explicitly state the version number to install. Instead, this goes to
the strategy used by OS package managers: Install the latest until told
to do otherwise.

Closes helm#1198
MichaelMorrisEst pushed a commit to Nordix/helm that referenced this issue Nov 17, 2023
@doughgle
Put tiller on path for use in helm-tiller plugin (helm#1198)
f39d792 
Put tiller binary on the path in helm v2.X container image so that helm-tiller plugin will use it instead of duplicating the download of helm tarball and performing version check.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@technosophos technosophos

Labels
None yet
Projects
None yet
Milestone
2.0.0-Alpha.5
Development

No branches or pull requests
2 participants
@technosophos @jackfrancis