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