Opened 11 months ago
Closed 11 months ago
#626 closed defect (fixed)
NM: error when initiating a new connection
| Reported by: | joanna | Owned by: | marmarek |
|---|---|---|---|
| Priority: | critical | Milestone: | Release 1 |
| Component: | core | Keywords: | |
| Cc: | marmarek |
Description
How to reproduce:
1) DISCONNECT your current connection (
2) Try to connect again to the same wifi network
You will get a popup from netvm:
Failed to add/activate connection (32) No session found for uid 1000 (unknown)
Probably some problem with dbus, again...
Change History (7)
comment:1 Changed 11 months ago by marmarek
- Owner changed from joanna to marmarek
- Status changed from new to assigned
comment:2 Changed 11 months ago by marmarek
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:3 Changed 11 months ago by joanna
Perhaps we can always pass only stdout, and always ignore stderr for the qvm-run -p?
comment:4 Changed 11 months ago by marmarek
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:5 Changed 11 months ago by joanna
Then we can perhaps add another switch to ignore stderr? --no-stderr?
comment:6 Changed 11 months ago by marmarek
This is named "2> /dev/null"...
comment:7 Changed 11 months ago by marmarek
- Resolution set to fixed
- Status changed from assigned to closed
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

pam_systemd was actually required...