feat: using preferredURLPattern if present#269
feat: using preferredURLPattern if present#269fstagni wants to merge 1 commit intoDIRACGrid:develfrom
Conversation
fstagni
force-pushed
the
preferredURLPattern
branch
from
c7abcdc to
5b02055
Compare
fstagni
force-pushed
the
preferredURLPattern
branch
from
5b02055 to
984c98d
Compare
|
How am I suposed to review this with hundreds of lines of formatting changes mixed in to the same commit? |
| return len(preferredURLPatterns) | ||
|
|
||
|
|
||
| def orderCSByPatterns(configServerList, preferredURLPatterns): |
There was a problem hiding this comment.
Why is this needed? I thought the pilot only put the CS URL(s) into the pilot.cfg which is then parsed by a full DIRAC installation which will randomize/order them correctly.
There was a problem hiding this comment.
The issue I was trying to solve here is the first contact to the CS: which one to use.
There was a problem hiding this comment.
As far as I can tell this is only ordering the URLs in the pilot.cfg file? Or am I missing something?
If so, the sorting/randomisation is done here: https://github.com/chrisburr/DIRAC/blob/b17cf052b5e23912da1a9e7accfe65b7632e7457/src/DIRAC/ConfigurationSystem/private/RefresherBase.py#L141-L144
2 participants
Uh oh!
There was an error while loading. Please reload this page.