Adding new `pipelines deploy` command by alyssa-db · Pull Request #3107 · databricks/cli

alyssa-db · 2025-06-23T21:02:36Z

Changes

Adds pipelines deploy command with:

--auto-approve flag for non-interactive deployments
--force-lock flag to override deployment locks
--var flag for runtime variable substitution
--verbose flag for detailed output (VSCode integration)

Why

Enables direct pipeline deployment from CLI with safety controls and CI/CD support, eliminating need to switch tools for deployment workflows.

Tests

Added acceptance tests covering:

Auto-approve behavior for destructive operations
Force lock functionality and concurrent deployment handling
Variable substitution with special characters, unicode, and precedence rules
Error cases for undefined variables and invalid formats
Basic deployment and recreation scenarios

All tests run against Terraform backend with proper cleanup.

kanterov

Agree with Andrew on not reusing the implementation as-is. I think as a principle we want to duplicate CLI commands, and customize which options they accept instead of reusing. We can extract sensible pieces of code to be shared, for example, mutator pipeline for deployment.

jefferycheng1

LGTM

jefferycheng1 · 2025-07-04T07:27:04Z

@kanterov seems like alyssa resolved the comments, forgot to re-request, mind taking another look?

## Changes Added acceptance test for rendering diagnostics warnings when unknown properties are encountered during pipeline deployment. ## Why Follow-up to [PR #3107](#3107) - ensures the diagnostics system correctly identifies and reports unknown properties in pipeline configurations, providing users with helpful feedback about potential configuration issues. ## Tests Added acceptance test case `render-diagnostics-warning` that verifies warning messages are properly displayed for unknown bundle configuration properties.

## Changes Adds `pipelines deploy` command with: - `--auto-approve` flag for non-interactive deployments - `--force-lock` flag to override deployment locks - `--var` flag for runtime variable substitution - `--verbose` flag for detailed output (VSCode integration) ## Why Enables direct pipeline deployment from CLI with safety controls and CI/CD support, eliminating need to switch tools for deployment workflows. ## Tests Added acceptance tests covering: - Auto-approve behavior for destructive operations - Force lock functionality and concurrent deployment handling - Variable substitution with special characters, unicode, and precedence rules - Error cases for undefined variables and invalid formats - Basic deployment and recreation scenarios All tests run against Terraform backend with proper cleanup.

alyssa-db requested review from andrewnester, anton-107, denik, pietern and shreyas-goenka as code owners June 23, 2025 21:02

alyssa-db temporarily deployed to test-trigger-is June 23, 2025 21:02 — with GitHub Actions Inactive

alyssa-db commented Jun 23, 2025
View reviewed changes

Comment threadacceptance/pipelines/install-pipelines-cli/output.txt Outdated

andrewnester requested a review from kanterov June 24, 2025 10:04

andrewnester reviewed Jun 24, 2025
View reviewed changes

Comment threadcmd/pipelines/deploy.go Outdated

kanterov reviewed Jun 24, 2025
View reviewed changes

deploy feat
365a31e

alyssa-db temporarily deployed to test-trigger-is June 24, 2025 21:53 — with GitHub Actions Inactive

alyssa-db force-pushed the pipelines-deploy branch from 2668f34 to 365a31eCompare June 24, 2025 22:02

alyssa-db temporarily deployed to test-trigger-is June 24, 2025 22:02 — with GitHub Actions Inactive

alyssa-db temporarily deployed to test-trigger-is June 25, 2025 00:30 — with GitHub Actions Inactive

modified deploy function
64ef843

alyssa-db force-pushed the pipelines-deploy branch from 5484a66 to 64ef843Compare June 25, 2025 00:33

alyssa-db temporarily deployed to test-trigger-is June 25, 2025 00:33 — with GitHub Actions Inactive

alyssa-db requested review from andrewnester, jefferycheng1 and kanterov June 25, 2025 09:04

andrewnester reviewed Jun 25, 2025
View reviewed changes

Comment threadcmd/bundle/deploy.go Outdated
Comment threadcmd/bundle/variables.go Outdated

modified bundle import functions
50e8b4b

alyssa-db temporarily deployed to test-trigger-is June 25, 2025 17:05 — with GitHub Actions Inactive

alyssa-db requested a review from andrewnester June 25, 2025 18:47

Merge branch 'main' into pipelines-deploy
2e786c6

alyssa-db temporarily deployed to test-trigger-is June 26, 2025 18:27 — with GitHub Actions Inactive

added init dependent tests
18950dd

alyssa-db temporarily deployed to test-trigger-is June 26, 2025 23:11 — with GitHub Actions Inactive

alyssa-db temporarily deployed to test-trigger-is June 27, 2025 04:00 — with GitHub Actions Inactive

Merge branch 'main' into pipelines-deploy
02a2a51

alyssa-db temporarily deployed to test-trigger-is July 3, 2025 14:04 — with GitHub Actions Inactive

tests
0a9a6ba

alyssa-db temporarily deployed to test-trigger-is July 3, 2025 18:42 — with GitHub Actions Inactive

unexporting
0919062

alyssa-db temporarily deployed to test-trigger-is July 3, 2025 19:42 — with GitHub Actions Inactive

Merge branch 'main' into pipelines-deploy
a70a18b

alyssa-db temporarily deployed to test-trigger-is July 3, 2025 19:43 — with GitHub Actions Inactive

tests
7db01ba

alyssa-db temporarily deployed to test-trigger-is July 4, 2025 03:04 — with GitHub Actions Inactive

alyssa-db mentioned this pull request Jul 4, 2025
Add diagnostics warning test for pipelines deploy #3179
Merged

jefferycheng1 approved these changes Jul 4, 2025
View reviewed changes

Comment threadacceptance/pipelines/deploy/create-pipeline/script

jefferycheng1 requested a review from kanterov July 4, 2025 07:25

Merge branch 'main' into pipelines-deploy
20da5cc

alyssa-db temporarily deployed to test-trigger-is July 5, 2025 05:53 — with GitHub Actions Inactive

alyssa-db temporarily deployed to test-trigger-is July 5, 2025 05:56 — with GitHub Actions Inactive

new out.test.toml files
0e0caf5

alyssa-db temporarily deployed to test-trigger-is July 5, 2025 06:14 — with GitHub Actions Inactive

Merge branch 'main' into pipelines-deploy
3f88a0b

alyssa-db temporarily deployed to test-trigger-is July 7, 2025 13:22 — with GitHub Actions Inactive

kanterov approved these changes Jul 7, 2025
View reviewed changes

Merge branch 'main' into pipelines-deploy
81ca535

alyssa-db temporarily deployed to test-trigger-is July 7, 2025 14:45 — with GitHub Actions Inactive

added pipelines check
897ba26

alyssa-db temporarily deployed to test-trigger-is July 7, 2025 20:59 — with GitHub Actions Inactive

andrewnester merged commit 4ce207e into mainJul 8, 2025
21 of 23 checks passed

andrewnester deleted the pipelines-deploy branch July 8, 2025 09:13

Navigation Menu

Uh oh!

Adding new `pipelines deploy` command - #3107

Adding new `pipelines deploy` command#3107
andrewnester merged 25 commits into
mainfrom
pipelines-deploy

alyssa-db commented Jun 23, 2025•
edited
Loading

Uh oh!

Uh oh!

Uh oh!

kanterov left a comment

Uh oh!

Uh oh!

Uh oh!

jefferycheng1 left a comment

Uh oh!

Uh oh!

jefferycheng1 commented Jul 4, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Uh oh!

Conversation

alyssa-db commented Jun 23, 2025• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Changes

Why

Tests

Uh oh!

Uh oh!

Uh oh!

kanterov left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

jefferycheng1 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

jefferycheng1 commented Jul 4, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

alyssa-db commented Jun 23, 2025•
edited
Loading