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

Update DENYLIST #154

Merged
merged 2 commits into from
Nov 1, 2022
Merged

Update DENYLIST #154

merged 2 commits into from
Nov 1, 2022

Conversation

chantra
Copy link
Contributor

@chantra chantra commented Nov 1, 2022

Add netcnt into ci/vmtest/configs/DENYLIST.x86_64 so it passes on kvm-enabled vm and copy ci/vmtest/configs/DENYLIST.aarch64 from bpf-next.

When ran under kvm, the test is failing with
`packets unexpected packets: actual 10001 != expected 10000`

this does not seem to reproduce under full qemu emulation.

In order to be able to introduce runners with kvm enabled, adding this
test to
the denylist will help keeping CI green while this is being
troubleshooted.

Signed-off-by: Manu Bretelle <chantr4@gmail.com>
Copy link
Contributor

@danielocfb danielocfb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use a .diff for the bpf-next commit instead of importing the raw denylist as discussed offline. Other than that the changes look fine to me. Should we open an issue for the netcnt investigation?

…ch64)

Until this makes it in bpf tree, we need to keep this DENYLIST to avoid
failing aarch64 on bpf tree.

Signed-off-by: Manu Bretelle <chantr4@gmail.com>
@chantra
Copy link
Contributor Author

chantra commented Nov 1, 2022

Should we open an issue for the netcnt investigation?

yes, I can create an issueon GH for visibility.

@chantra chantra merged commit 81886e1 into kernel-patches:master Nov 1, 2022
@chantra chantra deleted the denylist_netcnt branch November 1, 2022 21:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants