Friday, 2016-07-01

*** gothicmindfood has joined #storyboard01:36
*** openstackgerrit has quit IRC07:33
*** openstackgerrit has joined #storyboard07:33
*** mrmartin has joined #storyboard08:17
*** openstackgerrit has quit IRC08:18
*** openstackgerrit has joined #storyboard08:18
Zaramorning, storyboard!09:22
*** mrmartin has quit IRC09:30
* Zara looks at https://review.openstack.org/#/c/333409/10:59
openstackgerritMerged openstack-infra/storyboard: Add a way to retrieve the history of edited comments  https://review.openstack.org/33340911:50
Zaraaw, linting fails for the non-permanent filter buttons11:51
Zaranot sure why it wasn't caught before the gate, hm11:51
Zara2016-07-01 11:07:08.607800 | src/app/stories/controller/story_detail_controller.js11:51
Zara2016-07-01 11:07:08.607830 |   107:34  error  headers is defined but never used  no-unused-vars11:51
Zarathat seems like something that should have been caught11:51
Zara(for: https://review.openstack.org/#/c/331748/ )11:52
SotKI think something has changed, like the version of eslint used in the gate or something11:52
SotKI saw that in a totally unrelated patch too iirc11:52
Zarahuh, the same failure is present in the test that's marked as success11:53
SotKoh, the subscribe button xD11:53
Zara2016-06-21 11:28:28.920835 src/app/stories/controller/story_detail_controller.js11:53
Zara2016-06-21 11:28:28.920865   107:34  error  headers is defined but never used  no-unused-vars11:53
SotKhttps://review.openstack.org/#/c/324550/4/src/app/stories/controller/story_detail_controller.js is the fix you need11:53
SotKhmm, weird11:53
Zarayeah, I'm more bothered by the test reporting success than by the actual error, which seems sensible enough...11:53
*** mrmartin has joined #storyboard11:54
* SotK disappears again11:54
Zara:)11:54
Zaraand I was about to ask how the holiday was going!11:54
*** mrmartin has quit IRC12:05
Zaraasked in #infra, AJaeger has shed light on it; t'was a bug fixed by: https://review.openstack.org/#/c/334582/12:07
Zaraso I think it was just checked before that merged12:08
ZaraI'm still not au fait with changing other people's patches in gerrit. which is to say that last time I tried, everything went wrong and I panicked.12:41
ZaraI'd rather announce events timelineness after we have the non-permanent filters, so... atm plan to send out an email mid-next-week.12:42
Zarait's so quiet today!13:06
persiaZara: Context on the notes-read-only-by-default change: that was a consequence of notmorgan accidentally overwriting a few notes when looking around.13:22
Zaraokay, I agree with the motivations for it, I'm partly hoping that if someone asks about the possibility of having ephemeral notes in a few months' time, it'll ring a bell13:25
Zaramight be wishful thinking, but hoping people won't get caught up in redoing it.13:26
Zara(also, heh, I'd remembered it as dhellman but wasn't sure... I timed that bugsquash atrociously.)13:29
Zarapedroalvarez: I'm pinging you about the ansible role/docs since I said I'd bug you about it ages ago and forgot. :) did you get anywhere with it?13:45
*** sparkycollier has quit IRC14:23
*** sparkycollier has joined #storyboard14:24
*** jamesmcarthur has joined #storyboard15:22
Zarahm, looks like once you archive a lane, you can't add things from that lane to other lanes in the same board15:40
ZaraI thought we'd found and fixed that earlier, but maybe I'm getting it confused15:40
Zarathis is probably why my oldest board isn't finding a bunch of tasks and stories, anyway15:40
Zaragrr15:40
ZaraI've added a task for it so it doesn't get lost.15:45
Zaraugh, linting fails for task notes patch, too; gonna take a wild guess about what the error will be...16:26
Zarayup, same thing16:26
Zara('headers' var is unused)16:26
Zarawelp on monday I'll dig into changing others' patches via gerrit, since that's blocked... 2 patches today? there's no urgency to merging them today, but that's annoying.16:27
Zarahm, subscriptions page does one GET per type of subscription. not sure if that's avoidable or not16:32
Zarait seems a bit weird that it does both a GET for projects with subscriber foo, plus a GET for subscriptions that are to projects, with subscriber foo16:33
Zaraand similar for all other resources16:33
Zarathe page doesn't load particularly slowly, anyway; that just seems weird.16:33
*** jamesmcarthur has quit IRC16:46
Zaraheh, that archived cards order problem is very visible on this worklist I was just testing16:55
Zaraalso seems weird that it just increments the subscription id each time (so if you subscribe, then delete subscription, then subscribe again, the id is incremented the second time you subscribe). well, that's simplest.... I just wonder if it'll get weird wiht high numbers.16:58
Zaraprobably fine16:58
persiaAny given deployment probably needs thousands of developers making thousands of changes a day for thousands of years to overflow in any interesting way.17:01
Zarayup17:02
persiaErm, plus another factor of ten.17:02
ZaraI could see it being slower than it had to be if there were things that requested the full list of subscriptions, including all those deleted ones17:02
Zarabut that's the only thing I can think of17:03
Zara(ie: all the deleted ones when it didn't need to request those)17:03
Zaraand in practice I don't think they hang around; I think it's just the id that's incremented, but I might be wrong on that17:03
SotKyeah, its just that ids are not reused unless explicitly set17:08
Zara\o/17:11
Zarahow's the holiday?17:11
*** jamesmcarthur has joined #storyboard17:34
Zaraevidently absorbing! \o/ night, all17:43
*** jamesmcarthur has quit IRC17:44
*** jtomasek has quit IRC19:42
-openstackstatus- NOTICE: The infrastructure team is taking Gerrit offline for maintenance beginning shortly after 20:00 UTC to upgrade the Zuul and static.openstack.org servers. We aim to have it back online around 22:00 UTC.19:54
*** jtomasek has joined #storyboard19:55
-openstackstatus- NOTICE: Gerrit is offline for maintenance until approximately 22:00 UTC20:08
*** ChanServ changes topic to "Gerrit is offline for maintenance until approximately 22:00 UTC"20:08
*** nibalize1 has joined #storyboard21:50
*** nibalizer has quit IRC21:53
*** nibalize1 is now known as nibalizer21:58

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