databrickslabs / pytester

Python Testing for Databricks
https://pypi.org/project/databricks-labs-pytester/
Other
51 stars 5 forks source link

Use watchdog timeout to catalog properties #48

Closed ericvergnaud closed 1 month ago

ericvergnaud commented 1 month ago

Changes

Current implementation does not mark catalogs as being in use for the watchdog to skip. This PR fills the gap by adding a RemoveAfter entry to catalog properties

Linked issues

PR https://github.com/databrickslabs/watchdog/pull/58

Tests

Not tested

github-actions[bot] commented 1 month ago

This PR breaks backwards compatibility for databrickslabs/blueprint downstream. See build logs for more details.

Running from downstreams #28

github-actions[bot] commented 1 month ago

This PR breaks backwards compatibility for databrickslabs/ucx downstream. See build logs for more details.

Running from downstreams #23

github-actions[bot] commented 1 month ago

✅ 35/35 passed, 3 skipped, 7m52s total

Running from acceptance #69

ericvergnaud commented 1 month ago

@nfx following discussion with @asnare, switched to using properties instead of name suffix

asnare commented 1 month ago

@nfx following discussion with @asnare, switched to using properties instead of name suffix

Context: review conversations from databrickslabs/ucx#2388