HomeElementl

GH issue 3610 (add redis to requirements.yaml)

Description

GH issue 3610 (add redis to requirements.yaml)

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

Reviewers: nate, rexledesma

Reviewed By: rexledesma

Differential Revision: https://dagster.phacility.com/D6281

Details

Provenance
catherinewuAuthored on Feb 3 2021, 7:04 AM
Reviewer
rexledesma
Differential Revision
D6281: GH issue 3610 (add redis to requirements.yaml)
Parents
R1:88907f9473fb: GH issue 3609 (Expose labels field in celery config)
Branches
Unknown
Tags
Unknown