-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Fix cross-project listing subscriptions #13362
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
Fix cross-project listing subscriptions #13362
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: 14 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. |
Is this WIP? |
It is failing on the import step which I still need to fix. Working on that now |
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: 14 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
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All existing TestAccBigqueryAnalyticsHub tests passed in a manual Teamcity build
a8699fd
fixes hashicorp/terraform-provider-google#21402
google_bigquery_analytics_hub_listing_subscription is unfortunately unique in that the create_url and self_link differ completely. What was missed in the original implementation was a cross-project setup that would cause one project needed for the create_url, but a separate one for the self_link.
This fix rectifies the issue by inserting pre_read and pre_delete code to overwrite the template URL to use the correct destination dataset project instead of the
project
field.Release Note Template for Downstream PRs (will be copied)
See Write release notes for guidance.