Skip to content
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

Custom suppress diff for storage in databricks_pipeline #1574

Merged
merged 1 commit into from
Aug 31, 2022

Conversation

alexott
Copy link
Contributor

@alexott alexott commented Aug 31, 2022

When the storage option isn't provided, DLT generates a temp one in the form of
dbfs:/pipelines/<uuid>. But in the sequential apply the drift was detected, and
pipeline was recreated. Marking it with computed or suppress_diff doesn't help
because it prevents the pipeline from the detection of change of storage from value
to null.

@alexott alexott requested a review from a team August 31, 2022 11:46
@nfx nfx enabled auto-merge (squash) August 31, 2022 13:15
@nfx nfx force-pushed the dlt-storage-suppress-diff branch from e238cab to 89c30b7 Compare August 31, 2022 13:16
When the `storage` option isn't provided, DLT generates a temp one in the form of
`dbfs:/pipelines/<uuid>`.  But in the sequential apply the drift was detected, and
pipeline was recreated.  Marking it with `computed` or `suppress_diff` doesn't help
because it prevents the pipeline from the detection of change of `storage` from value
to null.
@nfx nfx force-pushed the dlt-storage-suppress-diff branch from 89c30b7 to b8d1522 Compare August 31, 2022 13:17
@nfx nfx merged commit a723577 into master Aug 31, 2022
@nfx nfx mentioned this pull request Sep 12, 2022
@nkvuong nkvuong deleted the dlt-storage-suppress-diff branch October 20, 2022 08:37
michael-berk pushed a commit to michael-berk/terraform-provider-databricks that referenced this pull request Feb 15, 2023
…ks#1574)

When the `storage` option isn't provided, DLT generates a temp one in the form of
`dbfs:/pipelines/<uuid>`.  But in the sequential apply the drift was detected, and
pipeline was recreated.  Marking it with `computed` or `suppress_diff` doesn't help
because it prevents the pipeline from the detection of change of `storage` from value
to null.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants