| === Ursinha is now known as Ursinha-afk | ||
| === Ursinha-afk is now known as Ursinha | ||
| === kgunn is now known as Guest14808 | ||
| imgbot | === trainguards: IMAGE 244 building (started: 20140917 02:05) === | 02:05 |
|---|---|---|
| === kgunn is now known as Guest35270 | ||
| imgbot | === trainguards: RTM IMAGE 45 building (started: 20140917 03:05) === | 03:05 |
| bfiller | looks like rtm silo 5 failing to build maliit because it doesn't have git: https://ci-train.ubuntu.com/job/ubuntu-rtm-landing-005-1-build/44/console | 03:08 |
| bfiller | robru: any ideas how to fix this? the ubuntu build was fine.. ^^^ | 03:09 |
| ToyKeeper | ... get git? | 03:10 |
| ToyKeeper | Sorry, I don't know the details... just had to take the opportunity to say that out loud. | 03:10 |
| imgbot | === trainguards: IMAGE 244 DONE (finished: 20140917 04:00) === | 04:00 |
| imgbot | === changelog: http://people.canonical.com/~ogra/touch-image-stats/244.changes === | 04:00 |
| Mirv | morning | 04:11 |
| Mirv | bfiller: for some reason it tries to download upstream release (which wouldn't exist even if git was there) instead of using the supplied orig tarball | 04:12 |
| Mirv | bfiller: since it's a sync, I'll just try what would it say without the ~rtm version mangling | 04:15 |
| Mirv | bfiller: it's building now, I'll run watch_only build for it in a few minutes so that the status will be correct too. | 04:16 |
| imgbot | === trainguards: RTM IMAGE 45 DONE (finished: 20140917 04:20) === | 04:20 |
| imgbot | === changelog: http://people.canonical.com/~ogra/touch-image-stats/rtm/45.changes === | 04:20 |
| sergiusens | Mirv: bfiller was an email flowing about lack of those tools in the rtm archive; I got tcpdump copied over... but that's not your error, get rid of the ~rtm | 04:31 |
| robru | Mirv: what did you do? Manual source package upload? | 05:15 |
| robru | Mirv: bfiller: doing a build with the "don't mangle version" option set should have worked, but the real solution is to make the package be split packaging, i guess that one was never onboarded for train usage | 05:17 |
| Mirv | robru: no, copy-package | 05:17 |
| Mirv | I forgot about that option | 05:17 |
| robru | Mirv: right, same thing, you put the package there without using a train job to do it | 05:18 |
| robru | Anyway, looks like you have it under control, goodnight! | 05:19 |
| Mirv | nighty night, all of you who have not yet waken up to the morning of 17th :) | 05:40 |
| ToyKeeper | tedg, charles: Not sure when you'll be around, but is there any chance you could expand the indicator-datetime's test plan with meaningful tests, or provide a link to an autopilot suite covering its functions? | 06:11 |
| popey | Mirv: please upload http://s-jenkins.ubuntu-ci:8080/job/calculator-app-click/lastSuccessfulBuild/artifact/out/com.ubuntu.calculator_1.3.329_all.click when you get a moment, thanks. | 06:49 |
| === tvoss is now known as tvoss|test | ||
| === tvoss|test is now known as tvoss | ||
| Mirv | popey: clock done | 07:28 |
| popey | Mirv: i hope you mean calculator? | 07:28 |
| popey | Mirv: checked store, you do. | 07:29 |
| popey | Thank you! | 07:29 |
| Mirv | popey: s/clock/calculator/... | 07:29 |
| Mirv | c something | 07:30 |
| popey | sil2100: are we going to get a promoted devel image at some point? | 07:37 |
| popey | (morning btw) | 07:37 |
| sil2100 | popey: morning! | 07:40 |
| sil2100 | popey: yeah, I wanted to have one today, Omer tested one of the images yesterday and said it didn't have any serious issues | 07:41 |
| sil2100 | popey: I'll just double check with one of our QA people that krillin is also fine on devel and we could promote | 07:41 |
| popey | awesome. the clock app hasn't been updated in the store for a while and has new stuff that needs to land in the image, but depends on an image being promoted | 07:42 |
| popey | then everyone gets snooze for their alarms! | 07:42 |
| psivaa_ | sil2100: need to be afk for a bit and may miss the early part of the meeting. | 08:21 |
| === psivaa_ is now known as psivaa_-afk | ||
| sil2100 | psivaa_-afk: ACK | 08:26 |
| === psivaa_-afk is now known as psivaa | ||
| Laney | is there a special procedure to get non-train packages into rtm silos or should I just copy it? | 08:52 |
| john-mcaleely | davmor2, http://people.canonical.com/~jhm/barajas/device_krillin-20140917-19d7408.tar.xz | 09:01 |
| davmor2 | john-mcaleely: thanks | 09:01 |
| === vila changed the topic of #ubuntu-ci-eng to: Train support: trainguards | Vanguard: vila | Train Dashboard: http://bit.ly/1mDv1FS | QA Signoffs: http://bit.ly/1qMAKYd | Known Issues: Latest flash update hung all makos on s-jenkins. | ||
| sil2100 | Laney: you can use CI Train, but for now you can just copy - what packages do you have in mind? Syncing from utopic? | 09:09 |
| Laney | sil2100: I thought it had to be QAed | 09:12 |
| Laney | yes, syncing | 09:12 |
| tsdgeos | vila: hi, our autopilot setup has broken again https://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/4548/console any idea? | 09:15 |
| === vrruiz_ is now known as rvr | ||
| vila | tsdgeos: apart from the all caps in: 'SKIPPING phablet-flash AS REQUESTED' nothing catch my un-educated eyes | 09:21 |
| tsdgeos | ok, so no autopilot CI then | 09:22 |
| vila | tsdgeos: i.e. not flashing and later on failing during the setup sounds... somwhow expected ? | 09:22 |
| tsdgeos | i have no clue how this is setup | 09:23 |
| tsdgeos | i know it worked | 09:23 |
| tsdgeos | and now it doesn't :D | 09:23 |
| brendand | ogra_, so robru's citrain script is currently completely unable to cope with the adb changes | 09:23 |
| vila | tsdgeos: ;-) We'll need experts then, I'll track it and let you know or get pinged back | 09:23 |
| ogra_ | brendand, as is phablet-test-run apparently ... sigh | 09:23 |
| tsdgeos | vila: appreciated :) | 09:23 |
| ogra_ | chmod: changing permissions of '/dev/uinput': Operation not permitted | 09:23 |
| ogra_ | it tries a chmod 666 /dev/uinput at the very start | 09:24 |
| ogra_ | :( | 09:24 |
| brendand | ogra_, oh yeah | 09:24 |
| ogra_ | so i will have to add a --password option to it | 09:24 |
| ogra_ | :( | 09:25 |
| ogra_ | i guess the same goes for the ci tools | 09:25 |
| * ogra_ will get this fixed today ... | 09:25 | |
| ogra_ | damn | 09:25 |
| ogra_ | this is the tenth re-implementation of "adb shell" calls in phablet-tools | 09:26 |
| ogra_ | :( | 09:26 |
| brendand | is --ppa option of phablet-config writable-image working at all?? | 09:37 |
| vila | tsdgeos: hold on, the job you mention is from 2014-09-12, almost a week ago, I've triggered a rebuild for that one | 09:43 |
| tsdgeos | errr | 09:43 |
| vila | tsdgeos: do you have more recent failures ? | 09:43 |
| tsdgeos | that's weird it's linked from a more recent thing | 09:44 |
| tsdgeos | let me see | 09:44 |
| vila | tsdgeos: the re-run seems to proceed better (at least it pased the point where it failed previously) | 09:45 |
| vila | tsdgeos: http://s-jenkins.ubuntu-ci:8080/job/generic-deb-autopilot-runner-mako/4646/console | 09:45 |
| tsdgeos | doh :/ | 09:45 |
| tsdgeos | must have been tricked by the so many confusing links | 09:46 |
| tsdgeos | sorry | 09:46 |
| vila | tsdgeos: no worries | 09:46 |
| brendand | pete-woods, do you know how testing is going for silo 16 in RTM? | 09:46 |
| brendand | pete-woods, and 9 and 3, although those have only been around since yesterday so probably haven't got an opportunity to test those yet | 09:47 |
| brendand | ogra_, so "adb shell 'SUDO_ASKPASS=/tmp/sudo_askpass.VA3s sudo -A add-apt-repository -y ppa:ci-train-ppa-service/landing-009'" seems to think /tmp/ is on the host | 09:50 |
| brendand | ogra_, that's broken... | 09:50 |
| brendand | does anyone test this stuff... | 09:51 |
| ogra_ | where is that from ? | 09:51 |
| brendand | ogra_, from phablet-config | 09:51 |
| Laney | hrm | 09:51 |
| ogra_ | indeed that has been tested | 09:51 |
| Laney | I got the rtm chroot from LP but it had an archive.u.c sources.list | 09:51 |
| Laney | is that usual? | 09:51 |
| Laney | I was expecting something different | 09:52 |
| Laney | I assume launchpad-buildd overrides it | 09:53 |
| * Laney does that manually | 09:53 | |
| ogra_ | brendand, i dont get what you mean, it cant thin it is on the host, it is executed inside an adb shell | 09:54 |
| brendand | ogra_, http://paste.ubuntu.com/8364004/ | 09:54 |
| ogra_ | brendand, can you give me the exact error ? | 09:54 |
| ogra_ | thanks | 09:54 |
| ogra_ | brendand, no, from phablet-config | 09:54 |
| ogra_ | (which creates this file before trying to use it) | 09:55 |
| brendand | ogra_, it doesn't give an error, it just silently does nothing | 09:55 |
| ogra_ | it works fine here | 09:55 |
| ogra_ | just tried | 09:55 |
| brendand | ogra_, but that's the command it runs | 09:55 |
| cjwatson | Laney: yes, that's usual, Launchpad always forces the appropriate-context sources.list into place at the start of a build | 09:56 |
| Laney | cjwatson: Kay, I figured that would be it | 09:57 |
| brendand | ogra_, what's the outcome, is there a new file in /etc/apt/sources.list.d/? | 09:57 |
| ogra_ | http://paste.ubuntu.com/8364019/ | 09:57 |
| ogra_ | ogra@anubis:~/image-stuff$ adb -s 0079a79f1857654a shell | 09:58 |
| ogra_ | phablet@ubuntu-phablet:~$ ls /etc/apt/sources.list.d/ | 09:58 |
| ogra_ | ci-train-ppa-service-ubuntu-landing-009-utopic.list | 09:58 |
| ogra_ | yes | 09:58 |
| brendand | ogra_, ah but that's not what phablet-config does | 09:58 |
| ogra_ | that is exactly what phablet-config does | 09:58 |
| brendand | ogra_, well i get '#!/bin/shecho <combination on my luggage>' in the /tmp file | 09:59 |
| mzanetti | sil2100: hey, I'd need your help once more: I removed a branch from the spreadsheet, then did a reconfigure and rebuild on the silo. but that branch still shows up in the dashboard | 10:00 |
| mzanetti | sil2100: what did I miss? | 10:00 |
| cjwatson | brendand: /me takes seconds to work out what the "shecho" program might be | 10:00 |
| mzanetti | sil2100: silo 11, branch https://code.launchpad.net/~nick-dedekind/ubuntu-settings-components/indicator-polishing/+merge/229083 | 10:01 |
| sil2100 | mzanetti: removal of branches I think requires a landing team reconfigure :) | 10:02 |
| sil2100 | mzanetti: let me reconfigure for you | 10:02 |
| brendand | cjwatson, yes - dodgy newlines | 10:02 |
| mzanetti | thanks | 10:02 |
| ogra_ | brendand, you did supply -r, right ? | 10:02 |
| brendand | ogra_, yes - otherwise it would never run that code | 10:03 |
| brendand | cjwatson, adb.shell('/bin/echo -e \\"#!/bin/sh\necho %s \\" > %s ' % (args.remotepassword[0], tempfile), ignore_errors=False) | 10:03 |
| ogra_ | but right, the newlines are garbled, sigh | 10:03 |
| brendand | ogra_, what does yours look like, since i see you supplied two newlines | 10:04 |
| brendand | ogra_, do you have a space between the two lines? | 10:05 |
| ogra_ | brendand, i guess the same as yours ... | 10:05 |
| ogra_ | let me try a fix ... | 10:05 |
| ogra_ | why did that work during testing (not only for me) | 10:05 |
| brendand | ogra_, could echo have broken? seems impossible, but why doesn't it insert the newline? | 10:06 |
| ogra_ | brendand, thats a quoting issue | 10:06 |
| brendand | ogra_, when did it last work for you? | 10:07 |
| ogra_ | brendand, when testing before uploading/merging | 10:07 |
| cjwatson | ogra_: ugh, seriously, use printf | 10:08 |
| cjwatson | consistent shell interpretation FTW | 10:08 |
| cjwatson | (yeah, I know /bin/echo is probably safe, but ...) | 10:08 |
| cjwatson | adb.shell is a pretty awful interface by the looks of it :) | 10:09 |
| ogra_ | it is | 10:09 |
| brendand | ogra_, i'm just puzzled how this ever worked | 10:10 |
| ogra_ | and that it has to wrap two bash calls inside a sudo call to get the right env doesnt help much either | 10:10 |
| brendand | ogra_, can you paste me the fix? | 10:11 |
| ogra_ | brendand, well, if i dont get pinged every 30sec i could actually work on it :P | 10:11 |
| cjwatson | mine more backslashes | 10:11 |
| ogra_ | yeah | 10:14 |
| davmor2 | john-mcaleely, sil2100: tarball testing done, I get at least 4 brightnesses to reproduce, A bright light (spot light will do, normal light, cover just the sensor but leave the camera uncovered, cover the camera and light sensor) also the brightening of the screen on shutdown is a lot dimmer :) I'm happier with that :) | 10:14 |
| ogra_ | well, convincing printf to accep ! in #!/bin/sh seems not so easy | 10:14 |
| sil2100 | SHIP IT | 10:14 |
| john-mcaleely | sil2100, ok, I'll push it now. thanks davmor2 | 10:14 |
| cjwatson | ogra_: printf doesn't interpret ! in any way | 10:15 |
| cjwatson | ogra_: you may of course have to quote it to protect it from the *shell* trying to do history expansion on it | 10:15 |
| ogra_ | right | 10:15 |
| ogra_ | single quotes help | 10:15 |
| cjwatson | (though that shouldn't happen in noninteractive shells) | 10:15 |
| brendand | ogra_, escaping it worked for me | 10:15 |
| brendand | or not... | 10:16 |
| john-mcaleely | sil2100, shipped | 10:16 |
| sil2100 | john-mcaleely, davmor2: thanks! | 10:16 |
| ogra_ | brendand, let me fix this properly using printf as cjwatson suggests | 10:17 |
| Laney | is there a manifest for rtm? | 10:18 |
| ogra_ | Laney, on cdimage | 10:18 |
| Laney | found it | 10:18 |
| ogra_ | Laney, and changelogs in a subdir at the usual place too | 10:18 |
| ogra_ | under rtm/ | 10:18 |
| Laney | grr! | 10:19 |
| Laney | ogra_: rtm doesn't even have those packages you were complaining about | 10:19 |
| Laney | because it has gst-bad 1.2.4 | 10:20 |
| ogra_ | Laney, well, the gst merge landed one day before rtm opened | 10:20 |
| Laney | https://launchpad.net/ubuntu-rtm/+source/gst-plugins-bad1.0 | 10:20 |
| ogra_ | cjwatson, hmm, now how do i combine printf's %s with pythons %s ? | 10:21 |
| cjwatson | %% | 10:21 |
| ogra_ | thanks ! | 10:22 |
| brendand | ogra_, might it be less confusing to not use it at all? | 10:26 |
| brendand | ogra_, and just use pythons? | 10:27 |
| ogra_ | - adb.shell('/bin/echo -e \\"#!/bin/sh\necho %s \\" > %s ' % | 10:28 |
| ogra_ | + adb.shell("printf '%%s\\n' '#!/bin/sh' 'echo %s' >> %s " % | 10:28 |
| ogra_ | there is your fix | 10:28 |
| cjwatson | err that looks undefined | 10:28 |
| cjwatson | %%s%%s surely? | 10:28 |
| cjwatson | I guess it works but seems like by luck | 10:29 |
| ogra_ | cjwatson, first one is fromat and then two times content | 10:30 |
| cjwatson | oh format is reused | 10:30 |
| ogra_ | right :) | 10:30 |
| cjwatson | hm, ok, news to me :) | 10:30 |
| cjwatson | but it's documented, so ... | 10:30 |
| ogra_ | yeah | 10:30 |
| ogra_ | and works nicely | 10:30 |
| ogra_ | i'm a bit uneasy about the >> | 10:31 |
| ogra_ | but then, the mktemp is in the line above | 10:31 |
| Laney | sil2100: would you drop line 56 please | 10:40 |
| === tvoss is now known as tvoss|lunch | ||
| brendand | ogra_, probably the reason people didn't notice is because phablet-config swallows all the output | 10:46 |
| ogra_ | brendand, yeah, currently working on that ;) | 10:46 |
| ogra_ | and i also dropped the need for rebooting if the image is already writable | 10:46 |
| brendand | ogra_, nice | 10:47 |
| ogra_ | i want to drop 80% of phablet-test run for it | 10:47 |
| ogra_ | the ppa setup, adb sudo setup etc should only live in one script | 10:47 |
| sil2100 | Laney: sure | 10:51 |
| Mirv | bah, request ID:s disappearing again | 11:01 |
| ogra_ | brendand, http://paste.ubuntu.com/8364323/ | 11:02 |
| === MacSlow is now known as MacSlow|lunch | ||
| brendand | ogra_, much better | 11:04 |
| ogra_ | ok let me prepare a landing for this | 11:06 |
| ogra_ | brendand, https://code.launchpad.net/~ogra/phablet-tools/phablet-tools-fix-writable-image/+merge/234946 if you like to test and approve | 11:10 |
| mzanetti | sil2100: hmm... that branch still shows up in the dashboard | 11:13 |
| fginther | balloons, when you get a moment, can you review https://code.launchpad.net/~fginther/cupstream2distro-config/core-apps-python3/+merge/234902 ? | 11:13 |
| sil2100 | mzanetti: hmmm, let me check then | 11:14 |
| brendand | ogra_, would dpkg-query also be broken on rtm? | 11:14 |
| sil2100 | mzanetti: you wanted to get rid of the indicator-polishing branch, right? | 11:16 |
| mzanetti | sil2100: yes | 11:16 |
| sil2100 | mzanetti: I still see it in the MR list in the spreadsheet, so it was simply not removed from there - even though I remember you said you did? | 11:16 |
| mzanetti | err, what? | 11:16 |
| * mzanetti checks again | 11:16 | |
| sil2100 | mzanetti: if you check line 25 of the google spreadsheet, the branch is still there :o | 11:16 |
| mzanetti | ffs... | 11:17 |
| mzanetti | sil2100: I removed it from line 35 | 11:17 |
| mzanetti | sil2100: ok... kgunn and me really need to sync better | 11:17 |
| mzanetti | sorry | 11:17 |
| sil2100 | Ok, let me reconfigure then - but maybe you can reconfigure that as well now :) | 11:20 |
| sil2100 | Ok, I see you reconfigure now | 11:21 |
| * sil2100 goes to prepare lunch | 11:21 | |
| mzanetti | sil2100: ok. seems to work | 11:22 |
| === tvoss|lunch is now known as tvoss | ||
| ogra_ | brendand, dpkg-query operates on local Pakcage files iirc | 11:23 |
| tvoss | sil2100, can I get a silo for line41? | 11:23 |
| brendand | ogra_, ok. i need to figure out why it's giving me the wrong result then | 11:24 |
| Mirv | tvoss: allocating | 11:26 |
| tvoss | Mirv, thank you | 11:26 |
| Mirv | tvoss: fixing missing space in your MR:s | 11:27 |
| tvoss | Mirv, oh sorry | 11:28 |
| ogra_ | cjwatson, hmm ... | 11:37 |
| ogra_ | doing something like: phablet-config -r 0000 --ppa ppa:ci-train-ppa-service/ubuntu-rtm/landing-004 | 11:37 |
| ogra_ | which simply calls add-apt-repository in the backend results in: | 11:37 |
| ogra_ | Command '"SUDO_ASKPASS=/tmp/sudo_askpass.B4Y7 sudo -A apt-get update -qq ; echo ADB_RC=\$?"' returned non-zero exit status 100 | 11:37 |
| ogra_ | W: Failed to fetch http://ppa.launchpad.net/ci-train-ppa-service/landing-004/ubuntu-rtm/dists/utopic/main/binary-armhf/Packages 404 Not Found | 11:37 |
| ogra_ | cjwatson, does add-apt-repository not automatically replace "utopic" with "14.09" here ? | 11:38 |
| ogra_ | (note, i'm not running rtm on the device) | 11:38 |
| cjwatson | ogra_: dunno, cyphermox did that work | 11:38 |
| ogra_ | ah, k, i'll wait for him then | 11:39 |
| ogra_ | sergiusens, i could need top approval of https://code.launchpad.net/~ogra/phablet-tools/phablet-tools-fix-writable-image/+merge/234946 | 11:40 |
| ogra_ | (fixes a lot of wrongs in the code :) ) | 11:41 |
| === MacSlow|lunch is now known as MacSlow | ||
| === vila changed the topic of #ubuntu-ci-eng to: Train support: trainguards | Vanguard: cihelp | Train Dashboard: http://bit.ly/1mDv1FS | QA Signoffs: http://bit.ly/1qMAKYd | Known Issues: Latest flash update hung all makos on s-jenkins. | ||
| === Ursinha is now known as Ursinha-afk | ||
| === cprov changed the topic of #ubuntu-ci-eng to: Train support: trainguards | Vanguard: cprov | Train Dashboard: http://bit.ly/1mDv1FS | QA Signoffs: http://bit.ly/1qMAKYd | Known Issues: Latest flash update hung all makos on s-jenkins. | ||
| === Ursinha-afk is now known as Ursinha | ||
| === kgunn is now known as Guest35530 | ||
| sil2100 | brendand: hey! | 12:55 |
| brendand | sil2100, hello | 12:55 |
| sil2100 | brendand: were you able to check rtm #37 for krillin? | 12:55 |
| sil2100 | brendand: wait | 13:02 |
| sil2100 | brendand: I mean utopic #37 | 13:02 |
| sil2100 | My brain is broken today | 13:02 |
| ogra_ | sergiusens, i cant use single quotes in the printf ... that breaks the format | 13:03 |
| ogra_ | sergiusens, the right combo of single and double quotes is essential in that line | 13:04 |
| * ogra_ fixes the other line | 13:04 | |
| === Ursinha is now known as Ursinha-afk | ||
| === Ursinha-afk is now known as Ursinha | ||
| brendand | sil2100, yes it seems fine. no major issues | 13:07 |
| sil2100 | ogra_: hey, two questions! First, how's the mako dbus smoketesting issues? | 13:11 |
| ogra_ | sil2100, well, see above, other stuff kept me away from it | 13:11 |
| sil2100 | ogra_: ACK, second thing! What do you think about promotion for the utopic channel? ;) | 13:12 |
| ogra_ | sil2100, sure | 13:12 |
| ogra_ | sergiusens, i fixed what i could ... and commented | 13:13 |
| ogra_ | please approve :) | 13:13 |
| Mirv | sil2100: if you're interested, I'm still getting error on merge&cleaning rtm silos (requiring ignore_missing_projects) even when landing has gone completely ok: https://ci-train.ubuntu.com/job/ubuntu-rtm-landing-013-3-merge-clean/8/console | 13:19 |
| pete-woods | brendand: I'm coming round to testing all my silos now | 13:21 |
| sil2100 | ogra_: so, we would need the mako's #243 promoted and krillin's #37 | 13:24 |
| sil2100 | Mirv: oh | 13:24 |
| satoris | brendand: hi, could we get qa approval for line 40? Thanks. | 13:29 |
| brendand | satoris, yes - it's just in our queue now | 13:30 |
| satoris | Great. | 13:30 |
| ogra_ | sil2100, done | 13:31 |
| sil2100 | ogra_: \o/ | 13:31 |
| sil2100 | popey: ^ | 13:31 |
| popey | yay | 13:32 |
| * ogra_ wonders how to bribe sergiusens to top approve that super urgent phablet-config fix | 13:32 | |
| popey | mhall119: ^^ | 13:32 |
| popey | (update yr phone) | 13:32 |
| mhall119 | popey: yay! | 13:33 |
| * mhall119 hugs sil2100 | 13:33 | |
| brendand | tedg, hey i'm just looking at rtm silo 002. it was noticed that indicator-display is not seeded, is that known? | 13:34 |
| tedg | brendand, Yeah, it's on ogra_'s todo to review | 13:34 |
| tedg | brendand, But, the system settings function to turn it on doesn't work yet either. So it's not high priority :-) | 13:35 |
| ogra_ | brendand, tedg, did it land in rtm already ? i cant seed non-existing packages | 13:35 |
| tedg | ogra_, Yes, it has. | 13:35 |
| ogra_ | k | 13:35 |
| brendand | tedg, so indicator-display is just landing some translations fixes? | 13:37 |
| tedg | brendand, No, it's just enabling translation in the packaging. | 13:38 |
| tedg | No binary changes | 13:38 |
| brendand | tedg, so what change should we see after installing it? | 13:38 |
| tedg | brendand, You shouldn't. It should be changing bits in launchpad as they're introspecting those changes when building language packs. | 13:39 |
| tedg | brendand, So you won't see changes on the device for a while. | 13:39 |
| brendand | tedg, ok it would be helpful to us if that was pointed out in the description | 13:40 |
| brendand | tedg, 'indicator-datetime has no binary changes'. blah blah blah | 13:40 |
| tedg | Oh, you can't get to the descriptions in the MRs for RTM builds. | 13:40 |
| tedg | That's annoying. | 13:40 |
| tedg | Hmm, okay. | 13:41 |
| brendand | tedg, well we can go digging and figure it out, but it would be helpful if that were stated directly | 13:41 |
| brendand | tedg, just sayin | 13:41 |
| tedg | brendand, Just to be clear, indicator-display has no binary changes, indicator-datetime was changed. | 13:41 |
| brendand | tedg, yes the fix for datetime is clearly mentioned | 13:41 |
| tedg | brendand, K, just making sure :-) | 13:42 |
| tedg | Make sure we test the right thing. | 13:42 |
| brendand | tedg, anyway i'm on it now - shouldn't take long | 13:43 |
| ogra_ | sergiusens, so no top approval until i rewrite the printf line ? | 13:43 |
| tedg | brendand, Great, thanks! Any questions bug me or charles | 13:43 |
| sergiusens | ogra_: no, I'm just in a meeting | 13:43 |
| ogra_ | oh, sorry | 13:44 |
| * ogra_ hugs sergiusens | 13:44 | |
| mzanetti | sil2100: can you please do me another reconfigure on silo 11? | 13:46 |
| sil2100 | mzanetti: sure | 13:47 |
| mzanetti | thanks | 13:47 |
| ogra_ | robru, can you please approve https://code.launchpad.net/~ogra/phablet-tools/phablet-tools-fix-phablet-shell/+merge/234976 so we unscrew the SDK guys | 13:48 |
| brendand | tedg, would you accept the statement that overall the datetime testplan could be a little more thorough? also do you have a plan for writing autopilot tests for it? | 13:53 |
| jhodapp | sil2100, can I get a silo for line 75? | 13:54 |
| tedg | brendand, Hmm, I guess. Probably need to update the wiki as well. | 13:54 |
| sil2100 | jhodapp: sure | 13:55 |
| tedg | brendand, There, updated from the branch. | 13:56 |
| bfiller | sil2100: where are the krillin smoketest logs? | 13:57 |
| === Guest35530 is now known as kgunn | ||
| ogra_ | sil2100, ^^^please :) | 14:16 |
| sil2100 | ogra_: sure :) | 14:17 |
| === ogra_ changed the topic of #ubuntu-ci-eng to: Train support: trainguards | Vanguard: cprov | Train Dashboard: oh, i see, only one free silo | QA Signoffs: http://bit.ly/1qMAKYd | Known Issues: Latest flash update hung all makos on s-jenkins. | ||
| ogra_ | argh | 14:17 |
| davmor2 | ogra_: you really need to stop doing that :D | 14:17 |
| sergiusens | plars: can you test run this https://code.launchpad.net/~nskaggs/phablet-tools/remove-python2-support/+merge/233754 on the infra? | 14:17 |
| === ogra_ changed the topic of #ubuntu-ci-eng to: Train support: trainguards | Vanguard: cprov | Train Dashboard: http://bit.ly/1mDv1FS | QA Signoffs: http://bit.ly/1qMAKYd | Known Issues: Latest flash update hung all makos on s-jenkins. | ||
| ogra_ | davmor2, tell that to our focus handling :P | 14:18 |
| davmor2 | ogra_: well if you insist on using it wrong that's your fault not ubuntu's you are meant to focus on one thing not 10 :P | 14:19 |
| ogra_ | :P | 14:20 |
| ogra_ | sergiusens, that sudo crap in the last line needs to go (in fact i'm just completely re-writing phablet-test-run since it is unusable today) | 14:20 |
| sergiusens | ogra_: well I thought the plan was to move to adt runners | 14:21 |
| sergiusens | fginther: plars thoughts on that? | 14:21 |
| ogra_ | sergiusens, smoketesting isnt using them today | 14:21 |
| sergiusens | thoughts lead to questions about roadmaps :) | 14:21 |
| ogra_ | and we have dbus issues since yesterday with phablet-test-run | 14:21 |
| ogra_ | which makes that re-write kind f urgent | 14:22 |
| cyphermox | ogra_: I could have sworn it did pick 14.09 rather than utopic when I wrote the patch, but it just gets that from aptsources, so it's in a way another indication that it would work better if base-files were updated to say ubuntu-rtm and such | 14:22 |
| ogra_ | if we dont need it i'd be super happy though :) | 14:22 |
| ogra_ | cyphermox, well, i expllicitly tried to add a rtm ppa on an utopic image | 14:23 |
| ogra_ | not sure thats even remotely supposed to work :) | 14:23 |
| cyphermox | no, on a utopic image it would definitely pick utopic | 14:23 |
| ogra_ | good | 14:23 |
| ogra_ | then it works as excpected | 14:23 |
| ogra_ | i was just wondering if it shouldnt :) | 14:24 |
| cyphermox | I'll spend a bit more time thinking about it, later today | 14:24 |
| cyphermox | it really should be picking the series for the distro you're trying to add ;) | 14:24 |
| ogra_ | not urgent for sure | 14:24 |
| cyphermox | no, that's why I said later | 14:24 |
| ogra_ | though definitely interesting the other way round | 14:24 |
| ogra_ | i can imagine people wanting to add utopic ppas to rtm | 14:24 |
| sergiusens | trainguards line 73 please | 14:28 |
| ogra_ | you smarted out the bot :) | 14:28 |
| sergiusens | ogra_: the spreadsheets says we should ping manually | 14:29 |
| ogra_ | yeah, the bot only sendds a notice | 14:29 |
| ogra_ | which most IRC clisents dont highlight | 14:29 |
| ogra_ | you were still faster than the bot :) | 14:29 |
| * barry is slower than queuebot | 14:36 | |
| Laney | sil2100: where's the bug for that? | 14:36 |
| Laney | I don't think you do have the FFe | 14:36 |
| Laney | or at least I'm not aware of it | 14:36 |
| ogra_ | barry, yeah, sergiusens could teach you though :) | 14:37 |
| barry | :) | 14:37 |
| barry | i'm trying to decide whether i need an ffe for this or not though | 14:37 |
| jhodapp | sil2100, what is wrong here, not sure what this is telling me: https://ci-train.ubuntu.com/job/ubuntu-rtm-landing-014-1-build/11/console | 14:38 |
| barry | i think not since there are no user-visible new features | 14:38 |
| plars | sergiusens: last time we tried using adt for autopilot tests, it didn't work at all | 14:38 |
| plars | sergiusens: we can revisit it though | 14:38 |
| ogra_ | plars, wihin this week? | 14:39 |
| plars | ogra_: doubtful | 14:39 |
| ogra_ | else i need to fix phablet-test-run to run everything as root i guess | 14:39 |
| ogra_ | right, thats what i thought | 14:39 |
| plars | ogra_: how is phablet-test-run broken? | 14:39 |
| ogra_ | plars, see all the dbus AP errors in the failed app tests | 14:39 |
| barry | ladeedah, no silos available | 14:39 |
| plars | ogra_: why do we want to run as root? Don't we want to run the apps as the actual user? | 14:39 |
| balloons | the mp for adt-run should be working now.. | 14:39 |
| ogra_ | plars, thats what i thgouth too, but AP doesnt seem to ship permissions for the phablet user to run the dbus introspection bits | 14:40 |
| sergiusens | ogra_: no, running as root is the wrong thing | 14:40 |
| ogra_ | sergiusens, it runs as root today | 14:40 |
| sergiusens | ogra_: that's phablet-config | 14:41 |
| balloons | this was the mp; fginther needs to update and give it a run: https://code.launchpad.net/~fginther/ubuntu-test-cases/add-reminders/+merge/226281 | 14:41 |
| ogra_ | except for the single test commands everything assumes a root shell | 14:41 |
| ogra_ | sergiusens, did you look at phablet-test-run ? | 14:41 |
| sergiusens | ogra_: I wrote it initially with mzanetti almost 2 years ago | 14:41 |
| ogra_ | it stops/starts unity as use and runs the actual test as user ... everything arounnd this needs root | 14:41 |
| sergiusens | ogra_: the start/stop unity isn't used afaik | 14:42 |
| ogra_ | it is in smoke tests | 14:42 |
| ogra_ | i see it in the logs | 14:42 |
| ogra_ | sergiusens, but even then ... 80% of the adb calls are as root | 14:42 |
| ogra_ | (i dont get why it doesnt use phablet-config to install packages and enable ppas though) | 14:43 |
| mzanetti | ogra_: because phablet-test-run is approx 1 year older than phablet-config | 14:44 |
| sergiusens | that ^ | 14:45 |
| ogra_ | mzanetti, sure | 14:46 |
| ogra_ | mzanetti, i'm not saying it is wrong, but sergiusens claimed it wouldnt run stuff as root ... it clearly expects root for many things | 14:46 |
| mzanetti | yes, it does | 14:46 |
| ogra_ | and i will need to fix these up | 14:46 |
| ogra_ | if we want working test results | 14:47 |
| mzanetti | all run run_with_adb calls need root | 14:47 |
| ogra_ | right | 14:47 |
| mzanetti | run_with_ssh is user stuff | 14:47 |
| tedg | Thanks brendand! | 14:47 |
| sergiusens | ogra_: I thought you took care of those during the adb as user migration though | 14:47 |
| ogra_ | sergiusens, i did ... and it worked when i tested here | 14:47 |
| ogra_ | sergiusens, except for the chmod 666 ... for which balloons told me back then we could drop it anyway | 14:48 |
| ogra_ | (since AP ships a udev rule for that device) | 14:48 |
| === nik90_ is now known as nik90 | ||
| ogra_ | sergiusens, it could well be that this extra "sudo -uphablet -i" wrapper everywhere causes the dbus issues though | 14:49 |
| ogra_ | havent checked that yet | 14:49 |
| sil2100 | jhodapp: let me take a look, I was in a meeting | 14:52 |
| sil2100 | Laney: for what? | 14:52 |
| jhodapp | sil2100, thanks | 14:52 |
| Laney | sil2100: the big FFe you mentioned on that hangout | 14:52 |
| sil2100 | Laney: uh, I was sure we had one, but I see that I might not be the case! Crap, but looking into that now | 14:56 |
| Laney | sil2100: I mailed ubuntu-release (see the archives) but some people optimistally said there would be no (or so few that they could individually get FFes) new features now. :) | 14:57 |
| Laney | asac might have more current info | 14:57 |
| sil2100 | Laney: yeah, sorry, I saw you mentioning that you can generate the list and saw that everyone generally agreed and I was sure it was filled with that | 14:58 |
| sil2100 | I just poked him, but I need to poke slangasek if he has any info regarding that | 14:58 |
| sil2100 | Damn, made a fool out of myself | 14:58 |
| sil2100 | ;) | 14:58 |
| brendand | pete-woods, i'm testing silo 18 | 15:01 |
| === kgunn is now known as Guest81961 | ||
| brendand | pete-woods, should translations work for infographics now? | 15:07 |
| === apw_ is now known as apw | ||
| pete-woods | brendand: they would, but the lang pack hasn't been released yet. the "no sources" message isn't translated yet | 15:21 |
| brendand | pete-woods, looks translated here | 15:22 |
| brendand | except for 'photos taken' | 15:22 |
| pete-woods | brendand: the normal sources should be translated | 15:22 |
| pete-woods | brendand: assuming translations are provided. but the build in messages like "no sources" don't work yet | 15:23 |
| brendand | pete-woods, so i can't really test this change? | 15:23 |
| pete-woods | brendand: not really no. to test it I stuffed some extra translations onto my image manually | 15:24 |
| pete-woods | which was quite a pain | 15:24 |
| sil2100 | ogra_: hey, can you proof-read something for me? ;) | 15:26 |
| ogra_ | sil2100, in a meeting, but i can try indeed :) | 15:27 |
| sil2100 | davmor2: btw. thanks for the test report e-mail, it's good to have that :) | 15:32 |
| sil2100 | jhodapp: ah! | 15:33 |
| sil2100 | jhodapp: sorry I only checked now, but it seems it's only worried about the last ~rtm upload that happened | 15:34 |
| sil2100 | jhodapp: as it was a sync, we don't save those versions anywhere in bzr | 15:34 |
| jhodapp | sil2100, ah so it was referencing the last thing that was in that silo for rtm? | 15:35 |
| brendand | tvoss, sergiusens, jdstrand, dbarth_, pete-woods, dobey, bfiller, Mirv - anyone have RTM silos that are almost done being tested? don't forget to mark them so on the spreadsheet | 15:36 |
| sil2100 | jhodapp: it's just confused, as the rtm archive has 0.3.3+14.10.20140911~rtm-0ubuntu1 while the last thing in the branch is 0.3.3+14.10.20140911-0ubuntu1 - the way out of here is just to flip the ignore flag when building the silo | 15:36 |
| tvoss | brendand, yup | 15:36 |
| sil2100 | jhodapp: check FORCE_REBUILD during the build job | 15:36 |
| jhodapp | sil2100, ok cool, let me try | 15:36 |
| pete-woods | will do, just doing the testing now I have sucked some eggs on how to shell into my krillin again | 15:36 |
| sil2100 | Ah, and you probably would like to check DO_NOT_APPEND_RTM_TO_VERSION as well | 15:37 |
| sil2100 | jhodapp: as you guys are doing RTM landings first, right? | 15:37 |
| jhodapp | sil2100, yes | 15:37 |
| jdstrand | cwayne: hey, did your features stuff land so we can put apparmor in the rtm branch? | 15:37 |
| sil2100 | jhodapp: yeah, then for now rememeber checking the not append flag as well :) | 15:38 |
| cwayne | jdstrand: so the ubuntu-touch-customization-hooks bits landed some time yesterday | 15:38 |
| cwayne | the custom tarball needs to be updated now too | 15:38 |
| cwayne | to actually make use of it | 15:38 |
| boiko | sil2100: quick question, is this rtm or utopic image: http://ci.ubuntu.com/smokeng/utopic/touch_stable/mako/42:20140917:20140903.1/10510/dialer_app/1686233/ ? | 15:38 |
| jdstrand | cwayne: cool, so that whole process is in place? | 15:39 |
| cwayne | jdstrand: yep | 15:39 |
| sil2100 | boiko: hey! This is RTM - whenever you see touch_stable in the name then it's ubuntu-rtm | 15:39 |
| jdstrand | cwayne: awesome. let me get a new apparmor-easyprof-ubuntu into utopic, then I can ping you | 15:40 |
| sil2100 | boiko: I mean, in the URL | 15:40 |
| boiko | sil2100: nice! thanks! | 15:40 |
| jdstrand | cwayne: then you can regenerate the tarball with what is in the silo and the new apparmor-easyprof-ubuntu | 15:40 |
| brendand | sil2100, why do we use those names and not just the distro name? | 15:40 |
| brendand | distro/series | 15:40 |
| brendand | sil2100, or the channel name even better | 15:41 |
| jhodapp | sil2100, seems to be building ok now | 15:41 |
| brendand | sil2100, it even says 'utopic/touch_stable' which is super-confusing | 15:41 |
| ogra_ | brendand, i think asac picked the names | 15:42 |
| brendand | ogra_, asac - i'd love to know the reason - i'm sure there's a good one | 15:44 |
| davmor2 | brendand: because he wanted to :P | 15:44 |
| === plars changed the topic of #ubuntu-ci-eng to: Train support: trainguards | Vanguard: plars | Train Dashboard: http://bit.ly/1mDv1FS | QA Signoffs: http://bit.ly/1qMAKYd | Known Issues: Latest flash update hung all makos on s-jenkins. | ||
| brendand | davmor2, you're *sooo* funny | 15:46 |
| brendand | davmor2, see - sarcasm :P | 15:46 |
| davmor2 | brendand: I know right :D | 15:47 |
| sil2100 | brendand: don't ask meeee | 15:47 |
| davmor2 | </new_level_sarcasm_for_brendand> | 15:47 |
| sil2100 | So many things | 15:48 |
| sil2100 | Mirv: let me try looking into that issue now | 15:48 |
| popey | sil2100: I'll be skipping the meeting today. | 15:51 |
| davmor2 | cyphermox: silo-001 how do I test this do I just tail syslog or is there something better? | 15:52 |
| sil2100 | popey: ACK | 15:52 |
| cyphermox | davmor2: you'd need to run dbus-monitor --system | grep ScanDone and check that there aren't two ScanDone in rapid succession | 15:53 |
| bfiller | sil2100: so these failures - which image are they using? http://ci.ubuntu.com/smokeng/utopic/touch_stable/mako/42:20140917:20140903.1/10510/dialer_app/ | 15:57 |
| bfiller | rtm, ubuntu, proposed, etc? | 15:58 |
| sil2100 | bfiller: those are using mako ubuntu-rtm #42, which is the, let's say, equivalent of ubuntu-rtm images for krillin #46 | 15:59 |
| sil2100 | bfiller: those two have different image numbering since we bump the image everytime a new device tarball is released... | 16:00 |
| sil2100 | bfiller: and since we have a new one for krillin frequently, the krillin ones are bumped more | 16:00 |
| sil2100 | ogra_: I'll be right there on the meeting | 16:03 |
| ogra_ | k | 16:03 |
| bfiller | boiko: ^^^^ | 16:03 |
| bfiller | boiko: see what sil2100 said about images being used in smoketests | 16:03 |
| boiko | bfiller: yep, thanks | 16:04 |
| === Ursinha is now known as Ursinha-afk | ||
| sil2100 | boiko, bfiller: it might be confusing indeed, especially that we have like 3-4 numbering schemes now happening that are completely out of sync | 16:07 |
| asac | ogra_: brendand: well, for me there is devel-proposed and stable-proposed which currently is mapped to rtm-proposed. thats where the name is coming from | 16:08 |
| ogra_ | asac, we're talking about touch, touch_stable etc on the smoke testing dashboard | 16:09 |
| asac | reality is that underlyaing archive names disagree and we talk about utopic everywhere which is confusing for those that use the devel-proposed channel | 16:09 |
| asac | ogra_: yes, touch_stable came out of morphong touch with the stable-proposed channel idea | 16:09 |
| asac | :) | 16:09 |
| asac | morphing | 16:09 |
| asac | note that i didnt pick that nmae | 16:10 |
| asac | i gave the high level input like above and someone did that then | 16:10 |
| ogra_ | hmm | 16:10 |
| asac | gave the same input to someone else and they picked the channel name on system-image :) | 16:10 |
| ogra_ | brendand, ^^^^ so we dont know who picked these names i guess | 16:10 |
| brendand | ogra_, color me not surprised :) | 16:10 |
| asac | is it that confusing really? feels like a simple mental mapping exercise to me :) | 16:13 |
| === gatox is now known as gatox_lunch | ||
| sil2100 | asac: to be completely fair, stable-proposed is not linked to rtm right now ;) | 16:21 |
| balloons | so plars ogra_ do we still plan to merge this? https://code.launchpad.net/~nskaggs/phablet-tools/remove-python2-support/+merge/233754 | 16:21 |
| sil2100 | asac: actually 'stable-proposed' does not exist at all | 16:21 |
| sil2100 | asac: there's 'stable', but it's not linked to anything, it's a manual channel where we didn't promote anything to since months | 16:22 |
| davmor2 | cyphermox: erm I see 1 ScanDone every few seconds what are you classing as quick succession? | 16:24 |
| ogra_ | balloons, sure, but i will have to make other changes to phablet-test-run | 16:24 |
| tvoss | sil2100, line 49 complaints about wrong format in testing done column, mind having a look? | 16:24 |
| sil2100 | tvoss: looking | 16:25 |
| ogra_ | balloons, if it has been properly tested, llets just land it before i start ? | 16:25 |
| * ogra_ checks if it is finally top approved | 16:25 | |
| sil2100 | tvoss: ah, don't worry, it's normal... sadly google docs doesn't support pattern-based field validation ;/ | 16:25 |
| tvoss | sil2100, ack | 16:25 |
| balloons | ogra_, that was my thought. It should fix a few of the worst issues atm | 16:26 |
| ogra_ | balloons, do you consider it ready ? i can top approve and quickly land it | 16:27 |
| robru | ogra_: i reviewed your branch | 16:27 |
| ogra_ | robru, yeah, i saw | 16:27 |
| balloons | ogra_, I do and sergiusens reviewed as well. He was just wanting plars to ack it | 16:27 |
| asac | sil2100: yeah i kind of said that :); doesnt really defeat the general way to look at it | 16:27 |
| asac | imo | 16:28 |
| ogra_ | robru, i would still like to keep the chmod (if ~/.ssh doesnt exist or authorized_keys doesnt, it still helps to get the permissions right) but i just dropped the chown (one bzr push away) | 16:28 |
| asac | i dont think we really know yet where this is heading, hence I don't think we shoudl too much effort in making these terms consistent for now | 16:28 |
| ogra_ | balloons, ah, k ... so we'll wait for plars | 16:28 |
| plars | one sec, I'll take a look | 16:29 |
| robru | ogra_: hm, k | 16:29 |
| ogra_ | robru, by default the files will have far broader permissions at creation time, i think ssh would choke | 16:30 |
| * ogra_ tests | 16:30 | |
| sil2100 | Mirv: ah! Damn! I think I see the problem ;/ | 16:31 |
| sil2100 | Mirv: it seems some leftover code from the old sync code is biting our asses again | 16:31 |
| robru | ogra_: yeah ssh freaks out if those files aren't *00 | 16:31 |
| ogra_ | robru, hmm, it doesnt actually ... drwxr-xr-x 2 phablet phablet 4096 Sep 17 16:31 .ssh | 16:32 |
| ogra_ | -rw-r--r-- 1 phablet phablet 221 Sep 17 16:31 authorized_keys | 16:32 |
| ogra_ | i'm logged in fine | 16:32 |
| ogra_ | robru, nontheless they shouldnt be world readable | 16:32 |
| robru | ogra_: not sure why it works for you, i specifically wrote that code because ssh was freaking out and blocking me with those files world-readable | 16:33 |
| ogra_ | yeah | 16:33 |
| davmor2 | cyphermox: ah hang on running the dbus as root not I see 2 in quick succession | 16:34 |
| robru | ogra_: that code was a direct response to what was happening at the time; if it had worked I wouldn't have thought to write the code ;-) | 16:34 |
| ogra_ | robru, bzr ppushed and commented | 16:34 |
| cjwatson | ssh usually only cares about world-writability, with the exception of private keys | 16:34 |
| ogra_ | cjwatson, well, see above | 16:35 |
| cjwatson | there's no problem with having ~/.ssh, or files that only contain public material, be world-readable | 16:35 |
| ogra_ | ah | 16:35 |
| cjwatson | but you do have to have the ownership be correct, normally | 16:35 |
| ogra_ | right, we got this | 16:36 |
| ogra_ | since adbd runs as the correct user with the correct env now | 16:36 |
| cjwatson | right | 16:36 |
| ogra_ | (which needed an awful hack though ... "inception of shells" :) ) | 16:37 |
| cjwatson | just wanted to clarify, afaik the only thing that's required to be *00 is private key files | 16:37 |
| davmor2 | cyphermox: should of read "ah hang on running the dbus as root now I see 2 in quick succession" | 16:37 |
| ogra_ | yay | 16:38 |
| dobey | brendand: mine doesn't seem to even have a silo yet | 16:38 |
| brendand | dobey, https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu-rtm/landing-010? | 16:41 |
| ogra_ | balloons, do you happen to know why phablet-test-run needs /tools/bin in its path ? is that still used | 16:41 |
| balloons | ogra_, hmm.. no can't say I do. ogra_ ohh, if you plan to re-write things btw, you should probably chat with pitti and have a look at adt click support. There's an ssh runner in there | 16:43 |
| dobey | brendand: so why is the spreadsheet not updated? | 16:43 |
| ogra_ | balloons, well, the quick fix is to keep it like it is and just inject sudo for the root needed parts ... long term we should probably just switch over | 16:44 |
| ogra_ | (i heard using adt is the long term plan anyway) | 16:44 |
| balloons | ogra_, that was one of the thoughts.. I'm starting to push using adt | 16:45 |
| brendand | dobey, ah i just realised yours is not required QA sign-off | 16:45 |
| ogra_ | right | 16:45 |
| balloons | it handles things sanely, though you are always free to ssh in and run them yourself :-) | 16:45 |
| balloons | ogra_, I'm curious about what root needs you have though.. outside of updating the click hooks? | 16:45 |
| ogra_ | well, i dont think ssh is actually an option for smoke testing | 16:45 |
| dobey | brendand: but still, the spreadsheet not having the proper status seems like a problem | 16:45 |
| balloons | ogra_, right.. I was speaking that adt can be seen as overkill for one-off developer runs | 16:46 |
| ogra_ | balloons, well, apparently APs introspection doesnt work anymore | 16:46 |
| brendand | dobey, i'm actually not sure what you're expecting to see. if there's something wrong ask sil2100/robru | 16:46 |
| ogra_ | and i assume there is an env issue when not running the tests as root | 16:46 |
| ogra_ | as they were before | 16:47 |
| sil2100 | The spreadsheet is a bother | 16:47 |
| dobey | brendand: column C in row 67 is still empty | 16:47 |
| dobey | sil2100: what's up with that? | 16:47 |
| ogra_ | oh ! | 16:47 |
| sil2100 | dobey: sorry, just my attitude towards the spreadsheet ;) Let me take a look at what's up | 16:47 |
| balloons | ogra_, really? is there a bug for what you are describing? That seems really off.. I'm not seeing such things | 16:48 |
| ogra_ | balloons, all app tests fail today | 16:48 |
| balloons | heh, magic | 16:48 |
| dobey | sil2100: oh i understand that. i hate spreadsheets :) | 16:48 |
| ogra_ | and no.. there isnt a bug .., thats enough of an emergency to just fix it | 16:48 |
| robru | brendand: dobey sil2100: ok i fixed row 67 in the spreadsheet | 16:49 |
| ogra_ | balloons, http://ci.ubuntu.com/smokeng/utopic/touch_stable/mako/42:20140917:20140903.1/10510/dialer_app/1686233/ scroll down, thats one example | 16:49 |
| ogra_ | it isnt in all apps | 16:49 |
| ogra_ | but most | 16:49 |
| balloons | ogra_, ahh.. I just flashed to the newest image.. haven't tried things yet | 16:49 |
| robru | sil2100: it lost the request id and the assigned silo name, just had to copy&paste from the dashbaord | 16:49 |
| ogra_ | balloons, but i think i understand whats going on, phablet-test-run uses "sudo -u phablet -i /bin/sh -c" | 16:50 |
| dobey | does emulator to OTA updates? | 16:50 |
| dobey | because my mako is totally borked right now and i have no idea how to fix it :( | 16:50 |
| ogra_ | balloons, that will actually drop the (now correct) bash env and use /bin/sh (which is dash and doesnt read .profile, so it doesnt have the dbus address) | 16:50 |
| balloons | ogra_, wild.. interesting | 16:51 |
| * ogra_ assumes just ripping out that sudo stuff will make it work | 16:51 | |
| balloons | a fair assumption | 16:52 |
| robru | sil2100: how's your watch-ppa work going? | 16:52 |
| cyphermox | davmor2: it depends how fast, you're supposed to see them come up every few seconds | 16:53 |
| robru | sil2100: I was looking at the way it scans for those .project files, but I couldn't find the place in build.py where those actually get created, so indeed it seems to me there's some kind of race condition, and we should pass the package list differently... | 16:53 |
| ogra_ | plars, could you try this hack to phablet-test-run and try re-running one of the failed apps with it http://paste.ubuntu.com/8366191/ ? | 16:53 |
| sil2100 | robru: one moment, OTP :) | 16:53 |
| ogra_ | (i'll work on the sudo root bits too) | 16:54 |
| plars | ogra_: sure | 16:54 |
| plars | balloons: I'm hitting some problems with that branch | 16:54 |
| ogra_ | awesome | 16:54 |
| plars | balloons: http://q-jenkins.ubuntu-ci:8080/job/plars-smoke-daily-test/34/console | 16:54 |
| davmor2 | cyphermox: so just running dbus-monitor --system | grep ScanDone I got one only every few second if I run sudo dbus-monitor --system | grep ScanDone then I get 2 every few second | 16:54 |
| cyphermox | show me? | 16:55 |
| dobey | i guess it doesn't | 16:55 |
| plars | balloons: you need to watch the way that is passed, or set the default value to [] | 16:55 |
| dobey | get the notification, but updates tab sits at "Checking for updates" indefinitely | 16:55 |
| plars | ogra_: do you have a branch with this pushed somewhere already? | 16:57 |
| ogra_ | plars, not yet, nope | 16:57 |
| plars | ogra_: it doesn't need to be proposed | 16:58 |
| ogra_ | there is more that needs to change ... | 16:58 |
| ogra_ | ok, let me quickly roll one | 16:58 |
| plars | ogra_: ok, I have an easier way to test it if you push it somewhere | 16:58 |
| ogra_ | yeah, gimme a sec | 16:58 |
| * balloons looks | 16:59 | |
| ogra_ | plars, https://code.launchpad.net/~ogra/phablet-tools/phablet-tools-fix-test-run/+merge/235015 | 17:00 |
| plars | ogra_: which one would be a good one to try? calendar app maybe? http://ci.ubuntu.com/smokeng/utopic/touch/mako/244:20140917:20140917/10503/calendar_app/1685292/ | 17:00 |
| balloons | plars, ahh, yes, fallout from the change from a global. | 17:00 |
| ogra_ | plars, i think elopio shoed other issues with calendar | 17:00 |
| elopio | plars: there are four expected failures on the calendar. | 17:01 |
| ogra_ | plars, well, but if you dont see dbus ones that might be enough | 17:01 |
| ogra_ | dialer seems liek a good candidate though | 17:02 |
| plars | ogra_: ok, I'll try that one | 17:02 |
| ogra_ | but that has so much extra stuff installed | 17:02 |
| ogra_ | (phonesim etc) | 17:02 |
| plars | ogra_: AssertionError: After 10.0 seconds test on DialerPage.isReady failed: True != dbus.Boolean(False, variant_level=1) is the type of error we're looking for? | 17:02 |
| ogra_ | no | 17:02 |
| ogra_ | autopilot introspection | 17:03 |
| ogra_ | plars, oh, indeed | 17:04 |
| plars | right | 17:04 |
| ogra_ | i was looking at console log ... but yeah,, thats the line you see in the test output | 17:04 |
| ogra_ | plars, did you re-run many tests on mako rtm ? | 17:05 |
| plars | ogra_: not yet, I was more focused on krillin | 17:05 |
| ogra_ | looking at the result from 42 here it doesnt look that bad actually | 17:05 |
| plars | ogra_: psivaa_-afk may have hit them earlier though | 17:06 |
| ogra_ | ah | 17:06 |
| === psivaa_-afk is now known as psivaa_ | ||
| balloons | plars, fixed the nonetype issue and pushed as rev 316 | 17:07 |
| * ogra_ looks at 244 mako instead, since that has all the latest stuff | 17:07 | |
| psivaa_ | plars: sorry missing context on what i may have hit :) | 17:12 |
| === gatox_lunch is now known as gatox | ||
| ogra_ | psivaa_, we were wondering if you re-ran any of the mako rtm tests | 17:13 |
| ogra_ | i think i remember they looked wrose this morning ... but probably i mis-remember | 17:13 |
| ogra_ | we have surely too many channels now to keep track by memory | 17:13 |
| psivaa_ | ogra_: i dont think i reran them. hmm sorry abou tit | 17:14 |
| psivaa_ | *about it | 17:14 |
| ogra_ | psivaa_, all fine ... nobody asked you to re-run them :) | 17:14 |
| psivaa_ | ogra_: ack :) | 17:14 |
| ogra_ | i was just wondering why mako rtm looked so good | 17:14 |
| ogra_ | :) | 17:14 |
| dobey | does anyone know how to fix a busted mako that won't accept flashing? | 17:14 |
| ogra_ | how does it not accept flashing ? did you try --bootstrap ? | 17:15 |
| dobey | yes i've tried everything i can think of at this point | 17:15 |
| balloons | I agree it's pretty difficult to completely brick a phone | 17:15 |
| ogra_ | (bootstrap should always work, since it runs from the bootloader) | 17:15 |
| dobey | it keeps complaining about /cache/recovery | 17:15 |
| ogra_ | dobey, when you run from bootloader and with --bootstrap ? | 17:16 |
| dobey | ogra_: yes, several times | 17:16 |
| balloons | dobey, I would re-flash twrm recovery,or even just do a full factory restore. fastboot should let you flash do your delight | 17:16 |
| dobey | balloons: have also tried reflashing android, and even it is failing :-/ | 17:16 |
| balloons | can you not get into fastboot? | 17:17 |
| ogra_ | well, if he ran with --bootstrap he has to | 17:17 |
| dobey | 10:12 < dobey> 2014/09/17 10:12:05 Cannot push /home/dobey/.cache/ubuntuimages/pool/ubuntu-cbe7b905c3a6b174f5e650d248091dbb6d69a7ed68d2be650d4495f6cbaa1a4a.tar.xz.asc to device: free space on /cache/recovery is unknown | 17:17 |
| dobey | stuff like that happening | 17:17 |
| ogra_ | --bootstrap actually runs the new recovery from the image (not the device) under fastboot ... then formats the partitions and flashes from recovery | 17:18 |
| ogra_ | and i dont see how such error could show up with --bootstrap | 17:18 |
| ogra_ | i have surely seen it before with less agressive options to u-d-f ... like --wipe | 17:19 |
| dobey | well now i am getting this: | 17:19 |
| dobey | 2014/09/17 13:19:06 Can't flash recovery image | 17:19 |
| balloons | bootloader unlocked? | 17:19 |
| dobey | yes of course | 17:19 |
| balloons | it relocks on factory reset I think.. so check :-) | 17:19 |
| dobey | it's not reset on factory reset in my experience. one has to actually lock it using the fastboot tool, after flashing android back on, iirc | 17:21 |
| dobey | sending 'radio' (45537 KB)... | 17:21 |
| dobey | FAILED (data transfer failure (Protocol error)) | 17:21 |
| dobey | getting that when trying to flash android on | 17:21 |
| ogra_ | dobey, try another cable | 17:22 |
| ogra_ | (Protocol error) ... | 17:22 |
| dobey | also tried that | 17:22 |
| dobey | FAILED (data transfer failure (Cannot send after transport endpoint shutdown)) | 17:22 |
| * dobey finds yet another cable to try | 17:23 | |
| dobey | and whee, protocol error with it too :( | 17:23 |
| dobey | all i wanted to do was flash back to devel-proposed from rtm to test things, and the world exploded :-/ | 17:25 |
| robru | lool: tvoss: silo utopic2 needs an rtm sync as well right? | 17:38 |
| plars | ogra_: that branch does not seem to be helping with the introspection errors: http://q-jenkins.ubuntu-ci:8080/job/plars-smoke-daily-test/35/console | 17:38 |
| lool | robru: in rtm? | 17:38 |
| lool | robru: yes | 17:38 |
| plars | still seeing | 17:38 |
| plars | File "/usr/lib/python3/dist-packages/autopilot/introspection/types.py", line 178, in wait_for | 17:38 |
| plars | failure_msg)) | 17:38 |
| plars | AssertionError: After 10.0 seconds test on KeypadEntry.value failed: '1 23' != dbus.String('', variant_level=1) | 17:38 |
| ogra_ | plars, danm | 17:39 |
| lool | robru: thomas pinged me earlier to add an entry for rtm, and I failed to create one; I wasn't sure whether I could insert lines in the spreadhsheet | 17:39 |
| robru | lool: yep it's best to keep the rtm sync row adjacent to the utopic landing, otherwise it's (even more) chaos | 17:40 |
| robru | lool: anyway building in rtm13 now | 17:40 |
| lool | robru: thanks | 17:41 |
| robru | lool: you're welcome | 17:41 |
| davmor2 | robru your favourite silo-001 is good :) | 17:42 |
| robru | davmor2: rtm1? i published that one already | 17:44 |
| robru | davmor2: I mean, uh, "thanks" | 17:44 |
| davmor2 | robru: \o/ sorry had to nip off | 17:44 |
| davmor2 | robru: no issues this time though which is good :) | 17:45 |
| robru | davmor2: great | 17:45 |
| cwayne | fginther: hi, so youve mentioned that we're moving from s-jenkins to 'the cloud'.. do you have any more information i could look at? | 17:47 |
| lool | mandel: may I remove row 8 of spreadsheet now? | 17:48 |
| sil2100 | robru: back from the phone - I actually tried working around it yesterday, but didn't test if that works currently... it's best to loop through the list of .project files as it's doing now since this way you're sure to check all packages that have been uploded and make sure that the backend is tracking them correctly | 17:49 |
| robru | sil2100: ok so why aren't the .project files being created by the time that watch-ppa is running for the first time? | 17:50 |
| robru | sil2100: actually your statement is ridiculous on it's face, because the exact problem we're having is that we're surely not checking all packages that have been uploaded. we're in fact checking zero of the packages that have been uploaded. | 17:51 |
| robru | sil2100: should we just put in a "sleep 30" and hope the .project files are created in time? | 17:51 |
| lool | robru: would you mind handing me a silo for row 71, Upstart job fixes for HERE wrappers? | 17:52 |
| robru | lool: please mark it ready | 17:52 |
| lool | robru: this would also land in RTM later on, but there's a landing for location service stuff already which was delayed by issues in QA; with this and thomas' landing, we can land | 17:53 |
| lool | done | 17:53 |
| lool | robru: marked ready | 17:53 |
| robru | lool: ok you got utopic 20 | 17:53 |
| lool | thanks | 17:53 |
| robru | you're welcome! | 17:54 |
| === Ursinha-afk is now known as Ursinha | ||
| sil2100 | robru: well, it's a problem that it's not seeing the .project files - there is no reason for them not to exist during watch-ppa | 18:01 |
| sil2100 | robru: and I see what's up now | 18:02 |
| robru | sil2100: oh? | 18:02 |
| sil2100 | There is a small problem with when the .project files are generated in this *particular* case | 18:02 |
| robru | sil2100: it's just the sync case, right? | 18:03 |
| robru | sil2100: if you don't fix it before you EOD I'm going to commit this to trunk: http://paste.ubuntu.com/8366598/ | 18:05 |
| sil2100 | robru: hoho, noooo! I have a possible solution, since the problem here is that I need to generate the .project files when I do the source uploads, just need to make sure we don't get conflicts | 18:07 |
| sil2100 | Since I see that it might actually try to re-generate the .project file if I don't change the logic | 18:08 |
| robru | sil2100: what's even in these project files? how bad can it be to regenerate them? | 18:08 |
| robru | brb | 18:23 |
| sil2100 | Nothing bad will happen for sure, so in the end I would leave it regenerating, but better not to do any useless operations | 18:24 |
| sil2100 | robru: actually one tweak needs to be done anyway | 18:34 |
| tvoss | robru, yup | 18:49 |
| robru | sil2100: do you have a branch i can review? | 18:56 |
| === Ursinha is now known as Ursinha-afk | ||
| sil2100 | robru: yes, but before a review I need to test it in preprod if it's the right way | 19:02 |
| robru | sil2100: ok | 19:03 |
| renatu | fginther, om26er, I am having a problem with jenkins: https://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-utopic/2970/testReport/junit/address_book_app.tests.test_delete_contact/TestDeleteSelectContact/test_select_single_cancel_/ | 19:08 |
| renatu | fginther, om26er : 17:43:18.078 ERROR content:47 - Could not add content object 'None' due to IO Error: [Errno 13] Permission denied: '/var/log/syslog' | 19:08 |
| sil2100 | robru: argh, 000 again doesn't use preprod ;/ | 19:10 |
| * sil2100 modifies it manually again | 19:11 | |
| sil2100 | robru: did you have a moment to take a look at this? | 19:11 |
| lool | trainguards, I'd like to get a rtm silo for "Fix Ubuntu Location Service to hook correctly with Ubuntu Espoo Service (HERE positioning engine)" on row 27 | 19:12 |
| lool | also, I'd like to put not 2 put 3 packages there frmo utopic | 19:12 |
| lool | location-service, lxc-android-config, ubuntu-location-provider-here | 19:12 |
| lool | do I just update the sync:2 to sync:3? | 19:12 |
| sil2100 | lool: those are already in utopic, yes? | 19:13 |
| lool | sil2100: yes; ubuntu-location-provider-here is on its way from proposed to utopic | 19:14 |
| sil2100 | lool: then fill in a landing `sync:ubuntu,utopic location-service lxc-android-config ubuntu-location-provider-here` | 19:14 |
| lool | sil2100: thanks | 19:14 |
| sil2100 | lool: just in case - when you press build, check the DO_NOT_APPEND_RTM_TO_VERSION flag maybe | 19:15 |
| sil2100 | lool: :) | 19:15 |
| lool | ok | 19:15 |
| sil2100 | (just in case citrain tries to add ~rtm to the lxc-android-config package) | 19:15 |
| lool | yup | 19:15 |
| robru | sil2100: sorry I didn't get a chance to look at the deploy job yet, was too busy being defeated by other things. | 19:21 |
| === plars changed the topic of #ubuntu-ci-eng to: Train support: trainguards | Vanguard: cihelp | Train Dashboard: http://bit.ly/1mDv1FS | QA Signoffs: http://bit.ly/1qMAKYd | Known Issues: Latest flash update hung all makos on s-jenkins. | ||
| sil2100 | Yeah, it's a busy period :/ | 19:22 |
| === Ursinha-afk is now known as Ursinha | ||
| lool | sil2100: I'm still on for the silo? | 19:29 |
| sil2100 | lool: sorry! I'll assign it in a moment, let me check if something's free :) I'm in the middle of testing some citrain fixes | 19:31 |
| lool | sil2100: 4 rtm are free | 19:31 |
| lool | sil2100: no hurry, thanks | 19:31 |
| lool | sil2100: ty | 19:35 |
| sil2100 | lool: yw! Sorry it took so long :) | 19:42 |
| AlbertA2 | cihelp: can I get a silo for row 71? | 19:43 |
| plars | AlbertA: ask trainguards for things like that | 19:44 |
| sil2100 | AlbertA2: you need to ping trainguards ;) We'll try to do that soon | 19:44 |
| AlbertA2 | thanks! | 19:45 |
| === Ursinha changed the topic of #ubuntu-ci-eng to: Need a silo? Ping train support: trainguards | Vanguard: cihelp | Train Dashboard: http://bit.ly/1mDv1FS | QA Signoffs: http://bit.ly/1qMAKYd | Known Issues: Latest flash update hung all makos on s-jenkins. | ||
| Ursinha | I think I added that to topic before but it disappeared :) | 19:47 |
| sil2100 | hah ;) | 19:48 |
| Ursinha | :) | 19:49 |
| Ursinha | as a helper :) I don't mind replying requests like that, but cihelp isn't always available and people might end up unnecessarily waiting for a silo | 19:50 |
| plars | Ursinha: well, I don't think we can assign them anyway though can we? | 19:51 |
| Ursinha | plars: nope, hence the topic change :) | 19:52 |
| Ursinha | ah, I see your point | 19:52 |
| robru | sil2100: don't merge your branch yet | 19:52 |
| plars | Ursinha: right, I wasn't sure what you meant about 'replying to requests like that' | 19:52 |
| plars | Ursinha: if I could, I wouldn't mind helping | 19:52 |
| sil2100 | robru: no no, I still need to test some smaller things | 19:52 |
| Ursinha | plars: I meant (and failed) that there won't be a cihelp person to say "hey, this is actually traing uards" :) | 19:52 |
| plars | right | 19:53 |
| plars | good edit | 19:53 |
| Ursinha | hehe | 19:53 |
| sil2100 | robru: but so far at least sync:silo are working | 19:54 |
| robru | AlbertA2: ok you got silo utopic 2 | 19:54 |
| sil2100 | robru: it catches the newly uploaded packages and such, just need to make sure nothing else is broken | 19:54 |
| robru | sil2100: nm, I thought I saw an unused variable, but you are using it. branch seems fine then | 19:55 |
| sil2100 | (created an MR so that your CI can be run against it) | 19:55 |
| robru | sil2100: you'll get a pep8 error due to line length | 19:57 |
| robru | or I guess not, that file isn't under pep8 yet. man I gotta finish that off | 19:57 |
| sil2100 | I tried making sure that all the ones that are under check are passing pep8 | 19:58 |
| sil2100 | robru: ok, the branch seems to work ok... sadly I have no time to test all the cases | 20:01 |
| * sil2100 needs to finish the e-mail now | 20:01 | |
| sil2100 | robru: in case something is broken - remember! Reverrrt | 20:01 |
| robru | sil2100: ok just land it and I'll revert if it explodes. | 20:01 |
| === Ursinha is now known as Ursinha-afk | ||
| robru | sil2100: did you deploy prod yet? | 20:15 |
| robru | sergiusens: ^ dont' build yet, were about to deploy something that might fix some stuff | 20:16 |
| sil2100 | robru: it didn't get merged yet by the bot | 20:18 |
| robru | sil2100: hm should soon. brb | 20:18 |
| sergiusens | robru: sure; btw, I'm not sure I got to sync nuntium or ubuntu-download-manager back into utopic | 20:27 |
| sergiusens | robru: it's kind of hard to keep track, I'm mostly sure it didn't happen | 20:27 |
| sergiusens | robru: so if you want to bundle all those even though unrelated; I'm fine | 20:28 |
| sil2100 | robru: redeployed | 20:28 |
| sil2100 | See you tomorrow o/ | 20:29 |
| bfiller | robru: can I get a silo for line 75 please? | 20:29 |
| robru | sergiusens: sounds good | 20:35 |
| robru | bfiller: can do | 20:35 |
| === Ursinha-afk is now known as Ursinha | ||
| robru | bfiller: ok you got utopic 20 | 20:36 |
| bfiller | robru: thank you sir | 20:36 |
| robru | bfiller: you're welcome | 20:36 |
| robru | sergiusens: ok, trying the build in silo 14. we'll see if it doesn't require a watch_only | 20:38 |
| robru | sergiusens: https://ci-train.ubuntu.com/job/ubuntu-landing-014-1-build/53/console | 20:38 |
| robru | sergiusens: well seems like it worked but worrisome that it didn't catch nuntium... | 20:47 |
| robru | sergiusens: of course udm and ciborium disappeared off the dashboard. faith in the code: rising | 20:48 |
| balloons | ogra_, should sudo be prompting me for a password on krillin? I don't have one to give it | 20:55 |
| brendand | balloons, you must have | 20:59 |
| brendand | balloons, what's your lock pin/passphrase? | 20:59 |
| brendand | balloons, don't tell me :) | 20:59 |
| balloons | brendand, :-) yes I have a pin set | 20:59 |
| brendand | balloons, it's that | 20:59 |
| balloons | brendand, brillant, thanks | 21:00 |
| === Ursinha is now known as Ursinha-afk | ||
| ToyKeeper | sigh, another silo with no test plan whatsoever and nobody around to ask about it. | 22:17 |
| thomi | ToyKeeper: that's easy though, right? reject! | 22:21 |
| barry | robru: ping | 22:57 |
| robru | barry: pong | 22:58 |
| === kgunn is now known as Guest93957 | ||
| barry | robru: hi. after several attempts i've gotten my silo ppa to build the package. but i don't remember what i have to do to kick the silo without causing it to actually rebuild | 23:17 |
| robru | barry: yeah things are pretty busted right now | 23:18 |
| robru | barry: what silo? | 23:18 |
| barry | robru: 19 | 23:18 |
| robru | barry: so, what, you think the silo has the right contents but the build job is making a false error? | 23:19 |
| sergiusens | robru: are we good with 14 now? | 23:19 |
| sergiusens | robru: just go back here | 23:19 |
| robru | sergiusens: yeah, all packages built. just give them a quick test and I'll publish | 23:20 |
| barry | robru: it's just that the ppa has trouble building the package and it often takes several rebuild attempts to get the package built. the ppa now has a good build, but the silo still thinks that the package has build failures. | 23:20 |
| robru | barry: ok so you what you wanna do is run the build job with WATCH_ONLY checked | 23:21 |
| sergiusens | barry: shouldn't a "watch" only build solve that? | 23:21 |
| robru | that should find the right status | 23:21 |
| barry | robru, sergiusens awesome, that was the thing i was forgetting. | 23:21 |
| * barry tries | 23:21 | |
| * sergiusens has been landing too much lately :-p | 23:21 | |
| barry | right, and you have to do two builds+watchonly in a row ;) | 23:22 |
| robru | barry: citrain is just imploding. I'm trying to clean it up and add tests but it's just a nightmare. | 23:22 |
| barry | robru: :( | 23:23 |
| robru | barry: yeah, the code makes me cry | 23:23 |
| robru | barry: did you see my G+ post when I realized it had 1200 pep8 violations? | 23:23 |
| barry | robru: i didn't. that would make me cry to | 23:23 |
| barry | *too | 23:23 |
| robru | barry: yeah, so it's getting better. but it's slow | 23:24 |
| robru | barry: I routinely delete functions and find they have no tests | 23:24 |
| barry | robru, sergiusens can you take a look at the dashboard. it says "Packages built Build failed: Some packages failed to build." | 23:24 |
| robru | barry: that's fine, it's just the stale status from the spreadsheet | 23:25 |
| robru | it'll fix itself after 5 mins | 23:25 |
| barry | robru: awesome, thanks. i'm eod (work/life balance and all :) so i'll test the packages tomorrow | 23:25 |
| === Ursinha-afk is now known as Ursinha | ||
| cwayne | ToyKeeper: which one? the unity-scopes-api one? | 23:48 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!