| *** eandersson2 is now known as eandersson | 05:56 | |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: amphorav1 removal https://review.opendev.org/c/openstack/octavia/+/876034 | 07:29 |
|---|---|---|
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: Remove specific handling of amphorav1 parameters https://review.opendev.org/c/openstack/octavia/+/876035 | 07:29 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: Merge amphorav2 provider doc into amphora doc https://review.opendev.org/c/openstack/octavia/+/876036 | 07:29 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: Update amphorav2 jobs https://review.opendev.org/c/openstack/octavia/+/876037 | 07:29 |
| skraynev | gthiemonge: finally it passed! https://review.opendev.org/c/openstack/octavia/+/877414 :) johnsom, could please help with review? | 07:33 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: Update amphorav2 jobs https://review.opendev.org/c/openstack/octavia/+/876037 | 08:13 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: Fix octavia-status with amphorav2 https://review.opendev.org/c/openstack/octavia/+/878816 | 08:13 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: [sqlalchemy2] Turn on RemovedIn20Warnings https://review.opendev.org/c/openstack/octavia/+/861288 | 08:45 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: [sqlalchemy2] move to Session.get() https://review.opendev.org/c/openstack/octavia/+/861289 | 08:45 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: [sqlalchemy2] use attribute to indicate relationship https://review.opendev.org/c/openstack/octavia/+/861310 | 08:45 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: [sqlalchemy2] positional arguments for case https://review.opendev.org/c/openstack/octavia/+/861311 | 08:45 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: [sqlalchemy2] Use subquery to select from a query https://review.opendev.org/c/openstack/octavia/+/861312 | 08:45 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: [sqlalchemy2] Using non-integer/slice indices on Row is deprecated https://review.opendev.org/c/openstack/octavia/+/861313 | 08:45 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: [sqlaclhemy2] Don't pass strings to session.execute https://review.opendev.org/c/openstack/octavia/+/861314 | 08:45 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: [sqlalchemy2] Removal of cascade backrefs https://review.opendev.org/c/openstack/octavia/+/861315 | 08:45 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: [sqlalchemy2] Added missing relationships in models https://review.opendev.org/c/openstack/octavia/+/875364 | 08:45 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: [sqlalchemy2] subtransactions & autocommit removal https://review.opendev.org/c/openstack/octavia/+/861316 | 08:45 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: [sqlalchemy2] Disabling 'create_engine.convert_unicode' filter https://review.opendev.org/c/openstack/octavia/+/875365 | 08:45 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: Fix sqlalchemy declarative_base import https://review.opendev.org/c/openstack/octavia/+/875892 | 08:45 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: WIP Adding a functional job with SQLAlchemy master https://review.opendev.org/c/openstack/octavia/+/875532 | 08:45 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Add octavia-grenade-slurp CI job https://review.opendev.org/c/openstack/octavia/+/860221 | 10:09 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Add octavia-grenade-slurp CI job https://review.opendev.org/c/openstack/octavia/+/860221 | 10:23 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Add octavia-grenade-slurp CI job https://review.opendev.org/c/openstack/octavia/+/860221 | 10:27 |
| opendevreview | Julian DA CUNHA proposed openstack/octavia master: Add new spec Let's Encrypt support https://review.opendev.org/c/openstack/octavia/+/877281 | 11:29 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Add octavia-grenade-slurp CI job https://review.opendev.org/c/openstack/octavia/+/860221 | 12:00 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: [sqlalchemy2] Disabling 'create_engine.convert_unicode' filter https://review.opendev.org/c/openstack/octavia/+/875365 | 12:50 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: Fix sqlalchemy declarative_base import https://review.opendev.org/c/openstack/octavia/+/875892 | 12:50 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: WIP Adding a functional job with SQLAlchemy master https://review.opendev.org/c/openstack/octavia/+/875532 | 12:50 |
| kevko | johnsom: well, patch is from me, i thought that owner shouldn't give vote himself ..and i am not core .. | 14:15 |
| johnsom | Ok, I didn't know you were the author. We have had multiple patches where people are asking for reviews and are not helping to review it. I'm trying to encourage people to do reviews even if they are not cores. All reviews matter. | 14:16 |
| opendevreview | Tom Weininger proposed openstack/octavia master: DNM: profile using py-spy https://review.opendev.org/c/openstack/octavia/+/878917 | 15:06 |
| opendevreview | Tom Weininger proposed openstack/octavia master: DNM: profile using py-spy https://review.opendev.org/c/openstack/octavia/+/878917 | 15:12 |
| opendevreview | Tom Weininger proposed openstack/octavia master: DNM: profile using py-spy https://review.opendev.org/c/openstack/octavia/+/878917 | 15:14 |
| opendevreview | Tom Weininger proposed openstack/octavia master: DNM: profile using py-spy https://review.opendev.org/c/openstack/octavia/+/878917 | 15:20 |
| opendevreview | Tom Weininger proposed openstack/octavia-tempest-plugin master: DNM profile o-* using py-spy https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/878919 | 15:24 |
| opendevreview | Tom Weininger proposed openstack/octavia master: DNM: profile using py-spy https://review.opendev.org/c/openstack/octavia/+/878917 | 15:30 |
| opendevreview | Tom Weininger proposed openstack/octavia master: DNM: profile using py-spy https://review.opendev.org/c/openstack/octavia/+/878917 | 15:56 |
| gthiemonge | #startmeeting Octavia | 16:00 |
| opendevmeet | Meeting started Wed Mar 29 16:00:26 2023 UTC and is due to finish in 60 minutes. The chair is gthiemonge. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:00 |
| opendevmeet | The meeting name has been set to 'octavia' | 16:00 |
| gthiemonge | Hi there | 16:00 |
| johnsom | o/ | 16:00 |
| gthiemonge | long time no see | 16:00 |
| tweining | o/ | 16:00 |
| johnsom | lol | 16:00 |
| gthiemonge | #topic Announcements | 16:01 |
| gthiemonge | * PTG week | 16:02 |
| gthiemonge | yesterday was the Octavia PTG session | 16:02 |
| gthiemonge | really good discussions | 16:02 |
| gthiemonge | in case you missed it, the notes are in the etherpad | 16:02 |
| gthiemonge | #link https://etherpad.opendev.org/p/bobcat-ptg-octavia | 16:02 |
| tweining | bobcat will come with quite a few improvements as it seems | 16:03 |
| tweining | (if things go as planned) | 16:04 |
| gthiemonge | I need to check the notes from other projects but I heard that there are some plans to deprecate more python-*client projects (barbican) | 16:04 |
| johnsom | I don't think they are going to be able to deprecate python-barbicanclient in Bobcat. I think there is work to do on sdk for them. | 16:06 |
| gthiemonge | right | 16:06 |
| gthiemonge | we need to think about that in the next releases | 16:06 |
| tweining | I guess there is no date for the migration to Launchpad yet, but it will be announced in time here. | 16:07 |
| johnsom | Yeah, we should probably check all of the clients. nova, glance, barbican, cinder | 16:07 |
| gthiemonge | tweining: I'll send an email and prepare the patches to update the links | 16:07 |
| gthiemonge | any other announcements? | 16:10 |
| gthiemonge | #topic CI Status | 16:11 |
| gthiemonge | CI is in a better shape this week :D | 16:12 |
| gthiemonge | grenade and pep8 jobs are fixed | 16:12 |
| gthiemonge | things are merging properly | 16:12 |
| gthiemonge | still not sure on the status of the ubuntu jammy nested virt hosts from the vexxhost nodepool | 16:12 |
| gthiemonge | no failure this week | 16:13 |
| johnsom | Yeah, I'm not sure either. Neutron was also having similar issues. | 16:13 |
| johnsom | It sounds like it was due to vexxhost still running bonic hypervisors with bugs | 16:14 |
| johnsom | #link https://bugs.launchpad.net/neutron/+bug/1999249/comments/4 | 16:14 |
| gthiemonge | it seems that they were progressively updating those hypervisors, maybe it will be fine | 16:15 |
| tweining | only the grenade job failed, right? | 16:15 |
| gthiemonge | nop | 16:15 |
| gthiemonge | the scenario jobs | 16:16 |
| gthiemonge | #topic Brief progress reports / bugs needing review | 16:19 |
| gthiemonge | Folks pinged us on IRC to get review for: | 16:19 |
| gthiemonge | #link https://review.opendev.org/c/openstack/octavia/+/877414 | 16:19 |
| gthiemonge | tweining: could you take a look ^ ? | 16:20 |
| tweining | sure | 16:20 |
| gthiemonge | on my side, I rebased the amphorav1-removal patch chain | 16:21 |
| gthiemonge | #link https://review.opendev.org/q/topic:amphorav1-removal | 16:21 |
| gthiemonge | I think it is ready | 16:21 |
| gthiemonge | I rebased the changes for sqlalchemy2 on top of it, but it fails because we need to merge a patch in taskflow first (and to get a new release) | 16:22 |
| tweining | all the persons present know it already, but I mention it for the record: I updated patch for testing SLURP upgrades, and it should be ready for review too | 16:23 |
| tweining | https://review.opendev.org/c/openstack/octavia/+/860221 | 16:23 |
| gthiemonge | tweining: can you proactively backport it to 2023.1? just to check if it runs properly there | 16:24 |
| tweining | sure | 16:24 |
| gthiemonge | thanks | 16:24 |
| opendevreview | Tom Weininger proposed openstack/octavia stable/2023.1: Add octavia-grenade-slurp CI job https://review.opendev.org/c/openstack/octavia/+/878928 | 16:25 |
| gthiemonge | #topic Open Discussion | 16:25 |
| tweining | maybe one more comment to the slurp patch | 16:26 |
| tweining | before there was octavia-grenade-ffu in "periodic". I moved it to check, because I saw some other project did it that way too | 16:27 |
| gthiemonge | I think we can remove the ffu job now we have the slurp job | 16:27 |
| tweining | yeah, the new job replaces the ffu job | 16:28 |
| gthiemonge | ack | 16:28 |
| gthiemonge | I think it's fine to have the slurp in the check | 16:29 |
| johnsom | +1 | 16:30 |
| gthiemonge | any other topics for this week? | 16:32 |
| tweining | I hope next week I can present some profiling results. Apart from that nothing from my side. | 16:33 |
| gthiemonge | +1 | 16:33 |
| gthiemonge | ok! thank you guys! | 16:33 |
| gthiemonge | #endmeeting | 16:33 |
| opendevmeet | Meeting ended Wed Mar 29 16:33:28 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:33 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/octavia/2023/octavia.2023-03-29-16.00.html | 16:33 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/octavia/2023/octavia.2023-03-29-16.00.txt | 16:33 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/octavia/2023/octavia.2023-03-29-16.00.log.html | 16:33 |
| tweining | the new skip-level jobs runs with the 2023.1 backport. that is a good sign. | 16:37 |
| opendevreview | Tom Weininger proposed openstack/octavia master: DNM: profile using py-spy https://review.opendev.org/c/openstack/octavia/+/878917 | 16:47 |
| *** JayF is now known as Guest9308 | 19:44 | |
| *** JasonF is now known as jayf | 19:44 | |
| *** jayf is now known as JayF | 19:44 | |
| opendevreview | Merged openstack/octavia master: Fix ORM caching for with_for_update calls https://review.opendev.org/c/openstack/octavia/+/877414 | 21:46 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!