| *** tongl has quit IRC | 00:16 | |
| *** livelace has quit IRC | 01:28 | |
| *** livelace has joined #openstack-neutron-ovn | 01:29 | |
| *** mestery has quit IRC | 02:17 | |
| *** afranc has quit IRC | 02:17 | |
| *** mestery has joined #openstack-neutron-ovn | 02:22 | |
| *** afranc has joined #openstack-neutron-ovn | 02:22 | |
| *** saphi has joined #openstack-neutron-ovn | 02:24 | |
| *** saphi has quit IRC | 02:38 | |
| *** yamamoto_ has joined #openstack-neutron-ovn | 02:46 | |
| *** yamamoto has quit IRC | 02:46 | |
| *** saphi has joined #openstack-neutron-ovn | 02:51 | |
| *** anilvenkata has joined #openstack-neutron-ovn | 03:56 | |
| *** janki has joined #openstack-neutron-ovn | 05:03 | |
| *** saphi_ has joined #openstack-neutron-ovn | 05:24 | |
| *** saphi has quit IRC | 05:25 | |
| *** lrichard has quit IRC | 05:47 | |
| *** lrichard has joined #openstack-neutron-ovn | 05:53 | |
| *** saphi__ has joined #openstack-neutron-ovn | 06:01 | |
| *** saphi_ has quit IRC | 06:02 | |
| *** livelace has quit IRC | 06:22 | |
| *** livelace has joined #openstack-neutron-ovn | 06:28 | |
| *** mmirecki has joined #openstack-neutron-ovn | 06:45 | |
| *** zefferno has joined #openstack-neutron-ovn | 06:57 | |
| *** pcaruana has joined #openstack-neutron-ovn | 07:00 | |
| *** gtrxcb has quit IRC | 07:11 | |
| *** yamamoto_ has quit IRC | 07:44 | |
| *** yamamoto has joined #openstack-neutron-ovn | 07:50 | |
| *** mwarad has joined #openstack-neutron-ovn | 08:15 | |
| *** _mwarad_ has joined #openstack-neutron-ovn | 08:15 | |
| *** _mwarad_ has quit IRC | 08:15 | |
| *** Guest14 has joined #openstack-neutron-ovn | 08:21 | |
| *** lucas-afk is now known as lucasagomes | 08:26 | |
| openstackgerrit | Jakub Libosvar proposed openstack/ovsdbapp master: Add RowView versions of db_find and db_list https://review.openstack.org/490199 | 08:32 |
|---|---|---|
| openstackgerrit | Jakub Libosvar proposed openstack/ovsdbapp master: Add RowView versions of db_find and db_list https://review.openstack.org/490199 | 08:33 |
| *** yamamoto has quit IRC | 08:49 | |
| *** mwarad has quit IRC | 08:59 | |
| *** yamamoto has joined #openstack-neutron-ovn | 09:15 | |
| *** yamamoto has quit IRC | 09:22 | |
| *** pmannidi has joined #openstack-neutron-ovn | 09:48 | |
| openstackgerrit | Merged openstack/networking-ovn master: Prepare for using ovsdbapp > 0.4.0 https://review.openstack.org/487442 | 09:50 |
| *** yamamoto has joined #openstack-neutron-ovn | 09:54 | |
| *** saphi__ has quit IRC | 10:01 | |
| *** yamamoto has quit IRC | 10:07 | |
| *** yamamoto has joined #openstack-neutron-ovn | 10:12 | |
| *** yamamoto has quit IRC | 10:17 | |
| *** yamamoto has joined #openstack-neutron-ovn | 10:18 | |
| *** yamamoto has quit IRC | 10:20 | |
| *** yamamoto has joined #openstack-neutron-ovn | 10:20 | |
| openstackgerrit | Miguel Angel Ajo proposed openstack/networking-ovn master: refarch: Update documentation and diagrams https://review.openstack.org/490812 | 10:54 |
| Guest14 | dalvarez can you have a look on metadata related changes here ^ ? | 10:54 |
| *** Guest14 is now known as ajo | 10:54 | |
| ajo | yikes | 10:54 |
| dalvarez | ajo sure | 10:55 |
| ajo | thanks :D | 10:56 |
| *** yamamoto has quit IRC | 10:58 | |
| *** yamamoto has joined #openstack-neutron-ovn | 10:59 | |
| *** yamamoto has quit IRC | 11:05 | |
| *** yamamoto has joined #openstack-neutron-ovn | 11:07 | |
| *** yamamoto has quit IRC | 11:08 | |
| *** yamamoto has joined #openstack-neutron-ovn | 11:10 | |
| *** yamamoto has quit IRC | 11:13 | |
| *** yamamoto has joined #openstack-neutron-ovn | 11:15 | |
| *** yamamoto has quit IRC | 11:16 | |
| *** yamamoto has joined #openstack-neutron-ovn | 11:16 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/networking-ovn master: Idea proposal: Neutron/OVN database consistency problem https://review.openstack.org/490834 | 11:38 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/networking-ovn master: Tox docs: Force the use of python2.7 https://review.openstack.org/491765 | 11:43 |
| *** lucasagomes is now known as lucas-hungry | 11:46 | |
| *** yamamoto has quit IRC | 12:18 | |
| *** yamamoto has joined #openstack-neutron-ovn | 12:20 | |
| *** yamamoto has quit IRC | 12:20 | |
| openstackgerrit | Merged openstack/networking-ovn master: call provisioning_complete conditionally https://review.openstack.org/490511 | 12:33 |
| *** yamamoto has joined #openstack-neutron-ovn | 12:40 | |
| *** lucas-hungry is now known as lucasagomes | 12:45 | |
| ajo | russellb , anilvenkata about the check db version / vs model introspection in networking-ovn | 12:51 |
| ajo | I was proposing more the introspection side, while, it could be marginally slower / or a bit more complex to program | 12:51 |
| ajo | because it's a bit more independent of backports | 12:51 |
| ajo | when maintaining neutron upstream I cried many times because it wasn't easy to backport a fix that was db dependent | 12:52 |
| ajo | making sure we don't use the version for checks would avoid us eventually ending in the same situation | 12:52 |
| anilvenkata | ajo, how would using version will affect backporting issues? | 12:53 |
| ajo | may be it's over engineering at this point, I don't know, but I remember hours of frustration trying to find a solution for the db backports, that... while they ended being possible, they were technically very complicated to handle | 12:53 |
| ajo | anilvenkata , you backport an specific patch from 2.9 to 2.8, for example, that exposes a new column/table via the db | 12:54 |
| ajo | while you cannot set the 2.9 version of the DB (because that would misslead later upgrades) | 12:54 |
| ajo | the colum/table would still be there | 12:54 |
| ajo | imagine that you had 5.8.1 in 2.8 | 12:54 |
| ajo | and in 2.9 the patch you try to backport upgrades db from 5.9.3 to 5.9.4 | 12:55 |
| ajo | you could backport the patch and... | 12:55 |
| ajo | update the db in 2.8 to 5.8.1.2 | 12:55 |
| ajo | or someyhing like that | 12:55 |
| openstackgerrit | Miguel Angel Ajo proposed openstack/networking-ovn master: refarch: Update documentation and diagrams https://review.openstack.org/490812 | 12:57 |
| anilvenkata | ajo, in this case our version check in networking-ovn will be, before backporting >=5.9.4 , after backporting >=5.8.1.2 | 12:57 |
| anilvenkata | ajo, can't we solve backporting issule like that ^ ? | 12:58 |
| numans | ajo anilvenkata how can you backport something if the corresponding DB table/column is not present in the older version ? | 12:58 |
| ajo | numans backporting the db changes too | 13:00 |
| anilvenkata | numans, ajo I think in backporting patch, he is also backporting DB changes | 13:00 |
| anilvenkata | :) | 13:00 |
| ajo | anilvenkata you're right, we could change the check for the DB version | 13:00 |
| ajo | to the new 5.8.1.2 | 13:00 |
| ajo | well | 13:01 |
| ajo | that'd make sense | 13:01 |
| ajo | anilvenkata++ | 13:01 |
| ajo | ok, so let it be db version checks :) | 13:01 |
| anilvenkata | ajo, no still that is a problem | 13:01 |
| ajo | why | 13:01 |
| anilvenkata | ajo, what if the user is not upgrading new networking-ovn | 13:02 |
| ajo | then they wouldn't be using the new feature/thing | 13:02 |
| *** mmichelson has joined #openstack-neutron-ovn | 13:02 | |
| ajo | northd interface would have to be backwards compatible (like we did for the options:redirect_chassis & Gateway_Chassis) | 13:03 |
| ajo | ovn-northd will take any of them | 13:03 |
| anilvenkata | ajo, but if the new feature deletes columns? | 13:03 |
| ajo | with priority for the new | 13:03 |
| ajo | anilvenkata that'd be an issue in either case | 13:03 |
| ajo | if you check the models or if you check the version | 13:03 |
| ajo | as soon as a column you arent' checking goes away | 13:03 |
| ajo | I guess deleting things from the DB can't be done lightly | 13:04 |
| ajo | it needs a deprecation window | 13:04 |
| anilvenkata | russellb, any thoughts ^ on if we can go with version number or other approach? | 13:05 |
| anilvenkata | ajo, russellb if we can conclude, I will update my patch :) | 13:05 |
| numans | ajo, got it. | 13:05 |
| numans | anilvenkata, ajo, although I feel its highly unlikely that a feature/db changes would be backported to a released branch | 13:05 |
| ajo | I think, based on the discusion, version check may be ok | 13:05 |
| ajo | numans yeah, it's always the case, until you find it | 13:06 |
| ajo | :D | 13:06 |
| ajo | but, it could be fixed with versioning to | 13:06 |
| ajo | too | 13:06 |
| ajo | you just change the check for the "new backported version" is not a big deal :) | 13:06 |
| anilvenkata | ajo, thanks Miguel | 13:07 |
| anilvenkata | ajo, russellb numans shall I go ahead with version checks? | 13:07 |
| numans | anilvenkata, version checks seems simpler in terms of code. so i am fine with it :) | 13:10 |
| anilvenkata | numans, thanks numans | 13:11 |
| russellb | i'm really ok with either ... i guess it's more my ovn maintainer hat on, thinking "why do we bother with the version if it doesn't help solve a problem" | 13:13 |
| anilvenkata | russellb, thanks Russell | 13:19 |
| russellb | but i also get how backports make version checking messy, so checking for the feature is the most explicit | 13:20 |
| anilvenkata | russellb, checking for feature means checking if columns and tables present or not? | 13:35 |
| anilvenkata | ajo, ^ | 13:35 |
| anilvenkata | numans, ^ | 13:37 |
| russellb | yes | 13:40 |
| *** mlavalle has joined #openstack-neutron-ovn | 13:43 | |
| anilvenkata | russellb, thanks Russell | 13:45 |
| *** anilvenkata is now known as anilvenkata|AFK | 13:45 | |
| numans | anilvenkata|AFK, even i was thinking the same. checking for the feature seems to be better | 13:49 |
| *** anilvenkata|AFK is now known as anilvenkata | 13:58 | |
| anilvenkata | numans, thanks Numa | 13:59 |
| *** pmannidi has quit IRC | 14:14 | |
| *** mmirecki has quit IRC | 15:05 | |
| *** mmirecki has joined #openstack-neutron-ovn | 15:25 | |
| *** livelace2 has joined #openstack-neutron-ovn | 15:48 | |
| *** janki has quit IRC | 15:48 | |
| *** mmirecki has quit IRC | 15:51 | |
| *** yamamoto has quit IRC | 15:54 | |
| *** zefferno has quit IRC | 15:57 | |
| *** ajo has quit IRC | 15:57 | |
| *** yamamoto has joined #openstack-neutron-ovn | 15:58 | |
| *** yamamoto has quit IRC | 16:03 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/networking-ovn master: Idea proposal: Neutron/OVN database consistency problem https://review.openstack.org/490834 | 16:23 |
| *** lucasagomes is now known as lucas-afk | 16:24 | |
| openstackgerrit | Terry Wilson proposed openstack/ovsdbapp master: Add RowView versions of db_find and db_list https://review.openstack.org/490199 | 16:25 |
| *** pcaruana has quit IRC | 16:27 | |
| *** saphi has joined #openstack-neutron-ovn | 16:32 | |
| openstackgerrit | venkata anil proposed openstack/networking-ovn master: Support for L3 gateway HA https://review.openstack.org/486971 | 16:48 |
| *** anilvenkata has quit IRC | 16:53 | |
| *** saphi has quit IRC | 16:56 | |
| *** livelace2 has quit IRC | 17:55 | |
| *** mlavalle has quit IRC | 19:00 | |
| *** yamamoto has joined #openstack-neutron-ovn | 21:09 | |
| *** yamamoto has quit IRC | 21:21 | |
| *** yamamoto has joined #openstack-neutron-ovn | 21:59 | |
| *** mmichelson has quit IRC | 22:24 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!