Implement check for missing evgenkeywords

Changes introduced

Make logParser throw a warning if the transform prints:

Py:Gen_tf         WARNING Could not find evgenkeywords.txt file

Tests

With log from #128 (closed)

and if the above line is not present in the log it will print

Issues resolved

Closes #128 (closed)

changed milestone to %S1.2021

added new feature label

mentioned in issue #132 (closed)

added 1 commit

• 15b70470 - A less hacky implementation

Compare with previous version

changed the description

Hi @cgutscho and @sargyrop sorry for the problem. It was fixed before Christmas and should be available in the next release. atlas/athena!39401 (merged)

Cheers, ewelinA

Thanks @ewelina! So in case the transform can't find it, why don't we let the CI print a warning that prints to screen an https link to the evgenkeywords.txt file and ask that the user / JO coordinator checks that key words used in the JO actually exist in that file? What do you think, @sargyrop?

Hmm yes this can be done. What's the link we should print? Should it be the one that corresponds to the HEAD of the 21.6 branch or the tags? (release/21.6.x)?

added 1 commit

• e1ca1d9e - Update check

Compare with previous version

added 1 commit

• 61e71ba4 - small change

Compare with previous version

changed the description

added 1 commit

• 1b8b8181 - actually pushing the previous change

Compare with previous version

just to add that we want to build a new release this week. At the moment we wait for a MG patch to be applied by Genser. When it is done we will provide a new release.

resolved all threads

added 20 commits

• 1b8b8181...98bfad85 - 15 commits from branch master
• 105cb02f - Implement check for missing evgenkeywords
• 82a5d49f - A less hacky implementation
• 5827bde9 - Update check
• ec284fdf - small change
• c54e04d5 - actually pushing the previous change

Compare with previous version

approved this merge request

unmarked as a Work In Progress

merged