-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
[CI:DOCS] podman-system-service.1.md: document systemd usage #19084
[CI:DOCS] podman-system-service.1.md: document systemd usage #19084
Conversation
b019869
to
125ea66
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
I plan to rewrite it a bit. (No need for reviews right now) |
f12dfc5
to
aa2cd13
Compare
I rewrote most parts of the first version. I moved some already existing text when I reorganised the document. Should I split up the commit into multiple commit to make it easier to follow how the already existing |
ef0745f
to
e959c7b
Compare
Regarding "The command does not support more than one listening socket for the API service." See this Podman source code: (a permalink into the main branch as of 2 July 2023) https://github.com/containers/podman/blob/539be58163a1730af0d84b39fcde585983cd9925/cmd/podman/system/service_abi.go#L48-L50 Move up the paragraph "The REST API provided ...". Move up the sentence "Note: The default systemd ...". Signed-off-by: Erik Sjölund <[email protected]>
e959c7b
to
f907768
Compare
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: eriksjolund, rhatdan, vrothberg The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Does this PR introduce a user-facing change?
(I didn't document how it works on macOS and Windows)