ashb edited a comment on issue #4984: Revert "[AIRFLOW-3992] 1-setup-env.sh 
should be re-runable"
URL: https://github.com/apache/airflow/pull/4984#issuecomment-477929088
 
 
   We only expect it to run in docker/docker-on-CI but if users do attempt to 
run this locally trying to follow run the CI locally we shouldn't overwrite 
files.
   
   (I've had a report of this happening, so it's not an "it might happen" but 
"it has happened")
   
   Given we also add to the end of authorized_keys maybe the better change here 
is to only run this script in the docker-container (by checking an env variable 
that we set somewhere?)
   
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to