Fix DaVinciTests/tests/options/Upgrade/DaVinci-Options.yaml

Indeed DaVinci-Options.yaml and DaVinci-Options.py were not in sync and the former missed a couple of options, which result in an ApplicationManager error status code.

To reproduce, just in case, e.g. with the nightlies from this morning:

This works: davinci --export configuration_dst.opts run-mc --inputfiledb Upgrade_Bd2KstarMuMu - --joboptfile DaVinciTests/tests/options/Upgrade/DaVinci-Options.py.

This did not: davinci --export configuration_dst.opts run-mc --inputfiledb Upgrade_Bd2KstarMuMu - --joboptfile DaVinciTests/tests/options/Upgrade/DaVinci-Options.yaml.

added bug fix testing labels

requested review from @pkoppenb and @dfazzini

assigned to @pkoppenb

BTW, this was not caught because the Python option file is tested but not the related YAML one (test DaVinciTests/tests/qmtest/davinci.qms/configure_dst.qmt) ;-).

changed the description

The relevant differences in the configuration between what gets set via the Python or the YAML file is the following, for completeness:

< DVAppOptions.enable_unpack = False;
---
> DVAppOptions.enable_unpack = True;
44c44
< DVAppOptions.unpack_stream = "/Event";
---
> DVAppOptions.unpack_stream = "/Event/Spruce";

Nothing strange, just showing what this source of the buglet is.

approved this merge request

Feel free to ci-test when ready

added 3 commits

• 679531d5 - added DaVinciSys from DaVinci (DaVinci/erodrigu-fix-yaml-testfile)
• 9cadb57e - adding test for .yaml joboption file
• 0f638b1d - fix copyright

Compare with previous version

resolved all threads

approved this merge request

/ci-test --merge

added ci-test-triggered label

• [2021-12-10 13:00] Validation started with lhcb-master-mr#3317

resolved all threads

assigned to @erodrigu

mentioned in commit 3d8f724c

merged

added DPA-WP3 label