This repository was archived by the owner on Oct 26, 2023. It is now read-only.
Merged
Conversation
… cgroup This page gives epxlains how to revert the change locally, explains what happens during an update, and how to migrate older nodes. Also add a brief discussion of relationships with Docker and Kubernetes versions and configuration. Signed-off-by: Jeremi Piotrowski <jpiotrowski@microsoft.com>
Merged
7 tasks
Contributor
|
Looks good, maybe we also cover containerd (https://kubernetes.io/docs/setup/production-environment/container-runtimes/#containerd-systemd) and cri-o (https://kubernetes.io/docs/setup/production-environment/container-runtimes/#cgroup-driver) but I haven't tried it yet to verify |
Document the findings around container runtimes. The default `dockershim` does not require special containerd configuration, when using `containerd` directly some changes to config.toml are needed.
pothos
reviewed
Aug 6, 2021
pothos
reviewed
Aug 6, 2021
pothos
reviewed
Aug 6, 2021
Add note that config snippets are CLC YAML and need to be transpiled and that changing the cgroup OEM grub.cfg has no effect on the first boot and requires a reboot to become active. Co-authored-by: Kai Lüke <pothos@users.noreply.github.com>
We added freezing of containerd config during post migration using a drop-in unit, so add that to the revert description. A community member asked for a description of how to migrate nodes to cgroupsv2 so add that too. Also extended the kubernetes doc with a kubeadm config snippet and some more context for the containerd config lines.
and other minor corrections to text.
pothos
reviewed
Aug 12, 2021
pothos
reviewed
Aug 12, 2021
…emd_cgroup_controller' with sed Co-authored-by: Kai Lüke <pothos@users.noreply.github.com>
pothos
reviewed
Aug 12, 2021
jepio
commented
Aug 12, 2021
Found one last issue during testing and added the suggested rewording of a sentence around
containerd configuration ("if you use your own configuration...").
Co-authored-by: Kai Lüke <pothos@users.noreply.github.com>
pothos
approved these changes
Aug 12, 2021
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add a page under
container runtimesthat explains the details of the cgroupv2 migration and impact.