Skip to content
This repository has been archived by the owner on Sep 12, 2023. It is now read-only.

Add --devel flag so helm can see rc.x versions #176

Merged
merged 1 commit into from Nov 7, 2018

Conversation

marcoderama
Copy link
Contributor

Unbelievably (to me at least) helm ignores by default versions that look like a.b.c.d-e. This means that the install script is currently ignoring our release candidates (e.g. 1.0.0-rc.3).

c.f. helm/helm#2407

Adding the --devel flag will pick them up.

@marcoderama marcoderama requested a review from a team November 7, 2018 01:52
@ghost ghost assigned marcoderama Nov 7, 2018
@ghost ghost added the review label Nov 7, 2018
@@ -196,7 +196,7 @@ if [ "false" != "$ES_EXPORT_YAML" ]; then

if [ -z "$ES_LOCAL_CHART" ]; then
TDIR=$(mktemp -d 2>/dev/null || mktemp -d -t 'install-es-tdir')
debug helm fetch -d $TDIR "$VERSION_ARG" "$chart_ref"
debug helm fetch -d $TDIR --devel "$VERSION_ARG" "$chart_ref"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

according to man page --devel option is ignored when version is specified.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah. $VERSION_ARG will have a value if and only if the user specifies a value on the install-es.sh command line. If they do, --devel will be ignored. If they don't, it'll be empty and --devel will come into play.

@jsravn
Copy link
Contributor

jsravn commented Nov 7, 2018

To be honest this sounds good to me - helm doesn't install pre-release versions by default. It's not intuitive at all, but the feature itself is nice.

Copy link
Contributor

@jsravn jsravn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you eliminate the duplication somehow? It's unfortunate we have essentially 3 duplications of the args we pass in.

@marcoderama
Copy link
Contributor Author

And now that I think about it, we may want to change the install script to allow for installation of "stable" or "bleeding edge" versions. If "stable", then we don't consider -rc.x versions and drop the --devel flag internally. Later...

@marcoderama marcoderama merged commit ae65ef5 into master Nov 7, 2018
@ghost ghost removed the review label Nov 7, 2018
@dbrinegar dbrinegar deleted the ariano.notiller branch February 1, 2019 04:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants