Merge branch 'checkForTriggerMetadata' into 'master'

Check for presence of trigger metadata before fetching

See merge request atlas/athena!37791

Trigger/TrigConfiguration/TrigConfxAOD/Root/xAODConfigTool.cxx

@@ -277,7 +277,9 @@ namespace TrigConf {
       // Try to read the R2 metadata object:
       m_tmc = nullptr;
       m_triggerMenuContainerAvailable = true;
-      if( inputMetaStore()->retrieve( m_tmc, m_metaName_run2 ).isFailure() ) {
+      if( !inputMetaStore()->contains<xAOD::TriggerMenuContainer>(m_metaName_run2) 
+          or inputMetaStore()->retrieve( m_tmc, m_metaName_run2 ).isFailure() )
+      {
          m_triggerMenuContainerAvailable = false;
       }
 
@@ -288,16 +290,24 @@ namespace TrigConf {
       m_l1psJson = nullptr;
       m_bgJson = nullptr;
       m_menuJSONContainerAvailable = true;
-      if( inputMetaStore()->retrieve( m_hltJson, m_metaNameJSON_hlt ).isFailure() ) {
+      if( !inputMetaStore()->contains<xAOD::TriggerMenuJsonContainer>(m_metaNameJSON_hlt) 
+          or inputMetaStore()->retrieve( m_hltJson, m_metaNameJSON_hlt ).isFailure() )
+      {
          m_menuJSONContainerAvailable = false;
       }
-      if( inputMetaStore()->retrieve( m_l1Json, m_metaNameJSON_l1 ).isFailure() ) {
+      if( !inputMetaStore()->contains<xAOD::TriggerMenuJsonContainer>(m_metaNameJSON_l1) 
+          or inputMetaStore()->retrieve( m_l1Json, m_metaNameJSON_l1 ).isFailure() )
+      {
          m_menuJSONContainerAvailable = false;
       }
-      if( inputMetaStore()->retrieve( m_hltpsJson, m_metaNameJSON_hltps ).isFailure() ) {
+      if( !inputMetaStore()->contains<xAOD::TriggerMenuJsonContainer>(m_metaNameJSON_hltps) 
+          or inputMetaStore()->retrieve( m_hltpsJson, m_metaNameJSON_hltps ).isFailure() )
+      {
          m_menuJSONContainerAvailable = false;
       }
-      if( inputMetaStore()->retrieve( m_l1psJson, m_metaNameJSON_l1ps ).isFailure() ) {
+      if( !inputMetaStore()->contains<xAOD::TriggerMenuJsonContainer>(m_metaNameJSON_l1ps) 
+          or inputMetaStore()->retrieve( m_l1psJson, m_metaNameJSON_l1ps ).isFailure() )
+      {
          m_menuJSONContainerAvailable = false;
       }
       // if( inputMetaStore()->retrieve( m_bgJson, m_metaNameJSON_bg ).isFailure() ) {

Trigger/TrigConfiguration/TrigConfxAOD/src/xAODConfigSvc.cxx

@@ -396,7 +396,9 @@ namespace TrigConf {
 
       // Read the R2 metadata object...
       const xAOD::TriggerMenuContainer* input_tmc = nullptr;
-      if( m_metaStore->retrieve( input_tmc, m_metaName ).isFailure() ) {
+      if( !m_metaStore->contains<xAOD::TriggerMenuContainer>(m_metaName) 
+          or m_metaStore->retrieve( input_tmc, m_metaName ).isFailure() )
+      {
          m_triggerMenuContainerAvailable = false;
       }
 
@@ -406,19 +408,29 @@ namespace TrigConf {
       const xAOD::TriggerMenuJsonContainer* input_hltps = nullptr;
       const xAOD::TriggerMenuJsonContainer* input_l1ps = nullptr;
       // const xAOD::TriggerMenuJsonContainer* input_bg = nullptr;
-      if( m_metaStore->retrieve( input_hlt, m_metaNameJSON_hlt ).isFailure() ) {
+      if( !m_metaStore->contains<xAOD::TriggerMenuJsonContainer>(m_metaNameJSON_hlt) 
+          or m_metaStore->retrieve( input_hlt, m_metaNameJSON_hlt ).isFailure() )
+      {
          m_menuJSONContainerAvailable = false;
       }
-      if( m_metaStore->retrieve( input_l1, m_metaNameJSON_l1 ).isFailure() ) {
+      if( !m_metaStore->contains<xAOD::TriggerMenuJsonContainer>(m_metaNameJSON_l1) 
+          or m_metaStore->retrieve( input_l1, m_metaNameJSON_l1 ).isFailure() )
+      {
          m_menuJSONContainerAvailable = false;
       }
-      if( m_metaStore->retrieve( input_hltps, m_metaNameJSON_hltps ).isFailure() ) {
+      if( !m_metaStore->contains<xAOD::TriggerMenuJsonContainer>(m_metaNameJSON_hltps) 
+          or m_metaStore->retrieve( input_hltps, m_metaNameJSON_hltps ).isFailure() )
+      {
          m_menuJSONContainerAvailable = false;
       }
-      if( m_metaStore->retrieve( input_l1ps, m_metaNameJSON_l1ps ).isFailure() ) {
+      if( !m_metaStore->contains<xAOD::TriggerMenuJsonContainer>(m_metaNameJSON_l1ps) 
+          or m_metaStore->retrieve( input_l1ps, m_metaNameJSON_l1ps ).isFailure() )
+      {
          m_menuJSONContainerAvailable = false;
       }
-      // if( m_metaStore->retrieve( input_bg, m_metaNameJSON_bg ).isFailure() ) {
+      // if( !m_metaStore->contains<xAOD::TriggerMenuJsonContainer>(m_metaNameJSON_bg) 
+      //     or m_metaStore->retrieve( input_bg, m_metaNameJSON_bg ).isFailure() ) 
+      // {
       //    m_menuJSONContainerAvailable = false;
       // }