megan-parker commented on a change in pull request #19758:
URL: https://github.com/apache/airflow/pull/19758#discussion_r765899157
##########
File path: airflow/api_connexion/openapi/v1.yaml
##########
@@ -441,6 +441,41 @@ paths:
'401':
$ref: '#/components/responses/Unauthenticated'
+ patch:
+ summary: Update DAGs
+ x-openapi-router-controller: airflow.api_connexion.endpoints.dag_endpoint
+ operationId: patch_dags
+ tags: [DAG]
+ parameters:
+ - $ref: '#/components/parameters/PageLimit'
+ - $ref: '#/components/parameters/PageOffset'
+ - $ref: '#/components/parameters/FilterTags'
+ - $ref: '#/components/parameters/UpdateMask'
+ - name: only_active
+ in: query
+ schema:
+ type: boolean
+ default: true
+ required: false
+ description: |
+ Only update active DAGs.
+ - name: dag_id_pattern
+ in: query
+ schema:
+ type: string
+ required: false
+ description: |
+ If set, only update DAGs with dag_ids matching this pattern.
Review comment:
@ashb I had modified both the `get` and `patch` bulk endpoints. `~` can
be supplied to return all dags if needed.
I had done it this way since `dag_id_parameter` is a reusable parameter as
per this thread. I am thinking to define `dag_id_pattern` separately in both
bulk endpoints and define it as required in `patch` and optional in `get`. WDYT?
--
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]
