feat: updated workflow parameters to ignore values without a default.

This commit is contained in:
Andrey Melnikov
2020-03-20 12:38:38 -07:00
parent cb821694c3
commit 0e100b83d0

View File

@@ -845,12 +845,22 @@ func filterOutCustomTypesFromManifest(manifest []byte) (result []byte, err error
return manifest, nil
}
// We might not want some parameters due to data structuring.
parametersToKeep := make([]interface{}, 0)
for _, parameter := range parametersList {
paramMap, ok := parameter.(map[string]interface{})
if !ok {
continue
}
// If the parameter does not have a value, skip it so argo doesn't try to process it and fail.
if _, hasValue := paramMap["value"]; !hasValue {
continue
}
parametersToKeep = append(parametersToKeep, parameter)
keysToDelete := make([]string, 0)
for key := range paramMap {
if key != "name" && key != "value" {
@@ -863,5 +873,7 @@ func filterOutCustomTypesFromManifest(manifest []byte) (result []byte, err error
}
}
argumentsMap["parameters"] = parametersToKeep
return yaml.Marshal(data)
}