Incorrect description for env_extra for HTCondorCluster

Hi,

The description for env_extra in HTCondorCluster is not correct: the job that HTCondorCluster creates calls dask-worker directly instead of through a bash wrapper script, so you cannot put arbitrary shell commands into env_extra.

The interface supports environment variables as key=value pairs, which will be inserted into dask-worker’s environment (via the “Environment” attribute in the submit file). (For consistency, you can write export foo=bar but the word “export” will be ignored.)

This is also important to keep in mind with regards to #323; renaming env_extra to job_script_extra or similar would be even more inaccurate (for the HTCondor case anyway).

[jolange]

Ok, I made a proposal in #563. I’d suggest to start the renaming (#323) after we agreed on the implementation and merged it. (I can also do that if you like.)

[jolange]

Heh /tmp is there to be cluttered 😆

😆

So by now I would prefer the in-lining solution to an extra temporary script. But maybe someone else has a good idea how to realize this.