From 5dc05ec43499e93e461245db9558ca773fb27572 Mon Sep 17 00:00:00 2001 From: Kaio Oliveira Date: Fri, 20 Feb 2026 00:46:05 -0300 Subject: [PATCH] feat: [AAP-59028] cleanup old redis artifacts Also, we add a fail-fast task that makes impossible for a user to silently run a misconfigured state thinking redis is backing EDA. We then explicit to the user what needs to be changed. Documentation update will be treated by a future change. --- roles/eda/tasks/deploy_eda.yml | 24 ++++++++++++++++++++++++ roles/eda/tasks/idle_deployment.yml | 24 ++++++++++++++++++++++++ roles/eda/tasks/main.yml | 15 +++++++++++++++ 3 files changed, 63 insertions(+) diff --git a/roles/eda/tasks/deploy_eda.yml b/roles/eda/tasks/deploy_eda.yml index 41480c7..51d0ad5 100644 --- a/roles/eda/tasks/deploy_eda.yml +++ b/roles/eda/tasks/deploy_eda.yml @@ -94,6 +94,30 @@ namespace: "{{ ansible_operator_meta.namespace }}" state: absent +- name: Remove legacy Redis deployment + k8s: + api_version: apps/v1 + kind: Deployment + name: "{{ ansible_operator_meta.name }}-redis" + namespace: "{{ ansible_operator_meta.namespace }}" + state: absent + +- name: Remove legacy Redis service + k8s: + api_version: v1 + kind: Service + name: "{{ ansible_operator_meta.name }}-redis-svc" + namespace: "{{ ansible_operator_meta.namespace }}" + state: absent + +- name: Remove legacy Redis secret + k8s: + api_version: v1 + kind: Secret + name: "{{ ansible_operator_meta.name }}-redis-configuration" + namespace: "{{ ansible_operator_meta.namespace }}" + state: absent + - name: Check for API Pod k8s_info: kind: Pod diff --git a/roles/eda/tasks/idle_deployment.yml b/roles/eda/tasks/idle_deployment.yml index 0ac181b..04104d2 100644 --- a/roles/eda/tasks/idle_deployment.yml +++ b/roles/eda/tasks/idle_deployment.yml @@ -29,6 +29,30 @@ replicas: 0 when: not ui_disabled +- name: Remove legacy Redis deployment + kubernetes.core.k8s: + api_version: apps/v1 + kind: Deployment + name: "{{ ansible_operator_meta.name }}-redis" + namespace: "{{ ansible_operator_meta.namespace }}" + state: absent + +- name: Remove legacy Redis service + kubernetes.core.k8s: + api_version: v1 + kind: Service + name: "{{ ansible_operator_meta.name }}-redis-svc" + namespace: "{{ ansible_operator_meta.namespace }}" + state: absent + +- name: Remove legacy Redis secret + kubernetes.core.k8s: + api_version: v1 + kind: Secret + name: "{{ ansible_operator_meta.name }}-redis-configuration" + namespace: "{{ ansible_operator_meta.namespace }}" + state: absent + - name: Combine postgres default and custom vars for each component ansible.builtin.import_role: name: postgres diff --git a/roles/eda/tasks/main.yml b/roles/eda/tasks/main.yml index 35ab8e9..c5c01a2 100644 --- a/roles/eda/tasks/main.yml +++ b/roles/eda/tasks/main.yml @@ -4,6 +4,21 @@ - name: Combine default and custom vars for each component include_tasks: combine_defaults.yml +- name: Fail if user still references redis configuration + ansible.builtin.fail: + msg: >- + Redis is no longer supported by the EDA operator. EDA now uses + dispatcherd (backed by PostgreSQL) for task queuing. + Please remove the 'redis' section (including 'redis.redis_secret') + from your EDA custom resource spec. No data migration from Redis + is required as task state is persisted in PostgreSQL. + when: + - redis is defined + - redis is mapping + - redis | length > 0 + - redis.redis_secret is defined + - redis.redis_secret | length > 0 + - name: Configure cluster include_role: name: common