Tuesday, 2019-11-26

*** macz has quit IRC00:05
*** mattw4 has quit IRC00:12
*** goldyfruit_ has quit IRC00:20
*** mmethot_ has quit IRC00:21
*** mmethot has joined #openstack-neutron00:22
openstackgerritMerged openstack/neutron master: Update security group rule if port range is all ports  https://review.opendev.org/68905000:23
openstackgerritMerged openstack/neutron stable/train: NetcatTester.stop_processes skip "No such process" exception  https://review.opendev.org/69565600:31
*** mvkr has quit IRC01:04
*** goldyfruit_ has joined #openstack-neutron01:05
*** awalende has joined #openstack-neutron01:05
*** awalende has quit IRC01:10
*** nanzha has joined #openstack-neutron01:10
*** ociuhandu has joined #openstack-neutron01:12
*** igordc has quit IRC01:13
*** mvkr has joined #openstack-neutron01:17
*** nanzha has quit IRC01:19
*** nanzha has joined #openstack-neutron01:19
*** ociuhandu has quit IRC01:28
*** ociuhandu has joined #openstack-neutron01:40
*** ociuhandu has quit IRC01:44
openstackgerritReedip proposed openstack/neutron master: Dont schedule Network, respecting network_auto_schedule config  https://review.opendev.org/69364501:55
*** armax has quit IRC01:57
*** nanzha has quit IRC01:58
*** nanzha has joined #openstack-neutron02:10
*** armax has joined #openstack-neutron02:31
*** ociuhandu has joined #openstack-neutron02:38
openstackgerritMerged openstack/neutron-vpnaas master: Use cidr from tempest config project_network_v6_cidr  https://review.opendev.org/69546102:47
*** ociuhandu has quit IRC02:48
*** nanzha has quit IRC02:52
*** nanzha has joined #openstack-neutron02:53
*** chenhaw has joined #openstack-neutron02:53
*** tonyb has joined #openstack-neutron03:00
*** awalende has joined #openstack-neutron03:13
*** awalende has quit IRC03:18
*** dsneddon has quit IRC03:40
*** ociuhandu has joined #openstack-neutron03:43
*** ociuhandu has quit IRC03:47
*** nanzha has quit IRC03:50
*** ociuhandu has joined #openstack-neutron03:53
*** nanzha has joined #openstack-neutron03:55
*** hongbin has joined #openstack-neutron04:00
*** ociuhandu has quit IRC04:01
*** ociuhandu has joined #openstack-neutron04:02
*** ociuhandu has quit IRC04:07
*** dsneddon has joined #openstack-neutron04:08
*** dsneddon has quit IRC04:13
*** ccamposr__ has joined #openstack-neutron04:33
*** ccamposr has quit IRC04:36
*** dsneddon has joined #openstack-neutron05:02
*** ratailor has joined #openstack-neutron05:11
*** nanzha has quit IRC05:20
*** nanzha has joined #openstack-neutron05:24
*** gcheresh_ has joined #openstack-neutron05:25
*** ociuhandu has joined #openstack-neutron05:30
*** ociuhandu has quit IRC05:35
*** tkajinam has quit IRC05:37
*** tkajinam has joined #openstack-neutron05:38
*** lajoskatona has joined #openstack-neutron05:43
*** gcheresh_ has quit IRC05:44
*** markmcclain has quit IRC05:56
*** markmcclain has joined #openstack-neutron05:57
*** nanzha has quit IRC06:05
*** nanzha has joined #openstack-neutron06:06
*** hongbin has quit IRC06:18
*** sapd1 has joined #openstack-neutron06:28
*** pcaruana has joined #openstack-neutron06:32
*** ratailor has quit IRC06:38
*** ratailor has joined #openstack-neutron06:42
*** yedongcan has joined #openstack-neutron06:42
*** nanzha has quit IRC06:45
*** nanzha has joined #openstack-neutron06:54
*** sapd1 has quit IRC06:57
*** nanzha has quit IRC06:59
*** nanzha has joined #openstack-neutron07:08
*** nanzha has quit IRC07:20
*** nanzha has joined #openstack-neutron07:21
*** slaweq has joined #openstack-neutron07:23
*** bobmel has joined #openstack-neutron07:25
*** chenhaw has quit IRC07:26
*** chenhaw has joined #openstack-neutron07:26
*** slaweq has quit IRC07:28
*** nanzha has quit IRC07:44
*** gcheresh_ has joined #openstack-neutron07:45
*** gcheresh has joined #openstack-neutron07:49
*** gcheresh_ has quit IRC07:50
openstackgerritEran Kuris proposed openstack/networking-ovn master: Added OSP16 & train to pluging spec.  https://review.opendev.org/69602707:51
openstackgerritEran Kuris proposed openstack/networking-ovn stable/train: Added OSP16 & train to pluging spec.  https://review.opendev.org/69602907:57
*** slaweq has joined #openstack-neutron07:57
*** nanzha has joined #openstack-neutron07:58
*** chenhaw has quit IRC08:01
*** maciejjozefczyk has joined #openstack-neutron08:02
slaweqamotoki: hi, will You maybe have time to review https://review.opendev.org/#/c/658414/ today?08:06
*** tesseract has joined #openstack-neutron08:16
*** ratailor has quit IRC08:17
*** ratailor_ has joined #openstack-neutron08:18
*** chenhaw has joined #openstack-neutron08:18
*** tkajinam has quit IRC08:19
*** rubasov has quit IRC08:26
*** rubasov has joined #openstack-neutron08:27
*** jlibosva has joined #openstack-neutron08:29
*** nanzha has quit IRC08:33
*** ccamposr__ has quit IRC08:41
*** ccamposr__ has joined #openstack-neutron08:42
*** nanzha has joined #openstack-neutron08:43
*** jpena|off is now known as jpena08:44
*** pbandark has joined #openstack-neutron08:45
*** dsneddon has quit IRC08:50
*** ralonsoh has joined #openstack-neutron08:53
*** lucasagomes has joined #openstack-neutron08:57
*** priteau has joined #openstack-neutron08:58
*** macz has joined #openstack-neutron08:59
*** macz has quit IRC09:03
*** tssurya has joined #openstack-neutron09:04
*** ociuhandu has joined #openstack-neutron09:04
openstackgerritMerged openstack/networking-odl master: Remove networking-odl python2 jobs  https://review.opendev.org/68785809:05
*** ociuhandu has quit IRC09:08
*** nanzha has quit IRC09:09
*** nanzha has joined #openstack-neutron09:10
amotokislaweq: hi, sure. I will check it today.09:12
slaweqamotoki: thx a lot09:12
amotokiI was PTO yesterday after returning from San Diego :)09:12
openstackgerritBin Lu proposed openstack/neutron master: add optional argument metric to set metric for routes  https://review.opendev.org/63958509:15
openstackgerritMaciej Józefczyk proposed openstack/networking-ovn master: Fallback to default network_driver value if octavia not present  https://review.opendev.org/69585809:15
*** dsneddon has joined #openstack-neutron09:24
*** nanzha has quit IRC09:29
openstackgerritNoy fridman proposed openstack/neutron-tempest-plugin master: Add test to create policy with multiple rules  https://review.opendev.org/69591409:29
*** nanzha has joined #openstack-neutron09:33
*** ociuhandu has joined #openstack-neutron09:37
*** davidsha has joined #openstack-neutron09:41
*** ociuhandu has quit IRC09:43
openstackgerritMerged openstack/neutron-lib master: Remove 'interconnection' API extension definition (neutron-interconnection)  https://review.opendev.org/69446609:44
*** ociuhandu has joined #openstack-neutron09:44
*** abaindur has quit IRC09:47
*** ociuhandu has quit IRC09:48
*** nanzha has quit IRC09:59
*** dsneddon has quit IRC09:59
*** ociuhandu has joined #openstack-neutron10:00
*** nanzha has joined #openstack-neutron10:09
*** ccamposr__ has quit IRC10:16
*** dsneddon has joined #openstack-neutron10:19
*** mkrai has joined #openstack-neutron10:20
mkraiHi, I am trying to understand Neutron code. Can some point the code to network create API method in neutron repo? Thanks in advance10:20
*** dsneddon has quit IRC10:25
*** ivve has joined #openstack-neutron10:35
ivvehello10:38
ivveseems neutron 14.0.2 is not compatible with 14.0.3? RemoteError: Remote error: InvalidTargetVersion Invalid target version 1.5 from ovs neutron agent as well as neutron server10:39
ivveim seeing this on a fresh deployed compute into a stack with 14.0.210:40
ivvethe fresh compute has 14.0.310:40
openstackgerritMerged openstack/neutron-vpnaas master: Switch to Ussuri jobs  https://review.opendev.org/69121610:45
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron master: [WIP] [OVN] Add DB tables for OVN backend  https://review.opendev.org/69596910:46
*** ccamposr has joined #openstack-neutron10:47
*** lpetrut has joined #openstack-neutron10:52
fricklerivve: the usual upgrade procedure would be to upgrade controllers and network nodes first. though I agree that this should not happen for a patch release. minor version bumps are happening far too seldom10:53
ivvefrickler: yeah i didn't bother to upgrade anything else as it is such a minor10:55
openstackgerritMaciej Józefczyk proposed openstack/networking-ovn master: Add support for multiple L4 protocols withing same LB  https://review.opendev.org/67129310:55
*** jlibosva has quit IRC10:55
ivveso neither are upgraded, and this went into stein stable (via kolla)10:55
*** takamatsu has quit IRC10:55
ivvebut also observe i said "seems" :)10:56
ivvemy conclusion isn't 100% yet10:56
ivveim testing an older container just to be sure10:56
ivvebut its 2 hosts with the exact 14.0.3 version that are misbehaving and i have some 55ish working with 14.0.210:56
ivveredeployed them just to be sure but same thing is happening10:57
*** dsneddon has joined #openstack-neutron10:58
fricklerivve: I'd say https://review.opendev.org/669360 is the culprit11:00
*** ociuhandu has quit IRC11:03
ivvethanks frickler11:04
ivveim testing and older version in about an hour, lunchtime now :)11:04
openstackgerritFederico Ressi proposed openstack/networking-ovn master: Use pip_install to install six python module  https://review.opendev.org/69498711:05
*** nanzha has quit IRC11:07
*** ccamposr has quit IRC11:08
*** nanzha has joined #openstack-neutron11:10
*** nanzha has quit IRC11:17
*** priteau has quit IRC11:19
*** nanzha has joined #openstack-neutron11:24
*** nanzha has quit IRC11:29
*** nanzha has joined #openstack-neutron11:29
*** dsneddon has quit IRC11:30
*** yedongcan has left #openstack-neutron11:32
*** tbachman has quit IRC11:40
*** rcernin has quit IRC11:40
*** ociuhandu has joined #openstack-neutron11:41
*** macz has joined #openstack-neutron11:41
openstackgerritMerged openstack/neutron stable/queens: Run revision bump operations en masse  https://review.opendev.org/69569311:41
*** mkrai has quit IRC11:44
*** macz has quit IRC11:45
*** ociuhandu has quit IRC11:46
openstackgerritAditya Reddy Nagaram proposed openstack/neutron-lib master: Support for stateless security groups  https://review.opendev.org/69607011:55
openstackgerritAditya Reddy Nagaram proposed openstack/neutron master: [WIP] Support for stateless security groups  https://review.opendev.org/57276711:59
*** mkrai has joined #openstack-neutron12:03
*** dsneddon has joined #openstack-neutron12:03
*** klippo has joined #openstack-neutron12:14
*** jlibosva has joined #openstack-neutron12:18
*** dtantsur|afk is now known as dtantsur12:20
*** jlibosva has quit IRC12:29
*** jlibosva has joined #openstack-neutron12:32
openstackgerritPedro Henrique Pereira Martins proposed openstack/neutron-lib master: Add description field in port forwarding API  https://review.opendev.org/69258012:37
*** sean-k-mooney has quit IRC12:39
*** sean-k-mooney has joined #openstack-neutron12:40
*** nanzha has quit IRC12:45
ivvehttps://bugs.launchpad.net/neutron/+bug/185405012:46
openstackLaunchpad bug 1854050 in neutron "minor versions 14.0.2 & 14.0.3 are not compatible in dvr-ha" [Undecided,New]12:46
ivvei have a feeling this is a critical bug12:47
*** jawad_axd has joined #openstack-neutron12:48
*** ratailor_ has quit IRC13:03
*** tbachman has joined #openstack-neutron13:04
*** dsneddon has quit IRC13:08
*** tbachman has quit IRC13:08
*** goldyfruit_ has quit IRC13:08
*** tbachman has joined #openstack-neutron13:16
*** ociuhandu has joined #openstack-neutron13:20
*** liuyulong has joined #openstack-neutron13:23
*** ociuhandu has quit IRC13:26
*** jpena is now known as jpena|lunch13:26
*** bobmel has quit IRC13:27
*** tbachman has quit IRC13:27
*** bobmel has joined #openstack-neutron13:27
*** tbachman has joined #openstack-neutron13:31
openstackgerritAditya Reddy Nagaram proposed openstack/neutron-lib master: Support for stateless security groups  https://review.opendev.org/69607013:32
*** dsneddon has joined #openstack-neutron13:36
openstackgerritAditya Reddy Nagaram proposed openstack/neutron-lib master: add security-group-stateful api extension  https://review.opendev.org/69607013:37
openstackgerritGregoire Mahe proposed openstack/neutron master: Allow to parse keywords in dns labels  https://review.opendev.org/68634313:46
openstackgerritLIU Yulong proposed openstack/neutron master: Not process port security groups twice  https://review.opendev.org/69348413:53
openstackgerritElod Illes proposed openstack/networking-bagpipe stable/queens: Add local bindep.txt and make unit tests pass  https://review.opendev.org/65782813:58
*** tkajinam has joined #openstack-neutron14:00
*** jpena|lunch is now known as jpena14:18
openstackgerritSlawek Kaplonski proposed openstack/neutron master: Rehome networking-ovn CI jobs to neutron repository  https://review.opendev.org/69609414:18
openstackgerritSlawek Kaplonski proposed openstack/neutron master: Rehome networking-ovn CI jobs to neutron repository  https://review.opendev.org/69609414:20
*** ociuhandu has joined #openstack-neutron14:22
liuyulongThese is an official python3.7 rpm release for CentOS7?14:24
*** ociuhandu has quit IRC14:27
openstackgerritNate Johnston proposed openstack/neutron master: [DNM] Test py36 unit tests  https://review.opendev.org/69609614:28
*** dsneddon has quit IRC14:33
openstackgerritLajos Katona proposed openstack/neutron master: use objects common_types from neutron-lib  https://review.opendev.org/66464614:35
*** goldyfruit has joined #openstack-neutron14:36
*** dsneddon has joined #openstack-neutron14:38
*** goldyfruit_ has joined #openstack-neutron14:38
*** goldyfruit has quit IRC14:41
*** dsneddon has quit IRC14:45
*** jawad_axd has quit IRC14:48
*** ociuhandu has joined #openstack-neutron14:49
openstackgerritAditya Reddy Nagaram proposed openstack/neutron-lib master: add security-group-stateful api extension  https://review.opendev.org/69607014:50
openstackgerritAditya Reddy Nagaram proposed openstack/neutron-lib master: add stateful-security-group api extension  https://review.opendev.org/69607014:52
*** ociuhandu has quit IRC14:54
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron master: [WIP] [OVN] Add hash ring methods  https://review.opendev.org/69610315:06
*** dsneddon has joined #openstack-neutron15:10
openstackgerritMaciej Józefczyk proposed openstack/networking-ovn master: Add support for multiple L4 protocols withing same LB  https://review.opendev.org/67129315:12
*** ociuhandu has joined #openstack-neutron15:19
*** tkajinam has quit IRC15:19
openstackgerritMaciej Józefczyk proposed openstack/networking-ovn master: Add gateway chassis rebalance script  https://review.opendev.org/66304615:25
openstackgerritMaciej Józefczyk proposed openstack/networking-ovn master: Disable voting for tripleo-ci-centos-7-containers-multinode  https://review.opendev.org/66509215:26
openstackgerritMaciej Józefczyk proposed openstack/networking-ovn master: Add gateway chassis rebalance script  https://review.opendev.org/66304615:26
*** ociuhandu has quit IRC15:28
*** lajoskatona has quit IRC15:32
*** ociuhandu has joined #openstack-neutron15:42
*** awalende has joined #openstack-neutron15:45
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron master: [OVN] Add DB tables for OVN backend  https://review.opendev.org/69596915:46
*** awalende has quit IRC15:50
*** mlavalle has joined #openstack-neutron15:51
*** ccamposr has joined #openstack-neutron15:53
*** spatel has joined #openstack-neutron15:54
openstackgerritAditya Reddy Nagaram proposed openstack/neutron master: [WIP] Support for stateless security groups  https://review.opendev.org/57276716:00
openstackgerritMaciej Józefczyk proposed openstack/neutron master: Move OVN constants to neutron tree  https://review.opendev.org/69611816:01
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron master: Do not inherit from built-in "dict"  https://review.opendev.org/69187416:01
*** ivve has quit IRC16:02
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron master: [OVN] Add hash ring methods  https://review.opendev.org/69610316:02
*** dtantsur is now known as dtantsur|afk16:03
openstackgerritTerry Wilson proposed openstack/networking-ovn master: Fix agent extension support after hashring merge  https://review.opendev.org/69484016:03
openstackgerritTerry Wilson proposed openstack/networking-ovn master: Fix agent extension support after hashring merge  https://review.opendev.org/69484016:08
*** gcheresh has quit IRC16:10
*** davidsha has quit IRC16:11
*** ijw has joined #openstack-neutron16:14
openstackgerritFederico Ressi proposed openstack/networking-ovn master: Use pip_install to install six python module  https://review.opendev.org/69498716:18
openstackgerritFederico Ressi proposed openstack/networking-ovn master: Use pip_install to install six python module  https://review.opendev.org/69498716:18
*** jangutter has quit IRC16:22
*** tesseract has quit IRC16:23
*** ociuhandu has quit IRC16:33
*** ociuhandu has joined #openstack-neutron16:34
openstackgerritBernard Cafarelli proposed openstack/networking-sfc master: Drop python 2 support and testing  https://review.opendev.org/69612816:38
gibimlavalle: hi! just a heads up I opened a bug about the missing api documentation for the multiple port binding api https://bugs.launchpad.net/neutron/+bug/1853873 if I know corretly you were the author of the implementation of that api16:38
openstackLaunchpad bug 1853873 in neutron "The /v2.0/ports/{port_id}/bindings APIs are not documented" [Medium,New] - Assigned to Chason Chan (chen-xing)16:38
mlavallegibi: Thanks!16:39
*** jlibosva has quit IRC16:39
*** jpena is now known as jpena|brb16:45
*** macz has joined #openstack-neutron16:45
*** lucasagomes has quit IRC16:46
*** mkrai has quit IRC16:47
njohnstonslaweq: FYI do you know who runs the Mellanox CI?  It looks like that has not run on Neutron changes since Nov 20th.16:56
slaweqnjohnston: I have no idea16:57
slaweqbut I would start asking from adrianc :)16:57
njohnstonmlavalle: Did you get the mail I sent about the profiling decorator?16:57
mlavallenjohnston: I just read it16:57
* slaweq is going afk for now, will be back online later tonight probably16:58
njohnstonadrianc: Hi!  Do you know who runs the Mellanox CI?  It looks like that has not run on Neutron changes since Nov 20th.16:58
njohnstonmlavalle: I am mostly worried that I am using it wrong16:58
mlavallenjohnston: yeah, it looks odd17:01
mlavallethis is an example: https://review.opendev.org/#/c/689640/4/neutron/plugins/ml2/plugin.py17:02
mlavalleare you using it like this?17:02
njohnstonmlavalle: I believe so: https://review.opendev.org/#/c/695307/2/neutron/plugins/ml2/plugin.py17:04
*** ivve has joined #openstack-neutron17:06
openstackgerritNate Johnston proposed openstack/neutron master: [DNM] profile create_port to find slowness  https://review.opendev.org/69530717:06
mlavallenjohnston: that paste you included in the email, came from one job in specific, right?17:13
njohnstonmlavalle: Yeah, it came from https://9e08b63720ebd1424e24-d183cc649f1ed183c61dad720f170c6c.ssl.cf1.rackcdn.com/695307/2/check/neutron-rally-task/030fbf9/controller/logs/screen-q-svc.txt.gz17:14
njohnstonI just picked an example of a port creation that was longer than the 60 second point when a client would timeout the request17:15
*** ociuhandu has quit IRC17:15
*** bobmel has quit IRC17:19
*** njohnston is now known as njohnston|lunch17:19
mlavalleok, I'll look at the results of the latest revision17:20
*** igordc has joined #openstack-neutron17:20
mlavalleat some point soon. Do you have a pressing deadline you want to meet using this tool?17:21
*** jpena|brb is now known as jpena17:21
*** spatel has quit IRC17:28
openstackgerritAditya Reddy Nagaram proposed openstack/neutron-lib master: add stateful-security-group api extension  https://review.opendev.org/69607017:41
openstackgerritGhanshyam Mann proposed openstack/python-neutronclient master: Move grenade neutron-lib job to py3 and in python-neutronclient repo  https://review.opendev.org/69578917:45
*** openstackgerrit has quit IRC17:49
*** tbachman has quit IRC18:10
*** tssurya has quit IRC18:12
*** armax has quit IRC18:14
*** ijw has quit IRC18:26
*** jpena is now known as jpena|off18:31
*** njohnston|lunch is now known as njohnston18:33
njohnstonmlavalle: No deadline, just trying to debug an issue :-)18:33
njohnstonmlavalle: Thanks for taking a look18:33
mlavallenjohnston: cool. I'll keep an eye on the patch and leave my comments there18:34
njohnstonmlavalle: The background is that I was contacted by one of the Octavia guys because port creation was so slow that it was causing their CI to break, because port creation was > 60s18:35
*** ijw has joined #openstack-neutron18:37
mlavallethat's useful backgraound. Yeah, I'll help as much as I can and lets get those Octavia guys (like rm_work of johnsom) off of our backs18:38
johnsomlol18:38
rm_worklol18:39
mlavalleThe Octavia Northwesterners18:39
johnsomI resemble that remark18:39
rm_workhey, everyone's northwest of something18:39
*** ijw has quit IRC18:40
rm_workso in a way, we're ALL northwesterners :)18:40
*** ijw has joined #openstack-neutron18:40
mlavallerm_work: that's true. as always, you have a smart response :-)18:40
*** goldyfruit___ has joined #openstack-neutron18:41
rm_workif i can't make sarcastic / witty responses, what's the point of life ;)18:42
*** goldyfruit_ has quit IRC18:43
*** ralonsoh has quit IRC18:51
*** goldyfruit_ has joined #openstack-neutron18:51
*** goldyfruit___ has quit IRC18:54
*** CeeMac has quit IRC18:57
*** aedc has joined #openstack-neutron18:57
*** gcheresh has joined #openstack-neutron19:02
*** ijw_ has joined #openstack-neutron19:03
*** ijw has quit IRC19:06
*** goldyfruit_ has quit IRC19:07
*** maciejjozefczyk has quit IRC19:08
*** lpetrut has quit IRC19:08
*** goldyfruit has joined #openstack-neutron19:09
*** goldyfruit_ has joined #openstack-neutron19:19
*** bobmel has joined #openstack-neutron19:20
*** goldyfruit has quit IRC19:22
*** bobmel has quit IRC19:25
*** lajoskatona has joined #openstack-neutron19:36
*** aedc has quit IRC19:38
*** aedc has joined #openstack-neutron19:39
*** abaindur has joined #openstack-neutron19:39
*** abaindur has quit IRC19:40
*** abaindur has joined #openstack-neutron19:40
abainduranyone know why this function does not actually do anything except LOG an error message? https://github.com/openstack/neutron/blob/master/neutron/db/agents_db.py#L53919:59
abaindurit's called from server side report_state RPC handler. the agent timestamp is not ignored as it is in case where its older than server start time20:01
abaindurhttps://github.com/openstack/neutron/blob/master/neutron/db/agents_db.py#L50920:01
abaindurshouldn't we discard instead of writing to DB in such case where time diff is > agent down time? whats the purpose of just logging error msg?20:02
*** numans has quit IRC20:04
*** aedc has quit IRC20:10
*** goldyfruit___ has joined #openstack-neutron20:17
*** goldyfruit_ has quit IRC20:20
*** goldyfruit_ has joined #openstack-neutron20:21
*** goldyfruit___ has quit IRC20:24
njohnstonabaindur: Read https://bugs.launchpad.net/neutron/+bug/1432582 for the reasoning that went into the decision20:25
openstackLaunchpad bug 1432582 in neutron "OVS agent shows connected to AMQP but neutron server rejects the request." [Undecided,Fix released] - Assigned to Sudipta Biswas (sbiswas7)20:25
abainduryea I read that - wondering why we don't return though? What is the purpose of writing to the DB an old timestamp, one that we know will still mark agent as dead?20:25
abainduras I understand, the purpose of that bug/fix was basically, to change it from a debug to raise an error20:27
njohnstonabaindur: What's the tangible difference between logging and writing to DB versus just logging?20:27
njohnstonto put it another way: what if it isn't the compute that has the wrong time, but rather the controller?20:28
abaindurWas looking at it in relation to https://bugs.launchpad.net/neutron/+bug/185307120:28
openstackLaunchpad bug 1853071 in neutron "AMQP disconnects, q-reports-plugin queue grows, leading to DBDeadlocks while trying to update agent heartbeats" [High,New]20:28
abaindurit puts uneccesary strain on RPC workers and extra DB connections, potentially leading to that issue20:28
abaindursuppose neutron-server is down for some time, or the DB is down. the q-reports-plugin Rabbit quueue grows as agents keep sending heartbeats.20:29
abaindurwhen server recovers, it has to process hundreds, thousands of heartbeats. the only effect is they will be stale, but neutron instead if busy processing those20:30
*** lajoskatona has quit IRC20:39
*** gcheresh has quit IRC20:44
abaindurnjohnston: and i guess that commit only even logs error message initially, when agent first reports state. I was talking about discarding all HB timestamps that are older than agent_down_time - not just the initial one20:45
*** pbandark has quit IRC20:45
njohnstonabaindur: I think the change you recommend is a good one, I would +2 it20:45
abaindurfyi - i tried and while it doesnt fix the DBDeadlock in bug above, it does help alleviate rabbit queue growth/size20:46
eanderssonYea the DBDeadlock hits us bad at larger scale20:48
eanderssonIt is especially bad after an upgrade for some reason.20:48
eanderssonTakes hours for the control plane to recover.20:48
*** bobmel has joined #openstack-neutron20:51
abaindureandersson: I found the fix to this issue20:51
abaindurwe need to enable new OVO engine facade for agent and port DB objects20:52
abaindurhttps://opendev.org/openstack/neutron/commit/6f83466307fb21aee5bb596974644d457ae1fa6020:53
abaindurNow I dont fully understand why the fix works (using the new context manager thats set when new_facade = True), but it works20:53
*** bobmel has quit IRC20:55
*** gcheresh has joined #openstack-neutron20:56
njohnstonslaweq mlavalle ^^ engine facade FTW21:01
*** armax has joined #openstack-neutron21:03
*** threestrands has joined #openstack-neutron21:27
abaindurWhy isn't everything using new_facade? or why is it not set in base base.NeutronDbObject?21:30
*** tbachman has joined #openstack-neutron21:30
*** Satarel has quit IRC21:31
*** igordc has quit IRC21:33
abaindurnjohnston: any reason why every object class isnt using new_facade?21:35
njohnstonabaindur: There are a variety of things that break when you turn on the new facade globally.  There are a number of context usages that need to be sorted out.  I believe it is currently being worked on by mlavalle.21:35
abainduryea i saw some commits that reverted new engine code, like https://bugs.launchpad.net/neutron/+bug/174482921:36
openstackLaunchpad bug 1744829 in neutron "Avoid mixed usage of old and new transaction styles" [High,Fix released] - Assigned to Lujin Luo (luo-lujin)21:36
njohnstonabaindur: If you set new_facade = True in the base object in a DNM change you'll see21:36
abaindurDNM?21:36
*** ijw_ has quit IRC21:36
*** openstackgerrit has joined #openstack-neutron21:37
openstackgerritNate Johnston proposed openstack/neutron master: [DNR] Check new engine facade effectiveness  https://review.opendev.org/60203521:37
abaindurFor now, in our easy to repro setup, we turned it on in Agent and PortBinding class only (since some of the stuck Deadlock'd transactions showed queries in history on those tables)21:37
njohnstonDNM = Do Not Merge21:38
abaindurbut i am wondering about side effects even from this21:38
*** igordc has joined #openstack-neutron21:39
njohnstonabaindur: I revivied an old change that checks this, we can watch https://review.opendev.org/#/c/602035/ and see how bad the breakage is21:39
abaindurDo you foresee a problem with for now, at least enabling it on Agent (and maybe portBinding classes?)21:40
abainduras a fix for https://bugs.launchpad.net/neutron/+bug/185307121:41
openstackLaunchpad bug 1853071 in neutron "AMQP disconnects, q-reports-plugin queue grows, leading to DBDeadlocks while trying to update agent heartbeats" [High,New]21:41
njohnstonabaindur: I would say, push an upstream change with just those and let's see how bad the tests break, to start21:41
njohnstonabaindur: I can help you work through the issues, I have some background with engine facade21:41
*** gcheresh has quit IRC21:52
eanderssonbtw talking about db. Has security group performs been fixed yet?21:58
eanderssonIn Rocky with our large site the API is unusable.21:58
*** abaindur has quit IRC22:01
*** abaindur has joined #openstack-neutron22:01
*** ociuhandu has joined #openstack-neutron22:02
*** rcernin has joined #openstack-neutron22:04
*** bobmel has joined #openstack-neutron22:06
*** ociuhandu has quit IRC22:08
*** bobmel has quit IRC22:11
*** pcaruana has quit IRC22:16
*** bobmel has joined #openstack-neutron22:20
abaindurnjohnston: failing unit tests on my end just now22:21
abainduroslo_db.exception.DBReferenceError: (sqlite3.IntegrityError) foreign key constraint failed [SQL: u'INSERT INTO ports (project_id, id, name, network_id, mac_address, admin_state_up, status, device_id, device_owner, ip_allocation, standard_attr_id) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)'] [parameters: (None, 'a9124d5d-6453-4d8f-83e7-c79e3720d3c9', None, 'bb9a094c-edf0-4d35-8553-d8296a97a73e', 'fa:16:3e:3d:46:76', 1, 'ACTIVE', '01422:21
abaindur6dea6-ff57-4e00-a23d-2a49a8518bd0', 'network:router_ha_interface', None, 2)] (Background on this error at: http://sqlalche.me/e/gkpj)22:21
abaindurin test__get_ha_router_interface_ids_with_ha_replicated_port22:22
abaindur17 failures total, some different: http://paste.openstack.org/show/786760/22:25
*** bobmel has quit IRC22:27
openstackgerritFlavio Fernandes proposed openstack/networking-ovn master: DNM: This is a hack, please ignore  https://review.opendev.org/69620022:30
*** slaweq has quit IRC22:37
*** armax has quit IRC22:42
*** tkajinam has joined #openstack-neutron23:08
*** slaweq has joined #openstack-neutron23:11
*** goldyfruit___ has joined #openstack-neutron23:13
*** slaweq has quit IRC23:15
*** goldyfruit_ has quit IRC23:16
*** goldyfruit_ has joined #openstack-neutron23:17
*** goldyfruit___ has quit IRC23:20
*** ociuhandu has joined #openstack-neutron23:30
*** ociuhandu has quit IRC23:35
*** armax has joined #openstack-neutron23:38
*** goldyfruit_ has quit IRC23:49

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!