o-nikolas commented on PR #60675:
URL: https://github.com/apache/airflow/pull/60675#issuecomment-3770661646

   > @o-nikolas I have tested the general functionality of celery worker with 
your changes as well as tested out the CLI. Things are working as expected. 
However, I don't see how I can test the `--team` option. Seems like if I set a 
team the jobs still seem to be launching on the worker. How do you propose I 
test the multi team support here?
   
   Thanks for the thorough review @dheerajturaga! I've addressed those issues 
(slightly differently for one than your suggested patch). I'm currently 
struggling with back compat tests. It's slow/difficult because those tests to 
do not run successfully in breeze on my laptop. So I have to push to the PR to 
test each change.
   
   As far as testing with the --team flag. For this you have to have a full 
multi-team setup, which we don't have great documentation for yet (coming 
soon). The most helpful testing is actually on the backcompat side (testing 
with airflow 2.11 and 3.1.X)


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