Friday, 2015-07-10

*** catherineD has quit IRC00:12
*** zehicle has joined #openstack-defcore02:39
*** markvoelker has quit IRC05:32
*** ig0r__ has joined #openstack-defcore05:52
*** ig0r_ has quit IRC05:55
*** markvoelker has joined #openstack-defcore06:32
*** markvoelker has quit IRC06:37
*** eglute has quit IRC07:30
*** alex_klimov has joined #openstack-defcore07:39
*** markvoelker has joined #openstack-defcore08:34
*** markvoelker has quit IRC08:38
*** alex_klimov has quit IRC10:27
*** markvoelker has joined #openstack-defcore10:35
*** alex_klimov has joined #openstack-defcore10:37
*** markvoelker has quit IRC10:39
*** markvoelker has joined #openstack-defcore11:35
*** markvoelker has quit IRC11:40
*** markvoelker has joined #openstack-defcore12:07
*** ig0r__ has quit IRC12:17
*** ig0r_ has joined #openstack-defcore12:23
markvoelkercatherineD: Yes, hogepodge and I talked about that a bit earlier and we can probably arrange things better next time around.12:32
markvoelkerchaterineD: in the meantime we have this problemw here they're not getting merged because they haven't got reviews, so having some actual reviews now is needed. =)12:33
markvoelkerIf a trivial rebase is needed, I think the Rob/Egle will probably be able to note the previous +1's/+2's and not be afraid to merge it before they're all re-accumulated.12:33
markvoelkerAt this point we're down to a handful of them, so babysitting them isn't too hard.  Would definitely be more work if there were a lot more of them though.12:34
*** hockeynut has quit IRC12:38
*** hockeynut has joined #openstack-defcore12:42
*** zehicle has quit IRC13:07
*** zehicle has joined #openstack-defcore13:28
*** galstrom_zzz is now known as galstrom13:32
*** zehicle has quit IRC13:37
*** galstrom is now known as galstrom_zzz14:17
*** galstrom_zzz is now known as galstrom14:26
*** markvoelker has quit IRC14:48
*** ig0r__ has joined #openstack-defcore15:19
*** ig0r_ has quit IRC15:22
*** galstrom is now known as galstrom_zzz15:35
*** catherineD has joined #openstack-defcore15:35
*** alex_klimov has quit IRC16:14
*** markvoelker has joined #openstack-defcore16:31
*** galstrom_zzz is now known as galstrom16:40
*** galstrom is now known as galstrom_zzz16:42
*** bitblt has joined #openstack-defcore17:01
*** jmckind has joined #openstack-defcore17:25
*** galstrom_zzz is now known as galstrom17:49
*** galstrom is now known as galstrom_zzz18:49
openstackgerritMerged openstack/defcore: Remove flag because of bug fix.  https://review.openstack.org/18993118:53
*** eglute has joined #openstack-defcore18:56
eglutehogepodge everyone is this ready to be merged? https://review.openstack.org/#/c/189927/18:56
eglutesome comments asked to wait18:57
openstackgerritMerged openstack/defcore: Flag compute resize tests in 2015.05  https://review.openstack.org/19615219:05
*** galstrom_zzz is now known as galstrom19:10
markvoelkereglute: I'm on the fence.  The BP referenced there has a host of changes that need to go in, and more than one is still outstanding.19:10
markvoelkerGiven that this is .next though, I don't think there's any real hurry to merge it.19:10
egluteok, that works for me.19:10
markvoelkerE.g. that Guideline won't be rolled out for months yet, so may as well give the tempest folks time to finish out the blueprint.19:10
eglutecool! I just don't want people think I am ignoring their patches or comments!19:11
markvoelkereglute: On that note, thanks for merging some patches today. =)  I think a few will probably fail since we're landing a couple at once and they'll create conflicts.  I'll rebase those.19:13
markvoelker(pretty trivial to do)19:13
egluteyeah, i think I will hold off on merging any more right now, unless there is one that should be merged this week19:14
egluteI will go over all the open patches before next meeting19:14
openstackgerritMark T. Voelker proposed openstack/defcore: Flag test that is not widely supported across hypervisors  https://review.openstack.org/19615319:15
markvoelkereglute: ^^ that's a rebase of one that conflicted, so it should be good to land again19:16
egluteok, and everyone agreed that we are moving all the flagged tests into 2015.5.json?19:17
openstackgerritMark T. Voelker proposed openstack/defcore: Flag tests that depend upon SSH in 2015.05.  https://review.openstack.org/19615719:18
markvoelkerThere are a couple we're not carrying into 2015.05 on account of tempest fixes19:18
*** ig0r__ has quit IRC19:19
markvoelkerBUt most of the rest still apply per the commit messages in gerrit. THose changes should have multiple reviewers now for the most part w/+1's and/or +2's and no -1's.19:19
markvoelkerhttps://review.openstack.org/196157 is also now rebased19:19
*** ig0r_ has joined #openstack-defcore19:19
eglutethank you mark!19:25
openstackgerritMerged openstack/defcore: Flag test that is not widely supported across hypervisors  https://review.openstack.org/19615319:28
openstackgerritMark T. Voelker proposed openstack/defcore: Flag tests that depend upon SSH in 2015.05.  https://review.openstack.org/19615719:31
markvoelkereglute: ^^ rebased again19:31
eglutesorry... that patch really does not want to be merged :)19:32
markvoelkerNo worries.  Bound to happen when carrying forward several flags at a time and it's completely trivial to rebase.19:33
egluteright. which one should I merge next with least damage?19:34
markvoelkerIt'd be useful to at least review this one (it only had a review from rockyg so far): https://review.openstack.org/#/c/196154/19:36
markvoelkerNo worries if you want others to comment on it before merging it though.19:37
markvoelkerhttps://review.openstack.org/#/c/199799/ would be good too, it's just a trivial doc fixup.  But let's wait a few minutes for 196157 to land and I'll re-run the scripts.19:38
eglutewhat happened with the comments there? https://review.openstack.org/#/c/196154/19:40
eglutemultiple submissions somehow?19:40
markvoelkereglute: no idea, I think rocky must have refreshed the page while her browser was issuing a POST somehow or something.19:40
egluteyeah, something didnt go properly19:41
openstackgerritMerged openstack/defcore: Flag tests that depend upon SSH in 2015.05.  https://review.openstack.org/19615719:42
markvoelkerhmm...looks like 196157 broke something.19:54
markvoelkerOh I see19:54
markvoelkertrailing comma on a list19:54
markvoelkernobody caught it in the reviews and I didn't catch it when writing it. =)  One-character fix coming momentarily.19:54
openstackgerritMark T. Voelker proposed openstack/defcore: Fix trailing comma in compute-volume list  https://review.openstack.org/20069519:57
markvoelkereglute: ^^19:57
markvoelkerWe really should add a gate check for that sort of thing at some point...19:58
eglutehow do we add a gate check?19:58
markvoelkerWe'd need to send a patch to infra to set it up.  Don't have the repo pointer handy at the moment, but it's doable with some legwork.19:59
*** galstrom is now known as galstrom_zzz19:59
eglutecool. maybe next week. right now it is way too late here, or it feels late at least.20:00
markvoelkereglute: where are you, anyway? =)20:00
egluteLithuania! We had snow yesterday.20:01
markvoelkerMust be a heck of a weather shock after summer in Texas. =)20:01
openstackgerritMerged openstack/defcore: Fix trailing comma in compute-volume list  https://review.openstack.org/20069520:02
openstackgerritMark T. Voelker proposed openstack/defcore: Update procedue to reflect current status  https://review.openstack.org/19979920:04
markvoelkerOk, https://review.openstack.org/#/c/199799/ should be good now.  I re-ran the script so it includes all the flags that just landed.20:04
eglutewell, it was more like hail. But yes, in texas this would be considered winter weather. https://twitter.com/eglute/status/61959796386686156820:05
markvoelkereglute: nice.  We got hail in a thunderstorm a week or so ago, but fortunately not enough to damage my tomato plants. =p20:06
markvoelkerNext week I'm off to Silicon Valley again so looking forward to a break from the heat/humidity though.20:07
eglutei was surprised plants in my grandmother's garden were not damaged, or at least nobody said anything. that hail was really intense20:07
markvoelkerhmm...I just noticed that when we moved all the process docs around, we broke a bunch of links on the wiki page.  I'll go fix those before I sign off.20:08
markvoelkerOK, fixed all the ones I saw.20:12
eglutethank you markvoelker20:12
markvoelkerI have to run out to an appointment shortly.  Have a good weekend everyone!20:12
markvoelkerSafe travels Egle20:13
openstackgerritMerged openstack/defcore: Update procedue to reflect current status  https://review.openstack.org/19979920:13
*** bitblt has quit IRC20:14
eglutethanks markvoelker!!20:15
*** jmckind has quit IRC20:22
*** ig0r_ has quit IRC21:10
*** purp has quit IRC22:53
*** purp has joined #openstack-defcore22:53
*** openstackgerrit has quit IRC23:39
*** openstackgerrit has joined #openstack-defcore23:39

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