project: add Resource Identity to iam_policy#17586
Conversation
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 2586083: Diff reportYour PR generated the following diffs in downstream repositories:
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode Action takenFound 19 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 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. 🔴 Replaying Rerun Failed: Some tests failed due to non-determinism when VCR replayed the response. 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 build log or the debug logs folder for detailed results. |
|
/gcbrun |
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 2586083: Diff reportYour PR generated the following diffs in downstream repositories:
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode Action takenFound 17 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. |
| // IamPolicyBaseIdentitySchema is intentionally empty: for IAM Policy resources | ||
| // the entire identity is composed of the parent attributes (project, folder, | ||
| // bucket, etc.) since there is one policy per parent. | ||
| var IamPolicyBaseIdentitySchema = map[string]*schema.Schema{} |
There was a problem hiding this comment.
Will this variable be used in anywhere?
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit f73f702: Diff reportYour PR generated the following diffs in downstream repositories:
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode Action takenFound 11 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. |
d88e68f
Release Note Template for Downstream PRs (will be copied)
a follow-up of:
but focused on
iam_policySee Write release notes for guidance.