-
Notifications
You must be signed in to change notification settings - Fork 14.6k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
add dag_run_ids and task_ids filter for the batch task instance API endpoint #32705
add dag_run_ids and task_ids filter for the batch task instance API endpoint #32705
Conversation
15d8f64
to
59d0c1e
Compare
Can you add “New in version” markers to the parameters in the OpenAPI spec please? There are various examples if you search in the file. |
64fb437
to
384e360
Compare
@uranusjr done, assumed it'd go in 2.6.4, please take a look. |
384e360
to
1074524
Compare
not sure if maintainers can do that right away but please feel free to modify any files here before you merge as you see fit, e.g. fixing the version. if you let me know what'd be the right version number I can also do that. |
I think will hold-off with those merges for a short while due to cutting out 2.7.0 release, but I think @pierrejeambrun looked at the open-api changes recently :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good to me and working as expected. As mentioned 2.7.0
is already being cut out and this will most probably go into 2.7.1
can you please update the New in version xxxx
part, and we should be good to go :)
edit: I just updated it.
Approved
56f1e74
to
78c5de0
Compare
This PR introduces two new filters for the batch task instance endpoint:
dag_run_ids
: a set of DAG Run IDs to filter the instances that were in those runstask_ids
: a set of task IDs that the instances were forLet me know if you have any feedback.