Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Parsing error in pbs plugin #312

Open
fgeorgatos opened this issue Oct 16, 2023 · 0 comments
Open

Parsing error in pbs plugin #312

fgeorgatos opened this issue Oct 16, 2023 · 0 comments

Comments

@fgeorgatos
Copy link
Collaborator

Currently, the qtop pbs plugin does not correctly work with PBS Pro. PBS Pro displays job occupancy for all cores like this:


2257887.cluster-pbs5/0, 2257887.cluster-pbs5/1, 2257887.cluster-pbs5/2, 2257887.cluster-pbs5/3, 2257887.cluster-pbs5/4

However qtop expects this format which is commonly used in Torque:


0/10102182.cluster-pbs5, 1/10102106.cluster-pbs5, 2/10102339.cluster-pbs5, 3/10104007.cluster-pbs5

The pbs plugin should be able to detect and handle both formats automatically.
sebinbash pushed a commit that referenced this issue Dec 4, 2023
fix: a few bugs in PBS plugin, add some error handling (fixes #312 and …
24400f5 
…fixes #314)

- qstat: display '#' as a default symbol if there are more users than symbols
- pbs: update regex statements to account for truncated strings in qstat output
- pbs: update regex statement to account for job arrays in qstat output
- pbs: parse both Torque and PBS Pro style job ids
- pbs: add unit tests for _get_jobs_cores

Signed-off-by: Sebastian Schnorbus <sebastian.schnorbus@hpe.com>
@fgeorgatos fgeorgatos mentioned this issue May 14, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@fgeorgatos