scylla-operator icon indicating copy to clipboard operation
scylla-operator copied to clipboard

Always resolve SCYLLA_OPERATOR_IMAGE to sha reference

Open tnozicka opened this issue 3 years ago • 9 comments

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 avatar Jul 07 '21 16:07 tnozicka

@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.

rzetelskik avatar Sep 30 '21 12:09 rzetelskik

the operator image is embedded in every scylla deployment for the side car so if the user:

  1. deploys the operator with latest, currently pointing to 1.4.0
  2. deploys a ScyllaCluster with 3 replicas, sidecar get's the image for 1.4.0
  3. :latest changes to 1.5.0
  4. :latest changes to 1.6.0
  5. now you have an unsupported skew because the image is effectively not reconciled
  6. one of the scylla pods gets evicted and the new pod resolves to 1.6.0 while the other two still run from 1.4.0

similar issues arise for the case when a tag is rewritten, sha is the only way to reliably reference the same image

tnozicka avatar Oct 26 '21 08:10 tnozicka

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

tnozicka avatar Jun 26 '24 13:06 tnozicka