| opendevreview | Merged openstack/neutron stable/yoga: Check the Chassis_Private nb_cfg_timestamp with current value https://review.opendev.org/c/openstack/neutron/+/854541 | 01:24 |
|---|---|---|
| *** dasm is now known as dasm|off | 02:00 | |
| opendevreview | ZhouHeng proposed openstack/neutron-lib master: [ovn]Floating IP adds distributed attributes https://review.opendev.org/c/openstack/neutron-lib/+/855053 | 02:40 |
| opendevreview | Ke Niu proposed openstack/neutron master: Cleanup py27 support https://review.opendev.org/c/openstack/neutron/+/853115 | 06:16 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Pin OVS version for OVN master https://review.opendev.org/c/openstack/ovn-octavia-provider/+/854008 | 07:24 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: [OVN] Pin OVS version for OVN master https://review.opendev.org/c/openstack/ovn-octavia-provider/+/854008 | 07:26 |
| *** elodilles_pto is now known as elodilles | 08:06 | |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp/rpc: retrieve network details with segments https://review.opendev.org/c/openstack/neutron/+/840416 | 08:10 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: utils: add option to generate dhcp device id per segmentation https://review.opendev.org/c/openstack/neutron/+/840417 | 08:10 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ml2/mech: bind port on segment that subnet belongs https://review.opendev.org/c/openstack/neutron/+/840418 | 08:10 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: make device manager to clean only not used devices https://review.opendev.org/c/openstack/neutron/+/840419 | 08:10 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: add/use cleanup stale devices API https://review.opendev.org/c/openstack/neutron/+/840420 | 08:10 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: support multiple segmentations per network https://review.opendev.org/c/openstack/neutron/+/840421 | 08:10 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: db: add segment_index to the unique constraint https://review.opendev.org/c/openstack/neutron/+/841459 | 08:10 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: segment: enable multisegments support for host https://review.opendev.org/c/openstack/neutron/+/841838 | 08:10 |
| opendevreview | ZhouHeng proposed openstack/neutron-tempest-plugin master: list floating ip api contains port forwarding id https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/840584 | 11:09 |
| opendevreview | ZhouHeng proposed openstack/neutron-tempest-plugin master: skip '_verify_created_pfs' https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/840584 | 11:11 |
| opendevreview | ZhouHeng proposed openstack/neutron-tempest-plugin master: skip '_verify_created_pfs' https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/840584 | 11:13 |
| opendevreview | ZhouHeng proposed openstack/neutron-lib master: [ovn]Floating IP adds distributed attributes https://review.opendev.org/c/openstack/neutron-lib/+/855053 | 11:27 |
| sahid | slaweq, obondarev hi, any chance to make https://review.opendev.org/c/openstack/neutron/+/839479 and https://review.opendev.org/c/openstack/neutron/+/840415/28 progressing when you have a moment? | 12:32 |
| obondarev | sahid: yes, on my list | 12:35 |
| lajoskatona | sahid: on my list also | 12:44 |
| sahid | thanks a lot guys :-) | 13:17 |
| sahid | feel free to let me know if you need help on reviewing specific work | 13:17 |
| slaweq | sahid: I will review them today or tomorrow morning | 13:19 |
| lajoskatona | sahid: if I get the last from the series (https://review.opendev.org/c/openstack/neutron/+/841838 I think) and I add a bridge_mappings = public:br-ex,physnet1:br0,physnet2:br0 I shall use the feature, am I right? | 13:24 |
| opendevreview | Merged openstack/ovn-octavia-provider master: Fix healthMonitor events affecting to unrelated LB https://review.opendev.org/c/openstack/ovn-octavia-provider/+/853681 | 13:26 |
| opendevreview | Merged openstack/ovn-octavia-provider master: [OVN] Pin OVS version for OVN master https://review.opendev.org/c/openstack/ovn-octavia-provider/+/854008 | 13:26 |
| sahid | lajoskatona: not exactly, with this feature, you will still have a phynet per bridge. You will be able to have more than one subnet on the network where is attached the physnet and have for each subnet a segment attached to it | 13:32 |
| sahid | today, only one subnet/segment is accepted per network that is attached to a physnet | 13:33 |
| sahid | basically if you ran out of ip on that network, you have to create a new network, right? | 13:33 |
| sahid | with this feature, if you ran out of ip, you can add a new subnet on that same network | 13:34 |
| lajoskatona | sahid: ack, ok | 13:34 |
| sahid | btw I have noticed your comment and I will update the doc, +1 | 13:37 |
| *** dasm|off is now known as dasm | 13:46 | |
| lajoskatona | sahid: thanks | 13:56 |
| lajoskatona | #startmeeting networking | 14:00 |
| opendevmeet | Meeting started Tue Aug 30 14:00:59 2022 UTC and is due to finish in 60 minutes. The chair is lajoskatona. 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:01 |
| lajoskatona | o/ | 14:01 |
| rubasov | o/ | 14:01 |
| obondarev | hi | 14:01 |
| ralonsoh | hi | 14:01 |
| frickler | \o | 14:01 |
| isabek | hi | 14:01 |
| slaweq | o/ | 14:01 |
| lajoskatona | Ok ,let's start | 14:02 |
| lajoskatona | #topic Announcements | 14:02 |
| lajoskatona | The usual Zed schedule: https://releases.openstack.org/zed/schedule.html | 14:03 |
| lajoskatona | and the release countdown mail: | 14:03 |
| lajoskatona | [release] Release countdown for week R-5, Aug 29 - Sep 02 (#link https://lists.openstack.org/pipermail/openstack-discuss/2022-August/030138.html ) | 14:03 |
| lajoskatona | Some dates from it: | 14:03 |
| lajoskatona | Zed-3 milestone (feature freeze): September 1st, 2022 (R-5 week) | 14:03 |
| lajoskatona | that is Thursday this week | 14:03 |
| lajoskatona | RC1 deadline: September 15th, 2022 (R-3 week) | 14:04 |
| lajoskatona | Final RC deadline: September 29th, 2022 (R-1 week) | 14:04 |
| lajoskatona | Final Zed release: October 5th, 2022 | 14:04 |
| lajoskatona | Next PTG: October 17-21, 2022 (virtual PTG!!!) | 14:04 |
| lajoskatona | The etherpad for it: https://etherpad.opendev.org/p/neutron-antelope-ptg | 14:04 |
| lajoskatona | I add a mental not to myself to ask around for the schedule | 14:05 |
| lajoskatona | Cycle Highlights, please check it: https://review.opendev.org/c/openstack/releases/+/853813 | 14:05 |
| lajoskatona | My memory faded so thanks for the good comments on it already | 14:05 |
| lajoskatona | Perhaps I add here a small news: there will be again edge session on the PTG: | 14:07 |
| lajoskatona | https://lists.openstack.org/pipermail/openstack-discuss/2022-August/030128.html | 14:07 |
| lajoskatona | and there is an etherpad for it: https://etherpad.opendev.org/p/ecg-ptg-october-2022 | 14:07 |
| lajoskatona | if you are interested check it | 14:07 |
| lajoskatona | My personal point is that I would like to see exactly what and why we need at the edge sites | 14:08 |
| lajoskatona | In Berlin I was on a session for edge, and I heard some bullshit (self driving cars, and things like that which are far from mass usage at the moment) and the only thing which sounded as real was gaming which needed some edge features | 14:09 |
| * slaweq don't wants to have self driving car with Openstack running on it :P | 14:10 | |
| ralonsoh | hehehe | 14:10 |
| lajoskatona | :-) | 14:10 |
| mlavalle | I wouldn't mind | 14:10 |
| lajoskatona | there was even a slide as I remember with a fighter jet with edge sites inside it :-) | 14:11 |
| lajoskatona | ok, that's it for the PTG from me | 14:11 |
| haleyb | can Openstack drive a manual transmission? :) | 14:11 |
| slaweq | haleyb: I don't think it will be fast enough :P | 14:12 |
| slaweq | and rabbitmq can be stucked somewhere in the middle :D | 14:12 |
| lajoskatona | :D | 14:13 |
| obondarev | anything - but let it not be brakes please! | 14:13 |
| slaweq | LOL | 14:13 |
| slaweq | obondarev++ | 14:13 |
| lajoskatona | Another thing from me for announcements: | 14:14 |
| lajoskatona | Please recheck with reason | 14:14 |
| lajoskatona | Please check the numbers from slaweq: https://etherpad.opendev.org/p/neutron-ci-meetings#L42 | 14:14 |
| lajoskatona | If there is no more comments or questions for announcements we can move on | 14:15 |
| frickler | PTL nominations? | 14:16 |
| lajoskatona | frickler: good comment, thanks | 14:16 |
| ralonsoh | I'm on it, I'll send a mail/patch today, in the next hour | 14:16 |
| lajoskatona | ralonsoh: thanks | 14:16 |
| ralonsoh | (and before that, lajoskatona thanks a lot!) | 14:16 |
| mlavalle | ralonsoh: ++ | 14:16 |
| lajoskatona | Ok, next topic | 14:18 |
| lajoskatona | #topic Bugs | 14:18 |
| lajoskatona | Report from slaweq: https://lists.openstack.org/pipermail/openstack-discuss/2022-August/030168.html | 14:18 |
| lajoskatona | bug numbers are going upper as summer period is over :-) | 14:18 |
| lajoskatona | I saw some which needs more attention | 14:18 |
| lajoskatona | https://bugs.launchpad.net/neutron/+bug/1987530 - Duplicate external_ip in NAT table lead to loss of N/S connectivity - needs attention | 14:18 |
| ralonsoh | I'll take a look at this one, this should not happen of course | 14:19 |
| lajoskatona | ralonsoh: thanks | 14:20 |
| lajoskatona | Another one without owner: | 14:20 |
| lajoskatona | https://bugs.launchpad.net/neutron/+bug/1987666 - Race condition when adding two subnet with same cidr to router - unassigned currently | 14:20 |
| slaweq | I think that froyo is working on it currently | 14:20 |
| slaweq | but maybe he didn't assign it to himself | 14:20 |
| froyo | slaweq, lajoskatona yeah! | 14:21 |
| lajoskatona | ahh, ok, thanks | 14:21 |
| slaweq | froyo++ thx | 14:21 |
| froyo | done! | 14:21 |
| mlavalle | :-) | 14:22 |
| lajoskatona | An old cloud-init bug: https://bugs.launchpad.net/cloud-init/+bug/1899487 - cloud-init hard codes MTU configuration at initial deploy time - bug which was reported for cloud-init first but now after long discussion it's on Neutron and we need to check it | 14:22 |
| slaweq | yeah, that one I wanted to raise today too :) | 14:22 |
| lajoskatona | :-) | 14:23 |
| lajoskatona | If you have some time please check it, I also try to check the history of this issue | 14:26 |
| lajoskatona | Do you have any bug which you would like to discuss? | 14:27 |
| slaweq | nothing more to discuss | 14:27 |
| frickler | there was the CI issue with designate yesterday | 14:27 |
| lajoskatona | ok, thanks | 14:27 |
| slaweq | but there is one https://bugs.launchpad.net/neutron/+bug/1988026 | 14:27 |
| slaweq | which looks like low-hanging-fruit for me | 14:28 |
| frickler | not sure if there is really a bug hidden there for neutron in terms of deleting SGs | 14:28 |
| slaweq | so maybe there would be someone who would like to take a look into it | 14:28 |
| slaweq | that's all from me | 14:28 |
| frickler | that bug came out of debugging things | 14:28 |
| frickler | but it doesn't seem to be new | 14:28 |
| frickler | the patch in tempest for what triggered the issue is https://review.opendev.org/c/openstack/tempest/+/854973 | 14:29 |
| lajoskatona | friskler: thanks for checking it | 14:29 |
| frickler | but that has been in place for a long time and things in CI only started to fail around friday | 14:29 |
| lajoskatona | frickler, sorry | 14:29 |
| frickler | so maybe there is yet another issue. I didn't get round to reproducing locally yet | 14:30 |
| lajoskatona | yes that is strange that it started to explode in designate CI from end of the last week | 14:30 |
| slaweq | frickler: do you have opened bug somewhere? | 14:31 |
| frickler | designate has a lot of tests that do not use network resources, which I think is why this was visible there most | 14:31 |
| lajoskatona | slaweq: this one I think: https://bugs.launchpad.net/neutron/+bug/1988026 | 14:31 |
| frickler | slaweq: not yet, because I'm not sure if there actually is a bug. maybe neutron now is only more correct than earlier | 14:32 |
| slaweq | lajoskatona: regarding that cloud-init bug, I was going to say that I will try to look into it this week but I wrote it in different window (internal irc) :D | 14:32 |
| slaweq | lajoskatona: ahh, ok | 14:32 |
| lajoskatona | slaweq: ack, the message arrived | 14:32 |
| slaweq | so that is the one which I think may be low-handing-fruit in neutron | 14:32 |
| lajoskatona | frickler: ok, thanks | 14:33 |
| frickler | 1988026 is related but not what caused CI to fail now | 14:33 |
| frickler | the underlying issue is fixed with the tempest patch, so also no CI impact any longer. at least none I'm aware of | 14:34 |
| frickler | and I think that's it for now. ;) | 14:34 |
| lajoskatona | ok, so let's keep our eyes open if similar issues appear | 14:35 |
| lajoskatona | One more sentence for the bug deputy week: This week haleyb is the deputy and next week amotoki will be. | 14:36 |
| amotoki | lajoskatona: ack | 14:36 |
| haleyb | lajoskatona: ack | 14:36 |
| lajoskatona | amotoki, haleyb: thanks | 14:37 |
| lajoskatona | #topic On Demand Agenda | 14:37 |
| lajoskatona | (slaweq) PTG - TC+Leaders Interaction Session - poll opened https://framadate.org/zsOqRxfVcmtjaPBC | 14:37 |
| slaweq | ahh, right | 14:37 |
| slaweq | I just wanted to mention that gmann opened poll to choose the best time slot for the TC+Leaders session during the PTG | 14:38 |
| slaweq | so if anyone is interested in participating in it, please vote there | 14:38 |
| slaweq | lajoskatona: ralonsoh looking at You especially ;) | 14:38 |
| ralonsoh | I'm updating it now | 14:38 |
| lajoskatona | slaweq: thanks, I voted on the time slots :-) | 14:39 |
| slaweq | thx | 14:39 |
| slaweq | that's all from me | 14:39 |
| lajoskatona | I have another one, more a question: | 14:40 |
| lajoskatona | (lajoskatona): OVS (&OVN) version in CI | 14:40 |
| lajoskatona | I just greped quickly and saw that even in our CI we have different hashes for on and ovs branch | 14:40 |
| lajoskatona | perhaps we should have a common version for all master jobs? | 14:41 |
| slaweq | yeah, we should use "master" and "main" branch basically | 14:41 |
| ralonsoh | right and we should update it anytime OVN master (or any other branch) update the dependency | 14:41 |
| ralonsoh | not really | 14:41 |
| ykarel | but those do not work correctly sometime | 14:41 |
| ralonsoh | master requires a specific OIVS version | 14:42 |
| ykarel | yeap ^ | 14:42 |
| ralonsoh | e.g.: https://review.opendev.org/c/openstack/ovn-octavia-provider/+/854008 | 14:42 |
| ralonsoh | tomorrow I'll check all OVN master references | 14:42 |
| lajoskatona | yeah I was looking for this one | 14:43 |
| ralonsoh | and update the needed OVS version | 14:43 |
| slaweq | ++ | 14:43 |
| ykarel | may be we can looking into adding an option so ovs version is auto detected based on ovn source | 14:43 |
| ykarel | atleast when ovn branch is main | 14:43 |
| lajoskatona | perhaps only some guide in docs is enough if it is hard to automate | 14:43 |
| ralonsoh | we can introduce an ansible role to read that yeah | 14:43 |
| ralonsoh | first I'll update the OVS versions statically | 14:44 |
| ralonsoh | then I'll open a LP for this improvement | 14:44 |
| lajoskatona | ralonsoh: thanks | 14:44 |
| lajoskatona | good idea | 14:44 |
| ykarel | +1 | 14:44 |
| lajoskatona | That's it from me, do you have anything more to discuss? | 14:46 |
| slaweq | nothing from me | 14:46 |
| frickler | actually yes | 14:46 |
| frickler | I added a topic to the PTG etherpad: neutron-legacy in devstack | 14:46 |
| frickler | we planned for new neutron code to be dropped in A cycle after it was deprecated earlier | 14:47 |
| frickler | maybe someone here is interested in actually doing this | 14:47 |
| frickler | would save you a lot of duplication when adding new features | 14:47 |
| frickler | and it would be good to do it early in the cycle, so preparing a patch could actually start soon | 14:48 |
| lajoskatona | frickler: good idea to discuss it, and write down/discuss the basics before removing anything | 14:48 |
| lajoskatona | and now I see that this could be a shared session with QA team | 14:48 |
| lajoskatona | but we use now neutron-legacy or am I wrong? | 14:49 |
| frickler | yes, though not sure if anyone other than me would join, but here's hoping | 14:50 |
| frickler | well currently the usage is mixed I think | 14:50 |
| frickler | would need to search for the deprecation message in logs | 14:50 |
| frickler | but good idea to check that before removing things, too | 14:51 |
| lajoskatona | +1 | 14:51 |
| lajoskatona | frickler: thanks for bringing it here | 14:52 |
| lajoskatona | If nothing more we can close the meeting | 14:52 |
| frickler | not from me, thx lajoskatona | 14:53 |
| lajoskatona | We have the CI meeting after this one, and it will be video meeting today: https://meetpad.opendev.org/neutron-ci-meetings | 14:53 |
| lajoskatona | see you there | 14:53 |
| slaweq | ++ | 14:53 |
| lajoskatona | #endmeeting | 14:53 |
| opendevmeet | Meeting ended Tue Aug 30 14:53:35 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:53 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/networking/2022/networking.2022-08-30-14.00.html | 14:53 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/networking/2022/networking.2022-08-30-14.00.txt | 14:53 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/networking/2022/networking.2022-08-30-14.00.log.html | 14:53 |
| lajoskatona | o/ | 14:53 |
| slaweq | o/ | 14:53 |
| mlavalle | o/ | 14:53 |
| ralonsoh | bye | 14:53 |
| obondarev | o/ | 14:53 |
| haleyb | o/ | 14:54 |
| slaweq | #startmeeting neutron_ci | 15:00 |
| opendevmeet | Meeting started Tue Aug 30 15:00:47 2022 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 | meeting is on video | 15:00 |
| slaweq | #link https://meetpad.opendev.org/neutron-ci-meetings | 15:00 |
| mlavalle | o/ | 15:00 |
| ykarel | o/ | 15:02 |
| slaweq | mlavalle: lajoskatona ykarel are You joining? | 15:02 |
| slaweq | and others of course too | 15:02 |
| lajoskatona | o/ | 15:02 |
| lajoskatona | sorry | 15:02 |
| slaweq | Grafana dashboard: https://grafana.opendev.org/d/f913631585/neutron-failure-rate?orgId=1 | 15:03 |
| slaweq | #topic Actions from previous meetings | 15:03 |
| slaweq | slaweq to fix functiona/fullstack failures on centos 9 stream: https://bugs.launchpad.net/neutron/+bug/1976323 | 15:03 |
| slaweq | #action slaweq to fix functiona/fullstack failures on centos 9 stream: https://bugs.launchpad.net/neutron/+bug/1976323 | 15:07 |
| ralonsoh | please continue, I don't know what is happening | 15:08 |
| slaweq | slaweq to check failing neutron-ovn-tempest-ovs-master-centos-9-stream periodic job | 15:08 |
| slaweq | ralonsoh to check route not found failure in functional job | 15:09 |
| slaweq | ralonsoh: ^^ can You update about this one? | 15:09 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/854986 | 15:09 |
| ralonsoh | ^^ | 15:09 |
| ralonsoh | yes, done | 15:09 |
| slaweq | thx ralonsoh | 15:10 |
| slaweq | #topic Stadium projects | 15:11 |
| slaweq | all green in stadium | 15:12 |
| slaweq | #topic Grafana | 15:12 |
| slaweq | #topic Rechecks | 15:13 |
| slaweq | #topic fullstack/functional | 15:15 |
| ralonsoh | maybe timeout moving the logs | 15:16 |
| slaweq | #action slaweq to check POST_FAILURE reasons | 15:17 |
| slaweq | https://zuul.openstack.org/build/cb4b93bb06034b178b63f54f705320d3 | 15:18 |
| slaweq | https://zuul.openstack.org/build/8f9683dfa1a74a43a5ae5b302eb53092 | 15:18 |
| slaweq | https://zuul.openstack.org/build/17044d0bce2f4da68e7e3016ddb7d318 | 15:18 |
| slaweq | network interface not found errors in all cases ^^ | 15:18 |
| slaweq | #action ykarel to check interface not found issues in the periodic functional jobs | 15:21 |
| slaweq | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_7e5/840415/28/check/neutron-fullstack-with-uwsgi/7e5372c/testr_results.html | 15:21 |
| ralonsoh | no no, this is most probably a timeout during the DB table deletion | 15:22 |
| ralonsoh | we have seen that before | 15:22 |
| slaweq | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_7e5/840415/28/check/neutron-fullstack-with-uwsgi/7e5372c/controller/logs/dsvm-fullstack-logs/TestLogging.test_logging.txt | 15:22 |
| ralonsoh | could be yes | 15:22 |
| ralonsoh | we have seen that before | 15:22 |
| ralonsoh | (and this is during the cleanup) | 15:23 |
| slaweq | #topic Periodic | 15:23 |
| slaweq | openstack-tox-py39-with-oslo-master failed today and 27.08 | 15:24 |
| slaweq | https://zuul.openstack.org/build/fb999a2bc6004e459d81d822e2ccaf37 | 15:24 |
| slaweq | https://zuul.openstack.org/build/8468b288c7b84861a827a9f9d2c402ef | 15:24 |
| slaweq | mlavalle: https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_846/periodic/opendev.org/openstack/neutron/master/openstack-tox-py39-with-oslo-master/8468b28/testr_results.html | 15:25 |
| lajoskatona | neutron.tests.unit.objects.test_quota.ReservationDbObjectTestCase.test_delete_expired | 15:25 |
| ralonsoh | ohhh | 15:26 |
| slaweq | #action mlavalle to check failing quota test in openstack-tox-py39-with-oslo-master periodic job | 15:26 |
| ralonsoh | could be the expiry date == reservations? | 15:26 |
| ralonsoh | no from my side | 15:26 |
| ralonsoh | sure | 15:27 |
| ralonsoh | thansk! | 15:27 |
| slaweq | #endmeeting | 15:27 |
| opendevmeet | Meeting ended Tue Aug 30 15:27:39 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:27 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_ci/2022/neutron_ci.2022-08-30-15.00.html | 15:27 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_ci/2022/neutron_ci.2022-08-30-15.00.txt | 15:27 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_ci/2022/neutron_ci.2022-08-30-15.00.log.html | 15:27 |
| slaweq | o/ | 15:27 |
| lajoskatona | Bye | 15:27 |
| mlavalle | o/ | 15:27 |
| mlavalle | ralonsoh: any additional comments/insights about that test failure? | 15:28 |
| ralonsoh | yeah, could be something related to the system date | 15:28 |
| ralonsoh | (maybe) | 15:28 |
| ralonsoh | and in Reservation.delete_expired | 15:28 |
| mlavalle | ok, I'll keep that in mind | 15:28 |
| lajoskatona | frickler: regarding devstack/lib/neutron, opensearch cant find this log msg, or I can't make it to find it :-) | 15:28 |
| ralonsoh | the expiring_time is bigger than the expiration time | 15:28 |
| ralonsoh | mlavalle, because in L68 in the test | 15:29 |
| ralonsoh | we actually check how many registers we have deleted | 15:29 |
| ralonsoh | self.assertEqual(2, self._test_class.delete_expired( | 15:29 |
| ralonsoh | self.context, dt, None)) | 15:29 |
| ralonsoh | mlavalle, ok ok ok | 15:30 |
| ralonsoh | one sec | 15:30 |
| ralonsoh | this is oslo-master, right? | 15:30 |
| mlavalle | yeap, it seems so | 15:30 |
| ralonsoh | we are deleting the registers *outside* a DB transaction | 15:31 |
| ralonsoh | L68-69 | 15:31 |
| ralonsoh | delete_expired must be called insied a db txn | 15:31 |
| ralonsoh | so you should commit that query | 15:31 |
| ralonsoh | in other words, wrap this call in L68 inside a writer context | 15:31 |
| mlavalle | what code are you looking at? | 15:32 |
| ralonsoh | ReservationDbObjectTestCase.test_delete_expired | 15:32 |
| mlavalle | ok | 15:32 |
| ralonsoh | and most probably you should do the same with "_create_test_reservation" | 15:33 |
| mlavalle | ralonsoh: ok, thanks for the comments. | 15:34 |
| frickler | lajoskatona: ack, that's a good thing at least | 15:41 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [WIP] Add an active wait during the port provisioning event https://review.opendev.org/c/openstack/neutron/+/855257 | 16:19 |
| opendevreview | Merged openstack/neutron master: Format the protocol number to be passed to pyroute2 https://review.opendev.org/c/openstack/neutron/+/854986 | 17:03 |
| *** dasm is now known as dasm|off | 19:21 | |
| opendevreview | Merged openstack/neutron stable/yoga: [OVN] Remove ACLs with remote SG during deletion of SG https://review.opendev.org/c/openstack/neutron/+/854558 | 21:16 |
| opendevreview | Merged openstack/neutron stable/ussuri: Bump revision number of objects when description is changed https://review.opendev.org/c/openstack/neutron/+/854909 | 21:30 |
| opendevreview | Merged openstack/neutron stable/ussuri: Allow operator to disable usage of random-fully https://review.opendev.org/c/openstack/neutron/+/854995 | 21:30 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!