Replies: 15 comments
-
Thanks for opening your first issue here! Be sure to follow the issue template! If you are willing to raise PR to address this issue please do so, no need to wait for approval. |
Beta Was this translation helpful? Give feedback.
-
If there's a button for this in the UI, we'd definitely want to make sure that only the highest level of roles within a given server have access to this feature. That's probably assumed and would be taken care of, but just want to state the obvious. |
Beta Was this translation helpful? Give feedback.
-
this is not ideal if the UI button doesn't revert dags back to its original state on unpause. having list of dags as params should be strictly required for this for pause /unpause to original state. should be covered in the caveat |
Beta Was this translation helpful? Give feedback.
-
Hey @potiuk I am willing to work on this. |
Beta Was this translation helpful? Give feedback.
-
I would rather this not be a UI feature. This is very dangerous operation and with a single click of a button that was done by mistake can cause serious problems to large deployments. If this must be a UI button it must come with seperated role with default of usage only to Admin. However I would rather this functionality to be API only. To my prespective (and correct me if I am wrong) this operation should be used rarely so having it as button in the main page might be not the best? WDYT? |
Beta Was this translation helpful? Give feedback.
-
Agreed that this is dangerous as the implied "unpause all" can resume DAGs which were previously paused for good reason. As an admin, I have the option to scale the scheduler down to 0 replicas (preventing any runs from advancing) or setting |
Beta Was this translation helpful? Give feedback.
-
Beta Was this translation helpful? Give feedback.
-
@potiuk I agree with @cmarteepants suggestion. This seems to be the right way. WDYT? |
Beta Was this translation helpful? Give feedback.
-
I am fine with that. @eladkal ? |
Beta Was this translation helpful? Give feedback.
-
Can you be specific into what view you are referring to? |
Beta Was this translation helpful? Give feedback.
-
I agree - the task instance and dag run view are not the right the place - it was the only view I could find that had a multi-selector. My idea was to replace the individual toggles to pause/unpause dags with a multi-selector + action button in the home page / dag list view. |
Beta Was this translation helpful? Give feedback.
-
Thinking about this further: Another idea is an extension of @yshkp's original proposal, which takes into account whether individual dags were paused before globally pausing all dags. We proceed with a button to pause/unpause all dags, however:
To be fair, this is becoming a lot more complicated. |
Beta Was this translation helpful? Give feedback.
-
I belive its complicated because you are combining the business logic of the dag with adminstative operation of upgrade. DAG author should not have business logic impacted (on/off) because cluster admin decided to make an upgrade. Meaning that if you switch my dag to off for upgrade then you need to turn it on once upgrade is done how would you do that for a deployemnt of 1000 dags authored by 200 different developers? I think you are consedring only deployments where the dag author and the cluster admin are the same person :) I am happy to discuss ideas of how to accommodate your request but you need to understand that we have to consider all kinds of deployments and the implication of such button would make. I would be more happy with adding such operation on a folder base rather than all dags once #24464 is implemented. (On side note I still don't quite follow why termination of scheduler is not good enough to handle your use case) |
Beta Was this translation helpful? Give feedback.
-
I'll defer to others on the use-cases. These were just two drive-by suggestions on how we can add UI capability for this while minimizing risk, assuming this is something the community truly wants :) Personally, I think the existing env var solution is fine. |
Beta Was this translation helpful? Give feedback.
-
Description
Slider button on UI of airflow to pause/unpause all DAGs.
Use case/motivation
During deployment, every time there is a use case to pause all dags to not run tasks in the old build. So this button will come handy during deployment.
Related issues
No response
Are you willing to submit a PR?
Code of Conduct
Beta Was this translation helpful? Give feedback.
All reactions