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 output of how jobs are printed during poll stage #664

Merged
merged 3 commits into from
Feb 26, 2021

Conversation

shahzebsiddiqui
Copy link
Member

Refactor batch executor class (slurm, lsf, cobalt)
print table of completed and pending jobs

list of jobs completed and pending with details such as name of test,
executor, jobID, job state, and command
jobs. Use "fancy_grid" table format to print table and we show list
of jobIDs in queue to be polled
@codecov
Copy link

codecov bot commented Feb 26, 2021

Codecov Report

Merging #664 (b29cb06) into devel (dbddad0) will increase coverage by 6.83%.
The diff coverage is 60.73%.

Impacted file tree graph

@@            Coverage Diff             @@
##            devel     #664      +/-   ##
==========================================
+ Coverage   57.69%   64.52%   +6.83%     
==========================================
  Files          21       34      +13     
  Lines        1553     2816    +1263     
  Branches      226      516     +290     
==========================================
+ Hits          896     1817     +921     
- Misses        606      875     +269     
- Partials       51      124      +73     
Impacted Files Coverage Δ
buildtest/log.py 63.16% <ø> (ø)
buildtest/executors/slurm.py 13.45% <13.45%> (ø)
buildtest/executors/cobalt.py 16.07% <16.07%> (ø)
buildtest/executors/lsf.py 16.67% <16.67%> (ø)
buildtest/buildsystem/compilerbuilder.py 24.35% <24.35%> (ø)
buildtest/menu/__init__.py 44.44% <34.78%> (+23.75%) ⬆️
buildtest/config.py 37.80% <35.44%> (-62.20%) ⬇️
buildtest/menu/compilers.py 37.96% <37.96%> (ø)
buildtest/executors/setup.py 44.32% <44.32%> (ø)
buildtest/menu/build.py 56.78% <54.92%> (+31.15%) ⬆️
... and 43 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 69e4ce6...b7904ad. Read the comment docs.

@shahzebsiddiqui shahzebsiddiqui merged commit ecda508 into devel Feb 26, 2021
@shahzebsiddiqui shahzebsiddiqui deleted the test_directory branch February 26, 2021 15:48
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