Matcher logging is incorrect

[chrisburr]

In #4978 subloggers were added for the matching process however the same DB instance is used for all matching requests meaning the logs are meaningless when multiple matching requests are being processed:

DIRAC/src/DIRAC/WorkloadManagementSystem/Service/MatcherHandler.py

Lines 60 to 68 in df96450

	matcher = Matcher(
	pilotAgentsDB=self.pilotAgentsDB,
	jobDB=self.jobDB,
	tqDB=self.taskQueueDB,
	jlDB=self.jobLoggingDB,
	opsHelper=opsHelper,
	pilotRef=pilotRef,
	)
	result = matcher.selectJob(resourceDescription, credDict)

[fstagni]

I don't want to move to

matcher = Matcher( 
     opsHelper=opsHelper, 
     pilotRef=pilotRef, 
 ) 
 result = matcher.selectJob(resourceDescription, credDict) 

so maybe I simply remove these lines:

DIRAC/src/DIRAC/WorkloadManagementSystem/Client/Matcher.py

Lines 54 to 57 in df96450

	self.pilotAgentsDB.log = gLogger.getSubLogger(f"[{pilotRef}]Matcher")
	self.jobDB.log = gLogger.getSubLogger(f"[{pilotRef}]Matcher")
	self.tqDB.log = gLogger.getSubLogger(f"[{pilotRef}]Matcher")
	self.jlDB.log = gLogger.getSubLogger(f"[{pilotRef}]Matcher")

[chrisburr]

I think the logging is valuable. I haven't thought it through but could you make log a contextvar?