scylla-operator
scylla-operator copied to clipboard
Always resolve SCYLLA_OPERATOR_IMAGE to sha reference
Is this a bug report or feature request?
- Feature Request
What should the feature do:
If someone useses a rolling update tag like latest
or 1.3
for SCYLLA_OPERATOR_IMAGE
, we won't initiate a rolling restart to update the sidecar image. We should always use sha in the objects we generate even if the user forgot to do so.
@tnozicka why would we want to initiate a rolling restart whenever a new image is published using an existing tag? I've consulted this behaviour with @zimnx and he expressed concern as well.
the operator image is embedded in every scylla deployment for the side car so if the user:
- deploys the operator with
latest
, currently pointing to1.4.0
- deploys a ScyllaCluster with 3 replicas, sidecar get's the image for
1.4.0
-
:latest
changes to1.5.0
-
:latest
changes to1.6.0
- now you have an unsupported skew because the image is effectively not reconciled
- one of the scylla pods gets evicted and the new pod resolves to
1.6.0
while the other two still run from1.4.0
similar issues arise for the case when a tag is rewritten, sha is the only way to reliably reference the same image
The Scylla Operator project currently lacks enough contributors to adequately respond to all issues.
This bot triages un-triaged issues according to the following rules:
- After 30d of inactivity,
lifecycle/stale
is applied - After 30d of inactivity since
lifecycle/stale
was applied,lifecycle/rotten
is applied - After 30d of inactivity since
lifecycle/rotten
was applied, the issue is closed
You can:
- Mark this issue as fresh with
/remove-lifecycle stale
- Close this issue with
/close
- Offer to help out
/lifecycle stale
This may be be replaced by splitting this logic into a separate image but until we have a plan for that let's keep this here. /remove-lifecycle stale