potiuk commented on pull request #20847: URL: https://github.com/apache/airflow/pull/20847#issuecomment-1022347709
Hey @Melodie97 . I just pushed some fixes. I also rebased your changes to latest `main` so you will have to do: `git fetch && git reset --hard origin/main` In your branch. Make sure that you rebuild your image with `./breeze build-image` before you start it. But it should all start now and we have lot-more jager traces. By quick check I saw: * SQL traces * jinja2 traces Some others might be only visible later or might need additional configuration, so this is somethign you should take a look afterwards. There are still some things from modularisation left from my comments, so I will mark all comments that are already addressed asr resolved. Please take a look at the others. There were quite a few problems: * The "airflow commands" were in `'` single quotes not `"` quotes which make them not work * --open-telemetry flag was not `store_true` so it expected argument * The --open-telemetry flag was not needed to except the webserver * `scheduler` was removed * Some screenshots here:   -- 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]
