PilotCSToJSONSnchronizer is not thread safe

[chaen]

We observed a race condition where the commitNewData method from the CS was called twice, in a short amount of time, the first one being still ongoing when the second one started. Of course, that does not work.

Two options:

• lock the commitNewData method

  DIRAC/ConfigurationSystem/Service/ConfigurationHandler.py

  Line 72 in 41c6ed2

  def export_commitNewData(self, sData):

• lock the sync method

  DIRAC/WorkloadManagementSystem/Utilities/PilotCStoJSONSynchronizer.py

  Line 73 in 41c6ed2

  def sync(self):

One more option, and probably my favourite one: go back to an agent doing the work, instead of triggering from the CS. Much less error prone...

Opinions ?

[fstagni]

Locking the commitNewData would probably be a good idea independently of this issue. @atsareg what do you think?

[chaen]

We need to move on with this. If no objection/reply, I'll do it next week

[fstagni]

Which solution?

[fstagni]

I coded first stub here: integration...fstagni:v7r2-useAgentForCSPilot

Please have a look.

[fstagni]

I believe this can be closed now.