15:05:49 <noonedeadpunk> #startmeeting openstack_ansible_meeting 15:05:49 <opendevmeet> Meeting started Tue Apr 26 15:05:49 2022 UTC and is due to finish in 60 minutes. The chair is noonedeadpunk. Information about MeetBot at http://wiki.debian.org/MeetBot. 15:05:49 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 15:05:49 <opendevmeet> The meeting name has been set to 'openstack_ansible_meeting' 15:05:58 <noonedeadpunk> #topic rollcall 15:06:04 <mgariepy> hey 15:06:07 <noonedeadpunk> hey 15:06:08 <jamesdenton> hi 15:06:21 <noonedeadpunk> sorry, I tried to run from my phone first but seems it doesn't work :( 15:07:51 <noonedeadpunk> or I connected to freenode there..... 15:08:05 <noonedeadpunk> anyway) 15:09:00 <damiandabrowski[m]> hey! 15:10:20 <noonedeadpunk> #topic office hours 15:11:09 <noonedeadpunk> first of all I want to congrat damiandabrowski[m] in our core team! 15:11:20 <jrosser> o/ hello 15:11:21 <noonedeadpunk> I just added him to the group, haven't written ML yet :) 15:11:30 <jamesdenton> congrats! 15:11:41 <mgariepy> congrats damiandabrowski[m] ! :D 15:12:07 <NeilHanlon> congrats damiandabrowski[m]! 15:12:23 <damiandabrowski[m]> thanks guys! 15:12:41 <noonedeadpunk> Then just wanted to say - I'm not near IRC this week so can't really check on things :( 15:12:58 <noonedeadpunk> but since we have one more reviewer now we likely covered :p 15:13:06 <opendevreview> Merged openstack/openstack-ansible-haproxy_server master: Add functionality to accept both HTTP and HTTPS during upgrade https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/829899 15:13:32 <mgariepy> damiandabrowski[m], so you only need to cover for noonedeadpunk work this week ;p no pressure ! :P 15:13:46 <noonedeadpunk> yeah, exactly :D 15:15:51 <damiandabrowski[m]> hahha :D I just want to kindly ask for few more reviews in my tempest patches https://review.opendev.org/q/topic:tempest-damian-2021-12+AND+-status:merged 15:17:07 <NeilHanlon> i'm always happy to provide a blind +1 ;) 15:17:23 <NeilHanlon> will take a look at those today out of interest damiandabrowski[m] 15:18:19 <damiandabrowski[m]> thanks! 15:18:28 <jrosser> damiandabrowski[m]: i did look but i think there is a *lot* of rebasing needed? it was quite confusing 15:19:25 <opendevreview> Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_octavia master: Use PKI role for certificate generation https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/839068 15:19:51 <damiandabrowski[m]> yeah the plan was to gather initial reviews -> apply corrections(if required) -> merge 15:20:17 <damiandabrowski[m]> i planned to do it like this to avoid unnecessary zuul jobs, but i can rebase them now if You think it's better idea 15:23:29 <noonedeadpunk> damiandabrowski[m]: I think you can do rebase now... 15:23:41 <noonedeadpunk> It was a plan to wait for a week and 2 passed till now ;) 15:24:23 <noonedeadpunk> To have that said, I pushed WIP patch regarding Octavia using PKI role 15:24:38 <noonedeadpunk> https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/839068 15:25:14 <jrosser> we have the upgrade case to handle yet? 15:25:16 <noonedeadpunk> It doesn't have upgrade path and my best thought to handle upgrade is to create a playbook to integrated repo 15:25:32 <noonedeadpunk> yeah 15:26:22 <noonedeadpunk> As I actually don't see better way then just to rename/move existing certs on the octavia_cert_setup_host 15:27:41 <noonedeadpunk> Btw have we checked 22.04 availability in CI? 15:28:26 <noonedeadpunk> Seems it's present?:) https://opendev.org/openstack/project-config/src/branch/master/nodepool/nodepool.yaml#L42 15:30:24 <noonedeadpunk> we should try adding this out:) 15:31:24 <noonedeadpunk> anything I missed and requires attention? 15:32:08 <mgariepy> should we try to clenaup the review queue ? there is patches there since 2017.. 15:32:40 <noonedeadpunk> I believe some of them could be still valid though.... 15:33:22 <noonedeadpunk> I can totally recall one that I'm not dropping on purpose not to forget to revise one day - about how we're handling limiting across hosts for nova/cinder 15:35:48 <mgariepy> yeah some of them are probably still good but there are a couple of them that i suspect not needed anymore.. 15:36:14 <mgariepy> like one of compatibility matrix update.. from 2018.. 15:37:01 <noonedeadpunk> Yeah..... 15:37:07 <noonedeadpunk> well.... 15:37:10 <mgariepy> we have about 240 patches on master.. 15:37:34 <mgariepy> but yeah we should try to clean up the queue a bit/ reorganize them 15:38:43 <jrosser> I think that we should use the dashboard more effectively 15:39:29 <jrosser> working on fixing the things that need reverify, actively reviewing the ones with +1 15:39:32 <noonedeadpunk> Do you see any changes to it? 15:39:53 <jrosser> i think i mean that the information is pretty clear in the dashboard as it is 15:40:09 <mgariepy> not all the patches show up there tho. 15:40:10 <jrosser> but without actively working on the patches to get them moved through the stages and merged, they will just pile up 15:40:13 <mgariepy> i think 15:42:26 <noonedeadpunk> mgariepy: I _think_ this should be arranged now when I parented all projects under openstack-ansible group 15:42:26 <opendevreview> Merged openstack/openstack-ansible-os_tempest master: do not include [*-feature-enabled] sections in tempest.conf https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/825164 15:42:56 <noonedeadpunk> But if you found any patch missing from it - just ping me and we will fix that :) 15:43:51 <mgariepy> https://review.opendev.org/q/parentproject:openstack/openstack-ansible+branch:master+status:open 15:44:13 <mgariepy> 10 pages. 15:44:22 <jrosser> we miss the ones which are +2+W but blocked? 15:44:51 <mgariepy> ho. 15:45:30 <jrosser> mgariepy: you can click the headings in the dashboards and get the full lists for each category? 15:46:10 <mgariepy> not sure. 15:46:38 <noonedeadpunk> I bet it's possible.... 15:47:15 <opendevreview> Dmitriy Rabotyagov proposed openstack/openstack-ansible-galera_server stable/xena: Use separate tmp directory https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/839338 15:48:01 <noonedeadpunk> jrosser: you mean we miss a category for that? 15:48:36 <noonedeadpunk> basically having +V+2+W ? 15:48:42 <jrosser> i think there is an "approved but cannot merge" category 15:49:23 <jrosser> like if there is an outstanding depends-on 15:49:52 <noonedeadpunk> iirc there was no such catregory 15:50:07 <noonedeadpunk> But I think I can try to add one 15:50:50 <jrosser> i do remember we mentioned this before, maybe there was also a zuul mergability flag that was disabled, and may now be returned 15:51:24 <noonedeadpunk> Oh, yes, we likely discussed that but I never happen to follow up on that... 15:51:35 <opendevreview> Merged openstack/openstack-ansible master: In AIO and CI set the internal vip to https only for 'deploy' actions. https://review.opendev.org/c/openstack/openstack-ansible/+/830431 15:51:37 <noonedeadpunk> Basically I still haven't added queues... 15:51:44 <jrosser> hmm 15:52:30 <noonedeadpunk> But I kind of tend to add queues after branching at least to experiment when we're done with main bunch of things we want to land before experimenting 15:53:07 <noonedeadpunk> Regarding adding such category to dashboard - I think it's usefull and will write down to take a look there one more time 15:54:00 <jrosser> i am just working on upgrade path for lsyncd -> glusterfs 15:56:34 <mgariepy> yay no more lsyncd soon !:D 16:01:29 <noonedeadpunk> hurray! really great to hear! 16:01:53 <noonedeadpunk> I will continue looking in octavia and stuff only next week :( 16:01:58 <noonedeadpunk> #endmeeting