*** catherineD has quit IRC | 00:12 | |
*** zehicle has joined #openstack-defcore | 02:39 | |
*** markvoelker has quit IRC | 05:32 | |
*** ig0r__ has joined #openstack-defcore | 05:52 | |
*** ig0r_ has quit IRC | 05:55 | |
*** markvoelker has joined #openstack-defcore | 06:32 | |
*** markvoelker has quit IRC | 06:37 | |
*** eglute has quit IRC | 07:30 | |
*** alex_klimov has joined #openstack-defcore | 07:39 | |
*** markvoelker has joined #openstack-defcore | 08:34 | |
*** markvoelker has quit IRC | 08:38 | |
*** alex_klimov has quit IRC | 10:27 | |
*** markvoelker has joined #openstack-defcore | 10:35 | |
*** alex_klimov has joined #openstack-defcore | 10:37 | |
*** markvoelker has quit IRC | 10:39 | |
*** markvoelker has joined #openstack-defcore | 11:35 | |
*** markvoelker has quit IRC | 11:40 | |
*** markvoelker has joined #openstack-defcore | 12:07 | |
*** ig0r__ has quit IRC | 12:17 | |
*** ig0r_ has joined #openstack-defcore | 12:23 | |
markvoelker | catherineD: Yes, hogepodge and I talked about that a bit earlier and we can probably arrange things better next time around. | 12:32 |
---|---|---|
markvoelker | chaterineD: 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 |
markvoelker | If 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 |
markvoelker | At 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 IRC | 12:38 | |
*** hockeynut has joined #openstack-defcore | 12:42 | |
*** zehicle has quit IRC | 13:07 | |
*** zehicle has joined #openstack-defcore | 13:28 | |
*** galstrom_zzz is now known as galstrom | 13:32 | |
*** zehicle has quit IRC | 13:37 | |
*** galstrom is now known as galstrom_zzz | 14:17 | |
*** galstrom_zzz is now known as galstrom | 14:26 | |
*** markvoelker has quit IRC | 14:48 | |
*** ig0r__ has joined #openstack-defcore | 15:19 | |
*** ig0r_ has quit IRC | 15:22 | |
*** galstrom is now known as galstrom_zzz | 15:35 | |
*** catherineD has joined #openstack-defcore | 15:35 | |
*** alex_klimov has quit IRC | 16:14 | |
*** markvoelker has joined #openstack-defcore | 16:31 | |
*** galstrom_zzz is now known as galstrom | 16:40 | |
*** galstrom is now known as galstrom_zzz | 16:42 | |
*** bitblt has joined #openstack-defcore | 17:01 | |
*** jmckind has joined #openstack-defcore | 17:25 | |
*** galstrom_zzz is now known as galstrom | 17:49 | |
*** galstrom is now known as galstrom_zzz | 18:49 | |
openstackgerrit | Merged openstack/defcore: Remove flag because of bug fix. https://review.openstack.org/189931 | 18:53 |
*** eglute has joined #openstack-defcore | 18:56 | |
eglute | hogepodge everyone is this ready to be merged? https://review.openstack.org/#/c/189927/ | 18:56 |
eglute | some comments asked to wait | 18:57 |
openstackgerrit | Merged openstack/defcore: Flag compute resize tests in 2015.05 https://review.openstack.org/196152 | 19:05 |
*** galstrom_zzz is now known as galstrom | 19:10 | |
markvoelker | eglute: 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 |
markvoelker | Given that this is .next though, I don't think there's any real hurry to merge it. | 19:10 |
eglute | ok, that works for me. | 19:10 |
markvoelker | E.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 |
eglute | cool! I just don't want people think I am ignoring their patches or comments! | 19:11 |
markvoelker | eglute: 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 |
eglute | yeah, i think I will hold off on merging any more right now, unless there is one that should be merged this week | 19:14 |
eglute | I will go over all the open patches before next meeting | 19:14 |
openstackgerrit | Mark T. Voelker proposed openstack/defcore: Flag test that is not widely supported across hypervisors https://review.openstack.org/196153 | 19:15 |
markvoelker | eglute: ^^ that's a rebase of one that conflicted, so it should be good to land again | 19:16 |
eglute | ok, and everyone agreed that we are moving all the flagged tests into 2015.5.json? | 19:17 |
openstackgerrit | Mark T. Voelker proposed openstack/defcore: Flag tests that depend upon SSH in 2015.05. https://review.openstack.org/196157 | 19:18 |
markvoelker | There are a couple we're not carrying into 2015.05 on account of tempest fixes | 19:18 |
*** ig0r__ has quit IRC | 19:19 | |
markvoelker | BUt 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 |
markvoelker | https://review.openstack.org/196157 is also now rebased | 19:19 |
*** ig0r_ has joined #openstack-defcore | 19:19 | |
eglute | thank you mark! | 19:25 |
openstackgerrit | Merged openstack/defcore: Flag test that is not widely supported across hypervisors https://review.openstack.org/196153 | 19:28 |
openstackgerrit | Mark T. Voelker proposed openstack/defcore: Flag tests that depend upon SSH in 2015.05. https://review.openstack.org/196157 | 19:31 |
markvoelker | eglute: ^^ rebased again | 19:31 |
eglute | sorry... that patch really does not want to be merged :) | 19:32 |
markvoelker | No worries. Bound to happen when carrying forward several flags at a time and it's completely trivial to rebase. | 19:33 |
eglute | right. which one should I merge next with least damage? | 19:34 |
markvoelker | It'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 |
markvoelker | No worries if you want others to comment on it before merging it though. | 19:37 |
markvoelker | https://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 |
eglute | what happened with the comments there? https://review.openstack.org/#/c/196154/ | 19:40 |
eglute | multiple submissions somehow? | 19:40 |
markvoelker | eglute: no idea, I think rocky must have refreshed the page while her browser was issuing a POST somehow or something. | 19:40 |
eglute | yeah, something didnt go properly | 19:41 |
openstackgerrit | Merged openstack/defcore: Flag tests that depend upon SSH in 2015.05. https://review.openstack.org/196157 | 19:42 |
markvoelker | hmm...looks like 196157 broke something. | 19:54 |
markvoelker | Oh I see | 19:54 |
markvoelker | trailing comma on a list | 19:54 |
markvoelker | nobody caught it in the reviews and I didn't catch it when writing it. =) One-character fix coming momentarily. | 19:54 |
openstackgerrit | Mark T. Voelker proposed openstack/defcore: Fix trailing comma in compute-volume list https://review.openstack.org/200695 | 19:57 |
markvoelker | eglute: ^^ | 19:57 |
markvoelker | We really should add a gate check for that sort of thing at some point... | 19:58 |
eglute | how do we add a gate check? | 19:58 |
markvoelker | We'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_zzz | 19:59 | |
eglute | cool. maybe next week. right now it is way too late here, or it feels late at least. | 20:00 |
markvoelker | eglute: where are you, anyway? =) | 20:00 |
eglute | Lithuania! We had snow yesterday. | 20:01 |
markvoelker | Must be a heck of a weather shock after summer in Texas. =) | 20:01 |
openstackgerrit | Merged openstack/defcore: Fix trailing comma in compute-volume list https://review.openstack.org/200695 | 20:02 |
openstackgerrit | Mark T. Voelker proposed openstack/defcore: Update procedue to reflect current status https://review.openstack.org/199799 | 20:04 |
markvoelker | Ok, 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 |
eglute | well, it was more like hail. But yes, in texas this would be considered winter weather. https://twitter.com/eglute/status/619597963866861568 | 20:05 |
markvoelker | eglute: nice. We got hail in a thunderstorm a week or so ago, but fortunately not enough to damage my tomato plants. =p | 20:06 |
markvoelker | Next week I'm off to Silicon Valley again so looking forward to a break from the heat/humidity though. | 20:07 |
eglute | i was surprised plants in my grandmother's garden were not damaged, or at least nobody said anything. that hail was really intense | 20:07 |
markvoelker | hmm...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 |
markvoelker | OK, fixed all the ones I saw. | 20:12 |
eglute | thank you markvoelker | 20:12 |
markvoelker | I have to run out to an appointment shortly. Have a good weekend everyone! | 20:12 |
markvoelker | Safe travels Egle | 20:13 |
openstackgerrit | Merged openstack/defcore: Update procedue to reflect current status https://review.openstack.org/199799 | 20:13 |
*** bitblt has quit IRC | 20:14 | |
eglute | thanks markvoelker!! | 20:15 |
*** jmckind has quit IRC | 20:22 | |
*** ig0r_ has quit IRC | 21:10 | |
*** purp has quit IRC | 22:53 | |
*** purp has joined #openstack-defcore | 22:53 | |
*** openstackgerrit has quit IRC | 23:39 | |
*** openstackgerrit has joined #openstack-defcore | 23:39 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!