Join GitHub today
GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.
Sign upNM: error when initiating a new connection #626
Comments
marmarek
assigned
rootkovska
Mar 8, 2015
marmarek
added this to the Release 1 milestone
Mar 8, 2015
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
marmarek
Mar 8, 2015
Member
Comment by marmarek on 14 Jul 2012 21:20 UTC
pam_systemd was actually required...
|
Comment by marmarek on 14 Jul 2012 21:20 UTC |
marmarek
added
bug
C: core
P: critical
labels
Mar 8, 2015
marmarek
assigned
marmarek
and unassigned
rootkovska
Mar 8, 2015
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
marmarek
Mar 8, 2015
Member
Comment by marmarek on 14 Jul 2012 21:51 UTC
Patch mentioned in #607 actually was backported to FC17 systemd package, but later was reverted because it caused regression (https://bugzilla.redhat.com/show_bug.cgi?id=823485). This bug is harmless for us, as we doesn't use gdm (or any other display manager) in the VM.
First of all we actually need pam_systemd, so previous workaroud isn't acceptable.
The other ways:
- Just revert workaround and live with "...killed" messages (perhaps adding 2>/dev/null somewhere, ex in qvm-sync-clock).
- Fork systemd package and apply the patch (actually remove patch which revert the fix).
|
Comment by marmarek on 14 Jul 2012 21:51 UTC First of all we actually need pam_systemd, so previous workaroud isn't acceptable.
|
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
marmarek
Mar 8, 2015
Member
Comment by joanna on 15 Jul 2012 08:55 UTC
Perhaps we can always pass only stdout, and always ignore stderr for the qvm-run -p?
|
Comment by joanna on 15 Jul 2012 08:55 UTC |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
marmarek
Mar 8, 2015
Member
Comment by marmarek on 15 Jul 2012 11:17 UTC
I think this is bad idea, as some programs use stderr for useful diagnostics (ie tar cv). This will also make difficult solving errors like typo in cmdline ("unknown argument", "file not found" etc).
|
Comment by marmarek on 15 Jul 2012 11:17 UTC |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
marmarek
Mar 8, 2015
Member
Comment by joanna on 15 Jul 2012 11:19 UTC
Then we can perhaps add another switch to ignore stderr? --no-stderr?
|
Comment by joanna on 15 Jul 2012 11:19 UTC |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
marmarek
Mar 8, 2015
Member
Comment by marmarek on 15 Jul 2012 11:38 UTC
This is named "2> /dev/null"...
|
Comment by marmarek on 15 Jul 2012 11:38 UTC |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
marmarek
Mar 8, 2015
Member
Comment by marmarek on 16 Jul 2012 11:38 UTC
http://git.qubes-os.org/gitweb/?p=marmarek/core.git;a=commit;h=0f6f445ece0ea05d41a1a20f2c5854ea61440013
When package with workaround was installed on system, pam_systemd must be reenabled manually:
cd /lib64/security; mv pam_systemd.so.disabled pam_systemd.so
|
Comment by marmarek on 16 Jul 2012 11:38 UTC
|
marmarek commentedMar 8, 2015
Reported by joanna on 14 Jul 2012 12:54 UTC
How to reproduce:
You will get a popup from netvm:
Probably some problem with dbus, again...
Migrated-From: https://wiki.qubes-os.org/ticket/626