Skip to content

Conversation

pchaigno
Copy link
Member

Some of the unit tests may cause the installation of XFRM states and policies on the host. Those are deleted as part of the test if it succeeds. But if it fails, we need to clean them up in the unit test tear down. This pull request implements that cleanup.

Some of the unit tests may cause the installation of XFRM states and
policies on the host. Those are deleted as part of the test if it
succeeds. But if it fails, we need to clean them up in the unit test
tear down. This commit implements that cleanup.

Signed-off-by: Paul Chaignon <paul.chaignon@gmail.com>
@pchaigno pchaigno added area/CI Continuous Integration testing issue or flake area/encryption Impacts encryption support such as IPSec, WireGuard, or kTLS. release-note/ci This PR makes changes to the CI. labels Nov 22, 2023
@pchaigno
Copy link
Member Author

/ci-integration

@pchaigno
Copy link
Member Author

/ci-ginkgo

@pchaigno
Copy link
Member Author

/ci-runtime

@pchaigno pchaigno marked this pull request as ready for review November 22, 2023 15:56
@pchaigno pchaigno requested a review from a team as a code owner November 22, 2023 15:56
@pchaigno pchaigno merged commit afff5e9 into cilium:main Nov 23, 2023
@pchaigno pchaigno deleted the xfrm-unit-test-cleanup branch November 23, 2023 07:31
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 area/encryption Impacts encryption support such as IPSec, WireGuard, or kTLS. release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants