#654 common.yaml: add groups: dialout, disk
Closed by siosm. Opened by bam.
bam/workstation-ostree-config etc-group-members  into  main

Download 654.patch

Fix https://github.com/fedora-silverblue/issue-tracker/issues/657

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci
https://fedora.softwarefactory-project.io/zuul/buildset/c355b644893e4d6baaf55800fa557bcc

I don't think we want this on all systems by default.

You can instead add your user to these groups with this:

grep -E '^dialout:' /usr/lib/group | sudo tee -a /etc/group
usermod -aG dialout $USER
grep -E '^disk:' /usr/lib/group | sudo tee -a /etc/group
usermod -aG disk $USER

The whole point of the patch is not having to do the hack above manually.
I had a hard times figuring that out.

I don't know what is the problem with doing it on all systems, but if you are strictly against that, let's please do it for SB at least.

Adding these two legacy groups to all SB users doesn't seem like a good solution to me, the vast majority doesn't need them.

People who have special use cases requiring them shouldn't have a problem simply doing that, and it's well documented too.

Adding these two legacy groups to all SB users doesn't seem like a good solution to me, the vast majority doesn't need them.

People who have special use cases requiring them shouldn't have a problem simply doing that, and it's well documented too.

I had a real problems with it already, despite it's "well documented" (I noticed it just occasionally after many failed attempts).

You are not expecting the odd behavior in the first place, and the debugging is hard - as the behavior is not self-explanatory and the real issue is hidden.

It doesn't worth people's nerves, sorry.

I clearly said in https://github.com/fedora-silverblue/issue-tracker/issues/657 that we don't want to do this. It should most likely be possible to do what you want via polkit if you try what I suggested in https://discussion.fedoraproject.org/t/add-groups-to-get-access-to-ttys-and-loopback-devices/155084.

Pull-Request has been closed by siosm

What you said is not truth in the last instance, sorry.

It's not clear what is the problem with the patch proposed.
But it's very well clear from the discussion above that Polkit approach you suggested is unacceptable.

Please stop with the grand accusations and the unsupported claims. I haven't seen any evidence that you've given the polkit approach a serious try. Even if there were, we would still not be doing that as explained by Christian above.

Metadata