/srv/irclogs.ubuntu.com/2014/03/20/#ubuntu-devel.txt

bluesabregreeting sponsors!02:34
bluesabreIs anybody interested in uploading...02:35
bluesabrehttps://bugs.launchpad.net/ubuntu/+source/xubuntu-docs/+bug/129493202:35
ubottuLaunchpad bug 1294932 in xubuntu-docs (Ubuntu) "[needs-packaging] xubuntu-docs 14.04.1" [Undecided,Confirmed]02:35
bluesabreand...02:35
bluesabrehttps://bugs.launchpad.net/ubuntu/+source/shimmer-themes/+bug/129445902:35
ubottuLaunchpad bug 1294459 in shimmer-themes (Ubuntu) "[needs-packaging] shimmer-themes-1.7.2" [Undecided,Confirmed]02:35
bluesabrePlease let me know if there are any questions or concerns02:36
=== OutOfControl is now known as benonsoftware
pittiGood morning06:16
dokopitti, could you have a look at https://launchpad.net/ubuntu/+source/ubuntu-drivers-common/1:0.2.91/+build/582726807:06
pittidoko: yes, will do; thanks07:07
dokothanks07:08
dholbachgood morning07:24
ionthat07:31
=== tkamppeter_ is now known as tkamppeter
YokoZarLaney: Can I kindly trouble you to remerge p11-kit to fix https://bugs.launchpad.net/ubuntu/+source/p11-kit/+bug/1027299  :)   Alternatively, lmk if I should do it and seek sponsorship.08:05
ubottuLaunchpad bug 1027299 in p11-kit (Ubuntu) "Seperate out p11-kit-trust.so into a multiarch package to prevent errors in Wine" [Undecided,Confirmed]08:05
pittiRAOF: ah, I got umockdev-record --evemu working :)08:33
pittiRAOF: (mostly, need to fiddle with the "record device path" still)08:34
dholbach@pilot in08:41
=== udevbot changed the topic of #ubuntu-devel to: Trusty Beta 1 released! | 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 -> saucy | #ubuntu-app-devel for app development on Ubuntu http://wiki.ubuntu.com/UbuntuDevelopment | See #ubuntu-bugs for http://bit.ly/lv8soi | Patch Pilots: dholbach
* pitti hugs dholbach08:41
* dholbach hugs pitti back :)08:41
dholbachhey happyaron - how are you doing? did you see https://bugs.launchpad.net/indicator-keyboard/+bug/1290881? (just checking because I'm doing some sponsoring right now)08:54
ubottuLaunchpad bug 1290881 in indicator-keyboard (Ubuntu) "indicator-keyboard suppresses IBus candidate window" [High,In progress]08:54
happyarondholbach: haven't looked at the patch yet, will do it this night08:55
dholbachhappyaron, thanks a lot!08:55
happyaronnp, :)08:59
seb128dholbach, hey, happy piloting ;-)09:00
dokomdeslaur, can you have a look at https://bugs.launchpad.net/ubuntu/trusty/+source/ca-certificates-java/+bug/1258286 ?09:02
ubottuLaunchpad bug 1258286 in ca-certificates-java (Ubuntu Trusty) "CAcert should not be trusted by default" [High,Confirmed]09:02
* dholbach hugs seb12809:13
* seb128 hugs dholbach back09:13
dholbachbluesabre, for bug 1294459 do you have a .orig.tar.gz file somewhere?09:17
ubottubug 1294459 in shimmer-themes (Ubuntu) "[needs-packaging] shimmer-themes-1.7.2" [Undecided,Confirmed] https://launchpad.net/bugs/129445909:17
tvossdoko, good morning09:45
tvossdoko, could you have a look here: https://code.launchpad.net/~robru/dbus-cpp/fix-ppc64el/+merge/21182009:45
tvoss?09:45
dokotvoss, you don't need the b-d on gcc-4-x, g++-4.x depends on it. please don't add an explicit b-d on g++-4.8, but use g++ instead09:47
tvossdoko, ack09:48
dokotvoss, maybe just define something like v_ext=4.7, and then use g++$(v_ext)09:49
dokotvoss, ahh, and g++ is a dependency of build-essential, so you don't need it at all09:52
tvossdoko, okay, so I should be good with just g++-4.7 for non ppc64el, correct?09:52
dokoyes09:53
damianatorrpmHi all :-)10:03
damianatorrpmHow are you doing?10:03
tvossdoko, https://code.launchpad.net/~thomas-voss/dbus-cpp/fix-ppc64-el/+merge/21189410:05
dokotvoss, 4.8 still hardcoded10:07
dokoifneq (,$(filter $(DEB_HOST_ARCH),ppc64el))10:07
doko  v_ext = -4.710:07
dokoendif10:07
dokoexport CC=$(DEB_HOST_GNU_TYPE)-gcc$(v_ext)10:07
dokoexport CXX=$(DEB_HOST_GNU_TYPE)-g++$(v_ext)10:07
damianatorrpmtiny question: libdubsmenu & libdbusmenu-qt, in ubuntu 13.10 I can see only in the repos libdbusmenu-qt, before there was libdbusmenu-gtk2 and libdusmenud-gtk3 which required patched gtk2/gtk310:08
damianatorrpmare they now dropped and only libdubsmenu-qt is still there?10:08
damianatorrpmlarsu: maybe you know ? :) :)10:08
LaneyMirv: hm, qtbase didn't get pushed back?10:16
=== FourDollars_ is now known as FourDollars
tvossdoko, updated: https://code.launchpad.net/~thomas-voss/dbus-cpp/fix-ppc64-el/+merge/21189410:30
dokotvoss, ok. still b-d's on gcc-4.7, but doesn't hurt10:32
tvossdoko, has to bd on gcc-4.7 until we update the papi to use > 4.710:33
dokotvoss, g++-4.7 depends on gcc-4.7, so it is not needed ...10:33
tvossdoko, sorry, got it now10:33
cjwatsondamianatorrpm: "apt-cache search -n dbusmenu-gtk" still returns results on 14.04 for me10:38
tseliotdirecthex: in case the email didn't make it, thanks! :)10:46
MirvLaney: it's not formally part of CI Train, so "the other CI" needs to merge it. trying that now.10:47
bluesabredholbach: yeah, its in the branch: https://bazaar.launchpad.net/~smd-seandavis/xubuntu-artwork/shimmer-themes-1.7.2/files11:08
dholbachah ok, thanks11:08
bluesabreoccasionally, there is some confusion with this package since its a multiple upstream tarball package, previous note with previous upload https://bugs.launchpad.net/ubuntu/+source/shimmer-themes/+bug/1227402/comments/711:11
ubottuLaunchpad bug 1227402 in shimmer-themes (Ubuntu) "Please update shimmer-themes to 1.6.2" [Undecided,Fix released]11:11
bluesabrethanks for taking a look :)11:11
mdeslaurdoko: sure, I'll take a look....it ftbfs?11:22
mdeslaurdoko: oh, nm, I see the test11:24
dokomdeslaur, ok, thanks11:34
=== MacSlow is now known as MacSlow|lunch
zequenceLaney: I see you are marked as janitor for ubuntustudio-live in the sponsors queue. Any more hinders in getting it through to universe?12:33
Laneyzequence: I don't know what that means12:34
zequenceLaney: From what I understand, it's uploaded, but needs approval from archive admins12:34
LaneyAll that means is that I made the last comment on the bug12:34
zequenceAh12:34
LaneyIt's been uploaded, yes, just waiting for approval12:35
zequenceI have a habit of misreading things :P12:36
bigondoko: hi12:41
bigonI was wondering, about https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=72054512:42
ubottuDebian bug 720545 in src:bash "bash: Please consider removing privmode.diff" [Important,Open]12:42
maxiaojunwhy usb-creator is so buggy these days?13:13
=== _salem is now known as salem_
=== cmagina-away is now known as cmagina
miraiEin Kubuntu, what app is similar with Password and Encryption Keys for uploading GPG key?13:43
sladenRiddell: ^^13:46
pittiLaney, stgraber: this should be a formality, but I filed it nevertheless: bug 129513313:48
ubottubug 1295133 in umockdev (Ubuntu) "FFE: record/replay of input devices using the evemu format" [Wishlist,New] https://launchpad.net/bugs/129513313:48
Laneyalready replying13:48
pittiLaney: wow, you're too fast :)13:49
Laneyjust happened to open the list :-)13:49
pittiLaney: yes, of course I'm reading incoming bugs, I watch the autopkgtest failures, and the Mir etc. guys know where I live :)13:51
LaneyGot to say these things for completionism :P13:52
barrypitti: ping, re LP: #127235913:53
ubottuLaunchpad bug 1272359 in aptdaemon (Ubuntu Trusty) "needs porting to Python 3.4" [High,In progress] https://launchpad.net/bugs/127235913:53
pittibarry: hey13:53
barrypitti: hi13:53
pittibarry: I see similar failures like you, also with current 3.313:54
barrypitti: there is no branch of aptdaemon that i can find that actually passes all its tests13:54
pittibarry: i. e. the "FAILED (failures=1, errors=2, skipped=4)" bits13:54
pittibarry: yes, that also happens in trunk13:54
barrypitti: while d/rules runs the test suite at build time, it doesn't ftbfs because the test failures are ignored ;)13:54
pittibarry: I guess trusty around it changed enough to now make it fail13:54
barrypitti: upstream test suite doesn't pass in any of 2.7, 3.3, or 3.413:55
pitti*nod*13:55
barrypitti: i am very much inclined to simply disable the dep8 tests for now and file an upstream bug about the test suite13:55
pittibarry: back then when I filed it it still did (and only failed with 3.4), but apparently it now fails with current 3.3, too13:55
barrypitti: 2.7 even ;)13:55
pittibarry: we can just retitle that bug13:55
barrypitti: yep, i'll take care of that.  do you have any objections to disabling dep8 for now?13:56
pittibarry: there's no aptdaemon stuck in -proposed due to it, so no need to disable this13:56
pittibarry: yes; that causes more technical liabilities, and the next upstream update might just silently get in with actual regressions13:56
barrypitti: okay, i'm going to just fiddle with the bugs then and let upstream take it from here13:58
pittibarry: thanks13:58
barrypitti: cheers13:58
argesstgraber: hey looking at bug 1254120. trying to isolate problems with ifup/ifdown on bonds. do you know which part of ifupdown would be setting static ips (ip addr ...) ?13:58
ubottubug 1254120 in ifenslave (Ubuntu) "Bonded interfaces don't come down with a ifdown -a" [High,In progress] https://launchpad.net/bugs/125412013:58
stgraberarges: looking14:02
stgraberthere isn't really "parts" in ifupdown, it's a gigantic source file :)14:02
argesstgraber: yea, i noticed :)14:02
stgraberbut the commands comme from the .defn files which are parsed and expanded at build time14:03
stgraberarges: so FWIW, I never expected my new package to fix bug 125412014:03
ubottubug 1254120 in ifenslave (Ubuntu) "Bonded interfaces don't come down with a ifdown -a" [High,In progress] https://launchpad.net/bugs/125412014:03
argesstgraber: yea i've been trying to use strace and 'ifup -v' to figure out where the issues are14:03
argesstgraber: figured as much, I looked at the change and didn't think it woudl either but tested it to make sure14:03
argesi'll look through the .defn files.14:04
stgraberarges: let me update my network testing box to see if I can reproduce that issue easily here14:04
argesstgraber: yea, i'm doing this all in VMs and its pretty trivial to reproduce14:04
stgrabermy test box is physical hardware with LACP but that shouldn't matter in this case14:05
argesyup14:06
Riddellsladen: you still into fonts?14:07
stgraberarges: I suspect something may be wrong somewhere in ifenslave which would explain why it's not obvious when running with -v but hopefully I can figure out what exactly once that box is done upgrading (I really should switch to something better than a single core 32bit Atom for that box :))14:08
Riddellsladen: fancy seeing if this tar is sane and can be built and works? starsky.19inch.net/~jr/tmp/oxygen-fonts-0.4.tar.xz14:08
darkxstdholbach, I have to run, but re Bug 129489114:08
ubottubug 1294891 in Ubuntu "Ubuntu GNOME community wallpapers" [Undecided,Incomplete] https://launchpad.net/bugs/129489114:08
darkxstwe got permission from image authors to release work under cc-by-sa 3.0, so copyright file should be ok?14:08
xnoxballoons: sergiusens: what can I help with to release calendar-app revision 205 or better? It's been blocking python2 removal for a while now.14:09
argesstgraber: yea something intiially is causing it to not create the proper files in /run/network and the ifstate file isn't updated correctly, so I think ifup needs to work. The other question is, do we expect 'ifup bond0' to bring up slave interfaces or that needs to be done before bringing up the bond?14:11
sergiusensxnox, just push balloons around14:11
xnoxballoons: i'm prepared to be a slave for you =) calendar-app, please please please =)))))14:12
stgraberarges: ifup bond0 won't bring up slaves, it'll just sit and wait until slaves join. But slaves will bring up bond0, so "ifup eth0" should bring up eth0, bring up bond0 and then add eth0 to bond014:12
argesstgraber: gotcha.14:13
argesstgraber: so if i do 'ifup <slave if>' and the bond comes up, do we expect the ifstate file to have bond0=bond0 in it?14:14
xnoxballoons: honestly changes from v201 to v205 are trivial, and are trivial to verify. We shouldn't need to wait on all the cool things to be developed and be ready.14:14
stgraberarges: we do, though I suspect that may be the problem14:15
balloonslol xnox14:15
argesstgraber i think so too14:16
balloonsxnox, see my perhaps confusing mail on the list. We need to release calendar too. But trunk is broken. So I have a merge reverting the bad changes in trunk that we can land14:16
balloonsalternatively, we could fix the trunk version of the app, then land the fixes for the test14:16
xnoxballoons: do you want me to prepare v205 click, with diff of what changed is no code change. http://paste.ubuntu.com/7125444/14:16
xnoxballoons: you don't have to release trunk.14:16
xnoxballoons: i'm asking to release revision 205.14:17
xnoxballoons: which has _no code changes_ to the app.14:17
xnoxballoons: merging back reverts is silly. It's a distributed version control system, checkout the revision you want to release, build the click for that revision # and upload to the store.14:17
xnoxballoons: my changes were ready eon ages ago, why should i be blocked by broken trunk? i can trivially push current trunk to a staging branch and do push --override back to revision 205 if you wish.14:18
xnoxballoons: then "release trunk" and push back the current trunk back in place. But it's silly if we need to do that.14:19
xnoxballoons: am I missing something?14:19
xnoxbzr branch lp:ubuntu-calendar-app -r205 is not broken, and is trunk.14:19
balloonshttps://bugs.launchpad.net/ubuntu-calendar-app/+bug/129418114:20
ubottuLaunchpad bug 1294181 in Ubuntu Calendar App "Autopilot tests crashing in switch_to_tab helper" [Critical,Confirmed]14:20
balloonshttps://bugs.launchpad.net/ubuntu/+source/qtdeclarative-opensource-src/+bug/129499514:20
ubottuLaunchpad bug 1294995 in qtdeclarative-opensource-src (Ubuntu) "qmlscene crashed with SIGSEGV in value() when running calendar-app" [Undecided,Confirmed]14:20
balloonsxnox, it sounds like we're going to go ahead with landing my mp which reverts the offending changes: https://code.launchpad.net/~nskaggs/ubuntu-calendar-app/revert-212/+merge/21181314:21
xnoxballoons: and?14:21
xnoxballoons: dude, we need to fix packaging and metadata of the app. The app itself would be identical but the manifest revision number of where to fetch the tests from.14:22
stgraberarges: hmm, nope, that's not it. In my setup here with eth0+eth1 in bond0 and bond0 in br1000, I get the expect behavior, proper bring down and consistent ifstate14:22
=== MacSlow|lunch is now known as MacSlow
stgraberarges: now trying with IP config directly on bond0 as you have14:22
xnoxballoons: with normal packages, i would have done direct to the archive upload weeks ago.14:22
maxiaojunhttp://udisks.freedesktop.org/docs/latest/gdbus-org.freedesktop.UDisks2.Block.html#gdbus-method-org-freedesktop-UDisks2-Block.Format14:22
maxiaojunseems like erase: '' no longer works on trusty14:23
balloonsxnox, I'm blocked on releasing anything unless tests pass14:23
balloonsand tests fail in trunk because the app has some critical bugs14:23
balloonsregardless, it sounds like we are going to be able to land something today that should solve your issue14:24
balloonsso details aren't important14:24
stgraberarges: nope, things still get brought up correctly with "ifup eth0 && ifup eth1" and brought down correctly with "ifdown -a"...14:24
stgrabernow trying with an exact copy of your config :)14:25
xnoxballoons:  "I'm blocked on releasing anything unless tests pass" that sounds really odd, can you ellaborate where this requirement is comming from? because that should be blocking merging and landing new features, this should be blocking any other no-change rebuilds. E.g. just like we did for qt5.2 landing and etc?!14:25
argeshmm14:25
xnoxballoons: the details are important, as I have been blocked for about 4 weeks due to inability to do click uploads.14:25
xnoxballoons: those that _do_ not match trunk.14:25
argesstgraber: it works if the bond is using dhcp, but not with a static ip14:26
argesstgraber: in fact it seems to affect any static interfaces... I just set eth1 to static and tried to ifdown it with no luck14:29
roadmrHey folks, I see (https://launchpad.net/builders) that ppc64el builders are available for official distro but not for PPAs. How can I provide ppc64el packages on a PPA?14:30
cjwatsonroadmr: that's only possible with devirtualised PPAs, which are only provided for special purposes14:31
stgraberarges: I don't use any dhcp here14:32
stgraberarges: here are the config I tried so far: http://paste.ubuntu.com/7125505/14:33
stgraberarges: both come up fine with either "ifup eth0 && ifup eth1" or "ifup -a" and they get brought down cleanly with "ifdown -a"14:33
argesstgraber: let me adjust my config here14:34
roadmrcjwatson: devirtualized meaning native hardware? oh those must be scarce... who can help me decide if my purpose is special enough and maybe give us access to this?14:37
mdeslaurLaney: do we have a codesearch set up on the ubuntu somewhere?14:37
maxiaojunask again, who takes care of usb-creator?14:37
argesstgraber: i also found another issue where if I have a bond my vm takes much longer to boot up because i get 'bonding: bond0: unable to update mode because interface is up'14:37
argesso i should file a separate bug for that14:37
Laneymdeslaur: ya: http://ubuntu-codesearch.surgut.co.uk/14:38
cjwatsonroadmr: #webops internal14:38
Laneybe patient with it and refresh a couple of times if you get timeouts14:38
maxiaojunxnox: ping14:38
mdeslaurLaney: sweet, thanks!14:38
roadmrcjwatson: great, thanks so much for the info. I'll ask there14:38
xnoxmaxiaojun: hey!14:38
argesstgraber: i can still easily reproduce with http://paste.ubuntu.com/7125534/14:39
stgraberarges: so even if your config (dropping the eth0 and replacing eth1 by eth0 and eth2 by eth1), I still get things to work here...14:39
argesif I change bond0 from static to dhcp it works14:39
maxiaojunxnox: usb-creator's "Erase Disk" seems totally broken in trusty14:40
xnoxmaxiaojun: yeah. I have seen patches posted to the bugs on usb-creator. But I didn't review / upload them yet. I was planning to do it today or tomorrow.14:40
argesstgraber: I'll try this on hardware. Mind reproducing this in a VM? (just to prove i'm not crazy)14:41
stgraberarges: if I can find a machine where I have some kind of VM technology installed :)14:42
stgraber(sorry, LXC maintainer here)14:42
maxiaojunxnox: i cannot fix it even if i traced to usb-creator-helper14:42
argesstgraber: ah. ok i'm testing on hw now14:42
jdstrandLaney: curious, is that just for the dev release?14:43
Laneyjdstrand: ya14:43
LaneyYokoZar: I uploaded that, thanks for pointing it out14:44
Cimimterry, how do I test on the phone? branch and compile?14:44
mterryCimi, the wizard?14:44
Cimiyup14:45
mterryCimi, yeah you could do that, and run it manually...14:45
Cimiok14:45
mterryCimi, alternatively, you could use the upstart job that we don't actually ship in the deb, but is in the source tree14:45
mterryCimi, place that in /usr/share/upstart/sessions14:45
argesstgraber: just to confirm you've been testing trusty right?14:46
maxiaojunxnox: call to UDisks2's format either error or timeout (cannot find a way to do 'quick format' and do not where the time limit is set)14:46
stgraberarges: yep, up to date trusty + ifupdown from my PPA14:46
argesstgraber: ok i'm using the same in my vm with your ppa, and still reproducing it14:47
balloonsxnox, so we are landing a fix to the store right now. As far as what happened, well when qt 5.2 landed the trunk and supposed stable version of calendar stopped working due to the bugs I linked. I understand you are upset because you made only manifest changes, however I cannot release an old revision, it has to be current trunk14:48
stgraberarges: testing in a clean vm here now14:49
argesstgraber: ok... on hardware. ifdown -a; ifconfig (bond0 is gone!), ifup -a && ifdown -a (bond0 is still there)14:49
stgrabergood, so I'm not insane :)14:49
stgraberoh no14:50
stgrabermisread14:50
xnoxballoons: that sounds very wrong that " we cannot release an old revision,"14:50
xnoxballoons: we do this all the time for all components in the archive, in main and the ubuntu-touch seeds, both click apps and .debs14:50
xnoxballoons: we should fix that.14:50
balloonsxnox, afaict releasing an old version would still have broken tests. So yes it would have fixed your issue, but not the issues with the landing team and wouldn't pass the store review14:51
xnoxballoons: ?! dude didrocks himself uploads things to revert, or do no-change uploads without regressing beyond what was on the image already.14:52
xnoxballoons: e.g. we go up and down broken versions all the time.14:52
xnoxballoons: i'd love to hear comments from the landing team about this.14:52
didrocksxnox: can you stop trolling and start helping in the bazillion of regressions we have please?14:53
balloonsxnox, I have things rejected all the time for the store..14:53
balloonsanyways, your pain is not just your own.. the calendar devs, landing team, everyone has felt the impact of calendar regressing. But it's fixed now14:54
didrocksxnox: and we do test the revert before uploading and ensuring we don't introduce regressions compared to latest promoted image FWIW14:54
argesstgraber: on hw: reboot interfaces are correct, ifdown -a - everything is down, ifup -a - I get a kernel message 'bonding: bond0: unable to update mode because interface is up' and I noticed in ifenslave it tries to set the bonding mode but that command fails...14:54
balloonswell, fixed in the sense you'll get a new build14:54
xnoxdidrocks: correct. and ballons seems to think that this is not the case here.14:55
argesand on my vm, I get that kernel message right on boot every time (in fact it delays boot up by a minute or so)14:55
xnoxdidrocks: sure, what do you want me to fix in the fork of calendar-app form revison 201?14:56
didrocksxnox: I'm not sure about anything right now, calendar-app has no issue with balloons's fixes from what I know of14:57
xnoxdidrocks: it appears to have been accepted to have a certain baseline from way before qt5.2 uploads et.al.14:57
xnoxdidrocks: gallery-app?14:58
didrocksxnox: read #ubuntu-touch for the regressions14:58
xnoxdidrocks: dude calendar_app is 100% green with no crashes http://ci.ubuntu.com/smokeng/trusty/touch/mako/248:20140320:20140304/7277/calendar_app/14:59
didrocksxnox: I know that link14:59
xnoxdidrocks: please explain how calendar-app is not in a releasible state up to revision 205.14:59
didrocksin case you wonder14:59
didrocksI don't know14:59
didrocksdid I say that?14:59
didrocksdid I write that?14:59
xnoxdidrocks: no. But balloons above claims landing team is blocking releasing calendar-app.15:00
didrocksxnox: so how do you want me to explain something I didn't write?15:00
didrocksballoons: what's about this? ^15:00
xnoxdidrocks: balloons: so what do I need to do to release 205? do you want to execute full manual testing plan?15:00
xnoxdidrocks: well, I'm trying to find out what needs doing to release calendar app to revision 205 =)))) but i don't know who would know =))))15:01
balloonsxnox, ?? What do you need. We are releasing a build from today that contains everything up through rev211, which has your changes15:01
pittibarry: big +1 on dropping 3.315:01
balloonsjust wait for it to land and you should be set?15:01
didrocksxnox: yeah but please don't assume and start telling things that aren't written15:01
didrocksxnox: there is enough pressure and confusion on the real regression15:01
didrocksno need for noise on top of that15:02
didrocksthat would be appreciated15:02
barrypitti: \o/15:02
xnoxballoons: what's special about today vs all the past 4 weeks. I mean fine, but what can I do to prevent 4week delays like this?!15:02
pittibarry: that's essentially "drop it from py3versions; rebuild $world; remove package", right?15:02
xnoxdidrocks: well i have spare cycles and time to test and release things, yet people who can release are over-worked with "the real regressions"15:03
didrocksxnox: yeah, and not sure why you are telling that I wrote we are blocking you15:03
barrypitti: yes, although $world only needs to include packages with extension modules (to be identified)15:03
xnoxdidrocks: i'm asking how to help. Can i have access to e.g. click packages releases? and be one more lander for that?15:03
didrocksxnox: I don't know, I can't release click package either15:03
=== qengho is now known as CardinalFang
=== CardinalFang is now known as qengho
xnoxdidrocks: cause i've checked all the ci-train stuff, but the click/core-apps are not listed. Who owns those?15:04
xnoxballoons: is it only you who can release core-app clicks?15:04
barrypitti: doko is going to upload 3.4 final tomorrow or this weekend, so let's wait for that15:04
pittibarry: ack15:04
didrocksxnox: and sergiusens15:04
didrocksbut you just started a mail on the ML15:04
stgraberarges: just for fun, can you comment out eth0 in both your setups and try again?15:04
didrocksif you want to discuss that on IRC, use IRC15:04
didrocksdon't double the discussion15:04
stgraberarges: if that magically makes the issue disappear for you, that'd at least explain why I can't reproduce your problem15:05
xnoxdidrocks: well there is no resolution. i basically got told off, to sit and wait and do nothing it will be done "today".15:05
argesstgraber: sure i'll try it15:05
didrocksxnox: on the ML? You wrote the email 20 minutes ago and you already get one answer15:06
argesstgraber: wierd... without the extra eth0 interface ifdown -a && ifup -a works fine15:08
argesstgraber: i looped it about 10 times with no problems15:09
argesstgraber: i'm going to have to write some additoinal tests for ifupdown : )15:09
stgraberarges: good, so that confirms what I'm seeing here.15:09
stgraberarges: now the question is wth is eth0 triggering the bond somehow...15:09
stgraberI guess diffing "ifup -a" with "ifup --exclude=eth0 -a" may point us in the righ direction15:11
stgraberwell, both of those with -v obviously15:11
argesgood idea. i'll give that a try15:12
stgraberjust ifup/ifdown of eth0 doesn't seem to do anything wrong, so it's pretty confusing15:13
=== FJKong is now known as FJKong_afk
xnoxdidrocks: no, i've emailed to the ML based on the conversations with balloons. After that, you, ballons and I continued talking about this topic.15:13
didrocksxnox: ack then, seems it's resolved now though, right? balloons is going to release it?15:14
argesstgraber: confirmed that ifdown -a && ifup -a --exclude=eth0 do the correct thing for 5 iterations15:14
stgraberarges: hmm, I'm getting an error from lockfile when doing ifup -a, I wonder if that could be the problem15:15
xnoxdidrocks: this landing is resolved, yes. the backlog in .click releases of core-apps is subpar and needs more people helping out.15:15
dholbachdarkxst, if the content is all cc-by-sa - all the files should say so15:15
xnoxsergiusens: balloons: can you train me up to prepare and help with releasing .click core-apps?15:15
sergiusensxnox, sure15:15
didrocksxnox: yeah, that was on the schedule to discuss this week to have one release process for debs and clicks, but regressions blocked to take the time for it15:16
sergiusensnot much training needed; I'll ping back later today15:16
sergiusensah, right15:16
sergiusenswe can do it then as well15:16
xnoxsergiusens: balloons: that would be appreciated and hopefully would free you up your development time on more urgent things.15:16
didrockssergiusens: no time due to regression and not around tomorrow15:17
argesstgraber: are you only seeing this with -v?15:18
sergiusensdidrocks, I'm not around tomorrow either15:18
balloonsxnox, do you have a device?15:18
stgraberarges: that can be ignored, it was just the ntpdate hook being unhappy, I moved it away for now15:19
Cimimterry, it's not optimal, takes ages15:19
xnoxballoons: i have grouper and mako.15:20
stgraberarges: "ifup eth0 eth1 eth2" => missing bond0 in ifstate, "ifup eth1 eth2 eth0" => it's in there and everything works15:21
xnoxdidrocks: "that was on the schedule to discuss this week" where would this discussion be?15:21
xnoxor did you mean last week / vUDS?15:22
didrocksxnox: this week15:23
didrocksdidn't get room last wekk15:23
didrocksweek15:23
argesstgraber: that's really wierd15:23
didrocksjust a hangout15:23
didrocksxnox: you're welcome to organize it15:23
didrocksand promote it15:23
xnoxdidrocks: i wouldn't know full current release process of either debs or clicks, and even less on how to apply common policies/current practice to both.15:24
xnoxdidrocks: so i'm not at all the right person to drive that session.15:24
didrocksok, so let's us driving it15:24
didrockswhen we'll get time for that15:25
Cimiseb128, how you test on the phone?15:25
Cimiseb128, it takes light years to compile15:25
xnoxdidrocks: i wish to be an ad-hoc lander and help releasing things, but so far that was not possible.15:25
seb128Cimi, light years is an unit of distance :p15:25
seb128Cimi, you can cross compile on your desktop15:26
Cimiseb128, I know it is, but in my mind imples very far away with current speeds :)15:26
seb128if you have a build tree it's not that slow to run make15:26
seb128also often we work on qml15:26
Cimiseb128, I ran debuild15:26
Cimion the phone15:26
Cimiafter fifteen years I run wizard/test.sh15:27
Cimiand it aborts15:27
Cimifantastic :D15:27
Cimicould not connect to display15:27
seb128yeah, you don't get to run things by hand on the phone15:29
seb128you need to --desktop_hint= if you do15:29
seb128but I never tried the wizard on the phone, maybe mterry did15:29
mterryseb128, Cimi: I  did, using the upstart job15:29
mterryseb128, Cimi: but really, integration with phone setup is a TODO (ideally, it will be launched by greeter if needed)15:30
Cimimterry, how can I test sim card?15:30
Cimimterry, I did make install15:30
mterryCimi, try using the upstart job15:30
mterryCimi, install your built deb15:31
Cimihow do I play with the upstart job?15:31
mterrythen copy the upstart job from the source to /usr/share/upstart/sessions15:31
mterrythen reboot15:31
seb128you copy it in the init dir (system or user)15:31
seb128reboot or just "start <job>"15:31
seb128well, you might need to ack it for that15:31
seb128tweak the start conditions15:32
Cimimterry, job name?15:33
mterryCimi, wizard/ubuntu-system-settings-wizard.conf15:34
xnoxmterry: ~/.config/upstart is read by session upstart as well.15:34
Cimimterry, start ubuntu-system-settings-wizard doesn't work15:34
mterryCimi, did you manually copy the job to the upstart directory?15:35
Cimimterry, /usr/share/upstart/sessions15:35
xnoxCimi: $ init-checkconf ubuntu-system-settings-wizard.conf15:36
Cimimterry, but I cannot run others in anyway15:36
xnoxCimi: and $ initctl list | grep settings-wizard15:36
mterryCimi, try rebooting?15:36
Cimixnox, maybe rebooting15:36
xnoxCimi: sudo -u phablet -i, should setup the right environment and join the existing upstart user session of the currently running phablet user.15:36
mterry(just because then the job will launch itself)15:36
dholbach@pilot out15:37
=== udevbot changed the topic of #ubuntu-devel to: Trusty Beta 1 released! | 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 -> saucy | #ubuntu-app-devel for app development on Ubuntu http://wiki.ubuntu.com/UbuntuDevelopment | See #ubuntu-bugs for http://bit.ly/lv8soi | Patch Pilots:
stgraberarges: so I have an idea for at least part of the problem, looking into ifupdown's code to confirm that one now.15:38
stgraberarges: I suspect that ifup only writes ifstate in one shot at the end of the bring up which will then race with anything happening from upstart...15:38
argesstgraber: this is update_state in main.c?15:40
stgraberarges: yeah, though reading it now, it looks safe...15:41
Cimimterry, xnox thanks works, and sim detection too15:44
Cimimterry, seb128 simple one https://code.launchpad.net/~cimi/ubuntu-system-settings/wizard.sim-detection/+merge/21197615:47
seb128Cimi, looks good, thanks15:50
stgraberarges: so the reason why bond0 isn't in ifstate is because upstart fails to bring up bond0, I'm trying to figure out exactly why that is though15:54
stgraberarges: oh, hold on, I think I know15:55
argesstgraber: network-interface.conf or networking.conf?15:55
argesstgraber: : ) ok15:55
stgraberarges: yep, figured it out, your config has been wrong all along :)15:55
stgraberarges: you can't have multiple default gateways15:56
stgraberwell, you can, but not with the same priority15:56
YokoZarLaney: Thanks!15:56
stgraberdhcp gives you a default gateway, you also have one in bond0, so bond0 fails to come up because ifupdown can't set the gateway15:56
stgraberhowever bond0 is also created at that point so the rest continues and blows up because bond0 was never initialized due to the failure15:57
argesstgraber: so two things : 1) how come this works on reboot? 2) do you think ifupdown should show a better error message?15:57
stgraberarges: it's a race. If for some reason bond0 comes up before eth0, then things appear to work because dhclient will just silently fail to set its gateway15:58
stgraberarges: ifupdown actually did log an error, just not anywhere where we were looking (it was in /var/log/upstart/network-interface-bond0.log)15:59
argesstgraber: well I just get things like 'Failed to bring up bond'15:59
stgraberarges: to confirm, can you try adding "metric 10" to your bond0 on both test machines and then confirm everything works as expected?15:59
stgraberarges: aren't you getting the netlink error for IP collisions?16:00
stgraberRTNETLINK answers: File exists16:00
argesstgraber: yes I get that error as well16:00
argesah16:00
argesthat's where ip addr sets that gateway and it fails16:00
argesok16:00
stgraberyep, it fails saying there's already an entry for the default gateway16:01
=== TheMaster is now known as Guest62851
argesstgraber: ok metric 10 added to the bond0 entry works on the VM16:01
stgraberok, good, so ifupdown and ifenslave work great, it's just my brain that doesn't, I should have noticed the conflict immediately...16:02
argesstgraber: so why can't we have different default gateways for two different interfaces?16:02
argesstgraber: and if we errored, why does bond0 still come up? and then we can't ifdown it again?16:04
argesso this clearly seems like a configuration error, but i'm wondering if there is something that can be done to make ifupdown more robust with this type of failure16:04
stgraberarges: the actual bond bringup is a bit special. Basically the first interface that's related to a bond (in our case, eth1) will do "echo bond0 > /sys/class/net/bonding_masters" which will create bond0. It'll then enter a loop waiting for ifenslave.bond016:06
stgraberthe device creation emits a udev event which is picked up by upstart which then starts network-interface with INTERFACE=bond016:06
stgraberthat in turns calls "ifup --allow=auto bond0" which is responsible for actually setting up all the parameters16:06
stgrabernow the problem we had here is that this ifup failed right at the end in the ifupdown code, so not something we can do anything about from the ifenslave code16:07
stgraberand since the device isn't created by ifupdown, it doesn't know how to delete it16:07
stgraberand as everything is done in parallel, the initial ifenslave hook can't get a return code from upstart so can't know that bond0 actually failed to be brought up16:08
stgraberhmm, actually I suppose we could grep for bond0=bond0 in ifstate and try to abort if it's not in there16:09
argesstgraber: ok i think that woudl help rather than getting into this error state16:10
argesstgraber: the other question is why can I do something like "route add default gw 192.168.122.1 netmask 255.255.255.0 dev bond0" even if I have a default gw for eth0? But for some reason I need to change the metric to 10 to get it working with ifupdown/ifenslave/upstart16:11
stgraberarges: http://paste.ubuntu.com/7125986/16:14
stgraberarges: you only get file exists if the entries are identical, any difference and it'll let you add it16:15
argesstgraber: so 'ip route add ' behaves differently than 'route add'16:16
stgraberarges: route add doesn't appear to set "proto static"16:17
argesstgraber: but isn't there a different in the dev used?16:22
argeswhat i'm saying is this feels like a workaround and not a fix. because if we're defining a gateway for bond0, it should be for that device and shouldn't conflict with eth0's default gw right?16:25
stgraberarges: there's no such thing as per-device gateway, the default gateway applies to all the trafic16:26
argesstgraber: ah. ok16:27
stgraberarges: so I guess you are only allowed one default gateway per metric and per scope/type16:28
stgraberthe device isn't considered at all, so if you try to add the same route for two difference devices, you'll still get File exists (as was the case with your config)16:28
argesstgraber: so the 'real fix would be don't have multiple gateways defined for your network configuration16:29
argesthis may explain another issue i'm looking at...16:30
stgraberarges: correct16:30
argesstgraber++ cool thank you so much for your help with this.16:30
stgrabermultiple gateways is a nightmare to get right. You can do it with different metric and source based routing but that's a major pain16:30
=== NishanthMenon__ is now known as NishanthMenon
argesstgraber: so looking at the configuration if I wanted eth0 to get an IP from dhcp but my routing table not create a default gw for eth0 how would I specify that?16:35
stgraberarges: I suspect you'd have to change your dhclient.conf16:37
argesstgraber: ok. so i have a couple of suggestions then. Thanks16:37
stgraberarges: hmm, so bad news, there isn't a good way to have bond0 commit suicide if ifupdown fails to set the gateway, that's because there isn't a good way to have ifup call a script on failure for something it does internally16:38
stgraberso I think the best we can do for 14.04 is make sure that we have a paragraph reminding people that this specific config isn't possible16:39
argesstgraber: that sounds good. Where would be the best place for this? ubuntu wiki?16:39
argesman pages?16:39
stgraberarges: probably in the serverguide or wherever caribou was planning on documenting that kind of stuff16:41
argesstgraber: ok i'll work with him on this16:41
juliankbdmurray: WRT the encoding stuff. Instead of hardcoding the encoding in python-apt, how about putting something like http://paste.debian.net/88736/ in front of all dbus daemons using it?16:48
juliankThis tells daemons like aptdaemon or software-properties-dbus to use the system's default locale, rather than the C locale, and thus ensures correct encoding, even in non-UTF-8 environments.16:49
juliankApparently Cyrillic and CJK use non-UTF-8  environments16:50
bdmurrayjuliank: I don't think I know enough to say one way or the other.16:54
bdmurraybarry: are you about?16:55
barrybdmurray: yes17:17
bdmurrayjuliank: wanted a second opinion about some encoding issues...17:18
* barry backscrolls17:19
barrygosh, i don't know about that pastebin17:23
mitya57ScottK: can I go ahead with sip/pyqt* update in Trusty? It doesn't add new features (except some new methods in sip) (so probably no FFe is needed), but it will require rebuilding some stuff.17:27
tvossrobru, ping17:46
=== bfiller is now known as bfiller_afk
juliankbarry: One of the problems is that dbus starts daemons without a locale, so Python chooses the ANSI_X3.4-1968 encoding. Which fails if the files we open contain non-ANSI stuff like UTF-8 or some other encodings (CJK, etc)18:12
juliankSo we need some way to override the preferred encoding of Python.18:12
juliankBut that does not seem possible without re-exec()ing18:13
juliankI do not want to force everyone to have their /etc/apt/sources.list in UTF-8.18:13
sladenjuliank: python -S   then  sys.setdefaultencoding("utf-8")  ?18:16
barryjuliank: i generally prefer to open text files with an explicit encoding, which of course you have to know ahead of time.  in py3, built-in open() has an encoding argument, which i almost always set to utf-8.  but if the file contains non-utf8, you still have to be explicit (so how would you know what it contains if you don't already know, if you know what i mean ;)18:17
julianksladen: setdefaultencoding() does not change the preferred encoding.18:17
rsalvetixnox: are you uploading a new android today still?18:18
juliankbarry: The user edits the files with a text editor, which chooses whatever the locale tells him. So clients using python-apt should just use the same encoding, as the normal admin does. Which is probably what /etc/defaults/locale18:18
julianksays.18:18
juliankPython 3 automatically chooses the default file encoding from LANG, and I think that's a good idea, because things work in almost all cases. It only breaks for dbus services, because dbus starts services in a hyper-clean environment18:20
sladenis this not what is loaded via the 'site' module;  which is disabled by python -S ?18:20
dokoLaney, could you have a look at https://launchpad.net/ubuntu/+archive/test-rebuild-20140307/+build/568043818:20
julianksladen: The defaultencoding in sys is different from the preferred encoding according to my tests.18:20
dokocjwatson, could you have a look at https://launchpad.net/ubuntu/+archive/test-rebuild-20140307/+build/567587718:21
dokozyga, could you have a look at https://launchpad.net/ubuntu/+archive/test-rebuild-20140307/+build/567366718:21
cjwatsondoko: tomorrow18:22
dokoinfinity, could you have a look at https://launchpad.net/ubuntu/+archive/test-rebuild-20140307/+build/567366718:22
dokoseb128, could you have a look at https://launchpad.net/ubuntu/+archive/test-rebuild-20140307/+build/5700287 ?18:24
seb128doko, sure, I meant to do that last week and got sidetracked with other issues, adding to my todo for tomorrow18:24
dokoSweetshark, could you have a look at https://launchpad.net/ubuntu/+archive/test-rebuild-20140307/+build/5703951 ?18:24
dokocjwatson, seb128: thanks!18:25
xnoxrsalveti: i'm still working on it. Let me check how far the build got to.18:25
xnoxrsalveti: there were a couple of bugs when building in the chroot, vs just on my local machine.18:25
xnoxrsalveti: ideally, yes i'd like it to be uploaded today.18:25
seb128doko, yw!18:25
Laneydoko: tomorrow too18:25
rsalvetixnox: ok, let me know if there's anything I can help you with that18:25
Laneydoko: in return I'd appreciate it if you could see if you have any bright ideas about https://buildd.debian.org/status/fetch.php?pkg=webkitgtk&arch=armhf&ver=2.3.92-1&stamp=139514639418:25
xnoxrsalveti: well, i'd want you to review the debdiff and test x86 emulator image shortly.18:26
rsalvetisure18:27
dokoLaney, do you have the preprocessed source somewhere?18:28
Laneynope18:28
dokomlankhorst, could you have a look at https://launchpad.net/ubuntu/+archive/test-rebuild-20140307/+build/5735335 ?18:29
LaneyI guess you could reproduce it locally ;-)18:29
Laneyor I can, but again tomorrow18:29
* Laney leaves for a bit, see you18:29
sladenjuliank: I'm don't dispute this:  http://paste.ubuntu.com/7126580/18:31
argesstgraber: fyi, bug 1295304 has the new ifdown -a / vlan/bonding issue we discussed earlier18:32
ubottubug 1295304 in ifupdown (Ubuntu) "ifdown -a fails with vlans on bonded interface" [High,In progress] https://launchpad.net/bugs/129530418:32
julianksladen: Right, if I run this with LC_CTYPE=C it shows that me "<module 'sys' (built-in)> None UTF-8" and then "UTF-8 ANSI_X3.4-1968"18:32
juliankAnd it does not even work in python3 which does not have reload. And we only care about Python 3, because Python 2 does not care about encoding at all18:33
barryjuliank: sorry, multiple conversations going on.  s/reload/imp.reload/ in py318:35
juliankbarry: Thanks.18:36
juliankBut this still won't work because we are missing some kind of setpreferredencoding() AFAICT18:36
juliankOK, seems I figured something out18:39
juliankbarry: So, I got rid of that nasty exec() by calling locale.setlocale(locale.LC_CTYPE, "") after setting the environment variables from /etc/default/locale. http://paste.debian.net/88751/18:42
juliankThis changes the value returned by locale.getpreferredencoding()18:42
juliankand should thus change the default encoding of open()18:43
barrygetdefaultencoding is pretty much hardcoded to utf-8 in py318:44
barrys/pretty much// even ;)18:44
juliankbarry: Yes, but not getpreferredencoding() which is what open() uses. That uses the locale.18:44
barryjuliank: yeah.  it just makes me uncomfortable not give an explicit encoding to open()18:45
barryjuliank: where do you propose that pastebin code would go?18:45
juliankbarry: In aptd and software-properties-dbus (both D-Bus daemons)18:46
juliankAre the other d-bus daemons using python-apt?18:46
barrynot sure.  check reverse-depends python3-apt?18:48
barryjuliank: so let me see if can summarize the problem (is there a bug?).  user writes some non-utf8 text to a file in their possibly non-utf8 locale.  some dbus daemon reads that file.  dbus does not set a locale so there's no way to know what encoding is used in the file.  boom.  if so, would seem to affect any such dbus daemon.  reading /etc/default/locale may get you closer, but could still not be perfect if user sets a different18:52
barrylocale.18:52
juliankbarry: Yes. But consider that the files are only writable by root and root configures the default locale. This is the closest we can get I think.18:53
juliankIt does not affect most d-bus daemons, I think; because most are written in C, and they simply read things in bytes.18:54
barryi guess that would be another option, though probably not much fun (e.g. open in binary mode, read bytes and then decode according to best guess)18:55
barryi'm rather surprised dbus doesn't specify a locale18:55
stgraberarges: found the issue with the vlans, it's actually ifupdown being a bit too clever18:56
argesstgraber: wow18:56
argesthat was fast18:56
stgraberarges: it notices that the device is a vlan and tries to destroy it, except the vlan hook did it already so you get the error18:56
barryanyway, it seems like a hack, but possibly one that will usually work, and "practicality beats purity"18:56
stgraberarges: I'm trying to disable the chunk of code in ifupdown that does that, that should fix some broken behaviours with bridges and vlans19:03
argesstgraber: cool let me know and i'll test it on this end19:04
juliankbarry: I just noticed that aptdaemon has something similar already in its script, but it simply hardcodes C.UTF-819:07
barryjuliank: i wonder if that's the cause of all the aptdaemon test failures ;)19:08
barrykidding, but still19:08
juliankbarry: It seems that my approach without the exec()ing does not change the file system encoding of Python.19:11
juliankaptd re()execs itself if sys.getfilesystemencoding() == "ascii" and not "LANG" in os.environ19:11
juliankIf we add the default locale loading in there, all things should just work in almost all cases.19:12
juliankThe combined approach would be http://paste.debian.net/88755/19:13
juliankIt first sets LANG to C.UTF-8 and then reads /etc/defaults/locale to try to find something besser.19:13
juliankIt would probably be easier to ship a wrapper script instead of re-exec()ing the script itself.19:15
juliankAlthough this should work.19:15
argesbdmurray: hi. i'm getting a few things done, then i'll have some time to help with sru queue19:16
bdmurrayarges: sounds good19:16
jtaylorinfinity: when are we getting a fixed libm?19:20
stgraberarges: fixed ifupdown uploaded to my ppa19:21
stgraberarges: this also contains xnox's change to prevent "restart networking" and "stop networking"19:21
argesstgraber: i'll give it a try. this is the experimental ppa?19:21
stgraberyep, just uploaded now so it'll be a few minutes before it's done building19:21
argesok19:22
stgraberarges: done building, feel free to just grab the binary from LP19:24
argesstgraber: ok will do19:25
zygadoko: looking19:26
zygadoko: how can I reproduce that rebuild? was there anything special about it?19:26
juliankbdmurray: I reassigned the original bug to software-properties, could you reset it from fix released to new or triaged?19:28
juliankWe should then sync python-apt 0.9.3.4 from sid tomorrow and close regression bug 129453119:29
ubottubug 1294531 in python-apt (Ubuntu) "Unicode traceback saving empty SourcesList regression" [Undecided,Confirmed] https://launchpad.net/bugs/129453119:29
argesstgraber: it works! ifdown -a only leaves lo interfaces, and ifup -a brings back everything as expected . I did 5 iterations of it19:31
bdmurrayjuliank: yes, will do19:32
juliankThanks.19:32
stgraberarges: good :)19:33
bdmurrayjuliank: Thanks for working on this.19:34
bdmurraybarry: any ideas about the update-manager tests?19:34
barrybdmurray: no, sorry, i've yet to get a block of time to dig into it19:35
juliankbdmurray: I'm happy that I do not have to think about update-manager... Enough other packages too maintain for me over in Debian.19:39
juliankWhich reminds me that I (or hopefully someone else) still need to write a PackageKit-based update-notifier replacement for Debian.19:40
=== bfiller_afk is now known as bfiller
stgraberxnox: what's the status on changing lsb so that init scripts which have a matching upstart job exit immediately when called?20:20
stgraberxnox: I believe slangasek told me in London that it was on your todo for 14.0420:21
slangaseknot sure I said 14.0420:22
slangasekI did say xnox did have a plan for fixing this20:23
stgraberok, maybe you didn't say 14.04, though I'd argue we probably ought to fix this for 14.04, it's starting to be a bit rdiculous for half our init scripts to manually check for upstart and for the other half to just confuse you or break your system/service20:24
stgraberand it's going to be quite a surprise for people upgrading from 12.04 as they'll move from the upstart-job symlink to a potentially harmful init script20:25
stgraber(sure, they shouldn't be calling those scripts directly to begin with, but well, they do...)20:25
xnoxstgraber: right, i do have an example patch, but i didn't submit it to the debian maintainer.20:26
xnoxstgraber: i'll add it to my todo for tomorrow.20:26
stgraberxnox: awesome! thanks20:26
xnoxstgraber: indeed it outght to be fixed, cause e.g. /etc/init.d/ssh start should start ssh server..... not exit zero without any output.20:27
xnoxcause that would make everyone think it did start up.20:27
xnoxstgraber: might get it done tonight after kids are asleep.20:27
xnoxrsalveti: hm, shell scripts for the emulator got dropped?! http://paste.ubuntu.com/7127213/20:45
rsalvetixnox: yes20:45
xnoxrsalveti: cool.20:46
rsalvetixnox: but they were already removed from the android package20:46
rsalvetinot sure why it tried to install them again20:46
xnoxrsalveti: hm, weird. I'll check my debdiff carefully, maybe i'm working against something old =/20:47
rsalvetithe ubuntu-emulator package is doing the logic to download and setup the image20:47
rsalvetiyeah20:47
xnoxyet with a new tarball lol.20:47
xnoxwell it does build all flavours including x86 emulator so it's okish.20:47
rsalvetixnox: maybe the pkging bzr repo is not in sync with the src package20:48
xnoxrsalveti: we have a pkging bzr repo? i'm working agains the archive .src packag.e20:48
xnoxrsalveti: it really ought to be a git repo on phablet with the rest off the code to be honest20:48
rsalvetiwe have but yeah, not in sync20:48
slangasekstgraber: which "other half" are breaking the system/service?20:48
rsalvetijust work against the src package20:48
rsalvetixnox: indeed20:49
rsalvetixnox: we might want to create another branch/repo in there for the packaging stuff20:49
slangasekstgraber: I thought the announcement I sent to u-d/u-d-a last summer was very clear about the migration requirements; have people been uploading packages with upstart jobs without properly transitioning the init scripts?20:49
xnoxrsalveti: well i kicked off another build, and let me inspect the debdiffs so far.20:49
xnoxslangasek: duh.20:50
xnoxslangasek: not me, but i deffo seen just upstart jobs added with nothing done on sysvinit side at all.20:50
slangaseksigh20:50
stgraberslangasek: the usual breakage is a package having both an upstart and sysvinit job but not having both do the same thing, database servers and anything storing some kind of data in /var/lib may fall into that case20:50
slangasekstgraber: there aren't supposed to have been any such packages in Ubuntu; whoever uploaded those packages without fixing the init scripts was negligent20:51
slangasekand there's no guarantee that fixing this centrally in the lsb hooks will fix all of those packages, either20:52
slangasekbecause use of the lsb functions interface is optional20:52
xnoxslangasek: sure but some of these came via syncs, not uploads direct into ubuntu20:52
slangasekxnox: so the Debian maintainer was the one violating Debian policy on use of upstart jobs?20:53
stgraberslangasek: sorry I don't remember what you sent last summer, but what were you advocating? for people to make sure the logic in the upstart job and the sysvinit script stays in sync so that if the service is started directly from init.d things don't blow up?20:53
slangasekstgraber: no; you should never be starting the service directly from init.d20:54
xnoxstgraber: the policy says that initscript should check if there is upstart running as pid1, and in that case do nothing.20:54
slangasekand it was the responsibility of the maintainer to make sure the init script became a no-op20:54
stgraberslangasek: sure, I agree and indeed, I don't believe we have anything in the distro doing that20:54
stgraberslangasek: my problem is our users20:54
xnoxslangasek: but that would also break, when systemd is pid1 and upstart is pid2, as the policy checks will succeed on upstart, yet it's not pid120:54
xnoxslangasek: and systemd units should be used.20:54
stgraberall the bug reports I've seen with /etc/init.d/* breaking stuff isn't because of maintainer scripts or other distro tools but because of users directly poking those20:54
stgraberbecause, well, it used to work back in 12.0420:55
slangasekxnox: that's a new problem, and not the problem in 14.0420:55
xnoxslangasek: ok.20:55
stgraberand yes, that's wrong, they shouldn't do that and our documentation tells them not to, but well, they still do and if we can catch a good chunk of those, we should20:55
xnoxstgraber: "you fight for the user!" =)))))20:55
slangasekstgraber: yes, but the only places where this has regressed are maintainers uploading broken packages20:55
stgraberxnox: no, I fight for less bug reports :)20:55
slangasekstgraber: this is NOT about what the users are doing20:55
xnoxstgraber: =)))))) _selfish_ ;-)20:56
slangasekthis is about the *uploader* having done something broken20:56
slangasek"I ran /etc/init.d/foo start and it didn't start" -- "so don't do that then"; "I ran /etc/init.d/foo start and it trashed my database" -- "the maintainer did something very wrong"20:57
stgraberslangasek: not sure I understand you. If I have a package in 12.04 that ships an upstart job, I can do "/etc/init.d/service restart" and it'll restart the service. Now rebuilding the exact same thing on trusty will call theactual init.d job, not the upstart-job symlink, so now I'm going through an entirely different code path and potentially causing data corruption20:57
slangasekstgraber: who's "rebuilding the exact same thing on trusty"?20:57
slangasekI don't like talking about this in generalities20:58
slangasekwhat packages are broken?20:58
slangasekI think it's fine that xnox will try to fix it for the lsb case by having a common hook, which can either do a pass-through or make the init script a no-op (I don't have a strong preference between the two)20:58
slangasekbut we need to sort out what packages are now shipping both initscript and upstart job without having been properly transitioned20:59
stgraberlast example I heard was from rbasak with mysql20:59
xnoxstgraber: slangasek: i have a branch / thing scanning all upstart jobs, i can add init.d scripts scanning to it and give you a quick way to check.20:59
slangasek(and make sure that we catch the non-LSB case)21:00
stgraberapparently both the upstart job and the sysvinit script in 14.04 do stuff, they just don't quite do the same thing and there is documentation out there pointing to /etc/init.d/mysql which used to work with 12.04 and now on 14.04 starts a second daemon or something21:00
slangasekstgraber: which is clearly neither a sync nor a no-change rebuild; the maintainer is at fault for uploading this package21:00
xnoxslangasek: stgraber: we had a period of time with broken dh_installinit in debhelper. Can it be at fault here?21:01
xnoxslangasek: especially if before on ubuntu it would override initd script with a symlink to upstart job, and we stopped doing that?!21:01
slangasekxnox: no, the expected new behavior of debhelper is to install both the init script and the upstart job from the package, so that the package works on either an upstart system or a sysvinit system.  The expected behavior of the *maintainer* was to fix the init script before uploading to Ubuntu21:01
stgraberslangasek: what is the maintainer supposed to fix? both scripts are fine on their own, it's just if you start mixing both that things go wrong.21:02
xnoxslangasek: how would maintainer find out about it? surely that's a debhelper compatibility level break.21:02
slangasekstgraber: it is *not* fine on its own, the init script is in violation of Debian Policy!21:03
xnoxslangasek: one simply does not start generating broken packages, based on changed debhelper version =)))))21:03
xnoxslangasek: and especially when one didn't bump the package to declare compliance with the new debian policy version number.21:03
slangasekstgraber: https://lists.ubuntu.com/archives/ubuntu-devel/2013-May/037150.html21:03
xnoxlet me check default installs here.21:03
cjwatsoneh, standards-version is purely informational, please don't start getting the idea that things should be keying off it for build behaviour21:04
slangasekxnox: all packages must comply with the current version of the policy; the Standards-Version field is advisory21:04
slangasekxnox: the way forward here is to land your change in the upstart package to add the lsb hook; but I don't think there's any excuse for uploaders not following ubuntu-devel.21:06
slangasekthis problem only happened as a result of maintainers uploading new versions of their packages without making the necessary changes to the init scripts21:06
Noskcajmdeslaur, Are you going to update libyaml to the new bugfix release soon? If not, i can. Debian has an NMU fixing a regression too21:07
stgraberslangasek: I'd be interested to see an archive wide search for packages shipping both an /etc/init job and an /etc/init.d job of the same name looking for init_is_upstart or "initctl version | grep -q upstart". It's also unfortunate that the policy just says to exit 1 as that tends to confuse the users...21:07
stgraberslangasek: the example in the policy also only covers the "start" argument, so if people just started copy/pasting, "/etc/init.d/service restart" has a good chance to cause chaos...21:11
stgraber(your e-mail on the other hand properly covers all those cases, it's just unfortunate that this didn't make it to the policy)21:11
slangasekstgraber: policy covers that 'restart' must be the equivalent of stop && start21:11
stgraberok, my bad, make that force-reload then21:13
xnoxok.21:15
=== seb128_ is now known as seb128
mdeslaurNoskcaj: which version are you referring to? AFAIK, I already have the regression fix in trusty...21:25
xnoxrsalveti: the package builds and the debdiff is reasonable, i'll throught it into virtualised ppa to build as it's too late for me to upload.21:48
rsalvetisure21:53
xnoxrsalveti: uploaded as https://launchpad.net/~ubuntu-toolchain/+archive/android/+build/583266021:57
rsalveticool, thanks21:57
xnoxrsalveti: debdiff is smallish - http://paste.ubuntu.com/7127592/22:00
rsalvetixnox: yeah, looks fine22:03
=== cmagina is now known as cmagina-away
barrybdmurray: so, the test error seems like a clear bug in the test suite.  i can fix that.  the others seem like a change in semantics somehow about package upgradability.  it's difficult for me to tell why the test assumptions there are incorrect now, so i think i will file a bug (and assign to mvo to give him fun stuff to work on :) and attach my branch that fixes the one problem so far22:04
bdmurraybarry: I've been digging at it and pkg.is_upgradable seems to be returning different results now22:05
bdmurraybarry: so then updates_list is a different size22:06
barrybdmurray: yep, that's what i've seen too22:06
bdmurraybarry: well, let me know the bug number22:07
=== kenvandine_ is now known as kenvandine
barrybdmurray: LP: #129539222:12
ubottuLaunchpad bug 1295392 in update-manager (Ubuntu) "test suite failures" [Undecided,New] https://launchpad.net/bugs/129539222:12
bdmurraybarry: thanks22:15
=== marrusl is now known as marrusl_afk
=== marrusl_afk is now known as marrusl
=== salem_ is now known as _salem
juliankpython-apt 0.9.3.4 is imported in launchpad now. It does fix three other bugs apart from the unicode-revert-stuff (most importantly: pre-build.sh now fails if the mirror lists cannot be downloaded  [e.g. if launchpad is overloaded which happens a lot] instead of simply writing empty files)22:50
juliankI'm still a bit unsure about how to request per-package upload permissions as a DD. If someone could enlighten me and tell me what the first step is, that would be helpful?22:55
juliankThe dynamic-ppu-procedure.txt in Laney's is not well-worded.The first "step" seems to be 10 - aka send an email with list of packages; but 11 says you must have attended a meeting before this, so is there a step before this?22:58
barryjuliank: probably best to start with an email to devel-permissions@lists.ubuntu.com.  explain that you're a dd maintaining the packages you want upload rights to.  it can *probably* be done over email, but then i'm no longer on the dmb :)22:58
* barry thinks https://wiki.ubuntu.com/UbuntuDevelopers#PerPackage should really be updated to include laney's text22:59
infinityhttps://wiki.ubuntu.com/DeveloperMembershipBoard/ApplicationProcess has a section for DDs.22:59
barryyep, which eventually links to laney's text23:00
Laneythat is not a guide to be followed23:00
LaneyI think I mith delete it23:00
Laneyit was a proposal to the DMB23:00
infinity"To exercise this process, the DD should first be an existing Ubuntu developer ..."23:00
infinitySo, it sounds like one needs to at least do PPU/MOTU/etc once first before getting additional PPU for freee.23:01
Laneyyes23:01
infinityjuliank: Anyhow, synced for you for now.23:02
juliankLaney: OK, thanks. The first paragraph in https://wiki.ubuntu.com/DeveloperMembershipBoard/ApplicationProcess linked to your text, and this confused me enough to not read the second one carefully23:02
LaneyLet me see what it says there23:02
infinityjuliank: Thanks for paying attention to bugs/patches in both distros.  I know it's a pain sometimes.23:03
LaneyHrm23:03
LaneyYeah, I added that, lemme clarify23:03
juliankinfinity: Thanks.23:04
LaneyI just removed the link, it works without it23:05
juliankLaney: OK, yes, much better..23:05
juliank!23:05
juliank(sorry, the "..." were wrong)23:05
juliankWhen I tried to become a MOTU in 2008 (IIRC), it was rejected, maybe a PPU now works. I did not do that much, as I do most of my stuff on Debian side and have them sync automatically, I mostly appear active on the Ubuntu side during import freeze, to request syncs.23:11
juliankI'm trying to copy DeveloperApplication to a place below my user page, but all I get is "Please use the interactive user interface to use action CopyPage!"23:20
juliankWell, now it tells me that JulianAndresKlode/DeveloperApplication-PPU already exists. Stupid wiki23:25
barrybdmurray: that is very interesting23:37
bdmurraybarry: yeah, there are lot of changes there though23:38
barrybdmurray: so apt 0.9.14.1ubuntu2 works.  does 0.9.14.2 fail?  probably can at least bisect the ubuntu uploads to see which one it starts to fail with23:39
barrybdmurray: or maybe even ping juliank :)23:40
bdmurraybarry: it jumps from 0.9.14 to 0.9.1523:40
bdmurraywell with some .1s in there23:40
bdmurrayhttps://launchpad.net/ubuntu/+source/apt/+changelog23:40
juliankThere were some changes, APT now switches candidates if the normal one is not satisfiable.23:40
juliankI don't know when that was introduced, though.23:41
juliankIn 0.9.15.123:41
barrybdmurray: i was looking at `bzr branch ubuntu:apt`23:41
juliankapt 0.9.15.1 has "discard impossible candidates in MarkInstall", was it in 0.9.15 itself or were you testing with 0.9.15.1 or newer23:42
juliankI don't know precisely which versions were synced.23:42
bdmurray0.9.15.123:42
* barry was looking at: http://bazaar.launchpad.net/~ubuntu-branches/ubuntu/trusty/apt/trusty/view/head:/debian/changelog23:43
juliankNow, if you're trying to upgrade to something impossible, it will not produce an error, but rather switch back to the installed version23:44
barryjuliank: so it would report pkg.is_upgradable == False?23:44
juliankbarry: It depends on whether someone called mark_install() or mark_upgrade() on that package in the meantime. Assume: Before, it was true. Now you try to mark an impossible candidate as an upgrade. APT will switch to the installed version. So is_upgradable would be false.23:46
barryjuliank: okay thanks.  this is all in upgrade-manager's test suite.  we narrowed the failures down to packages which were upgradable but now are not.  it's all test data, so i suspect something there didn't keep up with these changes23:47
barrybut anyway... it's dinner time for me!23:47
juliankOK, and it's time for me to sleep now. It's 50 minutes past midnight.23:48
juliankI'll be gone in about 15 minutes and back in about 11 to 12 hours.23:49
=== Mapley is now known as Mapleh
=== Mapleh is now known as Mapley

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