Skip to content

Make hltmonitoringJson optional in xAODConfigTool

Tim Martin requested to merge tamartin-master-patch-00829 into master

It was noted by @khamano that xAODConfigTool crashes on data22_900GeV.00423027.physics_Main.merge.AOD.f1216_m2093 in analysis base due to a present but zero-sized monitoring JSON collection in the AOD.

There is a mistake in the logic of the xAODConfigTool (but not the alternate xAODConfigSvc) which classes this as an error, this is a mistake as we do not quite yet write the monitoring JSON to data AODs.

The zero-sized collection is coming here a product of the AOD-MERGE stage.

Altering the logic of the xAODConfigTool to not crash also on an empty-collection.

cc @mark & @ballbroo

Note: MR opened via GitLab UI

Merge request reports