Skip to content

set SLURM_PRIO_PROCESS environment variable#19

Closed
anthropotron wants to merge 1 commit intonatefoo:masterfrom
anthropotron:master
Closed

set SLURM_PRIO_PROCESS environment variable#19
anthropotron wants to merge 1 commit intonatefoo:masterfrom
anthropotron:master

Conversation

@anthropotron
Copy link
Copy Markdown

No description provided.

@lparsons
Copy link
Copy Markdown

lparsons commented Nov 5, 2018

Related to issue #18.

@natefoo
Copy link
Copy Markdown
Owner

natefoo commented Nov 12, 2018

Thanks for the submission! Two concerns:

  1. It's being set in the process's environment, which means it persists outside the context of the job being submitted.
  2. Because of the previous issue, I don't think it'd be thread safe.

I am working on these issues in the always-add-envvars branch.

@lparsons
Copy link
Copy Markdown

Any updates on this? I'm still running into the occasional issue due to it.

@natefoo
Copy link
Copy Markdown
Owner

natefoo commented Mar 7, 2019

Interesting, I thought I'd finished this. I'll pick this back up ASAP, should be within the next week or so.

@natefoo
Copy link
Copy Markdown
Owner

natefoo commented Mar 9, 2019

Ok, I implemented this in #23. Thanks for the submission though, @anthropotron!

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.

3 participants