| *** salv-orlando has joined #openstack-neutron-ovn | 00:01 | |
| *** salv-orlando has quit IRC | 00:08 | |
| *** salv-orlando has joined #openstack-neutron-ovn | 01:30 | |
| *** salv-orlando has quit IRC | 01:32 | |
| *** salv-orlando has joined #openstack-neutron-ovn | 01:38 | |
| *** asuvvari has joined #openstack-neutron-ovn | 01:40 | |
| *** asuvvari has quit IRC | 01:44 | |
| *** salv-orlando has quit IRC | 01:45 | |
| *** asuvvari has joined #openstack-neutron-ovn | 04:07 | |
| *** asuvvari has quit IRC | 04:11 | |
| *** salv-orlando has joined #openstack-neutron-ovn | 04:15 | |
| *** subscope has joined #openstack-neutron-ovn | 04:22 | |
| *** salv-orlando has quit IRC | 04:25 | |
| *** shettyg has quit IRC | 05:10 | |
| *** salv-orlando has joined #openstack-neutron-ovn | 05:20 | |
| *** gangil has joined #openstack-neutron-ovn | 05:54 | |
| *** gangil has joined #openstack-neutron-ovn | 05:54 | |
| openstackgerrit | Han Zhou proposed openstack/networking-ovn: Update macs to addresses because of OVN schema update for l3 https://review.openstack.org/236817 | 06:42 |
|---|---|---|
| *** yamamoto has joined #openstack-neutron-ovn | 06:55 | |
| *** subscope has quit IRC | 07:18 | |
| *** salv-orl_ has joined #openstack-neutron-ovn | 07:45 | |
| *** salv-orlando has quit IRC | 07:48 | |
| *** fzdarsky__ has joined #openstack-neutron-ovn | 07:51 | |
| *** asuvvari has joined #openstack-neutron-ovn | 08:02 | |
| *** asuvvari has quit IRC | 08:06 | |
| *** salv-orl_ has quit IRC | 08:14 | |
| *** subscope has joined #openstack-neutron-ovn | 08:29 | |
| *** openstack has joined #openstack-neutron-ovn | 09:17 | |
| *** salv-orlando has joined #openstack-neutron-ovn | 10:11 | |
| *** salv-orlando has quit IRC | 11:11 | |
| *** salv-orlando has joined #openstack-neutron-ovn | 12:17 | |
| *** salv-orlando has quit IRC | 12:21 | |
| *** salv-orlando has joined #openstack-neutron-ovn | 12:27 | |
| *** salv-orlando has quit IRC | 12:34 | |
| *** subscope has quit IRC | 12:48 | |
| *** subscope has joined #openstack-neutron-ovn | 12:49 | |
| *** regXboi has joined #openstack-neutron-ovn | 12:49 | |
| *** flaviof has joined #openstack-neutron-ovn | 12:56 | |
| *** yamamoto has quit IRC | 13:11 | |
| *** salv-orlando has joined #openstack-neutron-ovn | 13:13 | |
| *** mestery has joined #openstack-neutron-ovn | 13:24 | |
| *** salv-orl_ has joined #openstack-neutron-ovn | 13:37 | |
| *** salv-orl_ has quit IRC | 13:37 | |
| *** salv-orlando has quit IRC | 13:37 | |
| *** salv-orlando has joined #openstack-neutron-ovn | 13:38 | |
| *** nate_gone is now known as njohnston | 13:50 | |
| *** yamamoto has joined #openstack-neutron-ovn | 13:52 | |
| *** shettyg has joined #openstack-neutron-ovn | 13:55 | |
| *** gsagie_ has joined #openstack-neutron-ovn | 14:52 | |
| gsagie_ | terryw ? Where is twills :) | 14:53 |
| gsagie_ | twilson | 14:53 |
| russellb | he's otherwiseguy on irc | 14:53 |
| russellb | don't see him on | 14:53 |
| *** terryw is now known as otherwiseguy | 14:53 | |
| otherwiseguy | hi | 14:53 |
| gsagie_ | ohh yeah | 14:53 |
| gsagie_ | :) | 14:53 |
| gsagie_ | hi, haven't seen you for a while | 14:54 |
| russellb | otherwiseguy: oh hi there :) | 14:54 |
| * otherwiseguy waves | 14:54 | |
| otherwiseguy | gsagie_: yeah, I've been stuck in DPDK world...but looks like I'm wrapping that up soon. Going to also start doing some DVR-related stuff. But will also try to sneak some time in for OVN-related stuff as well. | 14:55 |
| gsagie_ | cool :) yeah i remember i saw an email from you combining dpdk with OVS agent | 14:56 |
| *** asuvvari has joined #openstack-neutron-ovn | 14:56 | |
| gsagie_ | important work.. :) | 14:57 |
| gsagie_ | and its interesting to do something different sometimes.. at least thats how i see it | 14:59 |
| *** salv-orl_ has joined #openstack-neutron-ovn | 15:16 | |
| *** salv-orlando has quit IRC | 15:20 | |
| *** gangil has quit IRC | 15:29 | |
| openstackgerrit | Russell Bryant proposed openstack/networking-ovn: Update macs to addresses because of OVN schema update for l3 https://review.openstack.org/236817 | 15:41 |
| *** armax has joined #openstack-neutron-ovn | 15:52 | |
| * russellb stabs gerrit | 15:58 | |
| *** subscope has quit IRC | 16:06 | |
| openstackgerrit | Sisir Chowdhury proposed openstack/networking-ovn: Fix for Defect-1507445: Renaming of router is not supported https://review.openstack.org/237069 | 16:17 |
| *** armax_ has joined #openstack-neutron-ovn | 16:27 | |
| *** armax has quit IRC | 16:29 | |
| *** mestery has quit IRC | 16:29 | |
| *** armax_ is now known as armax | 16:29 | |
| *** gangil has joined #openstack-neutron-ovn | 16:30 | |
| *** gangil has joined #openstack-neutron-ovn | 16:30 | |
| *** salv-orl_ has quit IRC | 16:31 | |
| *** salv-orlando has joined #openstack-neutron-ovn | 16:32 | |
| *** asuvvari_ has joined #openstack-neutron-ovn | 16:36 | |
| *** asuvvari has quit IRC | 16:39 | |
| openstackgerrit | Russell Bryant proposed openstack/networking-ovn: Test security groups. https://review.openstack.org/223839 | 16:59 |
| openstackgerrit | Russell Bryant proposed openstack/networking-ovn: Add security group support using OVN ACLs. https://review.openstack.org/223817 | 16:59 |
| *** openstackgerrit has quit IRC | 17:01 | |
| *** openstackgerrit has joined #openstack-neutron-ovn | 17:02 | |
| *** chandrav has joined #openstack-neutron-ovn | 17:05 | |
| *** salv-orlando has quit IRC | 17:11 | |
| *** subscope has joined #openstack-neutron-ovn | 17:31 | |
| *** salv-orlando has joined #openstack-neutron-ovn | 17:41 | |
| openstackgerrit | Merged openstack/networking-ovn: Update macs to addresses because of OVN schema update for l3 https://review.openstack.org/236817 | 17:43 |
| *** gsagie_ has quit IRC | 17:44 | |
| *** gsagie_ has joined #openstack-neutron-ovn | 17:54 | |
| gsagie_ | russellb : Joining the meeting? | 17:58 |
| gsagie_ | in 2 mins | 17:58 |
| russellb | yep | 17:58 |
| *** yamamoto has quit IRC | 18:00 | |
| *** azbiswas has joined #openstack-neutron-ovn | 18:08 | |
| *** yamamoto has joined #openstack-neutron-ovn | 18:09 | |
| chandrav | russelb: We are making changes to the OVN plugin to reflect the changes Ben is making | 18:10 |
| russellb | chandrav: great thank you! | 18:10 |
| chandrav | russelb: Need help with one thing, if you are free we can discuss it now | 18:10 |
| *** armax has quit IRC | 18:10 | |
| russellb | on the phone this minute | 18:10 |
| russellb | i can try to multi-task though | 18:11 |
| russellb | what's the issue? | 18:11 |
| chandrav | russelb: The issue is with updating the options column in logical port with {'router-port': <UUID of router port>} | 18:12 |
| chandrav | The uuid that i get after the txn.insert of a logical router port gets changed after the transaction is succeded | 18:12 |
| *** gangil1 has joined #openstack-neutron-ovn | 18:13 | |
| russellb | have code you can share? | 18:13 |
| chandrav | if the UUID were to be directly used in a column, idl would take care of updating it, but here since we are using the UUID as a json body it remains the old uuid | 18:13 |
| chandrav | yes, i do have the code ready with a few minor things to work on | 18:14 |
| chandrav | i can share it | 18:14 |
| russellb | ok that'd be helpful i think | 18:14 |
| *** gangil has quit IRC | 18:14 | |
| azbiswas | basically the uuid (of the router port) changes but that change in not reflected in the port corresponding to the router port. | 18:15 |
| russellb | i'm also not up to date with how the schema has changed | 18:16 |
| *** azbiswas_ has joined #openstack-neutron-ovn | 18:20 | |
| chandrav | russellb: You can pull the code from https://github.com/cvejendla/networking-ovn-wip (branch: l3_ghe) | 18:21 |
| *** yamamoto has quit IRC | 18:21 | |
| *** azbiswas_ has quit IRC | 18:27 | |
| gsagie_ | chandrav: you can always work on networking-ovn and send the patch as WIP | 18:34 |
| russellb | yeah, true | 18:36 |
| russellb | i'm just trying to find the exact code that isn't doing what you expect | 18:36 |
| gsagie_ | the UUID is changed after an update? thats strange | 18:39 |
| openstackgerrit | Russell Bryant proposed openstack/networking-ovn: Add security group support using OVN ACLs. https://review.openstack.org/223817 | 18:39 |
| russellb | this sounds really familiar | 18:40 |
| russellb | i think you have to commit the transaction creating the object first, before you know its real UUID | 18:41 |
| gsagie_ | this is true for create | 18:41 |
| *** armax has joined #openstack-neutron-ovn | 18:41 | |
| gsagie_ | but he is talking about update here, if i got it right | 18:41 |
| gsagie_ | for create you wouldn't know the correct UUID just after the transaction finished | 18:42 |
| azbiswas | So create --> commit --> get? | 18:42 |
| gsagie_ | I don't see why Row UUID is suppose to be changed after an update (unless this "update" is doing delete and re-create) | 18:42 |
| gsagie_ | azbiswas : yes, but why do you need the UUID after you create? | 18:43 |
| azbiswas | gsagie_: after the lrouter_port is created, its uuid needs to put int the corresponding l_port table | 18:44 |
| gsagie_ | why do you need it? you put it as external id? | 18:45 |
| azbiswas | yes | 18:45 |
| azbiswas | according to the latest schema | 18:45 |
| gsagie_ | for what purpose if i may ask? | 18:45 |
| azbiswas | the restriction of single router port per lswitch is removed | 18:45 |
| azbiswas | <column name="options" key="router-port" type='{"type": "uuid"}'> | 18:46 |
| azbiswas | Required. The UUID of the <ref table="Logical_Router_Port"/> to | 18:46 |
| azbiswas | which this logical switch port is connected. | 18:46 |
| azbiswas | </column> | 18:46 |
| azbiswas | instead the port column can have the above | 18:47 |
| gsagie_ | sec, let me look at the schema | 18:47 |
| gsagie_ | this is for the logical port? | 18:47 |
| russellb | azbiswas: take a look at creating an lport on an lswitch | 18:48 |
| gsagie_ | ahh yes i see it now | 18:48 |
| russellb | it's the same thing | 18:48 |
| russellb | we create the lport, and put in the list of ports on the lswitch | 18:48 |
| russellb | whatever that code does seems to work :) | 18:48 |
| azbiswas | russellb: Thanks for the hint | 18:48 |
| chandrav | but in the lswitch case, the uuid is used directly in the column, and i guess, idl takes care of converting the old uuid to a new uuid | 18:49 |
| chandrav | in this case, the uuid is not a used directly in the coulmn, but as {'router-port': 'UUID'} | 18:50 |
| gsagie_ | chandrav : ahh and since its options string it doesnt work | 18:50 |
| chandrav | yes | 18:50 |
| gsagie_ | it sounds like a bug in the idl then, but you could do a quick fix and have another commant/transaction to update this in the plugin | 18:51 |
| gsagie_ | command | 18:51 |
| chandrav | yes, thats what i intend to do, as a separate transaction | 18:51 |
| gsagie_ | so after you add the logical router port, after its already added, have another command to update this column | 18:51 |
| russellb | if you do it as a separate transaction, the router port might get garbage collected | 18:52 |
| gsagie_ | but maybe document that this is a patch and add a bug | 18:52 |
| russellb | since it's not referenced anywhere | 18:52 |
| gsagie_ | russellb: its referenced from the logical router | 18:52 |
| russellb | oh ok | 18:52 |
| russellb | then yeah, separate transaction should be OK | 18:52 |
| gsagie_ | but please add a bug for this, sounds like an idl bug | 18:53 |
| chandrav | ok then, thanks for the input, i'll make that change and send for review | 18:53 |
| gsagie_ | otherwiseguy likes to find these ;) | 18:53 |
| chandrav | yes, i'll create a bug | 18:53 |
| gsagie_ | chandrav: thanks for continuing this work! | 18:54 |
| chandrav | if i submit for review, Jenkins will fail since Bens changes are not int yet, how do i go about this ? | 18:54 |
| otherwiseguy | Yay! IDL bugs! | 18:54 |
| gsagie_ | and azbiswas of course :) | 18:54 |
| gsagie_ | chandrav : put as [WIP], we will only merge when the changes are merged into OVN | 18:55 |
| chandrav | ok | 18:55 |
| russellb | chandrav: good question ... hopefully they'll be merged ASAP | 18:55 |
| russellb | chandrav: actually update ovs master again, i think they merged | 18:55 |
| russellb | a couple hours ago | 18:55 |
| gsagie_ | the schema update is there | 18:55 |
| russellb | talk about bleeding edge! | 18:55 |
| gsagie_ | :) | 18:56 |
| *** fzdarsky__ has quit IRC | 18:57 | |
| russellb | security groups are working for me locally, but they require a kernel backport in a branch | 18:58 |
| russellb | trying to make it work in the gate :/ | 18:58 |
| gsagie_ | cool! :) | 18:59 |
| gsagie_ | good work | 18:59 |
| gsagie_ | have to try that | 18:59 |
| russellb | gsagie: https://review.openstack.org/#/c/223817 should use the right ovs code to make it work | 18:59 |
| russellb | compiles and loads custom ovs kernel module though | 19:00 |
| russellb | instead of using distro ovs module | 19:00 |
| gsagie_ | ok, cool will try tommorow | 19:00 |
| *** gangil1 has quit IRC | 19:05 | |
| russellb | you know ... i wonder if now that all this L3 stuff is going in, if our use of the L3 agent is going to break things | 19:06 |
| russellb | argh | 19:06 |
| russellb | azbiswas: when testing, are you turning off the l3 agent? | 19:07 |
| azbiswas | russellb: We haven't yet got to that point | 19:08 |
| russellb | ok | 19:08 |
| russellb | no problem | 19:08 |
| azbiswas | that will be the real l3 test | 19:08 |
| gsagie_ | if the implementation is correct we will still need the l3-agent | 19:09 |
| gsagie_ | for north-south traffic | 19:09 |
| gsagie_ | but east-west will never reach the network node namespace | 19:10 |
| azbiswas | we can confirm that by running tcpdump | 19:11 |
| azbiswas | which is what I was doing previously | 19:11 |
| russellb | gsagie_: yes, today, though they expect to merge some north-south support this week too | 19:11 |
| russellb | so we'll support distributed north-south traffic with NAT, as long as you have a floating IP | 19:11 |
| gsagie_ | how? they already have NAT supported? | 19:11 |
| *** fzdarsky__ has joined #openstack-neutron-ovn | 19:12 | |
| russellb | working on it | 19:12 |
| russellb | at least an intermediary solution | 19:12 |
| gsagie_ | and how is the distributed SNAT going to work? | 19:12 |
| russellb | it's not | 19:12 |
| gsagie_ | so only for DNAT? | 19:12 |
| russellb | well, it will as long as you have a floating IP | 19:12 |
| russellb | i don't know how they're implementing it, this is all going lightning speed | 19:12 |
| gsagie_ | okie | 19:13 |
| gsagie_ | thanks for the update, interesting to see | 19:13 |
| russellb | SNAT without floating IPs will be centralized, and may not work at all this week | 19:13 |
| russellb | so, similar in traffic pattern to DVR, i suppose | 19:13 |
| *** gsagie_ has quit IRC | 19:20 | |
| openstackgerrit | Russell Bryant proposed openstack/networking-ovn: Add security group support using OVN ACLs. https://review.openstack.org/223817 | 19:21 |
| *** fzdarsky__ has quit IRC | 19:22 | |
| shettyg | russellb: I have sent the code for review for OVN+Docker (master branch)+OpenStack. So for your demo, in OpenStack, I think it will be nice to have it too… | 19:23 |
| russellb | awesome | 19:23 |
| russellb | i'd love to show it ... right now i'm super nervous about how much we'll get working in time | 19:23 |
| russellb | we can certainly at least talk about it | 19:24 |
| *** roger has joined #openstack-neutron-ovn | 19:40 | |
| *** chandrav has quit IRC | 19:43 | |
| *** subscope has quit IRC | 19:53 | |
| *** chandrav has joined #openstack-neutron-ovn | 19:57 | |
| *** flaviof has quit IRC | 20:21 | |
| openstackgerrit | Russell Bryant proposed openstack/networking-ovn: Add security group support using OVN ACLs. https://review.openstack.org/223817 | 20:22 |
| *** gangil has joined #openstack-neutron-ovn | 20:23 | |
| *** gangil has joined #openstack-neutron-ovn | 20:23 | |
| *** yamamoto has joined #openstack-neutron-ovn | 20:30 | |
| *** yamamoto has quit IRC | 20:34 | |
| *** salv-orlando has quit IRC | 20:45 | |
| openstackgerrit | Russell Bryant proposed openstack/networking-ovn: Add security group support using OVN ACLs. https://review.openstack.org/223817 | 20:56 |
| *** salv-orlando has joined #openstack-neutron-ovn | 20:58 | |
| openstackgerrit | Russell Bryant proposed openstack/networking-ovn: Add security group support using OVN ACLs. https://review.openstack.org/223817 | 20:59 |
| *** regXboi has quit IRC | 21:01 | |
| *** gangil has quit IRC | 21:04 | |
| *** VikramD has joined #openstack-neutron-ovn | 21:18 | |
| *** gangil has joined #openstack-neutron-ovn | 21:22 | |
| *** gangil has joined #openstack-neutron-ovn | 21:22 | |
| *** njohnston is now known as nate_gone | 21:36 | |
| *** shettyg has quit IRC | 21:56 | |
| *** salv-orl_ has joined #openstack-neutron-ovn | 21:56 | |
| *** yamamoto has joined #openstack-neutron-ovn | 21:58 | |
| *** salv-orlando has quit IRC | 21:59 | |
| *** asuvvari_ has quit IRC | 22:23 | |
| *** asuvvari has joined #openstack-neutron-ovn | 22:23 | |
| *** asuvvari_ has joined #openstack-neutron-ovn | 22:27 | |
| *** asuvvari has quit IRC | 22:28 | |
| *** asuvvari_ has quit IRC | 22:32 | |
| *** asuvvari has joined #openstack-neutron-ovn | 22:39 | |
| *** VikramD has quit IRC | 22:40 | |
| *** azbiswas has quit IRC | 22:42 | |
| *** asuvvari has quit IRC | 22:44 | |
| *** chandrav has quit IRC | 23:02 | |
| openstackgerrit | Sisir Chowdhury proposed openstack/networking-ovn: Closes-Bug: 1507445 -- This Commit will have the Change to support OVN NB Logical Router name Update. https://review.openstack.org/237069 | 23:12 |
| openstack | bug 1507445 in networking-ovn "Renaming of router is not supported i.e. router name change in Neutron DB is not getting reflected in OVN_NB" [Undecided,New] https://launchpad.net/bugs/1507445 - Assigned to Sisir Chowdhury (schowdh) | 23:13 |
| *** chandrav has joined #openstack-neutron-ovn | 23:16 | |
| *** flaviof has joined #openstack-neutron-ovn | 23:21 | |
| *** salv-orl_ has quit IRC | 23:26 | |
| *** gangil has quit IRC | 23:30 | |
| *** salv-orl_ has joined #openstack-neutron-ovn | 23:31 | |
| *** salv-orlando has joined #openstack-neutron-ovn | 23:32 | |
| *** gangil has joined #openstack-neutron-ovn | 23:35 | |
| *** gangil has joined #openstack-neutron-ovn | 23:35 | |
| *** salv-orl_ has quit IRC | 23:36 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!