Removed whitelist-var-run normalization as this breaks functionality (#1956)
Closes #1795
This commit is contained in:
		
							parent
							
								
									9969c74703
								
							
						
					
					
						commit
						86493d07cc
					
				|  | @ -58,18 +58,8 @@ func init() { | ||||||
| 
 | 
 | ||||||
| 	addKanikoOptionsFlags() | 	addKanikoOptionsFlags() | ||||||
| 	addHiddenFlags(RootCmd) | 	addHiddenFlags(RootCmd) | ||||||
| 	RootCmd.PersistentFlags().BoolVarP(&opts.IgnoreVarRun, "whitelist-var-run", "", true, "Ignore /var/run directory when taking image snapshot. Set it to false to preserve /var/run/ in destination image. (Default true).") | 	RootCmd.PersistentFlags().BoolVarP(&opts.IgnoreVarRun, "whitelist-var-run", "", true, "Ignore /var/run directory when taking image snapshot. Set it to false to preserve /var/run/ in destination image.") | ||||||
| 	RootCmd.PersistentFlags().MarkDeprecated("whitelist-var-run", "please use ignore-var-run instead.") | 	RootCmd.PersistentFlags().MarkDeprecated("whitelist-var-run", "Please use ignore-var-run instead.") | ||||||
| 	RootCmd.PersistentFlags().SetNormalizeFunc(normalizeWhitelistVarRun) |  | ||||||
| } |  | ||||||
| 
 |  | ||||||
| func normalizeWhitelistVarRun(f *pflag.FlagSet, name string) pflag.NormalizedName { |  | ||||||
| 	switch name { |  | ||||||
| 	case "whitelist-var-run": |  | ||||||
| 		name = "ignore-var-run" |  | ||||||
| 		break |  | ||||||
| 	} |  | ||||||
| 	return pflag.NormalizedName(name) |  | ||||||
| } | } | ||||||
| 
 | 
 | ||||||
| // RootCmd is the kaniko command that is run
 | // RootCmd is the kaniko command that is run
 | ||||||
|  | @ -212,7 +202,7 @@ func addKanikoOptionsFlags() { | ||||||
| 	opts.RegistriesCertificates = make(map[string]string) | 	opts.RegistriesCertificates = make(map[string]string) | ||||||
| 	RootCmd.PersistentFlags().VarP(&opts.RegistriesCertificates, "registry-certificate", "", "Use the provided certificate for TLS communication with the given registry. Expected format is 'my.registry.url=/path/to/the/server/certificate'.") | 	RootCmd.PersistentFlags().VarP(&opts.RegistriesCertificates, "registry-certificate", "", "Use the provided certificate for TLS communication with the given registry. Expected format is 'my.registry.url=/path/to/the/server/certificate'.") | ||||||
| 	RootCmd.PersistentFlags().VarP(&opts.RegistryMirrors, "registry-mirror", "", "Registry mirror to use as pull-through cache instead of docker.io. Set it repeatedly for multiple mirrors.") | 	RootCmd.PersistentFlags().VarP(&opts.RegistryMirrors, "registry-mirror", "", "Registry mirror to use as pull-through cache instead of docker.io. Set it repeatedly for multiple mirrors.") | ||||||
| 	RootCmd.PersistentFlags().BoolVarP(&opts.IgnoreVarRun, "ignore-var-run", "", true, "Ignore /var/run directory when taking image snapshot. Set it to false to preserve /var/run/ in destination image. (Default true).") | 	RootCmd.PersistentFlags().BoolVarP(&opts.IgnoreVarRun, "ignore-var-run", "", true, "Ignore /var/run directory when taking image snapshot. Set it to false to preserve /var/run/ in destination image.") | ||||||
| 	RootCmd.PersistentFlags().VarP(&opts.Labels, "label", "", "Set metadata for an image. Set it repeatedly for multiple labels.") | 	RootCmd.PersistentFlags().VarP(&opts.Labels, "label", "", "Set metadata for an image. Set it repeatedly for multiple labels.") | ||||||
| 	RootCmd.PersistentFlags().BoolVarP(&opts.SkipUnusedStages, "skip-unused-stages", "", false, "Build only used stages if defined to true. Otherwise it builds by default all stages, even the unnecessaries ones until it reaches the target stage / end of Dockerfile") | 	RootCmd.PersistentFlags().BoolVarP(&opts.SkipUnusedStages, "skip-unused-stages", "", false, "Build only used stages if defined to true. Otherwise it builds by default all stages, even the unnecessaries ones until it reaches the target stage / end of Dockerfile") | ||||||
| 	RootCmd.PersistentFlags().BoolVarP(&opts.RunV2, "use-new-run", "", false, "Use the experimental run implementation for detecting changes without requiring file system snapshots.") | 	RootCmd.PersistentFlags().BoolVarP(&opts.RunV2, "use-new-run", "", false, "Use the experimental run implementation for detecting changes without requiring file system snapshots.") | ||||||
|  |  | ||||||
		Loading…
	
		Reference in New Issue