| michi | robru: ping | 03:25 |
|---|---|---|
| robru | michi: hey hey hey waddya say? | 03:25 |
| michi | Hi | 03:25 |
| michi | Hey, I have an absolutely trivial change in silo 29, ticket 1374 | 03:26 |
| michi | There is no point in testing or QA. | 03:26 |
| michi | Can we short-cut that? | 03:26 |
| robru | michi: indeed that looks quite trivial. | 03:27 |
| michi | It broke on yakkety | 03:27 |
| robru | michi: not really my call though, you should ask qa to just rubberstamp it | 03:27 |
| michi | Because of the xenial test in teh script. | 03:27 |
| michi | OK | 03:27 |
| michi | Thanks! | 03:27 |
| robru | michi: you're welcome | 03:27 |
| robru | michi: also that silo won't fix yakkety, only xenial+vivid | 03:27 |
| michi | I know. | 03:28 |
| michi | Someone requested a no-change rebuild of the cache, which is how I found out. | 03:28 |
| michi | https://launchpad.net/ubuntu/+source/persistent-cache-cpp/1.0.4+16.04.20160117-0ubuntu2 | 03:28 |
| michi | You can see the failure in the build logs there. | 03:28 |
| michi | It’s not urgent for myself. | 03:29 |
| michi | So, if it goes through the normal QA thing, that’s OK too. | 03:29 |
| michi | I just thought we could save some people some additional work. | 03:29 |
| robru | michi: well, if you target yakkety in the silo, it doesn't need qa. qa is only for duals. | 03:30 |
| robru | michi: that broken rebuild is in yakkety | 03:30 |
| michi | Yes. | 03:30 |
| michi | I can rebuild for yakkety too. It’s just that, to get into trunk, it really needs to go through the dual-landing build. | 03:31 |
| michi | Hmmm... | 03:31 |
| michi | When I select yakkety in the train, it gives me a warning about an SRU. | 03:31 |
| michi | Shall I ignore that? | 03:31 |
| robru | michi: yeah sorry there's a branch that fixes that but it's blocked for other reasons. | 03:32 |
| michi | NP | 03:32 |
| robru | I should split that up | 03:32 |
| michi | Building for yakkety now. | 03:32 |
| michi | I guess, with that, I can just set QA to “not required”? | 03:32 |
| robru | michi: it should switch automatically with the yakkety build | 03:33 |
| michi | Sweet! | 03:33 |
| michi | robru: What’s with the changelog problem? That version is indeed in the changelog. | 04:08 |
| robru | michi: no it isn't. the manual upload at dest was never synced to trunk so when you made a branch and built it it doesn't include that changelog entry | 04:09 |
| michi | Hmmm... | 04:09 |
| michi | So, how do I fix this? Build again for vivid+xenial? | 04:09 |
| robru | michi: http://bazaar.launchpad.net/~ci-train-bot/persistent-cache-cpp/persistent-cache-cpp-ubuntu-yakkety-landing-029/view/head:/debian/changelog | 04:11 |
| robru | michi: what is there to fix? there's no harm in publishing a package that clobbers a no-change rebuild changelog entry, it's meaningless. | 04:11 |
| robru | michi: just drop your debian/changelog changes and build it and publish it | 04:11 |
| michi | Well, it won’t hurt to have the changelog entry. | 04:12 |
| robru | michi: ok, then copy the debian/changelog from distro over the one in your branch and let the train generate the new entry for you. | 04:13 |
| michi | I just uncommitted. | 04:13 |
| michi | I guess that’s easier after all. | 04:13 |
| michi | Thanks for your help! | 04:13 |
| robru | michi: you're welcome! | 04:14 |
| robru | michi: your MP still touches debian/changelog, that's probably wrong. | 04:15 |
| michi | I just pushed with overwrite. | 04:15 |
| michi | Sigh... | 04:16 |
| robru | michi: I'm still seeing this: http://bazaar.launchpad.net/~michihenning/persistent-cache-cpp/merge-devel/revision/26?start_revid=26#debian/changelog | 04:16 |
| michi | Me too :( | 04:16 |
| robru | michi: just uncommit and push --overwrite | 04:16 |
| michi | I forgot the revert after the uncommit. | 04:18 |
| michi | OK, should be good now. | 04:18 |
| michi | I’ll kick the build | 04:18 |
| === chihchun is now known as chihchun_afk | ||
| robru | looks good | 04:22 |
| robru | michi pls | 05:25 |
| michi | ? | 05:25 |
| michi | I just don’t get it. | 05:25 |
| michi | I can’t publish without the change because the upload at destination. | 05:26 |
| robru | michi: what are you even trying to do with this changelog entry? it's so wrong in so many ways. | 05:26 |
| michi | There was a change to the code. What’s wrong about having a changelog entry for that? | 05:26 |
| robru | michi: because the train generates that for you. the one you wrote does not do what you are thinking it does. | 05:27 |
| michi | So why is it bitching at me then? | 05:27 |
| michi | About the unexpected upload? | 05:27 |
| robru | michi: first of all, putting it as UNRELEASED means the train will munge your version number when it releases it. so it doesn't solve the issue of that version number missing from the changelog. | 05:27 |
| michi | Yes, it then create a new version, right? | 05:28 |
| robru | michi: second of all, if you fix UNRELEASED to 'yakkety', it's a lie, because your upload now does not correspond to that version number | 05:28 |
| michi | So, previously, whenever we landed something on trunk via the train, we had an unreleased changelog entry. | 05:29 |
| robru | michi: I guess you didn't look at this last time I linked it. this is what is in the silo: http://bazaar.launchpad.net/~ci-train-bot/persistent-cache-cpp/persistent-cache-cpp-ubuntu-yakkety-landing-029/view/head:/debian/changelog | 05:29 |
| michi | That then got rewritten, adn things worked just fine. | 05:29 |
| michi | Now I’m doing the same thing and it isn’t working. | 05:29 |
| robru | michi: it is working, in the sense that it's doing exactly what you're telling it to do. you're just doing the wrong thing. | 05:30 |
| michi | OK, I need to know what to do then. | 05:30 |
| michi | Because, without the changelog change, it doesn’t work, and with it, it doesn’t work either. | 05:30 |
| robru | michi: you need to take the changelog that is already in yakkety, copy that overtop of your debian/changelog (just totally clobber the one you wrote by hand) and commit and pusht hat. | 05:30 |
| robru | michi: at no point have you yet attempted to include that changelog change. | 05:30 |
| michi | Aha. | 05:30 |
| michi | So, the problem is that there is still a xenial entry at the top, but for the yakkety build, it wants a yakkety one? | 05:31 |
| robru | michi: no, the problem is that when you write UNRELEASED, the train clobbers your version number, so the fake changelog entry with the version number you are fabricating is just ignored. | 05:32 |
| robru | michi: if you take the real changelog from yakkety, the train will generate a new entry on top of that and it'll be fine. | 05:32 |
| michi | Ah, OK. | 05:32 |
| michi | The error messages don’t really make that clear. | 05:32 |
| robru | michi: this is standard distro syncing mechanics. any time there's a manual distro upload you are supposed to copy it manually from distro to trunk. | 05:33 |
| michi | I’ll try that, thanks. | 05:33 |
| michi | I see. | 05:33 |
| robru | michi: I'm not really sure how it could be clearer but please do file a bug if you have suggestions. | 05:33 |
| michi | So, this happens each time we start a new series. | 05:33 |
| robru | michi: no, this happens each time there's a manual upload in the archive that didn't use the train. | 05:33 |
| michi | I see. | 05:34 |
| robru | michi: the fact that this is the first yakkety upload is a completely unrelated coincidence. | 05:34 |
| michi | I didn’t eve know that there was a manual upload. | 05:34 |
| robru | michi: yes you did. the no change rebuild that you saw and copied the version number of. that's a manual upload. | 05:34 |
| michi | Aha. I live and learn :) | 05:35 |
| robru | michi: the train only ever generates version numbers that end in 0ubuntu1. anything else (in this case 0ubuntu2) is a manual upload. | 05:35 |
| michi | So, this is what the changelog looks like right now: http://bazaar.launchpad.net/~ci-train-bot/persistent-cache-cpp/persistent-cache-cpp-ubuntu-yakkety-landing-029/view/head:/debian/changelog | 05:37 |
| michi | So I’ll use that verbatim | 05:37 |
| robru | michi: no, what? that's the one IN the silo. you want the one from yakkety. | 05:39 |
| michi | Looking for that... | 05:39 |
| michi | Where the hell do I find that? | 05:39 |
| michi | Extract it from the yakkety source package, I guess. | 05:39 |
| robru | michi: yeah, 'apt-source persistent-cache-cpp' and then copy the changelog from that to your branch | 05:40 |
| michi | Yep | 05:40 |
| === chihchun_afk is now known as chihchun | ||
| robru | apt-get source, excuse me | 05:40 |
| robru | (1am here) | 05:40 |
| michi | Just pulled that. | 05:44 |
| michi | It looks the same as our current head without the incorrect changelog entry. | 05:44 |
| michi | Top version is s (1.0.4+16.04.20160117-0ubuntu1) xenial | 05:45 |
| robru | michi: i can't even | 05:46 |
| robru | michi: uncommit what you did and apply this patch: http://launchpadlibrarian.net/257871596/persistent-cache-cpp_1.0.4+16.04.20160117-0ubuntu1_1.0.4+16.04.20160117-0ubuntu2.diff.gz | 05:46 |
| michi | OK, will do, thanks! | 05:47 |
| robru | you're welcome | 05:47 |
| === Mirv changed the topic of #ubuntu-ci-eng to: Train trouble? ping trainguards | CI problems? Use JenkaaS: http://bit.ly/jenkins-docs | Train: http://bit.ly/1hGZsfS | QA Signoffs: http://bit.ly/1qMAKYd | Known issues: robru sprinting in Texas, public holiday in large portion of Europe | ||
| Mirv | including me and eg France/Germany based people but apparently not sil2100 | 06:23 |
| robru | yay | 07:06 |
| === chihchun is now known as chihchun_afk | ||
| === chihchun_afk is now known as chihchun | ||
| Saviq | sil2100, hey, can you please recycle the regression for us https://requests.ci-train.ubuntu.com/static/britney/xenial/landing-071/excuses.html thank you | 08:41 |
| sil2100 | Saviq: done | 08:47 |
| Saviq | davmor2, hey, can you mark https://trello.com/c/yhjTyyxt/3121-1078-ubuntu-landing-071-unity-api-unity8-unity-scopes-shell-pstolowski-cimi as unblocked? it's re-running the britney regression (known flaky test) so is good to test again | 09:24 |
| rvr | Saviq: Done | 09:25 |
| Saviq | rvr, thanks! | 09:25 |
| davmor2 | rvr: thanks | 09:28 |
| popey | ooh, found a nice unity crash. If you get a notification which has a Yes/No prompt - like "your credentials are outdate, do you want to fix" - press no, all is fine, press yes, unity8 dies | 11:34 |
| * popey waits for daisy to process the crash | 11:37 | |
| popey | https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1578601 | 11:45 |
| ubot5 | Ubuntu bug 1578601 in unity8 (Ubuntu) "Crash when choosing "yes" to some popups when locked" [Undecided,New] | 11:45 |
| rvr | popey: o_O | 11:59 |
| popey | yeah | 12:03 |
| renatu | robru, how I can mark this silo as landed? https://requests.ci-train.ubuntu.com/#/ticket/1362 | 12:41 |
| === _salem is now known as salem_ | ||
| * tedg notices sil2100's ust landing and is excited, as it will make people stop bitching to him about those lttng messages in their logs. | 13:28 | |
| popey | managed to reproduce https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1578601 again | 13:35 |
| ubot5 | Ubuntu bug 1578601 in unity8 (Ubuntu) "Crash when choosing "yes" to some popups when locked" [Undecided,New] | 13:35 |
| popey | added screenshot | 13:35 |
| renatu | sil2100, hey, what is the correct way to mark this silo as landed? https://requests.ci-train.ubuntu.com/#/ticket/1362 | 14:17 |
| renatu | sil2100, it only contains the click package which was already published on store | 14:18 |
| rvr | kenvandine: ping | 14:25 |
| kenvandine | rvr, pong | 14:25 |
| rvr | kenvandine: Should System Settings be resizeable? | 14:26 |
| kenvandine | no | 14:26 |
| kenvandine | that was something that was recently changed | 14:26 |
| kenvandine | i think we set the max size | 14:26 |
| kenvandine | because the design isn't optimized for a wide screen, etc | 14:26 |
| rvr | kenvandine: I see it can only be maximized | 14:27 |
| kenvandine | for now | 14:27 |
| kenvandine | i think that'll change eventually | 14:27 |
| kenvandine | something in the shell needs to change to support the max stuff | 14:27 |
| rvr | Ok, so current behavior is a feature and not a bug. | 14:28 |
| kenvandine | yes | 14:28 |
| rvr | Great, wasn't sure | 14:28 |
| kenvandine | i think once the shell handles it, when you maximize it the window will grow to the max size | 14:28 |
| kenvandine | or maybe the maximize control won't show, not sure what the plan is :) | 14:28 |
| kenvandine | but that's all outside of settings | 14:29 |
| sil2100 | renatu: hey! | 14:29 |
| sil2100 | renatu: hm, there's no easy way, but let me try something | 14:30 |
| kenvandine | sil2100, i tried to publish it even thought there was no package or MR | 14:30 |
| kenvandine | it didn't like that :) | 14:30 |
| kenvandine | i had hoped that it would just mark it as landed | 14:30 |
| sil2100 | hah, no, that wouldn't work sadly ;) | 14:32 |
| rvr | ChrisTownsend: Silo 31 approved | 14:43 |
| robru | renatu: sil2100 kenvandine: clicks are an abomination unto the train and I've abandoned the request in order to free the PPA. This should have never been assigned and there's no way currently to mark click tickets as landed. | 14:48 |
| kenvandine | robru, i expected as much | 14:50 |
| robru | kenvandine: clicks are on my list to completely redesign the ux for but unfortunately it's low on the list, after git and parallelism | 14:51 |
| kenvandine | robru, i'd like to see publish for clicks upload to the store :) | 14:52 |
| robru | kenvandine: file a bug i guess? Clicks are a totally unconsidered afterthought bolted on for no particular reason. Clicks would probably be better served by people creating trello cards directly, except that it's also my intention to eventually roll trello board into bileto one day too | 14:54 |
| robru | But that's even further down the line. | 14:55 |
| ChrisTownsend | rvr: Awesome, thanks! | 14:58 |
| rvr | ChrisTownsend: And congrats for your spotlight award :) | 14:59 |
| ChrisTownsend | rvr: thanks:) | 14:59 |
| Saviq | pmcgowan, robru, sil2100, just realized one big drawback of not landing into Y - bugs don't get auto-closed now on release :S | 15:49 |
| pmcgowan | Saviq, ah that is unfortunate, we will start soonish | 15:50 |
| pmcgowan | robru, is waiting for the go ahead | 15:50 |
| sil2100 | Indeed | 15:51 |
| sil2100 | If everyone's ready for dealing with tripple landings already then we can switch that on | 15:52 |
| sil2100 | I thought of maybe waiting at least till OTA-11 is done | 15:52 |
| sil2100 | Since triple landings will mean additional effort for everyone | 15:53 |
| sil2100 | (and I like to have some spare time in case the triple landing bits have some issues) | 15:53 |
| pmcgowan | sil2100, maybe once we snapshot, sometime next week? | 15:53 |
| sil2100 | I suppose it should be safeish already, velocity should be low | 15:54 |
| sil2100 | pmcgowan: let's coordinate it tomorrow on the status meeting then | 15:54 |
| sil2100 | :) | 15:54 |
| sil2100 | Since I'd need to batch copy what we have in xenial-overlay to yakkety once we enable triples | 15:54 |
| ChrisTownsend | trainguards: Hi! Do you have the powers to publish silo 31^^^ Or do I need to find someone else? | 15:58 |
| sil2100 | ChrisTownsend: let me take a look | 15:58 |
| ChrisTownsend | sil2100: Thanks! | 15:58 |
| sil2100 | ChrisTownsend: yeah, manual package uploads need either a trainguard or a core-dev for publishing, on it now | 15:59 |
| ChrisTownsend | sil2100: Ok, great, much appreciated! | 15:59 |
| sil2100 | yw! | 16:00 |
| robru | pmcgowan: sil2100: I'm going to poke at this a little today, as the train is showing SRU warnings against yakkety landings, so I'll clean that up and make sure the trio landings are more ready to go than they already are while I'm at it | 16:09 |
| pmcgowan | great | 16:11 |
| Saviq | sil2100, would you be so kind and publish https://requests.ci-train.ubuntu.com/#/ticket/1078 | 16:54 |
| Saviq | kenvandine, or you ↑↑ | 16:58 |
| Saviq | since mterry's hiding behind network issues | 16:59 |
| kenvandine | sure | 16:59 |
| Saviq | kenvandine, thanks! | 17:14 |
| kenvandine | saviqnp | 17:15 |
| kenvandine | lol :) | 17:15 |
| === marcusto_ is now known as marcustomlinson | ||
| slangasek | sil2100, jibel: fyi, the staging-proposed channels were created but the devel-proposed channels had not been moved to import from yakkety; doing this now | 18:20 |
| === josepht` is now known as josepht | ||
| === chihchun is now known as chihchun_afk | ||
| === chihchun_afk is now known as chihchun | ||
| ToyKeeper | jgdx: I think this is ready to land, but the automated signoff failed. What's the story there? https://requests.ci-train.ubuntu.com/#/ticket/1314 | 21:19 |
| === salem_ is now known as _salem | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!