Leondon9 opened a new pull request, #67060:
URL: https://github.com/apache/airflow/pull/67060
Fix variable import handling for structured falsy values.
This keeps core CLI and `airflowctl` import semantics aligned when a
variable entry is shaped like `{"value": ..., "description": ...}` and the
value is falsy, such as an empty string, `0`, `false`, or `null`.
It also makes `airflowctl variables import` reject non-object JSON input
with the existing invalid-file error instead of failing later while iterating.
Tests:
- `breeze run pytest
airflow-ctl/tests/airflow_ctl/ctl/commands/test_variable_command.py -xvs`
- `breeze run pytest
airflow-core/tests/unit/cli/commands/test_variable_command.py::TestCliVariables::test_variables_import_with_structured_falsy_values
-xvs`
- `prek run ruff --files
airflow-core/src/airflow/cli/commands/variable_command.py
airflow-core/tests/unit/cli/commands/test_variable_command.py
airflow-ctl/src/airflowctl/ctl/commands/variable_command.py
airflow-ctl/tests/airflow_ctl/ctl/commands/test_variable_command.py`
---
##### Was generative AI tooling used to co-author this PR?
- [X] Yes — Codex (GPT-5)
Generated-by: Codex (GPT-5) following [the
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
