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: allow airflow.users[].roles to specify a list of roles #539

Merged
merged 3 commits into from
Mar 31, 2022

Conversation

kmehkeri
Copy link
Contributor

@kmehkeri kmehkeri commented Mar 6, 2022

What issues does your PR fix?

What does your PR do?

  • Alows airflow.users[].roles to be either a list of flask_appbuilder role-names, or a single role-name string.

Checklist

For all Pull Requests

For releasing ONLY

Signed-off-by: kmehkeri <kmehkeri@gmail.com>
@thesuperzapper thesuperzapper added this to the airflow-8.6.0 milestone Mar 22, 2022
Signed-off-by: Mathew Wicks <thesuperzapper@users.noreply.github.com>
@thesuperzapper thesuperzapper changed the title feat: allow multiple roles on a user feat: allow airflow.users[].roles to specify a list of roles Mar 30, 2022
@thesuperzapper thesuperzapper added the status/ready-to-merge status - this will be merged into next release label Mar 30, 2022
@thesuperzapper
Copy link
Member

@kmehkeri thanks for the PR!

I have made a few changes in 7570f52, so I can merge it ASAP and release 8.6.0.

thesuperzapper
thesuperzapper previously approved these changes Mar 30, 2022
Signed-off-by: Mathew Wicks <thesuperzapper@users.noreply.github.com>
@thesuperzapper thesuperzapper merged commit 138bb83 into airflow-helm:main Mar 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/ready-to-merge status - this will be merged into next release
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

allow airflow.users[].roles to specify a list of roles
2 participants