16:00:01 #startmeeting Octavia 16:00:02 Meeting started Wed Jan 6 16:00:01 2021 UTC and is due to finish in 60 minutes. The chair is rm_work. Information about MeetBot at http://wiki.debian.org/MeetBot. 16:00:03 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 16:00:05 The meeting name has been set to 'octavia' 16:00:15 Welcome back folks, and welcome to 2021! 16:00:15 o/ 16:00:29 #chair johnsom 16:00:30 Current chairs: johnsom rm_work 16:00:53 hi 16:01:01 oh good, someone else is actually here :P 16:01:22 o/ 16:01:29 #topic Announcements 16:01:38 already covered this one a little bit 16:01:42 hi 16:01:54 Happy New Year! Welcome to 2021! We're living in the future! The future is now! 16:02:13 ohhh-Ahhhh-wow 16:02:14 grin 16:02:27 Does that mean I can retire? 16:02:41 Oh, maybe not *that* far into the future 16:02:46 In the words of my wife: no retiring! work until you die! :D 16:03:00 (who wins that argument is still up in the air) 16:03:23 (pretty sure it's gonna be me) 16:03:28 So any other annoucements? 16:03:31 Yeah, sadly it isn't coming any time soon for me 16:03:32 *announcements 16:03:44 just ride the BTC to retirement? :P 16:03:55 anywho... 16:04:06 I don't really have anything else. 16:04:30 How far are we from code freeze now? I feel like we've just started the cycle, so code freeze must be like next week, right? lol 16:05:13 cgoncalves are you around or out on vacation? 16:05:29 Vacation today 16:05:36 cool cool 16:05:40 me too (mentally) 16:05:48 MS2 is January 18t 16:05:51 physically, I'm right here with all of you. metaphorically. 16:06:13 lol ok great so kidding-not-kidding on these milestones sneaking up on us 16:06:19 Last day for libs is first week in March 16:06:27 Yes, VERY fast 16:06:27 anywho, moving on 16:06:30 #topic Brief progress reports / bugs needing review 16:06:45 I have been on vacation, so nothing to report 16:07:26 I have a few (lower-constraints fixes on stable branches) 16:07:47 https://review.opendev.org/c/openstack/octavia/+/768464 https://review.opendev.org/c/openstack/octavia/+/769268 https://review.opendev.org/c/openstack/octavia/+/769348 https://review.opendev.org/c/openstack/octavia/+/765853 16:07:57 My progress: none. I have done essentially nothing this cycle... except one bugfix attempt here: https://review.opendev.org/c/openstack/octavia/+/767648 16:08:16 last one is not passing the requirements-check yet, but I'm not losing hope 16:08:18 which I have no idea if it's the right approach or not, i just gave it a quick go, but turns out we don't need that patch internally so I kinda lost interest <_< 16:09:12 I'll try to pick it up again once i've gotten through the majority of the post-long-moratorium fires 16:11:23 so yeah... guess that's it 16:11:31 #topic Open Discussion 16:11:36 so how bout them Mets? 16:11:46 anyone else have anything? 16:12:34 No idea, are they even playing? 16:12:43 I don't have anything else this week 16:12:53 rm_work: have a question for you on https://review.opendev.org/c/openstack/octavia/+/656811/ - there was a comment about using threshold as a % 16:13:05 * haleyb snuck that one in under the wire 16:13:09 I'm going to try to pick up smoking (meats) so I am spending way too much of my free time watching brisket smoking videos. 16:13:21 really embracing the "open" in "open discussion" 16:13:28 that's all I've got 16:13:35 hmmm yeah percentages 16:13:53 rm_work: i could update, think it's pretty straight forward 16:14:03 Percentage is.... 16:14:13 I don't think that's what we want honestly? like... 16:14:21 I see why you could think you want that 16:14:39 but I gave it some thought and IMO at any given point there should be essentially zero 16:14:42 One thing on that RFE idea we need to consider is how to override that should the cloud situation require it. I.e. once we hit that threshold, then what..... 16:14:57 in a tiny cloud, i don't think i'd want to limit it to like 10% because that could be ONE and that doesn't tell you anything 16:15:15 in a huge cloud, 10% would be WAY too high, i'd still want it to stop after like 5-10 16:15:37 Make it a float. 0.00000001% 16:15:38 grin 16:15:49 lol i mean, point being the same percentage is wrong for both in opposite ways 16:16:02 10% too small for a tiny cloud and too big for a large one 16:16:13 I just want to set it to like ... 5 16:16:19 * rm_work shrugs 16:16:21 yeah, it's knowing what to set it at... 16:16:30 5? or 10 :D 16:16:34 you know your environment 16:16:41 and it's 3 clearly :) 16:16:43 you can change it as the env grows if you think you need to 16:16:52 I SAID FIVE, BRIAN 16:16:59 42 16:17:02 lol 16:17:09 5 is right out 16:17:35 anywho, percent is fine? but i think it isn't actually what people want, it's just what they think they want 16:17:37 Yeah, I know we started a discussion over should it be percent or absolute value. I don't remember where that came out. 16:17:40 dunno, could add it, but i'm not going to 16:17:52 someone else can do the work? but it needs to support BOTH 16:18:15 and not be hilariously confusing like "10" and "10%" being radically different numbers of things 16:18:39 Do we have a spec for this? 16:18:46 * rm_work sighs again 16:18:54 don't think so 16:19:00 * johnsom grins with an evil smirk 16:19:16 feel like i'm a stormtrooper getting some jedi-talk, but it was from https://etherpad.opendev.org/p/octavia-train-ptg 16:19:24 ooh, train 16:19:55 ah there is a story 16:20:05 ok so it does say % in there 16:20:49 but it mentions % over a time period 16:21:32 * haleyb could have looked at the story before asking... 16:21:42 HA, quoting michael: 16:21:43 Personally, I am not sure percentage is a good way to measure this as a large deployment may see a single rack outage as a small percentage of their amphora (2,000+ load balancers is not uncommon). 16:21:43 I am thinking the threshold might be best set to a number slightly larger than the maximum number of amphora you would expect on a single host. 16:21:56 --Michael Johnson, 2020-02-21 16:22:28 yeah, that makes more sense, so maybe just need to add to release note or comment in conf file 16:22:34 lol, yeah, I'm open to re-thinking this. 16:22:46 no, no rethinking, i agree with you 16:22:53 your opinion is set in stone right there :P 16:23:07 Oh, I meant the percentage thing in the story 16:23:09 grin 16:23:16 heh 16:26:30 ok soooo 16:26:37 i guess that's it 16:27:02 i guess 16:27:03 unless we want to continue this percentage discussion for another 30m 16:27:08 we CAN do that :P 16:27:09 This is a priority for us to get merged 16:27:23 IMO it is fine as-is 16:27:42 but if you want to add percentage parsing too ... i guess... you could do that if you want 16:28:43 it would have to be a separate option imo 16:29:17 Yeah, I agree 16:29:37 Personally I would implement one way and see if there is a need for the other. 16:31:28 johnsom: ack, maybe we can just add your "threshold slightly larger than # amps on a compute node" to the release note? 16:31:40 Sure 16:32:04 i'll send a quick update since it's needs a rebase anyway most likely 16:32:09 So it looks like an absolute value is what is implemented in the patch, so I think we should just move forward with that and see how it goes 16:33:48 sure 16:36:34 i commented with some examples 16:36:55 +1 16:40:02 Any thing else today? 16:40:34 nope 16:40:36 have fun ya'll 16:40:41 #endmeeting