Skip to content

Conversation

julianwiedmann
Copy link
Member

No description provided.

The test setup expects that k8s2 is selected as gateway node. But we
currently derive the EgressIP based off k8s1.

As long as k8s1 and k8s2 have similar IPs, the result is the same (as we
simply calculate the EgressIP as x.x.x.100). Nevertheless clean up the
confusion.

Signed-off-by: Julian Wiedmann <jwi@isovalent.com>
Don't hardcode the Gateway's ci-node label in various places, and let the
test manage it instead.

Signed-off-by: Julian Wiedmann <jwi@isovalent.com>
Assert that applying a policy will not disturb established connections that
fall in scope of the policy.

This currently is an issue for pods on non-gateway nodes. Their
connections will typically exit on their local node; but once the policy is
applied, the from-container program in bpf_lxc starts to redirect them over
to the gateway node.

Signed-off-by: Julian Wiedmann <jwi@isovalent.com>
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label May 19, 2022
@julianwiedmann
Copy link
Member Author

/test-only --focus="K8sEgressGatewayTest"

@julianwiedmann
Copy link
Member Author

/test-only --focus="K8sEgressGatewayTest" --kernel_version=net-next

@julianwiedmann
Copy link
Member Author

/test-only --focus="K8sEgressGatewayTest.established connection" --kernel_version=net-next

@github-actions
Copy link

This pull request has been automatically marked as stale because it
has not had recent activity. It will be closed if no further activity
occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale. label Jun 19, 2022
@github-actions
Copy link

github-actions bot commented Jul 4, 2022

This pull request has not seen any activity since it was marked stale.
Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant