-
Notifications
You must be signed in to change notification settings - Fork 8
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
enable authorization for observability metrics-powerscale #115
Conversation
877bc9a
to
c4c54f7
Compare
c4c54f7
to
606b053
Compare
Hi Code Owners, please kindly review the PR and let me know if any change needed. Thank you! |
606b053
to
a3a32ac
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM based on unit and e2e test results. Have yo tested the authorization unit and e2e tests with this config as well?
hi @hoppea2 , yes, the authorization unit tests are passed when testing modules package; And for the e2e tests, they are passed: enable authorization standalone, and enable both authorization and observability modules with different combinations. |
a3a32ac
to
536d32c
Compare
536d32c
to
eff690e
Compare
unit tests and e2e tests are run and passed again. Please help review this PR. |
Description
enable authorization for observability metrics-powerscale.
GitHub Issues
List the GitHub issues impacted by this PR:
Checklist:
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration
Passed: enable both modules observability and authorization
output:
Grafana UI display correctly