| *** yarkot has joined #storyboard | 00:29 | |
| *** yarkot_ has joined #storyboard | 01:18 | |
| *** yarkot_ has quit IRC | 01:29 | |
| *** coolsvap|away has joined #storyboard | 04:21 | |
| *** coolsvap|away is now known as coolsvap | 04:22 | |
| *** coolsvap is now known as coolsvap|away | 06:26 | |
| *** mrmartin has joined #storyboard | 08:01 | |
| Zara | krotscheck: heh, can have a go! thanks for your help! :) | 10:15 |
|---|---|---|
| * pedroalvarez tried, didn't understand the failure :/ | 10:47 | |
| Zara | we're all doomed | 10:50 |
| pedroalvarez | hah | 10:53 |
| *** mrmartin has quit IRC | 11:03 | |
| Zara | hmmm... I don't think this is my code, but the error logs aren't just timeout related, so not sure what to make of it https://review.openstack.org/#/c/270337/ | 11:15 |
| Zara | *I don't think this is my code causing the error | 11:15 |
| Zara | https://review.openstack.org/#/c/257509/ is also still hanging about in review, as is https://review.openstack.org/#/c/269707/ . (I'd say these are the most useful of the things we currently have waiting) | 11:17 |
| SotK | hmm, I wonder why that doesn't fail normally | 11:17 |
| Zara | (most useful that aren't failing tests, anyway) | 11:17 |
| Zara | yeah | 11:17 |
| Zara | that's why I don't *think* it's my code, because it seems to be an error that would apply to lots of things... | 11:18 |
| SotK | http://logs.openstack.org/37/270337/4/check/gate-storyboard-webclient-npm-run-test-functional/00118c7/console.html#_2016-01-21_12_52_09_649 looks like it should have happened in every test since my patch to change page titles... | 11:18 |
| Zara | yeah | 11:19 |
| Zara | I don't think I wrote anything that'd affect the title! | 11:19 |
| Zara | oh actually | 11:20 |
| Zara | it looks like I have an extra space first | 11:20 |
| Zara | so maybe that changes the title, too | 11:21 |
| * Zara checks | 11:21 | |
| Zara | given that the error is related to a space at the start of the title | 11:21 |
| Zara | oh yeah I shut my vm down last night | 11:21 |
| Zara | nope nvm it doesn't and it wouldn't affect the title anyway | 11:27 |
| Zara | oh, unless the title test doesn't actually mark the build as failed | 11:29 |
| Zara | I think that's it. it's failing the title test but the one that actually made the build fail was the timeout | 11:30 |
| Zara | so the title test could have been failing for a whiel | 11:30 |
| Zara | *while | 11:30 |
| * Zara tries rechecking again. | 11:32 | |
| *** openstackgerrit has quit IRC | 11:47 | |
| *** openstackgerrit has joined #storyboard | 11:47 | |
| Zara | Krotscheck gave me lots of background on the 401, which I'd like to put on the story to make sure the information is there for everyone, but I feel it'd be creepy to quote without permission. so I'm going to put my own interpretation up there, but it'd be good if someone could check it at some point in case I've misunderstood anything. will link when done. | 11:53 |
| openstackgerrit | Michael Krotscheck proposed openstack-infra/storyboard-webclient: Logging in no longer causes page refresh. https://review.openstack.org/270882 | 11:55 |
| openstackgerrit | Merged openstack-infra/storyboard-webclient: Fix width of "no result" and "loading" rows in list tables https://review.openstack.org/269663 | 11:57 |
| krotscheck | Ok | 12:06 |
| krotscheck | There are a lot of jobs failing right now because the people who are working on pip decided to break us. | 12:06 |
| krotscheck | I'm actually working on the long-term fix while others are dealing with the short term one. | 12:06 |
| krotscheck | (One big short term fix is to recheck the job during not-NorthAmerica-workinghours) | 12:06 |
| Zara | ahh, right | 12:07 |
| Zara | I've noticed much discussion of wheel mirrors and things but I don't really know what any of it means | 12:08 |
| Zara | OpenStack is enormous. | 12:08 |
| krotscheck | Yes it is | 12:09 |
| krotscheck | Wheel mirrors are basically a slightly faster version of pypi mirrors. | 12:10 |
| krotscheck | A "Wheel" is a python package whose dependencies have been precompiled per distro/version, so that it doesn't have to happen during pip install | 12:11 |
| krotscheck | In smaller projects it's not really a big deal. | 12:11 |
| krotscheck | In something the size of openstack we're looking at 10-20% build speed improvement | 12:11 |
| krotscheck | With things being as volatile as they are though, getting a patch to land that helps things is HARD | 12:12 |
| Zara | yeah, I've noticed the need for a lot more rechecks over the last week or so... | 12:14 |
| Zara | so, do python package versions state exact dependencies, or is it something like semver? I'm wondering if the latter could mean you'd get a mismatch between the precompiled dependencies and what you'd think you were installing... but guessing that's been accounted for (if that's even an issue) | 12:17 |
| *** mrmartin has joined #storyboard | 12:25 | |
| *** openstackgerrit has quit IRC | 12:33 | |
| *** openstackgerrit has joined #storyboard | 12:33 | |
| Zara | yay, https://review.openstack.org/#/c/270337/ is passing the tests this time | 12:36 |
| krotscheck | They can. What pip wheel does though is it builds multiple "wheels", one for each package, so if the SAT solver resolves to a different version of a transient dependency, that'll get picked up. | 13:00 |
| openstackgerrit | Merged openstack-infra/storyboard-webclient: Always go to Story Detail Page after adding Story https://review.openstack.org/270337 | 13:04 |
| *** mrmartin has quit IRC | 13:07 | |
| *** mrmartin has joined #storyboard | 13:08 | |
| *** mrmartin has quit IRC | 13:16 | |
| Zara | ah, I was confused about how that would help in the case of multiple precompiled wheels being ready, then a user building the packages later, using those wheels, where the packages' transitive dependencies had changed in the interim-- but I'm guessing pip wheel notices when the transitive deps change and they get precompiled again each time? | 13:33 |
| krotscheck | Well, sortof | 14:19 |
| krotscheck | Pip says: Hey, I need these things. | 14:19 |
| krotscheck | Pip then goes to all the configured mirror URL's to see what it's got. | 14:19 |
| krotscheck | If a wheel is available, it'll grab that. Otherwise it'll grab the dependency source package. | 14:20 |
| krotscheck | The wheels have to be prebuilt out of band. | 14:20 |
| *** mrmartin has joined #storyboard | 15:04 | |
| *** mrmartin has quit IRC | 15:40 | |
| Zara | ah, okay, I think I get it. :) | 15:50 |
| Zara | I didn't think about pip having record of what, say, the highest version of package foo *was* to compare to the wheels, but it sounds like that's what happens... or I'm just sleepy | 15:53 |
| *** mrmartin has joined #storyboard | 15:54 | |
| openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard: Move summarize_task_statuses into wmodels.Story https://review.openstack.org/271407 | 16:03 |
| openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard: Resolve Board and Worklist contents on the server-side https://review.openstack.org/271408 | 16:03 |
| *** mrmartin has quit IRC | 16:19 | |
| *** mrmartin has joined #storyboard | 16:31 | |
| * Zara tests those now! | 16:34 | |
| Zara | am excited; these should speed the kanban up loads | 16:34 |
| SotK | needs webclient patches to actually get the benefit, which I'm still working on | 16:35 |
| Zara | ahh, still | 16:35 |
| Zara | this was the improvement I thought would take the longest | 16:36 |
| Zara | so either I'm horribly wrong or it's a good sign (as far as boards and worklists go, I mean) | 16:36 |
| Zara | I imagine search will take the actual longest | 16:36 |
| SotK | probably :) | 16:39 |
| Zara | this is tangential but I really like the way cards/lanes look btw. not that they've changed. I just like how straightforward and clear they are! | 16:40 |
| Zara | I think it's partly the colour scheme on my storyboard. anyway. | 16:41 |
| SotK | heh, I agree | 16:41 |
| SotK | I thought they were a bit plain at first, but I like them now | 16:41 |
| Zara | tbf I've always liked storyboard's aesthetic, and I like how well they fit in with it | 16:43 |
| Zara | ~oh storyboard, you are so beautiful~ | 16:43 |
| paulsherwood | it's looking good, but... could the 'add lane' button go up beside the title of the board? it's taking up space where it is | 16:43 |
| paulsherwood | and there's quite a lot of blank space after the board description, before the lanes start | 16:44 |
| paulsherwood | hobokan early users were very keen to have as much info available as possible without having to scroll | 16:44 |
| * SotK tries moving things around a bit | 16:48 | |
| Zara | I'm tempted to do it by having the title at the top, then the lanes, then the description (rather than squashing the description into the lanes). since you'd only get a little bit of space extra by squashing them together. | 16:48 |
| Zara | but I think that would look dramatically worse. and the description is probably only going to be read the first time a user uses the board; after that they'd be interested in the lanes. (alternatively, maybe a small description that can expand with a click?) | 16:49 |
| paulsherwood | win 39 | 16:51 |
| paulsherwood | feck | 16:51 |
| *** mrmartin has quit IRC | 16:51 | |
| Zara | I think we put 'add lane' there because we thought it was the first place someone would look for the button (see the existing lanes, realise one's missing), but if they're normally at the top in kanbans, then people will look at the top, so makes sense to me. | 16:51 |
| *** mrmartin has joined #storyboard | 16:53 | |
| openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient: Wrap long words in the "Items to add" table https://review.openstack.org/271430 | 17:09 |
| *** mrmartin has quit IRC | 17:18 | |
| Zara | I should look at the other patches from today, but getting pretty tired and worried I won't test them thoroughly enough | 17:18 |
| * SotK realises that https://review.openstack.org/271408 isn't quite finished | 17:19 | |
| Zara | tempted to change my +2 to a +1 on the other one for that reason | 17:19 |
| Zara | yeah, I'm going to change it I think, just so you know the reasoning there | 17:19 |
| *** mrmartin has joined #storyboard | 17:20 | |
| Zara | 'night, storyboard! been a lot of quiet work on boards and worklists this week (well, quiet except for SotK weeping, but that's normal); expect more visible patches next week! (heheh, I feel like I'm giving the weather forecast) I'll try to shift from coding to reviewing again so that we can get things merged faster. | 17:32 |
| Zara | right now, I need sleep, as you can probably tell. :) | 17:34 |
| -openstackstatus- NOTICE: Restarting zuul due to a memory leak | 17:51 | |
| *** mrmartin has quit IRC | 17:56 | |
| *** mrmartin has joined #storyboard | 18:07 | |
| *** mrmartin has quit IRC | 18:15 | |
| *** mrmartin has joined #storyboard | 19:18 | |
| *** mrmartin has quit IRC | 19:38 | |
| *** jtomasek has quit IRC | 21:18 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!