Skip to content

Commit

Permalink
fix: Fix unit test with missing createRunningPods() (argoproj#6585)
Browse files Browse the repository at this point in the history
Signed-off-by: Yuan Tang <[email protected]>
  • Loading branch information
terrytangyuan committed Aug 23, 2021
1 parent 48d7ad3 commit d6f5acb
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions workflow/controller/controller_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -281,9 +281,7 @@ func withAnnotation(key, val string) with {

// createRunningPods creates the pods that are marked as running in a given test so that they can be accessed by the
// pod assessor
// This function is called `createRunningPods`, but since it is currently not used and should not be removed it has been
// renamed to `_` until a use is found.
func _(ctx context.Context, woc *wfOperationCtx) {
func createRunningPods(ctx context.Context, woc *wfOperationCtx) {
podcs := woc.controller.kubeclientset.CoreV1().Pods(woc.wf.GetNamespace())
for _, node := range woc.wf.Status.Nodes {
if node.Type == wfv1.NodeTypePod && node.Phase == wfv1.NodeRunning {
Expand Down

0 comments on commit d6f5acb

Please sign in to comment.