uranusjr commented on PR #24626: URL: https://github.com/apache/airflow/pull/24626#issuecomment-1165284348
Hmm, I was under the impression `breeze tests` also needs this, but apparently it does not and “correctly” picks up the AMD-based image. This leads me to think maybe we are doing something wrong with the overall structure. Currently: 1. `shell` and `start-airflow` only pick up an image if it matches the platform. In other words, if we only have an AMD64 base image available, they would attempt to rebuild the base image against ARM from scratch. 2. `tests` automatically picks up the image for AMD64 and run tests against it. So either `shell` and `start-airflow` should also automatically detect the base image is against AMD64 and build against it instead, _or_ `tests` should not blindly pick up the AMD64 image, and instead build against ARM from scratch by default. -- 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: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
