databricks-cli

Author	SHA1	Message	Date
Pieter Noordhuis	61a3a03fa4	Support lookup by name of notification destinations	2024-11-21 11:29:56 +01:00
Pieter Noordhuis	14fe03dcb9	Breakout variable lookup into separate files and tests (#1921 ) ## Changes While looking into adding variable lookups for notification destinations ([API][API]), I found the codegen approach for different classes of variable lookups a bit complex. The template had a custom field override (for service principals), the package had an override for the cluster lookup, and it didn't produce tests. The notification destinations API uses a default page size of 20 for listing. I want to use a larger page size to limit the number of API calls, so that would imply another customization on the template or a manual override. This code being rather mechanical, I used copilot to produce all instances of the resolvers and their tests (after writing one of them manually). [api]: https://docs.databricks.com/api/workspace/notificationdestinations ## Tests * Unit tests pass * Manual confirmation that lookups of warehouses still work	2024-11-21 11:28:50 +01:00

Author

SHA1

Message

Date

Pieter Noordhuis

61a3a03fa4

Support lookup by name of notification destinations

2024-11-21 11:29:56 +01:00

Pieter Noordhuis

14fe03dcb9

Breakout variable lookup into separate files and tests (#1921 )

## Changes

While looking into adding variable lookups for notification destinations
([API][API]), I found the codegen approach for different classes of
variable lookups a bit complex. The template had a custom field override
(for service principals), the package had an override for the cluster
lookup, and it didn't produce tests.

The notification destinations API uses a default page size of 20 for
listing. I want to use a larger page size to limit the number of API
calls, so that would imply another customization on the template or a
manual override.

This code being rather mechanical, I used copilot to produce all
instances of the resolvers and their tests (after writing one of them
manually).

[api]: https://docs.databricks.com/api/workspace/notificationdestinations

## Tests

* Unit tests pass
* Manual confirmation that lookups of warehouses still work

2024-11-21 11:28:50 +01:00

Compare commits

2 Commits

41539f97f3 ... 61a3a03fa4

Diff Content Not Available

Compare commits

2 Commits 41539f97f3 ... 61a3a03fa4

Diff Content Not Available

2 Commits

41539f97f3 ... 61a3a03fa4