Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@dylandreimerink
Copy link
Member

The table wasn't assigned to the reconciler config for the bandwidth map this causes an error on startup when bandwidth manager is enabled. This commit should resolve the issue.

Fixes: #32909

Fix bandwidth manager reconciler config

The table wasn't assigned to the reconciler config for the bandwidth map
this causes an error on startup when bandwidth manager is enabled.
This commit should resolve the issue.

Signed-off-by: Dylan Reimerink <[email protected]>
@dylandreimerink dylandreimerink added the release-note/bug This PR fixes an issue in a previous release of Cilium. label Jun 7, 2024
@dylandreimerink dylandreimerink requested a review from a team as a code owner June 7, 2024 09:25
@dylandreimerink dylandreimerink requested a review from jibi June 7, 2024 09:25
@julianwiedmann
Copy link
Member

/test

@julianwiedmann julianwiedmann added feature/bandwidth-manager Impacts BPF bandwidth manager. kind/bug This is a bug in the Cilium logic. release-note/misc This PR makes changes that have no direct user impact. area/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages. and removed release-note/bug This PR fixes an issue in a previous release of Cilium. labels Jun 10, 2024
@julianwiedmann
Copy link
Member

(I changed the release-note label, as afaict the breakage didn't make a release yet)

@dylandreimerink
Copy link
Member Author

Right, it was technically caught in a pre-release

@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Jun 10, 2024
@dylandreimerink dylandreimerink added this pull request to the merge queue Jun 10, 2024
Merged via the queue into cilium:main with commit 8b1f64a Jun 10, 2024
@dylandreimerink dylandreimerink deleted the feature/fix-32909 branch June 10, 2024 12:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages. feature/bandwidth-manager Impacts BPF bandwidth manager. kind/bug This is a bug in the Cilium logic. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fatal error in cilium-agent on 1.16.0-pre.3 with bandwidthManager + BBR

3 participants