MessageSvc format string in MainServicesConfig: Use new option to shorten source name

Same as !38479 (merged) does for RecExComon, now for the new config

added full-unit-tests label

Jenkins please retry a build

This merge request affects 1 package:

• Control/AthenaConfiguration

This merge request affects 1 file:

• Control/AthenaConfiguration/python/MainServicesConfig.py

Adding @ssnyder as watcher

added Core master labels

added 1 commit

• e26bec71 - MainServicesConfig: Use new option to shorten source name also in single-thread case

Compare with previous version

CI Result FAILURE (hash e75ed659)

	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 24005]

Hi Walter, Sometimes it would be actually better to see a complete (sometimes realy long) source name. This appears in the stack of tools. Can we make this somehow command line configurable? Maybe this format string can become a config flag?

Hi @tbold,

such a flag would set exactly one property. The same can be done by doing acc.getService("MessageSvc").Format("whateverIWant") at the end of teh configuration.

Since engineering the format string is a hard and error-prone task, I would see value in a flag like 'source-lenght' that just sets the lenght of the source-field.

• Walter

added 1 commit

• 3bfe1154 - introduce flag to set source-field length

Compare with previous version

unmarked as a Work In Progress

This merge request affects 1 package:

• Control/AthenaConfiguration

This merge request affects 2 files:

• Control/AthenaConfiguration/python/AllConfigFlags.py
• Control/AthenaConfiguration/python/MainServicesConfig.py

Adding @ssnyder as watcher

added review-pending-level-1 label

CI Result FAILURE (hash 3bfe1154)

	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 24149]

added 1 commit

• 7526143b - fix failing tests

Compare with previous version

This merge request affects 1 package:

• Control/AthenaConfiguration

This merge request affects 2 files:

• Control/AthenaConfiguration/python/AllConfigFlags.py
• Control/AthenaConfiguration/python/MainServicesConfig.py

Adding @ssnyder as watcher

CI Result FAILURE (hash 7526143b)

	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 24169]

code changed look fine, but the CI tests are failing and one unresolved discussion (@tbold are you fine with it?)

Roman(L1)

added review-user-action-required label and removed review-pending-level-1 label

resolved all threads

added 527 commits

• 7526143b...bb16411c - 523 commits from branch atlas:master
• 1ee2adea - Merge remote-tracking branch 'upstream/master' into MainServiceMsgFormat
• af05f063 - update reference files CaloTowerStore_test.ref LArCellDeadOTXCorr_test.ref
• f665cad9 - Revert "update reference files CaloTowerStore_test.ref LArCellDeadOTXCorr_test.ref"
• 64656e75 - update ref files, first round

Compare with previous version