Skip to content

DM-53098 : Convert ctrl_bps_htcondor to use htcondor2 - #101

Merged
tcjennings merged 8 commits into
mainfrom
tickets/DM-53098
Oct 8, 2026
Merged

DM-53098 : Convert ctrl_bps_htcondor to use htcondor2#101
tcjennings merged 8 commits into
mainfrom
tickets/DM-53098

fix: address PR feedback

541434d
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project failed Oct 8, 2026 in 1s

88.19% (-0.22%) compared to 69b3ae7

View this Pull Request on Codecov

88.19% (-0.22%) compared to 69b3ae7

Details

Codecov Report

❌ Patch coverage is 79.08745% with 55 lines in your changes missing coverage. Please review.
✅ Project coverage is 88.19%. Comparing base (69b3ae7) to head (541434d).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
python/lsst/ctrl/bps/htcondor/lssthtc.py 53.19% 21 Missing and 1 partial ⚠️
python/lsst/ctrl/bps/htcondor/htcondor_compat.py 47.05% 18 Missing ⚠️
python/lsst/ctrl/bps/htcondor/htcondor_service.py 66.66% 7 Missing and 1 partial ⚠️
python/lsst/ctrl/bps/htcondor/common_utils.py 58.82% 6 Missing and 1 partial ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##             main     #101      +/-   ##
==========================================
- Coverage   88.41%   88.19%   -0.22%     
==========================================
  Files          21       23       +2     
  Lines        5067     5059       -8     
  Branches      504      502       -2     
==========================================
- Hits         4480     4462      -18     
- Misses        499      510      +11     
+ Partials       88       87       -1

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

View more details on Codecov