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

deploy: provide a systemd unit managing containers. #13

Closed
wants to merge 1 commit into from

Conversation

henriquesimoes
Copy link
Collaborator

When attempting to run in a podman namespace through a udev-triggered script, filesystem mounting does not work out-of-the-box. When configured with overlayfs, a permission denied occurs when remouting containers/storage/overlay/ with flags 0x40000. Similar issues happen with vfs, and probably other filesystems.

Launching the container under systemd does not provoke such not understood issues. Let's provide a systemd-unit to be started by udev when the IOC must be run. This is similar to the AFC EPICS IOC deploy, also making services management uniform.

During stop, a timeout of zero is provided to use SIGKILL right away, since the SIGTERM signal does not reach the procServ process and its childs for a yet unknown reason.

When attempting to run in a podman namespace through a udev-triggered
script, filesystem mounting does not work out-of-the-box. When
configured with overlayfs, a permission denied occurs when remouting
`containers/storage/overlay/` with flags 0x40000. Similar issues happen
with vfs, and probably other filesystems.

Launching the container under systemd does not provoke such not
understood issues. Let's provide a systemd-unit to be started by udev
when the IOC must be run. This is similar to the AFC EPICS IOC deploy,
also making services management uniform.

During stop, a timeout of zero is provided to use SIGKILL right away,
since the SIGTERM signal does not reach the procServ process and its
childs for a yet unknown reason.
@henriquesimoes henriquesimoes requested a review from ericonr August 20, 2024 12:40
@ericonr
Copy link
Member

ericonr commented Sep 4, 2024

This is going to happen as part of #14 , so closing now.

@ericonr ericonr closed this Sep 4, 2024
@ericonr ericonr deleted the service branch September 4, 2024 20:54
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.

2 participants