Skip to content

Commit ba3441f

Browse files
authored
chore: Remove the "??" character from the error log (#14340)
Signed-off-by: HirazawaUi <[email protected]>
1 parent 893c4b0 commit ba3441f

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

server/artifacts/artifact_server.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -422,7 +422,7 @@ func (a *ArtifactServer) getArtifactAndDriver(ctx context.Context, nodeId, artif
422422
if templateName != "" {
423423
template := wf.GetTemplateByName(templateName)
424424
if template == nil {
425-
return nil, nil, fmt.Errorf("no template found by the name of '%s' (which is the template associated with nodeId '%s'??", templateName, nodeId)
425+
return nil, nil, fmt.Errorf("no template found for name %q associated with nodeId %q", templateName, nodeId)
426426
}
427427
archiveLocation = template.ArchiveLocation // this is case 4
428428
}

workflow/controller/artifact_gc.go

+4-4
Original file line numberDiff line numberDiff line change
@@ -177,7 +177,7 @@ func (woc *wfOperationCtx) processArtifactGCStrategy(ctx context.Context, strate
177177
node, err := woc.wf.Status.Nodes.Get(artifactSearchResult.NodeID)
178178
if err != nil {
179179
woc.log.Errorf("Was unable to obtain node for %s", artifactSearchResult.NodeID)
180-
return fmt.Errorf("can't process Artifact GC Strategy %s: node ID %q not found in Status??", strategy, artifactSearchResult.NodeID)
180+
return fmt.Errorf("can't process Artifact GC Strategy %s: node ID %q not found in Status", strategy, artifactSearchResult.NodeID)
181181
}
182182
templateName := util.GetTemplateFromNode(*node)
183183
if templateName == "" {
@@ -187,7 +187,7 @@ func (woc *wfOperationCtx) processArtifactGCStrategy(ctx context.Context, strate
187187
if !found {
188188
template = woc.wf.GetTemplateByName(templateName)
189189
if template == nil {
190-
return fmt.Errorf("can't process Artifact GC Strategy %s: template name %q belonging to node %+v not found??", strategy, templateName, node)
190+
return fmt.Errorf("can't process Artifact GC Strategy %s: template name %q belonging to node %+v not found", strategy, templateName, node)
191191
}
192192
templatesByName[templateName] = template
193193
}
@@ -219,7 +219,7 @@ func (woc *wfOperationCtx) processArtifactGCStrategy(ctx context.Context, strate
219219
// create the pod
220220
podAccessInfo, found := podNames[podName]
221221
if !found {
222-
return fmt.Errorf("can't find podInfo for podName %q??", podName)
222+
return fmt.Errorf("can't find podInfo for podName %q", podName)
223223
}
224224

225225
_, err := woc.createArtifactGCPod(ctx, strategy, tasks, podAccessInfo, podName, templatesToArtList, templatesByName)
@@ -401,7 +401,7 @@ func (woc *wfOperationCtx) createArtifactGCPod(ctx context.Context, strategy wfv
401401
for templateName, artifacts := range templatesToArtList {
402402
template, found := templatesByName[templateName]
403403
if !found {
404-
return nil, fmt.Errorf("can't find template with name %s???", templateName)
404+
return nil, fmt.Errorf("can't find template with name %q", templateName)
405405
}
406406

407407
if template.ArchiveLocation.HasLocation() {

0 commit comments

Comments
 (0)