Skip to content

Conversation

alexott
Copy link
Contributor

@alexott alexott commented Aug 19, 2025

Changes

In the upcoming changes in Groups API, the List API won't return members attribute, so reimplemented databricks_group data source to use List only to get id of the group, and then perform call to Get API to obtain all needed information.

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

…ion of List + Get API

In the upcoming changes in Groups API, the List API won't return `members` attribute, so
reimplemented `databricks_group` data source to use `List` only to get `id` of the group,
and then perform call to Get API to obtain all needed information.
@alexott alexott requested review from a team as code owners August 19, 2025 08:18
@alexott alexott requested review from renaudhartert-db and removed request for a team August 19, 2025 08:18
@alexott alexott temporarily deployed to test-trigger-is August 19, 2025 08:18 — with GitHub Actions Inactive
@alexott alexott temporarily deployed to test-trigger-is August 19, 2025 08:19 — with GitHub Actions Inactive
@alexott alexott enabled auto-merge August 19, 2025 12:42
@alexott alexott temporarily deployed to test-trigger-is August 28, 2025 10:41 — with GitHub Actions Inactive
@alexott alexott temporarily deployed to test-trigger-is August 28, 2025 10:41 — with GitHub Actions Inactive
Copy link

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: 4947
  • Commit SHA: 011a5c98ce3c1cfe5baba597d88352851d3087c2

Checks will be approved automatically on success.

@renaudhartert-db renaudhartert-db requested review from mgyucht and removed request for renaudhartert-db August 28, 2025 12:56
@alexott alexott added this pull request to the merge queue Aug 28, 2025
Merged via the queue into main with commit 6d62aef Aug 28, 2025
12 checks passed
@alexott alexott deleted the internal/group-data-source-not-rely-on-list branch August 28, 2025 13:50
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