-
Notifications
You must be signed in to change notification settings - Fork 40
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(IAM Policy Management): Add support for policy templates #203
Conversation
Signed-off-by: Shaun Colley <shaun.colley@ibm.com>
Signed-off-by: Shaun Colley <shaun.colley@ibm.com>
…lates starting with SDKPython Signed-off-by: Shaun Colley <shaun.colley@ibm.com>
Latest integration tests after change: pytest test/integration/test_iam_policy_management_v1.py
============================================================================================== test session starts ===============================================================================================
platform darwin -- Python 3.10.8, pytest-7.2.0, pluggy-1.0.0
rootdir: /Users/shaunsmacibm/Desktop/cloud-platform/iam/sdk/platform-services-python-sdk
plugins: cov-2.12.1
collected 26 items
test/integration/test_iam_policy_management_v1.py .......................... [100%]
============================================================================================== 26 passed in 13.26s ===============================================================================================
|
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.
I'm approving as the code looks fine (functional) as is. However, I'll just repeat my comment as in the other PRs... i.e. we need to update the API definition with links and example values to improve the generation of int tests and examples.
@@ -49,6 +49,10 @@ | |||
example_policy_etag = None | |||
example_custom_role_id = None | |||
example_custom_role_etag = None | |||
example_template_id = None |
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.
Same comments here re: links & example values as in the other PRs.
Signed-off-by: Vivek.Jain3 <vivek.jain3@ibm.com>
… Tests update Signed-off-by: Vivek.Jain3 <vivek.jain3@ibm.com>
… Tests update Signed-off-by: Vivek.Jain3 <vivek.jain3@ibm.com>
# [0.41.0](v0.40.2...v0.41.0) (2023-09-05) ### Features * **IAM Policy Management:** add support for policy templates ([#203](#203)) ([c9df540](c9df540))
🎉 This PR is included in version 0.41.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
PR summary
Added feature support for Enterprise IAM work, policy templates and get template assigments.
Issue: https://github.ibm.com/IAM/AM-issues/issues/1346
PR Checklist
Please make sure that your PR fulfills the following requirements:
Current vs new behavior
SDK adopters will be able to call policy template and assignment APIs.
Does this PR introduce a breaking change?
Other information
API defintion:
Staging
Prod
Test Information
Integration Tests:
examples tests: