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.

Sign up for GitHub

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

Jump to bottom

"storage-credentials list" has a broken template #1029

Closed
docwhat opened this issue Nov 29, 2023 · 5 comments · Fixed by #1094
Closed

"storage-credentials list" has a broken template #1029

docwhat opened this issue Nov 29, 2023 · 5 comments · Fixed by #1094
Labels
Bug Something isn't working CLI CLI related issues

Comments

@docwhat
Copy link

docwhat commented Nov 29, 2023

databricks storage-credentials list dies on an error due to a template error:

$ databricks storage-credentials list
Error: template: command:2:169: executing "command" at <.GcpServiceAccountKey>: can't evaluate field GcpServiceAccountKey in type catalog.StorageCredentialInfo
@docwhat docwhat added the CLI CLI related issues label Nov 29, 2023
@andrewnester andrewnester linked a pull request Nov 30, 2023 that will close this issue
fix: gcp_service_account_key is not available #910
Closed
@andrewnester andrewnester added the Bug Something isn't working label Nov 30, 2023
@hargut
Copy link

hargut commented Dec 19, 2023

Seeing the same error tying to use the command on Azure.

@docwhat
Copy link
Author

docwhat commented Dec 19, 2023

My PR (#910) needs me to sign a CLA (Contribution License Agreement) or for someone to approve it anyway (I just deleted a bit of the template).

I don't see any information about a CLA. Maybe someone can point me at it?

@NodeJSmith
Copy link

FYI - using --output json will let you get the data in the meantime, since the bug is just in the text rendering, not in the actual request.

@docwhat
Copy link
Author

docwhat commented Dec 19, 2023

@NodeJSmith Oooh... Thanks. I didn't think of that.

@hargut hargut mentioned this issue Dec 20, 2023
Closed
@andrewnester andrewnester mentioned this issue Dec 28, 2023
Merged
github-merge-queue bot pushed a commit that referenced this issue Jan 2, 2024
@andrewnester
Fixed storage-credentials list command in text output (#1094)
3e6e048 
## Changes
Fixes #1029 
Closes #910 

## Tests
Added regression test
@andrewnester
Copy link
Contributor

The fix has been merged and will be released in the upcoming CLI release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Something isn't working CLI CLI related issues
Projects
None yet
Milestone
No milestone
4 participants
@docwhat @hargut @andrewnester @NodeJSmith