Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: enable mysql keda support for triggerer #37365

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

claneys
Copy link
Contributor

@claneys claneys commented Feb 12, 2024

This follows the previous PR #36167 about MySQL Keda trigger support on workers and is the adaptation to the triggerer statefulset.

Copy link
Member

@hussein-awala hussein-awala left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it needs a test as you did in the other PR

chart/templates/triggerer/triggerer-kedaautoscaler.yaml Outdated Show resolved Hide resolved
@claneys claneys force-pushed the feat/enable-mysql-keda-support-triggerer branch from 86d32d9 to 244fe48 Compare February 12, 2024 19:18
@claneys claneys force-pushed the feat/enable-mysql-keda-support-triggerer branch 7 times, most recently from a6aab9a to f0636e4 Compare February 19, 2024 15:00
@claneys claneys force-pushed the feat/enable-mysql-keda-support-triggerer branch 5 times, most recently from a0492f6 to d804968 Compare February 23, 2024 17:04
@claneys claneys force-pushed the feat/enable-mysql-keda-support-triggerer branch from d804968 to e54cdaf Compare February 28, 2024 09:13
@claneys claneys force-pushed the feat/enable-mysql-keda-support-triggerer branch 5 times, most recently from b651318 to b0e5375 Compare March 7, 2024 10:50
@claneys claneys force-pushed the feat/enable-mysql-keda-support-triggerer branch 3 times, most recently from cb956bb to e502d3c Compare March 15, 2024 13:19
@claneys claneys force-pushed the feat/enable-mysql-keda-support-triggerer branch 7 times, most recently from 51dd63c to 5699e05 Compare April 15, 2024 19:12
@claneys claneys force-pushed the feat/enable-mysql-keda-support-triggerer branch 5 times, most recently from 59093b4 to 5632a5d Compare April 22, 2024 12:12
@claneys claneys force-pushed the feat/enable-mysql-keda-support-triggerer branch 6 times, most recently from 01b0a81 to d58108c Compare April 30, 2024 12:06
@claneys claneys force-pushed the feat/enable-mysql-keda-support-triggerer branch from d58108c to 4535f86 Compare May 1, 2024 21:15
@claneys claneys force-pushed the feat/enable-mysql-keda-support-triggerer branch from 4535f86 to d816009 Compare May 13, 2024 08:22
@claneys
Copy link
Contributor Author

claneys commented May 13, 2024

Hi,

Is there anything blocking this PR to be merge ? I might miss something. Thanks 🙏

cc: @jedcunningham

@claneys claneys force-pushed the feat/enable-mysql-keda-support-triggerer branch 2 times, most recently from 6d2e6dd to cd3519e Compare May 20, 2024 21:15
@claneys claneys force-pushed the feat/enable-mysql-keda-support-triggerer branch 2 times, most recently from da208a8 to df85d20 Compare May 22, 2024 20:50
Signed-off-by: Romain Forlot <romain.forlot@dailymotion.com>
Signed-off-by: Romain Forlot <romain.forlot@dailymotion.com>
Signed-off-by: Romain Forlot <romain.forlot@dailymotion.com>
Also fix typo with worker tests targeting triggerer resources.

Signed-off-by: Romain Forlot <romain.forlot@dailymotion.com>
@claneys claneys force-pushed the feat/enable-mysql-keda-support-triggerer branch from df85d20 to d5b358e Compare May 29, 2024 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:helm-chart Airflow Helm Chart
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants