Sync feature branch resource identity#14374
Sync feature branch resource identity#14374BBBmau wants to merge 476 commits intoGoogleCloudPlatform:FEATURE-BRANCH-resource-identityfrom
Conversation
…loudPlatform#13714) Co-authored-by: Meng Yang <mengyang@google.com>
Co-authored-by: Sam Levenick <slevenick@google.com>
…nal with default (GoogleCloudPlatform#14059) Co-authored-by: Stephen Lewis (Burrows) <stephen.r.burrows@gmail.com>
|
@BBBmau, checks still failed. Is it expected? |
Co-authored-by: Stephen Lewis (Burrows) <stephen.r.burrows@gmail.com>
ah the failing TGC tests come from go.mod being included in the tgc_next, we'll want to open a new PR that handles the SDK/v2 bump. We don't see failures in the FEATURE-BRANCH due to the branch not having a dedicated |
|
Thanks for the information. The errors for tgc unit tests are as the following But this PR Manage go.mod and go.sum of tgc doesn't generate any diffs. I don't get that these errors are related to the tgc PR. I remember I saw these errors before. Should the new PR that handles the SDK/v2 bump be merging before the current PR? |
|
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
|
5348c78 to
6d9dd95
Compare
)" This reverts commit 5629b75.
…NCH-resource-identity
1029bcb to
036c9f3
Compare
Non-exercised tests🔴 Tests were added that are skipped in VCR:
🔴 Tests were added that are GA-only additions and require manual runs:
Tests analyticsTotal tests: 5322 Click here to see the affected service packages
Found 112 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. |
|
@zli82016 This PR has been waiting for review for 3 weekdays. Please take a look! Use the label |
|
@GoogleCloudPlatform/terraform-team @zli82016 This PR has been waiting for review for 1 week. Please take a look! Use the label |
|
The checks still fail. |
|
@GoogleCloudPlatform/terraform-team @zli82016 This PR has been waiting for review for 2 weeks. Please take a look! Use the label |
Release Note Template for Downstream PRs (will be copied)
See Write release notes for guidance.