Skip to content

doc improvements#8150

Draft
calestyo wants to merge 3 commits into
dCache:masterfrom
calestyo:doc-improvements
Draft

doc improvements#8150
calestyo wants to merge 3 commits into
dCache:masterfrom
calestyo:doc-improvements

Conversation

@calestyo

Copy link
Copy Markdown
Contributor

Hey.

As for 34827be, please check someone whether there’s really no such command - I couldn't find it, but maybe pools refresh is simply shared by pool-info and pool-history.

As for 1e51e8b, this is based on @DmitryLitvintsev’s comments and my own look at the code (though I had a hard time to actually understand which paths are affected by which timeout properties).
The problem with it is that it's incomplete:

  • I haven't added the path infos to options at all, where I don't use the path in my prometheus exporter.
  • Even for those that I've added, I'm not 100% sure whether the info is complete.
    in particular:
    • Are any others paths (which I don't list) affected by each of the options, e.g. /pools/{name}/usage somewhere contains a copy of the /cells data for that pool, but that for example does not seem to be affected by frontend.service.cell-info.timeout but rather by frontend.service.pool-info.timeout. Or frontend.service.pool-info.timeout might also affect /poolgroups/
    • Do I list any paths, where subpaths are however not affected by the respective option? E.g. /pools/ contains a lot, but I've only really checked that frontend.service.pool-info.timeout affects /pools/{name}/usage.
    • Also I'm not sure whether the list for frontend.service.pool-history.timeout is complete/exact.

So maybe this should rather be only a draft, but in general I think these options are less useful if it's not really clear what exactly they do.

calestyo added 3 commits July 12, 2026 17:24
Based on dCache#8142, but still incomplete.

Signed-off-by: Christoph Anton Mitterer <mail@christoph.anton.mitterer.name>
Signed-off-by: Christoph Anton Mitterer <mail@christoph.anton.mitterer.name>
Signed-off-by: Christoph Anton Mitterer <mail@christoph.anton.mitterer.name>
@calestyo

Copy link
Copy Markdown
Contributor Author

See also #8142

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant