Skip to content

Conversation

@amoghrajesh
Copy link
Contributor

Building up on top of #57187 and #57189, there are no more usages of these utilities.

Adding a deprecation warning to warn users to move away from using these if they are using it. There are some related unit tests too, which I am not so sure if we should remove now or later because technically this bit of code still will work until we wipe it off the repo.

This is how the deprecation warning looks like:

execute_in_subprocess(cmd=["bash", "-c", "echo CAT; echo KITTY;"])
<ipython-input-3-0c1513dd948a>:1 DeprecatedImportWarning: The function `execute_in_subprocess` is deprecated and will be removed in a future version.
/Users/amoghdesai/Documents/OSS/repos/airflow/airflow-core/src/airflow/utils/process_utils.py:188 DeprecatedImportWarning: The function `execute_in_subprocess_with_kwargs` is deprecated and will be removed in a future version.
[2025-10-24T11:18:13.833774Z] {process_utils.py:209} INFO - Executing cmd: bash -c 'echo CAT; echo KITTY;'
[2025-10-24T11:18:13.844192Z] {process_utils.py:213} INFO - Output:
[2025-10-24T11:18:13.851401Z] {process_utils.py:217} INFO - CAT
[2025-10-24T11:18:13.851532Z] {process_utils.py:217} INFO - KITTY
execute_in_subprocess_with_kwargs(cmd=["bash", "-c", "echo CAT; echo KITTY;"])
<ipython-input-4-46ee57e90afd>:1 DeprecatedImportWarning: The function `execute_in_subprocess_with_kwargs` is deprecated and will be removed in a future version.
[2025-10-24T11:18:23.570528Z] {process_utils.py:209} INFO - Executing cmd: bash -c 'echo CAT; echo KITTY;'
[2025-10-24T11:18:23.578198Z] {process_utils.py:213} INFO - Output:
[2025-10-24T11:18:23.582314Z] {process_utils.py:217} INFO - CAT
[2025-10-24T11:18:23.582415Z] {process_utils.py:217} INFO - KITTY

^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@amoghrajesh amoghrajesh added this to the Airflow 3.2.0 milestone Oct 24, 2025
@amoghrajesh amoghrajesh self-assigned this Oct 24, 2025
@amoghrajesh amoghrajesh merged commit 0636439 into apache:main Oct 24, 2025
60 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants