Fix reading trigger menu from Frontier in Python 2/3, and add functional unit test

Python 3 fixes - see ATR-21805.

changed the description

This merge request affects 1 package:

• Trigger/TrigConfiguration/TrigConfigSvc

Adding @mark ,@orlando ,@cmorenom ,@asonay as watchers

added Trigger master review-pending-level-1 labels

CI Result FAILURE (hash 84a09ef0)

	Athena	AthSimulation	AthGeneration	AnalysisBase
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 1
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 19031]

added 1 commit

• 64d72a72 - Fix flake8 warning

Compare with previous version

This merge request affects 1 package:

• Trigger/TrigConfiguration/TrigConfigSvc

Adding @mark ,@orlando ,@cmorenom ,@asonay as watchers

CI Result FAILURE (hash 64d72a72)

	Athena	AthSimulation	AthGeneration	AnalysisBase
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 19073]

MC trigger failures should be resolved by !35766 (merged).

Jenkins please retry a build

This merge request affects 1 package:

• Trigger/TrigConfiguration/TrigConfigSvc

Adding @mark ,@orlando ,@cmorenom ,@asonay as watchers

Hi Mike,

It's not worth re-running the CI for this known failure - you can verify that it hasn't introduced any other changes.

Cheers,

Stewart

Thanks Stewart! Looks like you had discussion of this with @mark in https://its.cern.ch/jira/browse/ATR-21805, and code changes look OK. approving. (L1)

added review-approved label and removed review-pending-level-1 label

CI Result SUCCESS (hash 64d72a72)

	Athena	AthSimulation	AthGeneration	AnalysisBase
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 19094]

merged

mentioned in commit 3251633a

added sweep:ignore label