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

loader: rewrite tests to remove gocheck dependency #31841

Merged
merged 4 commits into from
Apr 10, 2024

Conversation

lmb
Copy link
Contributor

@lmb lmb commented Apr 8, 2024

Remove the gocheck dependency from loader test so that I can run them like a sane person. I had to add creating the cilium_net device in tests to make this pass on my laptop.

loader: convert TestWrap

Signed-off-by: Lorenz Bauer <lmb@isovalent.com>

loader: convert TestHashDatapath

Signed-off-by: Lorenz Bauer <lmb@isovalent.com>

loader: rewrite cache tests

Signed-off-by: Lorenz Bauer <lmb@isovalent.com>

loader: rewrite tests

Signed-off-by: Lorenz Bauer <lmb@isovalent.com>

@lmb lmb added sig/loader Impacts the loading of BPF programs into the kernel. release-note/misc This PR makes changes that have no direct user impact. labels Apr 8, 2024
@lmb
Copy link
Contributor Author

lmb commented Apr 8, 2024

/test

@lmb lmb marked this pull request as ready for review April 9, 2024 09:41
@lmb lmb requested review from a team as code owners April 9, 2024 09:41
@lmb lmb requested review from tklauser and rgo3 April 9, 2024 09:41
pkg/datapath/loader/loader_test.go Outdated Show resolved Hide resolved
pkg/datapath/loader/loader_test.go Show resolved Hide resolved
pkg/datapath/loader/loader_test.go Show resolved Hide resolved
@maintainer-s-little-helper maintainer-s-little-helper bot added ready-to-merge This PR has passed all tests and received consensus from code owners to merge. labels Apr 9, 2024
@lmb lmb force-pushed the pr/lmb/loader-remove-checkmate branch from cae9b75 to 885ae67 Compare April 10, 2024 09:04
@lmb
Copy link
Contributor Author

lmb commented Apr 10, 2024

/test

lmb added 4 commits April 10, 2024 13:24
Signed-off-by: Lorenz Bauer <lmb@isovalent.com>
Signed-off-by: Lorenz Bauer <lmb@isovalent.com>
Signed-off-by: Lorenz Bauer <lmb@isovalent.com>
Signed-off-by: Lorenz Bauer <lmb@isovalent.com>
@lmb lmb force-pushed the pr/lmb/loader-remove-checkmate branch from 885ae67 to e55fcfe Compare April 10, 2024 11:24
@lmb
Copy link
Contributor Author

lmb commented Apr 10, 2024

/test

@lmb lmb enabled auto-merge April 10, 2024 11:45
@lmb lmb added this pull request to the merge queue Apr 10, 2024
Merged via the queue into cilium:main with commit fb0451c Apr 10, 2024
60 of 61 checks passed
@lmb lmb deleted the pr/lmb/loader-remove-checkmate branch April 10, 2024 14:06
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. sig/loader Impacts the loading of BPF programs into the kernel.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants