connectivity: Add local redirect policy tests #2568
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
aditighag
force-pushed
the
pr/aditighag/lrp-tests
branch
from
731a6c5 to
1d0736c
Compare
aditighag
force-pushed
the
pr/aditighag/lrp-tests
branch
from
1d0736c to
d32aa42
Compare
aditighag
force-pushed
the
pr/aditighag/lrp-tests
branch
from
d32aa42 to
a3d742a
Compare
aditighag
force-pushed
the
pr/aditighag/lrp-tests
branch
from
a3d742a to
c76e70d
Compare
aditighag
force-pushed
the
pr/aditighag/lrp-tests
branch
from
c76e70d to
1a7cc8b
Compare
aditighag
force-pushed
the
pr/aditighag/lrp-tests
branch
from
1a7cc8b to
99fa7a6
Compare
aditighag
force-pushed
the
pr/aditighag/lrp-tests
branch
from
99fa7a6 to
09a48ef
Compare
aditighag
force-pushed
the
pr/aditighag/lrp-tests
branch
from
09a48ef to
e1c9556
Compare
aditighag
force-pushed
the
pr/aditighag/lrp-tests
branch
from
e1c9556 to
d218b6c
Compare
christarazi
reviewed
Jun 5, 2024
christarazi
approved these changes
Jun 6, 2024
Extend the features list with a flag to enable local redirect policy. Signed-off-by: Aditi Ghag <[email protected]>
Signed-off-by: Aditi Ghag <[email protected]>
It checks for a failed curl command without any network policy drops. Used in follow-up commits to validate expected result for LRP tests. The results pertain to network policy as well non network policy related checks, so move them accordingly. Signed-off-by: Aditi Ghag <[email protected]>
Tests LRP connectivity scenarios with a configured skipRedirectFromBackend flag: - client pod to LRP frontend - LRP backend to LRP frontend Signed-off-by: Aditi Ghag <[email protected]>
aditighag
force-pushed
the
pr/aditighag/lrp-tests
branch
from
d218b6c to
63c6d3a
Compare
maintainer-s-little-helper
bot
added
the
ready-to-merge
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR introduces local redirect policy tests to the connectivity suite.
In order to break the circular dependency, there is a separate PR on cilium/cilium repo that enables these tests using a custom version of cilium-cli obtained from this PR changes. Here are successful test runs from the cilium/cilium PR CI runs -
Runs after addressing review comments -
https://github.com/cilium/cilium/actions/runs/9409627836/job/25920752091
https://github.com/cilium/cilium/actions/runs/9409627836/job/25919911738
https://github.com/cilium/cilium/actions/runs/9387516652/job/25850621644
https://github.com/cilium/cilium/actions/runs/9377202073/job/25819021276
https://github.com/cilium/cilium/actions/runs/9375976798/job/25815109405
https://github.com/cilium/cilium/actions/runs/9377202073/job/25818391762