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

add olcf regression test for ascent system to get test coverage for LSF #663

Merged
merged 2 commits into from
Feb 25, 2021

Conversation

shahzebsiddiqui
Copy link
Member

No description provided.

There was bug where buildtest couldn't resolve path to generated output
because we were using wrong directory to submit job. We should submit jobs from
stage directory so the generated files are in the same location as the job script.
hardcode python module version when running olcf pipeline, that way if
system python changes we dont break workflow
@codecov
Copy link

codecov bot commented Feb 25, 2021

Codecov Report

Merging #663 (f929a71) into devel (dbddad0) will increase coverage by 6.96%.
The diff coverage is 60.91%.

Impacted file tree graph

@@            Coverage Diff             @@
##            devel     #663      +/-   ##
==========================================
+ Coverage   57.69%   64.66%   +6.96%     
==========================================
  Files          21       34      +13     
  Lines        1553     2804    +1251     
  Branches      226      513     +287     
==========================================
+ Hits          896     1813     +917     
- Misses        606      867     +261     
- Partials       51      124      +73     
Impacted Files Coverage Δ
buildtest/log.py 63.16% <ø> (ø)
buildtest/executors/slurm.py 13.95% <13.95%> (ø)
buildtest/executors/cobalt.py 15.52% <15.52%> (ø)
buildtest/executors/lsf.py 16.22% <16.22%> (ø)
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 43.82% <43.82%> (ø)
buildtest/docs.py 60.00% <60.00%> (ø)
... 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 811469c...5784f73. Read the comment docs.

@shahzebsiddiqui shahzebsiddiqui merged commit 69e4ce6 into devel Feb 25, 2021
@shahzebsiddiqui shahzebsiddiqui deleted the olcf_regtest branch February 25, 2021 21:29
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