dask / dask-yarn

Deploy dask on YARN clusters
http://yarn.dask.org
BSD 3-Clause "New" or "Revised" License
69 stars 41 forks source link

Add -L to the curl download of miniconda install script #119

Closed bobbruno closed 4 years ago

bobbruno commented 4 years ago

I had failures in the EMR boostrap action using the script as-is. After some tinkering, I realized that curl on line 84 was failing with a generated file size of 0. Tests on my personal machine replicated the problem, which I solved by adding the -L (follow redirects) option to it.

quasiben commented 4 years ago

I think this is a duplicate of https://github.com/dask/dask-yarn/pull/117/. @jcrist ?

jcrist commented 4 years ago

Apologies for the late review, this was indeed superseded by #117. Closing.