Skip to content

fix: Add retry for systemd to startup before running exec#4778

Open
Shubhranshu153 wants to merge 1 commit intocontainerd:mainfrom
Shubhranshu153:fix-systemd-test
Open

fix: Add retry for systemd to startup before running exec#4778
Shubhranshu153 wants to merge 1 commit intocontainerd:mainfrom
Shubhranshu153:fix-systemd-test

Conversation

@Shubhranshu153
Copy link
Contributor

@Shubhranshu153 Shubhranshu153 commented Mar 6, 2026

Fixes: #4746

Added a retry logic to allow sometime for systemd to start.

@Shubhranshu153 Shubhranshu153 changed the title fix: Add delay for systemd to startup before running exec fix: Add retry for systemd to startup before running exec Mar 6, 2026
@Shubhranshu153 Shubhranshu153 marked this pull request as ready for review March 6, 2026 21:13
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, but still flaky
https://github.com/containerd/nerdctl/actions/runs/22777960578/job/66076770568?pr=4778

=== Failing tests ===
TestLogs
TestLogs/since_60s
TestLogs/until_60s
TestRunWithSystemdTrueEnabled

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hmm, for logs i will have separate one.
TestRunWithSystemdTrueEnabled -> this one i will look more thoroughly. Locally its hard to reproduce.

@Shubhranshu153
Copy link
Contributor Author

The retry logic itself is working. Increased the time between retries to check if its just a slow process.

@Shubhranshu153 Shubhranshu153 reopened this Mar 9, 2026
@Shubhranshu153 Shubhranshu153 force-pushed the fix-systemd-test branch 2 times, most recently from fdb76c7 to 2b234de Compare March 9, 2026 23:50
Signed-off-by: Shubhranshu Mahapatra <shubhum@amazon.com>
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.

Flaky test: TestRunWithSystemdTrueEnabled

2 participants