Jarek Potiuk

Results 1574 comments of Jarek Potiuk

> Yeah. That's what I was afraid - the impersonation test fail in ALL tests now :) But that is good - because it failed intermittently and if find the...

Yeah. We might want to remove or simplify the test. BTW. There is one more thing that I noticed - all of the tests in this build take upwards of...

Also the "use public runner" and comparing regular PR with no coverage enabled with "with coverage" might show some surprises. I think adding coverage slows down the tests (in general...

> We need a two counters 🤣 > Time spend to adjusts something to coverage or codecov > Time spend to fix impersonation tests Yes. Second comment in second answer...

We already have `quarantined` tests for that kind of approach - they are skipped by default when the test is marked with `quarantined` and we can have a separate `isolated`...

> BTW we could mark temporary tests/core/test_impersonation_tests.py as quarantined (not a first time) and create a task for move it into isolated and implements it. Tempting :)

> @ephraimbuddy I think it worthwhile to check it again after we move impersonations as quarantined Agree :)

@aru-trackunit - seems that @Joffreybvn cannot reproduce the issue - can you please provide a bit more information as he asked in https://github.com/apache/airflow/issues/36838#issuecomment-1925889496 ?

> At current moment we know at lest incompatible with that pendulum 2 doesn't support of python 3.12, but this one could be solved by https://github.com/apache/airflow/pull/36281 Yeah. I believe this...

You should be able to build it locally with `breeze ci-image build --python 3.12` (after you add it to configuration)