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

Bundle summary throws an error if job is deleted #1913

Open
koopatroopa599 opened this issue Nov 19, 2024 · 2 comments
Open

Bundle summary throws an error if job is deleted #1913

koopatroopa599 opened this issue Nov 19, 2024 · 2 comments
Assignees
Labels
Bug Something isn't working DABs DABs related issues

Comments

@koopatroopa599
Copy link

Describe the issue

Deleting a job and running databricks bundle summary throws an error:

panic: runtime error: invalid memory address or nil pointer dereference [signal 0xc0000005 code=0x0 addr=0xa0 pc=0x13c44c4]

Configuration

bundle:
  name: example

include:
  - resources/*/*.yml

dev:
    mode: production
    workspace:
      host: https://xxxx.azuredatabricks.net
      root_path: /.bundle/${bundle.name}/${bundle.target}

Steps to reproduce the behavior

  1. Delete a job .yml file from the /resources/jobs/ folder
  2. Run databricks bundle summary

Expected Behavior

Command outputs the bundle summary

Actual Behavior

Command outputs an error message

OS and CLI version

Operating System: Windows 10 (22H2)
CLI version: 0.234.0

Is this a regression?

No

@koopatroopa599 koopatroopa599 added the DABs DABs related issues label Nov 19, 2024
@pietern pietern added the Bug Something isn't working label Nov 19, 2024
@pietern
Copy link
Contributor

pietern commented Nov 19, 2024

Thanks for reporting!

@PaoloClaudioGrigis
Copy link

should databricks bundle validate catch this condition? it seems it doesn't at the moment

@andrewnester andrewnester assigned andrewnester and unassigned pietern Nov 27, 2024
github-merge-queue bot pushed a commit that referenced this issue Nov 28, 2024
## Changes
This PR introduces use of new `isNil` method. It allows to ensure we
filter out all improperly defined resources in `bundle summary` command.
This includes deleted resources or resources with incorrect
configuration such as only defining key of the resource and nothing
else.

Fixes #1919, #1913

## Tests
Added regression unit test case
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working DABs DABs related issues
Projects
None yet
Development

No branches or pull requests

4 participants