#1257 closed defect (incomplete)
sugar-datastore not stopped on logout (+automatic re-login)
Reported by: | sascha_silbe | Owned by: | tomeu |
---|---|---|---|
Priority: | Urgent | Milestone: | |
Component: | Sugar | Version: | Git as of bugdate |
Severity: | Critical | Keywords: | |
Cc: | Distribution/OS: | Unspecified | |
Bug Status: | Needinfo |
Description
After choosing "Logout" sugar-datastore keeps running, so when the next Sugar session starts up (which is immediately afterwards on DebXO) two sugar-datastore instances are running, both trying to write the Xapian index and both failing. We should make sure sugar-datastore has stopped (wait for DBus signal Stopped) before exiting sugar-session.
On unclean shutdowns (e.g. zapping the X server with Alt+Ctrl+BS) the situation is similar, though it's less clear how to handle that case (as the old instance cannot be reached via DBus from the new session - maybe use a PID file?).
Attachments (4)
Change History (11)
Changed 14 years ago by sascha_silbe
comment:1 Changed 14 years ago by tomeu
The DBus session should be closed when the sugar-session client finishes, and the DS process should exit when the dbus connection closes. We should see why this isn't working.
comment:2 Changed 14 years ago by erikos
- Bug Status changed from New to Needinfo
- Milestone changed from Unspecified by Release Team to 0.86
- Priority changed from Unspecified by Maintainer to Urgent
comment:3 Changed 14 years ago by sascha_silbe
I've just discovered another reason for this to happen: My ~/.xsession (created a long time ago) used dbus-launch without --exit-with-session (probably because I didn't know this option). I need to investigate why I put the dbus-launch there / whether it's still necessary (since dbus support is activated in /etc/X11/Xsession.options).
comment:4 Changed 13 years ago by tomeu
Ok, so my understanding of the issue is that we need to make the shell process exit only after the DS has disappeared from the bus by either listening to the NameOwnerChanged signal or the less standard Closed signal in the Datastore iface as Sascha suggests. I'm not sure which one is best.
comment:5 Changed 10 years ago by dnarvaez
- Milestone changed from 0.86 to 1.0
Probably worth to take a look.
comment:6 Changed 10 years ago by dnarvaez
- Resolution set to incomplete
- Status changed from new to closed
old session: datastore.log