| opendevreview | Merged openstack/neutron master: Add CONTEXT_WRITER decorator to delete_floatingip_agent_gateway_port https://review.opendev.org/c/openstack/neutron/+/797280 | 00:08 |
|---|---|---|
| opendevreview | Merged openstack/neutron master: Report ExternalDNSOverQuota exception for recordset quota error https://review.opendev.org/c/openstack/neutron/+/786175 | 00:23 |
| opendevreview | Merged openstack/networking-ovn stable/train: [EM releases] Move non-voting jobs to the experimental queue https://review.opendev.org/c/openstack/networking-ovn/+/796474 | 00:25 |
| opendevreview | liuyulong proposed openstack/neutron master: Add agent extension 'dhcp' for ovs agent https://review.opendev.org/c/openstack/neutron/+/776567 | 01:40 |
| opendevreview | liuyulong proposed openstack/neutron master: Add fullstack test case for OVS DHCP extension https://review.opendev.org/c/openstack/neutron/+/776568 | 01:40 |
| opendevreview | liuyulong proposed openstack/neutron master: [L3] Add some logs for router processing https://review.opendev.org/c/openstack/neutron/+/736269 | 01:41 |
| opendevreview | liuyulong proposed openstack/neutron master: Bump neutron-lib to 2.12.0 https://review.opendev.org/c/openstack/neutron/+/796404 | 01:44 |
| opendevreview | liuyulong proposed openstack/neutron master: [QoS] Add rule type packet per second (pps) https://review.opendev.org/c/openstack/neutron/+/796363 | 01:44 |
| opendevreview | Brian Haley proposed openstack/networking-ovn stable/train: Revert "Make networking-ovn-octavia-v2-dsvm-scenario non-voting" https://review.opendev.org/c/openstack/networking-ovn/+/796660 | 01:45 |
| opendevreview | liuyulong proposed openstack/neutron master: [L2] no provisioning block for internal service port https://review.opendev.org/c/openstack/neutron/+/794130 | 01:53 |
| opendevreview | Brian Haley proposed openstack/networking-ovn stable/train: Add Health Monitor support https://review.opendev.org/c/openstack/networking-ovn/+/796063 | 02:38 |
| opendevreview | Brian Haley proposed openstack/networking-ovn stable/train: Add Health Monitor support https://review.opendev.org/c/openstack/networking-ovn/+/796063 | 03:07 |
| opendevreview | Merged openstack/neutron stable/stein: Remove update_initial_state() method from the HA router https://review.opendev.org/c/openstack/neutron/+/777868 | 03:25 |
| *** ricolin_ is now known as ricolin | 05:05 | |
| opendevreview | Dr. Jens Harbott proposed openstack/neutron master: Fix priority review dashboard https://review.opendev.org/c/openstack/neutron/+/797385 | 05:32 |
| frickler | bcafarel: slaweq: ^^ mlavalle noticed that issue yesterday, not sure how that ever worked, or maybe it didn't? | 05:34 |
| opendevreview | Lajos Katona proposed openstack/networking-bgpvpn stable/stein: stable only: Fix failing jobs and make l-c non-voting https://review.opendev.org/c/openstack/networking-bgpvpn/+/797311 | 05:54 |
| bcafarel | frickler: nice I saw yesterday it was broken too, I am pretty sure I had used that link maybe older gerrit handled overquoted queries better? | 06:04 |
| opendevreview | Kamil Sambor proposed openstack/neutron master: Enable querier for multicast (IGMP) in OVN https://review.opendev.org/c/openstack/neutron/+/796997 | 06:09 |
| opendevreview | Lajos Katona proposed openstack/networking-bagpipe stable/train: stable only: Fix failing doc build job https://review.opendev.org/c/openstack/networking-bagpipe/+/797386 | 06:38 |
| slaweq | Thx frickler | 06:42 |
| *** gthiemon1e is now known as gthiemonge | 06:46 | |
| opendevreview | Merged openstack/neutron master: Fix priority review dashboard https://review.opendev.org/c/openstack/neutron/+/797385 | 07:01 |
| *** rpittau|afk is now known as rpittau | 07:06 | |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Use ovs constants from neutron-lib https://review.opendev.org/c/openstack/neutron/+/797120 | 07:35 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Remove ovs agent's common constants module https://review.opendev.org/c/openstack/neutron/+/797121 | 07:35 |
| opendevreview | Mamatisa Nurmatov proposed openstack/neutron master: use callback payloads for SECURITY_GROUP_RULE https://review.opendev.org/c/openstack/neutron/+/792895 | 07:57 |
| opendevreview | Mamatisa Nurmatov proposed openstack/neutron master: use callback payloads for SUBNET https://review.opendev.org/c/openstack/neutron/+/796011 | 08:25 |
| opendevreview | Mamatisa Nurmatov proposed openstack/neutron master: use callback payloads for SECURITY_GROUP_RULE https://review.opendev.org/c/openstack/neutron/+/792895 | 08:29 |
| opendevreview | Bodo Petermann proposed openstack/neutron master: Config option to enable OVN IDL on other workers https://review.opendev.org/c/openstack/neutron/+/795781 | 08:33 |
| opendevreview | Mamatisa Nurmatov proposed openstack/neutron master: use callback payloads for SECURITY_GROUP_RULE https://review.opendev.org/c/openstack/neutron/+/792895 | 08:38 |
| opendevreview | Mamatisa Nurmatov proposed openstack/neutron master: use callback payloads for SECURITY_GROUP https://review.opendev.org/c/openstack/neutron/+/674044 | 08:40 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: [DNM] Remove warning about session with and without enginefacade https://review.opendev.org/c/openstack/neutron-lib/+/791978 | 08:46 |
| opendevreview | Mamatisa Nurmatov proposed openstack/neutron master: use payloads for PORT AFTER_DELETE events https://review.opendev.org/c/openstack/neutron/+/797004 | 08:52 |
| slaweq | ralonsoh lajoskatona hi, can You add https://review.opendev.org/c/openstack/neutron/+/787691 to Your review's pile? Thx in advance | 09:02 |
| slaweq | ralonsoh and also https://review.opendev.org/c/openstack/neutron/+/796230 :) | 09:02 |
| opendevreview | Mamatisa Nurmatov proposed openstack/neutron master: use payloads for PORT AFTER_UPDATE events https://review.opendev.org/c/openstack/neutron/+/795117 | 09:02 |
| ralonsoh | slaweq, sure | 09:03 |
| slaweq | thx | 09:06 |
| bcafarel | ralonsoh: can I add https://review.opendev.org/c/openstack/neutron/+/797273 too? ussuri gate fix | 09:09 |
| opendevreview | Mamatisa Nurmatov proposed openstack/neutron master: use payloads for PORT BEFORE_UPDATE events https://review.opendev.org/c/openstack/neutron/+/795964 | 09:09 |
| ralonsoh | bcafarel, let me check | 09:10 |
| ralonsoh | bcafarel, in the main node L115 should not be needed, I think | 09:10 |
| ralonsoh | anyway, we saw that the subnode does not inherit the parent node config | 09:11 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Populate self.floating_ips_dict using "ip rule" information https://review.opendev.org/c/openstack/neutron/+/794604 | 09:13 |
| bcafarel | ralonsoh: thanks for W+1 (I mostly reused the conf from parent job) | 09:14 |
| bcafarel | vars is for main node config right? in failing log https://abfd123151705ca58b12-2045be852d43868eb95da6cc3429b40d.ssl.cf1.rackcdn.com/797000/1/check/neutron-tempest-slow-py3/44d649a/job-output.txt it was the controller that was running py2 | 09:15 |
| ralonsoh | export PYTHON2_VERSION=2.7 | 09:15 |
| ralonsoh | that's weird | 09:15 |
| bcafarel | indeed :) | 09:16 |
| opendevreview | Elod Illes proposed openstack/networking-bagpipe stable/train: [EM releases] Move non-voting jobs to the experimental queue https://review.opendev.org/c/openstack/networking-bagpipe/+/796477 | 09:29 |
| opendevreview | Elod Illes proposed openstack/networking-bagpipe stable/train: [EM releases] Move non-voting jobs to the experimental queue https://review.opendev.org/c/openstack/networking-bagpipe/+/796477 | 09:31 |
| opendevreview | Yi Yang proposed openstack/neutron-specs master: Add spec for Openflow-based DVR L3 implementation https://review.opendev.org/c/openstack/neutron-specs/+/796746 | 09:40 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron master: [OVN] Disable mcast_flood on localnet ports https://review.opendev.org/c/openstack/neutron/+/797418 | 09:57 |
| gryf | lucasagomes, hi. | 10:06 |
| lucasagomes | gryf, hi there | 10:07 |
| gryf | I've noticed that you have changed this: https://review.opendev.org/c/openstack/neutron/+/793470/3/devstack/plugin.sh#b12 | 10:08 |
| gryf | I know, that docs says about creating special 'stack' user and set it's home to /opt/stack | 10:09 |
| lucasagomes | gryf, right | 10:09 |
| gryf | although it's not hard dependency. so today I | 10:09 |
| gryf | 've | 10:09 |
| lucasagomes | oh yeah, if u have /devstack outside /opt/stack it's a problem... gthiemonge noticed it and was going to fix it | 10:09 |
| gryf | bump on the fact, that ovs_source has been looked for in /opt/stack/decstack | 10:09 |
| lucasagomes | gryf, :-( | 10:09 |
| lucasagomes | gthiemonge, have u proposed a fix for it already ? | 10:10 |
| * lucasagomes looks | 10:10 | |
| gryf | i guess, the fix is easy as replacing mentioned line with "source $TOP_DIR/lib/neutron_plugins/ovs_source" | 10:10 |
| gthiemonge | lucasagomes: gryf: https://review.opendev.org/c/openstack/neutron/+/797128 | 10:10 |
| lucasagomes | gthiemonge++ | 10:10 |
| * lucasagomes reviews | 10:11 | |
| gryf | gthiemonge, right :) | 10:11 |
| lucasagomes | gryf, yeah that's the fix proposed there, sorry for breaking u guys | 10:11 |
| gryf | lucasagomes, no worries :) | 10:11 |
| lucasagomes | the patch is already approved and underway zuul, hopefully we get it in soon | 10:12 |
| gryf | yeah, I can live with manually change it, or… move devstack to /opt/stack ;) | 10:13 |
| velizarx_ | Hey ralonsoh. I want to raise my question about RBAC bgpvpn implementation again. As I see we cannot initialize RBAC Object class for bgpvpn inside networking-bgpvpn repository because of this dinamic load (https://github.com/openstack/neutron/blob/1ad9ca56b07ffdc9f7e0bc6a62af61961b9128eb/neutron/db/rbac_db_models.py#L85). I've already tried to do this and it did not work. So my question, can I add RBACObject and RBAC database model to the | 10:32 |
| velizarx_ | main neutron repo? | 10:32 |
| ralonsoh | velizarx_, what objects are you referring with those RBAC models? | 10:38 |
| ralonsoh | where are the main OVOs located? | 10:38 |
| slaweq | ralonsoh can You +W https://review.opendev.org/c/openstack/neutron/+/796844 maybe? | 10:41 |
| slaweq | thx in advance | 10:41 |
| ralonsoh | slaweq, let me check | 10:41 |
| opendevreview | Merged openstack/neutron stable/ussuri: [stable/ussuri] Set USE_PYTHON3 for neutron-tempest-slow-py3 job https://review.opendev.org/c/openstack/neutron/+/797273 | 10:48 |
| velizarx_ | ralonsoh networking-bgpvpn does not have versioned objects at all, just database models https://github.com/openstack/networking-bgpvpn/blob/master/networking_bgpvpn/neutron/db/bgpvpn_db.py and extension https://github.com/openstack/networking-bgpvpn/blob/master/networking_bgpvpn/neutron/extensions/bgpvpn.py#L97 | 11:27 |
| velizarx_ | this is the main problem that RBAC cannot work with objects which were not initialized in Neutron (OVO) | 11:28 |
| velizarx_ | looks like RBAC cannot control objects from API extensions at all | 11:31 |
| ralonsoh | velizarx_, what you have in bgpvpn are DB models, not OVOs | 11:35 |
| ralonsoh | what is the problem you have when you define those RBAC objects in your project? | 11:35 |
| velizarx_ | ralonsoh if I add special classes for RBAC (subclass for RBACObject and RBAC model as subclass for rbac_db_models.RBACColumns) it does not work. Neutron's RBAC system just cannot find this objects and as result RBAC does not work. Because RBAC tries to load subclasses which were initialized in the main repository (see https://github.com/openstack/neutron/blob/1ad9ca56b07ffdc9f7e0bc6a62af61961b9128eb/neutron/db/rbac_db_models.py#L85 and | 11:41 |
| velizarx_ | https://github.com/openstack/neutron/blob/1ad9ca56b07ffdc9f7e0bc6a62af61961b9128eb/neutron/objects/rbac.py#L56) | 11:41 |
| velizarx_ | need to change get_type_class_map and get_type_model_map to load objects via entrypoints also. Or something like that | 11:43 |
| ralonsoh | velizarx_, right, both "get_type_model_map" and "get_type_class_map" will work only with classes from the Neutron repository | 11:45 |
| ralonsoh | let me check | 11:45 |
| velizarx_ | in my devstack machine I've got error: | 11:46 |
| velizarx_ | File "/opt/stack/neutron-lib/neutron_lib/db/api.py", line 179, in wrapped | 11:46 |
| velizarx_ | return f(*dup_args, **dup_kwargs) | 11:46 |
| velizarx_ | File "/opt/stack/neutron/neutron/db/rbac_db_mixin.py", line 47, in create_rbac_policy | 11:46 |
| velizarx_ | rbac_obj.RBACBaseObject.get_type_class_map()[e['object_type']]) | 11:46 |
| velizarx_ | KeyError: 'bgpvpn' | 11:46 |
| ralonsoh | velizarx_, this is because the RBAC classes have not been loaded | 11:52 |
| ralonsoh | where in you project have you defined the classes? | 11:53 |
| ralonsoh | in your* | 11:53 |
| velizarx_ | ralonsoh I've tried to do it in networking-bgpvpn repository, in files networking_bgpvpn/neutron/db/bgpvpn_db.py and networking_bgpvpn/neutron/extensions/bgpvpn.py where they should be by logic | 11:55 |
| ralonsoh | velizarx_, the plugin class must be loaded. Try to import the module containing the RBAC classes from the plugin module neutron.services.plugin | 11:56 |
| ralonsoh | sorry, I need to go for lunch now | 11:57 |
| velizarx_ | will try, and also will push the code to gerrit (with WIP flag) for discussion | 11:58 |
| *** osmanlicilegi is now known as Guest187 | 12:25 | |
| *** ChanServ changes topic to "Discussion of OpenStack Networking || for support join #openstack" | 12:30 | |
| ralonsoh | velizarx_, is that working? | 12:47 |
| opendevreview | Vadim Ponomarev proposed openstack/neutron-lib master: Introduce rbac-bgpvpn api extension https://review.opendev.org/c/openstack/neutron-lib/+/795423 | 12:48 |
| opendevreview | Merged openstack/networking-ovn stable/train: ovn: Don't use dict.remove() for filtering dhcp ports in db-sync https://review.opendev.org/c/openstack/networking-ovn/+/797001 | 13:11 |
| opendevreview | Merged openstack/neutron stable/wallaby: Add a single option to override the default hypervisor name https://review.opendev.org/c/openstack/neutron/+/796230 | 13:21 |
| opendevreview | Merged openstack/neutron master: Allow to parse keywords in dns labels https://review.opendev.org/c/openstack/neutron/+/686343 | 13:32 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: Enforce new policy defaults and scopes in the API job https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/796612 | 13:39 |
| opendevreview | Merged openstack/neutron master: Force to close http connection after notify about HA router status https://review.opendev.org/c/openstack/neutron/+/796844 | 13:45 |
| opendevreview | Merged openstack/neutron master: Do a quick FIP check on port delete https://review.opendev.org/c/openstack/neutron/+/785633 | 13:46 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Make explicit the network backend used in the CI jobs https://review.opendev.org/c/openstack/neutron/+/797051 | 13:48 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: Add new API tests for active field in the allowed_address_pairs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/795358 | 13:51 |
| slaweq | #startmeeting networking | 14:00 |
| opendevmeet | Meeting started Tue Jun 22 14:00:14 2021 UTC and is due to finish in 60 minutes. The chair is slaweq. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| opendevmeet | The meeting name has been set to 'networking' | 14:00 |
| mlavalle | o/ | 14:00 |
| slaweq | hi | 14:00 |
| ralonsoh | hi | 14:00 |
| gibi | 0/ | 14:00 |
| bcafarel | o/ | 14:01 |
| obondarev | hi | 14:01 |
| slaweq | #topic announcements | 14:01 |
| lajoskatona | Hi | 14:01 |
| slaweq | Xena calendar https://releases.openstack.org/xena/schedule.html | 14:01 |
| slaweq | our next milestone is Xena-2 in the week of the Jul 12th | 14:02 |
| slaweq | so we have couple more weeks before that milestone | 14:02 |
| slaweq | and we should focus on review of the specs from priority dashboard | 14:03 |
| rubasov | o/ | 14:03 |
| slaweq | next one | 14:04 |
| manub | hi | 14:04 |
| slaweq | I don't know if You know but since few weeks there is OpenInfra Live even: https://openinfra.dev/live/#all-episodes | 14:04 |
| slaweq | it's around 1h session every Thursday | 14:04 |
| slaweq | maybe You are interested to watch some episodes or maybe You will have some ideas about presenting something there :) | 14:05 |
| slaweq | and that's all announcements and reminders from me for today | 14:05 |
| slaweq | do You want to share anything else with the team? | 14:05 |
| slaweq | ok, I guess this means "no" :) | 14:06 |
| slaweq | #topic Blueprints | 14:07 |
| slaweq | Xena-2 BPs https://bugs.launchpad.net/neutron/+milestone/xena-2 | 14:07 |
| slaweq | do You have any updates for today? | 14:07 |
| mlavalle | really at this point the priority should be in reviewing the specs, right? | 14:08 |
| slaweq | mlavalle yes, I think so, we can focus on implementation in next weeks still | 14:09 |
| mlavalle | ack | 14:09 |
| slaweq | but maybe there are any other updates about any of BPs | 14:09 |
| * mlavalle didn't mean to subtract importance to on-going BPs | 14:10 | |
| slaweq | :) | 14:11 |
| slaweq | my only update about BPs is to ask for reviews of the patches https://blueprints.launchpad.net/neutron/+spec/distributed-dhcp-for-ml2-ovs | 14:11 |
| slaweq | https://review.opendev.org/q/topic:%22bp%252Fdistributed-dhcp-for-ml2-ovs%22+(status:open%20OR%20status:merged) | 14:11 |
| slaweq | there are only 2 of them left and IMO are pretty ok | 14:12 |
| slaweq | if there are no other updates, let's move on | 14:14 |
| mlavalle | #link https://review.opendev.org/q/topic:%22bp%252Fdistributed-dhcp-for-ml2-ovs%22 | 14:14 |
| slaweq | next topic | 14:14 |
| slaweq | #topic Bugs | 14:14 |
| slaweq | lajoskatona was bug deputy last week | 14:14 |
| slaweq | any bugs You want to highlight now? | 14:14 |
| lajoskatona | the report is here: http://lists.openstack.org/pipermail/openstack-discuss/2021-June/023190.html | 14:14 |
| lajoskatona | As I checked today all what I thought as critical and without owner is handled (https://bugs.launchpad.net/neutron/+bug/1932483 & https://bugs.launchpad.net/neutron/+bug/1933026 ) | 14:15 |
| lajoskatona | so I think we are good | 14:15 |
| slaweq | ++ | 14:16 |
| slaweq | thx | 14:16 |
| slaweq | any other bugs anyone wants to discuss today? | 14:16 |
| lajoskatona | perhaps https://launchpad.net/bugs/1930401 wit hpathc: https://review.opendev.org/c/openstack/oslo.privsep/+/794993 | 14:17 |
| ralonsoh | yes, we should ask core reviewers to check this patch | 14:18 |
| lajoskatona | there's 1 +2 on privsep patch, is there a way to push it, to tell the truth I don't know who to ask for review around oslo | 14:18 |
| ralonsoh | I added them to the patch | 14:18 |
| bcafarel | oslo meeting was yesterday sorry I forgot we had this one to mention | 14:18 |
| ralonsoh | I'll ping them in the oslo channel today | 14:19 |
| bcafarel | though pinging about it in #openstack-oslo may help | 14:19 |
| slaweq | ralonsoh thx | 14:19 |
| lajoskatona | ok, thanks I should have checked if they have active channel | 14:19 |
| slaweq | I also have one bug to mention here | 14:20 |
| slaweq | https://bugs.launchpad.net/ubuntu/+source/neutron/+bug/1927868 - it's a bit older but seems like pretty serious issue potentially | 14:21 |
| slaweq | anyone saw such issues maybe? | 14:21 |
| ralonsoh | we had something similar | 14:21 |
| slaweq | I think we didn't noticed that in the u/s gate | 14:21 |
| ralonsoh | related to keepalived | 14:21 |
| ralonsoh | the problem is they didn't identify what the problem was | 14:21 |
| ralonsoh | https://bugs.launchpad.net/ubuntu/+source/neutron/+bug/1927868/comments/27 | 14:21 |
| slaweq | ralonsoh thx, I also asked them for more logs some time ago but there are no logs there | 14:22 |
| slaweq | ok, let's keep an eye on that one for now | 14:24 |
| mlavalle | looks ugly | 14:24 |
| slaweq | From other things, I would also ask You to review patches https://review.opendev.org/q/topic:%22bug%252F1928466%22+(status:open%20OR%20status:merged) which are related to very old bug https://bugs.launchpad.net/neutron/+bug/1774459 | 14:25 |
| slaweq | obondarev already reviewed it couple of times | 14:26 |
| slaweq | and ralonsoh too | 14:26 |
| slaweq | but I would like to move on with this one now | 14:26 |
| ralonsoh | slaweq, I'm on it | 14:26 |
| ralonsoh | give me one more day to check it | 14:26 |
| slaweq | thx ralonsoh | 14:26 |
| slaweq | no rush, I just wanted to get more attention on those patches :) | 14:27 |
| slaweq | and that's all | 14:27 |
| slaweq | I know it's pretty big and not easy to review | 14:27 |
| slaweq | and that's all from me about bugs today | 14:28 |
| slaweq | bug deputy this week is amotoki | 14:29 |
| slaweq | and next week is mlavalle's tunr | 14:29 |
| slaweq | *turn | 14:29 |
| mlavalle | ack | 14:29 |
| mlavalle | I still expect my notification email | 14:29 |
| amotoki | thanks for the reminder | 14:29 |
| slaweq | mlavalle sure :) | 14:30 |
| slaweq | ok, that's all what I had for today | 14:32 |
| slaweq | do You have anything else You want to discuss today? | 14:32 |
| slaweq | or if not, I will give You some time back today :) | 14:32 |
| mlavalle | \o/ | 14:33 |
| bcafarel | sounds like option 2 :) | 14:33 |
| slaweq | ok, so thx for attending the meeting today | 14:33 |
| slaweq | and see You online | 14:33 |
| ralonsoh | bye | 14:33 |
| slaweq | o/ | 14:33 |
| obondarev | :) | 14:34 |
| slaweq | #endmeeting | 14:34 |
| opendevmeet | Meeting ended Tue Jun 22 14:34:01 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:34 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/networking/2021/networking.2021-06-22-14.00.html | 14:34 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/networking/2021/networking.2021-06-22-14.00.txt | 14:34 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/networking/2021/networking.2021-06-22-14.00.log.html | 14:34 |
| amotoki | o/ | 14:34 |
| lajoskatona | o/ | 14:34 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/wallaby: Force to close http connection after notify about HA router status https://review.opendev.org/c/openstack/neutron/+/797407 | 14:36 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/victoria: Force to close http connection after notify about HA router status https://review.opendev.org/c/openstack/neutron/+/797408 | 14:36 |
| rubasov | o/ | 14:36 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/ussuri: Force to close http connection after notify about HA router status https://review.opendev.org/c/openstack/neutron/+/797409 | 14:37 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/train: Force to close http connection after notify about HA router status https://review.opendev.org/c/openstack/neutron/+/797410 | 14:38 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/stein: Force to close http connection after notify about HA router status https://review.opendev.org/c/openstack/neutron/+/797411 | 14:40 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/rocky: Force to close http connection after notify about HA router status https://review.opendev.org/c/openstack/neutron/+/797412 | 14:40 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/queens: Force to close http connection after notify about HA router status https://review.opendev.org/c/openstack/neutron/+/797473 | 14:41 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: Revert "Skip scenario tests if HA router will not be active" https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/796858 | 14:44 |
| amotoki | is there any reason we set tox_install_siblings=false in neutron-functional jobs | 14:55 |
| amotoki | https://opendev.org/openstack/neutron/src/branch/master/zuul.d/base.yaml#L37 | 14:55 |
| amotoki | I noticed this when debuggin slaweq's patch which add a functional job to the neutron-lib gate. https://review.opendev.org/c/openstack/neutron-lib/+/797281 | 14:56 |
| amotoki | and also remembered we explicitly set it to true in n-d-r functional job. | 14:57 |
| amotoki | if there is no side effect it would be convenient to set tox_install_siblings to true by default. | 14:57 |
| slaweq | amotoki hmm, I don't know but I can check that | 14:57 |
| slaweq | good catch :) | 14:57 |
| amotoki | looking at the git history, tox_install_siblings is set to false from the initial commit of the zuulv3 migration | 14:58 |
| slaweq | amotoki and I don't remember why it was set like that :/ | 14:59 |
| amotoki | so I guess it is set to false just to minimize the installation. | 14:59 |
| slaweq | #startmeeting neutron_ci | 15:00 |
| opendevmeet | Meeting started Tue Jun 22 15:00:08 2021 UTC and is due to finish in 60 minutes. The chair is slaweq. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| opendevmeet | The meeting name has been set to 'neutron_ci' | 15:00 |
| slaweq | hi (again) | 15:00 |
| ralonsoh | hi | 15:00 |
| obondarev | hi | 15:00 |
| bcafarel | o/ again | 15:00 |
| slaweq | Grafana dashboard: http://grafana.openstack.org/dashboard/db/neutron-failure-rate | 15:00 |
| lajoskatona | o/ | 15:00 |
| slaweq | #topic Actions from previous meetings | 15:01 |
| slaweq | ralonsoh to update our ci job's definitions | 15:01 |
| ralonsoh | #link https://review.opendev.org/c/openstack/neutron/+/797051 | 15:01 |
| ralonsoh | and | 15:01 |
| ralonsoh | #link https://review.opendev.org/c/openstack/project-config/+/797454 | 15:01 |
| slaweq | thx ralonsoh I will review those patches later tonight or tomorrow morning | 15:02 |
| ralonsoh | thanks | 15:02 |
| bcafarel | added to the pile too | 15:03 |
| slaweq | next one | 15:04 |
| slaweq | obondarev to check fullstack timeouts in L3 agent tests | 15:04 |
| obondarev | yes, so I checked the failure You mentioned on the last meeting: 'Lost connection to MySQL server during query' in neutron server logs | 15:04 |
| obondarev | seems like a single random failure | 15:04 |
| slaweq | ahh, ok, so it was oom-killer | 15:04 |
| obondarev | but I checked other failures | 15:04 |
| slaweq | yes, we have such problems from time to time | 15:04 |
| obondarev | and seems there is a pattern | 15:05 |
| obondarev | not a pattern but a test that fails from time to time | 15:05 |
| obondarev | https://bugs.launchpad.net/neutron/+bug/1933234 | 15:05 |
| obondarev | it's not related to Port failed to become active | 15:05 |
| obondarev | it's about MTU not updated for router interface | 15:05 |
| slaweq | yes, I saw that one at least twice this week too | 15:06 |
| obondarev | didn't have time to look closer into it | 15:06 |
| obondarev | might still be related to privsep hanging but not usre | 15:06 |
| obondarev | sure* | 15:06 |
| slaweq | if I will have some time, I will try to have a look into that one | 15:07 |
| obondarev | that's all update for now | 15:07 |
| slaweq | thx obondarev | 15:07 |
| obondarev | np | 15:07 |
| slaweq | #action slaweq to check failing update mtu fullstack test: https://bugs.launchpad.net/neutron/+bug/1933234 | 15:07 |
| slaweq | ok, I think we can move on | 15:08 |
| slaweq | #topic Stadium projects | 15:08 |
| slaweq | any updates here? | 15:08 |
| slaweq | or new ci issues? | 15:08 |
| lajoskatona | on old branches we ar fighting with things, | 15:09 |
| lajoskatona | but thanks to elod and everybody seems to be things are progressing | 15:09 |
| slaweq | that's good | 15:10 |
| slaweq | if You would need any help, please ping me | 15:10 |
| slaweq | I also recently proposed fix for some tap-as-a-service job https://review.opendev.org/c/x/tap-as-a-service-tempest-plugin/+/797318 | 15:10 |
| bcafarel | the exercise turned into a round of stable ci status though that's good to have yes | 15:10 |
| bcafarel | (still have to read elod's fix on that tox dependencies trick) | 15:11 |
| lajoskatona | I check it, to tell the truth I left taas sleep, and hoped that the rename happens soon, peraps I ping infra about it | 15:11 |
| lajoskatona | bcafarel: to tell the truth I can leave without understanding all these dependency hell solutions | 15:12 |
| slaweq | ahh, so we should wait with that my patch until it will be in openstack/ namespace first, correct? | 15:12 |
| bcafarel | lajoskatona: this is also a reasonable way to keep your sanity :) | 15:12 |
| lajoskatona | slaweq: not sure, if the rename happens in the far future.... | 15:12 |
| slaweq | lajoskatona so feel free to check that patch and either approve or stop it for now, I'm ok with both ways | 15:13 |
| lajoskatona | slaweq: You can add an action point on me to check with infra | 15:13 |
| lajoskatona | ok, I check it | 15:13 |
| slaweq | #action lajoskatona to check with infra status of the tap-as-a-service move | 15:13 |
| lajoskatona | bcafarel: LOL | 15:13 |
| slaweq | thx lajoskatona | 15:13 |
| slaweq | anything else regarding stadium or can we move on? | 15:14 |
| lajoskatona | we can move I think | 15:14 |
| slaweq | thx | 15:14 |
| slaweq | #topic Stable branches | 15:14 |
| slaweq | any updates/issues here? | 15:14 |
| slaweq | bcafarel? :) | 15:14 |
| bcafarel | ussuri had issue with neutron-tempest-slow, fix already merged https://review.opendev.org/c/openstack/neutron/+/797273 | 15:15 |
| slaweq | ++ | 15:15 |
| slaweq | thx for that fix | 15:15 |
| bcafarel | else overall as of right now all good :) | 15:15 |
| slaweq | good, so we can move on | 15:16 |
| slaweq | #topic Grafana | 15:16 |
| slaweq | http://grafana.openstack.org/dashboard/db/neutron-failure-rate | 15:16 |
| slaweq | biggest problem was recently with functional tests job | 15:17 |
| slaweq | but fix was merged last night and it seems to be better now | 15:17 |
| slaweq | thx for quick review of the fix btw. :) | 15:18 |
| slaweq | anything else related to grafana for today? | 15:19 |
| slaweq | or can we move on? | 15:19 |
| bcafarel | sounds like a +1 for moving on (and yes it was nice to see that functional fix quickly in) | 15:19 |
| obondarev | good catch slaweq! | 15:20 |
| slaweq | thx :) | 15:20 |
| slaweq | ok, let's move on | 15:20 |
| slaweq | #topic fullstack/functional | 15:20 |
| slaweq | here I had only one issue for today | 15:20 |
| slaweq | but obondarev already reported and mentioned it here actually :) | 15:21 |
| obondarev | :) | 15:21 |
| slaweq | other than that I didn't found any new issues | 15:21 |
| slaweq | btw. I proposed to add neutron-functional job to the neutron-lib queues: https://review.opendev.org/c/openstack/neutron-lib/+/797281 | 15:21 |
| slaweq | but amotoki found some issue with that job's definition | 15:21 |
| slaweq | I will need to have closer look to make it running properly | 15:22 |
| ralonsoh | will you propose the change to remove "tox_install_siblings"? | 15:22 |
| amotoki | I think we can switch it to false. | 15:22 |
| slaweq | ralonsoh I'm not sure what will be the consequences of that | 15:22 |
| amotoki | s/false/true/ | 15:22 |
| ralonsoh | let's change that in neutron, some stadium projects are already doing it | 15:23 |
| amotoki | the first thing we can is to set tox_install_siblings to true in neutron-lib patch to land it. | 15:23 |
| amotoki | then we can try to switch it in the neutron repo. | 15:23 |
| bcafarel | having it set to false may have been for jobs inheriting the base one (but this is a very vague memory) | 15:23 |
| ralonsoh | I'll check any other project inheriting from neutron-functional | 15:24 |
| slaweq | ralonsoh I will check it tomorrow :) | 15:24 |
| slaweq | thx for advices guys | 15:24 |
| ralonsoh | perfect | 15:24 |
| amotoki | fyi: n-d-r explicitly set it to true in its zuul.yaml. | 15:25 |
| slaweq | ok, so this seems like good way to go :) | 15:25 |
| slaweq | I will update my patch tomorrow morning | 15:25 |
| slaweq | ok, let's move one | 15:26 |
| slaweq | #topic Tempest/Scenario | 15:26 |
| slaweq | here I also didn't found any new issues for today | 15:26 |
| slaweq | I have just quick heads-up | 15:27 |
| lajoskatona | Tis playbook is executed if true: https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/tox/tasks/siblings.yam | 15:27 |
| slaweq | I proposed patch https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/795929 to enable tls-proxy where it wasn't enabled | 15:27 |
| slaweq | it looks like it works fine | 15:27 |
| ralonsoh | cool, I think we can merge it now again | 15:28 |
| slaweq | thx lajoskatona | 15:28 |
| slaweq | ralonsoh why "again"? | 15:29 |
| ralonsoh | that was disabled some years ago | 15:29 |
| ralonsoh | that was an error in tempest, if I'm not wrong | 15:29 |
| slaweq | ahh ok | 15:30 |
| slaweq | but now it's enabled by default in tempest jobs IIRC | 15:30 |
| slaweq | so also in most of our jobs which inherits from devstack-tempest | 15:30 |
| slaweq | we just need to enable it in some jobs where it is disabled explicitly | 15:31 |
| slaweq | and looks that it's good now :) | 15:31 |
| slaweq | ok, that's basically all what I had for today | 15:32 |
| slaweq | periodic jobs are fine this week | 15:32 |
| slaweq | anything else You want to discuss today? | 15:32 |
| slaweq | or if not we can finish earlier today | 15:32 |
| ralonsoh | nothing from me | 15:33 |
| bcafarel | nothing from me either | 15:33 |
| slaweq | ok, so thx for attending the meeting today | 15:34 |
| slaweq | o/ | 15:35 |
| slaweq | #endmeeting | 15:35 |
| opendevmeet | Meeting ended Tue Jun 22 15:35:07 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:35 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_ci/2021/neutron_ci.2021-06-22-15.00.html | 15:35 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_ci/2021/neutron_ci.2021-06-22-15.00.txt | 15:35 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_ci/2021/neutron_ci.2021-06-22-15.00.log.html | 15:35 |
| ralonsoh | bye | 15:35 |
| lajoskatona | Bye | 15:35 |
| bcafarel | o/ | 15:36 |
| *** gthiemon1e is now known as gthiemonge | 15:38 | |
| opendevreview | Merged openstack/neutron stable/stein: [EM releases] Move non-voting jobs to the experimental queue https://review.opendev.org/c/openstack/neutron/+/796483 | 17:02 |
| opendevreview | Merged openstack/neutron stable/train: Added common config and SR-IOV agent config to sanity check https://review.opendev.org/c/openstack/neutron/+/796029 | 17:07 |
| opendevreview | Merged openstack/neutron stable/train: [EM releases] Move non-voting jobs to the experimental queue https://review.opendev.org/c/openstack/neutron/+/796472 | 17:07 |
| opendevreview | Merged openstack/ovn-octavia-provider master: Fix functional jobs due to OVS file removal https://review.opendev.org/c/openstack/ovn-octavia-provider/+/796747 | 17:11 |
| *** rpittau is now known as rpittau|afk | 17:11 | |
| opendevreview | Merged openstack/neutron stable/ussuri: Add a single option to override the default hypervisor name https://review.opendev.org/c/openstack/neutron/+/796853 | 17:27 |
| opendevreview | Merged openstack/neutron stable/ussuri: Read keepalived initial state in parallel to interface monitoring https://review.opendev.org/c/openstack/neutron/+/796998 | 17:54 |
| *** iurygregory_ is now known as iurygregory | 18:39 | |
| opendevreview | Brian Haley proposed openstack/networking-ovn stable/train: Add Health Monitor support https://review.opendev.org/c/openstack/networking-ovn/+/796063 | 19:02 |
| opendevreview | Merged openstack/neutron master: Add "nova:live_migration_events" flag to subnode in multinode CI job https://review.opendev.org/c/openstack/neutron/+/795761 | 19:03 |
| opendevreview | Merged openstack/neutron master: SR-IOV agent can handle ports with same MAC addresses https://review.opendev.org/c/openstack/neutron/+/780055 | 19:22 |
| opendevreview | Brian Haley proposed openstack/networking-ovn stable/train: Add Health Monitor support https://review.opendev.org/c/openstack/networking-ovn/+/796063 | 19:34 |
| opendevreview | Merged openstack/neutron master: Fix devstack path in plugin.sh https://review.opendev.org/c/openstack/neutron/+/797128 | 19:57 |
| opendevreview | Ade Lee proposed openstack/neutron master: WIP/DNM - Add FIPS enabled jobs https://review.opendev.org/c/openstack/neutron/+/797537 | 19:58 |
| opendevreview | Ade Lee proposed openstack/neutron master: WIP/DNM - Add FIPS enabled jobs https://review.opendev.org/c/openstack/neutron/+/797537 | 20:01 |
| opendevreview | Brian Haley proposed openstack/networking-ovn stable/train: Add Health Monitor support https://review.opendev.org/c/openstack/networking-ovn/+/796063 | 20:36 |
| *** njohnston_ is now known as njohnston | 21:07 | |
| opendevreview | Merged openstack/neutron stable/ussuri: ovn: Don't use dict.remove() for filtering dhcp ports in db-sync https://review.opendev.org/c/openstack/neutron/+/797000 | 21:07 |
| opendevreview | Brian Haley proposed openstack/networking-ovn stable/train: Add Health Monitor support https://review.opendev.org/c/openstack/networking-ovn/+/796063 | 23:42 |
| opendevreview | Brian Haley proposed openstack/ovn-octavia-provider master: Add Health Monitor support https://review.opendev.org/c/openstack/ovn-octavia-provider/+/713253 | 23:44 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!