Changed intelligence config name field from url_param_only to ignore_read#16483
Conversation
|
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: 6 Click here to see the affected service packages
Action takenFound 6 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. 🟢 All tests passed! |
|
@modular-magician reassign-reviewer |
|
Googlers: For automatic test runs see go/terraform-auto-test-runs. @hao-nan-li, 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. |
4c6956c
context: yaqs/4010639086615265280
The primary purpose of this change is to ensure the field is properly detected as having an API equivalent. The API returns a full path with a project number instead of project id, so there's no point in trying to have drift detection on this field - hence, ignore_read.
Release Note Template for Downstream PRs (will be copied)
See Write release notes for guidance.