Artifact - Remove redis channel (#4320)

* Manually reverting 7df67de ([this branch](master...jeev/sandbox-redis)), not sure why readme isn't updating, will worry about it later. * Remove the redis publisher. * Remove values related to redis publisher. Signed-off-by: Yee Hing Tong <[email protected]>
flyteorg · Oct 28, 2023 · bdddc93 · bdddc93
1 parent ceb390d
commit bdddc93
Show file tree

Hide file tree

Showing 9 changed files with 2,005 additions and 1,373 deletions.
diff --git a/charts/flyte-sandbox/Chart.lock b/charts/flyte-sandbox/Chart.lock
@@ -14,8 +14,5 @@ dependencies:
 - name: postgresql
   repository: https://charts.bitnami.com/bitnami
   version: 12.1.9
-- name: redis
-  repository: https://charts.bitnami.com/bitnami
-  version: 17.15.6
-digest: sha256:1bcc5f102768a19c19d444b3460dabd0f90847b2d4423134f0ce9c7aa0a256ea
-generated: "2023-08-31T16:46:00.478623-07:00"
+digest: sha256:e7155e540bbdb98f690eb12e2bd301a19d8b36833336f6991410cb44d8d9bb5e
+generated: "2023-10-28T10:05:34.269916+08:00"
diff --git a/charts/flyte-sandbox/Chart.yaml b/charts/flyte-sandbox/Chart.yaml
@@ -44,7 +44,3 @@ dependencies:
     version: 12.1.9
     repository: https://charts.bitnami.com/bitnami
     condition: postgresql.enabled
-  - name: redis
-    version: 17.15.6
-    repository: https://charts.bitnami.com/bitnami
-    condition: redis.enabled
diff --git a/charts/flyte-sandbox/values.yaml b/charts/flyte-sandbox/values.yaml
@@ -49,12 +49,10 @@ flyte-binary:
           memory: 0
       cloudEvents:
         enable: true
-        transformToCloudEvents: true
-        type: redis
-        redis:
-          addr: "flyte-sandbox-redis-headless.flyte.svc.cluster.local:6379"
+        cloudEventVersion: v2
+        type: sandbox
       artifacts:
-        host: artifact-service.flyte.svc.cluster.local
+        host: localhost
         port: 50051
         insecure: true
       storage:
@@ -159,21 +157,6 @@ postgresql:
       tag: sandbox
       pullPolicy: Never
 
-redis:
-  enabled: true
-  image:
-    tag: sandbox
-    pullPolicy: Never
-  auth:
-    enabled: false
-  master:
-    service:
-      type: NodePort
-      nodePorts:
-        redis: 30004
-  replica:
-    replicaCount: 0
-
 sandbox:
   # dev Routes requests to an instance of Flyte running locally on a developer's
   # development environment. This is only usable if the flyte-binary chart is disabled.

diff --git a/docker/sandbox-bundled/images/manifest.txt b/docker/sandbox-bundled/images/manifest.txt
@@ -1,7 +1,6 @@
 docker.io/bitnami/bitnami-shell:sandbox=bitnami/bitnami-shell:11-debian-11-r76
 docker.io/bitnami/minio:sandbox=bitnami/minio:2023.1.25-debian-11-r0
 docker.io/bitnami/postgresql:sandbox=bitnami/postgresql:15.1.0-debian-11-r20
-docker.io/bitnami/redis:sandbox=bitnami/redis:7.2.0-debian-11-r3
 docker.io/envoyproxy/envoy:sandbox=envoyproxy/envoy:v1.23-latest
 docker.io/kubernetesui/dashboard:sandbox=kubernetesui/dashboard:v2.7.0
 docker.io/library/registry:sandbox=registry:2.8.1