Merge pull request #74032 from tedyu/master

Use Set to check whether flag name is allowed flag
pull/564/head
Kubernetes Prow Robot 2019-02-13 20:33:17 -08:00 committed by GitHub
commit d01b01beb2
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 10 deletions

View File

@ -416,23 +416,18 @@ func ValidateMixedArguments(flag *pflag.FlagSet) error {
} }
func isAllowedFlag(flagName string) bool { func isAllowedFlag(flagName string) bool {
isAllowed := false knownFlags := sets.NewString(kubeadmcmdoptions.CfgPath,
switch flagName {
case kubeadmcmdoptions.CfgPath,
kubeadmcmdoptions.IgnorePreflightErrors, kubeadmcmdoptions.IgnorePreflightErrors,
kubeadmcmdoptions.DryRun, kubeadmcmdoptions.DryRun,
kubeadmcmdoptions.KubeconfigPath, kubeadmcmdoptions.KubeconfigPath,
kubeadmcmdoptions.NodeName, kubeadmcmdoptions.NodeName,
kubeadmcmdoptions.NodeCRISocket, kubeadmcmdoptions.NodeCRISocket,
kubeadmcmdoptions.KubeconfigDir, kubeadmcmdoptions.KubeconfigDir,
"print-join-command", "rootfs", "v": "print-join-command", "rootfs", "v")
isAllowed = true if knownFlags.Has(flagName) {
default: return true
if strings.HasPrefix(flagName, "skip-") {
isAllowed = true
} }
} return strings.HasPrefix(flagName, "skip-")
return isAllowed
} }
// ValidateFeatureGates validates provided feature gates // ValidateFeatureGates validates provided feature gates