bugraoz93 commented on PR #50132:
URL: https://github.com/apache/airflow/pull/50132#issuecomment-3094704353
> @bugraoz93 ` airflowctl backfills list` shows
>
> ```
> root@3987193c411c:/opt/airflow# airflowctl backfills list
> Please enter password for encrypted keyring:
> 2025-07-20 10:08:47 [warning ] Server error
[airflowctl.api.client] extra={'detail': [{'type': 'missing', 'loc': ['query',
'dag_id'], 'msg': 'Field required', 'input': None}]}
> Traceback (most recent call last):
> File "/usr/local/bin/airflowctl", line 10, in <module>
> sys.exit(main())
> File "/opt/airflow/airflow-ctl/src/airflowctl/__main__.py", line 34, in
main
> safe_call_command(args.func, args=args)
> File "/opt/airflow/airflow-ctl/src/airflowctl/ctl/cli_config.py", line
66, in safe_call_command
> function(args)
> File "/opt/airflow/airflow-ctl/src/airflowctl/api/client.py", line 329,
in wrapper
> return func(*args, api_client=api_client, **kwargs)
> File "/opt/airflow/airflow-ctl/src/airflowctl/ctl/cli_config.py", line
568, in _get_func
> method_output = operation_method_object(**method_params)
> File "/opt/airflow/airflow-ctl/src/airflowctl/api/operations.py", line
121, in wrapped
> return _exit_if_server_response_error(response=func(self, *args,
**kwargs))
> File "/opt/airflow/airflow-ctl/src/airflowctl/api/operations.py", line
334, in list
> return super().execute_list(path="backfills",
data_model=BackfillCollectionResponse)
> File "/opt/airflow/airflow-ctl/src/airflowctl/api/operations.py", line
187, in execute_list
> raise e
> File "/opt/airflow/airflow-ctl/src/airflowctl/api/operations.py", line
165, in execute_list
> self.response = self.client.get(path, params=shared_params)
> File "/usr/local/lib/python3.10/site-packages/httpx/_client.py", line
1054, in get
> return self.request(
> File "/usr/local/lib/python3.10/site-packages/httpx/_client.py", line
827, in request
> return self.send(request, auth=auth, follow_redirects=follow_redirects)
> File "/usr/local/lib/python3.10/site-packages/httpx/_client.py", line
914, in send
> response = self._send_handling_auth(
> File "/usr/local/lib/python3.10/site-packages/httpx/_client.py", line
942, in _send_handling_auth
> response = self._send_handling_redirects(
> File "/usr/local/lib/python3.10/site-packages/httpx/_client.py", line
999, in _send_handling_redirects
> raise exc
> File "/usr/local/lib/python3.10/site-packages/httpx/_client.py", line
982, in _send_handling_redirects
> hook(response)
> File "/opt/airflow/airflow-ctl/src/airflowctl/api/client.py", line 102,
in raise_on_4xx_5xx
> return get_json_error(response) or response.raise_for_status()
> File "/opt/airflow/airflow-ctl/src/airflowctl/api/client.py", line 98,
in get_json_error
> raise err
> airflowctl.api.operations.ServerResponseError: Client error message:
{'detail': [{'type': 'missing', 'loc': ['query', 'dag_id'], 'msg': 'Field
required', 'input': None}]}
> ```
>
> Do I need to add dag_id as params?
If you can add it, it would be amazing! The API was updated in a way that it
doesn't work without dag_id. If you can add it as a method parameter and pass
it to your method, it would be great. Then it should work as well as cascaded
as a parameter to the command. Please let me know, I can also update that as a
separate PR.
This shows we need better integration testing with the API. At least we
should catch these while updating the API itself. Noted and will include
follow-ups. Thanks!
--
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]
