open-policy-agent / open-policy-agent/cert-controller
Delay when the certs are mounted and available for use
Nobody has claimed this yet.
- Dominant language
- Go
- Stars
- 109
- Forks
- 49
- Avg merge
- 19h 20m
- Merged PRs (30d)
- 6
Description
I'm using the cert-controller in one of the projects to bootstrap a mutating webhook. I've configured the rotator using the example provided in the doc. Interestingly in most of the CI runs and local testing, I'm seeing a delay when the certs are available in the mount. Seeing it take upto 1m30s in few instances before the certs are ready in the mount path. The delay could be because the Kubernetes secret update is delayed and the mount republish is missed at the first attempt.
Is this a known behavior? Is that why there is RestartOnSecretRefresh property in struct?
github.com/open-policy-agent/cert-controller v0.2.0
k8s.io/kubernetes v1.21.2
sigs.k8s.io/controller-runtime v0.9.2
Usage:
// Make sure certs are generated and valid if cert rotation is enabled.
setupFinished := make(chan struct{})
if !disableCertRotation {
entryLog.Info("setting up cert rotation")
if err := rotator.AddRotator(mgr, &rotator.CertRotator{
SecretKey: types.NamespacedName{
Namespace: util.GetNamespace(),
Name: secretName,
},
CertDir: webhookCertDir,
CAName: caName,
CAOrganization: caOrganization,
DNSName: dnsName,
IsReady: setupFinished,
Webhooks: webhooks,
}); err != nil {
entryLog.Error(err, "unable to set up cert rotation")
os.Exit(1)
}
} else {
close(setupFinished)
}
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
The issue names AddRotator and CertRotator and provides a Go configuration example, but no repository files or tests. Start by locating those entry points and reviewing how IsReady and RestartOnSecretRefresh relate to secret updates and mounted certificates. Done means explaining or correcting the observed delay and adding evidence or a regression test if the behavior is actionable.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go, kubernetes
- Domain
- infrastructure, security
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100