[GitHub] [airflow] potiuk commented on pull request #24664: Add ARM image building for regular PRs

2022-06-28 Thread GitBox
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

[GitHub] [airflow] potiuk commented on pull request #24664: Add ARM image building for regular PRs

2022-06-28 Thread GitBox
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

[GitHub] [airflow] potiuk commented on pull request #24664: Add ARM image building for regular PRs

2022-06-27 Thread GitBox
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

[GitHub] [airflow] potiuk commented on pull request #24664: Add ARM image building for regular PRs

2022-06-27 Thread GitBox
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

[GitHub] [airflow] potiuk commented on pull request #24664: Add ARM image building for regular PRs

2022-06-27 Thread GitBox
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

[GitHub] [airflow] potiuk commented on pull request #24664: Add ARM image building for regular PRs

2022-06-27 Thread GitBox
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

[GitHub] [airflow] potiuk commented on pull request #24664: Add ARM image building for regular PRs

2022-06-27 Thread GitBox
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

[GitHub] [airflow] potiuk commented on pull request #24664: Add ARM image building for regular PRs

2022-06-27 Thread GitBox
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

[GitHub] [airflow] potiuk commented on pull request #24664: Add ARM image building for regular PRs

2022-06-26 Thread GitBox
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