Thursday, 2017-07-27

*** Tim_Eberhard has joined #openstack-fwaas00:23
*** Tim_Eberhard has quit IRC00:28
*** yamamoto_ has quit IRC01:31
*** yamamoto has joined #openstack-fwaas01:35
*** reedip_ has joined #openstack-fwaas03:00
openstackgerritReedip proposed openstack/neutron-fwaas master: Add additional test case for Action  https://review.openstack.org/45577203:03
*** reedip_ has quit IRC03:20
*** trungnv has quit IRC03:40
*** trungnv has joined #openstack-fwaas03:48
*** yushiro has joined #openstack-fwaas05:04
reedipyushiro : hi05:35
reedipwhen do we have a feature freeze for P-3  for FWaaS ?05:36
yushiroreedip, hi05:36
yushiroYesterday I tried to discuss with Sridar and xgerman_ about that but I forgot..05:37
yushirowill contact via e-mail on fwaas team.05:38
reedip:)05:38
reedipno problem , I just want to know the date so that we can05:39
reedipa) Run an assesment for stadium05:39
yushiroOK.05:39
reedipb) merge all items which are possible before the feature freeze for FWaaS05:39
*** vishwana_ has quit IRC05:40
*** vishwanathj has joined #openstack-fwaas05:41
reedipyushiro : we need vote from xgerman_ , SridarK has already given +2 to https://review.openstack.org/#/c/478883/. I dont think you can give  +W to it, right ?07:14
*** obre has quit IRC07:36
*** obre has joined #openstack-fwaas07:36
yushiroreedip, hi sorry for late07:38
yushiroreedip, yes, I think so.  It's 'self-approve'.07:39
yushiro+1 is OK :)07:40
yushiromaybe07:40
reedip:)07:52
yushiroxgerman_, Hi.  Could you please review  https://review.openstack.org/#/c/478883/ ?08:01
*** chandanc has joined #openstack-fwaas08:40
yushirochandanc, hi09:19
chandancHello yushiro09:23
yushirochandanc, did you sleep yesterday? :)09:23
chandancnot much ;)09:24
yushirochandanc, haha, me too. I'm a little sleepy09:24
chandanci am sterting the tests now09:24
chandancgot your patch applied09:25
yushirochandanc, Ok.  Now, I'm finding a mechanism to add 'tag' information into 'other_config' in OVSDB.09:25
chandancoh, and what is that09:26
yushiroIn driver layer (your patch set), get_tag_from_other_config() is called to get tag vlan id from ovsdb.09:26
chandancyes09:27
yushiroCurrent call timing, there is no 'tag' info in 'other_config' dict.09:28
chandancyes09:28
yushiroAs a result, it raises OVSFWTagNotFound09:28
chandancyes that is the race09:28
yushiroYes.  If we can find a point that a mechanism to insert 'tag' info into 'other_config' dict in OVSDB, no need to change in driver layer.09:29
chandancyes, i think that should be in the l2-agent09:30
yushirochandanc, agree.  However, now we can get tag vlan id from local vlan manager and insert 'lvlan' into port dict, right?09:31
chandanci think the extension is called after the lvlan is allocated ny l2-agent, but before the tag gets updated in ovsdb09:31
chandancyes, but we should be catefull nopt to create any regression for l2-agent itself09:32
chandanc*careful09:32
yushiroright.09:32
yushiroOK, let's keep on using lvlan from local vlan manager.09:32
chandancIf you find a slick way to achive this i will be with you09:32
yushiroYes, In fact, I don't want to change driver layer from neutron-side impl.09:33
chandancya, that can call for heavy review process09:34
yushiro+109:34
yushirohttps://etherpad.openstack.org/p/fwaas-v2-l2-agent  you can use this etherpad for free memo :)09:35
chandancyes i am following this09:36
chandancyushiro: i was thinking09:50
chandanc+    def _add_local_vlan_to_ports(self, fwg_ports):09:50
chandanc+        fwg_ports_with_lvlan = []09:50
chandanc+        for port in fwg_ports:09:50
chandanc+            try:09:50
chandanc+                network_id = self._get_network_id(port)09:50
chandanc+                port['lvlan'] = int(self.vlan_manager.get(network_id).vlan)09:50
chandanc+                fwg_ports_with_lvlan.append(port)09:50
chandanc+            except vlanmanager.MappingNotFound:09:50
chandanc+                pass09:50
chandanc+        return fwg_ports_with_lvlan09:50
chandancfor the skip ports without lvlan, what do you think09:50
yushirohmm, sorry for my question to your question.  Are you going to call driver.update_firewall_group() or something after passed ?09:51
chandancyes09:52
chandanci am updating your paste bin latest09:52
chandanchttp://paste.openstack.org/show/616596/09:52
chandanci also added the int cast that you mentined y’day09:53
yushiroyeah, but I'm afraid of calling of_port = OFPort(port, ovs_port, port_vlan_id)09:54
yushiroin spite of port_vlan_id is None09:54
yushiroIf we cannot get local vlan id, I think we cannot set ovs flow.09:54
chandancbut i am filtering thouse ports in this function09:54
yushirothis function is '_add_local_vlan_to_ports' ?09:55
chandancys09:55
chandancse the exception and return part09:55
yushirocould you check etherpad?09:55
yushiroI think if we cannot get loca lvlan id, no need to call driver side.09:56
chandancyes09:56
chandancupdating etherpad09:57
yushiromy current draft patch is light green09:57
yushirowatching09:57
*** yamamoto has quit IRC10:00
yushirochandanc, Thanks.  I understood what you'd like to do.10:05
chandancok will reply to pastebin10:05
yushirochandanc, could you tell me how to confirm ovs flow is set as expected?   ovs-ofctl dump-flows br-int and check table=41 or 43?10:11
yushiroIn addition, when VM removing, it raises in neutron-side OVSFWPortNotFound.  I pasted in the etherpad.10:16
chandancok let me check the code10:20
yushiroI think related table numbers are 41(base_egress), 42(rule_egress), 43(accept_or_ingress), 51(base_ingress) and 52(rules_ingress)10:26
yushiroI'm checking following command but not sure :)  sudo ovs-ofctl dump-flows br-int | grep -e table=41 -e table=42 -e table=43 -e table=51 -e table=5210:29
*** yamamoto has joined #openstack-fwaas10:31
chandancgot pulled into another task, Still in office10:38
chandancyushiro: cmd looks correct10:39
yushirochandanc, Wow, you look so busy.10:40
yushirochandanc, OK, thanks.10:40
*** chandanc has quit IRC11:06
*** vishwanathj has quit IRC11:26
*** vishwanathj has joined #openstack-fwaas11:26
*** chandanc has joined #openstack-fwaas12:11
*** yamamoto has quit IRC12:14
*** chandanc has quit IRC12:31
*** Tim_Eberhard has joined #openstack-fwaas13:12
*** yamamoto has joined #openstack-fwaas13:14
*** yamamoto has quit IRC13:21
*** Tim_Eberhard has quit IRC13:51
*** SridarK has joined #openstack-fwaas14:33
*** chandanc has joined #openstack-fwaas14:52
*** chandanc has quit IRC14:52
*** vishwanathj has quit IRC14:53
*** vishwanathj has joined #openstack-fwaas14:54
openstackgerritMerged openstack/neutron-fwaas master: Use API Definitions from neutron-lib  https://review.openstack.org/47888315:08
*** vishwanathj has quit IRC15:29
*** vishwanathj has joined #openstack-fwaas15:29
*** vishwana_ has joined #openstack-fwaas15:33
*** vishwanathj has quit IRC15:33
*** vishwana_ has quit IRC15:35
*** vishwanathj has joined #openstack-fwaas15:35
*** Tim_Eberhard has joined #openstack-fwaas16:11
*** Tim_Eberhard has quit IRC16:11
*** Tim_Eberhard has joined #openstack-fwaas16:12
*** Tim_Eberhard has quit IRC16:27
*** bzhao has joined #openstack-fwaas16:30
*** bbzhao has quit IRC16:33
*** SridarK has quit IRC17:10
*** SumitNaiksatam has joined #openstack-fwaas17:38
*** SumitNaiksatam has quit IRC17:56
*** SumitNaiksatam has joined #openstack-fwaas17:56
*** SumitNaiksatam has quit IRC19:07
*** vishwanathj has quit IRC19:34
*** vishwanathj has joined #openstack-fwaas19:35
*** Tim_Eberhard has joined #openstack-fwaas20:22
*** Tim_Eberhard has quit IRC20:23
*** Tim_Eberhard has joined #openstack-fwaas20:23
*** Tim_Eberhard has quit IRC20:37
*** vishwanathj has quit IRC21:01
*** vishwanathj has joined #openstack-fwaas21:02
*** yamamoto_ has joined #openstack-fwaas21:13
*** vishwanathj has quit IRC21:19
*** vishwanathj has joined #openstack-fwaas21:20
*** yamamoto_ has quit IRC21:20
*** vishwanathj has quit IRC21:27
*** vishwanathj has joined #openstack-fwaas21:28
*** vishwanathj has quit IRC21:28
*** vishwanathj has joined #openstack-fwaas21:29
*** vishwanathj has quit IRC21:32
*** vishwanathj has joined #openstack-fwaas21:33
*** yamamoto has joined #openstack-fwaas22:04
*** openstack has joined #openstack-fwaas22:43
*** Tim_Eberhard has joined #openstack-fwaas23:56

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