| kenvandine | robru, i don't think so | 00:41 |
|---|---|---|
| kenvandine | been there for quite a while | 00:41 |
| robru | kenvandine: Hmmmmmmm, train only found them in the xenial package | 00:44 |
| kenvandine | ok, so maybe in the overlay ppa then and not vivid proper | 00:44 |
| robru | kenvandine: did the format change at some point? Train looks for debian/tests/control | 00:45 |
| kenvandine | i thought they landed before vivid was released, but likely wrong | 00:45 |
| kenvandine | nope, we use debian/tests/control | 00:45 |
| robru | kenvandine: but the package I'm testing i grabbed from overlay ;-) | 00:45 |
| kenvandine | that has to have them | 00:45 |
| kenvandine | there is no delta between overlay and xenial | 00:46 |
| === _salem is now known as salem_ | ||
| === salem_ is now known as _salem | ||
| robru | kenvandine: can you comment on this failure? Is this expected? https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-xenial-ci-train-ppa-service-landing-019/xenial/armhf/c/content-hub/20151110_225444@/log.gz | 02:35 |
| === chihchun_afk is now known as chihchun | ||
| === chihchun is now known as chihchun_afk | ||
| === chihchun_afk is now known as chihchun | ||
| tvoss | good morning | 08:25 |
| tvoss | robru, you happen to be around? | 08:25 |
| tvoss | robru, if so: what is our policy on landing to wily and xenial? | 08:25 |
| tvoss | robru, do we have a combined target for both available? | 08:25 |
| === alan_g is now known as alan_g|afk | ||
| sil2100 | davmor2, rvr: any news on silo 0? | 09:32 |
| sil2100 | Since jibel mentioned that was the last one needed to land | 09:32 |
| sil2100 | I don't see it even ready for QA.. | 09:32 |
| davmor2 | sil2100: you have as much info as I do, I just logged in ;) | 09:33 |
| rvr | davmor2: sil2100: Nope, I have no news | 09:59 |
| morphis | sil2100: ping | 10:03 |
| Mirv | OTA-8 candidate bug #1514173 <- jibel, sil2100 | 10:04 |
| ubot5 | bug 1514173 in ubuntu-ui-toolkit (Ubuntu) "[listitemlayout] setting RichText format with html image tag leads to crash (segfault)" [Critical,In progress] https://launchpad.net/bugs/1514173 | 10:04 |
| jibel | Mirv, k, do you have an example of this crash with an application? | 10:10 |
| Mirv | jibel: I'm asking faenil to show up and tell about it | 10:16 |
| Mirv | or zsombi can fill in if faenil doesn't appear | 10:17 |
| faenil | Mirv: here I am | 10:18 |
| zsombi | Mirv: I only know that there is a crash, not sure whether the app has been released with it or not. I assume if this is a blocker, it means the release or RC doesn't have the changes which would cause the crash | 10:18 |
| zsombi | faenil: ^ do you have an answer to this? | 10:18 |
| zsombi | faenil: Mirv: if the app changes are not in RC, then it's not OTA-8 | 10:18 |
| faenil | jibel: re the bug, it's a bug I found while playing with the component, apps are in the process of transitioning to ListItemLayout, so I can't say if any of them causes the crash | 10:18 |
| zsombi | jibel: faenil: then it smells to me it's OTA-9 | 10:19 |
| faenil | if there's any app released in OTA8 that uses ListItemLayout with an <img> tag in the text, it's going to crash | 10:19 |
| faenil | zsombi: let's hope no apps released for OTA8 trigger the bug then... | 10:20 |
| jibel | faenil, do you have an example of such app? | 10:21 |
| faenil | it's a risk...but if it's too much trouble to pull the fix in for OTA8, then yes, it's not a really common usecase | 10:21 |
| zsombi | faenil: show an app from RC image that has this crash | 10:21 |
| faenil | zsombi: jibel I don't know. Telegram uses <img> in its delegates, but Telegram is not using ListItemLayout yet | 10:22 |
| zsombi | OTA9 then | 10:22 |
| jibel | faenil, zsombi well a crash in the sdk sounds critical to me, if the fix is safe and you can kand just this fix then it's worth preparing a silo and then we can decide to land or not in ota8 | 10:23 |
| sil2100 | morphis: pong | 10:23 |
| jibel | s/kand/land | 10:23 |
| morphis | sil2100: OTA8 is now in freeze, right? | 10:23 |
| sil2100 | jibel: we still waiting for silo 0 or should I do the snapshot and first rc candidate without that? | 10:23 |
| zsombi | jibel: all AP tests are crashing here'n'there lately... no idea where the crash is so far... bzoltan_ suffers with this for days | 10:23 |
| sil2100 | morphis: yes, we don't accept anything else for ota8 | 10:23 |
| morphis | sil2100: just wanted to know when the doors are open for ota9 things | 10:23 |
| jibel | sil2100, wait, probably more fixes to land today | 10:24 |
| sil2100 | morphis: you want to land anything to the overlay? | 10:24 |
| bzoltan_ | jibel: I do... I really do | 10:24 |
| morphis | sil2100: nothing for ota8 | 10:24 |
| Mirv | faenil: it's not that much about trouble but the risk involved in changing UITK at this point. so it depends on how risky the fix is. | 10:24 |
| === alan_g|afk is now known as alan_g | ||
| sil2100 | jibel: uh, so we stay frozen? Since we could let ota9 landings land to the overlay, but it would make things confusing for me as I would have to cherry pick to the snapshot | 10:24 |
| zsombi | jibel: bzoltan_ so as doors are closed, nothing will go for ota8 anymore, ota9 will get the fix, but first we have to find what causes the random crashes on AP! | 10:24 |
| jibel | sil2100, yes, we stay frozen | 10:24 |
| faenil | Mirv: I'd say the fix has quite low risk, but if pulling that fix will pull more stuff from recent uitk releases, then I can't say | 10:25 |
| === sil2100 changed the topic of #ubuntu-ci-eng to: Train trouble? ping trainguards | CI problems? ping cihelp | Train: http://bit.ly/1hGZsfS | QA Signoffs: http://bit.ly/1qMAKYd | Known Issues: OTA-8 preparations in progress, overlay landings frozen temporarily | ||
| faenil | (I don't know if you cherrypick it) | 10:25 |
| Mirv | faenil: yeah it would definitely need to be single cherry-pick fix if something. but those AP problems zsombi mentions complicate the issue. | 10:25 |
| faenil | Mirv: ok | 10:25 |
| zsombi | faenil: we already have few bugs we cherrypicked, and those were OK with AP... but something got basted since then | 10:26 |
| faenil | zsombi: ok | 10:27 |
| bzoltan_ | zsombi: jibel: brendand knows about it and he knows how to reproduce the issue.. IMO it is a showstopper one. | 10:29 |
| faenil | I agree with jibel that a crash fix is a crash fix, it shouldn't be postponed. But given there are AP failures and the ListItemLayout bug requires a quite special usecase to be triggered, it could make sense... | 10:29 |
| faenil | jibel: just saw your tag. that bug is not a regression (fyi). | 10:30 |
| faenil | it is a new component (introduced in OTA7), the bug has always been there since the component was released | 10:31 |
| faenil | jibel: ^ | 10:31 |
| brendand | bzoltan_, btw i'm looking at that now, i reproduced the issue | 10:31 |
| jibel | faenil, ok, thanks for the clarification | 10:32 |
| faenil | jibel: np :) | 10:32 |
| bzoltan_ | brendand: thanks man! | 10:32 |
| brendand | bzoltan_, apart from the crash though what about all the failures? | 10:33 |
| bzoltan_ | brendand: I would be happy to jump on them and fix them.. but first is to bring back the AP tests to non crashing state | 10:34 |
| brendand | bzoltan_, i'm still not sure why the crash is happening but it is related to test failing, so no failures, no crashes :P | 11:09 |
| bzoltan_ | brendand: Nice try :) but no. The Unity/Mir should not crash after a failing AP test... AP tests to very simple user interaction stuff.. apps could do the same. | 11:10 |
| brendand | bzoltan_, it happens when trying to write some error logs to the result | 11:11 |
| bzoltan_ | brendand: then it is an autopilot bug + a regression in Mir/Unity | 11:11 |
| brendand | bzoltan_, it's neither of those | 11:11 |
| brendand | bzoltan_, it's a testtools bug that's being exposed by lots of failures in uitk tests | 11:12 |
| bzoltan_ | brendand: what else could it be? Can not be UITK because I do have logs from the very same UITK with a different image... image changed, UITK AP tests crash... so something has landed | 11:12 |
| bzoltan_ | brendand: nice try again... but please do not try to blame the UITK for something what it is not responisble for. No AP should crash the shell | 11:13 |
| brendand | bzoltan_, every time a test fails it tries to write some extra logs and at some point (probably on a race condition) that is not working and it crashes autopilot | 11:13 |
| bzoltan_ | brendand: Okey, then it is an autopilot bug | 11:13 |
| bzoltan_ | brendand: failing test must not crash the whole system | 11:13 |
| brendand | bzoltan_, of course | 11:14 |
| bzoltan_ | brendand: and we are not seeking 100% test success here... we are trying to prevent regression.. so tests failed yesterday can fail today... but tests failed yesterday can not block other test runs. | 11:14 |
| bzoltan_ | brendand: I am sure that something has landed in the last weeks what made this regression... because it is a serious regression. | 11:15 |
| brendand | bzoltan_, it's pretty unlikely to have been either testtools or autopilot though as they are practically static. i'm sure it's an existing issue there that's been exposed by some change | 11:27 |
| bzoltan_ | brendand: Most probably ... not the first time that somebody just dumped an untested crap on the Overlay... untested I mean not running the UITK tests. | 11:29 |
| bzoltan_ | brendand: do we have a dashboard to see since what image the UITK tests crash? | 11:29 |
| brendand | bzoltan_, nope | 11:30 |
| bzoltan_ | brendand: we used to have such dashboard :( | 11:30 |
| brendand | bzoltan_, we did, but only a handful of people took any responsibility for it | 11:31 |
| brendand | bzoltan_, whereas everyone was supposed to | 11:31 |
| bzoltan_ | brendand: :( | 11:31 |
| brendand | bzoltan_, and some other changes meant the tests stopped being run a while back | 11:31 |
| brendand | bzoltan_, you could get a jenkins instance from ci and set up a regular run | 11:32 |
| bzoltan_ | brendand: so what should we do now? | 11:40 |
| bzoltan_ | brendand: what disturbs me a lot is that stuff can land and screw up UITK tests without much control | 11:41 |
| === alan_g is now known as alan_g|afk | ||
| === alan_g|afk is now known as alan_g | ||
| mardy | rvr: hi! Did you manage to test silo 1 with the new click? | 12:11 |
| rvr | mardy: Nope, needed to land a higher priority silo | 12:12 |
| mardy | rvr: ok | 12:12 |
| === _salem is now known as salem_ | ||
| === xavigarcia is now known as xavigarcia_lunch | ||
| === alan_g is now known as alan_g|lunch | ||
| === xavigarcia_lunch is now known as xavigarcia | ||
| tvoss | sil2100, ping | 13:45 |
| boiko | jibel: bfiller: not sure if there is still time, but: https://requests.ci-train.ubuntu.com/#/ticket/638 | 13:49 |
| pmcgowan | boiko, yeah we want that one | 13:58 |
| pmcgowan | mark it when ready | 13:59 |
| boiko | pmcgowan: ready already | 13:59 |
| === awe is now known as awe|afk | ||
| pmcgowan | davmor2, then ^^ | 13:59 |
| pmcgowan | boiko, is it marked ready? | 14:00 |
| pmcgowan | oh it is | 14:00 |
| boiko | pmcgowan: yep | 14:01 |
| davmor2 | pmcgowan: is that silo 0 | 14:02 |
| davmor2 | ah no 55 | 14:03 |
| === barry` is now known as barry_ | ||
| === barry_ is now known as barry | ||
| davmor2 | rvr: ^^ silo 55 is in jibel's list of bug fixes | 14:03 |
| davmor2 | pmcgowan, boiko: any news on silo0 | 14:04 |
| boiko | davmor2: salem_ is working on it, fixing a bug bfiller found | 14:04 |
| davmor2 | boiko: nice thanks | 14:05 |
| pmcgowan | davmor2, which image has silo 2? its not working here for me | 14:17 |
| davmor2 | pmcgowan: 172 on krillin | 14:19 |
| pmcgowan | davmor2, not working on mako or mx4 for me | 14:19 |
| pmcgowan | with whatever I got this morning | 14:19 |
| pmcgowan | davmor2, I understood it would be in the next image | 14:20 |
| davmor2 | pmcgowan: yes the one that landed last night 172, 171 was the last build yesterday | 14:21 |
| pmcgowan | davmor2, I am saying the update I got an hour ago does not have it, is that expected? | 14:21 |
| pmcgowan | or is it me | 14:22 |
| davmor2 | pmcgowan: opening here I get a location on the front road almost instantly | 14:22 |
| ogra_ | on MX4 image 162 location works fine again for me too | 14:23 |
| davmor2 | pmcgowan: http://people.canonical.com/~lzemczak/landing-team/ubuntu-touch/rc-proposed/172.commitlog | 14:24 |
| pmcgowan | something wrong here | 14:25 |
| pmcgowan | about telss me I have 154 which I updated to 11/11 | 14:25 |
| pmcgowan | wtf | 14:25 |
| pmcgowan | I have the wrong channel | 14:28 |
| davmor2 | pmcgowan: indeed somehow | 14:28 |
| davmor2 | pmcgowan: that sounds most strange | 14:29 |
| pmcgowan | davmor2, last time I reflashed I picked poorly | 14:29 |
| pmcgowan | cockpit error | 14:29 |
| davmor2 | pmcgowan: https://www.youtube.com/watch?v=Ubw5N8iVDHI | 14:30 |
| pmcgowan | davmor2, I just bricked my phone with a wrong flash | 14:32 |
| pmcgowan | forgot to give it a channel so it installed v1 | 14:32 |
| === alan_g|lunch is now known as alan_g | ||
| davmor2 | pmcgowan: ouch yeah that will do it, for mx4 power and volume down to get to bootloader, for mako power and volume+ & volume- | 14:33 |
| pmcgowan | davmor2, its back | 14:33 |
| pmcgowan | now to fix it | 14:34 |
| davmor2 | pmcgowan: you all sorted now? | 14:42 |
| pmcgowan | davmor2, waiting to see, it keeps saying failed to enter recovery but then finishes | 14:42 |
| pmcgowan | ubuntu spinny logo | 14:42 |
| davmor2 | pmcgowan: https://wiki.ubuntu.com/QATeam/ChannelsToFlash does your flash command resemble the ones on that page? | 14:44 |
| pmcgowan | davmor2, I have 162 now | 14:45 |
| davmor2 | \o/ | 14:45 |
| davmor2 | pmcgowan: and does location work now? | 14:45 |
| pmcgowan | davmor2, at some pint I used ubuntu instead of mezu.en | 14:45 |
| === awe|afk is now known as awe | ||
| pmcgowan | davmor2, yes, bam | 14:46 |
| davmor2 | pmcgowan: \o/ | 14:46 |
| salem_ | trainguards, hi, can you guys help me with silo 0 and 20? we created two silos because we can't dual land libphonenumber as it's a source package, but we have to land dialer-app along with the libphonenumber change. But the vivid silo now complains about versioning of dialer-app. how should I proceed? | 14:53 |
| xavigarcia | trainguards: hey guys, one question... I see silo 46 landed. Just would like to know if it landed for ota-8 or ota-9 | 14:58 |
| xavigarcia | trainguards: please note that the silo adds some new strings that need to be translated | 14:58 |
| === chihchun is now known as chihchun_afk | ||
| sil2100 | salem_: I'm on holidays right now but let me try taking a quick look | 14:59 |
| sil2100 | salem_: ok, seeing the vivid silo it seems dialer-app needs a rebuild | 15:00 |
| xavigarcia | sil2100, Hi there... sorry to bother you, I know you are off today... silo 46 landed and in case it landed in ota-8 I think we might have problems... the silo adds new strings that I'm not sure are translated | 15:05 |
| salem_ | sil2100, I tried that, but I believe the problem here is that I am using the same MR on both silos. we usually do dual landing for dialer-app. | 15:06 |
| sil2100 | salem_: ah, yeah, that won't do... | 15:06 |
| sil2100 | salem_: you can just have a separate silo for just dialer-app if anything and dual land that there | 15:07 |
| sil2100 | If, of course, you don't need to build against the new libphonenumber explicitly | 15:07 |
| salem_ | sil2100, yes, we don't. I think I will end up doing that. thanks! | 15:07 |
| sil2100 | salem_: this would make some things easier, and you can ask QA to for instance signoff both at once | 15:08 |
| salem_ | sil2100, yeah, will do that. thanks for the help. enjoy your holidays :) | 15:08 |
| alecu | sil2100: sorry to bother, but we are concerned with xavigarcia about silo 46 being published | 15:09 |
| alecu | sil2100: we decided to delay it to OTA-9, but it seems that it was published yesterday: https://ci-train.ubuntu.com/job/ubuntu-landing-046-2-publish/27/ | 15:09 |
| alecu | was it somehow published to OTA-8? | 15:10 |
| alecu | any other trainguard around? ^^^ | 15:11 |
| sil2100 | alecu: it was signed off so we landed it, it was rebuilt without the dep-changes right? | 15:12 |
| alecu | sil2100: yes, it was rebuilt without that, but it was bounced by cyphermox after that | 15:13 |
| alecu | sil2100: and since it was already late, we decided to postpone it to ota-9 | 15:13 |
| alecu | we added a note about that to the silo: https://requests.ci-train.ubuntu.com/#/ticket/528 | 15:14 |
| alecu | sil2100: the thing is that it adds a few strings that are very user-visible | 15:14 |
| alecu | those are shown whenever you change the volume | 15:14 |
| alecu | jibel: ^ should we revert it? | 15:15 |
| sil2100 | hm, well, that's bad in that case | 15:15 |
| cyphermox | fwiw it wasn't just the dependency issues that bothered me, but also the fact that there are changes that have no relation to the changelog entry. | 15:15 |
| sil2100 | cyphermox: those all were fixed, there were no packaging changes | 15:15 |
| sil2100 | Since I was able to publish it with my powers | 15:15 |
| alecu | cyphermox: yes, I think you did right to bounce it at that point | 15:15 |
| cyphermox | I don't know; I still see these changes in the build job | 15:16 |
| sil2100 | cyphermox: if there were packaging changes I wouldn't be able to publish as it's a main package | 15:16 |
| cyphermox | and on the publish job | 15:16 |
| sil2100 | And I'm a MOTU | 15:16 |
| cyphermox | sil2100: it doesn't matter in this case, that wasn't the problem | 15:16 |
| cyphermox | my issue with it, and it looks unresolved, is that there are a dozen files added that aren't obviously mappable to adding OSD notifications | 15:17 |
| alecu | xavigarcia: cyphermox is right: the silo diff still seems to contain all of the testing harness | 15:18 |
| cyphermox | in other words, this is a new version, it probably should say that it's a new upstream version, or list the changes that introduce, for instance, the files in gmenuharness | 15:18 |
| renatu | Mirv, ping | 15:18 |
| cyphermox | I realize this is likely tests, but it's not so good to include changes that aren't listed in changelog | 15:19 |
| alecu | cyphermox: agreed. | 15:19 |
| alecu | so, should I ask to revert this? | 15:19 |
| cyphermox | alecu: I'm not sure I'd bother unless there is another reason to | 15:20 |
| alecu | cyphermox: the big reason is that this adds new strings, and that they are very user visible | 15:20 |
| xavigarcia | alecu, cyphermox: the files are present but are no longer compiled | 15:20 |
| cyphermox | xavigarcia: it's irrelevant, we don't necessarily see what is compiled and what isn't from a quick look at the diffs | 15:21 |
| cyphermox | alecu: strings aren't an issue for xenial, at least | 15:21 |
| cyphermox | for vivid, you need an SRU bug in which case it may be arguable. | 15:22 |
| boiko | trainguards: could you please remove dialer-app from silos 0 and 20 PPAs? | 15:22 |
| alecu | cyphermox: I think we only want to land in the phone vivid overlay, not in vivid itself | 15:23 |
| sil2100 | boiko: done | 15:24 |
| alecu | cyphermox: and I worry if this has landed in the overlay, because ota-8 will have those strings untranslated. | 15:24 |
| cyphermox | alecu: that's up to you I suppose | 15:25 |
| boiko | sil2100: thanks | 15:25 |
| renatu | Mirv, could you get a silo for this? https://code.launchpad.net/~renatofilho/ubuntu/vivid/qtpim-opensource-src/fix-1514350/+merge/277264 | 15:25 |
| cyphermox | alecu: sil2100: shouldn't we be trying to land things for the overlay as SRUs as well though? | 15:28 |
| cyphermox | ie. not just in the overlay but also in vivid | 15:28 |
| boiko | sil2100: so, on silo 0, I removed dialer-app from the source package names, but it stills shows the silo as dirty saying dialer-app needs rebuilding | 15:30 |
| alecu | cyphermox: sorry, otp. I understand that the changes here apply to the phone, so we need the on the overlay, and I'm not sure about SRUing them | 15:51 |
| alecu | xavigarcia_: can you confirm that? ^ | 15:51 |
| xavigarcia_ | alecu: yeah, the changes apply to the phone | 15:55 |
| xavigarcia_ | alecu: the desktop is not showing those strings | 15:56 |
| alecu | xavigarcia_: great | 15:56 |
| pmcgowan | another update just popped up | 16:05 |
| alecu | hi trainguards, may we ask to revert this landing? https://requests.ci-train.ubuntu.com/#/ticket/528 | 16:09 |
| greyback | trainguards: this should really go in for ota8: https://requests.ci-train.ubuntu.com/#/ticket/627 | 16:10 |
| greyback | I keep forgetting if I'm supposed to click "publish" or that's a guard's job | 16:10 |
| rvr | bfiller: ping | 16:12 |
| === ev_ is now known as ev | ||
| === Ursinha_ is now known as Ursinha | ||
| brendand | bzoltan_, can you return the favour and have someone look at https://code.launchpad.net/~brendan-donegan/ubuntu-ui-toolkit/swipe_into_view_keyboard/+merge/273805 | 16:21 |
| brendand | bzoltan_, it's been sitting around for a while | 16:21 |
| bzoltan_ | brendand: sure | 16:21 |
| brendand | bzoltan_, i just managed to do a full run without it crashing :/ | 16:22 |
| brendand | bzoltan_, not really a good thing, but interesting | 16:22 |
| pmcgowan | greyback, yeah 44 can publish | 16:22 |
| bzoltan_ | brendand: how did you do that??? | 16:22 |
| bzoltan_ | brendand: must be black magic | 16:22 |
| brendand | bzoltan_, luck maybe | 16:23 |
| bzoltan_ | brendand: tomorrow I will eliminate 50% of the failures ... about that much fails because of the popover button introspection. | 16:24 |
| brendand | bzoltan_, yeah there were a lot of those | 16:24 |
| greyback | wtf | 16:24 |
| greyback | lousy luck, newer qtmir pushed to -proposed, but problem to fix on armhf xenial | 16:31 |
| rvr | jgdx: ping | 16:41 |
| robru | greyback: that silo was already published when you published it | 17:12 |
| greyback | robru: yeah? Sorry, the train made it look like it wasn't | 17:13 |
| rvr | boiko: ping | 17:13 |
| robru | greyback: look at the audit log on the ticket. sil2100 published it, it was migrating, you published it again | 17:13 |
| robru | greyback: what did it look like when you clicked published? | 17:13 |
| greyback | robru: ok, then was my bad timing | 17:13 |
| greyback | as it definitely had the QA signoff as the last thing in the log when I clicked publish | 17:13 |
| robru | greyback: the full log is hidden by default. the status would have said "x is in the proposed pocket" which means "this is already published" when you published. | 17:14 |
| greyback | robru: sure, that I know. Perhaps I hadn't hit F5 soon enough | 17:15 |
| greyback | robru: anything broken? | 17:15 |
| robru | greyback: doubtful: sil published it ~20 hours before you, and the page auto-refreshes every 3 minutes. ;-) | 17:15 |
| robru | greyback: no nothing broken. but probably worth checking why it's in proposed for so long | 17:15 |
| greyback | robru: yeah I'm trying to figure that out, something weird with xenial | 17:16 |
| robru | alecu: still want that reverted? I can do it if you're sure. | 17:17 |
| robru | tvoss: the train is technically capable of doing a xenial+wily silo but this isn't exposed in the UI because I'm not aware of a use case for it. | 17:19 |
| alecu | robru: yes, we'd like to revert that silo, please. | 17:19 |
| alecu | robru: thanks! | 17:20 |
| robru | alecu: just to confirm: you want to go back to 12.10.2+15.10.20151019-0ubuntu1 right? that's the one before the most recent upload. | 17:24 |
| robru | hmm | 17:30 |
| boiko | rvr: pong | 17:31 |
| rvr | boiko: Hi | 17:31 |
| boiko | rvr: hello | 17:31 |
| rvr | boiko: I'm testing silos 49 and 0 | 17:31 |
| boiko | rvr: ok | 17:31 |
| rvr | boiko: I did a dial-number /ril_0 123456789#1 and worked | 17:32 |
| rvr | boiko: Do you need any specific test? | 17:32 |
| rvr | (also tried with the dialer app, manually) | 17:32 |
| boiko | rvr: nope, that's it, and also make sure you create a contact with a number including a #, and try to call from address-book-app | 17:33 |
| rvr | boiko: Nice, trying... | 17:33 |
| boiko | rvr: why is 55 blocked? | 17:34 |
| rvr | boiko: Cannot reproduce | 17:35 |
| boiko | rvr: ah ok, in any case, the fix is harmless, in dialer-app's live call view there will be only at most one header section, so we just set the selectedIndex to zero | 17:35 |
| alecu | robru: if that's the previous version, then yes. Where can I check all the versions? | 17:36 |
| robru | alecu: sorry, that didn't work, apparently reverts don't work on dual silos. switched to revert in vivid only and now it's reverting down to 20151014. but it looks like that is the second most recent version in the overlay: | 17:37 |
| robru | https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/stable-phone-overlay/+packages?field.name_filter=indicator-sound&field.status_filter=&field.series_filter=vivid | 17:37 |
| boiko | rvr: you tried to reproduce in the latest rc-proposed? | 17:40 |
| robru | alecu: once it's built I'll get you to review the diff to make sure it has sensible contents. | 17:40 |
| rvr | boiko: Yes | 17:41 |
| robru | alecu: also this revert will only be in vivid, it won't revert your trunk or anything, so you'll be able to keep developing on your same code until you're ready to release again | 17:41 |
| boiko | rvr: let me try here | 17:41 |
| rvr | boiko: Address book and messaging app can dial to numbers with # | 17:42 |
| boiko | rvr: great! so the problem is fixed | 17:42 |
| rvr | boiko: I'm going to do some extra sanity checks, and will approve is everything is fine. | 17:43 |
| boiko | rvr: thanks | 17:45 |
| alecu | robru: sounds great, thanks | 17:54 |
| alecu | robru: when you say that the revert will be in vivid, do you mean the overlay? | 17:55 |
| robru | alecu: yeah | 17:55 |
| robru | alecu: https://ci-train.ubuntu.com/job/ubuntu-landing-057-1-build/lastSuccessfulBuild/artifact/indicator-sound_content.diff/*view*/ this looks a little weird to me | 17:56 |
| robru | alecu: aside from the changelog being completely nuts, do the code changes make sense to you? | 17:59 |
| robru | alecu: my reading of that diff is that there are 52 files just being totally removed. does that sound right? the most recent landing added 52 new files? | 18:00 |
| === alan_g is now known as alan_g|EOD | ||
| pmcgowan | xavigarcia, ^^ | 18:05 |
| robru | pmcgowan: xavigarcia alecu: if that diff is not correct, I'm not sure what went wrong or how to fix it. You'll have to prepare your own merge to revert the changes you don't want. Sorry guys | 18:07 |
| === alexabreu is now known as alex-abreu | ||
| xavigarcia | robru: what error do you get? | 18:12 |
| robru | xavigarcia: what? I don't get an error. I built a package that is ostensibly a revert of the last release except the contents make no sense to me, i want you to review it: https://ci-train.ubuntu.com/job/ubuntu-landing-057-1-build/lastSuccessfulBuild/artifact/indicator-sound_content.diff/*view*/ | 18:13 |
| xavigarcia | robru: ok... let me take a look | 18:13 |
| alecu | robru: sorry, was having a late lunch | 18:16 |
| rvr | boiko: Could you reproduce the label issue with latest rc-proposed? | 18:21 |
| xavigarcia | robru: diff looks fine to me | 18:22 |
| xavigarcia | robru: there were many new files in that silo | 18:23 |
| robru | xavigarcia: ok, with your approval I'll release that to vivid overlay then. | 18:23 |
| boiko | rvr: nops, not happening, but still, the fix is correct, we cannot rely on the SDK selecting anything by default | 18:23 |
| robru | pmcgowan: alecu: xavigarcia: ok the package is now in vivid overlay, so ota8 won't contain the most recent landing. feel free to make a new landing when you're ready | 18:26 |
| alecu | robru: yay! \o/ | 18:27 |
| alecu | robru: thanks a lot. | 18:27 |
| xavigarcia | robru: thanks! | 18:27 |
| robru | alecu: xavigarcia: you're welcome | 18:27 |
| rvr | boiko: I see | 18:39 |
| rvr | boiko: I think it first needs to set a default SIM | 18:40 |
| boiko | rvr: well, shouldn't make a difference, but who knows? :) | 18:42 |
| rvr | boiko: Approving silos 49 and 0 | 18:55 |
| boiko | rvr: great! thanks! | 18:56 |
| boiko | rvr: with that in mind, I think it is a good idea to also approve silo 20 (which is the xenial silo for the libphonenumber changes) | 18:56 |
| boiko | pmcgowan: should I go ahead and merge those? | 18:57 |
| boiko | s/merge/publish/ | 18:57 |
| pmcgowan | sure they are happroved | 18:57 |
| boiko | \o/ | 18:57 |
| rvr | boiko: davmor2: Approving silo 55. | 19:24 |
| davmor2 | rvr: nice | 19:25 |
| rvr | Err | 19:25 |
| boiko | trainguards: publishing silo 0 failed because it says it's dirty, that dialer-app needs rebuilding, but dialer-app is not on that silo anymore | 19:38 |
| boiko | rvr: davmor2: great! thanks, I will just have to rebuild that after silo 49 lands | 19:39 |
| boiko | robru: could you please check what is going on with silo 0? | 19:41 |
| robru | boiko: one sec | 19:57 |
| boiko | robru: thanks | 19:59 |
| robru | boiko: ok, had to manually drop dialer-app from inside jenkins' local silo dir, just doing a WATCH_ONLY to clear up the status, should be ready to publish as soon as that finishes | 20:00 |
| robru | there | 20:00 |
| robru | brb | 20:01 |
| boiko | robru: thanks | 20:02 |
| boiko | robru: turns out I am not authorized to publish libphonenumber, could you please publish it? | 20:08 |
| robru | boiko: if you're not authorized, neither am i. You need a core dev | 20:08 |
| boiko | robru: anyone here who could do that? | 20:09 |
| davmor2 | boiko: slangasek might be able to help | 20:10 |
| robru | boiko: i usually poke kenvandine or mterry | 20:10 |
| slangasek | it's a US holiday today | 20:10 |
| davmor2 | boiko: don't forget to land 20 too | 20:11 |
| slangasek | but I happen to be around so I'll have a look | 20:11 |
| boiko | davmor2: that's my next question: do you guys need to validate it? | 20:11 |
| boiko | davmor2: it is the same changes as silo 0, but targetting xenial | 20:11 |
| boiko | slangasek: thanks | 20:11 |
| davmor2 | boiko: we don't touch xenial yet | 20:11 |
| boiko | davmor2: so it is just landing and that's it? | 20:12 |
| davmor2 | boiko: that can just land the important one was 49 which touches both | 20:12 |
| davmor2 | boiko: yeap | 20:12 |
| boiko | davmor2: ok, let me change the silo | 20:12 |
| boiko | slangasek: if that's not asking too much, could you please publish silo 20 too? (same changes as silo 0, but for xenial) | 20:13 |
| slangasek | boiko: I'm surprised by the version number of the libphonenumber package for vivid; is this targeted as an SRU, or for the phone overlay ppa? | 20:15 |
| boiko | slangasek: phone overlay | 20:16 |
| slangasek | boiko: then it must not use this version number, which is reserved for SRUs | 20:16 |
| boiko | slangasek: ah, I think that's missing in the silo | 20:16 |
| boiko | slangasek: what version number would be fine? | 20:17 |
| slangasek | boiko: the previous changelog entries show '6.0+r655-0ubuntu7vivid1', '6.0+r655-0ubuntu7vivid2' - something in that vein is probably ok | 20:17 |
| slangasek | robru: ^^ is there a specific numbering convention boiko should use here? | 20:17 |
| boiko | I think kenvandine advised salem_ to use that version number | 20:18 |
| slangasek | mm | 20:18 |
| boiko | not sure why | 20:18 |
| slangasek | well, unless it's targeted for SRU, it's wrong and a problem | 20:18 |
| slangasek | as it would lead to any later SRU of libphonenumber in vivid having the same number but different content | 20:19 |
| robru | slangasek: boiko: train only enforces a certain version for MPs, so no preference on my end for manual sources | 20:19 |
| slangasek | robru: sure, but what's the convention? maybe there's something we can use for consistency | 20:20 |
| boiko | I have no preference either (and no knowledge about the policies for that) | 20:20 |
| robru | slangasek: well it would be upstream+15.04.YYYYMMDD-0ubuntu1 | 20:21 |
| slangasek | right | 20:21 |
| slangasek | so no help for this ;) | 20:21 |
| robru | Maybe Ken thought it was an SRU | 20:21 |
| boiko | robru: would it be possible to tweak the versioning in the silo ppa, or do we have to go through all the uploading, building, etc again? | 20:21 |
| slangasek | the package needs reuploaded and rebuilt | 20:22 |
| robru | slangasek: I'm not sure why the existing version has +rNNN in it, not familiar with that convention. I mean i get it's a revno but dunno who's idea that was | 20:22 |
| robru | boiko: no way to change the version without a new upload | 20:22 |
| boiko | robru: ok, I will ask salem_ to provide a new package, which version you advise us to use? I have really no idea there :) | 20:23 |
| robru | boiko: i guess 6.0+15.04.20151111-0ubuntu1 unless anybody has a good reason to keep the revno in there | 20:24 |
| boiko | salem_: ^ | 20:24 |
| boiko | salem_: can you provide a package with that versioning? | 20:25 |
| boiko | slangasek: what about the version in silo 20? is that one ok? | 20:26 |
| salem_ | robru, boiko sure, but it doesn't match the previous versions in the overlay ppa, is that ok? | 20:26 |
| salem_ | last one was 6.0+r655-0ubuntu7vivid2 | 20:26 |
| boiko | slangasek: 7.0.8-0ubuntu3 | 20:26 |
| boiko | slangasek: (for xenial) | 20:26 |
| robru | boiko: salem_: wait, that version I said isn't actually higher than the one already there | 20:28 |
| robru | boiko: salem_: I guess just use 6.0+r655-0ubuntu7vivid3 or something. I don't know where that scheme came from | 20:29 |
| robru | boiko: salem_: if you guys ever do a 6.0.1 or a 6.1 it'd be nice to switch to the train style versions for consistency | 20:30 |
| boiko | robru: we are not upstreams for that, I think kenvandine wanted to change the packaging to be able to do dual landings, that might include fixing the versioning | 20:31 |
| slangasek | boiko: yes that version is ok for xenial | 20:36 |
| boiko | slangasek: would you mind publishing it then? | 20:36 |
| === salem_ is now known as _salem | ||
| slangasek | boiko: I haven't looked at the actual package yet - what's the ticket number? | 20:37 |
| boiko | slangasek: https://requests.ci-train.ubuntu.com/#/ticket/622 | 20:38 |
| slangasek | boiko: ok, publishing | 20:43 |
| boiko | slangasek: thanks! | 20:43 |
| boiko | robru: package with the right versioning (we used 6.0+r655-0ubuntu7vivid3) building in a ppa on my launchpad page, I'll let you know when it finishes so that you copy it to the silo 0 ppa | 20:45 |
| robru | boiko: is your ppa devirt? If not we can just do a source copy now | 20:46 |
| boiko | robru: what is devirt? If I am supposed to know, then my PPA is not :D | 20:47 |
| boiko | robru: https://launchpad.net/~boiko/+archive/ubuntu/source-uploads/+packages | 20:47 |
| robru | boiko: yeah train PPAs have special builders, so i can't binary copy, have to source copy and rebuild. | 20:49 |
| boiko | robru: that's fine | 20:49 |
| boiko | robru: thanks | 20:51 |
| robru | boiko: UGH that version is lower than the version in the PPA | 20:52 |
| boiko | robru: yeah, I guess so | 20:52 |
| robru | boiko: let's free the ppa and get you a new one | 20:52 |
| boiko | robru: ok | 20:52 |
| robru | boiko: I'll do it | 20:52 |
| robru | boiko: OK building in silo 6 now, when it finishes please give it a quick smoketest to make sure nothing exploded in the rebuild | 20:58 |
| boiko | robru: sure thing, thanks for the help | 20:58 |
| robru | boiko: you're welcome | 20:59 |
| boiko | robru: all good with silo 6 | 21:30 |
| boiko | slangasek: if you are still around, mind publishing it? we had to reassign the silo to fix the version: https://requests.ci-train.ubuntu.com/#/ticket/623 | 21:31 |
| boiko | pmcgowan: if slangasek is not around anymore, we just need a core dev to publish silo 6, all the other dialer-app related silos are publishing already | 21:35 |
| pmcgowan | boiko, wonder who is working | 21:37 |
| pmcgowan | rsalveti, ?;) | 21:38 |
| pmcgowan | maybe cyphermox is around | 21:39 |
| cyphermox | what's up? | 21:52 |
| pmcgowan | cyphermox, hey, we want to publish silo 6 | 21:56 |
| cyphermox | libphonenumber? | 21:57 |
| pmcgowan | yes | 21:57 |
| cyphermox | ok | 21:58 |
| pmcgowan | ty | 21:59 |
| cyphermox | tiagosh, boiko: usually I'd expect to see comments in the patch following the DEP-3 patch tagging guidelines, but I'm not going to block that publishing on it | 21:59 |
| cyphermox | http://dep.debian.net/deps/dep3/ | 21:59 |
| cyphermox | note that there's another issue with the package, the version number isn't sustainable, as soon we'll be at something that might not map as being bigger than z, so it would be better to use release numbers rather than release codenames in the version number. | 22:00 |
| cyphermox | (by that I mean the "vivid" in 6.0+r655-0ubuntu7vivid3) | 22:00 |
| * cyphermox presses the buttons | 22:01 | |
| cyphermox | please make sure this lands in xenial too; as well as an SRU in the appropriate releases as necessary. | 22:03 |
| cyphermox | oh, I see it's already in xenial | 22:03 |
| pmcgowan | cyphermox, thanks | 22:06 |
| cyphermox | fwiw: # as a dialable char, useful ;) | 22:07 |
| pmcgowan | indeed | 22:10 |
| boiko | cyphermox: thanks | 22:11 |
| robru | cyphermox: yeah i mentioned already getting them on the train version scheme but unfortunately "15.04" isn't newer than "r655" so it won't work until they bump the upstream number. | 22:12 |
| cyphermox | robru: you could just replace "vivid3" ".15.04.3" or something to that effect | 22:13 |
| robru | cyphermox: i think that would interfere with the SRU versioning | 22:14 |
| cyphermox | plus given that we just do snapshots all the time, the upstream version numbers are close to meaningless, except for here when we're doing sru | 22:14 |
| cyphermox | robru: not that much | 22:14 |
| robru | cyphermox: are we the upstream of this project? If so why is it not using MPs? | 22:14 |
| cyphermox | it just needs some extra fudging, like adding a .$n more above whatever version is SRU'd to vivid | 22:15 |
| robru | boiko: ^^ | 22:15 |
| cyphermox | robru: there's always a way to make the version numbering work for what we need it to do, without using release codenames | 22:15 |
| robru | cyphermox: yeah i agree the codenames are bad in version numbers | 22:16 |
| robru | cyphermox: ironically this whole mess started because Steve rejected "0ubuntu7.1" as a version because it would conflict with SRU versioning | 22:17 |
| cyphermox | sure | 22:17 |
| cyphermox | I mean, it's true that it would | 22:17 |
| cyphermox | what you want is probably always something higher than whatever might be in vivid-proposed until you merge between proposed and overlay | 22:17 |
| cyphermox | so, 0ubuntu8~15.04.1 ? | 22:18 |
| cyphermox | as opposed to the SRUs which will always grow from 0ubuntu7 | 22:19 |
| cyphermox | but all this quickly becomes quite ugly | 22:19 |
| robru | cyphermox: yeah tilde is probably better for wedging versions in between other versions | 22:19 |
| cyphermox | IMHO the overlay needs to disappear and we need to find some way to be more agile at doing SRUs, which is a process we already are pretty good at I think | 22:20 |
| cyphermox | either that or getting back to the ubuntu-rtm derived distro? | 22:21 |
| robru | cyphermox: agree that overlay PPA is a gross hack, but disagree about SRUs. In my experience they take months of pain each. Better solution is just build phone images on dev series | 22:22 |
| robru | cyphermox: rtm was a bad idea, let's never mention it again | 22:22 |
| cyphermox | robru: I don't know that I'm sure of that. | 22:27 |
| cyphermox | and let's not fool ourselves, the overlay PPA is SRUs, just with relaxed regression checking. | 22:28 |
| cyphermox | bbl, I need to make a phone call :/ | 22:28 |
| robru | cyphermox: yeah, "relaxed regression testing" meaning "not having to wait months for your SRU to get approved" | 22:28 |
| cyphermox | you don't have to wait months to get SRUs approved; if you do it right it can take just 7 days, just long enough to have a bit more assurance in proposed that things aren't breaking | 22:29 |
| robru | cyphermox: heh, "just 7 days", we fly by the seat of our pants in phone land. | 22:30 |
| robru | cyphermox: pat is currently freaking out because a 9 hour delay on an image build is too long. | 22:30 |
| cyphermox | I'm not saying either option is perfect. | 22:30 |
| === _salem is now known as salem_ | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!