=== Ursinha is now known as Ursinha-afk === Ursinha-afk is now known as Ursinha === kgunn is now known as Guest14808 [02:05] === trainguards: IMAGE 244 building (started: 20140917 02:05) === === kgunn is now known as Guest35270 [03:05] === trainguards: RTM IMAGE 45 building (started: 20140917 03:05) === [03:08] 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:09] robru: any ideas how to fix this? the ubuntu build was fine.. ^^^ [03:10] ... get git? [03:10] Sorry, I don't know the details... just had to take the opportunity to say that out loud. [04:00] === trainguards: IMAGE 244 DONE (finished: 20140917 04:00) === [04:00] === changelog: http://people.canonical.com/~ogra/touch-image-stats/244.changes === [04:11] morning [04:12] 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:15] bfiller: since it's a sync, I'll just try what would it say without the ~rtm version mangling [04:16] 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:20] === trainguards: RTM IMAGE 45 DONE (finished: 20140917 04:20) === [04:20] === changelog: http://people.canonical.com/~ogra/touch-image-stats/rtm/45.changes === [04:31] 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 [05:15] Mirv: what did you do? Manual source package upload? [05:17] 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] robru: no, copy-package [05:17] I forgot about that option [05:18] Mirv: right, same thing, you put the package there without using a train job to do it [05:19] Anyway, looks like you have it under control, goodnight! [05:40] nighty night, all of you who have not yet waken up to the morning of 17th :) [06:11] 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:49] 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. === tvoss is now known as tvoss|test === tvoss|test is now known as tvoss [07:28] popey: clock done [07:28] Mirv: i hope you mean calculator? [07:29] Mirv: checked store, you do. [07:29] Thank you! [07:29] popey: s/clock/calculator/... [07:30] c something [07:37] sil2100: are we going to get a promoted devel image at some point? [07:37] (morning btw) [07:40] popey: morning! [07:41] 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] popey: I'll just double check with one of our QA people that krillin is also fine on devel and we could promote [07:42] 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] then everyone gets snooze for their alarms! [08:21] sil2100: need to be afk for a bit and may miss the early part of the meeting. === psivaa_ is now known as psivaa_-afk [08:26] psivaa_-afk: ACK === psivaa_-afk is now known as psivaa [08:52] is there a special procedure to get non-train packages into rtm silos or should I just copy it? [09:01] davmor2, http://people.canonical.com/~jhm/barajas/device_krillin-20140917-19d7408.tar.xz [09:01] john-mcaleely: thanks === 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. [09:09] Laney: you can use CI Train, but for now you can just copy - what packages do you have in mind? Syncing from utopic? [09:12] sil2100: I thought it had to be QAed [09:12] yes, syncing [09:15] vila: hi, our autopilot setup has broken again https://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/4548/console any idea? === vrruiz_ is now known as rvr [09:21] tsdgeos: apart from the all caps in: 'SKIPPING phablet-flash AS REQUESTED' nothing catch my un-educated eyes [09:22] ok, so no autopilot CI then [09:22] tsdgeos: i.e. not flashing and later on failing during the setup sounds... somwhow expected ? [09:23] i have no clue how this is setup [09:23] i know it worked [09:23] and now it doesn't :D [09:23] ogra_, so robru's citrain script is currently completely unable to cope with the adb changes [09:23] tsdgeos: ;-) We'll need experts then, I'll track it and let you know or get pinged back [09:23] brendand, as is phablet-test-run apparently ... sigh [09:23] vila: appreciated :) [09:23] chmod: changing permissions of '/dev/uinput': Operation not permitted [09:24] it tries a chmod 666 /dev/uinput at the very start [09:24] :( [09:24] ogra_, oh yeah [09:24] so i will have to add a --password option to it [09:25] :( [09:25] i guess the same goes for the ci tools [09:25] * ogra_ will get this fixed today ... [09:25] damn [09:26] this is the tenth re-implementation of "adb shell" calls in phablet-tools [09:26] :( [09:37] is --ppa option of phablet-config writable-image working at all?? [09:43] 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] errr [09:43] tsdgeos: do you have more recent failures ? [09:44] that's weird it's linked from a more recent thing [09:44] let me see [09:45] tsdgeos: the re-run seems to proceed better (at least it pased the point where it failed previously) [09:45] tsdgeos: http://s-jenkins.ubuntu-ci:8080/job/generic-deb-autopilot-runner-mako/4646/console [09:45] doh :/ [09:46] must have been tricked by the so many confusing links [09:46] sorry [09:46] tsdgeos: no worries [09:46] pete-woods, do you know how testing is going for silo 16 in RTM? [09:47] 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:50] 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] ogra_, that's broken... [09:51] does anyone test this stuff... [09:51] where is that from ? [09:51] ogra_, from phablet-config [09:51] hrm [09:51] indeed that has been tested [09:51] I got the rtm chroot from LP but it had an archive.u.c sources.list [09:51] is that usual? [09:52] I was expecting something different [09:53] I assume launchpad-buildd overrides it [09:53] * Laney does that manually [09:54] brendand, i dont get what you mean, it cant thin it is on the host, it is executed inside an adb shell [09:54] ogra_, http://paste.ubuntu.com/8364004/ [09:54] brendand, can you give me the exact error ? [09:54] thanks [09:54] brendand, no, from phablet-config [09:55] (which creates this file before trying to use it) [09:55] ogra_, it doesn't give an error, it just silently does nothing [09:55] it works fine here [09:55] just tried [09:55] ogra_, but that's the command it runs [09:56] Laney: yes, that's usual, Launchpad always forces the appropriate-context sources.list into place at the start of a build [09:57] cjwatson: Kay, I figured that would be it [09:57] ogra_, what's the outcome, is there a new file in /etc/apt/sources.list.d/? [09:57] http://paste.ubuntu.com/8364019/ [09:58] ogra@anubis:~/image-stuff$ adb -s 0079a79f1857654a shell [09:58] phablet@ubuntu-phablet:~$ ls /etc/apt/sources.list.d/ [09:58] ci-train-ppa-service-ubuntu-landing-009-utopic.list [09:58] yes [09:58] ogra_, ah but that's not what phablet-config does [09:58] that is exactly what phablet-config does [09:59] ogra_, well i get '#!/bin/shecho ' in the /tmp file [10:00] 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] sil2100: what did I miss? [10:00] brendand: /me takes seconds to work out what the "shecho" program might be [10:01] sil2100: silo 11, branch https://code.launchpad.net/~nick-dedekind/ubuntu-settings-components/indicator-polishing/+merge/229083 [10:02] mzanetti: removal of branches I think requires a landing team reconfigure :) [10:02] mzanetti: let me reconfigure for you [10:02] cjwatson, yes - dodgy newlines [10:02] thanks [10:02] brendand, you did supply -r, right ? [10:03] ogra_, yes - otherwise it would never run that code [10:03] cjwatson, adb.shell('/bin/echo -e \\"#!/bin/sh\necho %s \\" > %s ' % (args.remotepassword[0], tempfile), ignore_errors=False) [10:03] but right, the newlines are garbled, sigh [10:04] ogra_, what does yours look like, since i see you supplied two newlines [10:05] ogra_, do you have a space between the two lines? [10:05] brendand, i guess the same as yours ... [10:05] let me try a fix ... [10:05] why did that work during testing (not only for me) [10:06] ogra_, could echo have broken? seems impossible, but why doesn't it insert the newline? [10:06] brendand, thats a quoting issue [10:07] ogra_, when did it last work for you? [10:07] brendand, when testing before uploading/merging [10:08] ogra_: ugh, seriously, use printf [10:08] consistent shell interpretation FTW [10:08] (yeah, I know /bin/echo is probably safe, but ...) [10:09] adb.shell is a pretty awful interface by the looks of it :) [10:09] it is [10:10] ogra_, i'm just puzzled how this ever worked [10:10] and that it has to wrap two bash calls inside a sudo call to get the right env doesnt help much either [10:11] ogra_, can you paste me the fix? [10:11] brendand, well, if i dont get pinged every 30sec i could actually work on it :P [10:11] mine more backslashes [10:14] yeah [10:14] 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] well, convincing printf to accep ! in #!/bin/sh seems not so easy [10:14] SHIP IT [10:14] sil2100, ok, I'll push it now. thanks davmor2 [10:15] ogra_: printf doesn't interpret ! in any way [10:15] ogra_: you may of course have to quote it to protect it from the *shell* trying to do history expansion on it [10:15] right [10:15] single quotes help [10:15] (though that shouldn't happen in noninteractive shells) [10:15] ogra_, escaping it worked for me [10:16] or not... [10:16] sil2100, shipped [10:16] john-mcaleely, davmor2: thanks! [10:17] brendand, let me fix this properly using printf as cjwatson suggests [10:18] is there a manifest for rtm? [10:18] Laney, on cdimage [10:18] found it [10:18] Laney, and changelogs in a subdir at the usual place too [10:18] under rtm/ [10:19] grr! [10:19] ogra_: rtm doesn't even have those packages you were complaining about [10:20] because it has gst-bad 1.2.4 [10:20] Laney, well, the gst merge landed one day before rtm opened [10:20] https://launchpad.net/ubuntu-rtm/+source/gst-plugins-bad1.0 [10:21] cjwatson, hmm, now how do i combine printf's %s with pythons %s ? [10:21] %% [10:22] thanks ! [10:26] ogra_, might it be less confusing to not use it at all? [10:27] ogra_, and just use pythons? [10:28] - adb.shell('/bin/echo -e \\"#!/bin/sh\necho %s \\" > %s ' % [10:28] + adb.shell("printf '%%s\\n' '#!/bin/sh' 'echo %s' >> %s " % [10:28] there is your fix [10:28] err that looks undefined [10:28] %%s%%s surely? [10:29] I guess it works but seems like by luck [10:30] cjwatson, first one is fromat and then two times content [10:30] oh format is reused [10:30] right :) [10:30] hm, ok, news to me :) [10:30] but it's documented, so ... [10:30] yeah [10:30] and works nicely [10:31] i'm a bit uneasy about the >> [10:31] but then, the mktemp is in the line above [10:40] sil2100: would you drop line 56 please === tvoss is now known as tvoss|lunch [10:46] ogra_, probably the reason people didn't notice is because phablet-config swallows all the output [10:46] brendand, yeah, currently working on that ;) [10:46] and i also dropped the need for rebooting if the image is already writable [10:47] ogra_, nice [10:47] i want to drop 80% of phablet-test run for it [10:47] the ppa setup, adb sudo setup etc should only live in one script [10:51] Laney: sure [11:01] bah, request ID:s disappearing again [11:02] brendand, http://paste.ubuntu.com/8364323/ === MacSlow is now known as MacSlow|lunch [11:04] ogra_, much better [11:06] ok let me prepare a landing for this [11:10] brendand, https://code.launchpad.net/~ogra/phablet-tools/phablet-tools-fix-writable-image/+merge/234946 if you like to test and approve [11:13] sil2100: hmm... that branch still shows up in the dashboard [11:13] balloons, when you get a moment, can you review https://code.launchpad.net/~fginther/cupstream2distro-config/core-apps-python3/+merge/234902 ? [11:14] mzanetti: hmmm, let me check then [11:14] ogra_, would dpkg-query also be broken on rtm? [11:16] mzanetti: you wanted to get rid of the indicator-polishing branch, right? [11:16] sil2100: yes [11:16] 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] err, what? [11:16] * mzanetti checks again [11:16] mzanetti: if you check line 25 of the google spreadsheet, the branch is still there :o [11:17] ffs... [11:17] sil2100: I removed it from line 35 [11:17] sil2100: ok... kgunn and me really need to sync better [11:17] sorry [11:20] Ok, let me reconfigure then - but maybe you can reconfigure that as well now :) [11:21] Ok, I see you reconfigure now [11:21] * sil2100 goes to prepare lunch [11:22] sil2100: ok. seems to work === tvoss|lunch is now known as tvoss [11:23] brendand, dpkg-query operates on local Pakcage files iirc [11:23] sil2100, can I get a silo for line41? [11:24] ogra_, ok. i need to figure out why it's giving me the wrong result then [11:26] tvoss: allocating [11:26] Mirv, thank you [11:27] tvoss: fixing missing space in your MR:s [11:28] Mirv, oh sorry [11:37] cjwatson, hmm ... [11:37] doing something like: phablet-config -r 0000 --ppa ppa:ci-train-ppa-service/ubuntu-rtm/landing-004 [11:37] which simply calls add-apt-repository in the backend results in: [11:37] Command '"SUDO_ASKPASS=/tmp/sudo_askpass.B4Y7 sudo -A apt-get update -qq ; echo ADB_RC=\$?"' returned non-zero exit status 100 [11:37] 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:38] cjwatson, does add-apt-repository not automatically replace "utopic" with "14.09" here ? [11:38] (note, i'm not running rtm on the device) [11:38] ogra_: dunno, cyphermox did that work [11:39] ah, k, i'll wait for him then [11:40] sergiusens, i could need top approval of https://code.launchpad.net/~ogra/phablet-tools/phablet-tools-fix-writable-image/+merge/234946 [11:41] (fixes a lot of wrongs in the code :) ) === 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 [12:55] brendand: hey! [12:55] sil2100, hello [12:55] brendand: were you able to check rtm #37 for krillin? [13:02] brendand: wait [13:02] brendand: I mean utopic #37 [13:02] My brain is broken today [13:03] sergiusens, i cant use single quotes in the printf ... that breaks the format [13:04] sergiusens, the right combo of single and double quotes is essential in that line [13:04] * ogra_ fixes the other line === Ursinha is now known as Ursinha-afk === Ursinha-afk is now known as Ursinha [13:07] sil2100, yes it seems fine. no major issues [13:11] ogra_: hey, two questions! First, how's the mako dbus smoketesting issues? [13:11] sil2100, well, see above, other stuff kept me away from it [13:12] ogra_: ACK, second thing! What do you think about promotion for the utopic channel? ;) [13:12] sil2100, sure [13:13] sergiusens, i fixed what i could ... and commented [13:13] please approve :) [13:19] 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:21] brendand: I'm coming round to testing all my silos now [13:24] ogra_: so, we would need the mako's #243 promoted and krillin's #37 [13:24] Mirv: oh [13:29] brendand: hi, could we get qa approval for line 40? Thanks. [13:30] satoris, yes - it's just in our queue now [13:30] Great. [13:31] sil2100, done [13:31] ogra_: \o/ [13:31] popey: ^ [13:32] yay [13:32] * ogra_ wonders how to bribe sergiusens to top approve that super urgent phablet-config fix [13:32] mhall119: ^^ [13:32] (update yr phone) [13:33] popey: yay! [13:33] * mhall119 hugs sil2100 [13:34] tedg, hey i'm just looking at rtm silo 002. it was noticed that indicator-display is not seeded, is that known? [13:34] brendand, Yeah, it's on ogra_'s todo to review [13:35] brendand, But, the system settings function to turn it on doesn't work yet either. So it's not high priority :-) [13:35] brendand, tedg, did it land in rtm already ? i cant seed non-existing packages [13:35] ogra_, Yes, it has. [13:35] k [13:37] tedg, so indicator-display is just landing some translations fixes? [13:38] brendand, No, it's just enabling translation in the packaging. [13:38] No binary changes [13:38] tedg, so what change should we see after installing it? [13:39] brendand, You shouldn't. It should be changing bits in launchpad as they're introspecting those changes when building language packs. [13:39] brendand, So you won't see changes on the device for a while. [13:40] tedg, ok it would be helpful to us if that was pointed out in the description [13:40] tedg, 'indicator-datetime has no binary changes'. blah blah blah [13:40] Oh, you can't get to the descriptions in the MRs for RTM builds. [13:40] That's annoying. [13:41] Hmm, okay. [13:41] tedg, well we can go digging and figure it out, but it would be helpful if that were stated directly [13:41] tedg, just sayin [13:41] brendand, Just to be clear, indicator-display has no binary changes, indicator-datetime was changed. [13:41] tedg, yes the fix for datetime is clearly mentioned [13:42] brendand, K, just making sure :-) [13:42] Make sure we test the right thing. [13:43] tedg, anyway i'm on it now - shouldn't take long [13:43] sergiusens, so no top approval until i rewrite the printf line ? [13:43] brendand, Great, thanks! Any questions bug me or charles [13:43] ogra_: no, I'm just in a meeting [13:44] oh, sorry [13:44] * ogra_ hugs sergiusens [13:46] sil2100: can you please do me another reconfigure on silo 11? [13:47] mzanetti: sure [13:47] thanks [13:48] 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:53] 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:54] sil2100, can I get a silo for line 75? [13:54] brendand, Hmm, I guess. Probably need to update the wiki as well. [13:55] jhodapp: sure [13:56] brendand, There, updated from the branch. [13:57] sil2100: where are the krillin smoketest logs? === Guest35530 is now known as kgunn [14:16] sil2100, ^^^please :) [14:17] ogra_: sure :) === 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. [14:17] argh [14:17] ogra_: you really need to stop doing that :D [14:17] plars: can you test run this https://code.launchpad.net/~nskaggs/phablet-tools/remove-python2-support/+merge/233754 on the infra? === 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. [14:18] davmor2, tell that to our focus handling :P [14:19] 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:20] :P [14:20] 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:21] ogra_: well I thought the plan was to move to adt runners [14:21] fginther: plars thoughts on that? [14:21] sergiusens, smoketesting isnt using them today [14:21] thoughts lead to questions about roadmaps :) [14:21] and we have dbus issues since yesterday with phablet-test-run [14:22] which makes that re-write kind f urgent [14:22] 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] if we dont need it i'd be super happy though :) [14:23] cyphermox, well, i expllicitly tried to add a rtm ppa on an utopic image [14:23] not sure thats even remotely supposed to work :) [14:23] no, on a utopic image it would definitely pick utopic [14:23] good [14:23] then it works as excpected [14:24] i was just wondering if it shouldnt :) [14:24] I'll spend a bit more time thinking about it, later today [14:24] it really should be picking the series for the distro you're trying to add ;) [14:24] not urgent for sure [14:24] no, that's why I said later [14:24] though definitely interesting the other way round [14:24] i can imagine people wanting to add utopic ppas to rtm [14:28] trainguards line 73 please [14:28] you smarted out the bot :) [14:29] ogra_: the spreadsheets says we should ping manually [14:29] yeah, the bot only sendds a notice [14:29] which most IRC clisents dont highlight [14:29] you were still faster than the bot :) [14:36] * barry is slower than queuebot [14:36] sil2100: where's the bug for that? [14:36] I don't think you do have the FFe [14:36] or at least I'm not aware of it [14:37] barry, yeah, sergiusens could teach you though :) [14:37] :) [14:37] i'm trying to decide whether i need an ffe for this or not though [14:38] 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] i think not since there are no user-visible new features [14:38] sergiusens: last time we tried using adt for autopilot tests, it didn't work at all [14:38] sergiusens: we can revisit it though [14:39] plars, wihin this week? [14:39] ogra_: doubtful [14:39] else i need to fix phablet-test-run to run everything as root i guess [14:39] right, thats what i thought [14:39] ogra_: how is phablet-test-run broken? [14:39] plars, see all the dbus AP errors in the failed app tests [14:39] ladeedah, no silos available [14:39] ogra_: why do we want to run as root? Don't we want to run the apps as the actual user? [14:39] the mp for adt-run should be working now.. [14:40] 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] ogra_: no, running as root is the wrong thing [14:40] sergiusens, it runs as root today [14:41] ogra_: that's phablet-config [14:41] 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] except for the single test commands everything assumes a root shell [14:41] sergiusens, did you look at phablet-test-run ? [14:41] ogra_: I wrote it initially with mzanetti almost 2 years ago [14:41] it stops/starts unity as use and runs the actual test as user ... everything arounnd this needs root [14:42] ogra_: the start/stop unity isn't used afaik [14:42] it is in smoke tests [14:42] i see it in the logs [14:42] sergiusens, but even then ... 80% of the adb calls are as root [14:43] (i dont get why it doesnt use phablet-config to install packages and enable ppas though) [14:44] ogra_: because phablet-test-run is approx 1 year older than phablet-config [14:45] that ^ [14:46] mzanetti, sure [14:46] 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] yes, it does [14:46] and i will need to fix these up [14:47] if we want working test results [14:47] all run run_with_adb calls need root [14:47] right [14:47] run_with_ssh is user stuff [14:47] Thanks brendand! [14:47] ogra_: I thought you took care of those during the adb as user migration though [14:47] sergiusens, i did ... and it worked when i tested here [14:48] sergiusens, except for the chmod 666 ... for which balloons told me back then we could drop it anyway [14:48] (since AP ships a udev rule for that device) === nik90_ is now known as nik90 [14:49] sergiusens, it could well be that this extra "sudo -uphablet -i" wrapper everywhere causes the dbus issues though [14:49] havent checked that yet [14:52] jhodapp: let me take a look, I was in a meeting [14:52] Laney: for what? [14:52] sil2100, thanks [14:52] sil2100: the big FFe you mentioned on that hangout [14:56] 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:57] 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] asac might have more current info [14:58] 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] I just poked him, but I need to poke slangasek if he has any info regarding that [14:58] Damn, made a fool out of myself [14:58] ;) [15:01] pete-woods, i'm testing silo 18 === kgunn is now known as Guest81961 [15:07]