Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[8.0] Add PreferredURLPattern for URL sorting #8065

Open
wants to merge 1 commit into
base: rel-v8r0
Choose a base branch
from

Conversation

chrisburr
Copy link
Member

See the included docs for motivation.

BEGINRELEASENOTES

*Configuration
NEW: Add PreferredURLPattern for URL sorting

ENDRELEASENOTES

@DIRACGridBot DIRACGridBot added the alsoTargeting:integration Cherry pick this PR to integration after merge label Feb 28, 2025
@chaen
Copy link
Contributor

chaen commented Mar 3, 2025

Looks good to me but I let somebody from another VO have a look, because it may be of interest to them (maybe @iueda @hmiyake , as they have services scattered around the globe ? )

@hmiyake
Copy link
Contributor

hmiyake commented Mar 3, 2025

That's interesting. We are categorizing our servers to some groups (e.g. having UPS or not), then sometimes modify URLs to change active servers.

For such operation I prefer to change single value than list, since error proofing.

Let me gather comments to the PR from our colleagues.

@chrisburr
Copy link
Member Author

At the moment this is on a per system basis, though maybe it would make sense to move globally for all URLs? Thoughts?

That's interesting. We are categorizing our servers to some groups (e.g. having UPS or not), then sometimes modify URLs to change active servers.

If it would be useful I could support a list of regexes, i.e.

    PreferredURLPatterns = .*\.preferred\.invalid/.*
    PreferredURLPatterns += .*\.secondary\.invalid/.*
    PreferredURLPatterns += .*\.tertiary\.invalid/.*

@hmiyake What would you prefer?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alsoTargeting:integration Cherry pick this PR to integration after merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants