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: add airflow.roles value to manage FAB roles #537

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kmehkeri
Copy link
Contributor

@kmehkeri kmehkeri commented Mar 2, 2022

What issues does your PR fix?

Enables sync of FAB roles via values.yaml.

What does your PR do?

This PR adds a section .roles to values.yaml, which is read and processed by roles-sync job to add defined roles and permissions to FAB database. It's very similar to how users and variables are synced, except that it doesn't need templates.

Checklist

For all Pull Requests

For releasing ONLY

@thesuperzapper thesuperzapper removed the request for review from gsemet March 2, 2022 22:04
@thesuperzapper thesuperzapper changed the title feat: add support for specifying FAB roles (#282) feat: add support for specifying FAB roles Mar 2, 2022
@thesuperzapper thesuperzapper added this to the airflow-8.7.0 milestone Mar 2, 2022
@MrBones757
Copy link

Hello!

this implementation looks fairly complete; is there any ETA on when we might see this merged / released?

@thesuperzapper thesuperzapper changed the title feat: add support for specifying FAB roles feat: add airflow.roles value to manage FAB roles May 1, 2024
@afreyermuth98
Copy link

afreyermuth98 commented Nov 6, 2024

Will this PR would be merge some day ? It would be really useful ! cc @thesuperzapper

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.

add support for specifying FAB roles (airflow.userRoles)
4 participants