nerdctl icon indicating copy to clipboard operation
nerdctl copied to clipboard

TestRunWithJsonFileLogDriver is flaky

Open manugupt1 opened this issue 3 years ago • 6 comments

Description

=== PAUSE TestRunStdin === RUN TestRunWithJsonFileLogDriver run_test.go:237: the maximum number of old log files to retain exceeded 2 files --- FAIL: TestRunWithJsonFileLogDriver (3.42s)

https://github.com/containerd/nerdctl/runs/7312918520?check_suite_focus=true

Steps to reproduce the issue

N/A

Describe the results you received and expected

Test should pass.

What version of nerdctl are you using?

built on master

Are you using a variant of nerdctl? (e.g., Rancher Desktop)

No response

Host information

Fedora 36

manugupt1 avatar Jul 13 '22 03:07 manugupt1

=== RUN TestRunWithJsonFileLogDriver run_test.go:237: the maximum number of old log files to retain exceeded 2 files, got: [/var/lib/nerdctl/1935db59/containers/nerdctl-test/76fbdf10f050749b2d84e4bb136cc4c8f02dddeb302b30244e33cc2c61fe01f4/76fbdf10f050749b2d84e4bb136cc4c8f02dddeb302b30244e33cc2c61fe01f4-json.log /var/lib/nerdctl/1935db59/containers/nerdctl-test/76fbdf10f050749b2d84e4bb136cc4c8f02dddeb302b30244e33cc2c61fe01f4/76fbdf10f050749b2d84e4bb136cc4c8f02dddeb302b30244e33cc2c61fe01f4-json.log.1 /var/lib/nerdctl/1935db59/containers/nerdctl-test/76fbdf10f050749b2d84e4bb136cc4c8f02dddeb302b30244e33cc2c61fe01f4/76fbdf10f050749b2d84e4bb136cc4c8f02dddeb302b30244e33cc2c61fe01f4-json.log.2 /var/lib/nerdctl/1935db59/containers/nerdctl-test/76fbdf10f050749b2d84e4bb136cc4c8f02dddeb302b30244e33cc2c61fe01f4/76fbdf10f050749b2d84e4bb136cc4c8f02dddeb302b30244e33cc2c61fe01f4-json.log.22 /var/lib/nerdctl/1935db59/containers/nerdctl-test/76fbdf10f050749b2d84e4bb136cc4c8f02dddeb302b30244e33cc2c61fe01f4/76fbdf10f050749b2d84e4bb136cc4c8f02dddeb302b30244e33cc2c61fe01f4-json.log.3 /var/lib/nerdctl/1935db59/containers/nerdctl-test/76fbdf10f050749b2d84e4bb136cc4c8f02dddeb302b30244e33cc2c61fe01f4/76fbdf10f050749b2d84e4bb136cc4c8f02dddeb302b30244e33cc2c61fe01f4-json.log.4 /var/lib/nerdctl/1935db59/containers/nerdctl-test/76fbdf10f050749b2d84e4bb136cc4c8f02dddeb302b30244e33cc2c61fe01f4/76fbdf10f050749b2d84e4bb136cc4c8f02dddeb302b30244e33cc2c61fe01f4-json.log.5 /var/lib/nerdctl/1935db59/containers/nerdctl-test/76fbdf10f050749b2d84e4bb136cc4c8f02dddeb302b30244e33cc2c61fe01f4/76fbdf10f050749b2d84e4bb136cc4c8f02dddeb302b30244e33cc2c61fe01f4-json.log.6] --- FAIL: TestRunWithJsonFileLogDriver (3.35s)

https://github.com/containerd/nerdctl/runs/7314776208?check_suite_focus=true

AkihiroSuda avatar Jul 13 '22 07:07 AkihiroSuda

Looks like an issue with go-logrotate or the sleep time not sure :/ , I will take a look

fahedouch avatar Jul 13 '22 08:07 fahedouch

In this case.

=== RUN   TestRunWithJsonFileLogDriver
    run_test.go:237: the maximum number of old log files to retain exceeded 2 files, got: []
--- FAIL: TestRunWithJsonFileLogDriver (3.44s)

https://github.com/containerd/nerdctl/runs/7319449412?check_suite_focus=true

manugupt1 avatar Jul 14 '22 00:07 manugupt1

I got another failure: https://github.com/manugupt1/nerdctl/runs/7366257926?check_suite_focus=true#step:5:695

Test summary: https://github.com/manugupt1/nerdctl/actions/runs/2679791706

manugupt1 avatar Jul 15 '22 23:07 manugupt1

Doesn't seem to be fully fixed.

=== RUN   TestRunWithJsonFileLogDriver
    run_test.go:237: the maximum number of old log files to retain exceeded 2 files, got: []
--- FAIL: TestRunWithJsonFileLogDriver (3.36s)

https://github.com/containerd/nerdctl/runs/7367118724?check_suite_focus=true#step:7:702

junnplus avatar Jul 16 '22 01:07 junnplus

OK I am investigating... making log message more specific https://github.com/containerd/nerdctl/pull/1241/files

fahedouch avatar Jul 16 '22 14:07 fahedouch

This one seems frequent recently:

https://github.com/containerd/nerdctl/actions/runs/3343599257/jobs/5536985776

https://github.com/containerd/nerdctl/actions/runs/3438150819/jobs/5733839743

https://github.com/containerd/nerdctl/actions/runs/3019700424/jobs/4855888202

djdongjin avatar Nov 10 '22 22:11 djdongjin

  • https://github.com/containerd/nerdctl/pull/1650

AkihiroSuda avatar Dec 16 '22 18:12 AkihiroSuda