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

CI: Test/EgressGatewayTestSuite/TestEgressGatewayManager fails in Runtime Test (privileged) job #27431

Closed
tklauser opened this issue Aug 10, 2023 · 1 comment · Fixed by #27432
Closed

CI: Test/EgressGatewayTestSuite/TestEgressGatewayManager fails in Runtime Test (privileged) job #27431

tklauser opened this issue Aug 10, 2023 · 1 comment · Fixed by #27432
Assignees
@julianwiedmann
Labels
area/CI Continuous Integration testing issue or flake ci/flake This is a known failure that occurs in the tree. Please investigate me! feature/egress-gateway Impacts the egress IP gateway feature.

Comments

@tklauser
Copy link
Member 

level=error msg="Conflict with policy /policy-1: Selects the same egress interface but uses a different egress IP (192.168.101.1/24)." ciliumEgressGatewayPolicyName=/policy-3 egressIP=192.168.103.1 interface= subsys=egressgateway
level=error msg="Conflict with policy /policy-3: Selects the same egress interface but uses a different egress IP (192.168.103.1/24)." ciliumEgressGatewayPolicyName=/policy-1 egressIP="<nil>" interface=cilium_egw1 subsys=egressgateway
--- FAIL: Test (0.18s)
    --- FAIL: Test/EgressGatewayTestSuite (0.18s)
        --- FAIL: Test/EgressGatewayTestSuite/TestEgressGatewayManager (0.18s)
            manager_privileged_test.go:712: 
                ... value *errors.errorString = &errors.errorString{s:"mismatched gateway IP"} ("mismatched gateway IP")
                
FAIL
	github.com/cilium/cilium/pkg/egressgateway	coverage: 74.4% of statements
FAIL	github.com/cilium/cilium/pkg/egressgateway	0.216s

https://github.com/cilium/cilium/actions/runs/5822370297/job/15786887583

Seen on #27427.

I looks like this started happening after #27281 /cc @julianwiedmann
@tklauser tklauser added area/CI Continuous Integration testing issue or flake ci/flake This is a known failure that occurs in the tree. Please investigate me! feature/egress-gateway Impacts the egress IP gateway feature. labels Aug 10, 2023
@tklauser tklauser mentioned this issue Aug 10, 2023
Merged
@julianwiedmann
Copy link
Member

😬 yep, I can guess what it is. Sorry. Will back the test out until we found something more robust.

julianwiedmann added a commit to julianwiedmann/cilium that referenced this issue Aug 10, 2023
@julianwiedmann
egressgw: back out test for policy conflict in ENI mode
472dcca 
cilium#27431 points out that we can't
really tell which policy is going to be processed first (and wins the
conflict). Revert the test until we have a mechanism to do so.

Signed-off-by: Julian Wiedmann <jwi@isovalent.com>
@julianwiedmann julianwiedmann mentioned this issue Aug 10, 2023
Merged
@julianwiedmann julianwiedmann linked a pull request Aug 10, 2023 that will close this issue
egressgw: back out test for policy conflict in ENI mode #27432
Merged
@julianwiedmann julianwiedmann self-assigned this Aug 10, 2023
@asauber asauber mentioned this issue Aug 11, 2023
Merged
ldelossa pushed a commit that referenced this issue Aug 11, 2023
@julianwiedmann @ldelossa
egressgw: back out test for policy conflict in ENI mode
92481c4 
#27431 points out that we can't
really tell which policy is going to be processed first (and wins the
conflict). Revert the test until we have a mechanism to do so.

Signed-off-by: Julian Wiedmann <jwi@isovalent.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@julianwiedmann julianwiedmann

Labels
area/CI Continuous Integration testing issue or flake ci/flake This is a known failure that occurs in the tree. Please investigate me! feature/egress-gateway Impacts the egress IP gateway feature.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

egressgw: back out test for policy conflict in ENI mode julianwiedmann/cilium
2 participants
@tklauser @julianwiedmann