*** yushiro-mobile has joined #openstack-fwaas | 00:05 | |
*** yushiro-mobile has quit IRC | 00:06 | |
*** yushiro-mobile has joined #openstack-fwaas | 00:06 | |
*** yushiro-mobile has quit IRC | 00:31 | |
*** hoangcx has joined #openstack-fwaas | 00:50 | |
*** yushiro-mobile has joined #openstack-fwaas | 01:32 | |
yushiro-mobile | I'm finding Sridar now... | 01:33 |
---|---|---|
*** yamamoto has joined #openstack-fwaas | 03:03 | |
reedip | yushiro-mobile +1 :) | 03:25 |
yushiro-mobile | reedip: do u know where he is? :-) | 03:26 |
reedip | I dont know, I am in India :D | 03:26 |
yushiro-mobile | reedip: Haha. Thanks. | 03:27 |
*** yushiro2-mobile has joined #openstack-fwaas | 03:32 | |
*** yushiro-mobile has quit IRC | 03:32 | |
*** annp has joined #openstack-fwaas | 03:46 | |
*** yushiro2-mobile has quit IRC | 03:46 | |
*** lnicolas has quit IRC | 06:02 | |
*** yamamoto_ has joined #openstack-fwaas | 06:09 | |
*** bbbzhao has joined #openstack-fwaas | 06:14 | |
*** yamamoto has quit IRC | 06:14 | |
*** bbzhao has quit IRC | 06:14 | |
*** yushiro-mobile has joined #openstack-fwaas | 06:30 | |
*** openstackgerrit has quit IRC | 06:31 | |
*** eN_Guruprasad_Rn has joined #openstack-fwaas | 06:54 | |
*** openstackgerrit has joined #openstack-fwaas | 07:11 | |
openstackgerrit | XieYingYun proposed openstack/neutron-fwaas master: Optimize the link address https://review.openstack.org/517881 | 08:07 |
*** AlexeyAbashkin has joined #openstack-fwaas | 08:33 | |
reedip | happy birthday yushiro-mobile :) | 08:42 |
*** eN_Guruprasad_Rn has quit IRC | 08:43 | |
*** eN_Guruprasad_Rn has joined #openstack-fwaas | 08:43 | |
*** yushiro2-mobile has joined #openstack-fwaas | 08:47 | |
*** yushiro-mobile has quit IRC | 08:47 | |
yushiro2-mobile | reedip, thanks a lot !!!!!!! | 08:47 |
reedip | :) | 08:48 |
*** AlexeyAbashkin has quit IRC | 08:51 | |
*** AlexeyAbashkin has joined #openstack-fwaas | 08:51 | |
*** yushiro2-mobile has quit IRC | 08:56 | |
*** yamamoto_ has quit IRC | 09:06 | |
*** openstackgerrit has quit IRC | 09:18 | |
*** openstackgerrit has joined #openstack-fwaas | 09:22 | |
openstackgerrit | Reedip proposed openstack/neutron-fwaas master: Fix validation with 'protocol' for firewall_rule https://review.openstack.org/423229 | 09:22 |
*** hoangcx has quit IRC | 09:32 | |
*** hoangcx has joined #openstack-fwaas | 09:32 | |
*** AlexeyAbashkin has quit IRC | 10:00 | |
*** AlexeyAbashkin has joined #openstack-fwaas | 10:02 | |
*** yamamoto has joined #openstack-fwaas | 10:06 | |
*** AlexeyAbashkin has quit IRC | 10:06 | |
*** yamamoto has quit IRC | 10:15 | |
*** yamamoto has joined #openstack-fwaas | 10:21 | |
*** yamamoto has quit IRC | 10:29 | |
*** yamamoto has joined #openstack-fwaas | 10:40 | |
*** yamamoto has quit IRC | 11:00 | |
*** AlexeyAbashkin has joined #openstack-fwaas | 11:06 | |
*** annp has quit IRC | 11:18 | |
*** yamamoto has joined #openstack-fwaas | 11:19 | |
openstackgerrit | Murali Annamneni proposed openstack/neutron-fwaas master: Enable MySQL Cluster Support for neutron-fwaas https://review.openstack.org/513392 | 11:20 |
*** AlexeyAbashkin has quit IRC | 11:52 | |
*** AlexeyAbashkin has joined #openstack-fwaas | 11:55 | |
*** AlexeyAbashkin has quit IRC | 12:00 | |
*** AlexeyAbashkin has joined #openstack-fwaas | 12:13 | |
*** yamamoto has quit IRC | 12:17 | |
*** reedip has quit IRC | 12:20 | |
*** AlexeyAbashkin has quit IRC | 12:34 | |
*** yamamoto has joined #openstack-fwaas | 13:18 | |
*** yamamoto has quit IRC | 13:27 | |
*** eN_Guruprasad_Rn has quit IRC | 13:32 | |
*** annp has joined #openstack-fwaas | 13:47 | |
*** chandanc has joined #openstack-fwaas | 13:58 | |
annp | hi chandanc, xgerman_ | 14:00 |
chandanc | Hello annp | 14:00 |
annp | thank you for your time. | 14:00 |
chandanc | sure, sorry i am not able to join more often | 14:01 |
chandanc | lets start | 14:01 |
annp | chandanc, ok. let get started. | 14:01 |
annp | have you check my patch? | 14:02 |
annp | https://review.openstack.org/#/c/515368/3 | 14:02 |
chandanc | ya, i looked at the patch | 14:02 |
*** AlexeyAbashkin has joined #openstack-fwaas | 14:02 | |
chandanc | so you are diverting traffic based on the eg_enabled flag per port | 14:02 |
annp | yes, it is | 14:03 |
chandanc | ok, good idea | 14:03 |
annp | my idead, send accept packets from fwg to sg | 14:03 |
annp | however, i have a problem with conntrack | 14:04 |
chandanc | ya got it. | 14:04 |
chandanc | ok, i have not tried the patch and the mail was not very clear on the conntrack part | 14:04 |
annp | yeah. | 14:05 |
annp | let me show you my idea | 14:05 |
chandanc | ok | 14:06 |
xgerman_ | o/ | 14:06 |
chandanc | hello xgerman_ | 14:06 |
annp | hi xgerman_ | 14:06 |
xgerman_ | hi | 14:06 |
annp | let my explain my idea first, | 14:06 |
chandanc | ok | 14:06 |
xgerman_ | k | 14:07 |
annp | My idea, accept packet from fwg to sg, and commit the connection to conntracker module will be done by sg | 14:08 |
chandanc | ok | 14:09 |
xgerman_ | ok | 14:09 |
annp | so you can see in fwg no action=(ct(commit), ...) except invalid packets | 14:09 |
chandanc | but in case sg is disabled you will handle the conntrack ? | 14:09 |
xgerman_ | +1 | 14:09 |
annp | chandanc, in case sg is disabled: fwg will be process as sg | 14:10 |
xgerman_ | ok | 14:11 |
chandanc | 1 sec | 14:11 |
annp | that's reason, i need option port_security_enabled | 14:11 |
chandanc | looking at your patch | 14:11 |
xgerman_ | if we block a port which was previously open we also need to pull the conntrack | 14:11 |
chandanc | ok, got it | 14:12 |
xgerman_ | the original design was to do contrack in both SG and FWG and and if the entry was there ignore the request | 14:12 |
xgerman_ | aka the conntrack piece was a singleton… | 14:13 |
xgerman_ | then nothing needs to change if we run standalone, etc. | 14:13 |
chandanc | xgerman_: i think that was a coincedence due to copying of code :) but yes with the same outcome as you mentioned | 14:14 |
annp | xgerman_, Do you mean that, we should make conntrack is a singleton class? | 14:14 |
xgerman_ | yes | 14:14 |
xgerman_ | the same state is shared between SG and FWG | 14:14 |
annp | from my understanding, conntracker is a module of ovs | 14:15 |
xgerman_ | yeah, our thinking was for the iptables workd | 14:15 |
annp | xgerman_, yeah! | 14:15 |
xgerman_ | where we would manage the conntrack on the linux on the host | 14:15 |
*** AlexeyAbashkin has quit IRC | 14:16 | |
chandanc | we have only one conntrack state for the host, maintained in the kernel, it can be accessed through ovs,cli, netlink , this is my understanding | 14:17 |
annp | chandanc, yes! you're right | 14:18 |
xgerman_ | ok, great | 14:18 |
xgerman_ | now I thought the neutron conntrack module had internal data structures we needed to share between FWH and SG — but it has been >1 year snce I looked at that | 14:19 |
chandanc | so, can you explain the issue you faced with the current patch and conntrack | 14:19 |
chandanc | xgerman_: the conntrack class still needs to be singleton | 14:19 |
annp | the problem: why conntrack state has been change from +new-est to +est-rep+rpl for first packet | 14:20 |
chandanc | else different parts of the agent can become out of sync | 14:20 |
xgerman_ | +1 | 14:20 |
chandanc | sorry but annp can you please explain the +est-rep+rpl | 14:21 |
chandanc | what is rpl | 14:21 |
chandanc | rep is reply right ? | 14:21 |
annp | rep: reply | 14:24 |
chandanc | annp: can you paste the flow you mentioned in the mail, here. | 14:24 |
chandanc | it came a bit jumbled | 14:25 |
chandanc | VMBàTRANSIENT_TABLEàEGRESS_BASE_TABLEàACCEPT_OR_INGRESS_TABLEàgo to switchàTRANSIENT_TABLEàFW_EGRESS_BASE_TABLEàFW_EGRESS_RULE | 14:25 |
chandanc | àFW_ACCEPT_OR_INGRESS_TABLEàFW_BASE_INGRESS_TABLEàFW_INGRESS_RULE | 14:25 |
annp | chandanc, sorry, i don't have it here | 14:25 |
chandanc | ok | 14:25 |
annp | chandanc, i'm in home now | 14:25 |
annp | tomorrow, i will share it with you | 14:25 |
chandanc | sure, may be put in paste/etherpad | 14:25 |
xgerman_ | https://www.irccloud.com/pastebin/Tzjnny4H/ | 14:26 |
xgerman_ | ^^ it arrived unscrambled for me | 14:26 |
xgerman_ | oh, now it’s scrambled | 14:26 |
xgerman_ | https://usercontent.irccloud-cdn.com/file/tCMbny1u/Screen%20Shot%202017-11-06%20at%206.28.23%20AM.png | 14:27 |
chandanc | may be locale | 14:27 |
xgerman_ | ok, I uploaded a screenshot :-) | 14:27 |
chandanc | thanks :) | 14:27 |
chandanc | what the “go to switch” mean ? Physical switch ? | 14:28 |
annp | rpl - Packet is in reply direction | 14:28 |
annp | rel - Related - ICMP, eg “dst_unreach” response / helper “related” connection | 14:28 |
chandanc | oh ok | 14:29 |
annp | go to switch mean br-int | 14:29 |
chandanc | i though the rules are on br-int | 14:30 |
chandanc | right ? | 14:30 |
annp | the problem: conntrack state change from +new-est to +est-rel+rpl so 1st packet come to fwg will be matched with flow at https://review.openstack.org/#/c/515368/3/neutron_fwaas/services/firewall/drivers/linux/l2/openvswitch_firewall/firewall.py@679 | 14:31 |
annp | yes. | 14:31 |
annp | In case vm1 and vm2 is same host | 14:31 |
annp | In my case vm1 and vm2 is same host | 14:32 |
chandanc | so the logs that you captured are from running the latest code in your patch ? | 14:33 |
annp | chandanc, yes! | 14:33 |
openstackgerrit | boden proposed openstack/neutron-fwaas master: DNM: testing 516456 https://review.openstack.org/517998 | 14:34 |
chandanc | one more question. Did you stop the ping and start the ping again ? or did you keep running the ping and changed the rule | 14:35 |
annp | chandanc, i did both | 14:35 |
chandanc | oh, first one should fail ping | 14:36 |
annp | both will be failed | 14:36 |
chandanc | yes if you remove the conntrack correctly the 2nd also should fail | 14:36 |
xgerman_ | +1 | 14:37 |
annp | because, if conntrack state will be set to ct_mark_invalid | 14:37 |
chandanc | ok | 14:37 |
chandanc | does it work correctly with SG only ? | 14:38 |
annp | yes, It works perfect | 14:38 |
annp | https://review.openstack.org/#/c/515368/3/neutron_fwaas/services/firewall/drivers/linux/l2/openvswitch_firewall/firewall.py@708 | 14:39 |
chandanc | then as per the patch if we enable both SG and FWG and SG manages the conntrack it should work | 14:39 |
annp | chandanc, yes. that's reason. I don't understanding why conntrack state has been change for 1st packet in MY CASE | 14:40 |
chandanc | the new scenario is only introduced if we have SG disabled and FWG is managing conntrack | 14:40 |
chandanc | yes annp | 14:40 |
annp | if we have SG disable, fwg will be works same as sg | 14:41 |
chandanc | hmm | 14:42 |
chandanc | i guess we will need to do some live debugging | 14:42 |
xgerman_ | mmh, so SG allows packet and we drop it | 14:42 |
annp | however, if we enable SG or operator forgot to disable, there is problem! | 14:42 |
chandanc | sorry annp , did not catch your last comment | 14:44 |
xgerman_ | well, if SG is disabled we work great — but that’s not our ultimate goal… | 14:44 |
annp | xgerman_, right | 14:44 |
chandanc | oh ok ok | 14:45 |
annp | chandanc, It's my concern! | 14:45 |
annp | chandanc, there is difficult for me to explain the problem, if you have chance please try to test my case | 14:46 |
chandanc | i will, but will need some time. | 14:47 |
annp | chandanc, yeah! it will be take time. | 14:48 |
chandanc | is this upto date https://etherpad.openstack.org/p/fwaas-v2-l2 | 14:48 |
annp | chandanc, yes | 14:49 |
chandanc | xgerman_: is the l2 agent merged already ? | 14:49 |
xgerman_ | don’t think so | 14:49 |
chandanc | oh ok | 14:49 |
xgerman_ | I merged defaukt FWG last week | 14:49 |
chandanc | ok | 14:50 |
*** AlexeyAbashkin has joined #openstack-fwaas | 14:51 | |
chandanc | ok, i will bring up a setup and try to figure out. But it will take some time for me (due to other issue at the office) | 14:51 |
chandanc | will keep you posted | 14:52 |
chandanc | annp: do you want me to try anything specific, or just ping test ? | 14:52 |
annp | chandanc, i think just ping test is enough | 14:53 |
chandanc | ok will try | 14:53 |
annp | tomorrow, it will update etherpad with ovs flows, so we can see ovs flows for more clear problem | 14:54 |
chandanc | ok | 14:54 |
chandanc | do we have a meeting this week ? | 14:55 |
annp | xgerman_, chandanc, sorry for today. I haven't provide enough information. | 14:55 |
xgerman_ | no worries — we got some important clarifucations | 14:55 |
chandanc | no, problem. I wanted to understand the problem | 14:55 |
annp | IIRC, this week won't have a meeting | 14:55 |
chandanc | ok | 14:56 |
chandanc | ok, then let me get back with test results | 14:57 |
chandanc | annp if you figureout something let me know | 14:57 |
annp | yeah! thanks in advance. | 14:57 |
chandanc | sure, welcome | 14:57 |
annp | Sure! i will share with you. | 14:57 |
annp | xgerman, chandanc, thanks for discussion today | 14:58 |
chandanc | thanks | 14:58 |
annp | I need more time to make the problem more clear | 14:58 |
xgerman_ | thanks | 14:58 |
annp | So I will let you know, after i make it clear on etherpad | 14:59 |
chandanc | sure send a reply to the mail thread | 14:59 |
chandanc | ok, bye for now | 14:59 |
annp | chandanc, sure, see you! | 14:59 |
annp | xgerman_, have you go to office? | 14:59 |
xgerman_ | no, I work from home :-) | 15:00 |
annp | xgerman_, oh really! | 15:00 |
annp | xgerman_, have a great day ahead | 15:01 |
annp | xgerman_ i will off now! see you | 15:01 |
xgerman_ | yes, see you | 15:01 |
*** annp has quit IRC | 15:01 | |
*** chandanc has left #openstack-fwaas | 15:07 | |
*** AlexeyAbashkin has quit IRC | 15:15 | |
*** jdavis has joined #openstack-fwaas | 20:12 | |
*** jdavis has quit IRC | 20:13 | |
*** jdavis has joined #openstack-fwaas | 20:49 | |
*** AlexeyAbashkin has joined #openstack-fwaas | 21:00 | |
*** AlexeyAbashkin has quit IRC | 21:04 | |
*** jdavis has quit IRC | 21:18 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!