Prevent worker_extra_args to adds up extra arguments

dask / dask-jobqueue

Deploy Dask on job schedulers like PBS, SLURM, and SGE

https://jobqueue.dask.org

BSD 3-Clause "New" or "Revised" License

235 stars 142 forks source link

Closed guillaumeeb closed 2 years ago

guillaumeeb commented 2 years ago

Fixes #589, but I'd like to understand better how this bug was introduced.

Maybe we should also add a test, not mandatory here.

theXYZT commented 2 years ago

I think it's because in jobqueue.yaml the default extra value is null, but the default value for worker-extra-args is [] which is mutable.

guillaumeeb commented 2 years ago

Superseded by #591