| openstackgerrit | Merged openstack/neutron master: Fix handling of network:shared field in policy module https://review.opendev.org/652636 | 00:09 |
|---|---|---|
| openstackgerrit | Merged openstack/networking-ovn master: Fix networking-ovn-tempest-dsvm-ovs-master-fedora https://review.opendev.org/656621 | 00:13 |
| *** mriedem has quit IRC | 00:15 | |
| *** yamamoto has quit IRC | 00:18 | |
| *** ijw has joined #openstack-neutron | 00:19 | |
| *** Weifan has quit IRC | 00:20 | |
| *** ijw has quit IRC | 00:27 | |
| *** SmearedBeard has joined #openstack-neutron | 00:39 | |
| *** hyang has left #openstack-neutron | 00:45 | |
| *** hyang has joined #openstack-neutron | 00:51 | |
| openstackgerrit | Reedip proposed openstack/networking-ovn master: DNM Testing support for Router Scheduling https://review.opendev.org/657794 | 01:04 |
| openstackgerrit | Reedip proposed openstack/networking-ovn master: DNM Testing support for Router Scheduling https://review.opendev.org/657794 | 01:07 |
| *** SmearedBeard has quit IRC | 01:11 | |
| *** dsneddon has quit IRC | 01:15 | |
| openstackgerrit | LIU Yulong proposed openstack/neutron master: Minimizing L3 agent QoS extensions lock granularity https://review.opendev.org/656163 | 01:15 |
| openstackgerrit | LIU Yulong proposed openstack/neutron master: Minimizing lock for port forwarding https://review.opendev.org/656165 | 01:15 |
| *** dsneddon has joined #openstack-neutron | 01:20 | |
| *** dsneddon has quit IRC | 01:29 | |
| *** baojg has joined #openstack-neutron | 01:32 | |
| *** dsneddon has joined #openstack-neutron | 01:39 | |
| *** yamamoto has joined #openstack-neutron | 01:41 | |
| *** sean-k-mooney has quit IRC | 01:42 | |
| *** tbachman has quit IRC | 01:49 | |
| *** tbachman has joined #openstack-neutron | 01:51 | |
| *** yamamoto has quit IRC | 02:03 | |
| *** iyamahat has quit IRC | 02:24 | |
| *** yamahata has quit IRC | 02:26 | |
| *** itlinux has joined #openstack-neutron | 02:28 | |
| *** partlycloudy has joined #openstack-neutron | 02:34 | |
| *** itlinux has quit IRC | 02:38 | |
| *** yamamoto has joined #openstack-neutron | 02:39 | |
| *** yamamoto has quit IRC | 02:51 | |
| *** yamamoto has joined #openstack-neutron | 02:54 | |
| *** JamesBenson has joined #openstack-neutron | 02:59 | |
| openstackgerrit | Guo Jingyu proposed openstack/neutron-lib master: Add sort_keys retreiving function https://review.opendev.org/657913 | 03:31 |
| *** dsneddon has quit IRC | 03:43 | |
| *** dsneddon has joined #openstack-neutron | 03:45 | |
| *** dsneddon has quit IRC | 03:49 | |
| *** lpetrut has joined #openstack-neutron | 03:53 | |
| *** ramishra has joined #openstack-neutron | 03:55 | |
| *** wwriverrat has quit IRC | 04:03 | |
| *** ramishra has quit IRC | 04:03 | |
| *** dsneddon has joined #openstack-neutron | 04:10 | |
| *** gcheresh has joined #openstack-neutron | 04:12 | |
| *** ratailor has joined #openstack-neutron | 04:14 | |
| *** lpetrut has quit IRC | 04:18 | |
| *** ramishra has joined #openstack-neutron | 04:18 | |
| *** gcheresh has quit IRC | 04:34 | |
| *** ramishra has quit IRC | 04:34 | |
| *** dsneddon has quit IRC | 04:37 | |
| *** janki has joined #openstack-neutron | 04:38 | |
| *** gcheresh has joined #openstack-neutron | 04:44 | |
| *** ivve has quit IRC | 04:45 | |
| *** gcheresh has quit IRC | 04:54 | |
| *** JamesBenson has quit IRC | 05:01 | |
| *** gcheresh has joined #openstack-neutron | 05:15 | |
| *** samueldmq has quit IRC | 05:19 | |
| *** gcheresh has quit IRC | 05:25 | |
| *** dsneddon has joined #openstack-neutron | 05:34 | |
| *** ramishra has joined #openstack-neutron | 05:41 | |
| *** yamamoto has quit IRC | 05:42 | |
| *** yamamoto has joined #openstack-neutron | 05:46 | |
| *** ivve has joined #openstack-neutron | 05:49 | |
| *** yamamoto has quit IRC | 05:51 | |
| *** ksambor has joined #openstack-neutron | 06:02 | |
| *** hjensas has joined #openstack-neutron | 06:04 | |
| *** slaweq has joined #openstack-neutron | 06:06 | |
| openstackgerrit | Reedip proposed openstack/networking-ovn master: Support for Router Scheduling on addition/removal of chassis https://review.opendev.org/591461 | 06:06 |
| *** ccamposr has joined #openstack-neutron | 06:09 | |
| *** whoami-rajat has joined #openstack-neutron | 06:20 | |
| openstackgerrit | zhongshengping proposed openstack/neutron-lib master: Update Python 3 test runtimes for Train https://review.opendev.org/657927 | 06:26 |
| *** Luzi has joined #openstack-neutron | 06:26 | |
| *** lajoskatona has joined #openstack-neutron | 06:26 | |
| *** yamamoto has joined #openstack-neutron | 06:28 | |
| openstackgerrit | zhongshengping proposed openstack/neutron master: Update Python 3 test runtimes for Train https://review.opendev.org/657940 | 06:28 |
| *** yamamoto has quit IRC | 06:29 | |
| *** yamamoto has joined #openstack-neutron | 06:30 | |
| *** dsneddon has quit IRC | 06:49 | |
| *** dsneddon has joined #openstack-neutron | 06:58 | |
| *** maciejjozefczyk has joined #openstack-neutron | 07:09 | |
| *** tesseract has joined #openstack-neutron | 07:09 | |
| *** dsneddon has quit IRC | 07:11 | |
| *** aedc has quit IRC | 07:15 | |
| *** dsneddon has joined #openstack-neutron | 07:15 | |
| *** gcheresh has joined #openstack-neutron | 07:16 | |
| *** panda|off is now known as panda | 07:19 | |
| *** orzel has left #openstack-neutron | 07:27 | |
| *** rpittau|afk is now known as rpittau | 07:35 | |
| *** dsneddon has quit IRC | 07:45 | |
| *** jpena|off is now known as jpena | 07:45 | |
| *** gcheresh has quit IRC | 07:47 | |
| *** aojea has joined #openstack-neutron | 07:57 | |
| *** aedc has joined #openstack-neutron | 08:17 | |
| *** lucasagomes has joined #openstack-neutron | 08:18 | |
| *** ralonsoh has joined #openstack-neutron | 08:20 | |
| *** dsneddon has joined #openstack-neutron | 08:21 | |
| *** tkajinam has quit IRC | 08:30 | |
| *** aedc has quit IRC | 08:31 | |
| *** dsneddon has quit IRC | 08:36 | |
| *** SmearedBeard has joined #openstack-neutron | 08:37 | |
| *** SmearedBeard has quit IRC | 08:38 | |
| *** jangutter has joined #openstack-neutron | 08:39 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/networking-ovn master: Distributed OVSDB lock: HashRing common methods and DB migration https://review.opendev.org/655407 | 08:41 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/networking-ovn master: Distributed OVSDB lock: Make use of the HashRing https://review.opendev.org/655408 | 08:41 |
| *** tobias-urdin has joined #openstack-neutron | 08:41 | |
| lucasagomes | bcafarel, around ? If you have few minutes mind taking a look at https://review.opendev.org/#/c/657584/ ? The stable/stein branch is broken and that will fix it | 08:47 |
| *** aedc has joined #openstack-neutron | 08:47 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/neutron master: FdbInterfaceTestCase interface names should be randomly generated https://review.opendev.org/657958 | 08:49 |
| openstackgerrit | Guo Jingyu proposed openstack/neutron-lib master: Add sort_keys retreiving function https://review.opendev.org/657913 | 08:49 |
| *** panda has quit IRC | 08:55 | |
| openstackgerrit | Guo Jingyu proposed openstack/neutron-lib master: Add sort_keys retreiving function https://review.opendev.org/657913 | 08:55 |
| *** panda has joined #openstack-neutron | 08:55 | |
| *** SmearedBeard has joined #openstack-neutron | 08:57 | |
| *** davidsha has joined #openstack-neutron | 08:58 | |
| *** gmann_pto has joined #openstack-neutron | 08:58 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/neutron master: [DNM] Remove rootwrap-daemon from neutron-keepalived-state-change https://review.opendev.org/657849 | 09:06 |
| *** dsneddon has joined #openstack-neutron | 09:13 | |
| *** dsneddon has quit IRC | 09:17 | |
| *** tssurya has joined #openstack-neutron | 09:18 | |
| openstackgerrit | zhongshengping proposed openstack/neutron-lib master: Update Python 3 test runtimes for Train https://review.opendev.org/657927 | 09:36 |
| openstackgerrit | zhongshengping proposed openstack/neutron master: Update Python 3 test runtimes for Train https://review.opendev.org/657940 | 09:37 |
| *** dsneddon has joined #openstack-neutron | 09:43 | |
| *** yamamoto has quit IRC | 09:45 | |
| *** dsneddon has quit IRC | 09:48 | |
| *** tssurya has quit IRC | 09:57 | |
| *** ramishra has quit IRC | 10:04 | |
| *** pbandark has joined #openstack-neutron | 10:05 | |
| *** yamamoto has joined #openstack-neutron | 10:06 | |
| *** ramishra has joined #openstack-neutron | 10:13 | |
| *** mugsie has quit IRC | 10:25 | |
| *** yamamoto has quit IRC | 10:27 | |
| *** yamamoto has joined #openstack-neutron | 10:27 | |
| *** yamamoto has quit IRC | 10:28 | |
| *** yamamoto has joined #openstack-neutron | 10:32 | |
| *** yamamoto has quit IRC | 10:32 | |
| *** yamamoto_ has joined #openstack-neutron | 10:32 | |
| *** davidsha has quit IRC | 10:33 | |
| *** yamamoto_ has quit IRC | 10:33 | |
| *** mugsie has joined #openstack-neutron | 10:35 | |
| *** mugsie has quit IRC | 10:35 | |
| *** mugsie has joined #openstack-neutron | 10:36 | |
| *** mugsie has quit IRC | 10:38 | |
| *** mugsie has joined #openstack-neutron | 10:39 | |
| *** tesseract has quit IRC | 10:40 | |
| *** tesseract has joined #openstack-neutron | 10:41 | |
| *** tssurya has joined #openstack-neutron | 10:42 | |
| *** priteau has joined #openstack-neutron | 10:43 | |
| *** dsneddon has joined #openstack-neutron | 10:44 | |
| *** tesseract has quit IRC | 10:45 | |
| *** tesseract has joined #openstack-neutron | 10:45 | |
| *** tbachman has quit IRC | 10:48 | |
| *** dsneddon has quit IRC | 10:49 | |
| *** aedc has quit IRC | 10:52 | |
| *** panda is now known as panda|lunch | 11:02 | |
| openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: DNM Try to delay a bit ip addr list https://review.opendev.org/657565 | 11:03 |
| *** yamamoto has joined #openstack-neutron | 11:08 | |
| openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: [DNM] Remove rootwrap-daemon from neutron-keepalived-state-change https://review.opendev.org/657849 | 11:08 |
| slaweq | ralonsoh: ^^ I "improved" a bit Your patch :) | 11:09 |
| ralonsoh | slaweq, sure, perfect! | 11:09 |
| *** yamamoto has quit IRC | 11:13 | |
| *** priteau has quit IRC | 11:23 | |
| *** yamamoto has joined #openstack-neutron | 11:34 | |
| *** jpena is now known as jpena|lunch | 11:39 | |
| *** dsneddon has joined #openstack-neutron | 11:44 | |
| *** dsneddon has quit IRC | 11:48 | |
| *** gmann_pto has quit IRC | 11:50 | |
| *** janki has quit IRC | 11:54 | |
| *** lukasz_asdf has joined #openstack-neutron | 11:55 | |
| *** zigo has quit IRC | 11:59 | |
| *** aedc has joined #openstack-neutron | 12:02 | |
| *** yamamoto has quit IRC | 12:02 | |
| *** tbachman has joined #openstack-neutron | 12:05 | |
| *** panda|lunch is now known as panda | 12:06 | |
| *** yamamoto has joined #openstack-neutron | 12:07 | |
| *** aedc has quit IRC | 12:07 | |
| *** baojg has quit IRC | 12:08 | |
| *** iyamahat has joined #openstack-neutron | 12:15 | |
| *** aedc has joined #openstack-neutron | 12:16 | |
| *** iyamahat has quit IRC | 12:21 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/networking-ovn master: PEP8: Fix bandit exclude path https://review.opendev.org/658064 | 12:22 |
| *** zigo has joined #openstack-neutron | 12:23 | |
| *** jpena|lunch is now known as jpena | 12:31 | |
| *** yamamoto has quit IRC | 12:40 | |
| slaweq | haleyb: hi | 12:46 |
| slaweq | haleyb: can You take a look at https://bugs.launchpad.net/neutron/+bug/1828364 ? I'm not sure if this is really a bug or it is like that "by design" and should stay like that | 12:46 |
| openstack | Launchpad bug 1828364 in neutron "DVR: Fip namespaces are created in all the controllers and computes with vm only when an external interface to the router is added" [Undecided,New] - Assigned to Slawek Kaplonski (slaweq) | 12:47 |
| *** njohnston has joined #openstack-neutron | 12:47 | |
| haleyb | slaweq: i'll look, i'm thinking it's by design. might have to ping Swami as well | 12:48 |
| slaweq | haleyb: thx | 12:49 |
| lajoskatona | slaweq, ralonsoh: Hi | 12:51 |
| slaweq | lajoskatona: hi | 12:51 |
| lajoskatona | slaweq, ralonsoh: I started to play with routed networks, and found that it is not working, if I understand well due to placement microversion things | 12:51 |
| *** mriedem has joined #openstack-neutron | 12:51 | |
| lajoskatona | slaweq: Is that ok if I open a kind of umbrella bug for that? | 12:52 |
| slaweq | lajoskatona: what you mean by "umbrella bug"? | 12:53 |
| lajoskatona | slaweq: as I see now there will be some small changes how we handle exceptions on different microversion | 12:53 |
| lajoskatona | slaweq: I mean on bug like "routed networks placement microversion error" or similar | 12:53 |
| slaweq | I think that it will be fine | 12:53 |
| lajoskatona | s'on/one' | 12:53 |
| lajoskatona | slaweq: ok | 12:54 |
| *** ratailor has quit IRC | 12:54 | |
| lajoskatona | slaweq: and a more philosophical question, what do you think would be better | 12:56 |
| *** ramishra has quit IRC | 12:56 | |
| *** ramishra has joined #openstack-neutron | 12:56 | |
| lajoskatona | slaweq: routed networks now documented to be used with placement 1.1, make it work with placement 1.1 | 12:56 |
| lajoskatona | slaweq: or "upgrade" it to use 1.20 which we use for the placement reported for QoS min bw reporting? | 12:57 |
| openstackgerrit | Merged openstack/networking-ovn master: PEP8: Fix bandit exclude path https://review.opendev.org/658064 | 12:58 |
| slaweq | lajoskatona: so currently in docs we have that is should works in placement 1.1 but it don't work like that? | 12:59 |
| slaweq | is my understanding correct? | 12:59 |
| lajoskatona | slaweq: yes | 13:00 |
| slaweq | lajoskatona: and is it like that in stable branch(es) also? | 13:00 |
| openstackgerrit | Daniel Alvarez proposed openstack/networking-ovn master: Update device_owner for subports https://review.opendev.org/656907 | 13:00 |
| lajoskatona | slaweq: correction: in doc (https://docs.openstack.org/neutron/latest/admin/config-routed-networks.html ) we have 1.1 as minimum microversion | 13:01 |
| slaweq | lajoskatona:so it is only matter of placement API microversion, not placement package version, right? | 13:01 |
| lajoskatona | slaweq: hmmmm, as I see from pike we have this in doc | 13:03 |
| slaweq | lajoskatona:so I think that we can fix docs if that will be enough :) | 13:04 |
| lajoskatona | slaweq: and to tell the truth I can't see in placement microversion history where/when the exception (if we receive back details in exception for get inventory if the resourceprovider is not existing) changed | 13:04 |
| lajoskatona | so on stable branches this can be different, but I have to read more the placement history | 13:04 |
| lajoskatona | slaweq: I check that | 13:05 |
| *** yamamoto has joined #openstack-neutron | 13:06 | |
| slaweq | lajoskatona: so basically if fix would require to bump requirements version than it's not possible for stable branches | 13:07 |
| slaweq | but if this is not the case and stable branches can also use newer placement API version than IMO this should be ok | 13:07 |
| lajoskatona | slaweq: ok I check that | 13:08 |
| *** ramishra has quit IRC | 13:09 | |
| *** boden has joined #openstack-neutron | 13:09 | |
| *** boden has quit IRC | 13:13 | |
| *** samueldmq has joined #openstack-neutron | 13:16 | |
| *** ramishra has joined #openstack-neutron | 13:20 | |
| *** baojg has joined #openstack-neutron | 13:20 | |
| openstackgerrit | Brian Haley proposed openstack/neutron master: Fix bandit exclude path https://review.opendev.org/658075 | 13:25 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/neutron master: Bump Pyroute2 version to 0.5.5 https://review.opendev.org/657752 | 13:29 |
| *** tesseract has quit IRC | 13:35 | |
| *** tesseract has joined #openstack-neutron | 13:35 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/neutron master: Revert "Lock privileged.agent.linux.ip_lib functions" https://review.opendev.org/657608 | 13:35 |
| *** gcheresh has joined #openstack-neutron | 13:42 | |
| *** altlogbot_0 has quit IRC | 13:43 | |
| *** dsneddon has joined #openstack-neutron | 13:45 | |
| *** altlogbot_1 has joined #openstack-neutron | 13:45 | |
| *** boden has joined #openstack-neutron | 13:49 | |
| *** dsneddon has quit IRC | 13:50 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/neutron master: [DNM] Remove rootwrap-daemon from neutron-keepalived-state-change https://review.opendev.org/657849 | 13:55 |
| openstackgerrit | Darragh O'Reilly proposed openstack/neutron master: Make OVS controller inactivity_probe configurable https://review.opendev.org/641681 | 13:59 |
| *** rpittau is now known as rpittau|afk | 14:08 | |
| *** mlavalle has joined #openstack-neutron | 14:13 | |
| *** JamesBenson has joined #openstack-neutron | 14:17 | |
| *** itlinux has joined #openstack-neutron | 14:17 | |
| mlavalle | haleyb, slaweq, ralonsoh, amotoki: do you know that today is njohnston birthday? | 14:19 |
| mlavalle | njohnston: Happy Birthday! | 14:19 |
| ralonsoh | mlavalle, of course hehehe | 14:19 |
| *** JamesBenson has quit IRC | 14:21 | |
| *** itlinux has quit IRC | 14:23 | |
| *** JamesBenson has joined #openstack-neutron | 14:23 | |
| *** tesseract has quit IRC | 14:24 | |
| *** liuyulong has joined #openstack-neutron | 14:25 | |
| *** JamesBenson has quit IRC | 14:28 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/neutron master: Limit monotonic to python < 3.3 https://review.opendev.org/658111 | 14:29 |
| *** JamesBenson has joined #openstack-neutron | 14:29 | |
| *** yamamoto has quit IRC | 14:35 | |
| *** mvkr has quit IRC | 14:37 | |
| *** tesseract has joined #openstack-neutron | 14:50 | |
| *** Luzi has quit IRC | 14:51 | |
| *** tesseract has quit IRC | 14:51 | |
| *** tesseract has joined #openstack-neutron | 14:51 | |
| *** ratailor has joined #openstack-neutron | 14:52 | |
| *** hongbin has joined #openstack-neutron | 14:56 | |
| *** davidsha has joined #openstack-neutron | 14:57 | |
| njohnston | Thank you mlavalle! I'm 44 today, not sure if that makes mea young whippersnapper or a golden oldie | 14:59 |
| mlavalle | njohnston: the former | 14:59 |
| mlavalle | njohnston: the golden oldies are the 59ers like me | 15:00 |
| *** tesseract has quit IRC | 15:01 | |
| *** tesseract has joined #openstack-neutron | 15:03 | |
| *** tesseract has quit IRC | 15:11 | |
| *** wwriverrat has joined #openstack-neutron | 15:11 | |
| *** imacdonn has quit IRC | 15:12 | |
| *** imacdonn has joined #openstack-neutron | 15:12 | |
| *** yamamoto has joined #openstack-neutron | 15:12 | |
| *** pbandark has quit IRC | 15:14 | |
| *** tesseract has joined #openstack-neutron | 15:16 | |
| amotoki | njohnston: happy birthday! | 15:16 |
| amotoki | everyone will be young if they believe themselves so :) | 15:16 |
| *** lajoskatona has quit IRC | 15:17 | |
| *** yamamoto has quit IRC | 15:17 | |
| *** tesseract has quit IRC | 15:18 | |
| *** tesseract has joined #openstack-neutron | 15:20 | |
| liuyulong | njohnston: happy birthday! | 15:23 |
| *** gcheresh has quit IRC | 15:24 | |
| njohnston | thank you amotoki and liuyulong! | 15:28 |
| *** trident has quit IRC | 15:33 | |
| *** trident has joined #openstack-neutron | 15:35 | |
| *** baojg has quit IRC | 15:37 | |
| *** dsneddon has joined #openstack-neutron | 15:39 | |
| *** ivve has quit IRC | 15:40 | |
| openstackgerrit | Reedip proposed openstack/networking-ovn master: DNM Testing support for Router Scheduling https://review.opendev.org/657794 | 15:41 |
| *** mvkr has joined #openstack-neutron | 15:42 | |
| openstackgerrit | Brian Haley proposed openstack/neutron master: Fix bandit exclude path https://review.opendev.org/658075 | 15:46 |
| liuyulong | https://bugs.launchpad.net/neutron/+bug/1609217, hi guys, I just reopen this bug. Opinions needed. : ) | 15:48 |
| openstack | Launchpad bug 1609217 in neutron "DVR: dvr router ns should not exist in scheduled DHCP agent nodes" [Medium,Opinion] | 15:48 |
| *** priteau has joined #openstack-neutron | 15:48 | |
| liuyulong | And it has a very very old fix: https://review.opendev.org/#/c/364793/. gerrit review may give some clue about the bug. | 15:50 |
| *** dsneddon has quit IRC | 15:52 | |
| *** priteau has quit IRC | 15:56 | |
| *** bnemec has quit IRC | 15:58 | |
| *** ijw has joined #openstack-neutron | 15:58 | |
| *** bnemec has joined #openstack-neutron | 15:58 | |
| *** mvkr has quit IRC | 15:59 | |
| *** tesseract has quit IRC | 16:01 | |
| *** davidsha has quit IRC | 16:02 | |
| *** ratailor has quit IRC | 16:03 | |
| *** lucasagomes has quit IRC | 16:13 | |
| *** enyinna1234 has joined #openstack-neutron | 16:18 | |
| openstackgerrit | boden proposed openstack/neutron master: DNM: use SqlFixture and StaticSqlFixture from neutron-lib https://review.opendev.org/657887 | 16:21 |
| *** tssurya has quit IRC | 16:38 | |
| *** maciejjozefczyk has quit IRC | 16:42 | |
| *** jpena is now known as jpena|off | 16:42 | |
| *** irclogbot_1 has quit IRC | 16:46 | |
| *** munimeha1 has joined #openstack-neutron | 16:46 | |
| openstackgerrit | Brian Haley proposed openstack/neutron master: Fix bandit exclude path https://review.opendev.org/658075 | 16:46 |
| ralonsoh | haleyb, http://paste.openstack.org/show/751179/ | 16:47 |
| *** irclogbot_1 has joined #openstack-neutron | 16:47 | |
| ralonsoh | haleyb, just the first part of the execution | 16:47 |
| ralonsoh | haleyb, with bandit==1.5.0 | 16:47 |
| *** aojea has quit IRC | 16:47 | |
| ralonsoh | with 1.6.0, not errors | 16:47 |
| haleyb | ralonsoh: ack, i finally did reproduce it just didn't trigger the first time somehow. One of those was the resource_allocator test code i've been trying to fix :( | 16:48 |
| *** dsneddon has joined #openstack-neutron | 16:49 | |
| *** ramishra has quit IRC | 16:50 | |
| *** dsneddon has quit IRC | 16:54 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/neutron master: Remove QoS related objects unneeded compatibility conversions https://review.opendev.org/658155 | 16:56 |
| *** aedc has quit IRC | 16:58 | |
| *** ccamposr has quit IRC | 17:02 | |
| *** liuyulong has quit IRC | 17:03 | |
| *** dsneddon has joined #openstack-neutron | 17:06 | |
| *** ralonsoh has quit IRC | 17:08 | |
| *** macza has joined #openstack-neutron | 17:12 | |
| enyinna1234 | haleyb, manjeets, slaweq, tidwellr, I'm canvassing for code reviews: https://review.opendev.org/#/c/628071/ | 17:19 |
| *** yamamoto has joined #openstack-neutron | 17:22 | |
| *** Weifan has joined #openstack-neutron | 17:23 | |
| *** ijw has quit IRC | 17:37 | |
| *** rubasov has joined #openstack-neutron | 17:38 | |
| *** ivve has joined #openstack-neutron | 17:42 | |
| *** rubasov has quit IRC | 17:46 | |
| *** livelace has joined #openstack-neutron | 17:47 | |
| *** hongbin has quit IRC | 17:54 | |
| openstackgerrit | Brian Haley proposed openstack/networking-ovn master: Bump bandit version to 1.6.0 https://review.opendev.org/658165 | 17:58 |
| *** ijw has joined #openstack-neutron | 18:00 | |
| *** yamamoto has quit IRC | 18:11 | |
| *** tbachman has quit IRC | 18:13 | |
| openstackgerrit | Merged openstack/networking-ovn master: Update device_owner for subports https://review.opendev.org/656907 | 18:21 |
| *** SmearedBeard has quit IRC | 18:21 | |
| *** hyang has left #openstack-neutron | 18:30 | |
| *** igordc has joined #openstack-neutron | 18:35 | |
| *** igordc has quit IRC | 18:44 | |
| *** Weifan has quit IRC | 18:44 | |
| *** tbachman has joined #openstack-neutron | 18:45 | |
| *** igordc has joined #openstack-neutron | 18:47 | |
| *** yamamoto has joined #openstack-neutron | 18:51 | |
| *** iyamahat has joined #openstack-neutron | 18:51 | |
| *** yamamoto has quit IRC | 19:02 | |
| *** SmearedBeard has joined #openstack-neutron | 19:07 | |
| *** SmearedBeard has quit IRC | 19:13 | |
| *** Weifan has joined #openstack-neutron | 19:15 | |
| *** SmearedBeard has joined #openstack-neutron | 19:15 | |
| *** maciejjozefczyk has joined #openstack-neutron | 19:15 | |
| *** Weifan has quit IRC | 19:19 | |
| *** jobewan has joined #openstack-neutron | 19:23 | |
| *** maciejjozefczyk has quit IRC | 19:23 | |
| *** SmearedBeard has quit IRC | 19:26 | |
| tidwellr | enyinna1234: see my latest comments on https://review.opendev.org/#/c/628071/ | 19:26 |
| *** cheng1 has quit IRC | 19:27 | |
| *** tbachman has quit IRC | 19:30 | |
| *** cheng1 has joined #openstack-neutron | 19:33 | |
| enyinna1234 | Thanks for the review, tidwellr, I replied your comment | 19:42 |
| *** hongbin has joined #openstack-neutron | 19:44 | |
| enyinna1234 | haleyb, thanks. I also replied https://review.opendev.org/#/c/628071/ | 19:44 |
| tidwellr | enyinna1234: I'm not seeing your comments after refreshing the page | 19:48 |
| *** Weifan has joined #openstack-neutron | 19:51 | |
| enyinna1234 | tidwellr: Look again. I replied to the post | 20:02 |
| *** Weifan has quit IRC | 20:03 | |
| *** boden_ has joined #openstack-neutron | 20:15 | |
| *** boden has quit IRC | 20:19 | |
| *** SmearedBeard has joined #openstack-neutron | 20:30 | |
| *** livelace has quit IRC | 20:31 | |
| *** dsneddon has quit IRC | 20:39 | |
| mriedem | mlavalle: i've got some port binding (extended) questions about the flow when messing with multiple port bindings during a migration | 20:40 |
| mriedem | one is, if we start a migration with an inactive port binding on the dest host and an active port binding on the source host, and the port's binding:host_id points at the source, if we activate the dest host binding does the port's binding:host_id value automatically get updated? | 20:41 |
| mriedem | it seems that it does during live migration with the new port binding stuff we added in rocky | 20:41 |
| mlavalle | yes, it should | 20:41 |
| mriedem | ok so what if rather than activate the dest host port binding directly, i update the port's binding:host_id to point at the dest host, would you expect that to automatically activate the dest host binding and update the source host binding to be inactive? | 20:42 |
| mriedem | right now we're getting a UniqueConstraint error if we do ^ | 20:44 |
| mriedem | i don't know if we're losing a race or what | 20:44 |
| mlavalle | no. when you activate a binding you are calling this: https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/plugin.py#L2458. If you update the port, you are updating the same port binding that pointed previously to the source host | 20:45 |
| mlavalle | you are not changing the binding | 20:45 |
| mlavalle | update port is not going to call the new port binding activation | 20:46 |
| mriedem | ok so that would explain the UniqueConstraint then because neutron is trying to have an active and inactive binding on the same host for the same port which violates the constraint | 20:48 |
| mriedem | sean mooney described it in this workaround patch https://review.opendev.org/#/c/653506/ but i wasn't sure if that's what was really happening on the neutron side | 20:48 |
| mlavalle | to your point: https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/models.py#L42 | 20:49 |
| mriedem | so to recap the issue, if we have binding for port1 on hostA as active, and port1 on hostB which is inactive, and then update the port['binding:host_id'] = hostB, neutron will try to change the binding for hostA to be hostB which blows up b/c we already have a binding for hostB | 20:49 |
| mriedem | yup we get something like this | 20:50 |
| mriedem | DBDuplicateEntry: (pymysql.err.IntegrityError) (1062, "Duplicate entry '4b9a5615-c534-4205-892b-947b8460c874-ubuntu-bionic-ovh-gra1-0005' for key 'PRIMARY'") [SQL: 'UPDATE ml2_port_bindings SET host=%(host)s, vif_type=%(vif_type)s, vif_details=%(vif_details)s WHERE ml2_port_bindings.port_id = %(ml2_port_bindings_port_id)s AND ml2_port_bindings.host = %(ml2_port_bindings_host)s'] [parameters: {'host': 'ubuntu-bionic-o | 20:50 |
| mriedem | ra1-0005970038', 'vif_type': 'unbound', 'vif_details': '', 'ml2_port_bindings_port_id': '4b9a5615-c534-4205-892b-947b8460c874', 'ml2_port_bindings_host': 'ubuntu-bionic-ovh-gra1-0005970036'}] | 20:50 |
| mlavalle | yes, looking at the db model makes it crystal clear | 20:50 |
| mriedem | what wasn't clear to me was what neutron does when the port's binding:host_id was updated | 20:51 |
| mriedem | that's a bit confusing since we have the bindings records and the thing on the port | 20:51 |
| mriedem | i'm actually really surprised more people aren't hitting this with live migration in rocky+ | 20:54 |
| mriedem | mnaser: you haven't hit https://bugs.launchpad.net/nova/+bug/1822884 right? | 20:55 |
| openstack | Launchpad bug 1822884 in OpenStack Compute (nova) "live migration fails due to port binding duplicate key entry in post_live_migrate" [Undecided,In progress] - Assigned to sean mooney (sean-k-mooney) | 20:55 |
| mlavalle | mriedem: in this patch we converted the relationship between the port and its binding(s) to a list and also changed the way to access the binding to use a utility function: https://review.opendev.org/#/c/414251/74 | 20:59 |
| mlavalle | https://review.opendev.org/#/c/414251/74/neutron/plugins/ml2/models.py | 21:00 |
| *** logan- has quit IRC | 21:00 | |
| mlavalle | https://review.opendev.org/#/c/414251/74/neutron/common/utils.py | 21:01 |
| mlavalle | and then this patch pluralized the relationship name | 21:02 |
| *** logan- has joined #openstack-neutron | 21:03 | |
| *** mchlumsky has quit IRC | 21:04 | |
| *** Weifan has joined #openstack-neutron | 21:20 | |
| openstackgerrit | boden proposed openstack/neutron master: use payloads for PORT AFTER_CREATE events https://review.opendev.org/634461 | 21:31 |
| *** dasp has quit IRC | 21:52 | |
| *** boden_ has quit IRC | 21:55 | |
| *** tbachman has joined #openstack-neutron | 21:58 | |
| mriedem | mlavalle: thanks for the details, i think i have another way to trigger a recreate of that bug in the gate to test the theory | 21:58 |
| mlavalle | mriedem: glad to help :-) | 21:58 |
| *** JamesBenson has quit IRC | 21:59 | |
| *** ijw has quit IRC | 22:02 | |
| *** ijw has joined #openstack-neutron | 22:03 | |
| *** hongbin has quit IRC | 22:04 | |
| *** hongbin has joined #openstack-neutron | 22:05 | |
| *** hongbin has quit IRC | 22:05 | |
| *** hongbin has joined #openstack-neutron | 22:06 | |
| *** hongbin has quit IRC | 22:06 | |
| *** hongbin has joined #openstack-neutron | 22:06 | |
| *** ijw has quit IRC | 22:07 | |
| *** hongbin has quit IRC | 22:08 | |
| *** hongbin has joined #openstack-neutron | 22:08 | |
| *** trident has quit IRC | 22:19 | |
| *** tbachman has quit IRC | 22:19 | |
| *** enyinna1234 has quit IRC | 22:21 | |
| *** enyinna1234 has joined #openstack-neutron | 22:22 | |
| *** trident has joined #openstack-neutron | 22:22 | |
| wwriverrat | Anyone have the time to review the PTG requested spec for: Multi-segment-per-host spec? Pretty please, with sugar on top? :-P | 22:32 |
| wwriverrat | https://review.opendev.org/#/c/657170/ | 22:32 |
| *** markmcclain has quit IRC | 22:35 | |
| *** markmcclain has joined #openstack-neutron | 22:37 | |
| mlavalle | wwriverrat: I am actually writing the summary and was looking at the spec. I will give feedback over the next few days | 22:39 |
| mlavalle | it will be in my radar screen | 22:40 |
| mlavalle | thanks for the quick submission | 22:40 |
| *** iyamahat has quit IRC | 22:42 | |
| wwriverrat | woohoo! :) | 22:47 |
| *** mlavalle has quit IRC | 22:49 | |
| *** slaweq has quit IRC | 22:51 | |
| *** munimeha1 has quit IRC | 22:52 | |
| *** SmearedBeard has quit IRC | 22:56 | |
| *** macza has quit IRC | 22:57 | |
| *** SmearedBeard has joined #openstack-neutron | 22:58 | |
| *** slaweq has joined #openstack-neutron | 22:58 | |
| *** slaweq has quit IRC | 23:03 | |
| *** tkajinam has joined #openstack-neutron | 23:03 | |
| *** slaweq has joined #openstack-neutron | 23:11 | |
| *** slaweq has quit IRC | 23:16 | |
| *** SmearedBeard has quit IRC | 23:17 | |
| *** samueldmq has quit IRC | 23:31 | |
| *** whoami-rajat has quit IRC | 23:34 | |
| *** jobewan has quit IRC | 23:36 | |
| *** enyinna1234 has quit IRC | 23:40 | |
| *** eandersson has joined #openstack-neutron | 23:41 | |
| *** dsneddon has joined #openstack-neutron | 23:44 | |
| *** dsneddon has quit IRC | 23:51 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!