Merge pull request #1051 from monopole/fixNits

Fix a plugin nit.
This commit is contained in:
Jeff Regan
2019-05-07 17:58:15 -07:00
committed by GitHub
2 changed files with 10 additions and 8 deletions

View File

@@ -12,26 +12,26 @@ func GeneratorArgsFromKunstruct(k ifc.Kunstructured) (
result.Name = k.GetName() result.Name = k.GetName()
// TODO: validate behavior values. // TODO: validate behavior values.
result.Behavior, err = k.GetFieldValue("behavior") result.Behavior, err = k.GetFieldValue("behavior")
if !isAcceptableError(err) { if !IsAcceptableError(err) {
return return
} }
result.EnvSources, err = k.GetStringSlice("envFiles") result.EnvSources, err = k.GetStringSlice("envFiles")
if !isAcceptableError(err) { if !IsAcceptableError(err) {
return return
} }
result.FileSources, err = k.GetStringSlice("valueFiles") result.FileSources, err = k.GetStringSlice("valueFiles")
if !isAcceptableError(err) { if !IsAcceptableError(err) {
return return
} }
result.LiteralSources, err = k.GetStringSlice("literals") result.LiteralSources, err = k.GetStringSlice("literals")
if !isAcceptableError(err) { if !IsAcceptableError(err) {
return return
} }
err = nil err = nil
return return
} }
func isAcceptableError(err error) bool { func IsAcceptableError(err error) bool {
return err == nil || return err == nil ||
strings.HasPrefix(err.Error(), "no field named") strings.HasPrefix(err.Error(), "no field named")
} }

View File

@@ -25,9 +25,11 @@ func (p *plugin) Config(
if err != nil { if err != nil {
return return
} }
// Ignore missing type - it defaults. p.args.Type, err = k.GetFieldValue("type")
p.args.Type, _ = k.GetFieldValue("type") if !resmap.IsAcceptableError(err) {
return return
}
return nil
} }
func (p *plugin) Generate() (resmap.ResMap, error) { func (p *plugin) Generate() (resmap.ResMap, error) {