| === soren__ is now known as soren | ||
| === Md_ is now known as Md | ||
| stgraber | jodh: hi, I noticed one pretty bad bug with the session init re-exec | 14:27 |
|---|---|---|
| stgraber | jodh: after the re-exec, it doesn't accept dbus connections on the private bus (though according to netstat, it still listens on the socket) | 14:28 |
| stgraber | jodh: and IIRC I was getting a "No such file or directory" error related to the re-exec in .xsession_errors | 14:29 |
| stgraber | jodh: that was when using: dbus-send --type=method_call --print-reply --address=$UPSTART_SESSION /com/ubuntu/Upstart com.ubuntu.Upstart0_6.Restart | 14:29 |
| jodh | stgraber: hmm. thanks. Session Init re-exec is still a TODO for me :) | 14:30 |
| stgraber | jodh: I'm happy to take a look at it as IIRC we actually saw that one working at some point... shouldn't be too difficult to fix | 14:36 |
| jodh | stgraber: that'd be great, thanks. | 14:36 |
| stgraber | (I already have the upstart job to trigger the re-exec of the session init and will push it as soon as it actually works) | 14:36 |
| jodh | any thoughts on the logrotate job? | 14:37 |
| stgraber | jodh: looks reasonable, can't find anything obviously wrong with it | 14:41 |
| jodh | stgraber: yeah, it's just ugly :) | 14:42 |
| stgraber | jodh: execve("init", ["init", "--user", "--state-fd", "25", "--debug", "--restart"], [/* 34 vars */]) = -1 ENOENT (No such file or directory) | 16:17 |
| jodh | stgraber: awesome :) better make that execvp then ;) | 16:21 |
| stgraber | jodh: yep, just surprised that it works for pid1 ;) | 16:23 |
| jodh | stgraber: the kernel hard-codes the path and you have to specify the full path using init= | 16:24 |
| stgraber | jodh: ok, trying to replace execv by execvp in state.c, hopefully that'll do the trick | 16:25 |
| jodh | stgraber: either that, or specify the full path in 59upstart. | 16:26 |
| stgraber | jodh: we try not to do that usually (I think there's something in the Debian policy saying you're not supposed to) | 16:28 |
| stgraber | jodh: worked! | 16:37 |
| stgraber | jodh: will send a MP | 16:37 |
| jodh | stgraber: great! thanks. | 16:37 |
| stgraber | 1-char diff ;) | 16:37 |
| jodh | stgraber: the simplest are always the best ;) | 16:39 |
| stgraber | jodh: https://code.launchpad.net/~stgraber/upstart/upstart-fix-reexec/+merge/151050 | 16:42 |
| mayhew | Is there a cleaner way than `su` of specifying the user of the process? | 21:54 |
| xnox | mayhew: setuid stanza, see http://upstart.ubuntu.com/cookbook/ | 22:05 |
| mayhew | thanks! | 22:13 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!