=== m_conley is now known as m_conley_away [05:28] Good morning [05:30] morning === FJKong_afk is now known as FJKong [07:58] cyphermox, hi, any updates on https://bugs.launchpad.net/ubuntu/+source/network-manager-applet/+bug/1116317 [07:58] Launchpad bug 1116317 in network-manager-applet (Ubuntu Raring) "[ffe] allow option to create user connections by default" [Medium,In progress] [09:04] morning! [09:06] /query JohnLea [09:06] oups :) [09:06] hey Laney [09:07] hey hikiko [09:07] good morning desktopers! [09:07] hello :) [09:07] hey seb128, how are you? good we? [09:08] I'm good thanks, very nice w.e [09:08] 18°C with sun, it feels like summer [09:08] or almost ;-) [09:08] you? [09:09] oh yeah, we had nice weather too [09:09] was up north helping some people move house [09:09] so a lot of carrying of heavy things [09:09] some exercice, good ;-) [09:14] yep [09:14] also I'm off the rest of the week so if there's something I should look at, better tell me soon [09:14] ;-) === mlankhor1t is now known as mlankhorst [09:40] oh morning laney [09:40] yeah crazy weather [09:42] * Laney hopes it continues [09:48] not m,e === tkamppeter__ is now known as tkamppeter [09:53] I prefer slightly cold [10:06] * TheMuso` can't wait for things to start cooling down here. [10:27] seb128, Hey can you point me who wrote the 'About' panel in the settings app ? (and the underlying backends) [10:27] om26er, I did [10:27] om26er, hey [10:27] seb128, where you you getting the Serial number of the device from ? (not IMEI) [10:28] because I couldn't find where UbuntuStorageAboutPanel{} is being defined [10:29] om26er, the ro.serialno property libhybris [10:29] 666 open bugs for LibreOffice on Ubuntu! [10:29] \m/ [10:29] om26er, hum? the backend is storageabout.cpp in plugins/about [10:30] seb128, assumingly there is nothing for python to talk to hybirs ? [10:30] *hybris [10:30] om26er, I don't know, there is a command line utility, you can probably exec it from python [10:31] om26er, btw I'm still unsure I agree with your dropping of localized tests [10:31] in fact I'm pretty sure I disagree [10:31] seb128, in my other branch I am actually deleting the tests are rely on 'Labels' thats not something to be tested [10:32] *that are [10:32] om26er, I can agree with that, but it doesn't mean you should nuke support for translations on the way [10:32] especially if you don't test for labels, why does it bother you if my UI is in french? [10:32] seb128, I believe that's one extra point where the tests could fail, making them less stable [10:33] well, it might also catch real bugs [10:33] it's fine if your runner force a locale [10:33] but don't drop support for other locales if somebody wants to run them in that configuration [10:34] seb128, well if we force a locale then there is less point in translating, but I'll try to bring that change back, its your source ;) [10:34] it uses the upstream translations so you get that for free-ish [10:34] om26er, well, your CI setup might want to force a locale if you think it's safer [10:35] om26er, but I still want to be able to run those tests with a french UI ;-) [10:35] I can imagine someone might want to run the tests in Chinese [10:35] especially being able to test a RTL locale might be useful [10:36] the who point is, I am changing (and deleting in some cases) tests that are actually testing "nothing" :D [10:36] om26er, your team added those tests that test "nothing" btw... [10:36] Victor wrote them when he added the autopilot support [10:36] and well, they do test that pages load [10:37] though you can test that differently I guess [10:37] seb128, yeah we are supposed to test deeper than that, things like changing the rington setting actually changes that in gsettings or now the accountservice [10:38] seb128, is there a way to check if a plugin successfully loaded ? [10:38] om26er, right, see https://blueprints.launchpad.net/ubuntu/+spec/client-t-system-settings-testing [10:39] look at the title of the page? :p [10:41] seb128, yeah, we need to find a better way :) [10:41] sure, patches are welcome ;-) [10:41] how? [10:41] it's generally just 'load this qml file' [10:42] You can ask Qt if it loaded, but I'm not sure how you would expose that to autopilot [10:42] Laney, I think qml have properties live 'active' or in some cases 'loaded'. I might need to add a custom property as well. will investigate [10:43] I think that it might be better to see if that can be added to the testability infrastructure [10:43] given an object name, tell me if it loaded [10:43] Laney, if qt knows and does not expose to autopilot we can probably add custom properties for that [10:43] speaking of tests [10:43] maybe it's already there [10:44] Laney, do you understand what's the purpose of https://code.launchpad.net/~chris.gagnon/ubuntu-system-settings/barebones-autopilot-emulator-for-updates-page/+merge/210036 [10:44] they do have .active but not sure if reliable or not [10:45] anyway, I think that's a better level than the application layer [10:45] seb128: It lets you have custom methods on the object [10:46] Laney, seb128 I believe Chris is working on upgrade testing [10:47] so you can have a more friendly interface to the component [10:47] k [10:47] I'm not sure why there are random indenation and other changes in that mp though === maclin__ is now known as maclin [10:56] seb128, so if I am to write a few new tests, I should take this BP as a preliminary test plan ? https://blueprints.launchpad.net/ubuntu/+spec/client-t-system-settings-testing [10:56] om26er, that's an example of tests that would be nice to have that we identified, but feel free to add others [10:57] feel free to steal WIs from people there too I think [10:57] seb128: hmm, do you know if robert_ancell is planning on looking at user switching? [10:57] just had to switch vt manually to get back in [10:57] i did work on an initial helper to check if changes successfully took place in gsettings [10:58] Laney, that's not a known issue afaik [10:58] oh I thought you said it was [10:58] Did I hear right, ringtones settings will be saved in accountsservice ? [10:58] Laney, no, mine is a vt taking over the active session [10:58] gnome-screensaver → switch user → type password [10:58] just kept putting me back to the greeter [10:58] weird [10:58] om26er: yes, they are now [10:59] bah [11:00] Laney, so entering the password on the greeter sends you back to the greeter? [11:00] yus [11:00] I'll file a bug with logs [11:00] do you see it? [11:00] yes, please [11:00] no I don't [11:00] ok [11:00] is that reproducible? [11:01] dunno, let me file it then restart [11:01] I'm running trunk atm, robert_ancell asked me to test it since he landed a fix for the issue I was having (where my session would be closed, replaced by a vt after user switching) [11:01] ok I never saw that particular one [11:01] but if I can reproduce it I'll try trunk [11:01] thanks [11:05] bug #1290315 [11:05] Launchpad bug 1290315 in lightdm (Ubuntu) "After 'Switch user', entering password doesn't return to the session" [Undecided,New] https://launchpad.net/bugs/1290315 [11:05] * Laney will upload d-conf first, then reboot [11:17] Laney, "Inappropriate ioctl for device" ... weird [11:21] hmm [11:21] laney@raleigh> ls -l /dev/console ~ [11:21] lrwxrwxrwx 1 root root 11 Mar 10 09:29 /dev/console -> lxc/console [11:21] that's fishy [11:27] Laney, indeed, that seems weird [11:27] Laney, should the test suite depend on pyflakes and pep8 now that I have added a tests for that [11:28] om26er, build-depending on those seems fine to me [11:29] yep [11:29] thanks for adding those [11:29] seb128, but they are not build-deps I put the tests under autopilot, so they are run with autopilot. should they run during build ? [11:29] they should be build-time imo [11:29] that would be nicer, that's when you want to catch those issues [11:36] hmm I'm very suspicious of lxc now [11:36] it also symlinks tty1-4, which are broken [11:36] 'Unable to determine tty n ame' [11:36] stgraber is taking over your box! [11:37] hax [11:37] hum, I'm unsure I like the pep8 recommendation to wrap to 79 chars [11:37] everyone has wide screens nowadays, that doesn't really makes much sense [11:39] I do, it lets me have two windows side-by-side [11:40] ok, fair enough [11:41] om26er, can you make those tests build-time rather than autopilot? [11:41] seb128, working on it atm [11:42] thanks === MacSlow is now known as MacSlow|lunch [12:36] pushed. Now the tests should be running during build. [12:38] om26er, thanks, I'm going to review that in a bit [12:46] didrocks, do you remember the command sam was using to debug stacking issue? xwmtree or something [12:46] seb128: xwininfo -tree -root [12:46] didrocks, thanks [12:48] yw ;) [12:49] trying to get info on those "can't type in lightdm" bug (I can't reproduce but seems quite some users get it) === alan_g is now known as alan_g|lunch [12:54] Hey, does anyone know if there is some progress on the alt+key keystrokes being fetched by appmenu when not needed (like, for irssi and vim)? [12:55] sil2100, it's fixing in unity-gtk-module trunk, that needs a landing though ... can you make that happen? ;-) [12:56] sil2100, that component is not under CI train, and not sure old landings still happen... [12:56] seb128: I think we will have to move it to CI Train - I'll disable it from cu2d and assign a silo if you can prepare the landing :) [12:56] I would love having it fixed :D [12:57] sil2100, I can do that [12:58] sil2100, I guess you would be happy to test the ppa once it's in a silo then ;-) [12:59] man [12:59] I installed a VM on my HDD instead of the SSD [12:59] upgrading pure pain [12:59] ^is [13:00] attente: hey [13:00] Trevinho: hey [13:01] attente: about https://code.launchpad.net/~attente/compiz/1284532/+merge/208908 instead of hardcoding the change, why not using migration scripts and removing that plugin from options? [13:02] Trevinho: how do migration scripts work? do they still run if the key has been explicitly set in the past? [13:02] attente: basically you've to edit the unity.ini inside compiz-gnome and adding some migration scripts (for both ccsm and proper migration script) [13:02] attente: yes [13:03] attente: see https://code.launchpad.net/~3v1n0/compiz/disable-decor-with-unity and https://code.launchpad.net/~3v1n0/compiz/unity-profile-remove-decor [13:03] Trevinho: ok, thanks [13:03] you've just to do the same [13:04] attente: also it would be nice to add a proper incompatibility between unityshell and gnomecompat inside unityshell.xml [13:04] so that it won't be enabled by ccsm either [13:05] seb128: ;) [13:05] Trevinho: attente: Err, should probably move that MP out of approved then... [13:06] ChrisTownsend: yeah, [13:06] ChrisTownsend: hey btw :) [13:06] ChrisTownsend, Trevinho, bregma: hey unity team, how are things going? ;-) [13:07] you guys should get your lockscreen lined up/ready if you want it to land [13:07] ChrisTownsend: and... since you're here, here's the long awaited window shrink fix: https://code.launchpad.net/~3v1n0/compiz/extents-clamp-workarea/+merge/210202 ;) [13:07] seb128: it's ready I think [13:07] Laney is off for the end of the week starting tonight [13:07] Trevinho, is the an MP for that? [13:07] you might want to get the ffe approved before he leaves [13:07] seb128: here in the office a person is trying out high dpi ubuntu [13:07] Trevinho: Hey, oh, sure, I'll review. [13:07] seb128, you mean the lockscreen FFe? [13:07] bregma: lockscreen or resize? [13:08] otherwise you might have less chance chassing up another release-team member to get it approved [13:08] bregma, yes [13:08] xnox, hey, good! how is it working? [13:08] seb128: and unity is scaling awesome, the desktop icons on the desktop did not. [13:08] xnox, right, the slider doesn't change the gtk setting [13:09] xnox, there is a vUDS session about that this week [13:09] xnox, you can "gsettings set org.gnome.desktop.interface scaling-factor 2" manually though [13:09] seb128: do we have the right bits in gtk/gnome to do things (hiDPI things) or will we just fudge it? [13:09] seb128: let me try that. [13:10] xnox, "right", GNOME has this setting I just pointed you at, but it's an "int" factor, so it's 1 or 2 [13:10] no nice scaling [13:10] I guess we are going to end up setting that to whatever int is closer [13:10] but nice support is going to work only for unity and qml softwares === MacSlow|lunch is now known as MacSlow [13:22] seb128: right, we _mus_ make that twiddle be paired, and somehow modify the setting system-wide. [13:22] s/mus/must/ [13:23] seb128: cause it will ship pre-installed on hidpi screens =/ [13:23] xnox, well, oems can tweak the default [13:23] seb128: well, tweak one option, not multiple ideally. [13:23] right [13:24] it's going to be, don't worry about _must_ing people [13:24] seb128: is it meant to "auto-guess" the correct dpi / factor? [13:24] Laney: ok, sorry. [13:24] seb128, so whats the name of that cmd tool for getting serial number from hybris ? [13:24] i got the IMEI out of ofono' dbus interface [13:25] xnox, no, see https://code.launchpad.net/~laney/unity-settings-daemon/hidpi-no-auto-detection/+merge/207685 [13:25] om26er: I think getprop ro.serialno [13:25] xnox, but as said, there is a vUDS session to discuss that this week [13:25] om26er, what Laney said [13:25] seb128: right. [13:26] yeah that works.. [13:26] om26er: try not to assume that it'll always be there [13:26] where to get last updated date of the image ? and also image build number [13:26] the test should be like "if it's there then it is this" [13:26] Laney, yeah its not on the desktop only worked on my phone [13:26] yeah [13:26] om26er, the system-image provides those info [13:27] https://wiki.ubuntu.com/ImageBasedUpgrades/Client [13:27] yeah Info() on the dbus api [13:27] see the wikipage for details [13:27] it has documentations about the api and what it returns [13:28] seb128, cool, will read. thanks [13:28] yw [13:49] hmm [13:50] kenvandine: is there some problem with content hub when running apps from the terminal (over ssh)? [13:50] unity was unhappy after I undocked my laptop, he wouldn't restart until I restarted my session [13:51] Laney, they need to be started with upstart-app-launch [13:51] or the shell [13:51] :/ [13:51] needs the click magic stuff [13:51] Laney, yeah... it's tough [13:51] but it depends on apparmor profile stuff [13:51] it isn't impossible :) [13:51] Trevinho, ^, it was erroring out on "(opengl) - Error: Couldn't bind 0-sized pixmap to texture: the width and height arguments must be nonzero." [13:51] you can just set APP_ID [13:52] to match what it would be if run from upstart [13:52] seb128: that's already fixed in a branch I need to propose [13:52] lol [13:52] Trevinho, keep the good work ;-) [13:52] Laney, but... note that when the hub tries to focus the app, it'll start a new instance [13:52] since upstart isn't tracking the on you have already running [13:52] s/on/one [13:52] can I get stdout & stderr when using upstart-app-launch? [13:52] yes [13:53] or do I have to tail the log [13:53] look in .cache/upstart [13:53] tail the log [13:53] brr [13:53] yeah... not fun [13:53] IN YOUR FACE, UNIX [13:53] i know :) [13:53] i live in this world daily now :/ [13:53] upstart owns us now :) [13:54] anyway, thanks [13:54] I thought my code was breaking this inexplicably [13:54] has some up sides... but it is "change" :-/ [13:54] Hi Laney [13:54] GunnarHj: going out to lunch $now [13:54] need to be back before dmb, sorry [13:55] Laney: Ok. [13:55] but if you leave a message after the tone i'll call you back [13:55] * Laney beeps [13:55] Laney: I'll get back after dmb then. ;-) [13:56] GunnarHj, or just ask your question, maybe somebody else can reply, if not Laney is going to do that once he's back [13:59] seb128: Well, I was going to ask if Laney has time to follow up on https://mentors.debian.net/package/mythes-sv [13:59] seb128: Since he already did a first review... === alan_g|lunch is now known as alan_g [14:00] GunnarHj, k [14:31] shrug [14:32] mitya57, that d-feet update is buggy, please revert it [14:34] seb128: works fine here, what's wrong with it? [14:34] mitya57, it's using GTK CSD, where do I start? [14:34] there is no wm bar under Unity [14:34] no way to resize the dialog [14:34] the background is fully transparent [14:34] you can't see the content [14:34] seb128: please stop, reverting :) [14:35] mitya57, thanks [14:35] * mitya57 thought it was a bugfix-only release [14:35] mitya57, I was filling a bug, do you want it for tracking purpose? [14:35] mitya57, did you try to run it, you say "works fine here" [14:35] seb128: no need for a bug. I ran it, but didn't compare with previous version. [14:36] mitya57, what theme do you use? I wonder why I see the bg being transparent and not you [14:36] mitya57, you use gnome-shell I guess? [14:37] hey guys, wondering if anyone has sucessfully setup RDP(or similiar enviornment) on ubuntu desktop(gnome) 12.04 [14:37] seb128: Right now I'm using gnome-shell and Adwaita... [14:37] *Adwaita Dark [14:37] mitya57, k, so I guess a theme issue (+ CSD not working out of gnome-shell) [14:38] slowcon, hey, try #ubuntu for user questions [14:38] seb128: will pop in there [14:49] seb128: https://launchpad.net/ubuntu/+source/d-feet/0.3.8+really0.3.6-0ubuntu1 [14:52] Oh, nice, it FTBFS. "xvfb-run: error: Xvfb failed to start" [14:52] * mitya57 retries [14:53] mitya57, thanks === cmiller_ is now known as CardinalFang === CardinalFang is now known as qengho [14:56] Retry helped. [15:01] good [15:11] seb128: so what we're missing on lockscreen front? I mean, do we want that screeensaver change in also? [15:12] Trevinho, yes, we need a silo which is "complete", e.g a ppa which gives you "use trusty, enable the ppa, upgrade, get the new experience fully working" [15:27] pitti: hey, are you planning to upload udisk(2) to debian? [15:27] bigon: I did this morning [15:27] oh [15:27] bigon: http://packages.qa.debian.org/u/udisks/news/20140310T102046Z.html [15:27] /o\ [15:27] bigon: http://packages.qa.debian.org/u/udisks2/news/20140310T102106Z.html [15:27] thx [15:28] bigon: why /o\, something isn't right? [15:28] shouldn't this be be updated through security? [15:28] no, non, I missed the fact that you already did it [15:28] bigon: yes, I sent wheezy and squeeze updates to the security team [15:28] bigon: Moritz is handling them [15:31] ah he just sent the DSA mail [15:50] seb128: ok, so how can we make a silo that has both gnome-screensaver patches and unity? I mean, no problem for unity, but what should I do in order to get the g-s patch to build? [15:50] seb128: should I make a branch against its sources (not against the debian/ folder only)? [15:50] Trevinho, silos can have manual uploads, just provide a debdiff to bregma (assuming he's going to be the lander for the silo) [15:51] mh ok [15:52] seb128, I hate to sound ignorant (I am, I just hate to sound that way), how does a manual upload to a silo work? [15:52] bregma, that's a good question, same way as any ppa upload I guess, "dput .changes" [15:52] didrocks, sil2100: ^ ? [15:53] ah, ok... that's fine [16:01] seb128: let me backlog [16:02] bregma, Trevinho: yes, direct uploading is a standard dput to the given silo's landing PPA [16:09] yeah, we'll get something better in the long term I guess, but for now, it's the way [16:17] Laney, before you leave us for holidays, what's the status of https://bugs.launchpad.net/unity/+bug/1282798 ? did you want extra details about it...? [16:17] Launchpad bug 1282798 in Unity "[FFe] Provide a lock screen and unlock dialogs in Unity" [Medium,In progress] [16:18] it wasn't ready to go [16:18] for example the changes to g-s and whatever else didn't/don't exist [16:20] the branch itself wasn't even approved until 3 hours ago [16:21] Laney, right, said differently, it seems ready now, any chance you could have another look before calling it a week? ;-) [16:21] so the status is that it seems like a fairly big change that's only now getting ready which is concerning to me [16:22] yep, where can I find the g-s changes? [16:22] bregma, Trevinho: ^ [16:23] Laney, I don't have a MR yet, but let me paste the aptch [16:23] patch* [16:23] attach it to the bug please [16:24] Laney I'm doing that in few minutes, but in the mean time that's the raw version: http://pastebin.ubuntu.com/7068484/ [17:02] kenvandine: can I pass an environment variable using upstart-app-launch? :( [17:02] nope [17:02] they get stripped [17:03] righto [17:03] fun... :/ [17:04] is APP_ID the filename of the desktop file then? [17:04] no [17:04] but you can find it in the .desktop file [17:04] what am I looking for? [17:05] X-Ubuntu-Application-ID=com.ubuntu.developer.ken-vandine.hub-sharer_hub-sharer_0.1 [17:05] for example [17:05] it's not there for system-settings [17:05] ah... [17:05] because that isn't a click package [17:05] so yeah [17:05] then the name of the .desktop file [17:05] nod [17:05] and upstart starts it as "legacy" [17:05] yep, saw that [17:11] I hacked the reset.settings file instead ^_^ [17:11] even making a wrapper script didn't work === Ursinha is now known as Ursinha-afk [17:37] Laney: so I've attached the patch to the bug report [17:37] ok === Ursinha-afk is now known as Ursinha === cmiller_ is now known as CardinalFang === CardinalFang is now known as qengho [17:57] cyphermox, do you know if ofono provides what we need for https://bugs.launchpad.net/bugs/1287267 or if that's being worked on? [17:57] Launchpad bug 1287267 in ubuntu-system-settings (Ubuntu) ""Data usage statistics" not shown" [Low,Triaged] [17:58] seb128: please also ping awe [17:59] rsalveti, k, moved to a channel where we can find him and asked again there [17:59] cool, thanks [17:59] yw, thanks for suggestion ;-) === alan_g is now known as alan_g|EOD [18:15] does anyone else's empathy not start on trusty complaining about missing libwayland-egl.so.1? [18:15] Saviq, libhybris! [18:16] Saviq, don't install that ;-) [18:17] ah [18:17] that reminds me, I forgot to push at that [18:17] I posted a patch and a suggestion for a fix/workaround to https://bugs.launchpad.net/kubuntu-ppa/+bug/1206371 [18:17] Launchpad bug 1206371 in libhybris (Ubuntu) "libhybris breaks mesa (cannot find libwayland-egl.so.1)" [High,Confirmed] [18:18] rsalveti: ^ could you look at either please? [18:18] I'm going away for the week now so just wanted to let you know [18:18] on that note, see you next week everyone :-) [18:19] happy hacking [18:19] Laney, have nice holidays! [18:19] enjoy the unity lock screen & UDS [18:19] hehe, thank [18:19] s [18:19] did you approve it? [18:19] * seb128 opens the bug [18:19] yeah for upload before weds [18:19] cool [18:19] the g-s patch needs reviewing properly [18:19] it's landing in a silo today, should be doable [18:19] right [18:20] it seems slightly weird in places but I didn't look at the whole thing in context of the code [18:20] maybe it's right if you do that [18:20] Trevinho: thanks for working on that! [18:20] Laney: np [18:21] alrighty, catch you in a few days [18:21] Laney: thank you for the approval ;) [18:21] if you see people pinging me for stuff then redirect them to the rest of the release team ;-) [18:21] * Laney waves [18:23] Laney, have fun, see you! [18:34] bregma, Trevinho: unity failed to build in the landig ppa, (the new nux was not published yet), seems you like you need to rebuild unity build if you can [18:35] seb128, yes, depwait, I'm keeping an eye on it [18:35] oh, right, it's actually in depwait, I got confused by launchpad [18:35] bregma, don't forgot to upload gnome-screensaver as well ;-) [19:06] seb128, bregma gnome-screensaver deb-src are at http://people.ubuntu.com/~3v1n0/Lockscreen-GS/ [19:13] Trevinho, I'm test-building locally [19:14] bregma: thanks [19:14] dunno if I have sufficient privs to upload to the silo PPA, we'll see [19:20] seb128, yeah, only it gets pulled in by unity8 - seems like we should fix that dependency somewhere [19:20] ah, well, -common1 is probably enough alone [19:23] Saviq, 'sudo update-alternatives --config x86_64-linux-gnu_egl_conf' should fix the problem locally forever, but fixing the hybris dependency would make me happier [19:56] tedg: got a weird bug with indicators and gtk-greeter... [19:57] http://paste.ubuntu.com/7069492/ [19:57] Saviq, did you say Laney's comment? [19:57] not sure if this is caused by the no-watchers thing or not [19:57] but it causes the indicator and init and gtk-greeter to get "zombified" [19:58] and then it shows on the panel like this: http://imagebin.org/298340 [19:58] bregma, if you have rights to use the jenkins I think you should be able to upload [19:58] ps waxf : http://paste.ubuntu.com/7069422/ [19:59] seb128, nope (tried it) only the PPA services team have the rights) [20:00] bregma, shame :/ [20:02] more work for someone else, that's all [20:03] yeah, I'm going to test that tomorrow morning, but for now calling it a day [20:23] ali1234, Hmm, that's odd. Is whoever is starting init in that case setting the SIGTERM to it? [20:23] gtk-greeter calls init to send the startup event [20:24] ali1234, After the TERM, upstart should clean up everything. [20:24] i'm not 100% sure what actually gets killed [20:24] ali1234, The greeter should get a TERM from lightdm, and should pass that down. [20:25] well, that's pretty much what it does... i'm told that it was just copied from unity0greeter... although that's vala of course [20:28] ok, it runs init --user --startup-event indicator-services-start using g_spawn_async [20:28] it save the pid and then does kill(), waitpid() on it at shutdown [20:29] Hmm, that *should* work. [20:29] it seems like waitpid() never returns for some reason, as indicator-sound-service, init, and lightdm-gtk-greeter are all still running when you get to the desktop [20:29] (in the pastebin there's multiple copies of them all) [20:31] this is a user-reported error, i'm not sure how to reproduce, but apparently it happens intermittently with a clean install in virtualbox. i had the user add the no-watchers workaround to see if it makes any difference [21:19] tedg: apparently allow-no-watchers doesn't help [21:24] ali1234, I'd guess it's not an indicator bug, but somehow the signal not getting to Upstart. Upstart is pretty brutal about destroying the indicators once it shuts down. [21:24] i have to disagree [21:25] http://paste.ubuntu.com/7069492/ [21:25] init sent TERM to the indicator but for some reason it respawned [21:26] the only thing i can really think is somehow the greeter is killing the indicator process instead of upstart, and then upstart is just restarting it [21:26] Hmm, it should give us 5 secs, and then STOP [21:27] Wonder if that 5 secs gets reset on respawn. [21:27] xnox might know if he's around ^ [21:27] well it probably respawned those 10 times in like 1 second [21:29] hmm... indicator-power got disabled and doesn't show up on ps waxf [21:29] indicator-sound respawned only once and does appear on ps waxf [21:34] i actually have some of these zombies on my machine... never noticed them [21:36] * tedg hands ali1234 a shotgun [21:37] tedg, im out of context, but i laughed [22:02] reported as bug 1290575 [22:02] Launchpad bug 1290575 in lightdm-gtk-greeter (Ubuntu) "lightdm-gtk-greeter does not exit cleanly when logging in" [Undecided,New] https://launchpad.net/bugs/1290575 [22:45] desrt, are you running gdm at the moment? [23:38] ali1234: if upstart is stopping things, it sends sigterm... delay... sigkill. [23:39] ali1234: looking at that log, someone external sent the sigterm (or job is dieing by itself?!) and upstart is trying to keep it alive (respawing) and eventually giving up. [23:39] lightdm-gtk-greeter sends the kill to init [23:48] ali1234: to shutdown it's mini init, ok. where is that code? can you point me to it? [23:49] ali1234: i too noticed in the past that killing indicators was not done cleanly before. [23:49] http://bazaar.launchpad.net/~lightdm-gtk-greeter-team/lightdm-gtk-greeter/trunk/view/head:/src/lightdm-gtk-greeter.c#L2800 [23:49] that's the shutdown [23:49] launch is at: http://bazaar.launchpad.net/~lightdm-gtk-greeter-team/lightdm-gtk-greeter/trunk/view/head:/src/lightdm-gtk-greeter.c#L372 [23:56] ali1234: that seems odd. [23:56] ali1234: so init is shutting down correctly, that is - it received SIGTERM, it sends "session-end" event, and then force shutsdown everything. [23:57] yeah. except it doesn't all shut down cleanly and all the processes just hang around [23:57] ali1234: but the indicators do not "stop on session-end" they stop on "stop on desktop-end or indicator-services-end" [23:57] ali1234: that's a bug, but you can fix this (probably) by having indicator jobs "stop on session-end or ..." [23:58] ali1234: or try "initctl emit indicator-session-end" before sigterm'ing the init. [23:58] ali1234: and i'll reconstruct such unit test-case using session init to check where it is buggy there. [23:59] well, indicator jobs are tedg's area :) [23:59] ali1234: which i've rightfully rejected from ubiquity =)