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

CI: L4LB: warning: unused variable 'buf' #17576

Closed
pchaigno opened this issue Oct 11, 2021 · 1 comment · Fixed by #17683
Closed

CI: L4LB: warning: unused variable 'buf' #17576

pchaigno opened this issue Oct 11, 2021 · 1 comment · Fixed by #17683
Assignees
Labels
area/CI Continuous Integration testing issue or flake good-first-issue Good starting point for new developers, which requires minimal understanding of Cilium. sig/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages.

Comments

@pchaigno
Copy link
Member

Everytime the L4LB XDP workflow runs, the following warning is emitted during compilation:

Run ssh default "sudo /bin/sh -c 'cd /vagrant/test/l4lb && ./test.sh cilium b49ab9592821760c2cb8d1ba49b3f90a7a84de9f'"
+ IMG_OWNER=cilium
+ IMG_TAG=b49ab9592821760c2cb8d1ba49b3f90a7a84de9f
+ clang -O2 -Wall -target bpf -c bpf_xdp_veth_host.c -o bpf_xdp_veth_host.o
+ clang -O2 -Wall -target bpf -c test_tc_tunnel.c -o test_tc_tunnel.o
test_tc_tunnel.c:456:7: warning: unused variable 'buf' [-Wunused-variable]
        char buf[sizeof(struct v6hdr)];
             ^
1 warning generated.
@pchaigno pchaigno added area/CI Continuous Integration testing issue or flake good-first-issue Good starting point for new developers, which requires minimal understanding of Cilium. sig/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages. labels Oct 11, 2021
@h3llix
Copy link
Contributor

h3llix commented Oct 18, 2021

hi @pchaigno. Can you assign this to me ?

h3llix added a commit to h3llix/cilium that referenced this issue Oct 24, 2021
Fixes: cilium#17576
Signed-off-by: Gaurav Genani <h3llix.pvt@gmail.com>
kkourt pushed a commit that referenced this issue Oct 25, 2021
Fixes: #17576
Signed-off-by: Gaurav Genani <h3llix.pvt@gmail.com>
vincentmli pushed a commit to f5devcentral/cilium that referenced this issue Oct 25, 2021
Fixes: cilium#17576
Signed-off-by: Gaurav Genani <h3llix.pvt@gmail.com>
michi-covalent pushed a commit that referenced this issue Oct 27, 2021
Fixes: #17576
Signed-off-by: Gaurav Genani <h3llix.pvt@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/CI Continuous Integration testing issue or flake good-first-issue Good starting point for new developers, which requires minimal understanding of Cilium. sig/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants