dstandish commented on PR #33103:
URL: https://github.com/apache/airflow/pull/33103#issuecomment-1670555673

   I was about to say that I think we can close this because after 
https://github.com/apache/airflow/pull/33102, setting deps in a context no 
longer "auto adds" tasks to the context.  So now the sample dag posted in 
https://github.com/apache/airflow/issues/33093 will produce this graph:
   
   <img width="690" alt="image" 
src="https://github.com/apache/airflow/assets/15932138/e350cd60-89c6-4404-9146-6846ace94a74";>
   
   
   I.e. nothing will be in the task group.
   
   BUT...
   
   then i added the tasks to the context with `add_task`:
   
   ```python
       with TaskGroup("my_group") as tg:
           setup1 >> work1 >> teardown1
           setup1 >> teardown1
           tg.add_task(setup1)
           tg.add_task(teardown1)
           tg.add_task(work1)
       tg >> work2
   ```
   
   And now we encounter the issue again:
   <img width="936" alt="image" 
src="https://github.com/apache/airflow/assets/15932138/087a51d5-cbc3-4832-aa97-4b8240522566";>
   
   I'll poke around and see if I can figure anything out.
   
   


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