-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
Implement some completion for "kops create cluster" #11940
Conversation
6258564
to
6afda4b
Compare
6afda4b
to
6d16c13
Compare
/retest |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: olemarkus The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
completions = append(completions, "amazonvpc", "lyftvpc") | ||
} | ||
|
||
if cloudup.AlphaAllowGCE.Enabled() && (options.CloudProvider == "gce" || options.CloudProvider == "") { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I noticed a few odd behaviors with checking the feature flag here.
- The feature flag needs to be set prior to
kops completion
evaluation rather than at the time of tab completion. I suppose this isn't a big deal since most users would have this particular feature flag set in their default shell environment. - With the feature flag set,
kops completion
logs some output that should be silenced:
export KOPS_FEATURE_FLAGS=+AlphaAllowGCE
eval $(./.build/local/kops completion zsh)
I0718 08:50:04.388774 17741 featureflag.go:167] FeatureFlag "AlphaAllowGCE"=true
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think the logging behavior of kops completion
changed. The old implementation didn't suppress that output either.
No description provided.