Skip to content

Conversation

SpaghettiBorgar
Copy link

@SpaghettiBorgar SpaghettiBorgar commented Sep 28, 2025

Contribution Guidelines

What does this PR include?

Short Description

A quick fix in ipv6_controller.sh to make sure /etc/docker exists before trying to create daemon.json as it is not guaranteed to exist by default.

Affected Containers

None

Did you run tests?

What did you test?

Running update.sh with non-existing /etc/docker directory and agreeing to create a new configuration file.

What were the final results? (Awaited, got)

Successfully created the config.

@SpaghettiBorgar SpaghettiBorgar force-pushed the fix/ipv6-controller-docker-dir branch from 84a19cd to 09c7d23 Compare September 28, 2025 17:44
Copy link
Member

@patschi patschi left a comment

Choose a reason for hiding this comment

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

LGTM.

dirname seems to be part of coreutils, which is already a required dependency as per our docs.

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.

2 participants