Charts: incubator/zookeeper: do not use gcr.io/google-samples images

[ahmetb]

incubator/zookeeper sample is using gcr.io/google_samples/k8szk:v2 image:

https://github.com/kubernetes/charts/blob/a7e0c197407204843f5a2fd098fbab88009926c1/incubator/zookeeper/templates/statefulset.yaml#L39

Please do not use this image.

Images in gcr.io/google-samples are:

• meant to be used only in GKE tutorials and samples
• not patched/rebuilt actively with security fixes etc
• not actively maintained in general
• should not be used for production purposes
• there no guarantees whether they simply won't be deleted tomorrow.

Please refrain from using gcr.io/google-samples images in production.

[fejta-bot]

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle stale

[stale[bot]]

This issue is being automatically closed due to inactivity.

[abraithwaite]

/remove-lifecycle stale

[abraithwaite]

incubator zookeeper is still using the sample image

[alokrajiv]

Is there any update on this? We were deploying Kafka and found the zookeeper (a dependency) was using this image. Would like to know if there is a plan to fix this issue?

[abraithwaite]

FWIW we found a bug in the image:

-    echo "autopurge.purgeInteval=$ZK_PURGE_INTERVAL" >> $ZK_CONFIG_FILE
+    echo "autopurge.purgeInterval=$ZK_PURGE_INTERVAL" >> $ZK_CONFIG_FILE

(We reverse engineered the image for our own internal use).

[stevie-]

@abraithwaite We also ran into the bug of zkGenConfig.sh in sample image.
Are you able to provide us the Dockerfile or reference so we can build a new image?

[MartinX3]

@stevie-
https://github.com/helm/charts/pull/11343

[stevie-]

Thanks @MartinX3 ,

we use another workaround, but better use the official zookeeper images

helm chart values-zk-fix.yaml

zookeeper:
  command:
    - /bin/bash
    - -xec
    - zkGenConfig.sh && sed -i 's/autopurge.purgeInteval/autopurge.purgeInterval/' /usr/etc/zookeeper/zoo.cfg && exec zkServer.sh start-foreground

[abraithwaite]

I tried using the official image but the GCR one was so much easier to use.

Clever fix with the start command though! Would have made our lives easier than re-hosting it. :-P

[duksis]

here is fix applied on origin of the typo bug https://github.com/kubernetes-retired/contrib/commit/584b78cfcb88a9e432ee22701fb22b51face8d0d

[liupeng0518]

FWIW we found a bug in the image:

-    echo "autopurge.purgeInteval=$ZK_PURGE_INTERVAL" >> $ZK_CONFIG_FILE
+    echo "autopurge.purgeInterval=$ZK_PURGE_INTERVAL" >> $ZK_CONFIG_FILE

(We reverse engineered the image for our own internal use).

Thanks，Searched for a long time.