You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "github-actions[bot] (via GitHub)" <gi...@apache.org> on 2025/11/02 17:18:13 UTC

[PR] [v3-1-test] Add pause and unpause commands for dags (#57685) [airflow]

github-actions[bot] opened a new pull request, #57714:
URL: https://github.com/apache/airflow/pull/57714

   * Add pause and unpause commands for dags
   
   * Convert exit in failure
   
   * Update airflow-ctl/src/airflowctl/ctl/commands/dag_command.py
   
   * Refactor error message for pause/unpause operations in DAG commands
   (cherry picked from commit 203acbb8ef0883efe7de06169d8a4fb6840d22ca)
   
   Co-authored-by: Bugra Ozturk <bu...@users.noreply.github.com>


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [v3-1-test] Add pause and unpause commands for dags (#57685) [airflow]

Posted by "ashb (via GitHub)" <gi...@apache.org>.
ashb commented on PR #57714:
URL: https://github.com/apache/airflow/pull/57714#issuecomment-3485408494

   @bugraoz93 I think we need to stop back porting airflowctl changes to the Airflow core release branches. If some part of the process needs this, we need to change that process.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [v3-1-test] Add pause and unpause commands for dags (#57685) [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk merged PR #57714:
URL: https://github.com/apache/airflow/pull/57714


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [v3-1-test] Add pause and unpause commands for dags (#57685) [airflow]

Posted by "bugraoz93 (via GitHub)" <gi...@apache.org>.
bugraoz93 commented on PR #57714:
URL: https://github.com/apache/airflow/pull/57714#issuecomment-3487206902

   > @bugraoz93 I think we need to stop back porting airflowctl changes to the Airflow core release branches. If some part of the process needs this, we need to change that process.
   
   I was planning to do it after the first release due to integration test limitations. Leveraging what we have automated backports etc made it easy to use the v3-1-test branch. I assumed it should be fine since the released parts are not even running these code pieces. I wasn't only sure about branch, commit, etc limitations or things to consider which I opened a discussion in Slack in the release channel. I was planning to open as a dev discussion but thought this was a technical detail and we could decide with knowledgeable people. I am not saying everyone should catch the chat or so but just to get an idea or speak out on this backporting case to see if there is big blockers to use it for the first round. I haven't heard hard no, I moved with the backporting option. I already planned this change for sure. As an initial plan, we should definitely release airflowctl from main, that is not questionable 
   
   Is it impacting anything in Airflow Core releases? I can speed up the plan if it is causing too much trouble. If we can live with this until the first release, there will be one or two more PRs not more for 0.1.0. This would be very helpful for the first release


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [v3-1-test] Add pause and unpause commands for dags (#57685) [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #57714:
URL: https://github.com/apache/airflow/pull/57714#issuecomment-3487566655

   I think a pre-requisite for running things from main is indeed adding back-compat tests in CI. Similarly as we do with task-sdk this is something we should **definitely** do in the future - the plan is to remove EVERYTHING except airlfow-core from v3-* branches eventually and this should be part of it.
   
   If that is something that holds back airflow-ctl release, I would be for merging it now in v3-1-test until we implement back-compat test harness like we have for providers (for both task-sdk and airflow-ctl).


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org