Tuesday, 2020-07-28

*** zhanglong has joined #openstack-neutron00:13
*** hamalq has quit IRC00:41
*** gmann has quit IRC00:51
*** gmann has joined #openstack-neutron00:51
*** nplanel has quit IRC00:51
*** CeeMac has quit IRC00:52
*** andrewbogott has quit IRC00:53
*** nplanel has joined #openstack-neutron00:54
openstackgerritMerged openstack/networking-ovn stable/train: [OVN] Enable qos service plugin in tempest jobs  https://review.opendev.org/73056700:54
*** CeeMac has joined #openstack-neutron00:55
*** andrewbogott has joined #openstack-neutron00:55
*** zhanglong has quit IRC00:59
*** zhanglong has joined #openstack-neutron00:59
*** ociuhandu has joined #openstack-neutron01:01
*** ociuhandu has quit IRC01:05
openstackgerritMerged openstack/neutron stable/ussuri: Fix neutron-ovn-tempest-ovs-master-fedora job  https://review.opendev.org/74251901:14
openstackgerritLi YaJie proposed openstack/neutron master: Fix delete-accepted-egress-direct-flow issue on deleting VM  https://review.opendev.org/73144601:21
openstackgerritFlavio Fernandes proposed openstack/neutron master: [ovn]: port forwarding -- core changes  https://review.opendev.org/74130301:35
*** baojg has quit IRC01:38
*** baojg has joined #openstack-neutron01:41
openstackgerritFlavio Fernandes proposed openstack/neutron master: [ovn]: port forwarding -- IDL changes  https://review.opendev.org/74100001:48
*** ramishra has joined #openstack-neutron01:53
openstackgerritFlavio Fernandes proposed openstack/neutron master: [ovn]: port forwarding -- IDL changes  https://review.opendev.org/74100002:15
*** zhanglong has quit IRC02:18
*** zhanglong has joined #openstack-neutron02:19
*** sapd1 has joined #openstack-neutron02:33
openstackgerritFlavio Fernandes proposed openstack/neutron master: [ovn]: port forwarding -- core changes  https://review.opendev.org/74130302:39
openstackgerritFlavio Fernandes proposed openstack/neutron master: [ovn]: port forwarding -- functional tests for maintenance task  https://review.opendev.org/74173102:42
*** ircuser-1 has joined #openstack-neutron02:46
*** nweinber has quit IRC02:46
openstackgerritFlavio Fernandes proposed openstack/neutron master: [ovn]: port forwarding -- functional tests for maintenance task  https://review.opendev.org/74173102:53
*** rcernin has quit IRC02:59
*** hongbin has joined #openstack-neutron03:00
*** rcernin has joined #openstack-neutron03:03
*** nweinber has joined #openstack-neutron03:12
*** nweinber has quit IRC03:15
*** armax has quit IRC03:33
*** lajoskatona has joined #openstack-neutron04:15
*** hongbin has quit IRC04:20
*** ratailor has joined #openstack-neutron04:37
*** gcheresh has joined #openstack-neutron04:56
*** abdysn has joined #openstack-neutron05:01
*** psahoo has joined #openstack-neutron05:02
*** ociuhandu has joined #openstack-neutron05:03
*** sapd1 has quit IRC05:04
*** ociuhandu has quit IRC05:07
*** waleedm has joined #openstack-neutron05:11
openstackgerritvinay harsha mitta proposed openstack/neutron-lib master: Neutron agent-list API added with sort_key keyword  https://review.opendev.org/73847105:13
*** sapd1 has joined #openstack-neutron05:47
*** zhanglong has quit IRC05:49
*** lxkong has quit IRC05:55
*** lpetrut has joined #openstack-neutron06:04
*** maciejjozefczyk has joined #openstack-neutron06:10
*** baojg has quit IRC06:15
*** marbindrakon has quit IRC06:17
*** sapd1 has quit IRC06:28
*** marbindrakon has joined #openstack-neutron06:47
*** slaweq has joined #openstack-neutron06:48
openstackgerritzhufl proposed openstack/neutron master: [Trivial]Add missing space between words  https://review.opendev.org/74347107:00
*** jlibosva has joined #openstack-neutron07:00
*** bengates has joined #openstack-neutron07:07
*** tesseract has joined #openstack-neutron07:11
*** baojg has joined #openstack-neutron07:21
*** sapd1 has joined #openstack-neutron07:22
*** ccamposr__ has joined #openstack-neutron07:26
*** ccamposr has quit IRC07:29
*** rcernin has quit IRC07:32
*** bjolo has joined #openstack-neutron07:44
zigoAs expected, my added unit test looks like working: https://review.opendev.org/#/c/669395/ and it fails excatly how I expected.07:50
zigoDuring a debugging session, it appears that while the added subnet is in the subnet object, its service_types aren't in the subnet_service_types object yet at the time of this PR check, which is why it fails. How can I make it so that subnet_service_types are added *PRIOR* the check, and both gets rollback if the test fails?07:50
*** ociuhandu has joined #openstack-neutron07:52
*** ociuhandu has quit IRC07:52
*** ociuhandu has joined #openstack-neutron07:53
*** ralonsoh has joined #openstack-neutron07:53
*** jpena|off is now known as jpena07:54
openstackgerritLajos Katona proposed openstack/networking-odl master: Fix pep8 job  and small fixes for functional jobs  https://review.opendev.org/74112407:58
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron master: Specify C shared library in Pyroute2 namespace context  https://review.opendev.org/72225408:04
*** lucasagomes has joined #openstack-neutron08:06
*** jlibosva1 has joined #openstack-neutron08:45
*** jlibosva is now known as Guest4696408:45
*** jlibosva1 is now known as jlibosva08:45
*** Guest46964 has quit IRC08:47
*** rcernin has joined #openstack-neutron08:48
*** priteau has joined #openstack-neutron08:50
*** ociuhandu_ has joined #openstack-neutron08:53
*** rcernin has quit IRC08:54
slaweqralonsoh: hi08:55
ralonsohhi08:55
slaweqralonsoh: according to https://bugs.launchpad.net/neutron/+bug/1887523 I just created BP: https://blueprints.launchpad.net/neutron/+spec/handle-deadlocks-in-oslo-way08:55
openstackLaunchpad bug 1887523 in neutron "Deadlock detection code can be stale" [High,In progress]08:55
slaweqto track progress with this08:55
slaweqand I added You as an approver of the BP - is that ok for You?08:55
slaweqI don't think we need to discuss that in the drivers meeting as regular RFEs08:56
ralonsohslaweq, sure, actually I wrote this topic for the next drivers meeting08:56
slaweqit's more like refactoring08:56
ralonsohok08:56
*** ociuhandu has quit IRC08:56
ralonsohslaweq, but I think this is a good idea08:56
ralonsohand that could be done in parallel to njohnston testing FW08:57
slaweqyes, I agree that we should use oslo code instead of our own08:57
openstackgerritLajos Katona proposed openstack/networking-bgpvpn master: Fix pep8 job  https://review.opendev.org/74028709:01
*** marbindrakon has quit IRC09:02
openstackgerritLajos Katona proposed openstack/networking-bgpvpn master: Switch to hacking 3.0.1  https://review.opendev.org/71044709:03
openstackgerritLajos Katona proposed openstack/networking-bgpvpn stable/ussuri: Fix pep8 job  https://review.opendev.org/74348709:04
openstackgerritLajos Katona proposed openstack/networking-bgpvpn stable/ussuri: Fix pep8 job  https://review.opendev.org/74348709:06
*** marbindrakon has joined #openstack-neutron09:09
*** jangutter has joined #openstack-neutron09:10
*** dtantsur|afk is now known as dtantsur09:12
*** jangutter_ has quit IRC09:14
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron master: Add Loki testing for Neutron  https://review.opendev.org/73913909:18
*** ociuhandu_ has quit IRC09:25
openstackgerritMerged openstack/ovn-octavia-provider master: Add installation of octavia-tempest-plugin to sample local.conf  https://review.opendev.org/74239109:26
openstackgerritMerged openstack/ovn-octavia-provider master: Fix member_batch_update function to follow api v2  https://review.opendev.org/74241009:26
openstackgerritMerged openstack/ovn-octavia-provider master: Set the CONF.load_balancer.enabled_provider_drivers tempest conf  https://review.opendev.org/74242809:26
*** ociuhandu has joined #openstack-neutron09:31
*** ociuhandu has quit IRC09:31
*** ociuhandu has joined #openstack-neutron09:31
openstackgerritLajos Katona proposed openstack/networking-odl stable/ussuri: This change is a squash of 2 different patches to pass the gate  https://review.opendev.org/72553309:33
openstackgerritLucas Alvares Gomes proposed openstack/neutron master: Change some tempest tests to explicitly use ML2/OVS  https://review.opendev.org/74096809:33
openstackgerritMaciej Józefczyk proposed openstack/ovn-octavia-provider stable/ussuri: Set the CONF.load_balancer.enabled_provider_drivers tempest conf  https://review.opendev.org/74349109:34
*** Luzi has joined #openstack-neutron09:45
zigoslaweq: Hi ! You told me that subnet creation starts here: https://github.com/openstack/neutron/blob/master/neutron/db/db_base_plugin_v2.py#L80509:48
zigohowever, I'm reaching https://github.com/openstack/neutron/blob/master/neutron/db/ipam_backend_mixin.py#L356 before that (and my code fails there for the moment).09:48
zigoSo, what am I missing?09:48
ralonsohzigo, I think you should go to the ml2plugin create_subnet method09:54
ralonsohhttps://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/plugin.py#L123509:55
slaweqzigo: yes, ralonsoh is right, it probably starts in ml2 plugin code and later method from base class is also executed09:56
slaweqsorry09:56
ralonsohBTW, you can use pycharm to debug in a graphical env. You can run the server in debug mode and see the trace call09:56
zigoThanks.09:56
zigoI think I'm about to tackle the issue anyways ! :)09:56
zigoWhen the subnet is created, I should just add the service_types from subnet_args as arguments to _validate_segments, and see in there if it's using network:routed09:57
zigoThis should work, I believe ! :)09:57
*** marbindrakon has quit IRC10:05
*** marbindrakon has joined #openstack-neutron10:10
*** ramishra has quit IRC10:15
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron master: Specify C shared library in Pyroute2 namespace context  https://review.opendev.org/72225410:18
*** rcernin has joined #openstack-neutron10:19
*** psahoo has quit IRC10:23
openstackgerritFlavio Fernandes proposed openstack/neutron master: [ovn]: port forwarding -- core changes  https://review.opendev.org/74130310:27
openstackgerritFlavio Fernandes proposed openstack/neutron master: [ovn]: port forwarding -- functional tests for maintenance task  https://review.opendev.org/74173110:30
*** psahoo has joined #openstack-neutron10:41
*** rcernin has quit IRC10:43
openstackgerritRodolfo Alonso Hernandez proposed openstack/ovsdbapp master: Add a cause description to "TimeoutException"  https://review.opendev.org/73982110:50
*** ramishra has joined #openstack-neutron10:58
*** sapd1 has quit IRC11:07
*** lucasagomes has quit IRC11:11
openstackgerritMerged openstack/neutron stable/ussuri: [OVN] Allow IP allocation with different segments for OVN service ports  https://review.opendev.org/74285111:12
openstackgerritMerged openstack/neutron stable/train: [OVN] Allow IP allocation with different segments for OVN service ports  https://review.opendev.org/74287211:12
*** lucasagomes has joined #openstack-neutron11:12
*** psahoo has quit IRC11:23
openstackgerritMerged openstack/neutron master: Remove DbBasePluginCommon._get_subnet unused method  https://review.opendev.org/74282911:27
openstackgerritMerged openstack/neutron master: Set OVS manager command timeout and inactiviy probe  https://review.opendev.org/72071111:27
*** jpena is now known as jpena|lunch11:28
*** jangutter_ has joined #openstack-neutron11:32
*** jangutter has quit IRC11:35
*** psahoo has joined #openstack-neutron11:40
*** psahoo has quit IRC11:45
*** ratailor has quit IRC11:45
*** spatel has joined #openstack-neutron11:50
*** psahoo has joined #openstack-neutron11:53
*** spatel has quit IRC11:54
*** priteau has quit IRC11:55
*** raildo has joined #openstack-neutron11:56
*** priteau has joined #openstack-neutron11:57
openstackgerritMerged openstack/neutron stable/pike: Switch isolated metadata proxy to bind to 169.254.169.254  https://review.opendev.org/73271212:01
openstackgerritMerged openstack/neutron stable/ussuri: ovn-migration: Remove docker references  https://review.opendev.org/74279812:01
*** tkajinam has quit IRC12:02
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron-lib master: Add "supported_extensions" method to MechanismDriver  https://review.opendev.org/74351912:13
*** sapd1 has joined #openstack-neutron12:20
*** ralonsoh has quit IRC12:34
*** ralonsoh has joined #openstack-neutron12:34
*** jpena|lunch is now known as jpena12:34
*** psahoo has quit IRC12:35
openstackgerritLajos Katona proposed openstack/networking-bgpvpn stable/ussuri: Fix pep8 job  https://review.opendev.org/74348712:47
*** psahoo has joined #openstack-neutron12:47
openstackgerritThomas Goirand proposed openstack/neutron master: Floating IP's for routed networks  https://review.opendev.org/66939512:49
zigoslaweq: ^ This worked for me on my test server, I'm sorry I asked so many questions, but it was kind of hard to dive in the code. :)12:50
zigoHopefully, this will pass the new unit test I added...12:50
openstackgerritLajos Katona proposed openstack/networking-bgpvpn stable/ussuri: DNM: test ussuri CI  https://review.opendev.org/74352412:52
*** sapd1 has quit IRC12:53
*** ociuhandu has quit IRC12:57
*** ociuhandu has joined #openstack-neutron12:57
*** rcernin has joined #openstack-neutron12:59
*** rcernin has quit IRC13:03
*** nweinber has joined #openstack-neutron13:15
*** baojg has quit IRC13:20
*** baojg has joined #openstack-neutron13:21
*** ramishra has quit IRC13:27
*** psahoo has quit IRC13:30
*** ramishra has joined #openstack-neutron13:39
*** jangutter_ is now known as jangutter13:39
*** Luzi has quit IRC13:40
*** ccamposr__ has quit IRC13:44
slaweqzigo: thx, I will try to review it today or tomorrow13:47
*** psahoo has joined #openstack-neutron13:50
*** ccamposr has joined #openstack-neutron13:56
openstackgerritSlawek Kaplonski proposed openstack/neutron master: [ovn] Use normalized remote prefix IPs in OVN driver  https://review.opendev.org/73638613:59
*** ramishra has quit IRC13:59
*** ramishra has joined #openstack-neutron14:00
*** liuyulong has joined #openstack-neutron14:00
slaweqhi team, neutron team meeting is starting now in openstack-meeting-3 channel14:01
*** maciejjozefczyk has quit IRC14:03
*** mlavalle has joined #openstack-neutron14:04
mlavalleslaweq: no meeting today?14:10
slaweqmlavalle: there is14:11
slaweqon openstack-meeting-3 channel14:11
slaweqit changed14:11
*** sapd1 has joined #openstack-neutron14:12
*** marbindrakon has quit IRC14:12
*** marbindrakon has joined #openstack-neutron14:18
*** abdysn has quit IRC14:31
openstackgerritLajos Katona proposed openstack/networking-bgpvpn stable/ussuri: DNM: test ussuri CI  https://review.opendev.org/74352414:32
*** psahoo has quit IRC14:37
*** waleedm has quit IRC14:39
*** lpetrut has quit IRC14:40
openstackgerritLajos Katona proposed openstack/networking-odl master: Switch networking-odl jobs to focal  https://review.opendev.org/73670314:44
openstackgerritLajos Katona proposed openstack/networking-bagpipe master: DNM: test bagpipe with focal  https://review.opendev.org/73967214:44
*** maciejjozefczyk has joined #openstack-neutron14:46
*** armax has joined #openstack-neutron14:47
*** psahoo has joined #openstack-neutron14:52
*** baojg has quit IRC15:13
*** baojg has joined #openstack-neutron15:25
*** baojg has quit IRC15:26
*** maciejjozefczyk has quit IRC15:27
*** bengates has quit IRC15:27
*** baojg has joined #openstack-neutron15:27
*** baojg has quit IRC15:33
*** lajoskatona has quit IRC15:34
*** waleedm has joined #openstack-neutron15:56
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron master: Use "replace" to remove a section of a string  https://review.opendev.org/74357716:08
*** priteau has quit IRC16:08
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron master: Use "replace" to remove a section of a string  https://review.opendev.org/74357716:09
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron master: [DNM] [FT] TestMaintenance.test_port failing  https://review.opendev.org/74286816:09
*** jlibosva has quit IRC16:11
*** lucasagomes has quit IRC16:18
*** gcheresh has quit IRC16:18
*** maciejjozefczyk has joined #openstack-neutron16:22
*** dtantsur is now known as dtantsur|afk16:32
*** maciejjozefczyk has quit IRC16:36
*** liuyulong has quit IRC16:36
*** tesseract has quit IRC16:39
*** maciejjozefczyk has joined #openstack-neutron16:41
mlavalleslaweq: https://blueprints.launchpad.net/neutron/+spec/dns-records-in-neutron can be closed16:43
*** ociuhandu has quit IRC16:45
*** hamalq has joined #openstack-neutron16:46
*** mlavalle has quit IRC16:48
*** mlavalle has joined #openstack-neutron16:49
*** psahoo has quit IRC16:50
openstackgerritLukas Steiner proposed openstack/neutron master: Flush ebtables arp protect chains before deleting them  https://review.opendev.org/74058816:50
*** waleedm has quit IRC16:51
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron master: Remove "six" library  https://review.opendev.org/74051016:56
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron master: [OVN] Implement floating IP QoS in OVN backend  https://review.opendev.org/72241516:59
*** psahoo has joined #openstack-neutron16:59
*** maciejjozefczyk has quit IRC17:01
ralonsohmlavalle, https://review.opendev.org/#/c/741784/, just a question I don't understand17:05
mlavalleralonsoh: yes, we need to extend the security gorup rules API. That's another patch, though17:07
ralonsohmlavalle, ok, thanks!17:07
mlavallenp17:07
*** ccamposr has quit IRC17:12
*** jpena is now known as jpena|off17:16
*** psahoo has quit IRC17:27
*** sapd1 has quit IRC17:42
*** ralonsoh has quit IRC17:49
*** ociuhandu has joined #openstack-neutron17:54
openstackgerritFlavio Fernandes proposed openstack/neutron master: [ovn]: port forwarding -- documentation  https://review.opendev.org/74095517:57
*** ociuhandu has quit IRC17:58
openstackgerritFlavio Fernandes proposed openstack/ovn-octavia-provider master: Add integration tests with port forwarding  https://review.opendev.org/74318918:16
*** tbachman has joined #openstack-neutron18:27
*** jamesden_ has joined #openstack-neutron18:27
*** jamesdenton has quit IRC18:30
*** prometheanfire has joined #openstack-neutron19:49
openstackgerritCharles Farquhar proposed openstack/neutron master: Lock sg updates while in _apply_port_filter()  https://review.opendev.org/74362419:49
*** gcheresh has joined #openstack-neutron20:20
openstackgerritSlawek Kaplonski proposed openstack/neutron-lib master: Add API definition for SG rule's normalized_cidr field  https://review.opendev.org/74363021:00
*** lxkong has joined #openstack-neutron21:03
*** gcheresh has quit IRC21:05
openstackgerritFlavio Fernandes proposed openstack/ovn-octavia-provider master: Add integration tests with port forwarding  https://review.opendev.org/74318921:09
openstackgerritSlawek Kaplonski proposed openstack/neutron master: [ovn] Use normalized remote prefix IPs in OVN driver  https://review.opendev.org/73638621:20
*** raildo has quit IRC21:23
*** ccamposr has joined #openstack-neutron21:28
*** slaweq has quit IRC21:34
*** slaweq has joined #openstack-neutron21:50
*** slaweq has quit IRC21:54
*** rcernin has joined #openstack-neutron22:09
*** rcernin has quit IRC22:22
*** yangyapeng has quit IRC22:30
*** yangyapeng has joined #openstack-neutron22:35
*** rcernin has joined #openstack-neutron22:37
*** ircuser-1 has quit IRC22:48
*** rcernin has quit IRC22:50
*** rcernin has joined #openstack-neutron22:50
*** tkajinam has joined #openstack-neutron22:52
*** mlavalle has quit IRC23:01
openstackgerritMerged openstack/networking-odl master: Fix pep8 job  and small fixes for functional jobs  https://review.opendev.org/74112423:03
*** ircuser-1 has joined #openstack-neutron23:05
*** hamalq has quit IRC23:48
openstackgerritMerged openstack/neutron-lib master: Neutron agent-list API added with sort_key keyword  https://review.opendev.org/73847123:53

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!