search

tryretool / retool-helm

MIT License
45 stars 57 forks source link

Add nodeSelector to code-executor as well #153

Closed erulabs closed 1 month ago

erulabs commented 6 months ago

Because retool images are not built for ARM64, a nodeSelector kubernetes.io/arch: amd64 needs to be added to all mixed architecture clusters.

Note that a reasonable change, in my mind, is to make kubernetes.io/arch: amd64 a default node selector. That should work on almost all clusters and will prevent crashes on mixed architecture clusters.

avimoondra commented 1 month ago

Hey @erulabs this is addressed in https://github.com/tryretool/retool-helm/pull/169. Thanks for the feedback & PR!