| *** ysandeep|out is now known as ysandeep | 04:31 | |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: WIP Testing bugs from story 2009887 and HM status https://review.opendev.org/c/openstack/octavia/+/837195 | 06:35 |
|---|---|---|
| gthiemonge | ^ after so many rechecks, I think I have a fix for these CI issues | 06:38 |
| opendevreview | lixuehai proposed openstack/octavia master: Remove unnecessary unicode prefixes https://review.opendev.org/c/openstack/octavia/+/839480 | 07:33 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia stable/yoga: Deny the creation of L7Policies for HTTPS/TCP/UDP listeners https://review.opendev.org/c/openstack/octavia/+/839486 | 08:10 |
| *** ysandeep is now known as ysandeep|lunch | 08:12 | |
| opendevreview | Gregory Thiemonge proposed openstack/octavia stable/xena: Deny the creation of L7Policies for HTTPS/TCP/UDP listeners https://review.opendev.org/c/openstack/octavia/+/839487 | 08:13 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia stable/wallaby: Deny the creation of L7Policies for HTTPS/TCP/UDP listeners https://review.opendev.org/c/openstack/octavia/+/839488 | 08:13 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia stable/victoria: Deny the creation of L7Policies for HTTPS/TCP/UDP listeners https://review.opendev.org/c/openstack/octavia/+/839489 | 08:14 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia stable/ussuri: Deny the creation of L7Policies for HTTPS/TCP/UDP listeners https://review.opendev.org/c/openstack/octavia/+/839490 | 08:18 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia stable/train: Deny the creation of L7Policies for HTTPS/TCP/UDP listeners https://review.opendev.org/c/openstack/octavia/+/839492 | 08:19 |
| opendevreview | Tom Weininger proposed openstack/octavia-tempest-plugin master: Add scenario tests for failover circuit breaker https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/837878 | 08:37 |
| *** ysandeep|lunch is now known as ysandeep | 08:52 | |
| opendevreview | Vadim Ponomarev proposed openstack/octavia-tempest-plugin master: Add waiting OPERATION_STATUS for the third listener in tests. https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/839502 | 09:35 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: Fix potential race conditions on update requests in the v2 worker https://review.opendev.org/c/openstack/octavia/+/837195 | 11:56 |
| opendevreview | Tom Weininger proposed openstack/octavia-tempest-plugin master: Add scenario tests for failover circuit breaker https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/837878 | 12:18 |
| *** ysandeep is now known as ysandeep|afk | 12:42 | |
| opendevreview | Gregory Thiemonge proposed openstack/octavia-tempest-plugin master: Remove old unmaintained jobs https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/838096 | 12:53 |
| opendevreview | Tom Weininger proposed openstack/octavia-tempest-plugin master: Add scenario tests for failover circuit breaker https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/837878 | 12:57 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia-tempest-plugin master: WIP Add a job that uses octavia master with stable amphora https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/838168 | 12:57 |
| *** ysandeep|afk is now known as ysandeep | 13:00 | |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: Fix unclear error messages in the API https://review.opendev.org/c/openstack/octavia/+/836314 | 13:07 |
| opendevreview | Tom Weininger proposed openstack/octavia-tempest-plugin master: Add scenario tests for failover circuit breaker https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/837878 | 13:54 |
| *** ysandeep is now known as ysandeep|out | 14:21 | |
| opendevreview | Merged openstack/octavia stable/yoga: Fix AttributeError in exception handler https://review.opendev.org/c/openstack/octavia/+/839041 | 14:55 |
| opendevreview | Merged openstack/octavia stable/wallaby: Fix AttributeError in exception handler https://review.opendev.org/c/openstack/octavia/+/839044 | 14:55 |
| opendevreview | Merged openstack/octavia stable/ussuri: Fix AttributeError in exception handler https://review.opendev.org/c/openstack/octavia/+/839046 | 14:56 |
| gthiemonge | #startmeeting Octavia | 16:00 |
| opendevmeet | Meeting started Wed Apr 27 16:00:57 2022 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 | 16:01 |
| johnsom | o/ | 16:01 |
| tweining | Hi | 16:01 |
| spencerharmon | Hello! | 16:01 |
| gthiemonge | #topic Announcements | 16:02 |
| gthiemonge | No announcement today... any announcements on your side? | 16:02 |
| tweining | not really | 16:02 |
| gthiemonge | ok | 16:03 |
| gthiemonge | #topic Octavia CI status | 16:04 |
| gthiemonge | The major CI outages are fixed, thank you Folks! | 16:04 |
| johnsom | FYI, devstack is dropping centos-8-stream support | 16:04 |
| gthiemonge | yeha I wanted to talk about c9s in this section ;-) | 16:04 |
| gthiemonge | FYI I've worked on reproducing and analyzing our (not) "random" CI failures from | 16:05 |
| johnsom | Ok, sorry for jumping ahead | 16:05 |
| gthiemonge | #link https://etherpad.opendev.org/p/octavia-ci-failures | 16:05 |
| gthiemonge | NP ;-) | 16:05 |
| gthiemonge | I found that 2 of these issues are race conditions when updating a resource in amphorav2 | 16:05 |
| gthiemonge | I proposed a fix for it: | 16:05 |
| gthiemonge | #link https://review.opendev.org/c/openstack/octavia/+/837195 | 16:06 |
| tweining | I'm currently reviewing it | 16:06 |
| gthiemonge | I already got a +2 from johnsom ;-) | 16:06 |
| tweining | I have at least one question I think. | 16:06 |
| tweining | but you will see ;) | 16:06 |
| gthiemonge | hehe | 16:06 |
| gthiemonge | basically I fixed the v2 code by reusing the code from v1 ;-) | 16:07 |
| gthiemonge | tweining: waiting for your review | 16:07 |
| gthiemonge | still about CI: | 16:07 |
| gthiemonge | we still have open patches for the CI configuration | 16:07 |
| gthiemonge | (adding yoga jobs, removing old jobs) | 16:07 |
| gthiemonge | #link https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/838060 | 16:08 |
| gthiemonge | #link https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/838096 | 16:08 |
| johnsom | Long term that should not be necessary as the v2 driver should migrate to using the native provider driver data, but this is an interim solution. | 16:08 |
| johnsom | Oh, along that line, today Victoria is officially EM | 16:09 |
| gthiemonge | yeah I agree, but this is a huge work | 16:09 |
| johnsom | Yep, incremental | 16:09 |
| gthiemonge | so... after the job cleanup, I think the next task will be to fix/update the centos-9-stream job | 16:10 |
| gthiemonge | (c9s is part of the tested runtimes for Zed - and not c8s) | 16:10 |
| gthiemonge | #link https://governance.openstack.org/tc/reference/runtimes/zed.html | 16:10 |
| gthiemonge | we have 2 pending patches in o-t-p and octavia for c9s, I've rechecked them | 16:11 |
| gthiemonge | still waiting to see if everything is alright | 16:12 |
| johnsom | Question, here: | 16:13 |
| johnsom | #link https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/838096/5/zuul.d/jobs.yaml#809 | 16:13 |
| johnsom | Why are we keeping spare pool job for wallaby? | 16:13 |
| gthiemonge | If I remember correctly this is the only supported(non-EM) release that supports spare pool | 16:14 |
| gthiemonge | but yeah, we could also remove it | 16:14 |
| johnsom | Oh, it wasn't out of deprecation cycle for Wallaby, ok that makes sense | 16:14 |
| gthiemonge | ok | 16:15 |
| gthiemonge | #topic Brief progress reports / bugs needing review | 16:16 |
| gthiemonge | well my activity was mostly related to the previous section ^^ | 16:16 |
| spencerharmon | It's the usual topic for me. | 16:17 |
| spencerharmon | #link https://review.opendev.org/c/openstack/octavia/+/831051 | 16:17 |
| spencerharmon | Since last week, I removed all v1 changes, added a bit of documentation and release notes, added an option to disable notifications, and changed methods in rpc.py to use assert syntax. | 16:17 |
| johnsom | I am doing reviews as time permits | 16:18 |
| spencerharmon | I noticed Tom reviewed since then (thank you!), but let me know if there are any further changes needed. | 16:18 |
| tweining | spencerharmon: I tested it and it seems to work well | 16:18 |
| spencerharmon | Sure thing. No rush! | 16:19 |
| gthiemonge | thanks spencerharmon I'll try to review it before our next meeting ;-) | 16:19 |
| tweining | but I needed this patch to get it working: https://review.opendev.org/c/openstack/octavia/+/801860 | 16:19 |
| tweining | so maybe it makes sense to rebase the patch? | 16:20 |
| gthiemonge | the CI merges the (branch of the) patch from gerrit with master, so it should work there | 16:21 |
| spencerharmon | I see. Should I rebase or would that cause a conflict for CI? | 16:22 |
| gthiemonge | it should not conflict on rebase, because zuul is already able to merge it | 16:23 |
| gthiemonge | I have no opinion on that | 16:23 |
| spencerharmon | Ok. That's a good call-out, thank you. | 16:24 |
| tweining | oh, and I'm done with my scenario test for the new failover circuit breaker feature | 16:27 |
| tweining | https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/837878 it would be nice to get some feedback on that and related patches (same topic) | 16:28 |
| gthiemonge | I'll test it! | 16:28 |
| gthiemonge | #topic Open Discussion | 16:31 |
| gthiemonge | any other topics? | 16:31 |
| tweining | no, but I commented on the change we discussed at the beginning | 16:34 |
| gthiemonge | tweining: thanks | 16:36 |
| gthiemonge | ok Folks, thank you! | 16:36 |
| gthiemonge | #endmeeting | 16:36 |
| opendevmeet | Meeting ended Wed Apr 27 16:36:24 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:36 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/octavia/2022/octavia.2022-04-27-16.00.html | 16:36 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/octavia/2022/octavia.2022-04-27-16.00.txt | 16:36 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/octavia/2022/octavia.2022-04-27-16.00.log.html | 16:36 |
| johnsom | Thanks Greg! | 16:36 |
| tweining | hm, about that zed tested runtimes thing. so we can use Python 3.8 features in code from now on? | 16:38 |
| johnsom | 3.8 and newer yes. The setup.cfg will need to be updated to reflect that | 16:39 |
| johnsom | If it hasn't been already | 16:39 |
| * tweining looks forward to using assignment expressions in the future | 16:40 | |
| johnsom | https://github.com/openstack/octavia/blob/master/setup.cfg#L20 | 16:40 |
| johnsom | https://github.com/openstack/octavia/blob/master/setup.cfg#L9 | 16:40 |
| tweining | ok. thanks | 16:40 |
| gthiemonge | tweining: keep in mind that you may also want to backport the code ;-) | 16:41 |
| johnsom | Yep | 16:41 |
| tweining | ok, that leaves it at least for major new features. | 16:43 |
| johnsom | You can always adjust them on backport too if you really feel they bring value | 16:43 |
| opendevreview | Merged openstack/octavia stable/victoria: Fix AttributeError in exception handler https://review.opendev.org/c/openstack/octavia/+/839045 | 16:47 |
| opendevreview | Merged openstack/octavia stable/wallaby: Pass timeout_dict to _get_haproxy_versions https://review.opendev.org/c/openstack/octavia/+/835616 | 16:47 |
| opendevreview | Merged openstack/octavia stable/victoria: Add keepalive for redis-based taskflow boards https://review.opendev.org/c/openstack/octavia/+/826029 | 16:55 |
| opendevreview | Merged openstack/octavia stable/ussuri: Add keepalive for redis-based taskflow boards https://review.opendev.org/c/openstack/octavia/+/826030 | 16:56 |
| opendevreview | Tom Weininger proposed openstack/octavia master: WIP Move to Py38 https://review.opendev.org/c/openstack/octavia/+/839604 | 17:02 |
| opendevreview | Merged openstack/octavia stable/wallaby: Restart rsyslog from cloud-init https://review.opendev.org/c/openstack/octavia/+/837548 | 17:13 |
| opendevreview | Merged openstack/octavia stable/wallaby: Add keepalive for redis-based taskflow boards https://review.opendev.org/c/openstack/octavia/+/826028 | 17:40 |
| opendevreview | Merged openstack/octavia-tempest-plugin master: Update 'Maintained' stable branch jobs on master gate https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/838060 | 18:33 |
| opendevreview | Merged openstack/octavia stable/xena: Fix AttributeError in exception handler https://review.opendev.org/c/openstack/octavia/+/839043 | 20:06 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!