osm
osm copied to clipboard
feat(certs): handle MRCUpdated event when intent set passive
Description:
Adds logic to update MRC conditions and components. Handles MRCUpdated event when intent set to passive.
Part of #5091
Testing done:
Affected area:
| Functional Area | |
|---|---|
| New Functionality | [ ] |
| CI System | [ ] |
| CLI Tool | [ ] |
| Certificate Management | [ ] |
| Control Plane | [ ] |
| Demo | [ ] |
| Documentation | [ ] |
| Egress | [ ] |
| Ingress | [ ] |
| Install | [ ] |
| Networking | [ ] |
| Observability | [ ] |
| Performance | [ ] |
| SMI Policy | [ ] |
| Security | [ ] |
| Sidecar Injection | [ ] |
| Tests | [ ] |
| Upgrade | [ ] |
| Other | [ ] |
Please answer the following questions with yes/no.
-
Does this change contain code from or inspired by another project?
- Did you notify the maintainers and provide attribution?
-
Is this a breaking change?
-
Has documentation corresponding to this change been updated in the osm-docs repo (if applicable)?
I'd personally prefer to see a PR like this come before https://github.com/openservicemesh/osm/pull/5073. I think there's a lot of caveats like how many can run at once (might infer if we need a mutex, how the MRCReconciler signifies that it is done reconciling, or other implementation details). Curious on your thoughts and how others may feel though
Closing in favor of #5201