Thursday, 2015-07-23

*** galstrom is now known as galstrom_zzz00:01
*** catherineD has joined #openstack-defcore00:22
catherineDHi eglute:  I double checked the new capability names in https://docs.google.com/document/d/1Bx5A9tdaL4mIHXehdfgptqxhH-tzjUP6CZY3IBHyn9I/edit ... all looks good  ...00:26
eglutethank you catherineD!00:26
*** catherine_d|1 has joined #openstack-defcore01:52
openstackgerritMerged openstack/defcore: Update RST document formatting  https://review.openstack.org/20283303:50
*** ChanServ changes topic to "Coming up: MidCycle in Austin! https://etherpad.openstack.org/p/DefCoreFlag.MidCycle"04:17
*** catherine_d|1 has quit IRC05:25
*** alex_klimov has joined #openstack-defcore07:46
*** alex_klimov has quit IRC08:26
*** alex_klimov has joined #openstack-defcore08:37
openstackgerritMark T. Voelker proposed openstack/defcore: Drop flag on tests that require SSH  https://review.openstack.org/18992712:46
openstackgerritMark T. Voelker proposed openstack/defcore: Drop flag on tests that require SSH  https://review.openstack.org/18992712:47
markvoelkerglute: zehicle: Looking over outstanding patches against .next this morning.  This one has been sitting for two months with only hogepodge/galstrom/myself chiming in on it.13:00
markvoelkerCould you have a look when you get a chance?  https://review.openstack.org/#/c/18705813:00
markvoelkereglute: ^^13:00
eglutemarkvoelker right now jenkins is not happy with it13:26
markvoelkerEglute: yes, but I fear it's not been rebased due to lack of review attention...I'd be happy to rebase it if folks agree that it's a desirable change.13:26
markvoelkerThe rebase is quite trivial.13:27
egluteyes, if you rebase, i will merge.13:27
openstackgerritMark T. Voelker proposed openstack/defcore: Add test_list_get_volume_attachments under compute-volume  https://review.openstack.org/18705813:28
markvoelkereglute: ^^13:28
eglutedone! thank you! sorry for the delay13:30
eglutei think this one is ready as well, right? https://review.openstack.org/#/c/189927/13:30
markvoelkeryeah, it should be13:33
openstackgerritMerged openstack/defcore: Add test_list_get_volume_attachments under compute-volume  https://review.openstack.org/18705813:33
openstackgerritMerged openstack/defcore: Drop flag on tests that require SSH  https://review.openstack.org/18992713:36
markvoelkereglute: opinions on https://review.openstack.org/#/c/196155/ would be welcome too...it's a carry-forward to 2015.05 that's been out for almost a month with 1 review13:38
eglutewas just going to ask about it13:39
eglutesorry, i should have posted something, i agree with you markvoelker on it. if hogepodge is ok, i want to merge it13:39
markvoelkerok, if you want to leave a +1 in gerrit we will circle back w/hogepodge when he's online13:40
eglutealso, I am in favor of merging this, if zehicle zehicle_IRL hogepodge are ok with it. Needs rebase, but i left a comment https://review.openstack.org/#/c/189979/13:46
markvoelkereglute: yeah, we did actually discuss keeping a list of permanently removed tests a meeting or two ago, but I don't think hogepodge has had a chance to do it yet.  I'd be fine with merging that to get it done for .next before it goes to the Board though.13:48
egluteok, that  soudns good to me13:49
markvoelkerI think we've had a total of maybe 2 or 3 removals (including that one) so it shouldn't be hard to create the list later in the interest of getting a clean spec up to the Board now.13:49
egluteagree13:51
eglutethis needs rebase as well https://review.openstack.org/#/c/189867/ if you could rebase it i could merge it13:51
markvoelkereglute: sure13:54
eglutethank you markvoelker sorry i should have addressed these earlier. I try to go over them regularly, but if i see two -1, i leave it uncommented.13:55
markvoelkerTotally understandable, but our list of regular reviewers is so small that it would be good to leave comments/votes whenever possible if you have the time. =)13:58
markvoelkerI think we only have four people who've reviewed more than 10 patches in the entire Liberty cycle so far...13:59
egluteagree! i think i would wait cause i been known to merge things too quickly ;)13:59
markvoelker=)  Always a fine balancing act.13:59
egluteyes... and i really appreciate all the work you put in keeping us patched14:00
egluteand on top of things in general!14:01
markvoelkerhappy to be of service14:02
eglute:D14:02
openstackgerritMark T. Voelker proposed openstack/defcore: Flag Compute tests that rely on the Glance V1 in 2015.next  https://review.openstack.org/18986714:27
openstackgerritMark T. Voelker proposed openstack/defcore: Flag Compute tests that rely on the Glance V1 in 2015.next  https://review.openstack.org/18986714:29
markvoelkerdarned whitespaces14:31
openstackgerritMark T. Voelker proposed openstack/defcore: Flag Compute tests that rely on the Glance V1 in 2015.next  https://review.openstack.org/18986714:31
markvoelkereglute: ^^ that should be rebased now.  It's sort of a longish one so double check that I got everything.14:31
*** alex_klimov has quit IRC15:00
eglutethank you markvoelker! it looked good to me15:02
openstackgerritMerged openstack/defcore: Flag Compute tests that rely on the Glance V1 in 2015.next  https://review.openstack.org/18986715:09
openstackgerritMark T. Voelker proposed openstack/defcore: Update supplemental files for 2015.05  https://review.openstack.org/20512615:51
catherineDJust a note on https://review.openstack.org/#/c/187058/ ... This patch adds a test to replace tempest.api.compute.volumes.test_attach_volume.AttachVolumeTestJSON.test_attach_detach_volume which was previousely flagged due to Tempest SSH test issue.  However  with https://review.openstack.org/#/c/189927/ (SSH fixed) merged, all flagged tests due to SSH issues are no longer flagged.16:58
*** zehicle_IRL has quit IRC17:06
*** dwalleck has joined #openstack-defcore17:42
*** dwalleck has quit IRC17:43
*** dwalleck has joined #openstack-defcore17:43
*** dwalleck has quit IRC17:50
*** dwalleck has joined #openstack-defcore18:04
*** dwalleck has quit IRC18:16
*** dwalleck has joined #openstack-defcore18:23
*** zehicle_IRL has joined #openstack-defcore18:44
*** dwalleck has quit IRC18:45
*** dwalleck has joined #openstack-defcore18:46
*** dwalleck has quit IRC18:54
*** dwalleck has joined #openstack-defcore18:55
*** bitblt has joined #openstack-defcore18:55
*** bitblt has quit IRC18:58
*** dwalleck has quit IRC18:58
*** dwalleck has joined #openstack-defcore19:00
*** dwalleck has quit IRC19:02
*** dwalleck has joined #openstack-defcore19:03
*** zehicle_IRL has quit IRC19:04
*** dwalleck has quit IRC19:05
*** galstrom_zzz is now known as galstrom19:09
*** catherineD|2 has joined #openstack-defcore19:41
*** catherineD has quit IRC19:44
*** hockeynut_afk has joined #openstack-defcore19:55
*** dwalleck has joined #openstack-defcore20:07
*** dwalleck has quit IRC20:36
*** markvoelker has quit IRC20:42
*** VanL has joined #openstack-defcore21:01
*** VanL has quit IRC21:30
*** VanL has joined #openstack-defcore21:42
*** galstrom is now known as galstrom_zzz22:52
*** VanL has quit IRC23:34
openstackgerritRob Hirschfeld proposed openstack/defcore: Add comments that limit removing flags from Guidelines.  https://review.openstack.org/20093923:51

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