Include test to check if HltPackedDataWriter checksums are reproducible

Includes a test which runs Moore with checksums and debug output enabled for the HltPackedDataWriter and compares the checksums with a reference. Currently, it fails because of LHCb#151 (closed), so it will remain disabled.

Needs !985 (merged).

FYI @sstahl @rmatev

marked this merge request as draft

mentioned in issue LHCb#151 (closed)

added 1 commit

• 0234c40e - Include test to check if HltPackedDataWriter checksums are reproducible.

Compare with previous version

changed the description

marked this merge request as ready

assigned to @msaur

Hi @lmeyerga, thanks for creating this test

I've just chatted a bit with Sascha about this to understand the details and I'd propose the following:

Let's turn this test here into a normal single run reference test. So pretty much just run Hlt/Hlt2Conf/tests/qmtest/test_hlt2_checksums_packed_data_1.qmt and have it validate against the reference.
For now this test should be skipped as it will just fail.

Then add the python script to the LHCb issue, as that makes for a nice reproducer for whoever will eventually try and fix this.

Once the problem is fixed, we then just have to enable the test in Moore

added 1 commit

• dbf3320e - Add qmt test that checks reference after reconstruction and (ThOr) hlt2 selections.

Compare with previous version

added 1 commit

• cbd5ffeb - Add qmt test that checks reference after reconstruction and (ThOr) hlt2 selections.

Compare with previous version

resolved all threads

changed the description

added lhcb-head-2 label

• [2021-09-19 00:11] Validation started with lhcb-head-2#347
• [2021-09-20 00:16] Validation started with lhcb-head-2#348

Nightlies look mostly fine, mainly timeouts in Moore at x86_64_v2-centos7-gcc10-opt, x86_64_v2-centos7-gcc10-dbg, x86_64_v2-centos7-clang11-dbg (12!), but that looks good.

There are two failed tests in Brunel at x86_64_v2-centos7-clang11-dbg namely:

 brunel-upgrade-test-protos  unexpected standard output
 xdst-to-xdst  unexpected Wrong Counters

as those tests are without dedicated references and error happened just on one platform, I think it is fine.

@chasse What do you think?

removed lhcb-head-2 label

added testing label

approved this merge request

merged

mentioned in commit 4b75e6ea

mentioned in issue #324 (closed)