Taragolis commented on code in PR #39920:
URL: https://github.com/apache/airflow/pull/39920#discussion_r1620356020


##########
docs/apache-airflow-providers-amazon/executors/ecs-executor.rst:
##########
@@ -187,6 +187,13 @@ Then you can build your image by ``cd``-ing to the 
directory with the Dockerfile
    docker build -t my-airflow-image \
     --build-arg aws_default_region=YOUR_DEFAULT_REGION .
 
+For users on Apple Silicon, you must specify the arch using ``docker buildx``:

Review Comment:
   So maybe instead of cover one case, better cover most of them with mention 
that is important to build and run under the same architecture. And also would 
be nice to have a link to the Docker documentation.
   
   And keep example with a small clarification, that if you build on Apple 
Silicone and intend to run on X86_64 aka AMD64 then make sure that you use 
appropriate platform. 



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to