amoghrajesh commented on code in PR #72100:
URL: https://github.com/apache/airflow/pull/72100#discussion_r4142174364


##########
airflow-ctl/src/airflowctl/api/datamodels/generated.py:
##########
@@ -297,6 +297,13 @@ class ClearTaskInstancesBody(BaseModel):
         ),
     ] = None
     prevent_running_task: Annotated[bool | None, Field(title="Prevent Running 
Task")] = False
+    keep_task_state: Annotated[

Review Comment:
   Ah nice catch. Fixed by setting `exclude_defaults=True` in 
`TasksOperations.clear`, so untouched fields (including keep_task_state) no 
longer get sent to servers that don't have them yet. Added a test asserting 
that the default payload has no keep_task_state key.
   
   



-- 
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]

Reply via email to