/srv/irclogs.ubuntu.com/2013/01/15/#upstart.txt

toabctlxnox, thanks for the d-feet sync10:58
xnoxtoabctl: no problem. Why this channel?11:01
xnox=))))11:01
toabctlxnox, because I know you are here:=11:07
toabctl;)11:07
ztanegot firefox zombie clinging to pid 1, which is a bit annoying because now for some reason firefox does not want to start using a same profile until I reboot my computer... (maybe it is using a pidfile to guard the profile dir).14:00
ztaneup-to-date freshish install of 121014:01
jodhztane: can't you kill the firefox pid?14:25
ztanejodh: ofc not, that is what a zombie process is14:32
ztanea pid14:32
ztane+ an entry in a process table14:33
jodhztane: I didn't notice the word 'zombie' there :) is Upstart still responding? Can you start a new job for example?14:41
ztanejodh: let me check14:44
ztanejodh: works14:46
ztanejodh: xnox said that there is no support for general zombie reap in upstart init?14:47
xnoxwell, for jobs that were not started by upstart.14:47
xnox(but I may be wrong ztane, maybe jodh can give a more full answer)14:48
ztanepstree says init-+-firefox---43*[{firefox}], hanging there tight14:49
ztanehmmm14:50
ztaneah :D14:51
ztanemy bad :P14:51
ztanethere was 1 parent process running, which is apparent from that pstree14:51
ztanedunno why it was not killed by killall14:51
xnox=)14:59
jodhxnox/ztane: upstart calls wait repeatedly so it will handle zombies and orphans.15:06
xnoxack.15:27
stgraberjodh: hey, any idea why upstart would return user jobs twice?16:09
stgraberjodh: initctl --user or --session lists all the jobs twice. I initially thought it was a bug in my code (connecting to both buses) but it's not.16:10
stgrabermethod return sender=:1.119 -> dest=:1.128 reply_serial=416:10
stgraber   array [16:10
stgraber      object path "/com/ubuntu/Upstart/jobs/dbus"16:10
stgraber      object path "/com/ubuntu/Upstart/jobs/201105/dbus"16:10
stgraber      object path "/com/ubuntu/Upstart/jobs/debug"16:10
stgraber      object path "/com/ubuntu/Upstart/jobs/201105/debug"16:10
stgraber      object path "/com/ubuntu/Upstart/jobs/upstart_2devent_2dbridge"16:10
stgraber      object path "/com/ubuntu/Upstart/jobs/201105/upstart_2devent_2dbridge"16:11
stgraber   ]16:11
stgraberthat's what initctl is getting back from upstart. Obviously that's confusing it slightly ;)16:11
jodhstgraber: probably because we haven't removed the existing user job handling code.16:11
* xnox ponders what 201105 in above means16:11
stgraberjodh: that's my uid16:12
jodhstgraber: try disabling Upstart.conf and seeing if behavious is the same: http://upstart.ubuntu.com/cookbook/#enabling16:12
xnoxstgraber: a bit high =)16:13
* xnox is 1000 everywhere =)16:13
stgraberxnox: RID-based UID in a samba4 domain gets you pretty big UIDs16:13
stgraberjodh: yeah, I think that's it. So hopefully that'll get fixed once we get rid of the current user job code.16:15
jodhstgraber: do you want to volunteer? ;)16:15
stgraberjodh: one thing that's "interesting" is if you have a job defined as both system and user, then you try to start the user one with "initctl start dbus" (for example). That'll get initctl in some kind of infinite loop and will almost freeze upstart in the process (the user upstart, not the system one)16:16
stgraberbut my guess is that this will be fixed once we no longer have duplicate entries16:16
stgraberjodh: hehe, I've got a reasonably long todo list before I can et to that, but I can take a look once I'm done with my higher priority items :)16:17
jodhstgraber: agreed. ok, ta.16:17
stgraberxnox: I think I found what's broken in the upstart dailies22:50
stgraberbasically, the ubuntu branch contains some ubuntu-specific patches that we don't have in the dailies22:51
stgraberthe most important one being a change to the rc-sysvinit.conf start condition22:51
stgraberwhich perfectly explains the breakage on my machine22:51
stgraberso we can't simply use debian/ from the ubuntu branch for the dailies... we need to also get those changes out and into debian/patches, make a second branch with that and use it as the packaging branch22:52
stgraber(or just finally make ubuntu:upstart non-native and use debian/patches for those ubuntu-specific deltas, making the packaging branch re-usable for the dailies)22:52
* xnox +1 for ubuntu:upstart to be 3.0 quilt. But I'm sure it's not slangasek's preferred upstart workflow.... =)23:34
xnoxstgraber: what about basing recipe on lp:ubuntu/upstart & merging in lp:upstart and hoping it all just works?=)23:35
stgraberI'd have to check if the recipe format allows merging and I guess we'd need a fair amount of hoping considering the number of changes we've made lately ;)23:36
slangasek3.0 quilt + bzr is my preferred workflow in the alternate universe where I am the Marquis de Sade23:39
stgraberxnox: 5 conflicts, so not terribly bad, but can't be done automatically, so won't quite work for the daily builds23:41
slangasekxnox: find a solution to 3.0 (quilt) that won't require us to do stupid merges for anything other than the upstream source tree, and propose it?23:42
xnoxstgraber: the way i deal with conflicts is created a third branch which resolves the conflicts and then you get a common base you merge up lp:ubuntu/upstart + common-conflict-resolve-branch + lp:upstart23:42
xnoxrecipes can handle that.23:42
xnoxslangasek: I did make a proposal as summer of code project. Store the tree unpatched and offer looms api to view the patches and diff/switch up & down. But looms are never committed only flat patches are. (v2) same but generate bundles out of patches such that you can create patched branch to rebase it -> and export a new patch series.23:44
xnoxslangasek: the project was not picked up =)23:45
slangasekheh23:45
xnoxmercurial-queues is the best for debian/patches. They are quilt compatible but each patch has hg metadata in them. And there is nifty commands to rebase/add/remove/merge patches in that quilt series.23:46
xnoxunfortunately nobody stepped up to write a good hg-buildpackage.23:46
slangasekxnox: why do you propose using the patches as the stored representation, rather than using the looms as the stored representation and serializing debian/patches from there?23:47
xnoxslangasek: because of 3.0 (quilt) package format, which inherintely from quilt doesn't have $ quilt merge-series command nor $ quilt rebase-series command.23:49
xnoxslangasek: if we get 3.0 ($vcs) format, there is no need for patches. But that opens a different kind of warms (no way to easily get the diff upstream vs packaged-changes)23:50
slangasekwhy would I ever need/want to do such a merge-series or rebase-series?  Shouldn't loom primitives be sufficient?23:51
xnoxslangasek: if everyone switches and there are no exceptions, we can policy enforce looms and use that and live will be wonderful.23:54
xnoxs/live/life/23:54
slangasekwe don't need to switch "everyone" in order to make a policy change on UDD23:54
xnoxslangasek: i somehow believe that unless everyone switches, there will always be packages that fail to import or do something funny which will break UDD.23:56
slangasekwho is "everyone"?23:56
slangasekI don't understand who you're saying would be switching, here23:56
slangasekUDD itself imports from the archive, the archive wouldn't know anything about looms23:56
xnox"everyone" - any package that is ever manually generated & uploaded into ubuntu.23:56
slangasekunless you mean a 3.0 (bzr) format would be a prereq?23:56
xnoxyes. prereq.23:57
slangasekah23:57
slangasekyeah, that's not a great solution23:57
xnoxexactly.23:57
slangasekwhy don't you think looms could be used to output 3.0 (quilt) reliably?23:57
xnoxslangasek: packages that have both /debian/patches/series and /debian/patches/series.ubuntu fail spectacularly in the current UDD workflow.23:58
xnoxguess which series the importer imported for the whole history?! =)23:58
xnoxis one practical point I can think of.23:59
slangasekhmm23:59

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!