-
Notifications
You must be signed in to change notification settings - Fork 2.1k
resourceIdentity: block Identity() support from resources that exclude_read
#16058
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
base: FEATURE-BRANCH-resource-identity
Are you sure you want to change the base?
resourceIdentity: block Identity() support from resources that exclude_read
#16058
Conversation
resourceIdentity: block Identity() support from resources that exclude_readresourceIdentity: add explicit Identity() support on ExternalAccountKey
|
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: 1 Click here to see the affected service packages
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
|
|
🔴 Tests failed during RECORDING mode: 🔴 Errors occurred during RECORDING mode. Please fix them to complete your 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.
|
resourceIdentity: add explicit Identity() support on ExternalAccountKeyresourceIdentity: block Identity() support from resources that exclude_read
Tests analyticsTotal tests: 3344 Click here to see the affected service packages
Action takenFound 72 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: 🔴 Several tests terminated during RECORDING mode. 🔴 Errors occurred during RECORDING mode. Please fix them to complete your PR. |
|
/gcbrun |
|
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: 3360 Click here to see the affected service packages
Action takenFound 42 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: 🔴 Several tests terminated during RECORDING mode. 🔴 Errors occurred during RECORDING mode. Please fix them to complete your PR. |
|
we get a This is due to the custom_update being utilized for the resource I can push a commit that adds support within this PR or open an entirely new one, would have to be after the triage meeting @shuyama1 |
|
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: 3337 Click here to see the affected service packages
Action takenFound 41 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: 🔴 Several tests terminated during RECORDING mode. 🔴 Errors occurred during RECORDING mode. Please fix them to complete your PR. |
|
@shuyama1 This PR has been waiting for review for 3 weekdays. Please take a look! Use the label |
Needed to be resolved after looking over tests from feature-branch sync
Originally went with adding the identity support for this resource however we run into read identity errors due to the exclude read being set. This is something that we'd need to look into in the future as currently we do not support the ability to set custom_read code that would allow us to be explicit with the identity support in this particular case.
For now we should prevent support on
exclude_readresources until we can be explicit within read. We only have 5 resources that have this set so the missing coverage wouldn't be major.Note
Although the resources do not allow us to obtain information about the resource its still worth including identity support on them as we'd still want to be able to identify them through searching across cloud environments where the value that is used to do such a search comes from
identitybeing set in state.Release Note Template for Downstream PRs (will be copied)
See Write release notes for guidance.