Skip to content

Commit

Permalink
Merge pull request #135 from lsst-dm/u/echarles/fix_bps_report_call
Browse files Browse the repository at this point in the history
Fix bps report call for htcondor
  • Loading branch information
eacharles authored Nov 18, 2024
2 parents 9bef8de + 76cb7b8 commit f236c4e
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
4 changes: 3 additions & 1 deletion src/lsst/cmservice/handlers/functions.py
Original file line number Diff line number Diff line change
Expand Up @@ -642,7 +642,9 @@ async def load_wms_reports(
if wms_run_report is None:
return job
if wms_run_report.job_summary is None:
wms_run_report.job_summary = compile_job_summary(wms_run_report.jobs)
compile_job_summary(wms_run_report)
if wms_run_report.job_summary is None:
raise RuntimeError("compile_job_summary did not compile a job summary")
for task_name, job_summary in wms_run_report.job_summary.items():
fullname = f"{job.fullname}/{task_name}"
wms_dict = {f"n_{wms_state_.name.lower()}": count_ for wms_state_, count_ in job_summary.items()}
Expand Down
2 changes: 1 addition & 1 deletion src/lsst/cmservice/handlers/jobs.py
Original file line number Diff line number Diff line change
Expand Up @@ -357,7 +357,7 @@ async def _load_wms_reports(
fake_status = kwargs.get("fake_status", None)

try:
wms_svc = self._get_wms_svc()
wms_svc = self._get_wms_svc(config={})
except ImportError as msg:
if not fake_status: # pragma: no cover
raise msg
Expand Down

0 comments on commit f236c4e

Please sign in to comment.