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

Give the user the ability to choose if podman service should be enabled+restarted #20

Closed
dmotte opened this issue Feb 6, 2023 · 2 comments

Comments

@dmotte
Copy link

dmotte commented Feb 6, 2023

Hi, first of all I want to thank you for this role! It's very useful :)

Your role enables and starts the podman.service systemd unit at the end, but in my setup I'm using Podman only as an unprivileged user and I don't want the Podman socket running in the background as root.

A possible solution to this problem is to implement a enable_and_start_podman_service variable in the Ansible role to let the user choose. I'm going to make a pull request for you. Let me know what you think 😉

@hswong3i
Copy link
Member

hswong3i commented Feb 6, 2023

Nice for suggestion but I had additional concern:

  1. My podman related packaging and Ansible Roles are not fully tested with unprivileged user, yet
  2. Sometime unprivileged user mode couldn't working as expected, see journald logging support #9
  3. As a fail-safe and fallback, I mainly use privileged mode with sudo for my daily operation, see https://github.com/alvistack/docker-ubuntu/blob/master/molecule/ubuntu-22.04-docker/molecule.yml#L17-L23
  4. This Ansible Role share AS MUCH AS possible homogeneous coding style and logic with else my ~120 Ansible Roles; this systemd service enable + start logic is designed with KISS, ensure functioning in most cases if possible (i.e. without additional configuration, both functioning for privileged or unprivileged mode)

Usually it looks like sexy for an additional configuration parameter; BTW, unless around 20~50% use case need this ability for switching, else I will not add a new parameters that I am not going to use daily, for myself and both my paid clients.

@hswong3i hswong3i closed this as not planned Won't fix, can't repro, duplicate, stale Feb 6, 2023
@dmotte
Copy link
Author

dmotte commented Feb 6, 2023

Ok no problem, I understand. Thanks for your time and for the detailed explanation :)

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 a pull request may close this issue.

2 participants