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

Remove unused variable in test_tc_tunnel.c #17683

Merged
merged 1 commit into from Oct 25, 2021
Merged

Conversation

h3llix
Copy link
Contributor

@h3llix h3llix commented Oct 24, 2021

Fixes: #17576
Signed-off-by: Gaurav Genani h3llix.pvt@gmail.com

Fixes: cilium#17576
Signed-off-by: Gaurav Genani <h3llix.pvt@gmail.com>
@h3llix h3llix requested a review from a team as a code owner October 24, 2021 22:13
@h3llix h3llix requested review from a team, nebril and jibi October 24, 2021 22:13
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Oct 24, 2021
@pchaigno pchaigno added the release-note/misc This PR makes changes that have no direct user impact. label Oct 25, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Oct 25, 2021
@pchaigno
Copy link
Member

ci-l4lb

@pchaigno
Copy link
Member

The only end-to-end test covering this, L4LB, passed. Changes are trivial and not specific to cilium/loadbalancer, so marking ready to merge.

@pchaigno pchaigno added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Oct 25, 2021
@kkourt kkourt merged commit ba4e73d into cilium:master Oct 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

CI: L4LB: warning: unused variable 'buf'
5 participants