Skip to content

Fix asset materialize CLI command #48391

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

Closed
vatsrahul1001 wants to merge 2 commits into from
Closed

Fix asset materialize CLI command #48391

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
af0c3e6
fix asset materized CLI command
vatsrahul1001 Mar 26, 2025
686cd90
Merge branch 'main' into fix-asset-materialize-command
vatsrahul1001 Mar 26, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion airflow-core/src/airflow/cli/cli_config.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -903,7 +903,7 @@ class GroupCommand(NamedTuple):
ActionCommand(
name="materialize",
help="Materialize an asset",
func=lazy_load_command("airflow.cli.commands.asset_command.asset_materialize"),
func=lazy_load_command("airflow.cli.commands.remote_commands.asset_command.asset_materialize"),
Copy link
Contributor

@bugraoz93 bugraoz93 Mar 26, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is now flat in airflow-core, where all commands are under airflow-core/cli/commands. It was before like this remote_commands and local_commands. Now the split is more on airflowctl containing remote and airflow containing local commands. Everything is moved in here: #48224. Let me check too

Copy link
Member

@Lee-W Lee-W Mar 27, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh didn't notice it. I guess that means we probably don't need this one? @vatsrahul1001 could you please test it out and see whether it works well in main already? Thanks!

args=(ARG_ASSET_NAME, ARG_ASSET_URI, ARG_OUTPUT, ARG_VERBOSE),
),
)
Expand Down