Page MenuHomeElementl

GH issue 3610 (add redis to requirements.yaml)
ClosedPublic

Authored by catherinewu on Feb 3 2021, 7:42 AM.

Details

Summary

https://github.com/dagster-io/dagster/issues/3610

bitnami chart is recommended: https://github.com/helm/charts/tree/master/stable/redis

need to make the schema more permissive since the bitnami chart supports a lot more stuff. From the schema mismatch error message.... "Helm install completed with stderr: b"WARNING: Kubernetes configuration file is group-readable. This is insecure. Location: /tmp/tmpk8ouex4e\nWARNING: Kubernetes configuration file is world-readable. This is insecure. Location: /tmp/tmpk8ouex4e\nError: values don't meet the specifications of the schema(s) in the following chart(s):\ndagster:\n- redis: Additional property cluster is not allowed\n- redis: Additional property sentinel is not allowed\n- redis: Additional property serviceAccount is not allowed\n- redis: Additional property clusterDomain is not allowed\n- redis: Additional property podDisruptionBudget is not allowed\n- redis: Additional property usePassword is not allowed\n- redis: Additional property common is not allowed\n- redis: Additional property global is not allowed\n- redis: Additional property persistence is not allowed\n- redis: Additional property securityContext is not allowed\n- redis: Additional property volumePermissions is not allowed\n- redis: Additional property usePasswordFile is not allowed\n- redis: Additional property configmap is not allowed\n- redis: Additional property password is not allowed\n- redis: Additional property redisPort is not allowed\n- redis: Additional property slave is not allowed\n- redis: Additional property tls is not allowed\n- redis: Additional property containerSecurityContext is not allowed\n- redis: Additional property image is not allowed\n- redis: Additional property rbac is not allowed\n- redis: Additional property master is not allowed\n- redis: Additional property metrics is not allowed\n- redis: Additional property networkPolicy is not allowed\n- redis: Additional property podSecurityPolicy is not allowed\n- redis: Additional property sysctlImage is not allowed\n\n""

Test Plan

integration

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Feb 3 2021, 8:05 AM
Harbormaster failed remote builds in B25223: Diff 30745!
catherinewu retitled this revision from GH issue 3610 to GH issue 3610 (add redis to requirements.yaml).
Harbormaster returned this revision to the author for changes because remote builds failed.Feb 3 2021, 8:11 PM
Harbormaster failed remote builds in B25244: Diff 30769!
Harbormaster returned this revision to the author for changes because remote builds failed.Feb 11 2021, 6:52 AM
Harbormaster failed remote builds in B25607: Diff 31249!
This revision is now accepted and ready to land.Feb 11 2021, 3:03 PM