Add MetadataFeed support to Dataplex#16631
Add MetadataFeed support to Dataplex#16631MingyanZhao wants to merge 1 commit intoGoogleCloudPlatform:mainfrom
Conversation
|
Googlers: For automatic test runs see go/terraform-auto-test-runs. @c2thorn, 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. |
| base_url: 'projects/{{project}}/locations/{{location}}/metadataFeeds/{{metadata_feed_id}}' | ||
| self_link: 'projects/{{project}}/locations/{{location}}/metadataFeeds/{{metadata_feed_id}}' | ||
| create_url: 'projects/{{project}}/locations/{{location}}/metadataFeeds?metadataFeedId={{metadata_feed_id}}' | ||
| update_verb: 'PATCH' |
|
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.
Missing test reportYour PR includes resource fields which are not covered by any test. Resource: resource "google_dataplex_metadata_feed" "primary" {
filters {
aspect_types = # value needed
change_types = # value needed
entry_types = # value needed
}
labels = # value needed
pubsub_topic = # value needed
scope {
entry_groups = # value needed
projects = # value needed
}
}
|
Tests analyticsTotal tests: 84 Click here to see the affected service packages
Action takenFound 4 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. |
Uh oh!
There was an error while loading. Please reload this page.