TriggerJobOpts: cleanup TriggerConfigGetter and flags

This is rather big cleanup of TriggerConfigGetter:

• Replace TriggerFlags.dataTakingConditions by ConfigFlags.Trigger.availableRecoMetadata: this flag is used to indicate the trigger systems that are enabled during reconstruction. It is set based on the L1/HLT keys in COOL or the metadata in the input file.
• Remove obsolete metadataItems setting as discussed with @tamartin (the key names are not relevant anymore, and we are setting this elsewhere already).
• Deprecate TriggerFlags.configForStartup: this flag was used to indicate whether trigger keys can be found in COOL. But this is essentially the same check as availableRecoMetadata performs already. The flag itself is still there but will print a WARNING if set. Once this is merged, should update reprocessing shifter instructions to not add this to the AMI tag anymore (discussed with @mark).
• Fix the broken getKeysFromCool method in TrigConfigSvc.

I did not re-implement the setting of the DQMonFlags based on the available systems. If this is really needed still for the legacy monitoring it should be done in the relevant DQ job options. There are also a few TODO items left in the code that will be cleaned up separately. And once the dust has settled, we can think about migrating this to CA.

Relates to ATR-23102.

cc @rbielski

Trigger/TriggerCommon/TriggerJobOpts/python/HLTTriggerResultGetter.py

@@ -84,7 +84,7 @@ class ByteStreamUnpackGetterRun1or2(Configured):
         extr.ExtraInputs += [("xAOD::TrigNavigation", "StoreGateSvc+TrigNavigation")]
         extr.ExtraOutputs += [("TrigBSExtractionOutput", "StoreGateSvc+TrigBSExtractionOutput")]
         
-        if 'HLT' in ConfigFlags.Trigger.enabledSystems:
+        if 'HLT' in ConfigFlags.Trigger.availableRecoMetadata:
             from TrigNavigation.TrigNavigationConfig import HLTNavigationOffline
             extr.NavigationForL2 = HLTNavigationOffline("NavigationForL2")
             # Ignore the L2 TrigPassBits to avoid clash with EF (ATR-23411)
@@ -200,14 +200,14 @@ class TrigDecisionGetterRun1or2(Configured):
 #           WritexAODTrigDecision() is called within WriteTrigDecision()
 
             # inform TD maker that some parts may be missing
-            if 'HLT' not in ConfigFlags.Trigger.enabledSystems:
+            if 'HLT' not in ConfigFlags.Trigger.availableRecoMetadata:
                 topSequence.TrigDecMaker.doL2=False
                 topSequence.TrigDecMaker.doEF=False
                 topSequence.TrigDecMaker.doHLT=False
                 topSequence.TrigNavigationCnvAlg.doL2 = False
                 topSequence.TrigNavigationCnvAlg.doEF = False
                 topSequence.TrigNavigationCnvAlg.doHLT = False
-            if 'L1' not in ConfigFlags.Trigger.enabledSystems:
+            if 'L1' not in ConfigFlags.Trigger.availableRecoMetadata:
                 from AthenaCommon.AlgSequence import AlgSequence
                 topSequence.TrigDecMaker.doL1=False