You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Oct 22, 2021. It is now read-only.
Describe the bug
After a QuarksSecret referenced in a BOSHDeployment has been rotated, the corresponding desired-manifest is not updated.
To Reproduce
Deploy KubeCF
Trigger a rotation on the var-locket-database-password QuarksSecret.
Watch the database-seeder job trigger.
Watch the diego-api QuarksStatefulSet not restart and pick up the new password.
Delete the diego-api pod(s) and see them fail to connect to the database (because they're using old credentials).
Expected behavior
The desired manifest should update.
The affected pods should restart.
Environment
CF-Operator v6.0.4-0.gb44bb859
KubeCF master-ish.
Additional context
This means a KubeCF user can rotate secrets, but once they do the whole cluster is busted.
If I spam the quarks.cloudfoundry.org/restart-on-update annotation on all my instance groups it seems to work. But I was under the impression that updateOnConfigChange was supposed to take care of that for me.
The text was updated successfully, but these errors were encountered:
Describe the bug
After a QuarksSecret referenced in a BOSHDeployment has been rotated, the corresponding
desired-manifest
is not updated.To Reproduce
var-locket-database-password
QuarksSecret.database-seeder
job trigger.diego-api
QuarksStatefulSet not restart and pick up the new password.diego-api
pod(s) and see them fail to connect to the database (because they're using old credentials).Expected behavior
Environment
v6.0.4-0.gb44bb859
master
-ish.Additional context
This means a KubeCF user can rotate secrets, but once they do the whole cluster is busted.
If I spam the
quarks.cloudfoundry.org/restart-on-update
annotation on all my instance groups it seems to work. But I was under the impression thatupdateOnConfigChange
was supposed to take care of that for me.The text was updated successfully, but these errors were encountered: