Skip to content

ci: fix test logs on linux#4892

Open
ningmingxiao wants to merge 1 commit into
containerd:mainfrom
ningmingxiao:fix_log
Open

ci: fix test logs on linux#4892
ningmingxiao wants to merge 1 commit into
containerd:mainfrom
ningmingxiao:fix_log

Conversation

@ningmingxiao
Copy link
Copy Markdown
Contributor

@ningmingxiao ningmingxiao commented May 12, 2026

call "nerdctl logs --untils " time is 16:03:19

2026-05-11T16:09:34.1135624Z 2026/05/11 16:03:19  binary is /usr/local/bin/nerdctl arg is [logs --until 60s testlogs001] 

but json logs time is 16:01:40.459948268Z

logs --until 60s testlogs001]*******content is:{"log":"+ echo foo\n","stream":"stderr","time":"2026-05-11T16:01:40.459948277Z"}
2026-05-11T16:09:34.1137555Z {"log":"+ echo bar\n","stream":"stderr","time":"2026-05-11T16:01:40.461456573Z"}
2026-05-11T16:09:34.1137691Z {"log":"foo\n","stream":"stdout","time":"2026-05-11T16:01:40.459948268Z"}

the root reason should be the number exceeds the maximum concurrent test case limit, and some test cases will be queued .
The default concurrency of t.Parallel() is the number of CPU cores.
so we can't run many testcase at a same time in test logs because it is Time-critical testcase.
fix #4889
@AkihiroSuda

@ningmingxiao ningmingxiao force-pushed the fix_log branch 4 times, most recently from f2f1e6f to 21922ee Compare May 12, 2026 06:22
@ningmingxiao ningmingxiao changed the title ci: fix test logs ci: fix test logs on linux May 12, 2026
@ningmingxiao ningmingxiao force-pushed the fix_log branch 2 times, most recently from deb7ce4 to f2f1e6f Compare May 12, 2026 11:24
@ningmingxiao
Copy link
Copy Markdown
Contributor Author

After hundreds of tests, it works perfectly.

Comment thread cmd/nerdctl/container/container_logs_test.go Outdated
Signed-off-by: ningmingxiao <ning.mingxiao@zte.com.cn>
@ningmingxiao
Copy link
Copy Markdown
Contributor Author

done thanks @sathiraumesh

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

test logs failed

2 participants