-
Notifications
You must be signed in to change notification settings - Fork 92
Expose SUR APIs #2220
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
Expose SUR APIs #2220
Conversation
Hello williamlardier,My role is to assist you with the merge of this Available options
Available commands
Status report is not available. |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
|
benzekrimaha
left a comment
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
- Share the same logic as the quota initialization - Improve the policy attachment step to be more configurable from the feature files - Test that SUR API is exposed, authv4 works for both Roles and Users, and ensure authz is working as well. The actual check on the metrics is flexible, as this is already tested in SUR repository. Issue: ZENKO-4838
- Required to build scubaclient Issue: ZENKO-4838
- Added support for location in the MetricsClass Issue: ZENKO-4838
Issue: ZENKO-4838
Issue: ZENKO-4838
ddb6df2 to
fd2fc2d
Compare
|
/approve |
|
I have successfully merged the changeset of this pull request
The following branches have NOT changed:
Please check the status of the associated issue ZENKO-4838. Goodbye williamlardier. The following options are set: approve |
Adding SUR API backend (exposed) to Zenko. CTST tests are added.