Display plugin setup instructions after `apps init` flow, remove empty files after template rendering by pkosiec · Pull Request #4549 · databricks/cli

pkosiec · 2026-02-19T09:38:49Z

Changes

Show setup notes for plugins which have such instructions (needed for Lakebase plugin temporarily)
Remove hardcoded paths for analytics plugin - this will be handled purely by Go templates in the template itself
Remove empty files and directories after template rendering
Setup listers for postgres resource
~~Run "postinit" npm command in context of template if it exists~~ - reverted as we don't need it for now

Those changes need to be synchronized with AppKit where we add the templatePaths property in the manifest.

Why

We're introducing new plugins, and depending on them, we need to configure the dynamic template. The hardcoded solution is not scalable, as I pointed out here: feat: use plugin manifest #4469 (comment)

See also https://databricks.atlassian.net/browse/LKB-9681

Screenshot

Demo

(just the listers for future Lakebase plugin changes databricks/appkit#132)

Screen.Recording.2026-02-24.at.15.43.45.mov

eng-dev-ecosystem-bot · 2026-02-19T09:48:27Z

Commit: 9d74e9d

Run: 22479451774

	Env	🟨KNOWN	💚RECOVERED	🙈SKIP	✅pass	🙈skip	Time
🟨	aws linux	7	1	7	268	768	7:52
🟨	aws windows	7	1	7	270	766	5:52
💚	aws-ucws linux		8	7	364	684	5:59
💚	aws-ucws windows		8	7	366	682	5:40
💚	azure linux		2	9	271	766	7:25
💚	azure windows		2	9	273	764	4:18
💚	azure-ucws linux		2	9	369	680	9:50
💚	azure-ucws windows		2	9	371	678	6:25
💚	gcp linux		2	9	267	769	6:22
💚	gcp windows		2	9	269	767	5:14

15 interesting tests: 7 KNOWN, 7 SKIP, 1 RECOVERED

	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	💚R	💚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	💚R	💚R	💚R	💚R	💚R	💚R	💚R	💚R

Top 24 slowest tests (at least 2 minutes):

duration	env	testname
5:29	azure-ucws linux	TestSecretsPutSecretStringValue
3:51	gcp linux	TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=terraform
3:38	gcp windows	TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=direct
3:22	aws linux	TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=terraform
3:12	azure-ucws windows	TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=direct
3:12	gcp windows	TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=terraform
3:09	gcp linux	TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=direct
2:58	azure linux	TestSecretsPutSecretStringValue
2:48	aws linux	TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=direct
2:43	aws windows	TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=terraform
2:43	aws-ucws linux	TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=terraform
2:40	aws-ucws windows	TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=terraform
2:39	aws windows	TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=direct
2:36	aws-ucws linux	TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=direct
2:34	aws-ucws windows	TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=direct
2:22	gcp linux	TestSecretsPutSecretStringValue
2:16	azure-ucws linux	TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=terraform
2:16	azure linux	TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=terraform
2:16	azure-ucws windows	TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=terraform
2:14	azure windows	TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=direct
2:11	azure windows	TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=terraform
2:08	azure linux	TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=direct
2:07	azure-ucws linux	TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=direct
2:05	aws linux	TestSecretsPutSecretStringValue

…y files after template rendering (#4549) ## Changes - Show setup notes for plugins which have such instructions (needed for Lakebase plugin temporarily) - Remove hardcoded paths for analytics plugin - this will be handled purely by Go templates in the template itself - Remove empty files and directories after template rendering - Setup listers for `postgres` resource - ~Run "postinit" npm command in context of template if it exists~ - reverted as we don't need it for now Those changes need to be synchronized with AppKit where we add the `templatePaths` property in the manifest. ## Why - We're introducing new plugins, and depending on them, we need to configure the dynamic template. The hardcoded solution is not scalable, as I pointed out here: #4469 (comment) See also https://databricks.atlassian.net/browse/LKB-9681 ## Screenshot <img width="1461" height="738" alt="image" src="https://github.com/user-attachments/assets/966f8f84-94a4-4d4a-856e-7f5cdc703a57" /> ## Demo (just the listers for future Lakebase plugin changes databricks/appkit#132) https://github.com/user-attachments/assets/c745baad-662b-4f82-b197-ed95fcb89ef7

pkosiec temporarily deployed to test-trigger-is February 19, 2026 09:39 — with GitHub Actions Inactive

pkosiec temporarily deployed to test-trigger-is February 19, 2026 13:07 — with GitHub Actions Inactive

pkosiec force-pushed the pkosiec/rm-hardcoded-plugin-paths branch from be69f43 to 4814591 Compare February 20, 2026 09:52

pkosiec temporarily deployed to test-trigger-is February 20, 2026 09:52 — with GitHub Actions Inactive

pkosiec changed the title ~~Read plugin-owned paths from manifest during the apps init~~ Run a postinit template command after apps init flow Feb 20, 2026

pkosiec temporarily deployed to test-trigger-is February 20, 2026 12:21 — with GitHub Actions Inactive

pkosiec temporarily deployed to test-trigger-is February 20, 2026 17:07 — with GitHub Actions Inactive

pkosiec force-pushed the pkosiec/rm-hardcoded-plugin-paths branch from f568792 to 6f37920 Compare February 23, 2026 12:47

pkosiec temporarily deployed to test-trigger-is February 23, 2026 12:47 — with GitHub Actions Inactive

pkosiec force-pushed the pkosiec/rm-hardcoded-plugin-paths branch from 6f37920 to b0d7806 Compare February 23, 2026 16:50

pkosiec temporarily deployed to test-trigger-is February 23, 2026 16:50 — with GitHub Actions Inactive

pkosiec changed the title ~~Run a postinit template command after apps init flow~~ Display plugin setup instructions after apps init flow Feb 23, 2026

pkosiec changed the title ~~Display plugin setup instructions after apps init flow~~ Display plugin setup instructions after apps init flow, remove hardcoded plugin paths Feb 23, 2026

pkosiec temporarily deployed to test-trigger-is February 23, 2026 16:56 — with GitHub Actions Inactive

pkosiec changed the title ~~Display plugin setup instructions after apps init flow, remove hardcoded plugin paths~~ Display plugin setup instructions after apps init flow Feb 23, 2026

pkosiec changed the title ~~Display plugin setup instructions after apps init flow~~ Display plugin setup instructions after apps init flow, remove empty files after template rendering Feb 23, 2026

pkosiec force-pushed the pkosiec/rm-hardcoded-plugin-paths branch from 4218939 to ba9c6f2 Compare February 23, 2026 16:59

pkosiec marked this pull request as ready for review February 23, 2026 17:00

pkosiec requested a review from a team as a code owner February 23, 2026 17:00

pkosiec temporarily deployed to test-trigger-is February 23, 2026 17:00 — with GitHub Actions Inactive

This was referenced Feb 23, 2026

feat(appkit): introduce Lakebase plugin databricks/appkit#126

Merged

chore: generate template dynamically for specific enabled plugins databricks/appkit#127

Merged

[DO NOT MERGE] Full Lakebase plugin + dynamic template changes databricks/appkit#128

Closed

pkosiec force-pushed the pkosiec/rm-hardcoded-plugin-paths branch from ba9c6f2 to 17fd2f5 Compare February 24, 2026 14:48

pkosiec temporarily deployed to test-trigger-is February 24, 2026 14:48 — with GitHub Actions Inactive

pkosiec added 4 commits February 27, 2026 09:52

Read plugin-owned paths from manifest during the apps init

f20a927

Run an optional "postinit" command in a template directory

9157f79

Refactor the templateVars, skip empty files

df3a1a6

Remove empty dirs

3f2b940

pkosiec added 4 commits February 27, 2026 09:52

show setup instructions

f066fcc

Remove unnecessary postinit function execution

e3ab851

Support postgres resource for upcoming plugins

f8e51ab

Bring back app.yaml env generation

9d74e9d

pkosiec force-pushed the pkosiec/rm-hardcoded-plugin-paths branch from 17fd2f5 to 9d74e9d Compare February 27, 2026 08:52

pkosiec temporarily deployed to test-trigger-is February 27, 2026 08:52 — with GitHub Actions Inactive

MarioCadenas approved these changes Feb 27, 2026

View reviewed changes

pkosiec enabled auto-merge February 27, 2026 09:10

pkosiec added this pull request to the merge queue Feb 27, 2026

Merged via the queue into main with commit 93db3ca Feb 27, 2026
18 checks passed

pkosiec deleted the pkosiec/rm-hardcoded-plugin-paths branch February 27, 2026 09:22

Navigation Menu

Repository navigation

Display plugin setup instructions after `apps init` flow, remove empty files after template rendering - #4549

Display plugin setup instructions after `apps init` flow, remove empty files after template rendering#4549
pkosiec merged 8 commits into
mainfrom
pkosiec/rm-hardcoded-plugin-paths

pkosiec commented Feb 19, 2026 •

edited

Loading

Uh oh!

eng-dev-ecosystem-bot commented Feb 19, 2026 •

edited

Loading

Uh oh!

Uh oh!

This branch was previously deployed

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

pkosiec commented Feb 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Changes

Why

Screenshot

Demo

Uh oh!

eng-dev-ecosystem-bot commented Feb 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

This branch was previously deployed

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

pkosiec commented Feb 19, 2026 •

edited

Loading

eng-dev-ecosystem-bot commented Feb 19, 2026 •

edited

Loading