Skip to content

Fix bug introduced when adapting to removal of execute from LbAppInit, i.e. increase number of events processed

Gloria Corti requested to merge fix-init into master

Add missing call to increase number of events processed that was left to zero and hence the event number.

This bug was introduced when adapting to removal of execute from LbAppInit as a new function now needs to be called, increaseEventCounter().

This is also needed to properly set the random number seed otherwise it is the same for all events within the same Run

This should solve all the qmtest failing because of missing the correct number of events processed, e.g. https://lhcb-nightlies.cern.ch/logs/tests/nightly/lhcb-gauss-dev/1746/x86_64-slc6-gcc7-opt/Gauss/

due to the current validator:

expected_string = "Application Manager Terminated successfully"
if stdout.find(expected_string) == -1:
	causes.append('missing string')
	result['GaudiTest.expected_string'] = result.Quote(expected_string)
else:
	expected_string2 = "100 events processed"
	if stdout.find(expected_string2) == -1:
		causes.append('missing string')
		result['GaudiTest.expected_string'] = result.Quote(expected_string2)

Merge request reports