Skip to content

Conversation

@alexott
Copy link
Contributor

@alexott alexott commented Nov 30, 2025

Changes

This includes support for:

  • databricks_account_network_policy
  • databricks_workspace_network_option

Tests

  • make test run locally
  • relevant change in docs/ folder
  • covered with integration tests in internal/acceptance
  • using Go SDK
  • using TF Plugin Framework
  • has entry in NEXT_CHANGELOG.md file

This includes support for:

- `databricks_account_network_policy`
- `databricks_workspace_network_option`
@alexott alexott requested review from a team as code owners November 30, 2025 10:06
@alexott alexott requested review from renaudhartert-db and removed request for a team November 30, 2025 10:06
@alexott alexott added exporter TF configuration generator ai-assisted labels Dec 2, 2025
Copy link
Contributor

@rauchy rauchy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add some test coverage?

@alexott
Copy link
Contributor Author

alexott commented Dec 3, 2025

Regarding tests, the current test framework doesn't work for account-level resources. It requires a completely new implementation

@alexott alexott requested a review from rauchy December 3, 2025 14:36
@alexott alexott temporarily deployed to test-trigger-is December 8, 2025 15:27 — with GitHub Actions Inactive
@github-actions
Copy link

github-actions bot commented Dec 8, 2025

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

Trigger:
go/deco-tests-run/terraform

Inputs:

  • PR number: 5238
  • Commit SHA: da1242bc342e672c87e9c61d44bb38ef533bf9be

Checks will be approved automatically on success.

@alexott alexott temporarily deployed to test-trigger-is December 8, 2025 15:28 — with GitHub Actions Inactive
@alexott alexott requested a review from tanmay-db December 8, 2025 15:35
@alexott
Copy link
Contributor Author

alexott commented Dec 8, 2025

@rauchy I've added tests, and then #5262 adds even more

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-assisted exporter TF configuration generator

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants