Improve unused options report from JobOptionsSvc

After !1179 (merged) we discovered issues in LHCb Run2 support branches as some jobs/tests are still configured with .opts files.

In principle we should have detected issues in properties names earlier, but somehow it didn't happen, so I decided to add something to help in detection of typos of property names.

This MR includes:

• sorting of the JobOptionsSvc report of unused options
• mark with the comment // unused the unused options in the JOBOPTSDUMPFILE
• add a Python script (gaudi_list_opts_typos) that scans the new JOBOPTSDUMPFILE and prints possible typos in the names of the unused properties

To be noted that the options dump is generated at start so it might mark as unused options for configurables that are created during the event loop. Moreover the report of the script is not failure proof as it only checks the name of the property against options that have the same prefix (up to the last dot), but it should be enough to detect the mistakes that were accepted with the old JobOptionsSvc.

/cc @cattanem @rmatev @fwinkl

changed milestone to %v36r0

added lhcb-gaudi-head label

added 1 commit

• c93ca63d - rename list_opts_typos to gaudi_list_opts_typos

Compare with previous version

added 1 commit

• f418257e - Usea argparse instead of click

Compare with previous version

added 1 commit

• a7d155c6 - Use argparse instead of click

Compare with previous version

resolved all threads

approved this merge request

• [2021-06-19 00:04] Validation started with lhcb-gaudi-head#2963
• [2021-06-19 01:12] Validation started with lhcb-run2-gaudi-head#150
• [2021-06-21 00:04] Validation started with lhcb-gaudi-head#2964
• [2021-06-21 00:42] Validation started with lhcb-run2-gaudi-head#151

changed the description

assigned to @clemenci

merged

mentioned in commit 3b6fb2c5