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

Add kernel module xt_statistic #543

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add kernel module xt_statistic #543

wants to merge 1 commit into from

Conversation

LoeFe
Copy link

@LoeFe LoeFe commented Apr 22, 2024

When a k3s-agent is installed to the recent image and a hazelcast pod is deployed to the iot-2050-node, ip-tables can not be modified by k3s-agent since XT_STATISTIC Kernel module is missing.
This behavior has only been observed jet in the described setup and can be avoided by deleting and recreating the hazelcast pod.

@LoeFe LoeFe force-pushed the master branch 5 times, most recently from d869441 to 824f163 Compare April 23, 2024 14:35
@jan-kiszka
Copy link
Collaborator

Thanks for change - and sorry for the late reaction. Looks good - except for the from/signed-off-by line in your commit (real email address, please).

When a k3s-agent is installed to the recent image and a hazelcast pod is deployed to the iot-2050-node, ip-tables can not be modified by k3s-agent since XT_STATISTIC Kernel module is missing.
This behavior has only been observed jet in the described setup and can be avoided by deleting and recreating the hazelcast pod.

Signed-off-by: Felix Loesch <felix.loesch@siemens.com>
@LoeFe
Copy link
Author

LoeFe commented Jul 30, 2024

Sorry back for my late reply. Should be fixed now @jan-kiszka

@huaqianli
Copy link
Collaborator

Hi @LoeFe ,

Sorry for the late reply. Everything looks good, but could you please improve one thing? Wrap the commit message body to 72 characters or less.

Thanks.

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