Wednesday, 2018-01-10

*** AlexeyAbashkin has joined #openstack-powervm00:01
*** AlexeyAbashkin has quit IRC00:05
*** gman-tx has joined #openstack-powervm02:03
*** gman-tx has quit IRC03:17
*** AlexeyAbashkin has joined #openstack-powervm03:52
*** AlexeyAbashkin has quit IRC03:57
*** chhavi__ has joined #openstack-powervm04:25
*** AlexeyAbashkin has joined #openstack-powervm04:25
*** AlexeyAbashkin has quit IRC04:29
*** edmondsw has joined #openstack-powervm06:00
*** edmondsw has quit IRC06:04
*** AlexeyAbashkin has joined #openstack-powervm07:57
*** AlexeyAbashkin has quit IRC11:18
*** AlexeyAbashkin has joined #openstack-powervm11:18
*** chhavi__ has quit IRC11:35
*** edmondsw has joined #openstack-powervm13:01
*** apearson__ has joined #openstack-powervm13:55
*** chhavi__ has joined #openstack-powervm14:34
*** gman-tx has joined #openstack-powervm14:56
-openstackstatus- NOTICE: Gerrit is being restarted due to slowness and to apply kernel patches14:58
*** openstackgerrit has quit IRC15:01
edmondswesberglu posted more comments on the vscsi patch15:55
edmondswI've reviewed all the non-test code at this point15:56
esbergluedmondsw: Ok. I ended up reworking test_vscsi. I'm finishing up the last few tests, should be done before lunch. Will address your comments in the next patch15:59
*** AlexeyAbashkin has quit IRC16:55
*** zerick has joined #openstack-powervm17:39
*** zerick has quit IRC17:41
*** zerick has joined #openstack-powervm17:42
*** zerick has quit IRC17:45
*** zerick has joined #openstack-powervm17:46
esbergluYou know the answer to that?18:05
*** openstack has joined #openstack-powervm18:31
*** ChanServ sets mode: +o openstack18:31
*** AlexeyAbashkin has joined #openstack-powervm18:31
*** AlexeyAbashkin has quit IRC18:35
esbergluBased on that it looks like that only applies to the live migration case. Just rip it out?18:51
esbergluedmondsw: test_vscsi is ready for review19:18
edmondswesberglu as long as we remember to put it back when we do port in live migration...19:26
edmondswand yeah, the OOT comment should help with that19:28
*** zerick has quit IRC19:51
*** zerick_ has joined #openstack-powervm19:51
efriedesberglu Responded, sorry for the delay.20:49
*** gman-tx has quit IRC20:56
*** gman-tx has joined #openstack-powervm20:57
*** AlexeyAbashkin has joined #openstack-powervm21:00
*** AlexeyAbashkin has quit IRC21:04
*** kylek3h has joined #openstack-powervm21:16
*** kylek3h has quit IRC21:38
*** chhavi__ has quit IRC21:41
edmondswesberglu finished reviewing the vscsi tests22:02
edmondswcomments were against an older PS (19)22:02
edmondswesberglu you around?22:20
esbergluedmondsw: Yeah22:20
edmondswgood catch on adding the support-matrix... I meant to comment on that and then forgot22:21
edmondswbut couple questions in the networking one22:21
edmondswyou said complete for security groups... did we do something for that?22:21
edmondswand if complete is correct for that, then we probably need to say the same for networking.firewallrules22:22
edmondswwhich appears to be a duplicate22:22
-openstackstatus- NOTICE: The zuul system is being restarted to apply security updates and will be offline for several minutes. It will be restarted and changes re-equeued; changes approved during the downtime will need to be rechecked or re-approved.22:23
edmondswesberglu ^22:25
edmondswesberglu other question would be does the networking support we've added work with nova-network as well as neutron, or just the latter?22:26
edmondswbecause I'm not sure if the networking stuff in the support matrix is supposed to just be re: nova-network22:27
edmondswat least networking.topology.vlan sounds that way22:27
edmondswthen again, probably better to say we have networking support there and be told to remove it than to not say it and someone things we don't have networking implemented22:28
esbergluedmondsw: security groups are supported using ovs22:38
esbergluI can mark the same for firewallrules22:38
edmondswsounds good22:38
edmondswit should be supported, but I didn't recall seeing anything related to that in the patch22:39
esbergluedmondsw: Nothing needed in our code afaik. Devstack creates a default group and I confirmed you can create your own22:40
edmondswwhat about the nova-network question... does our driver care whether you're using nova-network vs. neutron?22:40
esbergluI thought it had to be neutron? I haven't tried it with nova-network though22:41
esbergluIsn't nova-network deprecated?22:42
edmondswI suspect networking.topology.vlan is just wrong to specifically call out nova-network in its description22:42
edmondswmaybe throw up a separate patch to remove " when using nova-network" there?22:43
esbergluedmondsw: Sure22:44
edmondswwhen you've fixed the firewallrules one ping me and efried to re +122:45
edmondswand let's not make any more changes after that without talking first :)22:46
esbergluedmondsw: efried: Done22:47
efriedI haven't been following.  What action is required on my part?22:48
efried(my upstream to-do list blew up today)22:48
edmondswesberglu added support matrix changes to the OVS patch, so we need to rereview and +1 that and SEA (rebased)22:49
edmondswesberglu you rebased SEA, right?22:49
esbergluedmondsw: Yep, that doesn't lose +1 though22:49
edmondswoh, nice22:49
*** edmondsw has quit IRC22:59
*** apearson__ has quit IRC23:15
*** gman-tx has quit IRC23:18
*** efried has quit IRC23:44
*** edmondsw has joined #openstack-powervm23:55

Generated by 2.15.3 by Marius Gedminas - find it at!