bugraoz93 commented on PR #45300:
URL: https://github.com/apache/airflow/pull/45300#issuecomment-2726800317

   > Speaking of what we are going to release - we should be able to release a 
separate "apache-airflow-cli" distribution and only that distribution should 
have`keyring` as paraemeter and should authenticate with the server.
   > 
   > Now - we need to figure out how to keep the `remote` commands still 
working remotely in a separate package, while also have them available as 
airflow commands in `apache-airflow-core` distribution (without the keyring).
   > 
   > That is a bit tricky, but I am sure we can work it out.
   
   Thanks Jarek! How about I remove the connection command integration from the 
PR? Let's merge the API integration without integrating into any command so 
that maintaining that part would be a lot easier even while moving out of the 
package. I then make the split into `apache-airflow-cli` and add this 
connection integration there. From there we start the transition to other 
remote commands. 
   
   For the having all in the `apache-airflow-core` too is indeed tricky. We 
should come up with something that shouldn't need duplicate work for API and DB 
calls separately for each command if we don't want to remove remote commands 
from `apache-airflow-core`.


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