Skip to content

Prevent worker_extra_args to adds up extra arguments#590

Closed
guillaumeeb wants to merge 1 commit into
dask:mainfrom
guillaumeeb:interface-flag
Closed

Prevent worker_extra_args to adds up extra arguments#590
guillaumeeb wants to merge 1 commit into
dask:mainfrom
guillaumeeb:interface-flag

Conversation

@guillaumeeb

Copy link
Copy Markdown
Member

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

theXYZT commented Oct 2, 2022

Copy link
Copy Markdown

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

Copy link
Copy Markdown
Member Author

Superseded by #591

@guillaumeeb guillaumeeb closed this Oct 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cluster keeps appending "interface" flag to job script

2 participants