Skip to content
Snippets Groups Projects

[Webservice] Ignore final empty line from squeue

Merged Thomas Kluyver requested to merge fix/squeue-trailing-nl into master

Description

Job monitor logs contain lots of extraneous warnings:

2024-12-10 09:13:05,038 - __main__ - WARNING - [job_monitor.py:84] Could not parse squeue status line ''

The squeue output has a trailing newline, so we were trying to parse an empty line.

How Has This Been Tested?

Using subprocess.run() in a shell.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • My code follows the code style of this project.

Reviewers

@schmidtp

Merge request reports

Checking pipeline status.

Merged by Thomas KluyverThomas Kluyver 3 months ago (Dec 10, 2024 9:27am UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading