Migrated compute metadata to use transport_tpg#17154
Conversation
|
Googlers: For automatic test runs see go/terraform-auto-test-runs. @BBBmau, a repository maintainer, has been assigned to review your changes. If you have not received review feedback within 2 business days, please leave a comment on this PR asking them to take a look. You can help make sure that review is quick by doing a self-review and by running impacted tests locally. |
125c555 to
10aa5ff
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: 1417 Click here to see the affected service packages
Action takenFound 15 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
|
|
🔴 Tests failed during RECORDING mode: 🔴 Errors occurred during RECORDING mode. Please fix them to complete your PR. |
Updated compute library calls to methods : resourceInstancePartnerMetadata , resourceInstancePatchPartnerMetadata flattenPartnerMetadata , ComparePartnerMetadataDiff , BetaMetadataUpdate , MetadataUpdate
10aa5ff to
1d378cc
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: 1418 Click here to see the affected service packages
🟢 All tests passed! View the build log |
| } | ||
|
|
||
| jsonString, err := json.Marshal(&value) | ||
| func convertPartnerMetadataToCompute(pm map[string]interface{}) (map[string]compute.StructuredEntries, error) { |
There was a problem hiding this comment.
LGTM!
I assume that once the migration is complete where transport_tpg is used across compute resources these methods will no longer be needed as a result of the migration?
0c55b33
Release Note Template for Downstream PRs (will be copied)
See Write release notes for guidance.