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