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

[Feature] cgroupv2fix: enable by default and move to nodestart #603

Merged
merged 1 commit into from
May 12, 2021

Conversation

iwilltry42
Copy link
Member

This PR enables the custom not-so-experimental-anymore entrypoint script for servers/agents if k3d detects that the runtime is on CgroupsV2.
This can be disabled by setting the environment variable K3D_FIX_CGROUPV2=false.
Additionally, the fix is now active also for k3d node create, not only for k3d cluster create.

Fixes #599

@iwilltry42 iwilltry42 added the enhancement New feature or request label May 12, 2021
@iwilltry42 iwilltry42 added this to the v4.4.4 milestone May 12, 2021
@iwilltry42 iwilltry42 self-assigned this May 12, 2021
@iwilltry42 iwilltry42 merged commit 770aaa4 into main May 12, 2021
@iwilltry42 iwilltry42 deleted the feature/cgroup2-fix-auto branch May 12, 2021 08:19
rancherio-gh-m pushed a commit that referenced this pull request May 12, 2021
Author: Thorsten Klein <[email protected]>
Date:   Wed May 12 10:19:15 2021 +0200

    [Feature] cgroupv2fix: enable by default and move to nodestart (#603)
@jlian
Copy link

jlian commented Jun 30, 2023

Why is this the default? Isn't this fixed in k3s upstream with k3s-io/k3s#4086?

Reason: to get NFS to work with k3d we have to use both a custom k3s image as well as a custom entrypoint (#1109 (comment)). The custom entrypoint cannot be applied because K3D_FIX_CGROUPV2 overrides it. Maybe you have a better way?

@iwilltry42
Copy link
Member Author

@jlian , the reason is in the comment of the issue you linked: k3s-io/k3s#4086 (comment)
... but we need the init process as per #129

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE] Make CgroupV2 workaround automatic and add it for node create
2 participants