*** Tim_Eberhard has joined #openstack-fwaas | 00:23 | |
*** Tim_Eberhard has quit IRC | 00:28 | |
*** yamamoto_ has quit IRC | 01:31 | |
*** yamamoto has joined #openstack-fwaas | 01:35 | |
*** reedip_ has joined #openstack-fwaas | 03:00 | |
openstackgerrit | Reedip proposed openstack/neutron-fwaas master: Add additional test case for Action https://review.openstack.org/455772 | 03:03 |
---|---|---|
*** reedip_ has quit IRC | 03:20 | |
*** trungnv has quit IRC | 03:40 | |
*** trungnv has joined #openstack-fwaas | 03:48 | |
*** yushiro has joined #openstack-fwaas | 05:04 | |
reedip | yushiro : hi | 05:35 |
reedip | when do we have a feature freeze for P-3 for FWaaS ? | 05:36 |
yushiro | reedip, hi | 05:36 |
yushiro | Yesterday I tried to discuss with Sridar and xgerman_ about that but I forgot.. | 05:37 |
yushiro | will contact via e-mail on fwaas team. | 05:38 |
reedip | :) | 05:38 |
reedip | no problem , I just want to know the date so that we can | 05:39 |
reedip | a) Run an assesment for stadium | 05:39 |
yushiro | OK. | 05:39 |
reedip | b) merge all items which are possible before the feature freeze for FWaaS | 05:39 |
*** vishwana_ has quit IRC | 05:40 | |
*** vishwanathj has joined #openstack-fwaas | 05:41 | |
reedip | yushiro : 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 IRC | 07:36 | |
*** obre has joined #openstack-fwaas | 07:36 | |
yushiro | reedip, hi sorry for late | 07:38 |
yushiro | reedip, yes, I think so. It's 'self-approve'. | 07:39 |
yushiro | +1 is OK :) | 07:40 |
yushiro | maybe | 07:40 |
reedip | :) | 07:52 |
yushiro | xgerman_, Hi. Could you please review https://review.openstack.org/#/c/478883/ ? | 08:01 |
*** chandanc has joined #openstack-fwaas | 08:40 | |
yushiro | chandanc, hi | 09:19 |
chandanc | Hello yushiro | 09:23 |
yushiro | chandanc, did you sleep yesterday? :) | 09:23 |
chandanc | not much ;) | 09:24 |
yushiro | chandanc, haha, me too. I'm a little sleepy | 09:24 |
chandanc | i am sterting the tests now | 09:24 |
chandanc | got your patch applied | 09:25 |
yushiro | chandanc, Ok. Now, I'm finding a mechanism to add 'tag' information into 'other_config' in OVSDB. | 09:25 |
chandanc | oh, and what is that | 09:26 |
yushiro | In driver layer (your patch set), get_tag_from_other_config() is called to get tag vlan id from ovsdb. | 09:26 |
chandanc | yes | 09:27 |
yushiro | Current call timing, there is no 'tag' info in 'other_config' dict. | 09:28 |
chandanc | yes | 09:28 |
yushiro | As a result, it raises OVSFWTagNotFound | 09:28 |
chandanc | yes that is the race | 09:28 |
yushiro | Yes. 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 |
chandanc | yes, i think that should be in the l2-agent | 09:30 |
yushiro | chandanc, agree. However, now we can get tag vlan id from local vlan manager and insert 'lvlan' into port dict, right? | 09:31 |
chandanc | i think the extension is called after the lvlan is allocated ny l2-agent, but before the tag gets updated in ovsdb | 09:31 |
chandanc | yes, but we should be catefull nopt to create any regression for l2-agent itself | 09:32 |
chandanc | *careful | 09:32 |
yushiro | right. | 09:32 |
yushiro | OK, let's keep on using lvlan from local vlan manager. | 09:32 |
chandanc | If you find a slick way to achive this i will be with you | 09:32 |
yushiro | Yes, In fact, I don't want to change driver layer from neutron-side impl. | 09:33 |
chandanc | ya, that can call for heavy review process | 09:34 |
yushiro | +1 | 09:34 |
yushiro | https://etherpad.openstack.org/p/fwaas-v2-l2-agent you can use this etherpad for free memo :) | 09:35 |
chandanc | yes i am following this | 09:36 |
chandanc | yushiro: i was thinking | 09: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 | + pass | 09:50 |
chandanc | + return fwg_ports_with_lvlan | 09:50 |
chandanc | for the skip ports without lvlan, what do you think | 09:50 |
yushiro | hmm, sorry for my question to your question. Are you going to call driver.update_firewall_group() or something after passed ? | 09:51 |
chandanc | yes | 09:52 |
chandanc | i am updating your paste bin latest | 09:52 |
chandanc | http://paste.openstack.org/show/616596/ | 09:52 |
chandanc | i also added the int cast that you mentined y’day | 09:53 |
yushiro | yeah, but I'm afraid of calling of_port = OFPort(port, ovs_port, port_vlan_id) | 09:54 |
yushiro | in spite of port_vlan_id is None | 09:54 |
yushiro | If we cannot get local vlan id, I think we cannot set ovs flow. | 09:54 |
chandanc | but i am filtering thouse ports in this function | 09:54 |
yushiro | this function is '_add_local_vlan_to_ports' ? | 09:55 |
chandanc | ys | 09:55 |
chandanc | se the exception and return part | 09:55 |
yushiro | could you check etherpad? | 09:55 |
yushiro | I think if we cannot get loca lvlan id, no need to call driver side. | 09:56 |
chandanc | yes | 09:56 |
chandanc | updating etherpad | 09:57 |
yushiro | my current draft patch is light green | 09:57 |
yushiro | watching | 09:57 |
*** yamamoto has quit IRC | 10:00 | |
yushiro | chandanc, Thanks. I understood what you'd like to do. | 10:05 |
chandanc | ok will reply to pastebin | 10:05 |
yushiro | chandanc, 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 |
yushiro | In addition, when VM removing, it raises in neutron-side OVSFWPortNotFound. I pasted in the etherpad. | 10:16 |
chandanc | ok let me check the code | 10:20 |
yushiro | I think related table numbers are 41(base_egress), 42(rule_egress), 43(accept_or_ingress), 51(base_ingress) and 52(rules_ingress) | 10:26 |
yushiro | I'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=52 | 10:29 |
*** yamamoto has joined #openstack-fwaas | 10:31 | |
chandanc | got pulled into another task, Still in office | 10:38 |
chandanc | yushiro: cmd looks correct | 10:39 |
yushiro | chandanc, Wow, you look so busy. | 10:40 |
yushiro | chandanc, OK, thanks. | 10:40 |
*** chandanc has quit IRC | 11:06 | |
*** vishwanathj has quit IRC | 11:26 | |
*** vishwanathj has joined #openstack-fwaas | 11:26 | |
*** chandanc has joined #openstack-fwaas | 12:11 | |
*** yamamoto has quit IRC | 12:14 | |
*** chandanc has quit IRC | 12:31 | |
*** Tim_Eberhard has joined #openstack-fwaas | 13:12 | |
*** yamamoto has joined #openstack-fwaas | 13:14 | |
*** yamamoto has quit IRC | 13:21 | |
*** Tim_Eberhard has quit IRC | 13:51 | |
*** SridarK has joined #openstack-fwaas | 14:33 | |
*** chandanc has joined #openstack-fwaas | 14:52 | |
*** chandanc has quit IRC | 14:52 | |
*** vishwanathj has quit IRC | 14:53 | |
*** vishwanathj has joined #openstack-fwaas | 14:54 | |
openstackgerrit | Merged openstack/neutron-fwaas master: Use API Definitions from neutron-lib https://review.openstack.org/478883 | 15:08 |
*** vishwanathj has quit IRC | 15:29 | |
*** vishwanathj has joined #openstack-fwaas | 15:29 | |
*** vishwana_ has joined #openstack-fwaas | 15:33 | |
*** vishwanathj has quit IRC | 15:33 | |
*** vishwana_ has quit IRC | 15:35 | |
*** vishwanathj has joined #openstack-fwaas | 15:35 | |
*** Tim_Eberhard has joined #openstack-fwaas | 16:11 | |
*** Tim_Eberhard has quit IRC | 16:11 | |
*** Tim_Eberhard has joined #openstack-fwaas | 16:12 | |
*** Tim_Eberhard has quit IRC | 16:27 | |
*** bzhao has joined #openstack-fwaas | 16:30 | |
*** bbzhao has quit IRC | 16:33 | |
*** SridarK has quit IRC | 17:10 | |
*** SumitNaiksatam has joined #openstack-fwaas | 17:38 | |
*** SumitNaiksatam has quit IRC | 17:56 | |
*** SumitNaiksatam has joined #openstack-fwaas | 17:56 | |
*** SumitNaiksatam has quit IRC | 19:07 | |
*** vishwanathj has quit IRC | 19:34 | |
*** vishwanathj has joined #openstack-fwaas | 19:35 | |
*** Tim_Eberhard has joined #openstack-fwaas | 20:22 | |
*** Tim_Eberhard has quit IRC | 20:23 | |
*** Tim_Eberhard has joined #openstack-fwaas | 20:23 | |
*** Tim_Eberhard has quit IRC | 20:37 | |
*** vishwanathj has quit IRC | 21:01 | |
*** vishwanathj has joined #openstack-fwaas | 21:02 | |
*** yamamoto_ has joined #openstack-fwaas | 21:13 | |
*** vishwanathj has quit IRC | 21:19 | |
*** vishwanathj has joined #openstack-fwaas | 21:20 | |
*** yamamoto_ has quit IRC | 21:20 | |
*** vishwanathj has quit IRC | 21:27 | |
*** vishwanathj has joined #openstack-fwaas | 21:28 | |
*** vishwanathj has quit IRC | 21:28 | |
*** vishwanathj has joined #openstack-fwaas | 21:29 | |
*** vishwanathj has quit IRC | 21:32 | |
*** vishwanathj has joined #openstack-fwaas | 21:33 | |
*** yamamoto has joined #openstack-fwaas | 22:04 | |
*** openstack has joined #openstack-fwaas | 22:43 | |
*** Tim_Eberhard has joined #openstack-fwaas | 23:56 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!