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

Intents operator creates separate network policies for ingress and egress #305

Closed
orishoshan opened this issue Dec 5, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@orishoshan
Copy link
Collaborator

Is your feature request related to a problem? Please describe.
The intents operator by default creates only ingress network policies. An experimental flag also allows to add egress network policies.
Ingress network policies are created to allow access TO a pod, so they apply to the destination.
Egress network policies are created to allow access FROM a pod, so they apply to the source.
This means that, when this experimental feature is enabled, one ClientIntent will result in two network policies being created: one on the server and one on the client.

In some cases this is fine, such as when the client is not also a server, which results in the minimal amount of network policies.
However, in some cases a client is also a server, so it is possible to create a single network policy to handle both ingress and egress for this pod.

This is in-line with what the intents-operator currently does for ingress, where it will create just 1 network policy for all clients. Keeping a small number of network policies aids debuggability.

Describe the solution you'd like
When possible, the intents operator should create a single network policy per pod, for both ingress and egress.

@orishoshan orishoshan added the enhancement New feature or request label Dec 5, 2023
@amitlicht
Copy link
Contributor

This was since implemented and available. The intents-operator will, by default, create a single network policy for pod, for both ingress and egress. This is controllable using configuration.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants