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

Annotate kubernetes resources #82

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

asimpleidea
Copy link
Member

This PR allows the egress watcher to annotate a supported Kubernetes object -- i.e. a ServiceEntry or NetworkPolicy -- with key that notify whether it was inserted or enabled on the chosen SDWAN.

For example, after kubectl applying a ServiceEntry correctly, egress watcher will annotate it with egress-watcher.cnwan.io/sdwan-populated after it receives a confirmation that it was successfully inserted, and `"egress-watcher.cnwan.io/sdwan-enabled" after a successful configuration/policy update from SDWAN.

The annotations handler contains variables and functions that will
create or remove annotations on supported Kubernetes objects.

Signed-off-by: Elis Lulja <elulja@cisco.com>
Signed-off-by: Elis Lulja <elulja@cisco.com>
Signed-off-by: Elis Lulja <elulja@cisco.com>
The SDWAN uses the annotations channel to notify the annotations handler
that an operation -- i.e. inserted or enabled -- was successful.

Signed-off-by: Elis Lulja <elulja@cisco.com>
This commit runs the goroutine that watchese for operations updates
coming from SDWAN.

Signed-off-by: Elis Lulja <elulja@cisco.com>
The provided `ClusterRole` is updated to allow the egress watcher to
update supported objects to be annotated by it.

Signed-off-by: Elis Lulja <elulja@cisco.com>
This commit includes the `NamespacedName` to SDWAN, so that it can
notify the annotations handler that that object can be annotation after
a successful operation.

Signed-off-by: Elis Lulja <elulja@cisco.com>
@asimpleidea asimpleidea added the enhancement New feature or request label Jan 17, 2023
@asimpleidea asimpleidea self-assigned this Jan 17, 2023
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

Successfully merging this pull request may close these issues.

1 participant