Skip to content

Commit

Permalink
[bugfix] Fix regression in ExternalAssetNode metadata (#20558)
Browse files Browse the repository at this point in the history
## Summary & Motivation

A previous refactor introduced a regression by failing to normalize
`ExternalAssetNode` metadata with `allow_invalid=True`. This fixes the
regression.

## How I Tested These Changes

Existing test suite.
  • Loading branch information
smackesey authored Mar 19, 2024
1 parent 091f096 commit d3720b4
Showing 1 changed file with 3 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -1239,7 +1239,9 @@ def __new__(
auto_observe_interval_minutes: Optional[float] = None,
owners: Optional[Sequence[str]] = None,
):
metadata = normalize_metadata(check.opt_mapping_param(metadata, "metadata", key_type=str))
metadata = normalize_metadata(
check.opt_mapping_param(metadata, "metadata", key_type=str), allow_invalid=True
)

# backcompat logic for execution type specified via metadata
if SYSTEM_METADATA_KEY_ASSET_EXECUTION_TYPE in metadata:
Expand Down

0 comments on commit d3720b4

Please sign in to comment.