opendevreview | Ghanshyam proposed openstack/octavia-tempest-plugin master: Remove unused dead code https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/797715 | 03:23 |
---|---|---|
opendevreview | Gregory Thiemonge proposed openstack/octavia-tempest-plugin master: Set the act-stdby jobs as voting jobs https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/798791 | 06:07 |
opendevreview | Gregory Thiemonge proposed openstack/octavia-tempest-plugin master: Fix two-node job configuration https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/773888 | 07:28 |
opendevreview | Carlos Gonçalves proposed openstack/octavia-tempest-plugin master: Set the act-stdby jobs as voting jobs https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/798791 | 07:30 |
opendevreview | Carlos Gonçalves proposed openstack/octavia-tempest-plugin master: Add stable/wallaby jobs to check and gate queues https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/798795 | 07:30 |
gthiemonge | New pylint release, pep8 job is broken: https://zuul.openstack.org/build/87738f6acb9c483aa8d453f6729d8d76 | 08:16 |
opendevreview | Gregory Thiemonge proposed openstack/octavia-tempest-plugin master: Fix two-node job configuration https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/773888 | 08:57 |
opendevreview | Gregory Thiemonge proposed openstack/octavia master: Fix new pylint issues https://review.opendev.org/c/openstack/octavia/+/798811 | 09:16 |
opendevreview | Gregory Thiemonge proposed openstack/octavia-tempest-plugin master: Fix two-node job configuration https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/773888 | 09:33 |
opendevreview | Gregory Thiemonge proposed openstack/octavia-tempest-plugin master: Fix two-node job configuration https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/773888 | 11:55 |
*** gthiemon1e is now known as gthiemonge | 12:49 | |
gthiemonge | #startmeeting Octavia | 16:00 |
opendevmeet | Meeting started Wed Jun 30 16:00:40 2021 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:00 |
johnsom | o/ | 16:01 |
haleyb | hi | 16:01 |
gthiemonge | #topic Announcements | 16:02 |
gthiemonge | CI is RED today | 16:02 |
gthiemonge | 1) PEP8 job is broken | 16:02 |
gthiemonge | A new pylint release introduced new checkers | 16:03 |
gthiemonge | and some older checkers detects new errors | 16:03 |
gthiemonge | I proposed a fix, I think it's already CR+2 W+1 | 16:03 |
johnsom | Yay for linters.... | 16:03 |
gthiemonge | #link https://review.opendev.org/c/openstack/octavia/+/798811 | 16:03 |
gthiemonge | I got it when rechecking a patch after the neutron SG rule deletion issue was merged :/ | 16:04 |
gthiemonge | 2) octavia-tox-functional-py37-tips is unstable | 16:04 |
gthiemonge | there's a timeout issue when testing the DriverAgent | 16:04 |
gthiemonge | Probably related to the slowness of the CI host | 16:05 |
gthiemonge | I have a workaround, I'll propose it after the meeting | 16:05 |
johnsom | Do you have a link to the logs of one of these failures? | 16:05 |
gthiemonge | but I still don't understand why it occurs only on the -tips job | 16:05 |
gthiemonge | #link https://zuul.openstack.org/build/d7abd5b0ef7b49c69e3325313fb46603 | 16:06 |
johnsom | This is a "learning opportunity" as that is my code. grin | 16:06 |
johnsom | Thanks | 16:06 |
gthiemonge | basically there's a settimeout(5) on a socket in the driver_lib | 16:06 |
gthiemonge | and we're hitting the timeout | 16:06 |
gthiemonge | in my env, receiving a message in the functional tests takes less than 200ms | 16:07 |
rm_work | there's some stuff that periodically I realize is not mocked that probably should be in certain situations | 16:07 |
rm_work | this is functional tho? | 16:07 |
gthiemonge | I managed to reproduce it with the cpulimit tool | 16:07 |
gthiemonge | functional yes | 16:07 |
rm_work | hmm k | 16:07 |
johnsom | Yeah, it's a very simple unix domain socket protocol, so ... interesting that it can't send data in 5 seconds | 16:08 |
gthiemonge | yeah i agree | 16:08 |
gthiemonge | any other announcements? | 16:10 |
johnsom | PTG is scheduled and registration is open. | 16:10 |
gthiemonge | Oh, I missed that :D | 16:11 |
johnsom | #link https://www.openstack.org/ptg/ | 16:11 |
gthiemonge | johnsom: thanks | 16:12 |
rm_work | I need these to be in-person again T_T | 16:12 |
gthiemonge | in... person... what is that? | 16:12 |
johnsom | We are all virtual AIs | 16:13 |
johnsom | There is also a lively discussion of community goals for the next release on the discuss list. | 16:14 |
johnsom | #link http://lists.openstack.org/pipermail/openstack-discuss/2021-June/023350.html | 16:14 |
johnsom | If you are interested in the goal process | 16:14 |
gthiemonge | #topic Brief progress reports / bugs needing review | 16:17 |
gthiemonge | I found an issue regarding haproxy worker cleanup in active standby | 16:19 |
gthiemonge | I opened a story that describes the behavior: | 16:19 |
gthiemonge | #link https://storyboard.openstack.org/#!/story/2009005 | 16:19 |
gthiemonge | and I proposed a patch: | 16:19 |
gthiemonge | #link https://review.opendev.org/c/openstack/octavia/+/797882 | 16:19 |
johnsom | That one is super interesting and a good find! | 16:20 |
gthiemonge | I wanted to get some reviews, but it seems it's already W+1 | 16:20 |
johnsom | In theory broken for quite a while. | 16:20 |
gthiemonge | yeah the stick tables are not synchronized after a config reload | 16:21 |
johnsom | I still need to go back and read up as I thought that was done over unix domain sockets, so wouldn't need lo, but that is probably something I remember wrong. | 16:21 |
johnsom | This goes back to design sessions we had in Seattle with a HAProxy core | 16:21 |
gthiemonge | I read somewhere that haproxy uses the peers in the peer list even for the local sync | 16:22 |
johnsom | I have been working on changing how we use tempest service clients to align with some "new-ish" guidelines for tempest plugins. | 16:22 |
johnsom | #link https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/797715 | 16:22 |
johnsom | Huge thanks to g-mann (no need to ping him) for helping with that. | 16:23 |
gthiemonge | johnsom: and it started with a one-line patch | 16:23 |
johnsom | It did.... sigh | 16:23 |
johnsom | A conversation in #openstack-qa with the ironic team about service client credentials.... | 16:24 |
johnsom | Ah, well, that should be pretty close if not good. I need to re-review today | 16:24 |
johnsom | Otherwise my time has been on reviews and Designate things | 16:25 |
gthiemonge | I'm still working on the two-node job, a commit was in review, but it was broken by the switch to ML2/OVN | 16:25 |
gthiemonge | with some help from the neutron folkks, I managed to fix the last issue (floating ips not reaachable) | 16:25 |
gthiemonge | #link https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/773888 | 16:26 |
gthiemonge | but now: TIMED_OUT | 16:26 |
gthiemonge | great | 16:26 |
johnsom | Grin | 16:27 |
johnsom | We should look into the "parallel" devstack install code. I thought that was default now, but maybe not? It would in theory stop devstack from installing each node sequentially. | 16:27 |
johnsom | Hnmm, looks like that is working there, so... maybe another issue | 16:29 |
gthiemonge | yeah | 16:29 |
gthiemonge | it looks like controller2 is ready 50 min after the start of the job | 16:29 |
opendevreview | Merged openstack/octavia stable/wallaby: Fix race conditions between API and worker DB calls https://review.opendev.org/c/openstack/octavia/+/798255 | 16:29 |
johnsom | Hmm yeah, so maybe not | 16:29 |
gthiemonge | I'll take a look at it | 16:30 |
johnsom | 50 minutes.... I wonder if devstack still says "will be done in 10 minutes". lol | 16:30 |
gthiemonge | #topic Open Discussion | 16:32 |
gthiemonge | any other topics? | 16:32 |
* haleyb is still neck deep in the ovn provider, so nothing else from me | 16:34 | |
gthiemonge | ok folks | 16:35 |
gthiemonge | thank you! | 16:35 |
gthiemonge | #endmeeting | 16:35 |
opendevmeet | Meeting ended Wed Jun 30 16:35:20 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:35 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/octavia/2021/octavia.2021-06-30-16.00.html | 16:35 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/octavia/2021/octavia.2021-06-30-16.00.txt | 16:35 |
opendevmeet | Log: https://meetings.opendev.org/meetings/octavia/2021/octavia.2021-06-30-16.00.log.html | 16:35 |
johnsom | gthiemonge https://opendev.org/openstack/devstack/commit/e53142ed0d314f07d974a104005be2120056d629 | 16:37 |
gthiemonge | hmmm | 16:37 |
gthiemonge | isn't it for the async tasks? | 16:38 |
johnsom | I don't know a lot about it, but something to check. | 16:38 |
gthiemonge | I don't think it deploys 2 nodes in // | 16:38 |
johnsom | Yeah, maybe that doesn't help with multi-node stuff | 16:38 |
johnsom | Anyway, that was what I was thinking about, so maybe not useful | 16:39 |
opendevreview | Merged openstack/octavia master: Fix new pylint issues https://review.opendev.org/c/openstack/octavia/+/798811 | 18:17 |
opendevreview | Michael Johnson proposed openstack/octavia-tempest-plugin master: Update service client access in tempest tests https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/797715 | 23:16 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!