Skip to content

Remove bundle/internal/tf and related code - #6932

Merged
denik merged 2 commits into
mainfrom
denik/bundle-internal-tf-removal
Oct 6, 2026
Merged

denik merged 2 commits into
mainfrom
denik/bundle-internal-tf-removal

Conversation

@denik

@denik denik commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Removes the generated Terraform provider schema (bundle/internal/tf/schema) and its codegen module (bundle/internal/tf/codegen), plus what only existed to feed or consume them:

  • The Terraform install in the acceptance harness (install_terraform.py).
  • The terraform_dabs_map generator and the generate-tf-schema / generate-schema-map tasks. generated.go stays as a frozen snapshot used by the state migration.
  • The bump-tf skill and related Taskfile, lint, deadcode, and doc entries.

This pull request and its description were written by Isaac.

@denik denik changed the title Remove bundle/internal/tf Remove bundle/internal/tf and related code Oct 5, 2026
@denik
denik force-pushed the denik/bundle-internal-tf-removal branch from 37da6c6 to c9c8db2 Compare October 5, 2026 18:40
@denik
denik changed the base branch from denik/delete-terraform to denik/remove-debug-terraform October 5, 2026 18:40
@denik
denik marked this pull request as ready for review October 5, 2026 18:44
@denik
denik requested review from a team as code owners October 5, 2026 18:44
@eng-dev-ecosystem-bot

eng-dev-ecosystem-bot commented Oct 5, 2026 •

Copy link
Copy Markdown
Collaborator

Integration test report

Commit: 75bfeaf

Run: 37466683768

Env 🔄​flaky ✅​pass 🙈​skip Time
✅​ aws linux 276 16 8:39
✅​ aws windows 278 14 5:26
✅​ azure linux 275 16 9:42
✅​ azure windows 277 14 6:07
🔄​ gcp linux 1 275 16 10:02
🔄​ gcp windows 2 276 14 6:35
Test Name gcp linux gcp windows
🔄​ TestFsCpDirToDirFileNotOverwritten ✅​p 🔄​f
🔄​ TestFsCpDirToDirFileNotOverwritten/uc-volumes_to_local ✅​p 🔄​f
🔄​ TestSyncIncrementalSyncPythonNotebookToFile 🔄​f ✅​p
Top 6 slowest tests (at least 2 minutes):
duration env testname
4:03 azure windows TestAccept
4:01 gcp linux TestAccept
4:01 azure linux TestAccept
3:50 aws linux TestAccept
3:23 gcp windows TestAccept
3:01 aws windows TestAccept

@denik
denik force-pushed the denik/remove-debug-terraform branch from 6034baf to dd4bcec Compare October 6, 2026 10:05
@denik
denik requested a review from a team as a code owner October 6, 2026 10:05
@denik
denik force-pushed the denik/bundle-internal-tf-removal branch from a30335c to bcedda7 Compare October 6, 2026 10:05
Base automatically changed from denik/remove-debug-terraform to main October 6, 2026 11:26
@denik
denik force-pushed the denik/bundle-internal-tf-removal branch from bcedda7 to cbce67f Compare October 6, 2026 12:39
@github-actions github-actions Bot added the DABs DABs related issues label Oct 6, 2026
denik and others added 2 commits October 6, 2026 14:51
The terraform provider schema and its codegen module are unused now that the
terraform engine is gone. Also remove what only existed to feed or consume it:
the terraform install in the acceptance harness and the terraform_dabs_map
generator (generated.go is kept as a frozen snapshot for the state migration).

Co-authored-by: Isaac <no-reply@databricks.com>
Co-authored-by: Isaac <no-reply@databricks.com>
@denik
denik force-pushed the denik/bundle-internal-tf-removal branch from cbce67f to 75bfeaf Compare October 6, 2026 12:53
@denik
denik enabled auto-merge October 6, 2026 12:56
@denik
denik added this pull request to the merge queue Oct 6, 2026
Merged via the queue into main with commit 7d4688f Oct 6, 2026
36 checks passed
@denik
denik deleted the denik/bundle-internal-tf-removal branch October 6, 2026 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DABs DABs related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants