Skip to content

scripts/ci-runner.sh: wait for systemd to settle#516

Merged
Luap99 merged 1 commit intocoreos:mainfrom
kolyshkin:ci-sleep
Mar 16, 2026
Merged

scripts/ci-runner.sh: wait for systemd to settle#516
Luap99 merged 1 commit intocoreos:mainfrom
kolyshkin:ci-sleep

Conversation

@kolyshkin
Copy link
Collaborator

The sleep stage, originally added by commit 07d38cb, is there to wait for systemd to finish its initialization stage (which includes wiping /tmp).

Commit 743c5ad added a step of building tests inside a container, and then commit 82dc44e moved the code into scripts/ci-runner.sh.

Now, sometimes building the tests fails with errors like this:

github.com/coreos/go-systemd/v22/activation.test: open /tmp/go-build4063755751/b001/importcfg: no such file or directory

or

asm: open $WORK/b006/symabis: no such file or directory

(as reported in #513 comments).

All this points to /tmp being wiped while the tests are being built.

Moving the sleep to before building tests (right after starting the container) should fix this. Yet better, let's use

systemctl is-system-running --wait

to wait for systemd to settle.

@kolyshkin kolyshkin changed the title scripts/ci-runner.sh: move sleep kludge up scripts/ci-runner.sh: wait for systemd to settle Mar 14, 2026
The sleep stage, originally added by commit 07d38cb, is there to wait
for systemd to finish its initialization stage (which includes wiping
/tmp).

Commit 743c5ad added a step of building tests inside a container,
and then commit 82dc44e moved the code into scripts/ci-runner.sh.

Now, sometimes building the tests fails with errors like this:

> github.com/coreos/go-systemd/v22/activation.test: open /tmp/go-build4063755751/b001/importcfg: no such file or directory

or

> asm: open $WORK/b006/symabis: no such file or directory

All this points to /tmp being wiped while the tests are being built.

Moving the sleep to before building tests (right after starting the
container) should fix this. Yet better, let's implement a proper wait
for systemd to settle.

While at it:
 - modify ci-runner.sh to accept multiple arguments;
 - remove GO15VENDOREXPERIMENT as it is no longer needed.

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
Copy link
Collaborator

@Luap99 Luap99 left a comment

Choose a reason for hiding this comment

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

LGTM

@Luap99 Luap99 merged commit 359a569 into coreos:main Mar 16, 2026
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants