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.

Sign up for GitHub

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

Jump to bottom

[helm] Redis does not include priorityClassName key #34878

Closed
2 tasks done
dntosas opened this issue Oct 12, 2023 · 0 comments · Fixed by #34879
Closed
2 tasks done

[helm] Redis does not include priorityClassName key #34878

dntosas opened this issue Oct 12, 2023 · 0 comments · Fixed by #34879
Labels
area:helm-chart Airflow Helm Chart kind:bug This is a clearly a bug

Comments

@dntosas
Copy link
Contributor

dntosas commented Oct 12, 2023 •
edited

Official Helm Chart version

1.11.0 (latest released)

Apache Airflow version

2.x

Kubernetes Version

1.25+

What happened

There is no way to configure via parent values the priorityClassName for Redis, which is a workload with PV constraints that usually needs increased priority to be scheduled wherever its PV lives.

What you think should happen instead

Able to include priorityClassName

How to reproduce

Install via Helm Chart

Anything else

No response

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct
@dntosas dntosas added area:helm-chart Airflow Helm Chart kind:bug This is a clearly a bug needs-triage label for new issues that we didn't triage yet labels Oct 12, 2023
@dntosas dntosas changed the title Redis does not include priorityClassName key [helm] Redis does not include priorityClassName key Oct 12, 2023
@dntosas dntosas mentioned this issue Oct 12, 2023
Merged
@nathadfield nathadfield removed the needs-triage label for new issues that we didn't triage yet label Oct 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area:helm-chart Airflow Helm Chart kind:bug This is a clearly a bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(helm): add support for priorityClassName on Redis pods dntosas/airflow
2 participants
@nathadfield @dntosas