databricks / cli

Databricks CLI
Other
150 stars 56 forks source link

Upgrade TF provider to 1.58.0 #1900

Closed pietern closed 1 week ago

pietern commented 1 week ago

Changes

Notable changes:

See:

Tests

Integration tests pass.

github-actions[bot] commented 1 week ago

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger: go/deco-tests-run/cli

Inputs:

Checks will be approved automatically on success.

eng-dev-ecosystem-bot commented 1 week ago

Test Details: go/deco-tests/11837584085

pietern commented 1 week ago

Beware: this bump does NOT include the proposed changes to the quality_monitor resource.

The TF provider implementation was changed to use the plugin framework for this resource, and since then, it no longer includes whether a nested object is a single block (list with max 1 item) or multiple blocks (list with any number of items).

We use the former condition to inline a nested object as a pointer type. Without this condition, we must embed it as a slice type, and we can no longer convert between our representation and the TF representation.

I manually confirmed that we can continue to use the nester object as a pointer type. TF performs the conversion (I think) and interprets the nested object correctly as a list of size 1.