Skip to content

Commit

Permalink
fix: Fix swapped artifact repository key and ref in error message (#5272
Browse files Browse the repository at this point in the history
)

Signed-off-by: terrytangyuan <terrytangyuan@gmail.com>
  • Loading branch information
terrytangyuan committed Mar 3, 2021
1 parent 69c40c0 commit b062cbf
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion workflow/artifactrepositories/artifactrepositories.go
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ func (s *artifactRepositories) get(ctx context.Context, ref *wfv1.ArtifactReposi
key := ref.GetKeyOr(cm.Annotations["workflows.argoproj.io/default-artifact-repository"])
value, ok := cm.Data[key]
if !ok {
return nil, nil, fmt.Errorf(`config map missing key %s for artifact repository ref "%v"`, ref, key)
return nil, nil, fmt.Errorf(`config map missing key "%s" for artifact repository ref "%v"`, key, ref)
}
repo := &config.ArtifactRepository{}
// we need the fully filled out ref so we can store it in the workflow status and it will never change
Expand Down

0 comments on commit b062cbf

Please sign in to comment.