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

bpf/tests: fix redundant usage of variable offset #22390

Merged
merged 1 commit into from
Nov 30, 2022

Conversation

sahid
Copy link
Member

@sahid sahid commented Nov 28, 2022

Fix typo when initializing offset

Signed-off-by: Sahid Orentino Ferdjaoui <sahid.ferdjaoui@industrialdiscipline.com>
@sahid sahid requested a review from a team as a code owner November 28, 2022 10:34
@sahid sahid requested a review from aspsk November 28, 2022 10:34
@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 Nov 28, 2022
@aspsk aspsk removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Nov 28, 2022
@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 Nov 28, 2022
@aspsk aspsk added release-note/minor This PR changes functionality that users may find relevant to operating Cilium. and removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Nov 28, 2022
@joestringer
Copy link
Member

@cilium/sig-datapath / @aspsk which github workflow triggers this test code? Can you check whether that workflow ran successfully or trigger it if it did not yet run?

@YutaroHayakawa
Copy link
Member

I think it's BPF checks. It already worked and passed.

https://github.com/cilium/cilium/actions/runs/3563921555/jobs/5987609089

@pchaigno pchaigno added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Nov 30, 2022
@jrajahalme jrajahalme merged commit 5317243 into cilium:master Nov 30, 2022
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/minor This PR changes functionality that users may find relevant to operating Cilium.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants