| *** yadnesh|away is now known as yadnesh | 04:02 | |
| opendevreview | Gregory Thiemonge proposed openstack/octavia stable/zed: Fix grenade job & pin pylint on stable/branches https://review.opendev.org/c/openstack/octavia/+/872866 | 07:21 |
|---|---|---|
| opendevreview | Tobias Urdin proposed openstack/octavia master: Use new get_rpc_client API from oslo.messaging https://review.opendev.org/c/openstack/octavia/+/871163 | 08:25 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Add octavia-grenade-tick-tock CI job https://review.opendev.org/c/openstack/octavia/+/860221 | 10:51 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Add octavia-grenade-tick-tock CI job https://review.opendev.org/c/openstack/octavia/+/860221 | 11:06 |
| opendevreview | Merged openstack/octavia stable/yoga: Fix grenade job https://review.opendev.org/c/openstack/octavia/+/872868 | 11:48 |
| opendevreview | Merged openstack/octavia stable/zed: Fix grenade job & pin pylint on stable/branches https://review.opendev.org/c/openstack/octavia/+/872866 | 12:53 |
| *** yadnesh is now known as yadnesh|away | 13:24 | |
| *** oschwart__ is now known as oschwart | 14:25 | |
| gthiemonge | #startmeeting Octavia | 16:01 |
| opendevmeet | Meeting started Wed Feb 8 16:01:27 2023 UTC and is due to finish in 60 minutes. The chair is gthiemonge. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:01 |
| opendevmeet | The meeting name has been set to 'octavia' | 16:01 |
| johnsom | o/ | 16:01 |
| tweining | o/ | 16:01 |
| gthiemonge | Hi Folks | 16:01 |
| matfechner | o/ | 16:03 |
| oschwart | o/ | 16:03 |
| gthiemonge | #topic Announcements | 16:03 |
| johnsom | octavia-lib for Antelope released | 16:04 |
| gthiemonge | Next week is Antelope-3 milestone: Feature Freeze | 16:04 |
| gthiemonge | +1 | 16:04 |
| johnsom | Yep, and client releases | 16:04 |
| gthiemonge | I'll focus on reviewing feature patches | 16:04 |
| gthiemonge | there's only one patch for the client | 16:04 |
| gthiemonge | reminder, the review priority list is: | 16:05 |
| gthiemonge | #link https://etherpad.opendev.org/p/octavia-priority-reviews | 16:05 |
| johnsom | I have +2'd that already. Maybe tweining can have a look | 16:06 |
| tweining | ok | 16:07 |
| gthiemonge | any other announcement folks? | 16:07 |
| gthiemonge | #topic CI Status | 16:09 |
| gthiemonge | we fixed issues with grenade and pylint this week, I think all the fixes are merged now | 16:10 |
| gthiemonge | (master and stable branches) | 16:10 |
| gthiemonge | another topic: | 16:11 |
| gthiemonge | froyo is adding new tests (missing tests) to test_traffic_ops in octavia-tempest-plugin | 16:11 |
| gthiemonge | it increases the duration of the scenario jobs, the c9s job seems to time out really often now | 16:12 |
| gthiemonge | I tried to split the class into 2 classes but c9s still fails from time to time | 16:12 |
| gthiemonge | so maybe it's time to take another approach: spliting the scenario job into 2 jobs (for instance traffic classes vs the rest) | 16:13 |
| gthiemonge | what do you think about that? | 16:13 |
| tweining | if it makes the execution faster, fine with me | 16:14 |
| oschwart | Sounds good to me | 16:14 |
| johnsom | We have done that in the past. Though, I have not looked at this patch from froyo to see if we can optimize it better. | 16:14 |
| johnsom | I would like to review it before we merge it. | 16:14 |
| gthiemonge | running the test_traffic_ops class takes 2h in my env | 16:15 |
| gthiemonge | sure | 16:15 |
| gthiemonge | we will not merge it until we get acceptable durations for the jobs | 16:15 |
| johnsom | plus we probably don't want to merge that at this point in the release as we probably have priority patches we don't want to impact | 16:15 |
| gthiemonge | right | 16:15 |
| johnsom | repeat=19 | 16:16 |
| johnsom | Yeah, that is probably more time than we really need to spend in the test | 16:16 |
| gthiemonge | in most jobs, this test class finished more than one hour after the other classes (ok we have only 2 concurrent classes at a time) | 16:17 |
| gthiemonge | anywys, I will propose a test patch that splits the job, so we can see the durations | 16:18 |
| johnsom | Each one of those new test cases will make 19 requests now, so yeah, huge jump in test runtime needed. This needs some review time. | 16:19 |
| gthiemonge | note that c9s job fails already with a lot of timeouts | 16:19 |
| johnsom | Yeah, agreed, C9S is slow... | 16:20 |
| johnsom | I'm just not sure we need to add 228 round trips to the members to have a good test. | 16:20 |
| gthiemonge | ok | 16:21 |
| gthiemonge | #topic Brief progress reports / bugs needing review | 16:24 |
| gthiemonge | on my side, I've worked on these gate issues (pylint/grenade) | 16:25 |
| gthiemonge | + reviews | 16:25 |
| gthiemonge | + a lot of downstream stuff | 16:25 |
| oschwart | Would this failure mean that we still have pylint issues on stable branches? | 16:26 |
| johnsom | Lots of review for me | 16:26 |
| oschwart | #link https://review.opendev.org/c/openstack/python-octaviaclient/+/869806 | 16:26 |
| gthiemonge | oschwart: I fixed only the octavia repo, I'll take a look at the octaviaclient, but here it passed 10min ago | 16:27 |
| tweining | I updated the patch that adds an tick-tock CI grenade upgrade job | 16:27 |
| tweining | https://review.opendev.org/c/openstack/octavia/+/860221 | 16:27 |
| gthiemonge | oschwart: it seems that we need to backport | 16:28 |
| gthiemonge | https://review.opendev.org/c/openstack/python-octaviaclient/+/872357 | 16:28 |
| johnsom | Oh, nice. The terminology is a mess for tick-tock vs slurp and the docs seem to contradict themselves. I'm glad someone took the time to figure out what needs to happen there. | 16:28 |
| oschwart | gthiemonge: I will take care of it if you guys are ok with it | 16:29 |
| gthiemonge | oschwart: ok! it seems that it is only needed on zed | 16:29 |
| tweining | yeah, I guess SLURP is the correct name now. I can update the patch if you want | 16:30 |
| oschwart | gthiemonge: ack | 16:31 |
| oschwart | tweining: should we keep the ffu u/s job then? | 16:31 |
| tweining | and regarding oschwart's comment about removing octavia-grenade-ffu. I guess he's right. I will update the patch | 16:31 |
| tweining | is stein even supported anymore? | 16:32 |
| gthiemonge | tweining: haha no | 16:32 |
| gthiemonge | tweining: I tried to update the grenade-ffu job, but it was not working at all | 16:33 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Add octavia-grenade-slurp CI job https://review.opendev.org/c/openstack/octavia/+/860221 | 16:35 |
| gthiemonge | #topic Open Discussion | 16:38 |
| opendevreview | Omer Schwartz proposed openstack/python-octaviaclient stable/zed: Add munch explicitly to requirements https://review.opendev.org/c/openstack/python-octaviaclient/+/873151 | 16:41 |
| gthiemonge | any other topics folks? | 16:42 |
| oschwart | Nothing from me. I will prob ask you guys to review the munch backport patches as we have until next week (?) for the client patches | 16:43 |
| gthiemonge | I guess it will be merged before the end of week ;-) | 16:43 |
| oschwart | gthiemonge: ack | 16:44 |
| gthiemonge | ok folks, thank you! | 16:44 |
| gthiemonge | have a great week | 16:44 |
| gthiemonge | #endmeeting | 16:44 |
| opendevmeet | Meeting ended Wed Feb 8 16:44:19 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:44 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/octavia/2023/octavia.2023-02-08-16.01.html | 16:44 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/octavia/2023/octavia.2023-02-08-16.01.txt | 16:44 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/octavia/2023/octavia.2023-02-08-16.01.log.html | 16:44 |
| opendevreview | Omer Schwartz proposed openstack/python-octaviaclient stable/yoga: Add munch explicitly to requirements https://review.opendev.org/c/openstack/python-octaviaclient/+/873153 | 16:44 |
| opendevreview | Omer Schwartz proposed openstack/python-octaviaclient stable/xena: Add munch explicitly to requirements https://review.opendev.org/c/openstack/python-octaviaclient/+/873154 | 16:46 |
| opendevreview | Omer Schwartz proposed openstack/python-octaviaclient stable/wallaby: Add munch explicitly to requirements https://review.opendev.org/c/openstack/python-octaviaclient/+/873155 | 16:47 |
| opendevreview | Merged openstack/python-octaviaclient master: Add missing flavor set --description option https://review.opendev.org/c/openstack/python-octaviaclient/+/866850 | 16:52 |
| opendevreview | Fernando Royo proposed openstack/octavia-tempest-plugin master: Spliting healthmonitor test for each pool protocol https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/870824 | 20:09 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!