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

Revert "config: Fix incorrect packet path with IPsec and endpoint rou… #17057

Merged
merged 1 commit into from Aug 5, 2021

Conversation

aanm
Copy link
Member

@aanm aanm commented Aug 4, 2021

…tes"

This reverts commit 7ef59aa.

After performing a git bisect where connectivity test had to pass 3
times consecutively this commit was the first that hit the current
ci-aks failures that we have been having in our CI, thus it is being
reverted.

Signed-off-by: André Martins andre@cilium.io

Fixes #17022

cc @pchaigno

…tes"

This reverts commit 7ef59aa.

After performing a git bisect where connectivity test had to pass 3
times consecutively this commit was the first that hit the current
ci-aks failures that we have been having in our CI, thus it is being
reverted.

Signed-off-by: André Martins <andre@cilium.io>
@aanm aanm added the release-note/misc This PR makes changes that have no direct user impact. label Aug 4, 2021
@aanm aanm requested a review from a team as a code owner August 4, 2021 23:51
@aanm aanm requested a review from ti-mo August 4, 2021 23:51
@aanm
Copy link
Member Author

aanm commented Aug 4, 2021

ci-aks

@aanm
Copy link
Member Author

aanm commented Aug 5, 2021

ci-aks has passed https://github.com/cilium/cilium/runs/3247037335

@aanm
Copy link
Member Author

aanm commented Aug 5, 2021

test-me-please

Job 'Cilium-PR-K8s-1.16-net-next' failed and has not been observed before, so may be related to your PR:

Click to show.

Test Name

K8sServicesTest Checks service across nodes Tests NodePort BPF Tests with vxlan Test NodePort with netfilterCompatMode=true

Failure Output

FAIL: Can not connect to service "http://192.168.36.11:32621" from outside cluster (2/10)

If it is a flake, comment /mlh new-flake Cilium-PR-K8s-1.16-net-next so I can create a new GitHub issue to track it.

@aanm
Copy link
Member Author

aanm commented Aug 5, 2021

/mlh new-flake Cilium-PR-K8s-1.16-net-next

👍 created #17060

@aanm aanm requested review from a team and jibi and removed request for a team August 5, 2021 12:54
@aanm aanm removed the request for review from jibi August 5, 2021 12:54
@aanm aanm unassigned jibi Aug 5, 2021
@aanm aanm requested review from a team and joamaki and removed request for a team August 5, 2021 12:55
@aanm aanm removed the request for review from joamaki August 5, 2021 12:55
@aanm aanm unassigned joamaki Aug 5, 2021
@aanm aanm requested review from a team and nathanjsweet and removed request for a team August 5, 2021 13:15
@aanm
Copy link
Member Author

aanm commented Aug 5, 2021

Merging to unblock other PRs that are blocked by #17022 and opened #17060 to track the current flake.

@aanm aanm merged commit 346713b into cilium:master Aug 5, 2021
@aanm aanm deleted the pr/fix-aks branch August 5, 2021 13:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI: ConformanceAKS many[no-policies] test commands fail (packets disappear) with ipsec encryption
5 participants