IIRC, Xrootd 4.9 allows config.d-style directories. That would allow a packager drop-in a override of the all.adminpath without having to do this inside systemd configs.
well, thats a very good news but is somehow beside the point i try to make.
i was trying to make an argument that any service should also have a cli switch to specify the location
of runtime generated files (pid, sockets etc...). adminpath is such a location and IMHO the service should have an cli option to specify it.
I agree on the comment about splitting out persistent-vs-transient files, but that seems to be a completely different ticket, no?
yes, that would be another issue, but i think a lot more complex than to add an "-a adminpath"
(-s for pidfile was taken) as argument of xrootd/cmsd
What are you trying to fix, exactly? The fact that the various admin files appear to be incorrectly placed?
well, the issue of redundant (and inconsistent) contents of adminpath is another thing. i am just trying to ask a feature that consist of an argument of executable that would set the location of the sockets used by the service...
Thank you!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/xrootd/xrootd","title":"xrootd/xrootd","subtitle":"GitHub repository","main_image_url":"https://assets-cdn.github.com/images/email/message_cards/header.png","avatar_image_url":"https://assets-cdn.github.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/xrootd/xrootd"}},"updates":{"snippets":[{"icon":"PERSON","message":"@adriansev in #803: \u003e IIRC, Xrootd 4.9 allows config.d-style directories. That would allow a packager drop-in a override of the `all.adminpath` without having to do this inside systemd configs.\r\n\r\nwell, thats a very good news but is somehow beside the point i try to make. \r\ni was trying to make an argument that any service should also have a cli switch to specify the location\r\nof runtime generated files (pid, sockets etc...). adminpath is such a location and IMHO the service should have an cli option to specify it.\r\n\r\n\u003e \r\n\u003e I agree on the comment about splitting out persistent-vs-transient files, but that seems to be a completely different ticket, no?\r\n\r\nyes, that would be another issue, but i think a lot more complex than to add an \"-a adminpath\"\r\n(-s for pidfile was taken) as argument of xrootd/cmsd\r\n\r\n\u003e What are you trying to fix, exactly? The fact that the various admin files appear to be incorrectly placed? \r\n\r\nwell, the issue of redundant (and inconsistent) contents of adminpath is another thing. i am just trying to ask a feature that consist of an argument of executable that would set the location of the sockets used by the service... \r\nThank you!"}],"action":{"name":"View Issue","url":"https://github.com/xrootd/xrootd/issues/803#issuecomment-428167860"}}}
[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/xrootd/xrootd/issues/803#issuecomment-428167860",
"url": "https://github.com/xrootd/xrootd/issues/803#issuecomment-428167860",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
},
{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB",
"title": "Re: [xrootd/xrootd] adminpath : no cli option for adminpath (#803)",
"sections": [
{
"text": "",
"activityTitle": "**Adrian Sevcenco**",
"activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png",
"activitySubtitle": "@adriansev",
"facts": [
]
}
],
"potentialAction": [
{
"name": "Add a comment",
"@type": "ActionCard",
"inputs": [
{
"isMultiLine": true,
"@type": "TextInput",
"id": "IssueComment",
"isRequired": false
}
],
"actions": [
{
"name": "Comment",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"IssueComment\",\n\"repositoryFullName\": \"xrootd/xrootd\",\n\"issueId\": 803,\n\"IssueComment\": \"{{IssueComment.value}}\"\n}"
}
]
},
{
"name": "Close issue",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"IssueClose\",\n\"repositoryFullName\": \"xrootd/xrootd\",\n\"issueId\": 803\n}"
},
{
"targets": [
{
"os": "default",
"uri": "https://github.com/xrootd/xrootd/issues/803#issuecomment-428167860"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"MuteNotification\",\n\"threadId\": 370663529\n}"
}
],
"themeColor": "26292E"
}
]
Use REPLY-ALL to reply to list
To unsubscribe from the XROOTD-DEV list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1