/srv/irclogs.ubuntu.com/2015/06/18/#ubuntu-ci-eng.txt

=== salem_ is now known as _salem
michialecu: ping01:05
michialecu: bug 146627301:50
ubot5bug 1466273 in Thumbnailer "gstreamer fails intermittently" [Undecided,New] https://launchpad.net/bugs/146627301:50
alecumichi: thanks! hope everything goes alright with your achilles heel02:05
michialecu: I shall live. Sorry, I posted this in the wrong channel by mistake.02:05
michiMeant to send it in unity-api.02:05
=== ljp is now known as lpotter
Mirvmorning04:54
sil2100jibel, davmor2: since I had my evening off yesterday, I probably missed the ping... but did the ubuntu images pass sanity?07:44
jibelsil2100, it did08:11
sil2100jibel: excellent, so I can promote those, right?08:12
sil2100jibel: QA tested mako, or any other platform too?08:12
davmor2sil2100: yes you missed the ping08:14
jibelsil2100, just mako08:14
davmor2sil2100: you also missed the cussing for there being a 164 when I went to test it too :P08:14
sil2100davmor2: oh, uh, then good I missed that I guess! I made 164 since people needed shell rotation back08:15
sil2100hm, I wonder how broken flo and manta are08:15
davmor2sil2100: yeah only I didn't know about it so install expecting 163 fortunately I notice the new icons and shell rotation so look into what was wrong08:17
davmor2sil2100: is manta even supported anymore?08:18
sil2100I mentioned the image number which was the OTA-4, so I thought you would install it explicitly :)08:18
sil2100Well, no one told us not to support it, we still build images for it08:18
sil2100The discussion never got resumed about deprecating it08:19
sil2100Maybe it's a good time to start that agin08:19
sil2100*again08:19
sil2100For now I'll promote mako, generic* and krillin08:22
sil2100Yeah... we anyway don't seem to have OTA-4 images for flo, manta and generic_x86 right now...08:34
sil2100Might need to think of a smart way to make them08:34
popeysil2100:  i added a line to the sheet for clock but I see no trello card, am I doing something wrong?08:37
seb128popey, are the system connected now?08:38
seb128like adding a line to the gdoc talks to trello?08:38
popeyi was told there was a bot08:38
seb128interesting ;-)08:38
ogra_seb128, since forever (for the "testing done" filed at least)08:38
seb128ogra_, only when QA is required no?08:39
ogra_the click package stuff not as long though08:39
ogra_seb128, yeah08:39
seb128k, right08:39
jibelpopey, the landing team has to assign a landing id.08:39
jibelI just did, it should appear on the board soon08:39
jibelpopey, well, when it'll be ready for QA08:40
popeyok.08:40
=== chihchun_afk is now known as chihchun
jibelpopey, if you tested it, mark it so and it should be enough to set it ready for qa08:41
Mirvsil2100: FYI QA now has a Qt security update in their hands... not sure if security updates to overlay contents have been discussed or not, but this relied on me noticing it (vivid proper was updated but it does not affect our images)08:44
jibelsil2100, https://wiki.ubuntu.com/LandingTeam/SupportedDevices doesn't mention flo or manta for stable channels08:46
sil2100jibel: ah, you're right08:46
sil2100jibel: we decided that we'll only do them for the devel channels08:47
sil2100My bad, forgot about that08:47
jibelMirv, security updates should follow the same process than any other security updates independently of the target archive/ppa08:48
Mirvjibel: what I meant as a general topic is that our security team does not currently consider overlay PPA at all08:48
Mirvso if we have a component there, it does not get security attention unless the component owner does so08:49
sil2100Mirv: who is working on the Qt security update?08:49
Mirvsil2100: ?08:50
Mirvsil2100: so as I just said I did this overlay update now ready for signoff08:50
Mirvsil2100: security team did it for normal vivid two weeks ago08:50
Mirvand I had a newer version in wily already08:50
sil2100Mirv: ah, ok, and the reason why we didn't get the security update 2 weeks ago is because we have our own Qt in the overlay08:51
sil2100Ok08:51
Mirvsil2100: so I just raised the topic that currently our security related things are not consolidated, as security team handles only stable Ubuntu releases but not phone if a package happens to be in overlay ppa08:51
Mirvsil2100: yeah, that :)08:52
sil2100Mirv: yeah, we know about it, and the reason it's like that because by principle anything that lands in the overlay should be backported as an SRU to vivid (if possible)08:52
Mirv then if we use SRU style versioning we run into the same thing as network-manager that overlay PPA package might be overwritten by normal SRU and losing overlay commits08:52
sil2100Well, that was the idea in the past, I know slangasek and olli are still re-discussing that08:53
Mirvsil2100: riiight, yes, I remember this now.08:53
Mirvsil2100: ok, let's see later. so now we have examples of Qt that needs manual intervention and network-manager that might suddenly lose features on phone images - two approaches08:53
sil2100For Qt the situation is anyway a bit harder, as it's not a touch-specific component08:53
Mirvsil2100: so is not network-manager, so that's another good example08:54
sil2100Yeah...08:54
Mirvand some of the indicators too08:54
Mirvand UITK08:55
Mirvand webbrowser08:55
Mirvthese latter examples are such that have actual bumped upstream versions so they are essentially same as Qt08:55
jibelsil2100, did anything change recently with the spreadsheet, the bot cannot parse the click tab anymore09:42
jibel?09:42
sil2100jibel: hm, nothing from our side at least09:46
sil2100jibel: we did notice that the spreadsheets have changed, so maybe they modified the API somehow?09:46
jibelsil2100, I'm just doing a get passing the id of the tab in the url but now it always returns the 1st tab09:48
sil2100You doing it as CSV or using the API for that?09:50
=== chihchun is now known as chihchun_afk
jibelsil2100, CSV, the format of the url has changed09:51
jibelfixing no09:51
jibelw09:51
sil2100Ok, probably due to the switch to the new API09:52
sil2100Actually, I don't want to jinx it, but maybe they fixed the race conditions in the new version and we won't have things broken anymore09:53
jibelfixed09:57
jibelpopey, https://trello.com/c/kTTrqblZ/1802-click-clock-app-popey09:57
popeythanks jibel09:58
jibelMirv, silo 10 is the security fix you mentioned earlier?10:07
sil2100\O/10:15
sil2100(big head since cat enabled capslock)10:15
Mirvjibel: yes10:15
sil2100wgrant, cjwatson: hey guys! I wanted to poke about the overlay translation updates - are all the bits in LP ready for that? Or is that still just a proposition and you're waiting for us?11:09
cjwatsonIt's a proposition.11:09
cjwatsonIf you folks sign off on it, we can implement it, but we weren't going to do it entirely on our own initiative.11:10
sil2100cjwatson: ok, need to move this forward then, but I suppose it might be linked with the big derived-vs-overlay discussion11:18
cjwatsonsil2100: That's what I'd feared.  I think it would be a bad outcome to get stalled in some kind of perfect-is-the-enemy-of-the-good thing11:20
cjwatsonsil2100: Just don't quite have the bandwidth to be able to give a day to something that might not be used11:21
sil2100At best I would like to have this implemented without waiting on the decision, but if by any chance we decide that its best to switch to a derived distro (uh) then it owuld be wasted work11:21
sil2100Yeah11:21
sil2100cjwatson: ok then, we'll be in touch regarding that11:22
cjwatsonWe can do it if there's a decent chance11:23
=== _salem is now known as salem_
=== MacSlow is now known as MacSlow|lunch
* sil2100 off to lunch12:02
=== alan_g is now known as alan_g|lunch
dobeyif cmake-extras is installed, why would EnableCoverageReport not be found?!12:24
=== MacSlow|lunch is now known as MacSlow
pete-woodstrainguards: I don't like to nag, but if someone could push the land button on silo 43 it would probably help out quite a few people :) (anyone using cmake-extras)12:51
pete-woodsI may or may not be one of these folks ;)12:52
Mirvpete-woods: ok :)12:58
Mirvpete-woods: unapproved! https://code.launchpad.net/~pete-woods/cmake-extras/cmake-3.2/+merge/262315 ..12:59
Mirv(just top-approve if you have the authority)12:59
pete-woodsMirv: done!12:59
Mirvpete-woods: ok, it's publishing now and I acked the packaging changes13:00
pete-woodsMirv: awesome, thanks!13:01
=== mvo_ is now known as mvo
=== alan_g|lunch is now known as alan_g
barryrobru: after cjwatson diagnosed the problem, i wonder if there isn't a quick hack/fix to add an option to include -sa when building the package.  adding .bzr-builddeb doesn't work13:53
barry(i can see that the orig.tar.gz isn't included in the upload to the ppa)13:53
barryrobru: or hmm... maybe i can just upload to the ppa manually and then let the train take it all from there?13:54
=== mvo_ is now known as mvo
kalikianajibel: ping https://code.launchpad.net/~bzoltan/ubuntu-ui-toolkit/landing_12-6-2015/+merge/261831 the branch is top-approved now14:05
kalikiana(and tested)14:05
kalikiana(for silo 011)14:05
dbarth_o/ silo 001 ready to publish for wily; though some qa smoke testing might be good for the dual/vivid task as well14:12
sil2100o/14:13
jibelkalikiana, thanks14:17
* jibel adds a task to update the bot to update existing cards.14:17
robrubarry: yeah you can always upload manually. I have no idea why your package alone requires -sa, literally no other train packages require this and it's never been a problem before.15:12
barryrobru: just trying to keep things interesting for you15:14
barryrobru: i'll upload manually for now15:16
robrubarry: would -sa option be a nop for packages that are already including orig.tar in the upload anyway? I'd be open to adding that to all builds if it a) helped you and b) was harmless for everybody else. I don't like the idea of adding extra options to the build, too many checkboxes already15:18
barryrobru: i think it would be harmless.  i'm not actually sure how it's currently automagically including the orig.tar.gz or not, or why my branch isn't getting it15:19
robrubarry: my only guess is that it's because you have a debian/watch. nobody else does. without it, dput must think "there's no way to find the orig.tar so I have to include it" but with yours dput is like "oh yeah lp can find the orig.tar on it's own"15:20
barryrobru: hmm...  but that would be an easy theory to test.  what's the command used to create the source package from the mp?15:22
robrubarry: "bzr bd -S"15:24
robrubarry: makes me wonder why lp doesn't just use your debian/watch to find the orig.tar15:26
barryrobru: that is very strange, because it's the same command i use here, and my _sources.changes *does* have an orig.tar.gz.  otoh, i'm doing it from the branch that already has the merge.  let me try to branch the packaging branch and then merge in my mp, and do that command15:27
sil21007~~~~~~~~~~~4~15:46
ogra_sil2100, meow ?16:00
oSoMoNtrainguards: can I have a silo for line 68, please?16:00
popeylanding meeting?16:01
barryrobru, slangasek yeah, so my source package building test worked just fine here locally.  in a completely different, fresh directory i did the following:16:14
barrybzr branch lp:~ubuntu-managed-branches/ubuntu-system-image/system-image16:14
barrycd system-image/16:14
barrybzr merge lp:~barry/ubuntu-system-image/citrain30116:14
barrybzr bd -S16:14
barrythen...16:15
barry% grep orig ../system-image_3.0.1-0ubuntu1_source.changes16:15
barry 4fa47281da9e6f313c2ab27c83e129bf4ae747fe 203601 system-image_3.0.1.orig.tar.gz16:15
barry 3a0d73bd37a67bcc1606d4f082d2072af5555f8413c36e56ad4b50f3b9ddb277 203601 system-image_3.0.1.orig.tar.gz16:15
barry 447b49e1a59655a16451bc98543190f8 203601 python optional system-image_3.0.1.orig.tar.gz16:15
barry 16:15
barryso yeah, i have no clue why the train is unhappy16:15
barryfor now, i am just going to upload manually to the ppa16:15
barryrobru: in the meantime, please remove the line that says "if user == 'barry': raise MysteriousFailureError"16:16
robrubarry: no i like that one!16:17
barryrobru: somehow that got past my code review16:17
robrubarry: I'll push a branch that adds -sa and see what happens16:17
barryrobru: should i wait so we can test that out for reals?16:17
robrubarry: sure one sec16:18
robrubarry: ok started a build16:20
barrycool16:20
cjwatsonrobru: debian/watch is totally irrelevant for this16:20
robrucjwatson: ok well I added -sa so we'll see if that works16:21
cjwatson-sa is implied if the previous version in the changelog had a different upstream, iirc16:21
robrucjwatson: but it's bizarre to me that all other packages work without -sa, I'm not sure how barry's differs16:21
barryrobru: oops16:21
barryrobru: you need to put the -sa after the --16:21
robrubarry: cjwatson but 3.0.1 differs from 3.0?16:21
cjwatsonit does16:22
barrycjwatson: yep, and locally no -sa is needed16:22
barryhang on16:22
barryrobru: this doesn't look right:16:23
barryCreating source with bzr bd -S -sa -- -k55EFCAA8 -d -v3.0-0ubuntu216:23
barryfirst, the -sa needs to go after --16:23
barrybut -v3.0-0ubuntu2?16:23
barrythat's probably breaking debuild16:23
barrywhere's that coming from?16:23
robrubarry: no the -v tag means "include the diff since this version", that's the version currently in archive16:23
robrubarry: fixed the -sa though16:23
barryrobru: oh right of course16:24
barrybingo16:24
barry  Uploading system-image_3.0.1.orig.tar.gz: done.16:24
barry 16:24
barrynfc why that's not autodetected, but it looks like that might have solved the problem16:25
robrubarry: cool. I'll keep an eye on that to make sure nobody else gets broken by this change ;-)16:25
barry:)16:25
jhodapprobru, can you please reconfigure silo 1916:25
cjwatsonusing -sa all the time is inefficient but shouldn't actually be harmful16:25
barryrobru: it could be a problem if someone uploads a change that only affects the packaging version, but given that most are not split like s-i i bet that won't happen in practice like it does in the distro16:26
robrubarry: yeah in the train we auto-generate new upstream versions16:26
cjwatsonbarry: that just means the orig is reuploaded unnecessarily, but LP won't care as long as it's identical16:26
barrycjwatson: ah, cool.  thanks guys!  i'm off to get some lunch but will check back in a few16:27
cjwatsonit's in https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/landing-040/+packages and building16:28
cjwatsonbut you can see there why there was a problem!16:28
cjwatsonrobru: look at https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/landing-040/+packages and expand the expander16:28
cjwatsonobserve the duplicate changelog version16:28
robrucjwatson: oh... k...16:29
jhodapprobru, thanks16:29
robrucjwatson: I blame barry16:29
robrujhodapp: you're welcome16:29
cjwatsonthat confuses dpkg-buildpackage into thinking it doesn't need to reupload the .orig because look there was a "previous" version already16:29
cjwatson(dpkg-genchanges, strictly)16:30
robrucjwatson: other than looking retarded, is that going to cause any further problems?16:30
cjwatsonshouldn't16:30
robrualright16:30
kenvandinedavmor2, how do i need to bug to get silo 4 verified by the eod of the week?16:30
kenvandines/eod/end/16:31
davmor2kenvandine: not me I'm off tomorrow :)  ↑ jibel16:31
davmor2kenvandine: it is however only 3rd in the queue so will most likely be done tomorrow anyway16:32
kenvandinedavmor2, i was told at one point that they aren't verified in that order16:32
kenvandineit was first in the queue a few days ago :)16:33
davmor2kenvandine: yeah if something critical lands then everything else gets bumped one.  but on the whole that is the order we pull from16:33
kenvandineok16:34
camakoricmm, you asked in the past to get a heads up on MPs for papi. This has lingered for a while now : https://code.launchpad.net/~mir-team/platform-api/delete-deprecations/+merge/254170 and it's blocking slangasek. Can you please take a look? We'll be landing it soon.16:40
camakoracarr, AlbertA ^16:41
=== alan_g is now known as alan_g|EOD
barrycjwatson: well, wtf.  yay.  course, it was happening even before that17:20
barrywell, wait a sec17:20
barrymy branch doesn't have that17:20
barryand neither apparently does: https://code.launchpad.net/~barry/ubuntu-system-image/citrain301/+merge/26225517:21
barrycjwatson, robru ^^17:21
* barry blames robru's == 'barry' line17:22
robrubarry: yeah the train adds that because reasons. ideally you'd allow the train to generate versions for you and it would all be one big entry but because you force your own versions this happens17:22
barrysweet17:22
* barry thinks of more ways to break robru's software17:23
=== mhall119_ is now known as mhall119
=== alex-abreu is now known as alex-abreu|afk
robrucyphermox: happy birthday! ;-)18:32
barrysil2100: why can't i find s-i 2.5.1 in the dashboard?18:39
barryor robru ^^?18:41
robrubarry: 2.5.1? hm?18:42
barryrobru: row 5518:42
robrubarry: because it was published: https://ci-train.ubuntu.com/job/ubuntu-landing-028-2-publish/42/console18:43
robrubarry: and it's in the PPA: https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/stable-phone-overlay/+packages?field.name_filter=system-image&field.status_filter=published&field.series_filter=18:44
barryrobru: then why row 55, column P?18:44
barryis that just leftover artifact or is there Something To Do?18:44
robrubarry: seems stale. spreadsheet rubbish18:45
robrubarry: here's something to do: https://code.launchpad.net/~barry/ubuntu-system-image/citrain301/+merge/262255 please top approve18:45
barryrobru: cool.  have you killed the spreadsheet yet?18:45
robruit's close, barry. so close.18:45
barry:)18:45
barrydone18:45
robrubarry: if you wanna help with spreadsheet, visit http://requests.ci-train.staging.ubuntu.com/ sign in, create a request, fiddle around, and file bugs against https://bugs.launchpad.net/bileto18:46
barryrobru: will do.  can you actually build and publish packages there currently?18:48
robrubarry: not publish, no18:49
robrubarry: it's a staging instance, it can build packages in ~ci-train-staging-area ppas, which are virtualized, and thus not eligible for publishing.18:49
barryrobru: ack18:49
* barry thinks that's exactly where system-image belongs18:53
barryrobru: one nice thing about bileto - i won't have to have a dual monitor spanning chrome tab open to interact with it18:54
robrubarry: yes one of the primary goals is to be more streamlined than a spreadsheet.18:57
robrubarry: it's not perfect, but it's an order of magnitude better18:57
robrubrb18:57
charlestrainguards, can I get a silo for row 6919:20
charlesqueuebot, indeed19:21
cjwatsonrobru: virtualised> soooo close19:21
cjwatsonvirt ppc64el builds almost work and we have hardware installed for ARM ...19:22
robrucharles: need MPs, not branches19:23
robrucjwatson: sweeeeet19:23
charlesd'oh19:23
robrucjwatson: still though even with that I'm not sure publishing from ci-train-staging-area is a good idea. the creds are less credible.19:23
charlesrobru, corrected19:24
robrucharles: thanks19:24
robrucharles: silo 2219:24
cjwatsonrobru: no quite19:25
cjwatsonwill make ephemeral PPAs simple though19:25
cjwatson(ish)19:25
robrucjwatson: right, that'll be nice19:25
charlesrobru, thanks19:25
robrucharles: you're welcome19:25
=== alex-abreu|afk is now known as alex-abreu
=== salem_ is now known as _salem

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