Dataplex-DataProduct: Adding icon support for DataProduct#18001
Conversation
|
Googlers: For automatic test runs see go/terraform-auto-test-runs. @melinath, 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. |
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit eb5b908: Diff reportYour PR generated the following diffs in downstream repositories:
Missing test reportYour PR includes resource fields which are not covered by any test. Resource: resource "google_dataplex_data_product" "primary" {
icon = # value needed
}
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode 🟢 All tests passed in Replaying mode! No Recording was needed. View the replaying VCR build log @shashambhavi, @melinath VCR tests complete for eb5b908! |
melinath
left a comment
There was a problem hiding this comment.
Could you add this to a test as well?
Probably easiest to do would be to add it to the update test so that the update path is exercised as well: https://github.com/GoogleCloudPlatform/magic-modules/blob/main/mmv1/third_party/terraform/services/dataplex/resource_dataplex_data_product_test.go
Done! Added the new field in the tests. |
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 6ea5491: Diff reportYour PR generated the following diffs in downstream repositories:
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode 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
View the replaying VCR build log Step 2: Recording Mode
🟢 All tests passed! View the recording VCR build log or the debug logs folder for detailed results. @shashambhavi, @melinath VCR tests complete for 6ea5491! |
7a04a5b
Added support for icon field in Dataplex DataProduct.