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): Stop rootful Docker daemon #81

Merged
merged 1 commit into from
Jul 13, 2022
Merged

fix(action): Stop rootful Docker daemon #81

merged 1 commit into from
Jul 13, 2022

Conversation

Kurt-von-Laven
Copy link
Contributor

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

GitHub Actions started managing the Docker daemon via systemd unannounced. That caused this action to fail when trying to start the rootless Docker daemon, because the rootful Docker daemon is already running. Stop the rootful Docker daemon before attempting to start the rootless Docker daemon. When Docker is managed via systemd, there is no need to run dockerd-rootless.sh separately since the rootless Docker install script starts the rootless Docker daemon automatically.

@Kurt-von-Laven Kurt-von-Laven self-assigned this Jul 13, 2022
@Kurt-von-Laven Kurt-von-Laven changed the title fix(action): Disable rootful Docker daemon fix(action): Stop rootfull Docker daemon Jul 13, 2022
GitHub Actions started managing the Docker daemon via systemd
unannounced. That caused this action to fail when trying to start the
rootless Docker daemon, because the rootful Docker daemon is already
running. Stop the rootful Docker daemon before attempting to start the
rootless Docker daemon. When Docker is managed via systemd, there is no
need to run dockerd-rootless.sh separately since the rootless Docker
install script starts the rootless Docker daemon automatically.
@Kurt-von-Laven Kurt-von-Laven changed the title fix(action): Stop rootfull Docker daemon fix(action): Stop rootful Docker daemon Jul 13, 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.

Looks good as far as I can tell

@Kurt-von-Laven Kurt-von-Laven removed the request for review from davisgtaylor July 13, 2022 18:01
@Kurt-von-Laven Kurt-von-Laven merged commit 3a02565 into main Jul 13, 2022
@Kurt-von-Laven Kurt-von-Laven deleted the action branch July 13, 2022 18:01
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.

None yet

2 participants