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

Remove outdated comment from chart values in the workers KEDA conf section #32300

Merged
merged 3 commits into from
Jul 4, 2023

Conversation

hussein-awala
Copy link
Member

In #13209, we have added support for KEDA with workers StatefulSet, and since V2.0.0, we can use KEDA with .Values.workers.persistence.enabled: true.

This PR removes the outdated comment:

`Persistence.enabled` must be set to false to use KEDA.

to avoid confusing the users.


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

@boring-cyborg boring-cyborg bot added the area:helm-chart Airflow Helm Chart label Jul 1, 2023
@hussein-awala hussein-awala added the type:doc-only Changelog: Doc Only label Jul 1, 2023
@potiuk potiuk merged commit d162518 into apache:main Jul 4, 2023
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 type:doc-only Changelog: Doc Only
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants