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

Add support to show prometheus metrics when rbac is enabled in airflow #28

Merged
merged 1 commit into from
Nov 23, 2018
Merged

Add support to show prometheus metrics when rbac is enabled in airflow #28

merged 1 commit into from
Nov 23, 2018

Conversation

phanindhra876
Copy link
Contributor

This enables prometheus metrics view on airflow when rbac is enabled.
Support for plugins in rbac is not available in release 1.10. This will be available in dev/future releases of airflow.

@phanindhra876 phanindhra876 changed the title Add support to show prometheus metrics when rbac enabled in airflow Add support to show prometheus metrics when rbac is enabled in airflow Nov 14, 2018
@phanindhra876
Copy link
Contributor Author

Can anyone have a look at this PR?

@elephantum
Copy link
Contributor

@phani8996 sorry for the delay, we’ll get to it today/tomorrow

@elephantum
Copy link
Contributor

@phani8996 can you please update your branch from current master? this way we could see whether tests in Travis are passing

This enables prometheus metrics view on airflow when rbac is enabled.
Support for plugins in rbac is not available in release 1.10. This will be available in dev/future releases of airflow.
@phanindhra876
Copy link
Contributor Author

phanindhra876 commented Nov 21, 2018

@elephantum Can you help me with testing. I have no idea what's failing the tests.

@elephantum
Copy link
Contributor

@cleverCat подключись плз

@elephantum
Copy link
Contributor

@phani8996 we're on it

@elephantum elephantum merged commit 3636512 into epoch8:master Nov 23, 2018
@elephantum
Copy link
Contributor

@phani8996 thanks again for your help, merged PR to master!

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.

2 participants