potiuk commented on code in PR #37134:
URL: https://github.com/apache/airflow/pull/37134#discussion_r1474625756
##########
scripts/ci/images/ci_start_arm_instance_and_connect_to_docker.sh:
##########
@@ -20,7 +20,7 @@ SCRIPTS_DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd)"
WORKING_DIR="/tmp/armdocker"
INSTANCE_INFO="${WORKING_DIR}/instance_info.json"
ARM_AMI="ami-0e43196369d299715" # AMI ID of latest arm-docker-ami-v*
Review Comment:
I rebuild it occasionally (it needs some of our extra stuf - very few but
still) -
https://github.com/apache/airflow-ci-infra/blob/main/arm-docker-ami/packer/linux-ami.pkr.hcl
And yeah, it's prety old image. It has very small surface attack though - we
only run remote docker build with it and that's about it (and it is
ephemeral/spot instance with built-in kill switch). But yes we could refresh it.
But I hope we will be able to get rid of it completely with the new M1
public runners, we need to just figure out how to build multi-stage images with
them (this is tricky because you need to run them on two parallel-running
hardwares ideally, unless you want to coordinate building them separately and
generating manifest for two different images.
--
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
