[9.0] more WMS refactoring

[fstagni]

closes #8272

[Robin-Van-de-Merghel]

https://github.com/DIRACGrid/DIRAC/actions/runs/16800399792/job/47580269126?pr=8280

Raised after DIRACGrid/diracx@46d54fe#diff-b98996af9344a5261c4c226a471b99b2487ac28369b2607a3f13b048934f33cf:

class JobParameters(BaseModel, extra="forbid"):

For the test itself, change the parameters:

res = jobStateUpdateClient.setJobParameters(jobID, [("par1", "par1Value"), ("par2", "par2Value")])
# par1 is not found, set a real parameter

[fstagni]

Thank you @Robin-Van-de-Merghel , I was actually looking into this. I do not think this is fully correct, in the sense that job parameters should probably not have a extra="forbid". But I see that DIRACGrid/diracx#626 already addresses that.

[aldbr]

I had to remove the extra=forbid because in lhcbdirac we have additional job parameters and attributes, and this was causing errors in the production instances.
I will check which parameters we use, but may be it would be better to let extra=forbid to know what "metadata" we can use within the code. Then extensions (like LHCb) would potentially need to extend the pydantic models to include their extra parameters. Any opinion?

[chrisburr]

I think most of src/DIRAC/WorkloadManagementSystem/Client/JobStatus.py should be moved to Utilities and reexported into the old location.

[fstagni]

I (re)moved one utilities from there. I do not see the need to move other content.