Thursday, 2017-09-07

*** yamamoto_ has joined #openstack-fwaas02:09
*** yamamot__ has joined #openstack-fwaas04:20
*** mfranc213 has quit IRC04:20
*** mfranc213 has joined #openstack-fwaas04:21
*** yamamoto_ has quit IRC04:21
*** lnicolas has quit IRC04:59
*** yamamot__ has quit IRC05:07
*** yamamoto_ has joined #openstack-fwaas05:07
openstackgerritReedip proposed openstack/neutron-fwaas master: Improve test cases for distributed routers  https://review.openstack.org/50156705:10
reedipxgerman_ ping05:17
openstackgerritReedip proposed openstack/neutron-fwaas master: Improve test cases for distributed routers  https://review.openstack.org/50156705:25
reedipyamamoto_ : u there?05:33
yamamoto_?05:33
reedipHi05:33
reedipyamamoto_ : Currently fwaas is not linked directly with neutron , IIUC05:34
reedipyamamoto_ : is there a way to verify that if there is a change in neutron, we can map it onto fwaas as well ? I mean if there is a change in neutron's code, but the same change has not occurred in fwaas, then notification ( like a test failure ) should be generated05:35
*** yushiro has joined #openstack-fwaas05:35
openstackgerritReedip proposed openstack/neutron-fwaas master: Update DVR condition in FWaaS  https://review.openstack.org/50157005:39
amotokireedip: what would you really like to do? do you want to test if fwaas works fine using some neutron side change ?05:40
reedipamotoki : yes05:40
amotokireedip: simply Depends-On05:40
amotokireedip: you can send a test patch to neutron-fwaas with a proper Depends-On05:41
reedipamotoki : umm , thats true, but not what I wanted to know.05:41
amotokireedip: so perhaps i don't understand what you want correctly05:41
reedipamotoki : just a minute05:42
reedipamotoki : like 3162846a7b42d860533df6dc32d9c553669df45b changed05:42
reedipin neutron, but fwaas which uses similar code didnt know about it. I think the best way to handle such things is with functional testing or fullstack testing05:43
reedipamotoki : but still need your opinion05:43
reedipamotoki : bug 1715395 was raised yesterday because Commit ID: # 3162846a7b42d860533df6dc32d9c553669df45b which changed in neutron was not reflected in fwaas05:44
openstackbug 1715395 in neutron "FWaaS: Firewall creation fails in case of distributed routers (Pike)" [High,In progress] https://launchpad.net/bugs/1715395 - Assigned to Reedip (reedip-banerjee)05:44
amotokireedip: it depends on cases. does fwaas impl depend on the internal details of l3-agent or non-well-defined methods05:46
amotoki?05:46
reedipamotoki : loosly dependent  but yes dependent05:46
amotokiif so, this kind of things cannot be avoided theoretically, and we need to explore what is more modular appraoch05:47
reedipamotoki : ok, I think improving the secnarios might be the best way forward, atleast having Fullstack tests05:47
amotokithis needs a feedback from out-of-tree users like fwaas05:47
reedipok05:47
amotokiwe are in the early stage of a dev cycle, and it is nice to have this kind of thing now :)05:48
reedipamotoki : I think a similar approach was used by boden for migration of neutron stuff to neutron-lib and he raised emails to all consumers. I think such a model works atleast to a good degree05:49
amotokireedip: yes, in neutron-lib case, we confidently know we break the current interface and recommend to use new interfaces05:50
amotokireedip: but this case is a bit different. we were not aware of breaking something AND did not think it is a public interface change05:50
amotokireedip: we thought it was just an usual neutron-internal change05:51
reedipamotoki : yes, I agree with that. The issue is that neutron change didnot consider that someone else would be using the attributes of the object directly05:51
amotokireedip: it is what we have now, but it is not a good thing to access internals05:52
reedipamotoki: so this bug is valid, but to prevent future issues, atleast to a certain degree, some method can be found. Obviously neutron cannot announce all its changes to out-of-tree consumers05:52
amotokireedip: so the stadium health scorecard has this entry N3 http://specs.openstack.org/openstack/neutron-specs/specs/stadium/ocata/neutron-fwaas.html#n305:54
reedipcheckin05:57
reedipamotoki : ok ...06:09
yamamoto_reedip: you can review every neutron patches and put -1 for breaking ones. :-)06:18
reedipyamamoto_ : thats a great idea :D06:18
*** eezhova has joined #openstack-fwaas06:49
*** yushiro has quit IRC06:56
*** eezhova has quit IRC07:45
*** eezhova has joined #openstack-fwaas08:09
*** yamamoto_ has quit IRC09:28
reedipamotoki : there?09:42
*** yamamoto has joined #openstack-fwaas09:46
*** yamamoto has quit IRC10:14
*** yamamoto has joined #openstack-fwaas10:19
openstackgerritReedip proposed openstack/neutron-fwaas master: Update fwaas configuration for user-defined L3 Agents  https://review.openstack.org/50166310:19
amotokireedip: what?10:42
reedip  amotoki : never mind, resolved the isue, thanks :)10:43
*** reedip is now known as reed_afk10:53
*** reed_afk is now known as reedip_afk10:53
openstackgerritInessa Vasilevskaya proposed openstack/neutron-fwaas master: Generate default firewall group via project  https://review.openstack.org/42576911:55
*** eezhova_ has joined #openstack-fwaas12:12
*** eezhova has quit IRC12:15
openstackgerritÉdouard Thuleau proposed openstack/neutron-fwaas master: Implements a plugable backend driver  https://review.openstack.org/48026513:35
*** eezhova_ has quit IRC13:44
*** eezhova has joined #openstack-fwaas14:25
xgerman_reedip what’s up>?14:30
*** eezhova has quit IRC15:05
*** lnicolas has joined #openstack-fwaas15:09
*** reedip has joined #openstack-fwaas15:33
reedip0/15:38
openstackgerritReedip proposed openstack/neutron-fwaas master: Update DVR condition in FWaaS  https://review.openstack.org/50157015:40
openstackgerritÉdouard Thuleau proposed openstack/neutron-fwaas master: Implements a plugable backend driver  https://review.openstack.org/48026515:41
reediphi doude : fixed the conflicts?15:45
*** reedip is now known as outofmemory15:59
doudeyes reedip_afk16:06
doudebut I was not able to run tempest on my devstack. I  don't understand why but the scenario fails here https://github.com/openstack/neutron-fwaas/blob/master/neutron_fwaas/tests/tempest_plugin/tests/scenario/test_fwaas_v2.py#L21716:07
*** mwynne has joined #openstack-fwaas16:08
doudeeffectively, the VM cannot be reach through its floating IP16:08
doudeI check and the VM receive correctly ICMP request and send a ack (I see that on the VM tap), but the icmp ack is lost somewhere in the OVS mess (qbr, qvo...)16:09
doudeI set up a classic devstack install16:10
doudeI'll see if CI fails16:10
*** eezhova has joined #openstack-fwaas16:19
outofmemorydoude : no issues, I will also pull it up tomorrow :) ( reedip here )16:36
doudeit seems the gate dsvm-tempest succeed with the last patch set16:37
doudeI mean gate-neutron-fwaas-v2-dsvm-tempest gate, the v1 not yet finished16:38
doudeand v2 multinode also succeed16:38
outofmemorygood :)16:52
*** eezhova has quit IRC16:55
*** outofmemory has quit IRC17:08
*** SumitNaiksatam has joined #openstack-fwaas17:19
doudeall green17:36
*** eezhova has joined #openstack-fwaas18:54
*** eezhova has quit IRC18:55
*** SumitNaiksatam has quit IRC19:10
*** eezhova has joined #openstack-fwaas19:24
*** eezhova has quit IRC19:56
*** eezhova has joined #openstack-fwaas20:03
*** eezhova has quit IRC21:19
*** yushiro has joined #openstack-fwaas23:39

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