Skip to content

Commit

Permalink
Merge pull request #1823 from buildtesters/fix_bug_with_executor_dete…
Browse files Browse the repository at this point in the history
…ction

fix bug with slurm partition detection
  • Loading branch information
shahzebsiddiqui authored Oct 7, 2024
2 parents b566047 + bf21cc9 commit d992f86
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion buildtest/scheduler/detection.py
Original file line number Diff line number Diff line change
Expand Up @@ -151,7 +151,7 @@ def validate_partition(self, slurm_executor):
f"{self.sched_cmds['sinfo']} -p {slurm_executor['partition']} -h -O available"
)

if part_state != "up":
if "up" not in part_state:
self.logger.error(
f"partition - {slurm_executor['partition']} is in state: {part_state}. It must be in 'up' state in order to accept jobs"
)
Expand Down

0 comments on commit d992f86

Please sign in to comment.