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

Enhance k3s check-config #7091

Merged
merged 3 commits into from
Mar 29, 2023
Merged

Enhance k3s check-config #7091

merged 3 commits into from
Mar 29, 2023

Conversation

dereknola
Copy link
Member

@dereknola dereknola commented Mar 14, 2023

Proposed Changes

  • Reports the embedded iptables shipped with K3s if no Iptables exists on the host
  • Checks correct embedded iptables links
  • Moves CGROUP_PIDS to from optional to required

Types of Changes

Verification

  • Remove Iptables from a host
  • Install K3s
  • Run k3s check-config, validate that something like:
System:
- /var/lib/rancher/k3s/data/e41fb3fc61cae853bd497dcbe628059f2fd8f196745962a7bec3dec999149452/bin/aux iptables v1.8.8 (legacy): ok

Should shows up.

  • CONFIG_CGROUP_PIDS is now under Generally Necessary:

Testing

Linked Issues

User-Facing Change


Further Comments

Signed-off-by: Derek Nola <derek.nola@suse.com>
Signed-off-by: Derek Nola <derek.nola@suse.com>
@dereknola dereknola marked this pull request as ready for review March 14, 2023 19:36
@dereknola dereknola requested a review from a team as a code owner March 14, 2023 19:36
Signed-off-by: Derek Nola <derek.nola@suse.com>
@dereknola dereknola merged commit d13ee64 into k3s-io:master Mar 29, 2023
dereknola added a commit to dereknola/k3s that referenced this pull request Mar 29, 2023
* Move  CONFIG_CGROUP_PIDS to Required

Signed-off-by: Derek Nola <derek.nola@suse.com>
dereknola added a commit to dereknola/k3s that referenced this pull request Mar 29, 2023
* Move  CONFIG_CGROUP_PIDS to Required

Signed-off-by: Derek Nola <derek.nola@suse.com>
dereknola added a commit that referenced this pull request Mar 29, 2023
* Add missing kernel config checks (#6946)

Add additional kernel config checks for NETFILTER_XT_MATCH_COMMENT and
NETFILTER_XT_MATCH_MULTIPORT as they are both required to run k3s.

Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>

* Enhance `k3s check-config` (#7091)

* Move  CONFIG_CGROUP_PIDS to Required

Signed-off-by: Derek Nola <derek.nola@suse.com>

---------

Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
Signed-off-by: Derek Nola <derek.nola@suse.com>
Co-authored-by: Richard Steinmetz <richard@steinmetz.cloud>
dereknola added a commit that referenced this pull request Mar 29, 2023
* Add missing kernel config checks (#6946)

Add additional kernel config checks for NETFILTER_XT_MATCH_COMMENT and
NETFILTER_XT_MATCH_MULTIPORT as they are both required to run k3s.

Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>

* Enhance `k3s check-config` (#7091)

* Move  CONFIG_CGROUP_PIDS to Required

Signed-off-by: Derek Nola <derek.nola@suse.com>

---------

Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
Signed-off-by: Derek Nola <derek.nola@suse.com>
Co-authored-by: Richard Steinmetz <richard@steinmetz.cloud>
@dereknola dereknola deleted the check_config branch May 2, 2023 18:44
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.

3 participants