/srv/irclogs.ubuntu.com/2015/04/07/#ubuntu-devel.txt

=== _salem is now known as salem_
psusicyphermox, I'm looking for someone other than cjwatson to pester about this and you seem to have done some work on ubiquity lately... could you have a look at bug #1418706?  It seems to be a very nasty bug in how ubiquity is executing the partman scripts and seems to me to be rather release critical, with 15.04 coming up fast... I suspect the underlying bug will cause many serious problems other than the one named in the bug.00:35
ubottubug 1418706 in ubiquity (Ubuntu) "Vivid: UEFI: blank drive incorrectly detected as existing BIOS-mode install" [Critical,Triaged] https://launchpad.net/bugs/141870600:35
infinitypsusi: I think I already heaped that on his TODO over the weekend.00:41
psusihehe... ok, good, as long as it gets taken care of before release... I just get nervous that something this serious will be released ;)00:42
cyphermoxpsusi: I'm already aware of it, and that's what I was looking at when you pinged00:49
psusiwhoohoo ;)00:49
cyphermoxit's kind of suspicious00:49
psusiI'm available if you have any questions about it ;)00:49
cyphermoxbut I need to setup my qemu to boot with EFI00:49
psusialready did some preliminary tracing, just don't have the familiarity with ubiquity or the time to fix it myself... been real busy lately with my day job... damn FDA and silly MISRA requirements...00:50
psusihere's the command I used to reproduce it: sudo qemu-system-x86_64 -enable-kvm -drive if=ide,file=/home/psusi/Music/isos/vivid-desktop-amd64.iso,media=cdrom -drive if=ide,file=/dev/faldara/test -vga std -m 102400:51
cyphermoxthanks, that's helpful00:51
psusisorry, that was bios mode... make it this one: sudo qemu-system-x86_64 -enable-kvm -bios /usr/share/qemu/OVMF.fd -drive if=ide,file=/home/psusi/Music/isos/vivid-desktop-amd64.iso,media=cdrom -drive if=ide,file=/dev/faldara/test -vga std -m 102400:51
psusiafter installing the ovmf package of course00:52
infinityYeah, I was going to mention the lack of '-bios /usr/share/qemu/OVMF.fd' on that first line.00:52
cyphermoxI already know the details for EFI bios :)00:52
psusikk ;)00:52
psusiI looked over the partman logs and added a few set -x's and found that the visual.d script from partman-efi was actually creating the new partition prior to running the init.d scripts, which then detected the new, unformatted partition as an existing bios mode install.. the scripts are NOT supposed to be run in that order00:53
psusithen of course, even when the condition is detected and the prompt displayed, ubiquity needs to wait for the reply instead of proceeding to the next task and leaving the prompt window orphaned/hung00:54
psusihrm... that reminds me... maybe a topic we might discuss at the next uds: debian now has support for running d-i under Xwin for a graphical install.. this may be a good replacement for ubiquity entirely?00:58
infinitypsusi: Using live-installer instead of base-installer, perhaps, but it would need some (a lot of) tweaking.01:01
infinitypsusi: Plus, ubiquity also doubles as oem-config, which bare d-i doesn't have an equivalent for.01:02
infinitypsusi: So, while convergence of the installers would be nice, it's non-trivial.01:02
psusiahh, didn't know about oem-config... yea.. convergence would be really nice ;)01:05
psusiespecially given some of the long standing and unfixed bugs in ubiquity, such as the fact that whenever grub-install fails, and you get the dialog box asking to pick an alternate location to install it to, any subsequent tries are doomed to failure because /proc, /sys, and /dev have already been unmounted from /target01:06
psusiand the lack of raid support in ubiquity01:06
psusiand the lvm support is really sad01:07
cyphermoxpsusi: if you have a list of the bugs you feel are important, I can add them to some form of a todo list for me to get to eventually01:08
psusiprobably a good foundation for a discussion on whether to proceed with fixing those bugs, or converge with debian and abandon ubiquity instead.01:09
* psusi misses having beer time at face to face UDS01:11
psusianyone remember when it was called espesso instead of ubiquity?  Christ, how the years do go by...01:14
=== salem_ is now known as _salem
=== doko_ is now known as doko
pittiGood morning04:59
Unit193Howdy.04:59
pittiinfinity: did you already talk to mbiebl about http://paste.ubuntu.com/10752817/ ? (I'd rather have his consensus for uploads to jessie)05:04
=== jamesh_ is now known as jamesh
Mirv@pilot in06:20
=== udevbot changed the topic of #ubuntu-devel to: Archive: Feature Freeze | Devel of Ubuntu (not support or app devel) | build failures -> http://qa.ubuntuwire.com/ftbfs/ | #ubuntu for support and discussion of lucid -> utopic | #ubuntu-app-devel for app development on Ubuntu http://wiki.ubuntu.com/UbuntuDevelopment | See #ubuntu-bugs for http://bit.ly/lv8soi | Patch Pilots: Mirv
=== marcusto_ is now known as marcustomlinson
=== marcusto_ is now known as marcustomlinson_
=== marcustomlinson_ is now known as marcustomlinson
LocutusOfBorg1good morning developers07:53
seb128hey LocutusOfBorg107:57
Laney@pilot in08:12
=== udevbot changed the topic of #ubuntu-devel to: Archive: Feature Freeze | Devel of Ubuntu (not support or app devel) | build failures -> http://qa.ubuntuwire.com/ftbfs/ | #ubuntu for support and discussion of lucid -> utopic | #ubuntu-app-devel for app development on Ubuntu http://wiki.ubuntu.com/UbuntuDevelopment | See #ubuntu-bugs for http://bit.ly/lv8soi | Patch Pilots: Mirv, Laney
zygagood morning08:31
zygavivid still prefers wifi over ethernet today08:31
mgedminvery modern08:34
flexiondotorgdidrocks, Do you have a moment to help clear to fog in my mind regarding - https://bugs.launchpad.net/ubuntu-mate/+bug/143938809:09
ubottuLaunchpad bug 1439388 in mate-tweak (Ubuntu) "Updated translations for mate-tweak [debdiff attached]" [Low,Incomplete]09:09
didrocksflexiondotorg: what fog? did you try to apply the patch yourself?09:11
flexiondotorgdidrocks, I don't understand the remark about translators.txt not being in the patch. translators.txt is the output from a utility script and not intended for packaging. This is why it is missing from the debdiff.09:11
didrocksnot sure how you generated your patch, but it wasn't against upstream ones09:11
didrocksflexiondotorg: it's in the pathc09:12
didrocksdiff -Nru mate-tweak-3.4.6/translators.txt mate-tweak-3.4.7/translators.txt09:12
didrocks--- mate-tweak-3.4.6/translators.txt2015-03-10 19:29:50.000000000 +000009:12
didrocks+++ mate-tweak-3.4.7/translators.txt2015-04-01 21:01:28.000000000 +010009:12
didrocksflexiondotorg: open it and look :p09:12
flexiondotorgI used dget -u -x to grab the existing package from Ubuntu. I created a new package locally and ran debdiff.09:12
didrocksflexiondotorg: seems you didn't do that with the upstream tarball you provided, see my remark about the diffs09:13
didrocksI took the time to list them all on purpose09:13
flexiondotorgdidrocks, Right. No idea how translators.* got in there. I'll re-create the diff.09:15
didrocksflexiondotorg: translators isn't the only one to be wrong09:15
didrocksflexiondotorg: so yeah, please always double check :p09:15
didrocksflexiondotorg: once you have your debdiff, you need to:09:16
didrocksdpkg-source -x <old_package>09:16
didrockswget your tarball, mv tarball_version.orig.tar.gz09:16
didrockscd old_package_dir/09:16
didrockspatch -p1 < /path/to/your/debdiff09:16
didrocksdebuild -S09:16
didrocksand that should succeed09:16
didrocksif not, you did something wrong in the generation09:17
Mirv@pilot out09:18
=== udevbot changed the topic of #ubuntu-devel to: Archive: Feature Freeze | Devel of Ubuntu (not support or app devel) | build failures -> http://qa.ubuntuwire.com/ftbfs/ | #ubuntu for support and discussion of lucid -> utopic | #ubuntu-app-devel for app development on Ubuntu http://wiki.ubuntu.com/UbuntuDevelopment | See #ubuntu-bugs for http://bit.ly/lv8soi | Patch Pilots: Laney
flexiondotorgdidrocks, Thanks for the explanation. Very helpful.09:22
didrocksflexiondotorg: yw ;)09:23
Laney@pilot out09:58
=== udevbot changed the topic of #ubuntu-devel to: Archive: Feature Freeze | Devel of Ubuntu (not support or app devel) | build failures -> http://qa.ubuntuwire.com/ftbfs/ | #ubuntu for support and discussion of lucid -> utopic | #ubuntu-app-devel for app development on Ubuntu http://wiki.ubuntu.com/UbuntuDevelopment | See #ubuntu-bugs for http://bit.ly/lv8soi | Patch Pilots:
Laneyfail... retry later :(09:58
Odd_BlokeI'm seeing failures of the CPC image builds on the buildds because cpio is missing; does anyone have any insight in to this issue?10:13
Odd_Blokemvo: ogra_: ^ ?10:13
ogra_Odd_Bloke, i saw a fix for that uploaded somewhere10:14
ogra_i think slangasek did it10:14
Odd_Blokeogra_: We're still seeing the failures at the moment; was the fix in livecd-rootfs, or somewhere else?10:15
ogra_Odd_Bloke, https://launchpad.net/ubuntu/+source/live-build/3.0~a57-1ubuntu1610:15
Odd_BlokeAh, yeah, just got there.10:16
Odd_BlokeUrgh, looks like we have a version of live-build lying around in our PPA; that'll be the problem.10:16
ogra_hah, yeah10:17
tseliotRiddell: switching between GPUs won't work in sddm, as it doesn't restart X on log out. So, for now, you might just want to add an entry with the DisplayCommand line in sddm.conf. No further patches are needed. Just make sure that /etc/X11/default-display-manager comes with sddm. I'll simply update gpu-manager not to exit when dealing with sddm10:28
pittihey tseliot, happy birthday!10:36
ogra_oh !10:39
ogra_happy birthday tseliot !10:39
tseliotpitti, ogra_: thanks :)10:39
flexiondotorgdidrocks, Could you give this a once over and let me know if this suitable? https://bugs.launchpad.net/ubuntu/+source/mate-menu/+bug/1439380/comments/510:49
ubottuLaunchpad bug 1439380 in mate-menu (Ubuntu) "Updated translations for mate-menu [debdiff attached]" [Low,Incomplete]10:50
flexiondotorgdidrocks, Test the patch applies cleanly and if this is suitable I'll prepare the other patch in the same fashion.10:50
flexiondotorgdidrocks, *I've tested the patch applies cleanly10:50
didrocksflexiondotorg: please check with the patch pilot first rathe than direct ping :)10:50
didrocksrather*10:50
flexiondotorgdidrocks, OK10:51
* flexiondotorg waits for a pilot.10:51
=== LocutusOfBorg is now known as LocutusOfBorg1
shadeslayermvo: any news on the rootfs for Kubuntu Plasma 5?11:02
=== LocutusOfBorg1 is now known as LocutusOfBorg
=== LocutusOfBorg is now known as LocutusOfBorg1
Odd_Blokemvo: Any idea who I can poke to get a live-build patch looked at?  (Specifically https://bugs.launchpad.net/ubuntu/+source/live-build/+bug/1411310)11:42
ubottuLaunchpad bug 1411310 in live-build (Ubuntu) "[PATCH] Enable tuning of EXT images produced by lb_binary_rootfs" [Undecided,New]11:42
flexiondotorgdidrocks, I know you said not to ping directly. Sorry. But I've hit a snag. Need some advice, because I'm guessing this is something that has been seen before.11:53
flexiondotorghttps://launchpad.net/ubuntu/+source/mate-tweak/11:53
flexiondotorgHas a patch accepted into the Ubuntu package.11:53
flexiondotorgThat patch was never shared with upstream (me).11:54
flexiondotorgI have incorporated the fix upstream and have a new upstream tarball.11:54
flexiondotorgBut creating a debdiff between 3.4.6-0ubuntu2 and the new 3.4.8-0ubuntu1 results in a rejection.11:55
didrockshum? what rejection?11:55
flexiondotorgOne sec...11:55
didrocksyou need to remove the patch11:55
didrocksafter unapplying it11:55
flexiondotorgdidrocks, I have.11:55
didrocksquilt pop -a11:55
didrocksrm debian/patches/<your_patch>11:55
flexiondotorgOh, this is new information :)11:56
didrocksvim debian/patches/series -> remove your patch11:56
flexiondotorgdidrocks, I don't have any patches. The patch has only ever existed in Ubuntu.11:56
didrocks13:53:48  flexiondotorg | Has a patch accepted into the Ubuntu package.11:57
didrocksthis contradicts yourself? ^11:57
flexiondotorgdidrocks, The patch was accepted into Ubuntu. Yes. I didn't not submit it.11:58
flexiondotorgdidrocks, The 'debian' packaging for mate-tweak is maintained in alioth11:58
didrocksdouble negation "didn't not submit" -> you did submit?11:58
flexiondotorgI did not submit the patch to Ubuntu.11:59
didrocksflexiondotorg: I still don't get what you mean at all, if the patch wasn't in the ubuntu package, why talking about it?11:59
didrocksand how would it interfere in your update?11:59
flexiondotorghttps://bugs.launchpad.net/ubuntu/+source/mate-tweak/+bug/143356211:59
ubottuLaunchpad bug 1433562 in mate-tweak (Ubuntu) "Translations are not reflected" [Undecided,Fix released]11:59
didrocksso12:00
didrocks  * debian/patches/translationfix.patch: added for fixing a wrong path.12:00
didrocks    (LP: #1433562)12:00
didrocksyou did add the patch12:00
didrocksright?12:00
flexiondotorgNo.12:00
flexiondotorgI only saw that bug after it had been uploaded.12:00
didrockshttp://launchpadlibrarian.net/201198800/mate-tweak_3.4.6-0ubuntu1_3.4.6-0ubuntu2.diff.gz12:01
didrocks-> the patch is in the package12:01
didrocksso ubuntu has this patch12:01
didrockshence, follow what I told above to remove it, before updating ^12:01
Odd_Blokemvo: (I just resubmitted our livecd-rootfs MP as well)12:02
flexiondotorgdidrocks, OK. I will follow what you've explained. Thank for helping.12:04
flexiondotorgdidrocks, One last question though.12:04
flexiondotorgdidrocks, I am maintaining the package in Debian and, in general, syncing to Ubuntu.12:04
flexiondotorgdidrocks, Would a syncrequest from Debian (where that Ubuntu patch has never existed) still work?12:05
didrocksflexiondotorg: sure, just justify why the diff between Debian and Ubuntu can be dropped in the bug report, and we can override the sync request with it12:05
flexiondotorgdidrocks, I'm not going to sync on this occasion. I think it is important I learn all these techniques.12:06
flexiondotorgdidrocks, Also forgive my typos. I have a badly mashed right hand and sometime my text prediction is a little off.12:06
didrocksflexiondotorg: ah ok, so when you sync request, there is a process to read: https://wiki.ubuntu.com/SyncRequestProcess12:06
didrocksheh, no worry :)12:07
flexiondotorgdidrocks, I'm familiar with the syncrequest process. I've used it quite extensively :)12:07
didrocksflexiondotorg: see the lines about the diff between Ubuntu and Debian12:08
didrocksor reread them rather :)12:08
flexiondotorgdidrocks, Will do :)12:08
flexiondotorgdidrocks, I've got this this - dpkg-source: info: you can integrate the local changes with dpkg-source --commit12:21
flexiondotorgdidrocks, I'm clearly missing a step here.12:37
flexiondotorgdidrocks, http://paste.ubuntu.com/10761683/12:41
flexiondotorgWhat next?12:41
cyphermoxgood morning!12:52
mdeslaurhi cyphermox!12:54
=== _salem is now known as salem_
didrocksflexiondotorg: well, then, update your package with uupdate as you usually do13:50
wuzhongchengi need help14:48
wuzhongchengi"m a chinese14:48
wuzhongchengnow i use puppy 5.7.114:48
wuzhongchengi like ubuntu14:58
flexiondotorgdidrocks, I've never had to use uupdate before. I have just used it yet I am still in the position where I have rejections :(15:00
wuzhongchenghoe do you do15:03
wuzhongchenghow do you do15:03
infinitypitti: I'd had a long discussion with him on IRC about it, and he was on CC in my response to you afterward and didn't object to the "belt and bracers" approach of implementing both options, but it never hurts to ask again, I suppose.15:04
wuzhongcheng:)15:04
wuzhongcheng:)15:04
wuzhongchengsorry my english is bad15:05
didrocksflexiondotorg: seems you need to read some packaging documentation :)15:06
flexiondotorgdidrocks, Well, I am doing :)15:06
flexiondotorgdidrocks, But I have rebuilt the debdiff about 4 different ways now and end up at the same place.15:07
rbasakhallyn: https://bugs.launchpad.net/ubuntu/+source/init-system-helpers/+bug/143998815:07
ubottuLaunchpad bug 1439988 in init-system-helpers (Ubuntu) "package init-system-helpers 1.22ubuntu4 failed to install/upgrade: trying to overwrite '/lib/init/apparmor-profile-load', which is also in package upstart-bin 1.13.2-0ubuntu9" [High,Confirmed]15:07
rbasakhallyn: this was my mistake that you spotted - s/upstart/upstart-bin/ needed in Breaks/Replaces.15:07
rbasakhallyn: I guess I'll just upload this fixed?15:08
dobeywuzhongcheng: i think you are looking for #ubuntu as itis the general help channel15:08
hallynrbasak: oh?  but after i had said that i checked the pkg again and it actually was coming from upstart, not upstart-bin15:08
dobeywuzhongcheng: i think there might be another channel for kylin too15:09
hallynrbasak: hm.  maybe i looked at the upstart version when i was doing that15:10
hallynrbasak: yes, please do, thx15:10
hallyngrrr15:10
hallyns/upstart/utopic/15:10
hallyn!4:s/upstart/utopic15:10
rbasakhallyn: no, I think you were right.15:12
rbasakSo I'm confused now. I see it present in upstart_1.13.2-0ubuntu10_amd64.deb15:12
* rbasak checks 915:12
hallynrbasak: changelog says it got moved, but maybe the changelog lied?15:12
rbasakhallyn: I also see it in upstart_1.13.2-0ubuntu9_amd64.deb but also in upstart-bin_1.13.2-0ubuntu9_amd64.deb.15:15
rbasakOnly in upstart in ubuntu10.15:15
hallynweird15:15
rbasakEasiest to just declare Breaks/Replaces against both in ubuntu10 I guess.15:15
hallynbc debian/upstart.install:debian/apparmor-profile-load lib/init/15:15
hallynyeah15:15
rbasakI'll do it.15:16
hallynyou'd think ppl would have gotten conflicts between upstart and upstart-bin then15:16
hallynrbasak: upstart-bin seems to be an empty package though, and depends on upstart.15:17
rbasakIndeed. I don't see B/R across ubuntu9 upstart and upstart-bin that is relevant. Yeah - upstart-bin is an empty package now, but it wasn't in ubuntu9.15:17
hallynok so b/r both is safest.  thanks15:18
rbasakAnyway, I'll just fix it in init-system-helpers and not worry about the upstart side of things. That would be a separate temporary bug that is gone now.15:18
hallynright - again surprised there were no upgrade bugs due to that15:20
infinitypitti: To be fair, when talking to him about it, he said he didn't want to do anything without your opinion.  You're like two parents who are never home at the same time. :P15:20
infinityrbasak: You're seeing things, it wasn't in upstart_1.13.2-0ubuntu9_amd64.deb15:25
rbasakinfinity: now I look again and you're right.15:26
rbasakThe fix I uploaded should still be good. I presume having a slightly higher than necessary version for one of the Breaks/Replaces shouldn't matter.15:27
infinityrbasak: Yeah, it'll be fine.  If LP ever gives me a diff.15:28
* infinity downloads it from the queue instead.15:28
infinityrbasak: Minor complaint that (<= 1.13.2-0ubuntu10) is more safely written as (<< 1.13.2-0ubuntu11) in case people are building derivative packages from ours.15:31
infinityrbasak: But since upstart has moved on to -0ubuntu12 since then, and it's unlikely anyone picked that day to fork, the point's probably moot.15:31
rbasakinfinity: point taken, thanks.15:32
* rbasak will amend his ways15:32
hallyn15:31 < infinity> rbasak: Minor complaint that (<= 1.13.2-0ubuntu10) is more safely written as (<< 1.13.2-0ubuntu11) in case people are building derivative packages from ours.15:46
hallynah.  i've been looking for reasons to go one way or th eother15:46
hallynthat makes sense15:46
infinityhallyn: It's not exactly foolproof, as someone could build a 1.13.2-0ubuntu10billy02 that cherrypicks our changes from 11, but it's statistically safer to assume they didn't.15:48
infinityhallyn: (much like it's statistically safer to assume 1.2.3-3ubuntu1 is similar to 1.2.3-3)15:48
rbasakAlso if they're going to cherry-pick changes from the future then they should get to keep the pieces when it breaks (since nobody else can).15:49
rbasakcf. it makes sense to make things work by default when there are no related changes15:49
* infinity nods.15:49
pittiinfinity: haha; sounds good then :)15:49
hallynwell, yeah, i am also inconsistent about whether i do 1.13.2-2test1 or 1.13.2-3~test115:50
hallynbut both are << 1.13.2-3, so taht's ok15:50
hallyni guess15:50
infinityhallyn: Right.15:50
rbasak1.13.2-3~test1 is useful before 1.13.2-3 is published if you want 1.13.2-3~test1 to eventually upgrade to 13.2-3.15:51
rbasakSay I'm testing something before upload in a PPA for example.15:51
infinityBoth upgrade, though.  So, meh.15:51
rbasakOh. -*2*. I see.15:51
pittiinfinity: so that change is in vivid-proposed now, FTR15:52
pittiinfinity: I'm just not sure whether we should squeeze it into jessie still15:52
infinityMy personal non-concrete rule for such things is "-2test1" is for a package that is "-2 with a quick patch on top" and "-3~test1" is "the package that will eventually be -3, but not quite ready yet".15:52
infinitypitti: Well, I intend to file an unblock for the sysvinit upload I did to jessie, unblocking both together would make sense.15:52
hallynheh, logical15:53
infinitypitti: It doesn't catch a lot of extra cases, but it still catches some different cases, so feels reasonable.15:53
pittiinfinity: ack15:53
infinitypitti: I still would prefer doing it "right" in C, but at least this is provably non-invasive.15:53
Anupkumarpitti: ping15:55
infinitypitti: Oh, and I see you (hopefully) fixed your racy tests.  \o/15:56
mitya57cyphermox, hi, can you please take a look at https://code.launchpad.net/~mitya57/network-manager-applet/lp1440009/+merge/255224 ?15:57
=== matsubara__ is now known as matsubara
mitya57I understand that it is an ugly hack, but it will be only for this cycle — the patch to solve it properly on our side is quite large to apply it during the final freeze.15:58
cyphermoxisn't there a way to integrate this better into the shell test?15:59
* mitya57 looks16:02
rbasakpitti: could you respond to https://bugs.launchpad.net/ubuntu/+source/init-system-helpers/+bug/1436691/comments/19 please? I take it this isn't an issue as the manifest file doesn't reflect what will actually get installed?16:02
ubottuLaunchpad bug 1436691 in upstart (Ubuntu) "Essential files are missing" [High,Confirmed]16:02
rbasakpitti: the only issues I've seen are on systems I suspect didn't use the normal installer path, which is why I asked for apport-collect. But I don't think that's the case here. It might just be that the user has a heavily customised system.16:03
=== MacSlow is now known as MacSlow|latelunc
infinityrbasak: An upgrade should be pulling in systemd-sysv.  If that's not true for ubuntu-gnome, we've messed up somehow.16:09
rbasakinfinity: I don't think we're seeing enough bug reports for that to be the case. The few we're getting are I think users who have heavily modified their systems in some way, where we can't really expect to support the upgrade path.16:10
rbasakI just want to get the bugs resolved and invalidated where appropriate.16:11
mitya57cyphermox: Currently nm_shell_watcher thinks that if ShellVersion is not present, then it is gnome-shell but it is not properly initialized. I don't think I want to break that behaviour.16:11
infinityRight, so #1436956 is clearly a non-standard install, but the sort we were afraid of...16:11
mitya57cyphermox, So checking the XDG_CURRENT_DESKTOP is the easiest temporary solution.16:12
cyphermoxfair enough16:12
infinitypitti: What was the reason we decided not to make init Essential, like Debian did?16:12
cyphermoxmitya57: can you upload or do you need sponsoring?16:12
mitya57I can upload if you ack it16:13
infinitypitti: It does seem to be biting people on upgrades without -minimal installed, as we feared would be the case.16:13
cyphermoxmitya57: just did :)16:13
mitya57Thanks!16:13
pittiAnupkumar: hello16:19
infinityrbasak: I dunno, I might be in the minority, but I still see this as a real bug.  We can jump up and down and say "we don't support systems without ubuntu-minimal installed" all we want, but that doesn't make their upgrades less broken.16:19
infinitypitti: ^16:19
=== quadrispro is now known as alessio_test
pittirbasak, infinity: "init" should be essential, and pull in either systemd-sysv or upstart-sysv16:21
pittiah no, it's "required"16:21
infinitypitti: Okay, so we just messed up and forgot to mark it Essential?16:21
pittimaybe for schroots or so, where neither is required; xnox?16:22
infinitypitti: Sure, neither is actually required in a chroot, but a tiny bit of chroot bloat to make upgrades actually work is a small price to pay (my Debian chroots have init installed).16:22
pittiinfinity: yes, I'm fine with making it essential16:23
pitti+    - init: Drop Essential:yes, in Ubuntu upstart is not essential.16:24
pittiwas the changelog entry16:24
pittiinfinity: fixing race conditions> apparently not hard enough, argh16:25
infinitypitti: D'oh.16:25
infinitypitti: Alright, let's try making it Essential again and see if that blows up any of our carefully-orchestrated ubuntu/ubuntu-touch madness.16:26
infinityIt shouldn't, since we never remove "init", only swap its deps.16:26
pittiinfinity: right16:26
infinitypitti: I'll toss that upload at the archive.16:26
infinitypitti: Yeah, so, my vague recollection is that we actually intended to make it Essential as part of the systemd switch and just kinda maybe forgot. :P16:34
infinitypitti: But I'm old, so I could be fabricating memories to make my life more pleasant.16:34
infinitypitti: I suppose the other gross hack that would sort of work would be to have upstart "Recommends: systemd-sysv | upstart-sysv", but that would also break people who install without recommends (and a Depends would be a lie).16:39
infinitypitti: So, meh.  One wart or the other, we can drop init out of Essential after 16.04, don't really care.  upstart was transitively Essential for years too, entirely by accident. :P16:39
=== salem_ is now known as _salem
=== _salem is now known as salem_
tewardis jenkins broken again?16:52
strikovHi pitti. We have juju-1.22.0 uploaded to -proposed which passes juju team's QA tests (it can provision trusty and precise) and my manual adt-run against vivid image with upstart installed. It doesn't pass automatic adt-run due to the lack of systemd support. I made decision not to update autotests because we'll be required to revert all these changes back while packaging 1.23. Could you help us to manually move the package to release pocket?16:53
strikovpitti: Additional information can be found in the bug: https://bugs.launchpad.net/ubuntu/+source/juju-core/+bug/1416051 Thanks!16:53
ubottuLaunchpad bug 1416051 in juju-core (Ubuntu Trusty) "juju-core 1.22.0 is not packaged in Ubuntu" [Undecided,New]16:53
infinitystrikov: I thought the whole reason we weren't letting in was because of the lack of systemd support, and we were told that was happening ASAP.16:56
strikovinfinity: that is fixed in 1.23 which is in beta; my plan is to upload 1.22 and start to work on 1.2316:57
strikovinfinity: it usually takes some time because d/copyright is pretty complex for golang package16:57
infinitystrikov: "Start work on" doesn't sound as "ASAP" as we were promised weeks ago. :/16:57
strikovinfinity: well, we can't upload 1.23 until juju team releases it :) i can do *some* licensing checks earlier but we need to wait until the release anyway I think16:59
infinitystrikov: Anyhow, the bug log looks "good enough" for letting it in.16:59
infinitystrikov: But gating on juju upstream to polish off 1.23 rather than backporting/cherrypicking the required bits to support systemd is going to make the release team very grumpy with all of you, given that the release is 2.5 weeks out.17:00
infinitystrikov: Of course, the problem with your tests not passing in any way is also that your deps (like lxc) could break juju completely, and we'll never catch it.17:01
infinityWell, not until it's too late.17:01
strikovinfinity: adt tests pass when i use vivid image with upstart installed; i may misunderstood your point but i run these tests that way to avoid missing anything which is verified by tests17:04
infinitystrikov: My point is that we re-run the tests automatically when your dependencies change.17:04
infinitystrikov: You miss out on the benefit of that when your tests can't be run automatically.17:04
strikovinfinity: Ah, that's a good point. I agree. The only argument I have is that I plan to upload 1.23 very soon. it won't have systemd issue.17:05
tewardhey infinity, do you have access to see whether a job that jenkins emailed me about actually exists?17:11
infinityteward: Maybe.17:12
infinityteward: Depends on the job, I suppose. :P17:13
infinitypitti: So, making init essential plumps up my chroots by ~20-30 MB (depending on arch) after I'd just cleaned them out to make them init system free, but I don't think that's world-ending.17:15
infinityBesides, chubby chroot is pleasantly alliterative.17:15
tewardinfinity: well jenkins mailed me on a failure at https://jenkins.qa.ubuntu.com/job/vivid-adt-nginx/47 but I get 404s.17:16
infinityteward: Ahh, the public mirror seems to be a bit behind reality.17:17
infinityteward: 47 exists and did indeed fail, and 48 is in progress as a retry.17:17
infinityAnd I completely forget who's in charge of the public mirror bits.17:18
tewardmmm17:18
tewardinfinity: ok.  i don't have the private access so i have no idea what's up with the failures, if anything.17:18
tewardkinda curious every time i see those emails xD17:19
infinityUgh.  The failure was not yours, if that's comforting.17:19
tewardthat's a positive.17:19
teward'tis all i really worried about :017:19
teward:) *17:19
infinityI'm less enthused.17:20
=== FunnyLookinHat_ is now known as FunnyLookinHat
tewardmmm17:20
tewardwell i'm more comforted, but still curious, alas that's my nature :P17:20
infinitymvo: test-apt-download-progress really, really, really doesn't like you.17:22
infinityteward: Not sure what it was that broke your tests, actually.  It smells like an apt bug, but on a retry it was fine, so maybe just a weird archive inconsistency...17:24
tewardinfinity: i've had nondeterministic failures in nginx builds too, so i'm used to seeing that every once in a while.17:24
tewardinfinity: if it failed twice i'd be more concerned, but one fail and one complete on retry makes me even less concerned17:25
infinityteward: Yeah.  Well, any failures concern me, because it's hard to have a stable baseline to compare to when none of the tests (or the infrastructure) are stable.  Getting there, though.  Slowly.17:25
pittistrikov: yes, I already unblock packages which are held up by the failing juju17:39
pittistrikov: why would you need to revert the changes? I thought you wanted to update the tests so that they work on any relese?17:40
pittiinfinity: sounds ok to me, especially as we can revert it in 16.10 if we care17:40
strikovpitti: they will work w/o any changes at all :) on trusty they will test upstart (because it's default init there), on vivid they will test systemd17:40
strikovpitti: i had an idea to test both systemd and upstart on vivid but came to conclusion that it's not needed17:43
strikovpitti: because it leads to some corner cases when host runs vivid/upstart but juju-local lxc container runs vivid/systemd17:44
strikovpitti: and i came to conclusion that simplest option is the best one17:44
infinitystrikov: FWIW, I already (grudgingly) let it in.17:45
pittistrikov: ack17:45
infinitystrikov: But please push your upstream to get us 1.23 ASAP, like actually ASAP, and if you can work on the packaging from their current trunk, rather than waiting for the code to drop, that would be great too.17:45
strikovinfinity: pitti: thanks; i'll start looking into 1.23 tomorrow morning; will be using what i have for now17:48
pittistrikov: yay17:49
mdeslaurcyphermox: can you take a look at my debdiff in bug 1440166 when you get a minute?18:04
ubottubug 1440166 in network-manager-applet (Ubuntu) "network indicator displays notifications for virtual devices" [Undecided,New] https://launchpad.net/bugs/144016618:04
=== MacSlow|latelunc is now known as MacSlow
infinitypitti: FWIW, the occasional apt failures on systemd-sysv/systemd are probably because of the Pre-Depends, which should just be a Depends.18:05
infinitypitti: Not that that isn't an apt ordering bug (it is), but making apt's life less painful there might be nice.  Unless systemd-sysv actually has a preinst that requires systemd.18:05
infinitypitti: Which it doesn't.18:06
infinitymbiebl: Was there a reason why systemd-sysv Pre-Depends on systemd instead of just Depends, or just an oops/misunderstanding of how dpkg deps work?18:09
tewardinfinity: indeed.  i can understand that18:11
infinityAhh, eww.18:14
infinity  * Add Pre-Depends on systemd to systemd-sysv, to avoid risking that the18:14
infinity    sysv-compatible symlinks become dangling on a partial install.18:14
stormbrew-infinity: I was told to bring this up with you (or doko? Who doesn't appear to be in here right now): There's a pretty severe bug in the libstdc++6 version 5 package in ubuntu-toolchain-r/test where std::error_codes are all comparing unequal: https://gist.github.com/stormbrew/e877b39aa060f19f16dc18:17
stormbrew-(told by ogra_ in the #ubuntu-bugs channel)18:18
infinitystormbrew-: Probably best to file a bug upstream or email doko, or both.18:19
infinitystormbrew-: FWIW, it looks like it's an ABI break.  If you recompile with g++-5 from the same PPA, it returns 0 as you expect, it's only compiling with 4.9 and running with 5.0 that breaks.18:28
infinitystormbrew-: But, yeah, I'd take that knowledge and file an upstream bug report and email doko about it before he decides to unleash that compiler on the world.18:29
infinitystormbrew-: Upstream's answer might be "sucks to be you, we warned you that -std=c++11 was experimental", but still worth reporting.18:30
stormbrew-ah, interesting. That's good to know. I'll do that as soon as I have the cycles for it, thanks.18:36
cyphermoxmdeslaur: yup19:20
cyphermoxmitya57: did you already upload your gnome flashback fix?19:21
mdeslaurcyphermox: cool, thanks!19:21
cyphermoxnevermind, I see it was19:21
mvoinfinity: meh, is that test still failing?20:20
hallynok, systemd continues to confound me:20:27
hallynubuntu@lv1:~/new$ sudo systemctl stop libvirt-bin20:27
hallynubuntu@lv1:~/new$ ps -ef | grep libvirt20:27
hallynubuntu   19003   908  0 16:26 pts/1    00:00:00 grep --color=auto libvirt20:27
hallynroot     25509     1  0 15:58 ?        00:00:00 /usr/sbin/libvirtd -d20:27
ubottuUbuntu bug 19003 in xorg (Ubuntu) "Error when trying to use other keyboard layout than us in breezy" [Medium,Fix released] https://launchpad.net/bugs/1900320:27
hallynsystemctl stop doesn't seem to work20:27
hallynoh, bc systemd thinks that /etc/init.d/libvirt-bin stop is the way to stop it20:29
hallynpitti: ^ i guess we really do need a systemd unit for libvirt, or at least a aysvinit script back, bc the current way really is broken20:30
mbieblhallyn: http://paste.debian.net/165569/20:33
mbieblthe sysv init script and the service file should have the same name20:33
mbieblor if they use different names, provide a symlink20:34
mbieblwhich package does provide /etc/init.d/libvirt-bin?20:35
mbieblis that an obsolete conffile?20:35
dobeymbiebl: dpkg -S tells you what installed package owns it, if any20:38
mbiebldobey: I do know that20:38
mbiebl(I do run Debian here, which doesn't ship /etc/init.d/libvirt-bin)20:39
Unit193!find /etc/init.d/libvirt-bin vivid20:39
ubottuFound: W:, W:, W:20:39
Unit193!find /etc/init.d/libvirt-bin vivid20:39
ubottuFile /etc/init.d/libvirt-bin found in libvirt-bin20:39
infinitymvo: It sure it.20:47
infinitymvo: is...20:47
hallynmbiebl: hm, i was thinking that /etc/init.d/libvirt-bin was a wrpaper around the /etc/init/libvirt-bin (as it used to be at one point),20:47
hallynstill i think it's better to provide a proper systemd unit for it than to figure out why the sysv script isn't working under systemd :)20:48
infinityhallyn: Stopping works for me...20:54
hallyninfinity: ok, good to know - must be something i've got locally then, thanks20:55
infinityhallyn: http://paste.ubuntu.com/10766403/20:55
hallyninfinity: i have no clue what's going on then, but on this instance it's 100% reproducible that libvirt doesn't die, and after the fact, systemctl status libvirt-bin shows:20:57
hallyn  Process: 20817 ExecStop=/etc/init.d/libvirt-bin stop (code=exited, status=0/SUCCESS)20:57
hallynoh COME ON20:58
hallynnow its gone away20:58
=== salem_ is now known as _salem

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