-
Notifications
You must be signed in to change notification settings - Fork 168
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
Add support for systemd-homed #2018
Conversation
Thank you for the PR, I am afraid it will take me some time to go through it. What did you use for testing? |
Building a new home with systemd-homed on fedora Tested on:
|
Also tested on
|
Thank you, the link is also quite helpful. I meant if it was "testing by usage" or some automated test or features which can be automated. We have very basic test. |
@zpytela, I have tried almost every combination available to the homectl tool for the last 3 years. This policy has more than basic support to function properly. Every now and then upstream modifies the code, and I have adjusted the policy along side them to maintain robustness. |
Thank you, that sounds very good, so you think this is ready to merge to F40? |
Yes Sir. |
Cockpit tests failed for commit 7d68e9e. @martinpitt, @jelly, @mvollmer please check. |
This breaks a lot of Cockpit tests, all which want to connect to the user's session bus:
I picked a random journal and it has a lot of
Note that these tests don't run with homed, this is a "classic" user account. Aside from this: I'm really excited, thanks for this work! I've run homed on my laptop for many years now, and switched off SELinux because of it. |
Reproducer without cockpit, in a standard Fedora 40 VM: Install the COPR:
Then log in as user. This doesn't start the user's systemd session, nor a D-Bus, and the journal says
(and lots more) |
This repo policy for some reason didn't include the systemd-homed module for testing. sudo semanage module -l | grep systemd-homed
|
Cockpit tests failed for commit 6de9bf4. @martinpitt, @jelly, @mvollmer please check. |
This last run still fails all tests. The journal shows lots of
(and maybe I missed some more) |
Cockpit tests failed for commit e032607. @martinpitt, @jelly, @mvollmer please check. |
The rawhide failures still look very grim. Pretty much all cockpit tests fail on
I think this is still a regression in this PR. selinux/rawhide has been broken for a longer time already, see e.g. this recent run. But that "only" broke two tests, not all of them, and the failure is much more specific. |
Cockpit tests failed for commit e2378ed. @martinpitt, @jelly, @mvollmer please check. |
Cockpit tests failed for commit 17042b1. @martinpitt, @jelly, @mvollmer please check. |
This still breaks the user bus:
|
Cockpit tests failed for commit fe8ac44. @martinpitt, @jelly, @mvollmer please check. |
Now it breaks the session bus (I didn't check user bus again):
|
Cockpit tests failed for commit 3a3ecf4. @martinpitt, @jelly, @mvollmer please check. |
Cockpit tests failed for commit 71626fa. @martinpitt, @jelly, @mvollmer please check. |
Cockpit tests failed for commit c36d8b8. @martinpitt, @jelly, @mvollmer please check. |
Cockpit tests failed for commit a29dfa8. @martinpitt, @jelly, @mvollmer please check. |
Cockpit tests failed for commit d20397d. @martinpitt, @jelly, @mvollmer please check. |
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.
Here is my review, I am sorry for the delay. Please do not make any changes right now until things are clarified. In general it looks very well. We'd appreciate any hint for automated testing on some basic level, in addition to the previous link.
Cockpit tests failed for commit 8b10286. @martinpitt, @jelly, @mvollmer please check. |
Cockpit tests failed for commit 7edaac6. @martinpitt, @jelly, @mvollmer please check. |
Cockpit tests failed for commit d135996. @martinpitt, @jelly, @mvollmer please check. |
Cockpit tests failed for commit ce8040b. @martinpitt, @jelly, @mvollmer please check. |
One more question: systemd_homed_tmpfs_t is used for generic temporary filesystem and e.g. /dev/shm, but the comment refers to /tmp which uses tmp_t. Which one is it actually used for? |
It's tmpfs_t, I removed the comments. Thank you for the review. |
Cockpit tests failed for commit 86ec558. @martinpitt, @jelly, @mvollmer please check. |
Cockpit tests failed for commit 97a648e. @martinpitt, @jelly, @mvollmer please check. |
I think it is now ready to be merged, thank you for your patience. |
59b9dd9
to
7d967ef
Compare
7d967ef
to
830966c
Compare
Merging, thank you. |
Testing
Enable homed features on Fedora
sudo systemctl enable --now \ systemd-homed
Relabel homed context
Create a testuser for homed