Skip to content

Instantly share code, notes, and snippets.

@jsturtevant
Last active October 11, 2022 16:22
Show Gist options
  • Save jsturtevant/fed2556449370fdbd0c54b95155e312b to your computer and use it in GitHub Desktop.
Save jsturtevant/fed2556449370fdbd0c54b95155e312b to your computer and use it in GitHub Desktop.

issue: openservicemesh/osm#4817

Webhooks that need updates:

Rotation subscriptions: https://github.com/openservicemesh/osm/blob/7abf10d787d37cbc79c207888e70248c3a9eef66/pkg/certificate/manager.go#L341

Cert management details: https://release-v1-2.docs.openservicemesh.io/docs/guides/certificates/

rotation steps: https://release-v1-2.docs.openservicemesh.io/docs/guides/certificates/#root-certificate-rotation

need to update webhook config bundle: https://github.com/openservicemesh/osm/blob/fa17242a34b39d87b6555774795563aede46efaa/pkg/validator/patch.go#L79

Questions:

controller resync: https://groups.google.com/g/kubernetes-sig-api-machinery/c/PbSCXdLDno0

A resync is different than a relist. The resync plays back all the events held in the informer cache. A relist hits the API server to re-get all the data.

Since we introduced the rate limited work queue a few releases ago, the need to wait for a resync to retry has largely disappeared since an error during processing gets requeued on an incrementing delay.

Think of the resync as insurance. You probably want it set more than a few minutes, less than a few hours. If you're using requeuing correctly and avoiding panics, you aren't likely to benefit from rescanning all the data very often.

apiVersion: config.openservicemesh.io/v1alpha2
kind: MeshRootCertificate
metadata:
name: osm-mesh-root-certificate-v2
namespace: osm-system
spec:
provider:
tresor:
ca:
secretRef:
name: osm-ca-bundle-v2
namespace: osm-system
trustDomain: cluster.local
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment