Skip to content

Pin apps init to appkit v0.7.4 by default#4585

Open
arsenyinfo wants to merge 1 commit intomainfrom
pin-appkit-default-version
Open

Pin apps init to appkit v0.7.4 by default#4585
arsenyinfo wants to merge 1 commit intomainfrom
pin-appkit-default-version

Conversation

@arsenyinfo
Copy link
Contributor

Summary

  • Default databricks apps init now clones appkit tag v0.7.4 instead of main
  • --version latest still points to main branch for bleeding edge
  • --version <REF> and --branch <REF> work as before

Test plan

  • Unit tests pass (go test ./cmd/apps/)
  • Manual test: default → clones v0.7.4 (48 files)
  • Manual test: --version latest → clones main (41 files)
  • Manual test: --version v0.6.0 → clones v0.6.0 (41 files)
  • Manual test: --branch main → clones main (41 files)

🤖 Generated with Claude Code

@arsenyinfo arsenyinfo requested a review from a team as a code owner February 24, 2026 11:34
@arsenyinfo arsenyinfo force-pushed the pin-appkit-default-version branch from 9ba12ef to 42c0a2e Compare February 24, 2026 11:37
@arsenyinfo arsenyinfo force-pushed the pin-appkit-default-version branch from 42c0a2e to 975d0f2 Compare February 24, 2026 11:39
@eng-dev-ecosystem-bot
Copy link
Collaborator

eng-dev-ecosystem-bot commented Feb 24, 2026

Commit: 18341fd

Run: 22351561867

Env 🟨​KNOWN 🔄​flaky 💚​RECOVERED 🙈​SKIP ✅​pass 🙈​skip Time
🟨​ aws linux 7 1 7 268 763 7:52
🟨​ aws windows 7 1 7 270 761 6:37
🔄​ aws-ucws linux 2 6 7 364 679 7:19
💚​ aws-ucws windows 8 7 366 677 5:29
💚​ azure linux 2 9 271 761 5:43
💚​ azure windows 2 9 273 759 5:18
💚​ azure-ucws linux 2 9 369 675 9:02
💚​ azure-ucws windows 2 9 371 673 7:20
💚​ gcp linux 2 9 267 764 6:53
💚​ gcp windows 2 9 269 762 5:48
15 interesting tests: 7 KNOWN, 7 SKIP, 1 flaky
Test Name aws linux aws windows aws-ucws linux aws-ucws windows azure linux azure windows azure-ucws linux azure-ucws windows gcp linux gcp windows
🟨​ TestAccept 🟨​K 🟨​K 🔄​f 💚​R 💚​R 💚​R 💚​R 💚​R 💚​R 💚​R
🙈​ TestAccept/bundle/resources/permissions 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S
🟨​ TestAccept/bundle/resources/permissions/jobs/destroy_without_mgmtperms/with_permissions 🟨​K 🟨​K 💚​R 💚​R 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S
🟨​ TestAccept/bundle/resources/permissions/jobs/destroy_without_mgmtperms/with_permissions/DATABRICKS_BUNDLE_ENGINE=direct 🟨​K 🟨​K 💚​R 💚​R
🟨​ TestAccept/bundle/resources/permissions/jobs/destroy_without_mgmtperms/with_permissions/DATABRICKS_BUNDLE_ENGINE=terraform 🟨​K 🟨​K 💚​R 💚​R
🟨​ TestAccept/bundle/resources/permissions/jobs/destroy_without_mgmtperms/without_permissions 🟨​K 🟨​K 💚​R 💚​R 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S
🟨​ TestAccept/bundle/resources/permissions/jobs/destroy_without_mgmtperms/without_permissions/DATABRICKS_BUNDLE_ENGINE=direct 🟨​K 🟨​K 💚​R 💚​R
🟨​ TestAccept/bundle/resources/permissions/jobs/destroy_without_mgmtperms/without_permissions/DATABRICKS_BUNDLE_ENGINE=terraform 🟨​K 🟨​K 💚​R 💚​R
🙈​ TestAccept/bundle/resources/postgres_branches/basic 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S
🙈​ TestAccept/bundle/resources/postgres_branches/recreate 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S
🙈​ TestAccept/bundle/resources/postgres_branches/update_protected 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S
🙈​ TestAccept/bundle/resources/postgres_branches/without_branch_id 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S
🙈​ TestAccept/bundle/resources/postgres_endpoints/recreate 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S
🙈​ TestAccept/bundle/resources/synced_database_tables/basic 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S
🔄​ TestAccept/ssh/connection 💚​R 💚​R 🔄​f 💚​R 💚​R 💚​R 💚​R 💚​R 💚​R 💚​R
Top 22 slowest tests (at least 2 minutes):
duration env testname
4:08 gcp linux TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=direct
3:45 azure-ucws windows TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=terraform
3:37 gcp windows TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=direct
3:15 azure-ucws linux TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=terraform
3:10 gcp windows TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=terraform
3:08 gcp linux TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=terraform
2:57 aws linux TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=terraform
2:49 aws-ucws windows TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=terraform
2:48 aws windows TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=terraform
2:47 aws-ucws linux TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=terraform
2:45 aws linux TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=direct
2:45 aws-ucws windows TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=direct
2:45 aws windows TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=direct
2:42 aws-ucws linux TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=direct
2:27 gcp linux TestSecretsPutSecretStringValue
2:14 azure linux TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=terraform
2:11 azure windows TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=direct
2:11 aws linux TestSecretsPutSecretStringValue
2:11 azure windows TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=terraform
2:06 azure-ucws linux TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=direct
2:06 azure-ucws windows TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=direct
2:04 azure linux TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=direct

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@arsenyinfo arsenyinfo force-pushed the pin-appkit-default-version branch from 975d0f2 to 18341fd Compare February 24, 2026 12:49
templatePathEnvVar = "DATABRICKS_APPKIT_TEMPLATE_PATH"
appkitRepoURL = "https://github.com/databricks/appkit"
appkitTemplateDir = "template"
appkitDefaultVersion = "v0.7.4"
Copy link
Member

Choose a reason for hiding this comment

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

The tag v0.7.4 points to AppKit package release. The template from that tag still uses 0.6.0 https://github.com/databricks/appkit/blob/1cf359641efca70abd287e6f4fd2c091a62fa713/template/package.json#L34-L35

We should use template-vx.y.z tag, but apparently we have some issues with the CI job that doesn't push them, so we'll need to push it manually.

The question is: does it mean you folks were evaluating the template with AppKit 0.6?

Copy link
Member

Choose a reason for hiding this comment

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

Have we agreed when exactly we can bump the tag? What are the prerequisites to make sure it's the new default version?

@MarioCadenas
Copy link
Contributor

@arsenyinfo can you rebase with main as we now have the --version have working properly, and make sure we point to a template-vX.X.X tag instead of the version tag? thank you!

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.

4 participants