Wednesday, 2024-03-06

*** promethe- is now known as prometheanfire01:30
opendevreviewMerged openstack/octavia-dashboard master: reno: Update master for unmaintained/wallaby  https://review.opendev.org/c/openstack/octavia-dashboard/+/91130907:25
tobias-urdingthiemon1e: troubleshooting octavia-dashboard eating away resources, trying this https://bugs.launchpad.net/octavia/+bug/2055161 did not really help07:57
tobias-urdinthe decision to use threading in octavia-dashboard is questionable, we open a lot of thread and never keep a handle or close them, when running httpd with mod_wsgi these are real OS threads that then just lay around07:59
tobias-urdineach thread opens the log files and keep them in memory, i can easily DoS anybody using octavia-dashboard by just doing a lot of requests and eventually httpd will eat up all resources08:00
tobias-urdini understand the retry logic was added with https://storyboard.openstack.org/#!/story/2008798 but that the threading was there even before, imo nuking all that code and returning an error to the client like anybody using the API would get would be preferable instead08:01
tobias-urdini don't see any good way to do background threading/tasks as horizon does not have any such logic08:01
tobias-urdin(the _thread import is also the internal low-level threading code that will be removed from python eventually, same issue as with eventlet)08:04
gthiemon1etobias-urdin: for the creation of a loadbalancer, we could use the fully-populated API (all the resources are created in a single POST call) instead of multiple calls, but that works only with a new LB, we cannot add listener+pool+members to an existing LB with this method08:05
gthiemon1etobias-urdin: yeah maybe we should consider a new approach08:06
tobias-urdinsince _thread contains dead code, i guess we could try using real threading module and trace git history to when start_new_thread() was used and why, but i have no idea if that would even help08:07
tobias-urdinbut imo using threads at all in a API by a horizon plugin should probably not be done at all08:08
tobias-urdinit would make more sense if octavia-worker queued up the work08:08
opendevreviewMerged openstack/octavia master: reno: Update master for unmaintained/wallaby  https://review.opendev.org/c/openstack/octavia/+/91131508:47
opendevreviewMerged openstack/octavia master: reno: Update master for unmaintained/xena  https://review.opendev.org/c/openstack/octavia/+/91132408:47
opendevreviewMerged openstack/octavia master: reno: Update master for unmaintained/victoria  https://review.opendev.org/c/openstack/octavia/+/91130509:09
opendevreviewTobias Urdin proposed openstack/python-octaviaclient master: Support filtering amphora list by image ID  https://review.opendev.org/c/openstack/python-octaviaclient/+/91095914:56
gthiemon1e#startmeeting Octavia16:00
opendevmeetMeeting started Wed Mar  6 16:00:40 2024 UTC and is due to finish in 60 minutes.  The chair is gthiemon1e. Information about MeetBot at http://wiki.debian.org/MeetBot.16:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:00
opendevmeetThe meeting name has been set to 'octavia'16:00
gthiemon1ehi16:00
tweiningo/16:00
gthiemon1e#topic Announcements16:02
gthiemon1e* 2024.1 Caracal Release Schedule16:02
gthiemon1eLast week was the Feature Freeze and final release for client libraries16:03
gthiemon1ewe released python-octaviaclient 3.7.016:03
gthiemon1ewe will not approve feature commits on master until the stable branch is cut16:04
gthiemon1ethis week, it's the deadline for the cycle highlights16:04
gthiemon1ejohnsom proposed an highlight for the SR-IOV feature (thanks!)16:05
tweiningi think its merged now16:05
tweininghttps://review.opendev.org/c/openstack/releases/+/91101316:05
gthiemon1e+1 nice!16:05
gthiemon1eNext week is RC116:06
gthiemon1eplease focus on reviewing the bugfixes that are on top of this list (or any other bugfix if you think it's worth it)16:06
gthiemon1ehttps://etherpad.opendev.org/p/octavia-priority-reviews16:06
gthiemon1edon't hesite to ping us if one of your commits is stuck/not reviewed16:09
gthiemon1e#topic Brief progress reports / bugs needing review16:10
gthiemon1eany significant progress/issues to report?16:14
tweiningdont ask. I have nothing for today. :)16:14
gthiemon1e:D16:14
gthiemon1e#topic Open Discussion16:14
gthiemon1eif anyone has an additional topic for this meeting, now is the time!16:15
gthiemon1eok that was quick!16:18
gthiemon1ethanks tweining 16:18
gthiemon1esee you next week16:18
gthiemon1e#endmeeting16:18
opendevmeetMeeting ended Wed Mar  6 16:18:39 2024 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:18
opendevmeetMinutes:        https://meetings.opendev.org/meetings/octavia/2024/octavia.2024-03-06-16.00.html16:18
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/octavia/2024/octavia.2024-03-06-16.00.txt16:18
opendevmeetLog:            https://meetings.opendev.org/meetings/octavia/2024/octavia.2024-03-06-16.00.log.html16:18
tweiningo/16:18
johnsomtobias-urdin: I really don’t recommend using mod_wsgi for the exact reasons mentioned. It eats resources unnecessarily.16:44
johnsomtobias-urdin: Why aren’t you running it in uesgi or gunicorn?16:45
opendevreviewGaudenz Steinlin proposed openstack/octavia master: Config option to allow ICMP echo requests on VIP  https://review.opendev.org/c/openstack/octavia/+/91096317:19

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!