Feature: Create datalineage config provider#17073
Feature: Create datalineage config provider#17073pawelmarut wants to merge 2 commits intoGoogleCloudPlatform:mainfrom
Conversation
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
|
Googlers: For automatic test runs see go/terraform-auto-test-runs. @malhotrasagar2212, 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. |
afd15cc to
6ae490b
Compare
|
@slevenick This PR has been waiting for review for 3 weekdays. Please take a look! Use the label |
9157862 to
dd3b07b
Compare
dd3b07b to
049dc90
Compare
|
@modular-magician reassign-reviewer |
|
Reassigned the review to @malhotrasagar2212, as the original reviewer is OOO |
|
|
||
| examples: | ||
| - name: 'data_lineage_config_basic' | ||
| config_path: 'templates/terraform/examples/data_lineage_config_basic.tf.tmpl' |
There was a problem hiding this comment.
config_path is not needed, as the file templates/terraform/examples/data_lineage_config_basic.tf.tmpl will be used to generate the test automatically by matching the example name.
| Integration to which the rule applies. | ||
| required: true | ||
| enum_values: | ||
| - 'INTEGRATION_UNSPECIFIED' |
There was a problem hiding this comment.
INTEGRATION_UNSPECIFIED is meaningless and can be removed.
| @@ -0,0 +1,23 @@ | |||
| # Copyright 2024 Google Inc. | |||
There was a problem hiding this comment.
| # Copyright 2024 Google Inc. | |
| # Copyright 2026 Google Inc. |
| custom_code: | ||
| test_check_destroy: 'templates/terraform/custom_check_destroy/datalineage_config.go.tmpl' | ||
|
|
||
| examples: |
There was a problem hiding this comment.
Can you also add the examples for folder config and org config?
|
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 service labelsThe following new resources do not have corresponding service labels:
If you believe this detection to be incorrect please raise the concern with your reviewer. Googlers: This error is safe to ignore once you've completed go/fix-missing-service-labels. |
Tests analyticsTotal tests: 6302 Click here to see the affected service packages
Action takenFound 16 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: 🔴 Errors occurred during RECORDING mode. Please fix them to complete your PR. |
|
Hi, |
|
I enabled the datalineage API |
|
/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.
Missing service labelsThe following new resources do not have corresponding service labels:
If you believe this detection to be incorrect please raise the concern with your reviewer. Googlers: This error is safe to ignore once you've completed go/fix-missing-service-labels. |
Tests analyticsTotal tests: 6311 Click here to see the affected service packages
Action takenFound 12 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: 🔴 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.
Missing service labelsThe following new resources do not have corresponding service labels:
If you believe this detection to be incorrect please raise the concern with your reviewer. Googlers: This error is safe to ignore once you've completed go/fix-missing-service-labels. |
Terraform resource and datasource for data lineage configuration.
Release Note Template for Downstream PRs (will be copied)