| *** ysandeep|out is now known as ysandeep|ruck | 03:12 | |
| *** ysandeep|ruck is now known as ysandeep|ruck|afk | 04:32 | |
| *** beagles_ is now known as beagles | 04:45 | |
| *** ysandeep|ruck|afk is now known as ysandeep|ruck | 05:08 | |
| opendevreview | Tom Weininger proposed openstack/octavia master: Add WebTest as an indirect test dependency https://review.opendev.org/c/openstack/octavia/+/848816 | 08:22 |
|---|---|---|
| opendevreview | Tom Weininger proposed openstack/octavia master: Add WebTest as an indirect test dependency https://review.opendev.org/c/openstack/octavia/+/848816 | 08:31 |
| *** ysandeep|ruck is now known as ysandeep|ruck|lunch | 08:43 | |
| opendevreview | Tom Weininger proposed openstack/octavia master: Add WebTest as an indirect test dependency https://review.opendev.org/c/openstack/octavia/+/848816 | 08:50 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Add WebTest as an indirect test dependency https://review.opendev.org/c/openstack/octavia/+/848816 | 09:02 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Add WebTest as an indirect test dependency https://review.opendev.org/c/openstack/octavia/+/848816 | 09:07 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Add WebTest as an indirect test dependency https://review.opendev.org/c/openstack/octavia/+/848816 | 09:29 |
| *** ysandeep|ruck|lunch is now known as ysandeep|ruck | 09:53 | |
| opendevreview | Tom Weininger proposed openstack/octavia master: Add WebTest as an indirect test dependency https://review.opendev.org/c/openstack/octavia/+/848816 | 09:56 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: Apply openstack-selinux policies in Centos amphorae https://review.opendev.org/c/openstack/octavia/+/840315 | 10:17 |
| gthiemonge | Hi Folks, the functional job is broken because of a new release of pecan, here is a patch from tweining: https://review.opendev.org/c/openstack/octavia/+/848816 | 12:59 |
| gthiemonge | could you review it or tell us if you have some concerns about it, we might merge it today with only one CR+2 | 13:00 |
| fkr | gthiemonge: johnsom: fyi - I'll not be able to make it to todays meeting. | 14:19 |
| gthiemonge | fkr: ack, np | 14:24 |
| *** ysandeep|ruck is now known as ysandeep|dinner | 15:28 | |
| gthiemonge | #startmeeting Octavia | 16:00 |
| opendevmeet | Meeting started Wed Jul 6 16:00:35 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 Folks! | 16:00 |
| tweining | o/ | 16:00 |
| oschwart | o/ | 16:01 |
| gthiemonge | #topic Announcements | 16:02 |
| gthiemonge | * New core reviewer in the Octavia group | 16:02 |
| tweining | seems it's going to be a RH-internal meeting :) | 16:02 |
| gthiemonge | Great news, tweining is now part of the core reviewer group for Octavia! | 16:02 |
| gthiemonge | congrats and thank you for your work tweining ;-) | 16:03 |
| tweining | yay, that's great. thank you for all your support. | 16:03 |
| oschwart | Congratulations tweining!!! | 16:03 |
| *** ysandeep|dinner is now known as ysandeep | 16:03 | |
| tweining | I already gave my first CR+2 and CR-2 ;) | 16:04 |
| gthiemonge | any other announcements? | 16:04 |
| gthiemonge | -2? | 16:04 |
| tweining | the duplicate change | 16:04 |
| gthiemonge | duplicate, ok | 16:05 |
| matfechner | o/ | 16:05 |
| gthiemonge | #topic CI Status | 16:06 |
| gthiemonge | FYI the functional job is broken because of an update of pecan | 16:06 |
| gthiemonge | tweining has proposed a patch | 16:06 |
| gthiemonge | #link https://review.opendev.org/c/openstack/octavia/+/848816 | 16:06 |
| gthiemonge | johnsom approved it few minutes ago, the gates should be fixed soon | 16:06 |
| gthiemonge | the change includes an update of l-c.txt | 16:07 |
| tweining | yes, next we will try to remove the l-c.txt | 16:07 |
| gthiemonge | but I think that now, we need to merge the patch that removes that job | 16:07 |
| gthiemonge | #link https://review.opendev.org/c/openstack/octavia/+/840108 | 16:07 |
| gthiemonge | yup | 16:07 |
| gthiemonge | I will approve it whe nthe fix is merged | 16:08 |
| *** ysandeep is now known as ysandeep|out | 16:08 | |
| tweining | btw. did we announce that registration for PTG is open? | 16:09 |
| gthiemonge | nop, not yet | 16:09 |
| gthiemonge | there was also an email for the team signup, I think I need to ping our contributors to know if they plan to go to Columbus | 16:10 |
| tweining | yes, it would be interesting to know who plans to go there | 16:12 |
| gthiemonge | yeah | 16:12 |
| gthiemonge | #topic Brief progress reports / bugs needing review | 16:13 |
| gthiemonge | no much activity from me, I've spent a lot of time on downstream issues | 16:13 |
| gthiemonge | I know that we have a long list of backports to review, I hope I'll be able to review them before Friday | 16:14 |
| spencerharmon | I'm just catching up on @tweining's comments regarding the documentation for notifications. I'm making those edits now. I'm unsure how to proceed about the default notifications.info topic. I haven't seen any documentation that oslo messaging is changing this behavior and in my lab environment, this topic is created. | 16:15 |
| gthiemonge | spencerharmon: your lab env is devstack? | 16:18 |
| spencerharmon | No, it isn't | 16:18 |
| tweining | https://review.opendev.org/c/openstack/octavia/+/831051 for the record, this is the notifications change | 16:18 |
| gthiemonge | ok, we could try to figure out why the notification queue doesn't get any messages | 16:20 |
| gthiemonge | I will re-test it | 16:20 |
| spencerharmon | Yes, thank you! I can work on setting that up, but my thought is that it's likely to result in the same behavior you both saw. In that event, I think the thing to do is to document this, right? Alternatively, I can see about adding logic to create this if it doesn't exist. | 16:20 |
| gthiemonge | so, in my env, the notifications.info topic exists, but it doesn't get the messages | 16:21 |
| spencerharmon | Makes sense. I suppose I could set this as a default if no topic is provided. | 16:22 |
| gthiemonge | and it exists, maybe not because of octavia, but because of other services, I don';t know | 16:22 |
| opendevreview | Spencer Harmon proposed openstack/octavia master: Add event notifications for load balancers. https://review.opendev.org/c/openstack/octavia/+/831051 | 16:22 |
| gthiemonge | BTW, I forgot this topc, I recreated the review list: | 16:24 |
| gthiemonge | #link https://etherpad.opendev.org/p/octavia-priority-reviews | 16:24 |
| gthiemonge | you can change the priority/order of the items if you want | 16:24 |
| gthiemonge | notifications is on top of the feature list | 16:25 |
| spencerharmon | Top of the list is ok with me! ;) | 16:25 |
| tweining | from my perspective the notifications change is okay as it is now | 16:25 |
| gthiemonge | yeah I agree, except with this notifications.info topic ;-) | 16:27 |
| spencerharmon | I have not concerns leaving it, but I'm also happy to investigate explicitly setting the default notifications topic to ensure it behaves the way I've documented. | 16:27 |
| gthiemonge | spencerharmon: yeah that would be nice if you could take a look at it | 16:27 |
| spencerharmon | It says "You may specify," but as is, you *must* specify. | 16:27 |
| tweining | at least in devstack I tried both the default topic (which is notification) as well as setting the topic to notifications explictly. same result | 16:28 |
| spencerharmon | I'll work on that. Don't worry about retesting until I get that in. | 16:29 |
| tweining | cool | 16:30 |
| gthiemonge | spencerharmon: ok thanks, don't hesitate to ping us before the next meetign if you have an update! | 16:30 |
| tweining | also, we have about 2 month till feature freeze so there is no rush | 16:30 |
| spencerharmon | Sure thing! I'll be out of town next week, but I'll let yall know once I have something for you to review on this issue. | 16:31 |
| spencerharmon | Thanks for the reviews! :) | 16:31 |
| gthiemonge | #topic Open Discussion | 16:33 |
| gthiemonge | any other topics? | 16:34 |
| tweining | hm, not sure. I'm working on a POC implementation for CPU pinning of HAProxy in the amp | 16:35 |
| tweining | and I have one design problem. CW sends the rendered HAProxy.cfg to the API server on the amp. | 16:36 |
| tweining | what I need is to get the number of vCPUs from the amp and add it in the config. | 16:37 |
| tweining | so my idea was to extent the api so CW queries the number of vcpus from the API server on the amp before rendering the haproxy cfg. | 16:38 |
| tweining | there are probably other ways to do this, but that seems to be the easiest way to me. | 16:39 |
| tweining | but I didn't spend a lot of thought on it yet, so maybe it's a bit too early to discuss | 16:40 |
| gthiemonge | there's an info endpoint in the amphora | 16:40 |
| gthiemonge | https://opendev.org/openstack/octavia/src/branch/master/octavia/amphorae/backends/agent/api_server/server.py#L170-L172 | 16:40 |
| gthiemonge | https://opendev.org/openstack/octavia/src/commit/d590d6c7051a9d9941900efc85959e9158609e08/octavia/amphorae/backends/agent/api_server/amphora_info.py#L34-L45 | 16:40 |
| gthiemonge | I think it's pretty safe to extend it | 16:40 |
| gthiemonge | but the worker might communicate with amphorae that do not include the new fields, so you have to take it into account | 16:41 |
| tweining | I also need to generate another file on the amp, a file with variables for TuneD. for that I also need the number of vCPUs. I guess I can handle that entirely on amp side though. | 16:42 |
| tweining | thanks, I will have a look | 16:42 |
| gthiemonge | https://opendev.org/openstack/octavia/src/commit/d590d6c7051a9d9941900efc85959e9158609e08/octavia/amphorae/backends/agent/api_server/amphora_info.py#L47 | 16:42 |
| gthiemonge | this function incldues more info | 16:43 |
| gthiemonge | but I don't think it is used by the controller services | 16:43 |
| opendevreview | Merged openstack/octavia master: Add WebTest as an indirect test dependency https://review.opendev.org/c/openstack/octavia/+/848816 | 16:43 |
| gthiemonge | yeah! gates are fixed^^ | 16:43 |
| tweining | :D | 16:44 |
| oschwart | \o/ | 16:44 |
| spencerharmon | Woot! | 16:44 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/octavia/+/840108 | 16:44 |
| gthiemonge | ^ merge conflict. tweining: could you review it again? | 16:45 |
| tweining | sure | 16:46 |
| gthiemonge | thanks | 16:46 |
| gthiemonge | anything else folks? | 16:47 |
| tweining | I don't think so | 16:47 |
| oschwart | Nothing from me | 16:47 |
| gthiemonge | ok! | 16:47 |
| gthiemonge | well, thank you all! | 16:48 |
| gthiemonge | #endmeeting | 16:48 |
| opendevmeet | Meeting ended Wed Jul 6 16:48:09 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:48 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/octavia/2022/octavia.2022-07-06-16.00.html | 16:48 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/octavia/2022/octavia.2022-07-06-16.00.txt | 16:48 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/octavia/2022/octavia.2022-07-06-16.00.log.html | 16:48 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!