-
Couldn't load subscription status.
- Fork 3
feat: add two new service roles #427
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
Conversation
|
/run pipeline |
|
Im expecting upgrade test to fail as it will want to update the auth policy. Once it updates, as opposed to destroys / re-creates, its safe to skip |
|
@jor2 you need to bump the test wrapper for addons fix |
|
/run pipeline |
|
/run pipeline |
|
There was a bug in the test which I pushed a fix for. re-running.. |
|
/run pipeline |
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.
@jor2 You need to skip the upgrade test. This is an update in place and expected, so safe to skip..
Messages: Resource(s) identified to be updated
Name: iam_identity_policy
Address: module.secrets_manager.module.secrets_manager.ibm_iam_authorization_policy.iam_identity_policy[0]
Actions: [update]
DIFF:
Before:
{"description":"Allows Secrets Manager instance dde9d8d7-0a37-4c31-a3cb-0d54c9b2e885 `Operator` access to the IAM Identity service to enable creating IAM credentials.","roles":"SECURE_VALUE_HIDDEN_HASH:-a61e9a8ffac694e1beff22f79c8445e563a8622b20c417c0133f32bf"}
After:
{"description":"Allows Secrets Manager instance dde9d8d7-0a37-4c31-a3cb-0d54c9b2e885 access to the IAM Identity service to enable creating IAM credentials.","roles":"SECURE_VALUE_HIDDEN_HASH:-f69404ba44ac8ee69a7230748679d59c2da072aaeb2eaf42a4d23d74"}
|
/run pipeline |
|
/run pipeline |
|
/run pipeline |
|
@jor2 you need to resolve conflicts first |
|
/run pipeline |
1 similar comment
|
/run pipeline |
=== NAME TestAddonsExistingSecretsManager
setup_teardown.go:513:
Error Trace: /go/pkg/mod/github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper@v1.60.13/testaddons/setup_teardown.go:513
/go/pkg/mod/github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper@v1.60.13/testaddons/setup_teardown.go:451
/usr/local/go/src/runtime/asm_amd64.s:1693
Error: Received unexpected error:
failed after 5 attempts: This project cannot be deleted because there is at least one configuration running a pipeline action. Try again after the following configurations are done their pipeline actions: extsm-qvu-deploy-arch-ibm-secrets-manager(8ce40b73-a194-4435-aefd-c0de25fe787c)
Test: TestAddonsExistingSecretsManager
[TestAddonsExistingSecretsManager] TEST EXECUTION END: TestAddonsExistingSecretsManager - RESULT: PASSED
--- FAIL: TestAddonsExistingSecretsManager (2993.00s) |
|
I have bumped up to test wrapper 1.60.15 and also common-dev-assets in a renovate related change (#430) which are newer than the versions in this PR. So it should be appropriate to remove them from here. |
|
/run pipeline |
|
🎉 This PR is included in version 2.11.2 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Description
#425
Release required?
x.x.X)x.X.x)X.x.x)Release notes content
Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
Checklist for reviewers
For mergers