AIP-81 Transition of Version Command
closes : #45670
Implement Version Command on the CLI side
^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.
Adding ActionCommand requires a deep dive since it is not working as expected. I'm working on fixing it
cc: @bugraoz93
If that's okay, I'll add the tests tomorrow.
I need to update some code. I will make the changes tomorrow morning as soon as possible
Using Version_operation and api_client.version.get() both cases it will show this logging :
[2025-05-29T11:14:08.670+0000] {_client.py:1026} INFO - HTTP Request: GET http://localhost:8080/api/v2/version "HTTP/1.1 200 OK" then version_dict
Could you please help me prevent this message from appearing
cc : @bugraoz93
Using
Version_operationandapi_client.version.get()both cases it will show this logging :
[2025-05-29T11:14:08.670+0000] {_client.py:1026} INFO - HTTP Request: GET http://localhost:8080/api/v2/version "HTTP/1.1 200 OK"then version_dictCould you please help me prevent this message from appearing
Thanks for tagging @Prab-27! I missed this. I will look, but we can handle this disabled log in a separate PR. Let's first finish the version command and merge it. I think that part should be disabled in the client that will apply to all commands. Great catch!
Using
Version_operationandapi_client.version.get()both cases it will show this logging :[2025-05-29T11:14:08.670+0000] {_client.py:1026} INFO - HTTP Request: GET http://localhost:8080/api/v2/version "HTTP/1.1 200 OK"then version_dict Could you please help me prevent this message from appearingThanks for tagging @Prab-27! I missed this. I will look, but we can handle this
disabled login a separate PR. Let's first finish the version command and merge it. I think that part should be disabled in the client that will apply to all commands. Great catch!
Thanks !! I'll start working on this in a couple of days
@bugraoz93 The static checks are not automatically fixed. Could you please tell me how to resolve these checks?
Can I copy - paste from the main branch to fix this check? I don't have enough knowledge about these files
@bugraoz93 The static checks are not automatically fixed. Could you please tell me how to resolve these checks?
Fixed
@bugraoz93 The static checks are not automatically fixed. Could you please tell me how to resolve these checks?
Fixed
It was quite busy days for me so I couldn't give some pointers. It's great to see it is fixed! Thanks a lot, @Prab-27! I will check this in detail soon
Hey @Prab-27, can you please rebase the branch? We can directly merge it afterwards. I can also do it if you don't have time