argo-workflows icon indicating copy to clipboard operation
argo-workflows copied to clipboard

Intermittent CI failures in Unit Tests

Open juliev0 opened this issue 3 years ago • 9 comments

My PR doesn't make any code changes, yet I just had two different unit test failures that passed later after an "empty commit".

Test name: TestRetriesVariableInPodSpecPatch File name: workflow/controller/operator_test.go Line: 1196 Link: https://github.com/argoproj/argo-workflows/runs/7333472812?check_suite_focus=true

Test name: TestFormulateRetryWorkflow/Nested_DAG File name: workflow/util/util_test.go Line: 910 Link: https://github.com/argoproj/argo-workflows/runs/7332453716?check_suite_focus=true

Update 7/15/22: @terrytangyuan has now fixed the second one

Test name: TestEmissary/Stdout and TestEmissary/Comined File name: emissary_test.go Lines: 45,52 Link: https://github.com/argoproj/argo-workflows/runs/7643460317?check_suite_focus=true

Test name: TestGlobalVarsOnExit File name: operator_test.go Line: 5609 Link: https://github.com/argoproj/argo-workflows/runs/7656323758?check_suite_focus=true


Message from the maintainers:

Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

juliev0 avatar Jul 14 '22 04:07 juliev0

note there's a separate issue related to intermittent e2e test failures in CI here

juliev0 avatar Jul 14 '22 16:07 juliev0

@terrytangyuan I'm not sure why I saw these two intermittent unit test failures yesterday (and now they're working), but interestingly the second one above is related to a new test you added. Could you take a look and see if you have any idea why it might have failed? Thank you!

juliev0 avatar Jul 14 '22 22:07 juliev0

I'll see if I can reproduce locally.

terrytangyuan avatar Jul 14 '22 23:07 terrytangyuan

I'll see if I can reproduce locally.

Thank you!

juliev0 avatar Jul 15 '22 04:07 juliev0

I ran it 20 times and it failed only once locally.

terrytangyuan avatar Jul 15 '22 14:07 terrytangyuan

@juliev0 That test case should be fixed in https://github.com/argoproj/argo-workflows/pull/9156.

terrytangyuan avatar Jul 15 '22 14:07 terrytangyuan

@juliev0 That test case should be fixed in #9156.

Thank you so much for doing that right away!!

juliev0 avatar Jul 15 '22 18:07 juliev0

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If this is a mentoring request, please provide an update here. Thank you for your contributions.

stale[bot] avatar Jul 31 '22 04:07 stale[bot]

keeping this alive...

juliev0 avatar Jul 31 '22 05:07 juliev0

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If this is a mentoring request, please provide an update here. Thank you for your contributions.

stale[bot] avatar Oct 01 '22 17:10 stale[bot]

This issue has been closed due to inactivity. Feel free to re-open if you still encounter this issue.

stale[bot] avatar Oct 16 '22 00:10 stale[bot]

I encountered a similar issue today Here is the log https://github.com/argoproj/argo-workflows/actions/runs/5653773150/job/15315538326#step:4:98

binkoni avatar Jul 25 '23 08:07 binkoni