14:00:17 <rafaelweingartner> #startmeeting cloudkitty
14:00:17 <opendevmeet> Meeting started Mon Nov 10 14:00:17 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:17 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:00:17 <opendevmeet> The meeting name has been set to 'cloudkitty'
14:00:56 <rafaelweingartner> Hello guys!
14:00:59 <rafaelweingartner> Roll count
14:02:29 <rafaelweingartner> \O
14:11:11 <rafaelweingartner> nobody ?
14:11:51 <jwysogla_> I'm here. sorry o/
14:13:45 <rafaelweingartner> cool
14:13:48 <rafaelweingartner> how are you doing?
14:14:30 <jwysogla> haha.. yea, fine. I thought the meeting was starting in an hour? I guess I just don't remember it correctly.
14:14:30 <mrunge> hey, good afternoon o/
14:15:38 <rafaelweingartner> #topic vPTG meeting
14:16:00 <rafaelweingartner> It was awesome to have you guys iun the vPTG meeting. The summary of the meeting is at #link https://etherpad.opendev.org/p/oct2025-ptg-cloudkitty
14:16:58 <mrunge> thank you for having us Rafael
14:17:19 <jwysogla> It was nice to be able to chat with you guys
14:18:31 <rafaelweingartner> =)
14:18:32 <rafaelweingartner> cool
14:19:40 <priteau> Hello.
14:20:32 <rafaelweingartner> #topic Target reviews
14:21:25 <mrunge> Can I please get another set of eyes on https://review.opendev.org/c/openstack/cloudkitty/+/961609 ?
14:21:41 <rafaelweingartner> #link https://review.opendev.org/c/openstack/cloudkitty/+/961609
14:21:53 <rafaelweingartner> I was going to talk about this one :)
14:21:56 <rafaelweingartner> it seems to be fine now
14:22:05 <rafaelweingartner> and we just need other pair of eyes there :)
14:22:13 <mrunge> indeed
14:24:34 <rafaelweingartner> #link https://review.opendev.org/c/openstack/cloudkitty/+/953827, this other is a nice to have
14:25:11 <priteau> I added a comment to the previous one
14:25:20 <jwysogla> I left a few comments there last week
14:26:09 <rafaelweingartner> yes, I have to respond your comment
14:26:14 <rafaelweingartner> I Will do so this week
14:29:11 <jwysogla> Also this one: https://review.opendev.org/c/openstack/cloudkitty/+/964303 should be a quick review. It's just adding a line to the Loki config deployed by devstack.
14:30:23 <priteau> Done
14:30:49 <jwysogla> thank you
14:34:33 <opendevreview> Juan Larriba proposed openstack/cloudkitty master: Shard the requests to Loki  https://review.opendev.org/c/openstack/cloudkitty/+/961609
14:37:15 <rafaelweingartner> Moving on, what do you guys about this one https://review.opendev.org/c/openstack/cloudkitty/+/958255?
14:37:38 <mrunge> Juan addressed the latest comment from Pierre on his patch
14:38:49 <mrunge> I like the idea of dumping  out the config like Takashi proposed. We have the change in aodh or ceilometer or both? I don't recall
14:39:17 <mrunge> My comment was related to the config dump patch https://review.opendev.org/c/openstack/cloudkitty/+/958255
14:39:57 <jwysogla> I like the idea. It was really useful to see the loaded config at the start of the logs when debugging stuff in other services. I'll take a detailed look at the change later today and +1 or leave comments.
14:42:11 <rafaelweingartner> me too
14:42:23 <rafaelweingartner> I find it useful depending on the context, this can help with the troubleshooting
14:44:41 <rafaelweingartner> Moving on, the #link https://review.opendev.org/c/openstack/cloudkitty/+/957656 seems to make sense.
14:45:51 <rafaelweingartner> what do you guys think?
14:45:56 <rafaelweingartner> I guess we can remove the code
14:49:21 <jwysogla> I'd say if it's broken (and so I guess not really used), it makes sense to remove it. But I'm probably not the right person to give opinions here as I'm not familiar with that feature at all :D
14:50:00 <rafaelweingartner> i guess nobody is using that one
14:52:03 <jlarriba> yeah, it does not make a lot of sense to keep something that is broken
14:52:08 <mrunge> to me it absolutely makes sense to remove
14:52:11 <jlarriba> we either commit to fix it if it is needed, or we remove it
14:53:34 <jlarriba> sorry for being late, didn't realized about the hour change respect UTC
15:09:09 <mrunge> if not happened, can you please recheck https://review.opendev.org/c/openstack/cloudkitty/+/961609 ?
15:09:38 <mrunge> sorry for being pushy on the patch, I'd really love to land it asap
15:35:03 <opendevreview> Merged openstack/cloudkitty master: Add reject_old_samples_max_age to loki config  https://review.opendev.org/c/openstack/cloudkitty/+/964303
15:56:16 <opendevreview> Merged openstack/cloudkitty master: sqlalchemy: Use built-in declarative  https://review.opendev.org/c/openstack/cloudkitty/+/953762
16:10:14 <opendevreview> Juan Larriba proposed openstack/cloudkitty master: Shard the requests to Loki  https://review.opendev.org/c/openstack/cloudkitty/+/961609
17:44:46 <rafaelweingartner> sorry guys
17:45:04 <rafaelweingartner> I lost track of time, while checking an issue and forgot the meeting :(
17:45:12 <rafaelweingartner> #endmeeting