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

NfTables chain does not get updated with latest KubeSpan peer when allowDownPeerBypass: true #8498

Closed
Tracked by #8481
utkuozdemir opened this issue Mar 27, 2024 · 0 comments · Fixed by #8538
Closed
Tracked by #8481
Assignees

Comments

@utkuozdemir
Copy link
Member

Bug Report

Description

When KubeSpan is enabled with allowDownPeerBypass: true, kubespan.ManagerController does not update the NfTablesChains resources to include the IPs of the newly joined peer.

It seems to be caused by the combination of:

Environment

  • Talos version: 1.6.4
  • Kubernetes version: 1.28.6
  • Platform: Metal
@smira smira self-assigned this Apr 1, 2024
smira added a commit to smira/talos that referenced this issue Apr 3, 2024
Fixes siderolabs#8498

Before KubeSpan was reimplemented to use resources for firewall rules,
the update was happening always, but it got moved to a wrong section of
the controller which gets executed on resource updates, but ignores
updates of the peer statuses.

Signed-off-by: Andrey Smirnov <andrey.smirnov@siderolabs.com>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 5, 2024
smira added a commit to smira/talos that referenced this issue Jul 22, 2024
Fixes siderolabs#8498

Before KubeSpan was reimplemented to use resources for firewall rules,
the update was happening always, but it got moved to a wrong section of
the controller which gets executed on resource updates, but ignores
updates of the peer statuses.

Signed-off-by: Andrey Smirnov <andrey.smirnov@siderolabs.com>
(cherry picked from commit e7d8041)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants