potiuk commented on PR #24664:
URL: https://github.com/apache/airflow/pull/24664#issuecomment-1168450379
Updated with only running stuff when we are upgrading to newer dependencies
:). It should be super-optimized this way
--
This is an automated message from the Apache Git Service.
To re
potiuk commented on PR #24664:
URL: https://github.com/apache/airflow/pull/24664#issuecomment-1168445200
Actually @jedcunningham - after sleeping over it, it seems we can furter cut
down the overhead and cost. It seems that in regular PRs we only need to build
it when some depdnencies chang
potiuk commented on PR #24664:
URL: https://github.com/apache/airflow/pull/24664#issuecomment-1168098448
BTW. @jedcunningham - by writing this answer and looking at the workflows I
realized that there is a potential for a small optimization (of cost). I added
additional "needs" for the "arm
potiuk commented on PR #24664:
URL: https://github.com/apache/airflow/pull/24664#issuecomment-1168093297
> @potiuk, just want to double check there is no security/resource
considerations here with us spinning up ARM machines for normal, non-committer
PRs, right?
Good questions. I :he
potiuk commented on PR #24664:
URL: https://github.com/apache/airflow/pull/24664#issuecomment-1167446632
BTW. This one is crucial for those of us who use M1 on a daily basis and
rely on the images continue to work for them @dstandish @bbovenzi :eyes:
--
This is an automated message from
potiuk commented on PR #24664:
URL: https://github.com/apache/airflow/pull/24664#issuecomment-116779
Nice. All ARM images built in under 10m :). Looks green.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
potiuk commented on PR #24664:
URL: https://github.com/apache/airflow/pull/24664#issuecomment-1167191019
Seems it's going to work this time (I had to make the builds run
sequentially, but it works now. just fine.
--
This is an automated message from the Apache Git Service.
To respond to t
potiuk commented on PR #24664:
URL: https://github.com/apache/airflow/pull/24664#issuecomment-1167173732
cc: @gmsantos - this will prevent the dramas you faced last week. I am
running it from "apache" repo, to make sure it actually works. The nice thing
is that for now we are **just** going
potiuk commented on PR #24664:
URL: https://github.com/apache/airflow/pull/24664#issuecomment-1166467913
Testing execution of in-airlfow-repo PR.
--
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 t