Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DO NOT MERGE]readd parent path #1990

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

VictorAtIfInsurance
Copy link

Changes

The parent path flag to the lakeview create command has been removed in some recent cli version.

> databricks --version
Databricks CLI v0.236.0
> databricks lakeview create --help
Create dashboard.
  
  Create a draft dashboard.

Usage:
  databricks lakeview create [flags]

Flags:
      --display-name string           The display name of the dashboard.
  -h, --help                          help for create
      --json JSON                     either inline JSON string or @path/to/file.json with request body (default JSON (0 bytes))
      --serialized-dashboard string   The contents of the dashboard in serialized string form.
      --warehouse-id string           The warehouse ID used to run the dashboard.

Global Flags:
      --debug            enable debug logging
  -o, --output type      output type: text or json (default text)
  -p, --profile string   ~/.databrickscfg profile
  -t, --target string    bundle target to use (if applicable)

But it is a valid flag in e.g. v0.232.0.

It should probably still be part of the cli as it is a valid bundle argument, and it still exist in the REST API under /api/2.0/lakeview/dashboards

As this file is automatically generated this should not be merged, but created the PR for reference.

Tests

Copy link

github-actions bot commented Dec 9, 2024

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/cli

Inputs:

  • PR number: 1990
  • Commit SHA: 412f336cb107a6cc68b17fe8c91347c38cd3a99a

Checks will be approved automatically on success.

@andrewnester andrewnester added the CLI CLI related issues label Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLI CLI related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants