Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(action): Only run dockerd-rootless.sh if told #84

Merged
merged 1 commit into from
Jul 18, 2022
Merged

Conversation

Kurt-von-Laven
Copy link
Contributor

@Kurt-von-Laven Kurt-von-Laven commented Jul 16, 2022

GitHub Actions made an unannounced change impacting how we determine whether the rootless Docker install script started the Docker daemon. Previously, "API listen on $XDG_RUNTIME_DIR/docker.sock" was present in the script's output if the Docker daemon was started, but this is no longer guaranteed. Running dockerd-rootless.sh when the Docker daemon has already been started results in an error, so only run it when the install script outputs an explicit instruction to do so.

@Kurt-von-Laven Kurt-von-Laven self-assigned this Jul 16, 2022
GitHub Actions made an unannounced change impacting how we determine
whether the rootless Docker install script started the Docker daemon.
Previously, "API listen on $XDG_RUNTIME_DIR/docker.sock" was present in
the script's output if the Docker daemon was started, but this is no
longer guaranteed. Running dockerd-rootless.sh when the Docker daemon
has already been started results in an error, so only run it when the
install script outputs an explicit instruction to do so.
@Kurt-von-Laven Kurt-von-Laven changed the title fix(action): Install containerd.io preemptively fix(action): Start containerd service preemptively Jul 17, 2022
@Kurt-von-Laven Kurt-von-Laven changed the title fix(action): Start containerd service preemptively fix(action): Only run dockerd-rootless.sh if told Jul 17, 2022
Copy link
Contributor

@mwarres mwarres left a comment

Choose a reason for hiding this comment

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

LGTM!

@Kurt-von-Laven Kurt-von-Laven merged commit b401704 into main Jul 18, 2022
@Kurt-von-Laven Kurt-von-Laven deleted the action branch July 18, 2022 17:28
@Kurt-von-Laven Kurt-von-Laven removed the request for review from davisgtaylor July 18, 2022 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants