Accounting hook breaks after adding 2nd hook

[kreczko]

@simonecid here is the ticket

Last working version is 8f4b497.
Which provides these lines in the log file (/var/log/condor/JobRouterLog):

07/05/19 13:02:54 JobRouter (src=4151.0,route=Add accounting information to jobs): Done editing job.

Somewhere between a774652 and e5bbed2 the code breaks:

• a774652: htcondor_hooks/accounting_hook.py → hooks/accounting.py
• 5ed2275: replaced functions in hooks/accounting.py with htcondor_hooks.core.*
• 83e8c6b: added blacklist
• 58632ed: insert htcondor_hooks path to sys.path
• deb5fc4: removal of logger from htcondor_hooks.core.*
• 2707999, 6b8c260 & e5bbed2: fixes for blacklist

Error message:

07/05/19 11:26:57 TranslateClient::hookExited (src=7388.0,route=Blacklist certain machines for local users): Failed to insert "EnteredCurrentSt" into ClassAd, ignoring invalid hook output
07/05/19 11:26:57 TranslateClient::hookExited (src=7396.0,route=Blacklist certain machines for local users): Failed to insert "EnteredCurrent" into ClassAd, ignoring invalid hook output
07/05/19 11:26:57 TranslateClient::hookExited (src=4158.0,route=Blacklist certain machines for local users): Failed to insert "C" into ClassAd, ignoring invalid hook output

First I am going to replay all the code changes and check with

service condor restart

(condor_reconfig does not try to re-apply the hooks --> no messages)

[kreczko]

From @EmyrClement: "Machine requirements need to be in quotes"

The current implementation, https://github.com/uobdic/htcondor-hooks/blob/master/hooks/blacklist.py#L36, relies on the JobAds to parse things correctly, but that does not happen for some reason