Monday, 2025-11-10

*** mrunge_ is now known as mrunge07:30
rafaelweingartner#startmeeting cloudkitty14:00
opendevmeetMeeting 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
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:00
opendevmeetThe meeting name has been set to 'cloudkitty'14:00
rafaelweingartnerHello guys!14:00
rafaelweingartnerRoll count14:00
rafaelweingartner\O14:02
rafaelweingartnernobody ?14:11
jwysogla_I'm here. sorry o/14:11
*** jwysogla_ is now known as jwysogla14:13
rafaelweingartnercool14:13
rafaelweingartnerhow are you doing?14:13
jwysoglahaha.. yea, fine. I thought the meeting was starting in an hour? I guess I just don't remember it correctly.14:14
mrungehey, good afternoon o/14:14
rafaelweingartner#topic vPTG meeting14:15
rafaelweingartnerIt 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-cloudkitty14:16
mrungethank you for having us Rafael14:16
jwysoglaIt was nice to be able to chat with you guys14:17
rafaelweingartner=)14:18
rafaelweingartnercool14:18
priteauHello.14:19
rafaelweingartner#topic Target reviews14:20
mrungeCan I please get another set of eyes on https://review.opendev.org/c/openstack/cloudkitty/+/961609 ?14:21
rafaelweingartner#link https://review.opendev.org/c/openstack/cloudkitty/+/96160914:21
rafaelweingartnerI was going to talk about this one :)14:21
rafaelweingartnerit seems to be fine now14:21
rafaelweingartnerand we just need other pair of eyes there :)14:22
mrungeindeed14:22
rafaelweingartner#link https://review.opendev.org/c/openstack/cloudkitty/+/953827, this other is a nice to have14:24
priteauI added a comment to the previous one14:25
jwysoglaI left a few comments there last week14:25
rafaelweingartneryes, I have to respond your comment14:26
rafaelweingartnerI Will do so this week14:26
jwysoglaAlso 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:29
priteauDone14:30
jwysoglathank you14:30
opendevreviewJuan Larriba proposed openstack/cloudkitty master: Shard the requests to Loki  https://review.opendev.org/c/openstack/cloudkitty/+/96160914:34
rafaelweingartnerMoving on, what do you guys about this one https://review.opendev.org/c/openstack/cloudkitty/+/958255?14:37
mrungeJuan addressed the latest comment from Pierre on his patch14:37
mrungeI like the idea of dumping  out the config like Takashi proposed. We have the change in aodh or ceilometer or both? I don't recall14:38
mrungeMy comment was related to the config dump patch https://review.opendev.org/c/openstack/cloudkitty/+/95825514:39
jwysoglaI 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:39
rafaelweingartnerme too14:42
rafaelweingartnerI find it useful depending on the context, this can help with the troubleshooting14:42
rafaelweingartnerMoving on, the #link https://review.opendev.org/c/openstack/cloudkitty/+/957656 seems to make sense.14:44
rafaelweingartnerwhat do you guys think?14:45
rafaelweingartnerI guess we can remove the code14:45
jwysoglaI'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 :D14:49
rafaelweingartneri guess nobody is using that one14:50
jlarribayeah, it does not make a lot of sense to keep something that is broken14:52
mrungeto me it absolutely makes sense to remove14:52
jlarribawe either commit to fix it if it is needed, or we remove it14:52
jlarribasorry for being late, didn't realized about the hour change respect UTC14:53
mrungeif not happened, can you please recheck https://review.opendev.org/c/openstack/cloudkitty/+/961609 ?15:09
mrungesorry for being pushy on the patch, I'd really love to land it asap15:09
opendevreviewMerged openstack/cloudkitty master: Add reject_old_samples_max_age to loki config  https://review.opendev.org/c/openstack/cloudkitty/+/96430315:35
opendevreviewMerged openstack/cloudkitty master: sqlalchemy: Use built-in declarative  https://review.opendev.org/c/openstack/cloudkitty/+/95376215:56
opendevreviewJuan Larriba proposed openstack/cloudkitty master: Shard the requests to Loki  https://review.opendev.org/c/openstack/cloudkitty/+/96160916:10
rafaelweingartnersorry guys17:44
rafaelweingartnerI lost track of time, while checking an issue and forgot the meeting :(17:45
rafaelweingartner#endmeeting17:45
opendevmeetMeeting ended Mon Nov 10 17:45:12 2025 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)17:45
opendevmeetMinutes:        https://meetings.opendev.org/meetings/cloudkitty/2025/cloudkitty.2025-11-10-14.00.html17:45
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/cloudkitty/2025/cloudkitty.2025-11-10-14.00.txt17:45
opendevmeetLog:            https://meetings.opendev.org/meetings/cloudkitty/2025/cloudkitty.2025-11-10-14.00.log.html17:45

Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!