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.

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

Change Poll commands for PBS and Torque and add support for retrieving job data #1746

Merged

Conversation

shahzebsiddiqui
Copy link
Member

No description provided.

… poll job. The command

options will differ slightly when using PBS vs Torque.

With PBS we will use 'qstat -xf' to get job history whereas Torque doesnt have job history so we will
use 'qstat -f'

Also we add method retrieve_jobdata for Torque class which will get job data and store value as string.

For PBS we will get output as JSON content
Copy link

codecov bot commented Apr 3, 2024

Codecov Report

Attention: Patch coverage is 19.04762% with 17 lines in your changes are missing coverage. Please review.

Project coverage is 33.69%. Comparing base (9c9b820) to head (dc6dce2).
Report is 68 commits behind head on devel.

Files Patch % Lines
buildtest/scheduler/pbs.py 19.05% 17 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##            devel    #1746       +/-   ##
===========================================
- Coverage   80.85%   33.69%   -47.16%     
===========================================
  Files          57       58        +1     
  Lines        6453     6531       +78     
===========================================
- Hits         5217     2200     -3017     
- Misses       1236     4331     +3095     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@shahzebsiddiqui shahzebsiddiqui merged commit b79dd92 into devel Apr 4, 2024
32 of 36 checks passed
@shahzebsiddiqui shahzebsiddiqui deleted the change_pbs_and_torque_poll_commands_and_jobdata branch April 4, 2024 17:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant