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

wireguard: Fix traffic counters in cilium debuginfo #16178

Merged

Conversation

gandro
Copy link
Member

@gandro gandro commented May 17, 2021

Wireguard keeps track of the traffic sent and receive for each peer. We
print this information in cilium debuginfo. Because the agent failed
to populate them the JSON response, the counters were always wrongly
reported as zero.

Wireguard keeps track of the traffic sent and receive for each peer. We
print this information in `cilium debuginfo`. Because the agent failed
to populate them the JSON response, the counters were always wrongly
reported as zero.

Signed-off-by: Sebastian Wicki <sebastian@isovalent.com>
@gandro gandro added release-note/bug This PR fixes an issue in a previous release of Cilium. area/encryption Impacts encryption support such as IPSec, WireGuard, or kTLS. release-priority/best-effort The project for target version is not a hard requirement. needs-backport/1.10 labels May 17, 2021
@gandro gandro requested a review from a team as a code owner May 17, 2021 14:30
@gandro gandro requested a review from brb May 17, 2021 14:30
@gandro gandro added this to the 1.10.0 milestone May 17, 2021
@gandro
Copy link
Member Author

gandro commented May 17, 2021

test-me-please

@gandro
Copy link
Member Author

gandro commented May 18, 2021

GH actions seems stuck on the non-required performance tests. Marking this as ready to merge, as all required tests have passed. I have also manually tested this build on EKS.

@gandro gandro added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label May 18, 2021
@tklauser tklauser merged commit f3fed9a into cilium:master May 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/encryption Impacts encryption support such as IPSec, WireGuard, or kTLS. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/bug This PR fixes an issue in a previous release of Cilium. release-priority/best-effort The project for target version is not a hard requirement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants