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.

Sign up for GitHub

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

Jump to bottom

Re-enable tapv2 #270

Closed
edwarnicke opened this issue Jun 16, 2021 · 2 comments
Closed

Re-enable tapv2 #270

edwarnicke opened this issue Jun 16, 2021 · 2 comments
Assignees
@glazychev-art
Labels
bug Something isn't working

Comments

@edwarnicke
Copy link
Member

edwarnicke commented Jun 16, 2021 •
edited
Loading

tapv2 was disabled to work around a vpp bug. Needs to be re-enabled once that bug is fixed.
@denis-tingaikin denis-tingaikin added the bug Something isn't working label Jun 16, 2021
@glazychev-art glazychev-art mentioned this issue Jul 9, 2021
Merged
@glazychev-art
Copy link
Contributor

After reverting PR that disabled tapv2, I found, that we have unstable cmd-forwarder-vpp docker tests TestForwarderTestSuite/TestCombinations/Ethernet/*.
Output:

--- FAIL: TestForwarderTestSuite (12.91s)
    --- FAIL: TestForwarderTestSuite/TestCombinations (11.73s)
        --- FAIL: TestForwarderTestSuite/TestCombinations/Ethernet (11.73s)
            --- FAIL: TestForwarderTestSuite/TestCombinations/Ethernet/Kernel (11.73s)
                --- PASS: TestForwarderTestSuite/TestCombinations/Ethernet/Kernel/Kernel (0.45s)
                --- FAIL: TestForwarderTestSuite/TestCombinations/Ethernet/Kernel/Memif (10.63s)
                --- PASS: TestForwarderTestSuite/TestCombinations/Ethernet/Kernel/Vxlan (0.65s)
    --- PASS: TestForwarderTestSuite/TestHealthCheck (0.00s)
FAIL

Tested with and without a new l3xc patch.

@glazychev-art glazychev-art mentioned this issue Jul 12, 2021
Merged
@glazychev-art
Copy link
Contributor

Found that the instability of these tests is not related to tapv2.
Created an issue: networkservicemesh/cmd-forwarder-vpp#274

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@glazychev-art glazychev-art

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@edwarnicke @glazychev-art @denis-tingaikin