Skip to content

Commit

Permalink
Merge pull request #3184 from ekouts/bugfix/squeue_nodelist
Browse files Browse the repository at this point in the history
[bugfix] Retrieve the test job's nodelist when using the `squeue` backend
  • Loading branch information
vkarak committed May 13, 2024
2 parents d6ed34d + ff41341 commit 94d9086
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
Expand Up @@ -35,8 +35,9 @@ RUN chmod +rx /etc/slurm-llnl/docker-entrypoint.sh

# Install reframe
ARG REFRAME_TAG=develop
ARG REFRAME_REPO=reframe-hpc
WORKDIR /usr/local/share
RUN git clone --depth 1 --branch $REFRAME_TAG https://github.com/reframe-hpc/reframe.git && \
RUN git clone --depth 1 --branch $REFRAME_TAG https://github.com/$REFRAME_REPO/reframe.git && \
cd reframe/ && ./bootstrap.sh
ENV PATH=/usr/local/share/reframe/bin:$PATH

Expand Down
3 changes: 3 additions & 0 deletions reframe/core/schedulers/slurm.py
Expand Up @@ -621,6 +621,9 @@ def poll(self, *jobs):

# Join the states with ',' in case of job arrays
job._state = ','.join(s.group('state') for s in job_match)

# Use ',' to join nodes to be consistent with Slurm syntax
job._nodespec = ','.join(m.group('nodespec') for m in job_match)
self._cancel_if_blocked(
job, [s.group('reason') for s in state_match]
)
Expand Down

0 comments on commit 94d9086

Please sign in to comment.