Skip to content

Comments

feat: [AAP-58755] Remove scheduler deployment due to dispatcherd migration#319

Merged
kaiokmo merged 1 commit intomainfrom
AAP-58755
Feb 13, 2026
Merged

feat: [AAP-58755] Remove scheduler deployment due to dispatcherd migration#319
kaiokmo merged 1 commit intomainfrom
AAP-58755

Conversation

@kaiokmo
Copy link
Member

@kaiokmo kaiokmo commented Feb 13, 2026

https://issues.redhat.com/browse/AAP-58755

After dispatcherd [1], there's no need for a separate deployment for task queueing, as it is now builtin into the defaultworker. That means we need to remove the leftover extra scheduler deployment that was used as rq-scheduler. Previously, the operator required 5 types of eda deployments: activation-worker, api, default-worker, event-stream and scheduler. Now we drop the scheduler deployment and remove all of its references (CRD, CSV, example CRs, and related files from the eda role).

The complete Redis removal will be done in a different change:
https://issues.redhat.com/browse/AAP-46540

[1] ansible/eda-server#1428

After dispatcherd [1], there's no need for a separate deployment for
task queueing, as it is now builtin into the defaultworker. That means
we need to remove the leftover extra scheduler deployment that was
used as rq-scheduler. Previously, the operator required 5 types of eda
deployments: activation-worker, api, default-worker, event-stream and
scheduler. Now we drop the scheduler deployment and remove all of its
references (CRD, CSV, example CRs, and related files from the eda
role).

[1] ansible/eda-server#1428
@kaiokmo kaiokmo requested review from a team, dsavineau, hsong-rh and ttuffin February 13, 2026 14:29
@kaiokmo
Copy link
Member Author

kaiokmo commented Feb 13, 2026

@hsong-rh Redis removal is a different beast, and will be taken care of in another change (https://issues.redhat.com/browse/AAP-46540)

@kaiokmo kaiokmo requested a review from ptoscano February 13, 2026 18:56
@kaiokmo kaiokmo merged commit c9905d1 into main Feb 13, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants