/srv/irclogs.ubuntu.com/2015/10/28/#ubuntu-ci-eng.txt

=== _salem is now known as salem_
=== salem_ is now known as _salem
=== vrruiz_ is now known as rvr
ogra_sil2100, where do i commitr system image config changes, i see the etc/ dir is a bzr branch, is it only there or is there a matching external one10:12
=== greyback__ is now known as greyback
sil2100ogra_: commit your changes and just do a bzr push10:30
ogra_sil2100, so no external LP branch then, ok10:32
morphissil2100: can you drop the wily packages from silo 52 for me?10:34
ogra_sil2100, eeek, there are uncommited changes10:34
sil2100ogra_: yeah, temporary stuff usually is left uncommited10:37
morphissil2100: or is that anymore true that we should lands things also for wily?10:37
sil2100But I suppose you can just commit that10:37
sil2100morphis: on it, removing10:37
ogra_sil2100, well, how do i commit my stuff then ?10:37
morphissil2100: thanks!10:37
ogra_ah ok10:37
sil2100xenial should be the focus now10:37
sil2100ogra_: those 'temporary changes' are usually changes that have no relevence if they're reverted or not ;)10:38
sil2100At least that's the overall idea10:38
ogra_sil2100, well the prob is that you have no history if every now and then youor log has "committed uncommited changes that someone did"10:39
ogra_i.e. you dont now what they were for10:39
sil2100morphis: deleted10:39
sil2100ogra_: yeah, I guess we'll have to make sure to keep the config clean next time10:39
sil2100hm, I think I might have had some uncommited changes there10:39
ogra_or just commit the temporaray stuff too ...then you have proper explanation ...10:40
sil2100But those, as I said, were irrelevant things - will commit them next time10:40
sil2100Sorry about that ;)10:40
ogra_no prob10:40
Saviqsil2100, is there a xenial channel yet?10:55
jibelSaviq, it'll be devel-proposed once there is a build for it which sil2100 said he would work on today11:00
Saviqjibel, ack, thanks11:01
sil2100Yeah, actually my expectation was it would switch automatically, but seems it needs a bit manual love11:01
sil2100On my plate today11:01
rvrjgdx: ping11:05
pstolowskitrainguards, hey, what's happening to silo 24? it's been in a weird state for 2-3 days11:44
sil2100Let me take a look11:46
sil2100hm, looks like it got rejected11:47
=== _salem is now known as salem_
jgdxrvr, ponmg12:36
rvrjgdx: Hey12:36
rvrjgdx: Silo 47. How do I check the OTA version in ubuntu-system-settings?12:37
rvrjgdx: I see no info in the About page12:37
jgdxrvr, i think you have to have ota8, but I'm not sure. sil2100 ^ do you know?12:37
rvrjgdx: Phone was flashed in rc-proposed channel12:38
jgdxrvr, if not, let me know if you want me to create a mock for this.12:38
jgdxcommand line that will give you an ota12:38
rvrjgdx: How did you test the feature?12:39
jgdxrvr, using a mock12:39
jgdxsil2100, if you flash ota7 now, will you get a tag=ota7 in version_details? You know?12:40
rvrversion_detail: ubuntu=20151028,device=20150911,custom=20151028,version=27412:40
sil2100jgdx: no, we won't be supporting tags from the past12:52
sil2100jgdx: since this requires an image to be re-created (with the new tag)12:52
jgdxsil2100, yup. Thanks!12:52
jgdxrvr, so impossible to test w/o trickery12:52
sil2100If you need it for testing, I can prepare a private system-image server12:53
sil2100I mean, hm, I would need to find a public place first12:53
jgdxsil2100, I was going to suggest a mocked s-i, but whatever's fine with me.12:53
sil2100jgdx, rvr: let me quickly take a look if I can export my s-i somewhere, you guys could then flash from there for testing12:54
rvrI tried to add the tag to the file, but no luck12:54
jgdxsil2100, thanks12:54
=== alan_g is now known as alan_g|lunch
=== chihchun_afk is now known as chihchun
pstolowskisil2100, rejected? why?13:21
pstolowskisil2100, also.. are the builders overloaded atm or something? i'm rebuild the mediascanner scope in silo 4 and it has been over 2,5 hrs already13:25
cjwatsonpstolowski: arm64 still has a day or so left of a test rebuild13:26
cjwatsonpstolowski: I've scored up your builds, though they were pretty close already13:26
pstolowskicjwatson, ah i see13:27
pstolowskisil2100, anything i can do about silo 24? it would be great to land it as i have another one in the pipeline :)13:33
sil2100pstolowski: let me take care of it now, had a busy afternoon today ;/13:35
sil2100pstolowski: ah, I think I see why it failed!13:37
sil2100Let me publish it by hand13:37
pstolowskiuhm13:37
pmcgowanmterry, Saviq silo21 still dirty :(13:38
mterrypmcgowan, looks like it got some builds in last night though -- you can try it and it should have the shutdown dialog fix13:39
pmcgowanmterry, I was thinking, are we doing too little in powerd and too much in unity? seems it should be simple to have two distinct events13:40
mterrypmcgowan, yeah it might make more sense to have USC/powerd emit a "long power press" signal that u8 consumes13:42
mterryI think there are still intentions to review that whole power signaling mess13:42
pmcgowanmterry, exactly, insteaad now there are multiple button pressed and released signals13:43
pmcgowanshould just be short and long13:43
pmcgowanthen filter multiple presses that are too frequent13:43
dobeyrvr: ping13:49
rvrdobey: pong13:58
dobeyrvr: so, the issue with the online-accounts is very improbable to be hit by pay-ui in real production devices, as once it lands in the store, and new images get built, the new version will already be installed, and anyway installing it as an update through system-settings will likely not have done an immediate --wipe flash of their device after the new click was built, as the click which will land in the store will be built b14:05
dobeyrvr: so i'm wondering if we could please move forward with the landings14:05
rvrdobey: As I commented on the trello card, the plan is to land silo 1 first, then pay-ui, and then 26.14:09
rvrmardy is already working on it, so seems it won't take too long14:10
=== alan_g|lunch is now known as alan_g
Saviqpmcgowan, it's dirty because there were new commits, but it's sane14:13
pmcgowanSaviq, ack14:14
mardycjwatson: is it possible to score up https://ci-train.ubuntu.com/job/ubuntu-landing-056-1-build/33/console and https://ci-train.ubuntu.com/job/ubuntu-landing-001-1-build/231/console too?14:46
cjwatsonmardy: the first is already building; done the second14:48
alan_gcihelp: we're seeing consistent problems on krillin in CI (https://jenkins.qa.ubuntu.com/job/mir-mediumtests-runner-touch/) that we're having trouble reproducing locally. What image is installed on these runners?14:49
fgintheralan_g, one moment14:50
oSoMoNtrainguards: is https://requests.ci-train.ubuntu.com/#/ticket/576 correctly configured so that vivid packages will land in the overlay PPA and xenial packages in the archive?14:51
fgintheralan_g, ubuntu-touch/rc-proposed/ubuntu14:53
fgintheralan_g, which currently appears to be image 31214:53
alan_gfginther: thanks.14:54
rvrxavigarcia: ping14:55
mardycjwatson: thanks a lot!14:55
sil2100oSoMoN: looking14:56
sil2100oSoMoN: looks ok, although I'm not sure if the Destination PPA shouldn't be empty14:58
sil2100oSoMoN: since dual silos anyway by default land the vivid part to the overlay14:58
sil2100But I need to check the code as robru made a lot of modifications in the past few weeks ;)14:58
oSoMoNyeah, that’s what I’m unsure about too14:58
xavigarciarvr pong15:03
rvrxavigarcia: Hi15:04
rvrxavigarcia: I'm taking a look at silo 46, I think it contains new strings, right?15:05
rvrxavigarcia: I plugged my headphones and a notification appeared with "Headphones" as title15:05
xavigarciarvr: yeah, it has new string to show the output device15:05
rvrxavigarcia: Ok. I think we are blocking it, we are in string freeze.15:06
xavigarciarvr: ok :(15:06
rvrsil2100: Any news about the s-i for OTA tag?15:12
sil2100rvr: I'm testing my local s-i on my krillin right now to see if it flashes/works correctly from the local server15:13
rvrsil2100: Ack15:14
sil2100rvr: once that's done I'll export it for you to test for a moment15:14
rvrsil2100: Cool, thanks15:14
barryrobru: ping15:16
sil2100rvr: how can I get my krillin into recovery?15:17
rvrsil2100: Power + Volume Up for 20 seconds or so15:17
rvrWhen you see a red light, release Power15:18
rvrWith Up select Recovery and with Volume Down enter that option15:18
sil2100rvr: thanks!15:18
sil2100I suspect the GPG keys might be causing problems15:19
sil2100Ah, I think I see what's broken15:33
dbarth__uh, sorry if that a faq, but how do i transition an old silo to xenial?15:36
dbarth__https://requests.ci-train.ubuntu.com/#/ticket/44615:36
dbarth__hi trainguards ^^15:36
sil2100dbarth__: hey!15:49
dbarth__sil2100: hi15:52
sil2100dbarth__: so first of all, you need to switch the silo type to xenial+vivid15:52
sil2100dbarth__: so edit the 'Target Series' field and save15:53
sil2100Next, rebuild your silo15:53
sil2100I'll remove the wily binaries from the PPA15:53
sil2100rvr: still investigating, my phone doesn't want to properly update to any image from my local s-i instance15:53
rvrsil2100: Ack15:55
sil2100(unrelated to the tag= syntax, simply issues with the local server or my flashing mechanisms)15:55
sil2100dbarth__: should I remove the wily binaries now? :)15:56
dbarth__sil2100: ok done15:57
=== chihchun is now known as chihchun_afk
sil2100dbarth__: packages removed16:09
dbarth__sil2100: thank you16:10
jgdxsil2100, any progress on the private s-i-s for rvr ?16:15
jgdxanything I can do before I eod?16:15
pete-woodstrainguards: hi folks, we have a new project we want to get into xenial, and indeed vivid overlay (https://launchpad.net/gmenuharness), I can't remember the process to do it, though :$ (been a while)16:15
pete-woodsname of the right person to ping is all I'm after, really :)16:27
rvrmardy: dobey: Great, pay-ui can be installed with silo 1 :))16:39
rvrand it shows the Ubuntu One login screen in the Store16:40
dobeyrvr: good16:43
Saviqrobru, any reason for the silo branches having $source-name duplicated? like ~ci-train-bot/unity8/unity8-...?16:55
rvrmardy: jgdx: There is a problem in the online accounts page in System Settings16:57
rvrmardy: jgdx: When editing an account (Google, Facebook), there is no bar with button to go back16:57
rvrjgdx: Maybe related to components 1.3 migration16:58
jgdxrvr, hm, right.. mardy, was online accounts migrated to 1.3?17:19
rvrhttps://bugs.launchpad.net/canonical-devices-system-image/+bug/151105517:20
ubot5Launchpad bug 1511055 in Online Accounts setup for Ubuntu Touch "Cannot go back to Accounts" [Undecided,New]17:20
sil2100rvr: ok, I flashed my device from my local s-i instance and confirmed that the tag= field is there17:26
sil2100rvr: how long will you work today still?17:26
rvrsil2100: I will finish current silo17:27
rvrsil2100: Can we check it tomorrow morning?17:27
sil2100rvr: sure, all is ready, I'll just open it up for you tomorrow then17:28
rvrThanks17:28
sil2100rvr: but please note you'll need to flash with a script for disabling GPG verification and the adb unlocking recovery...17:28
rvrsil2100: Ok17:28
rvr--run-script=/home/vrruiz/bin/disable-gpg.sh17:29
sil2100hah ;)17:29
rvrI'm already disabling gpg17:29
sil2100Prepared already I see!17:29
robrubarry: pong17:43
robruSaviq: yes, because the first one is the project name and the second one is the source package name, they're not always the same (mostly it's so gles twins don't clobber the main trunks but there's other cases too)17:44
robrusil2100: it's always been the case that vivid goes to overlay and dest ppa controls only where primary series goes (wily or xenial).17:46
Saviqrobru, ack17:46
robrudbarth__: rebuild wasn't necessary, especially since you already had qa approval, sorry sil2100 gave bad info.17:51
robruMy email announcing xenial switch explained the correct steps17:52
robruToo late to go back now though!17:52
dbarth__robru: uh, my bad; it had the info, i just forgot17:52
dbarth__note for others: read robru's email !17:53
robruYes!17:53
=== alan_g is now known as alan_g|eod
sil2100Sometimes I copy existing wily binaries to xential, yes18:05
sil2100But I usually noticed that people anyway want to rebuild since there are toolchain changes etc.18:05
sil2100So sorry if you had to rebuild without any reason18:05
robrusil2100: oh did the toolchain stuff land already? I didn't think it had yet.18:21
robrubarry: with regards to deadlocking, how likely is it for python to exit without closing it's fd's? In my (limited) testing i wasn't able to see a deadlock, it always freed the lock on process exit18:23
barryrobru: well, i'd have to go to my stevens to remind myself of generic process semantics, but python itself won't do anything special.  you can always set an atexit handler to explicitly close fds (or do whatever), but of course those handlers running are subject to the usual caveats, e.g. os._exit() or kill -9 won't run them18:25
robrubarry: Hmmmmmmm OK I'll need to test further in order to decide if it's worth switching.18:27
robrubarry: never heard of os._exit but sys.exit gets called in some cases. Most common signal will be 15 of user cancelled job in Jenkins18:28
barryrobru: sys.exit is fine; it raises an exception and if uncaught, it does a normal shutdown procedure.  same with catchable signals18:30
barryrobru: https://docs.python.org/3/library/os.html#os._exit18:30
robrubarry: OK great, the signal handler i have for signal 15 calls sys.exit18:31
sil2100Xenial is now open for development, so the most important stuff is in18:31
rvrmardy: Silo 1 approved18:54
dobeyhmm19:13
robrubarry: what's 'stevens'?19:24
barryrobru: http://tinyurl.com/ndcpys3  - it's the definitive book on unix semantics19:25
robrubarry: oooh19:26
barryevery *nix developer should own a copy (i really need to get the 3rd edition)19:26
barryw. richard stevens died a long while ago, but i believe it's been updated by other pepole and is still relevant19:26
robrubarry: yikes, $30 even on kindle19:26
barryhe also has books on network programming which are really excellent too19:27
robrubarry: must be printed with golden electrons19:27
barryyeah, but worth every penny :)19:27
robrubarry: maybe I'll pick it up then ;-)19:27
barrylooks liked 3rd ed even covers "ubuntu 12.04 (based on linux 3.2)"19:28
sil2100robru: uh: https://ci-train.ubuntu.com/job/ubuntu-landing-001-2-publish/123/console <- you saw anything like this before?19:33
robrusil2100: yeah, the silo is locked19:34
robrusil2100: that should only happen if you try to run two jobs simultaneously.19:34
sil2100The silo was ready for publish and built, maybe some lock that's left-over?19:35
robrusil2100: not sure, the train has code to check that locks are stale but if you feel it's in error you can delete ~/silos/$SILONAME/lock19:35
sil2100https://requests.ci-train.ubuntu.com/#/ticket/573 <- the silo looks fine here19:35
robrusil2100: that is weird19:36
robrusil2100: I'm working on a refactor of locking that should hopefully eliminate these issues19:36
sil2100robru: ok, thanks :)19:38
robrubarry: I have a question about generators19:47
barryrobru: sure19:47
renaturobru, do you know what is the problem with this silo? https://ci-train.ubuntu.com/job/ubuntu-landing-002-1-build/241/console19:48
robrubarry: let's say I'm iterating on a generator and it takes a long time (~5 minutes total).19:48
renatuI am getting silo locked too, on silo 219:48
robrubarry: if the generator instantiates an object for each iteration, what's the lifecycle of the object? is the object freed when the next iteration starts? no reference to the objects is kept as far as I know19:48
robrurenatu: looking19:49
barryrobru: the generator is a closure so if the object is referenced by a local variable in the generator, that reference will keep the object alive.  from there, it's normal python lifecycle semantics.  if you reassign or del the name binding and the object's refcount goes to zero, it gets freed.  if the object participates in a cycle, the cyclic garbage collector will eventually break the cycle, subject to those semantics (e.g. based on19:51
barrythe presence of __del__'s)19:51
robrurenatu: sil2100: oh apparently i goofed up and rolled out a half-finished merge to production, awesome.19:52
robruwill fix that asap19:52
robrubarry: so like this: http://bazaar.launchpad.net/~cupstream2distro-maintainers/cupstream2distro/trunk/view/head:/cupstream2distro/silomanager.py#L163 it just yeilds the object without assigning it, those'll be freed after each iteration right? or does python hang onto them until the full generation is complete?19:53
robrurenatu: sil2100: ok trunk is fixed, should hit production in ~20 minutes19:59
barryrobru: here's a good boiled down example which should explain what's going on: http://paste.ubuntu.com/12992694/20:02
barrybut yes, they should get freed20:02
barryrobru: er, try this instead: http://paste.ubuntu.com/12992722/20:04
robrubarry: seems to work even without the del at the end20:07
barryrobru: right, the del is just for printing a message when the object is collected20:07
barryrobru: if the del created a cycle, then you'd have problems :)20:07
barry*or resurrected the object20:07
barry__del__'s can be evil :)20:07
robrubarry: but I mean I dropped the del and it still prints 'freed' in each iteration instead of all at the nd20:08
barryrobru: impossible :)20:08
barrywell, if you replace the def with a pass20:08
robrubarry: what?20:08
barryrobru: what does "dropped the del" mean?20:09
barryoh, you mean `del obj`20:09
barryrobru: that makes sense20:09
robrubarry: http://paste.ubuntu.com/12992782/20:09
robruyeah20:09
robrubarry: that's great though, that answers my question and it's the behavior I wanted ;-)20:10
barryright?  "obj" gets rebound to a new Foo instance, and rc() returns so both refcounts are dropped, and the object's refcount goes to 0 so it gets del'd.  at the end of the for loop, obj gets unbound20:10
barrys/end of the for loop/script exits/20:10
robrubarry: right20:11
robrubarry: this means that the generator loop should only lock each object for 5 seconds each instead of locking them all for the full 5 minutes, so that's perfect ;-)20:12
barrycool :)20:12
robrusil2100: renatu: ok try again?20:21
=== salem_ is now known as _salem
robruSaviq: oh btw, not sure if you noticed yet, but "bug NNN" and "# NNN" are both linkified now. You don't need to use pad.lv anymore if you don't want to21:00
Saviqrobru, oh good, thanks21:00
robruSaviq: you're welcome!21:01
=== greyback__ is now known as greyback

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