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