14:00:19 <rafaelweingartner> #startmeeting cloudkitty
14:00:19 <opendevmeet> Meeting started Mon Sep 15 14:00:19 2025 UTC and is due to finish in 60 minutes.  The chair is rafaelweingartner. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:00:19 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:00:19 <opendevmeet> The meeting name has been set to 'cloudkitty'
14:00:21 <rafaelweingartner> Hello guys!
14:00:24 <rafaelweingartner> Roll count
14:00:25 <rafaelweingartner> \o
14:03:46 <priteau> o/
14:04:26 <rafaelweingartner> Nice to see you Priteau!
14:04:52 <priteau> Sorry, I have had conflicts with another meeting for many weeks now
14:05:00 <priteau> It finished early today
14:05:00 <rafaelweingartner> no worries :)
14:05:05 <rafaelweingartner> glad you are here now
14:06:04 <rafaelweingartner> so let's start
14:06:08 <rafaelweingartner> #topic vPTG
14:06:27 <opendevreview> Merged openstack/cloudkitty master: Fix FileBackend in Python 3  https://review.opendev.org/c/openstack/cloudkitty/+/957435
14:06:30 <rafaelweingartner> I have registered the team, and we just need to book the room. However, I think that the room registration is not open yet.
14:06:37 <jlarriba> o/, sorry im late :)
14:06:46 <rafaelweingartner> no worries, we just started =)
14:07:06 <rafaelweingartner> as soon as the room registration is open, I will do so in our usual vPTG time slot
14:07:16 <rafaelweingartner> which is basically, our usual meeting time
14:07:24 <rafaelweingartner> is that ok for you guys?
14:07:56 <priteau> October 27?
14:09:01 <rafaelweingartner> yes
14:09:28 <priteau> I might be on holiday, unsure. But if I am working I will make myself available.
14:09:34 <opendevreview> Merged openstack/cloudkitty master: Remove unused [state] options  https://review.opendev.org/c/openstack/cloudkitty/+/957436
14:09:34 <rafaelweingartner> ok
14:09:37 <opendevreview> Merged openstack/cloudkitty master: Drop unused BaseIOBackend  https://review.opendev.org/c/openstack/cloudkitty/+/957437
14:09:38 <jlarriba> i am eager to see you guys again there
14:09:46 <rafaelweingartner> me too
14:11:36 <rafaelweingartner> so, moving on
14:11:41 <rafaelweingartner> #topic Target reviews
14:11:56 <rafaelweingartner> #link https://review.opendev.org/c/openstack/cloudkitty/+/960374
14:12:09 <rafaelweingartner> is an interesting one, I just asked the author to remove one extra space in the code
14:12:24 <rafaelweingartner> other reviews would be welcome in this one, and in the others that we will go through today
14:15:49 <rafaelweingartner> #link https://review.opendev.org/c/openstack/cloudkitty/+/950542, this one Pedro and I reviewed. IF you are ok with it, priteau: I guess we can merge it
14:17:15 <priteau> Is this a bug fix that we would backport? If yes, can you add a release note?
14:17:32 <rafaelweingartner> hmm
14:17:40 <rafaelweingartner> I think it would be an interesting patch to backport
14:17:55 <rafaelweingartner> I am not the owner of the patch, but I guess we can ask the author to create a reno release notes
14:18:24 <priteau> Also the Git author should be updated from "root <root@cloudkitty.dev.rc.nectar.org.au>" ;-)
14:18:50 <priteau> I will respond
14:18:57 <rafaelweingartner> I did not get it
14:20:22 <rafaelweingartner> now, I see what you mean
14:20:44 <rafaelweingartner> moving on, we have #link https://review.opendev.org/c/openstack/cloudkitty/+/960032
14:20:52 <rafaelweingartner> it is changing a default for the cloudkitty threads
14:21:14 <rafaelweingartner> It changes a default configuration, I do not know what should be our policies. However, the change seem to make sense
14:21:22 <rafaelweingartner> We never use those configs as default
14:22:54 <priteau> That's a good one
14:22:58 <rafaelweingartner> cool
14:23:04 <rafaelweingartner> can you do the honours then?
14:23:08 <priteau> We are sometimes working with nodes that have 192 cores
14:23:29 <rafaelweingartner> exactly
14:23:48 <priteau> That would be 184320 threads?
14:24:33 <rafaelweingartner> a lot:)
14:24:47 <priteau> Approved
14:25:09 <rafaelweingartner> thanks!
14:25:23 <rafaelweingartner> moving on #link https://review.opendev.org/c/openstack/cloudkitty/+/950868, we have this one
14:25:26 <rafaelweingartner> we are fine with it
14:25:35 <rafaelweingartner> therefore, we are just waiting your feedback
14:26:10 <priteau> I will review in more depth soon. It's going to be for the G release anyway
14:26:24 <rafaelweingartner> ok
14:26:34 <priteau> An important one is https://review.opendev.org/c/openstack/cloudkitty/+/961242
14:26:41 <priteau> Cherry pick for 2025.2
14:27:08 <priteau> I thought it would be included in 2025.2 directly, I didn't realise you had not updated the release SHA
14:27:15 <rafaelweingartner> I merged some of those
14:27:36 <rafaelweingartner> I also did not paid attention to this one
14:27:46 <priteau> We should have updated https://review.opendev.org/c/openstack/releases/+/960069 as soon as it merged.
14:28:12 <rafaelweingartner> ok
14:28:42 <priteau> The cleanup patches merged to master recently should probably have waited until after feature freeze ended too
14:29:03 <rafaelweingartner> did not it finish?
14:29:08 <priteau> Sorry, I see they just merged this afternoon
14:29:12 <priteau> Git history confused me
14:29:25 <rafaelweingartner> I mean, the feature freeze was already over
14:29:39 <rafaelweingartner> I merged today only
14:29:55 <priteau> Yeah, as I said Git history confused me just now, sorry.
14:30:01 <rafaelweingartner> :)
14:30:32 <priteau> I know some projects don't start merging to master right after RC1 because they want to keep the master branch stable in case they need to make some important fixes + backports
14:30:44 <rafaelweingartner> I see
14:30:53 <rafaelweingartner> but in our case is more or less controlled, right
14:33:14 <rafaelweingartner> Moving one, we also have #link https://review.opendev.org/c/openstack/cloudkitty/+/953827
14:33:44 <rafaelweingartner> It is a patch proposed by me, which is an interesting feature to be used to enable people to configure methods/expressions to skip persisting some of the processed rating points
14:40:08 <rafaelweingartner> moving one, we have this other small patch, which is interesting as well to have #link https://review.opendev.org/c/openstack/cloudkitty/+/953762
14:40:48 <rafaelweingartner> on a similar line, we have #link https://review.opendev.org/c/openstack/cloudkitty/+/958374 and #link https://review.opendev.org/c/openstack/cloudkitty/+/954880
14:45:25 <rafaelweingartner> as we are reaching our final minutes. Instead of going over the other patches, I would like to open for the open floor, for people to discuss/comment things that they find important.
14:48:27 <jlarriba> i dont know if you noticed, but we submitted a patch to add the Loki testing to the CI: https://review.opendev.org/c/openstack/cloudkitty/+/960455
14:48:44 <jlarriba> which might be very useful to be able to ensure that everything works as expected
14:50:38 <priteau> rafaelweingartner: Should we make an RC2 to include the security patch?
15:00:38 <rafaelweingartner> I would say so
15:00:55 <rafaelweingartner> jlarriba: yes, I did notice it
15:01:49 <jlarriba> ack
15:02:37 <priteau> rafaelweingartner: will you propose a release patch?
15:02:44 <rafaelweingartner> sure I can
15:02:50 <rafaelweingartner> I took some note here to do
15:02:57 <rafaelweingartner> I will just wait for the patches to get merged
15:03:02 <rafaelweingartner> I am not sure if they were already all merged
15:03:05 <priteau> Someone is asking if we can backport the thread count patch
15:03:14 <priteau> https://review.opendev.org/c/openstack/cloudkitty/+/960032
15:03:56 <rafaelweingartner> maybe
15:03:56 <priteau> Maybe we backport it but don't try to include it in the rc2?
15:04:03 <rafaelweingartner> ok
15:04:09 <priteau> It could have bad effects that we haven't noticed yet
15:04:23 <rafaelweingartner> exactly
15:04:27 <rafaelweingartner> and it is not that important
15:04:30 <priteau> For the security fix, you need to wait until it has merged
15:04:36 <rafaelweingartner> someone can always override the defautls
15:05:02 <priteau> true
15:05:30 <rafaelweingartner> and that is basically it =)
15:05:32 <rafaelweingartner> great job guys!
15:05:42 <rafaelweingartner> thank you all for participating. Have a nice week!
15:07:09 <rafaelweingartner> #endmeeting