artifactregistry: Remove Immutable from upstream_credentials#17871
artifactregistry: Remove Immutable from upstream_credentials#17871NA2047 wants to merge 7 commits into
Conversation
|
Googlers: For automatic test runs see go/terraform-auto-test-runs. @trodge, 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. |
trodge
left a comment
There was a problem hiding this comment.
Do we have an update test for these fields?
Hey @trodge I'll write one :) |
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 2dcd3ab: Diff reportYour PR generated the following diffs in downstream repositories:
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode Action takenFound 1 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
View the replaying VCR build log Step 2: Recording Mode
Caution Issues requiring attention before PR completion 🔴 Initial Recording Failed: Some tests failed during the recording step. See the table above for details. Please address these issues to complete your PR. If you believe these detections are incorrect or unrelated to your change, please raise the concern with your reviewer. View the recording VCR build log or the debug logs folder for detailed results. @NA2047, @Subserial, @trodge VCR tests complete for 2dcd3ab! |
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 5e40bc5: Diff reportYour PR generated the following diffs in downstream repositories:
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode Action takenFound 1 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
View the replaying VCR build log Step 2: Recording Mode
Caution Issues requiring attention before PR completion 🔴 Initial Recording Failed: Some tests failed during the recording step. See the table above for details. Please address these issues to complete your PR. If you believe these detections are incorrect or unrelated to your change, please raise the concern with your reviewer. View the recording VCR build log or the debug logs folder for detailed results. @NA2047, @Subserial, @trodge VCR tests complete for 5e40bc5! |
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 16927dd: Diff reportYour PR generated the following diffs in downstream repositories:
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode Action takenFound 1 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
View the replaying VCR build log Step 2: Recording Mode
Caution Issues requiring attention before PR completion 🔴 Initial Recording Failed: Some tests failed during the recording step. See the table above for details. Please address these issues to complete your PR. If you believe these detections are incorrect or unrelated to your change, please raise the concern with your reviewer. View the recording VCR build log or the debug logs folder for detailed results. @NA2047, @Subserial, @trodge VCR tests complete for 16927dd! |
16927dd to
fb277f0
Compare
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit fb277f0: Diff reportYour PR generated the following diffs in downstream repositories:
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode Action takenFound 1 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
View the replaying VCR build log Step 2: Recording Mode
Caution Issues requiring attention before PR completion 🔴 Initial Recording Failed: Some tests failed during the recording step. See the table above for details. Please address these issues to complete your PR. If you believe these detections are incorrect or unrelated to your change, please raise the concern with your reviewer. View the recording VCR build log or the debug logs folder for detailed results. @NA2047, @Subserial, @trodge VCR tests complete for fb277f0! |
|
@trodge This PR has been waiting for review for 3 weekdays. Please take a look! Use the label |
|
The test is failing with However, I don't see a |
…citly supported in Cloud Asset Inventory
72384d2 to
9ddef40
Compare
Fixes for hashicorp/terraform-provider-google#20520 and hashicorp/terraform-provider-google#20520
Release Note Template for Downstream PRs (will be copied)
See Write release notes for guidance.