-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
ephemeral
: add google_service_account_id_token
#12141
base: FEATURE-BRANCH-ephemeral-resource
Are you sure you want to change the base?
ephemeral
: add google_service_account_id_token
#12141
Conversation
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are.
Errors
|
Tests analyticsTotal tests: 0 Click here to see the affected service packages
View the build log |
e6dfed7
to
c92521f
Compare
Marking as ready for review with tests passing locally: === RUN TestEphemeralServiceAccountIdToken_basic
=== PAUSE TestEphemeralServiceAccountIdToken_basic
=== RUN TestEphemeralServiceAccountIdToken_withDelegates
=== PAUSE TestEphemeralServiceAccountIdToken_withDelegates
=== RUN TestEphemeralServiceAccountIdToken_withIncludeEmail
=== PAUSE TestEphemeralServiceAccountIdToken_withIncludeEmail
=== CONT TestEphemeralServiceAccountIdToken_basic
=== CONT TestEphemeralServiceAccountIdToken_withIncludeEmail
=== CONT TestEphemeralServiceAccountIdToken_withDelegates
2024/11/06 13:57:01 [INFO] Authenticating using configured Google JSON 'credentials'...
2024/11/06 13:57:01 [INFO] -- Scopes: [https://www.googleapis.com/auth/cloud-platform https://www.googleapis.com/auth/userinfo.email]
2024/11/06 13:57:01 [INFO] Authenticating using configured Google JSON 'credentials'...
2024/11/06 13:57:01 [INFO] -- Scopes: [https://www.googleapis.com/auth/cloud-platform https://www.googleapis.com/auth/userinfo.email]
2024/11/06 13:57:01 [INFO] Authenticating using configured Google JSON 'credentials'...
2024/11/06 13:57:01 [INFO] -- Scopes: [https://www.googleapis.com/auth/cloud-platform https://www.googleapis.com/auth/userinfo.email]
2024/11/06 13:57:01 [INFO] Authenticating using configured Google JSON 'credentials'...
2024/11/06 13:57:01 [INFO] -- Scopes: [https://www.googleapis.com/auth/cloud-platform https://www.googleapis.com/auth/userinfo.email]
2024/11/06 13:57:01 [INFO] Authenticating using configured Google JSON 'credentials'...
2024/11/06 13:57:01 [INFO] -- Scopes: [https://www.googleapis.com/auth/cloud-platform https://www.googleapis.com/auth/userinfo.email]
2024/11/06 13:57:01 [DEBUG] Waiting for state to become: [success]
2024/11/06 13:57:01 [DEBUG] Waiting for state to become: [success]
2024/11/06 13:57:01 [INFO] Authenticating using configured Google JSON 'credentials'...
2024/11/06 13:57:01 [INFO] -- Scopes: [https://www.googleapis.com/auth/cloud-platform https://www.googleapis.com/auth/userinfo.email]
2024/11/06 13:57:01 [DEBUG] Waiting for state to become: [success]
2024/11/06 13:57:01 [INFO] Terraform is using this identity: mauricio-alvarezleon@hc-terraform-testing.iam.gserviceaccount.com
2024/11/06 13:57:01 [DEBUG] Verifying projects/hc-terraform-testing/serviceAccounts/tf-bootstrap-sa-idtoken@hc-terraform-testing.iam.gserviceaccount.com as bootstrapped service account.
2024/11/06 13:57:01 [INFO] Instantiating Google Cloud IAM client for path https://iam.googleapis.com/
2024/11/06 13:57:01 [DEBUG] Retry Transport: starting RoundTrip retry loop
2024/11/06 13:57:01 [DEBUG] Retry Transport: request attempt 0
2024/11/06 13:57:01 [INFO] Terraform is using this identity: mauricio-alvarezleon@hc-terraform-testing.iam.gserviceaccount.com
2024/11/06 13:57:01 [DEBUG] Verifying projects/hc-terraform-testing/serviceAccounts/tf-bootstrap-sa-idtoken-email@hc-terraform-testing.iam.gserviceaccount.com as bootstrapped service account.
2024/11/06 13:57:01 [INFO] Instantiating Google Cloud IAM client for path https://iam.googleapis.com/
2024/11/06 13:57:01 [DEBUG] Retry Transport: starting RoundTrip retry loop
2024/11/06 13:57:01 [DEBUG] Retry Transport: request attempt 0
2024/11/06 13:57:01 [INFO] Terraform is using this identity: mauricio-alvarezleon@hc-terraform-testing.iam.gserviceaccount.com
2024/11/06 13:57:01 [DEBUG] Verifying projects/hc-terraform-testing/serviceAccounts/tf-bootstrap-sa-target@hc-terraform-testing.iam.gserviceaccount.com as bootstrapped service account.
2024/11/06 13:57:01 [INFO] Instantiating Google Cloud IAM client for path https://iam.googleapis.com/
2024/11/06 13:57:01 [DEBUG] Retry Transport: starting RoundTrip retry loop
2024/11/06 13:57:01 [DEBUG] Retry Transport: request attempt 0
2024/11/06 13:57:02 [DEBUG] Retry Transport: Stopping retries, last request was successful
2024/11/06 13:57:02 [DEBUG] Retry Transport: Returning after 1 attempts
2024/11/06 13:57:02 [DEBUG] Setting service account permissions.
2024/11/06 13:57:02 [INFO] Instantiating Google Cloud IAM client for path https://iam.googleapis.com/
2024/11/06 13:57:02 [DEBUG] Retry Transport: starting RoundTrip retry loop
2024/11/06 13:57:02 [DEBUG] Retry Transport: request attempt 0
2024/11/06 13:57:02 [DEBUG] Retry Transport: Stopping retries, last request was successful
2024/11/06 13:57:02 [DEBUG] Retry Transport: Returning after 1 attempts
2024/11/06 13:57:02 [DEBUG] Setting service account permissions.
2024/11/06 13:57:02 [INFO] Instantiating Google Cloud IAM client for path https://iam.googleapis.com/
2024/11/06 13:57:02 [DEBUG] Retry Transport: starting RoundTrip retry loop
2024/11/06 13:57:02 [DEBUG] Retry Transport: request attempt 0
2024/11/06 13:57:02 [DEBUG] Retry Transport: Stopping retries, last request was successful
2024/11/06 13:57:02 [DEBUG] Retry Transport: Returning after 1 attempts
2024/11/06 13:57:02 [DEBUG] Setting service account permissions.
2024/11/06 13:57:02 [INFO] Instantiating Google Cloud IAM client for path https://iam.googleapis.com/
2024/11/06 13:57:02 [DEBUG] Retry Transport: starting RoundTrip retry loop
2024/11/06 13:57:02 [DEBUG] Retry Transport: request attempt 0
2024/11/06 13:57:02 [DEBUG] Retry Transport: Stopping retries, last request was successful
2024/11/06 13:57:02 [DEBUG] Retry Transport: Returning after 1 attempts
--- PASS: TestEphemeralServiceAccountIdToken_basic (22.84s)
--- PASS: TestEphemeralServiceAccountIdToken_withIncludeEmail (23.23s)
--- PASS: TestEphemeralServiceAccountIdToken_withDelegates (23.35s)
PASS
ok github.com/hashicorp/terraform-provider-google/google/services/resourcemanager 30.963s |
.../third_party/terraform/services/resourcemanager/ephemeral_google_service_account_id_token.go
Outdated
Show resolved
Hide resolved
.../third_party/terraform/services/resourcemanager/ephemeral_google_service_account_id_token.go
Outdated
Show resolved
Hide resolved
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are.
|
Tests analyticsTotal tests: 4267 Click here to see the affected service packages
Action takenFound 234 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
|
🟢 Tests passed during RECORDING mode: 🔴 Tests failed when rerunning REPLAYING mode: Tests failed due to non-determinism or randomness when the VCR replayed the response after the HTTP request was made. Please fix these to complete your PR. If you believe these test failures to be incorrect or unrelated to your change, or if you have any questions, please raise the concern with your reviewer. 🔴 Tests failed during RECORDING mode: 🔴 Errors occurred during RECORDING mode. Please fix them to complete your PR. |
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.
Some initial review, based off the feedback I gave in #12140
...d_party/terraform/services/resourcemanager/ephemeral_google_service_account_id_token_test.go
Show resolved
Hide resolved
.../third_party/terraform/services/resourcemanager/ephemeral_google_service_account_id_token.go
Outdated
Show resolved
Hide resolved
c92521f
to
f4825f5
Compare
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are.
|
Tests analyticsTotal tests: 4267 Click here to see the affected service packages
Action takenFound 4 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
|
🟢 Tests passed during RECORDING mode: 🟢 No issues found for passed tests after REPLAYING rerun. 🔴 Tests failed during RECORDING mode: 🔴 Errors occurred during RECORDING mode. Please fix them to complete your PR. |
.../third_party/terraform/services/resourcemanager/ephemeral_google_service_account_id_token.go
Outdated
Show resolved
Hide resolved
f4825f5
to
a5c6da5
Compare
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are.
|
Tests analyticsTotal tests: 4267 Click here to see the affected service packages
Action takenFound 5 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
|
🟢 Tests passed during RECORDING mode: 🟢 No issues found for passed tests after REPLAYING rerun. 🔴 Tests failed during RECORDING mode: 🔴 Errors occurred during RECORDING mode. Please fix them to complete your PR. |
Release Note Template for Downstream PRs (will be copied)
See Write release notes for guidance.