| opendevreview | Takashi Kajinami proposed openstack/cloudkitty master: Drop unused BaseIOBackend https://review.opendev.org/c/openstack/cloudkitty/+/957437 | 08:59 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/cloudkitty master: Drop unused BaseIOBackend https://review.opendev.org/c/openstack/cloudkitty/+/957437 | 09:07 |
| opendevreview | Takashi Kajinami proposed openstack/cloudkitty master: Remove cloudkitty-writer https://review.opendev.org/c/openstack/cloudkitty/+/957656 | 09:07 |
| opendevreview | Takashi Kajinami proposed openstack/cloudkitty master: Remove cloudkitty-writer https://review.opendev.org/c/openstack/cloudkitty/+/957656 | 09:08 |
| opendevreview | Takashi Kajinami proposed openstack/cloudkitty master: Remove cloudkitty-writer https://review.opendev.org/c/openstack/cloudkitty/+/957656 | 11:38 |
| priteau | rafaelweingartner: are you around? | 14:07 |
| rafaelweingartner | #startmeeting cloudkitty | 14:08 |
| opendevmeet | Meeting started Mon Aug 18 14:08:30 2025 UTC and is due to finish in 60 minutes. The chair is rafaelweingartner. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:08 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:08 |
| opendevmeet | The meeting name has been set to 'cloudkitty' | 14:08 |
| rafaelweingartner | Sorry the delay | 14:08 |
| rafaelweingartner | Hello guys! | 14:08 |
| rafaelweingartner | Roll count | 14:08 |
| rafaelweingartner | \o | 14:08 |
| mrunge_ | o/ | 14:09 |
| *** mrunge_ is now known as mrunge | 14:09 | |
| rafaelweingartner | nice to see you all | 14:10 |
| rafaelweingartner | =) | 14:10 |
| rafaelweingartner | #topic OpenStack Cloudkitty PTL | 14:11 |
| rafaelweingartner | It is that time of the year, and we need to nominate somebody to be the PTL fo rCloudKitty | 14:11 |
| rafaelweingartner | I can still be the PTL, if nobody else wants to step up. | 14:11 |
| rafaelweingartner | What do you guys think? Would somebody be willing to be the next PTL for CloudKitty? | 14:12 |
| mrunge | thank you for stepping up or with continuing your role | 14:12 |
| mrunge | I also don't see any disagreement here ;-) | 14:14 |
| rafaelweingartner | I agree | 14:15 |
| rafaelweingartner | Ok, so I will put my name forward | 14:15 |
| rafaelweingartner | thanks for the support | 14:15 |
| mrunge | +1 | 14:16 |
| rafaelweingartner | moving on | 14:16 |
| rafaelweingartner | #topic next vPTG | 14:17 |
| rafaelweingartner | we will need to schedule the room, but it is not open yet | 14:17 |
| rafaelweingartner | as soon as it opens, I will register/schedule the room | 14:17 |
| mrunge | how many slots are you planning? | 14:17 |
| rafaelweingartner | #topic Target reviewsI would say the usual amount | 14:17 |
| mrunge | or days? | 14:17 |
| rafaelweingartner | 1 or 2 | 14:17 |
| rafaelweingartner | we have done in the past with 1 slot, for instance | 14:18 |
| rafaelweingartner | but it is up to us | 14:18 |
| mrunge | right, that's mostly based on the demand | 14:18 |
| rafaelweingartner | I guess one slot works for us | 14:18 |
| mrunge | ack | 14:18 |
| mrunge | sorry, I interrupted you | 14:19 |
| rafaelweingartner | no worries | 14:19 |
| rafaelweingartner | I had finished already :) | 14:19 |
| mrunge | on the question of reviews (probably later?) , I'd be interested to move forward with https://review.opendev.org/c/openstack/cloudkitty/+/950868 | 14:20 |
| rafaelweingartner | me too | 14:21 |
| rafaelweingartner | so, moving on to the reviews | 14:21 |
| rafaelweingartner | #topic Target reviews | 14:21 |
| rafaelweingartner | #link https://review.opendev.org/c/openstack/cloudkitty/+/876643, we have this one which I would say that we should merge it | 14:21 |
| rafaelweingartner | and then, move on with this feature | 14:21 |
| rafaelweingartner | as it has been in the making for a long time | 14:21 |
| rafaelweingartner | and all of the concerns seem to be already addressed | 14:22 |
| mrunge | from quick glance, this looks like it was ready to merge some time ago already | 14:24 |
| mrunge | I see a +W | 14:24 |
| mrunge | and it didn't merged due to an unrelated issue | 14:24 |
| mrunge | the diff between the last and the reversion before is just in the commit message | 14:25 |
| rafaelweingartner | ah, yes | 14:25 |
| rafaelweingartner | then we addressed that | 14:25 |
| rafaelweingartner | and we strive to look for consensus | 14:26 |
| rafaelweingartner | that is why I did not merge it right away, as it comes from our side | 14:26 |
| mrunge | I just +1'd, if that means anything | 14:27 |
| mrunge | it looks like a sensible change | 14:27 |
| mrunge | the change conflicts with https://review.opendev.org/c/openstack/cloudkitty/+/953762, which also makes sense | 14:29 |
| rafaelweingartner | exactly | 14:30 |
| mrunge | I would say that rebasing one patch on the other should not be impossible | 14:31 |
| tkajinam | I can rebase 953762 if 876643 is merged | 14:31 |
| tkajinam | first | 14:31 |
| mrunge | perfect | 14:32 |
| rafaelweingartner | ok | 14:32 |
| rafaelweingartner | I will do the honor then | 14:32 |
| rafaelweingartner | priteau: are you ok with this getting in? | 14:32 |
| mrunge | not here in the channel? | 14:32 |
| rafaelweingartner | hmm | 14:33 |
| rafaelweingartner | funny I thought that I saw him | 14:33 |
| mrunge | pinged you earlier | 14:33 |
| rafaelweingartner | exactly | 14:33 |
| rafaelweingartner | well, will move on then | 14:33 |
| mrunge | since the patch already had +2/+W, I would suggest to merge it, especially since there was no real change between then and now | 14:34 |
| rafaelweingartner | yes | 14:34 |
| rafaelweingartner | Moving on | 14:34 |
| rafaelweingartner | #link https://review.opendev.org/c/openstack/cloudkitty/+/950868 | 14:34 |
| rafaelweingartner | I have reviewed the patch, and it looks ok now | 14:35 |
| rafaelweingartner | the support for the custom fields is not there, but there is a log saying that | 14:35 |
| rafaelweingartner | the reprocessing is supported now | 14:35 |
| mrunge | yes | 14:35 |
| rafaelweingartner | therefore, I am fine with it | 14:35 |
| mrunge | excellent, thank you! | 14:35 |
| rafaelweingartner | We just need to hear the others CR to merge it | 14:36 |
| mrunge | I would expect so | 14:36 |
| mrunge | out of curiosity, how many active cores do you have currently? | 14:36 |
| rafaelweingartner | 3 | 14:36 |
| opendevreview | Takashi Kajinami proposed openstack/cloudkitty master: Remove cloudkitty-writer https://review.opendev.org/c/openstack/cloudkitty/+/957656 | 14:37 |
| mrunge | similar to telemetry... | 14:37 |
| rafaelweingartner | yes | 14:38 |
| rafaelweingartner | :) | 14:38 |
| mrunge | :D | 14:38 |
| rafaelweingartner | I noticed there when we propose things in Ceilometer :) | 14:38 |
| mrunge | we were adding cores, like tkajinam | 14:38 |
| rafaelweingartner | similar to Gnocchi as well I would say. I know it is not part of the OpenInfra and so on... | 14:38 |
| mrunge | indeed | 14:39 |
| rafaelweingartner | moving on | 14:41 |
| rafaelweingartner | We have #link https://review.opendev.org/c/openstack/cloudkitty/+/953827, which is a patch that I did | 14:41 |
| rafaelweingartner | to add a new option for CloudKitty to skip persisting datapoints that have zero as quantity | 14:41 |
| rafaelweingartner | this is helping us to control the load that the system applies in the storage backend (InfluxDB, ElasticSearch/OpenSerach/ and no Loki) | 14:42 |
| rafaelweingartner | Revies are welcome :) | 14:42 |
| * mrunge earmarks the patch | 14:42 | |
| rafaelweingartner | reviews* | 14:45 |
| rafaelweingartner | moving on, we have #link https://review.opendev.org/c/openstack/cloudkitty/+/953762 | 14:45 |
| rafaelweingartner | thanks tkajinam! | 14:48 |
| rafaelweingartner | for the patches | 14:48 |
| tkajinam | :-) | 14:48 |
| tkajinam | Can I ask one question regarding the series starting from https://review.opendev.org/c/openstack/cloudkitty/+/957435/ ? | 14:48 |
| rafaelweingartner | sure | 14:50 |
| tkajinam | I noticed that FileBackend has been broken since we bumped the min python version to 3.6 during Ussuri cycle. ( https://review.opendev.org/c/openstack/cloudkitty/+/957435 may fix it ) | 14:51 |
| tkajinam | this hasn't affected server side because FileBackend hasn't been unused after state persistence was moved to database from file | 14:51 |
| tkajinam | however cloudkitty-writer has been using FileBackend... it means that this CLI has been broken for multiple years | 14:52 |
| tkajinam | my question here is... is cloudkitty-writer still needed ? I found a few old note saying that it will be replaced by report API, which already exists IIUC | 14:52 |
| rafaelweingartner | I would say no | 14:53 |
| rafaelweingartner | I never used it :) | 14:53 |
| tkajinam | because it has been broken for some time without any bug report I tend to just drop it now, assuming it is not actually used (that's what I proposed in https://review.opendev.org/c/openstack/cloudkitty/+/957656 ) | 14:53 |
| rafaelweingartner | and I have been using CloudKitty and working with it since 2018 :) | 14:53 |
| rafaelweingartner | 2019* | 14:53 |
| rafaelweingartner | I would say that we drop it then | 14:53 |
| rafaelweingartner | I have never seen anybody using it | 14:54 |
| tkajinam | I wanted to ask for opinions from people actually using cloudkitty. | 14:54 |
| tkajinam | ok then I feel like I'm in the right direction :-) | 14:54 |
| rafaelweingartner | I will add a topic to this meeting agenda. Then we might discuss it again, and hear others such as pierre and maybe zigo | 14:55 |
| rafaelweingartner | if he is around in our next meeting | 14:55 |
| tkajinam | thx | 14:55 |
| rafaelweingartner | and that was basically it for today | 14:55 |
| rafaelweingartner | do you guys have something else to add? | 14:55 |
| mrunge | nope, thank you | 14:56 |
| tkajinam | no | 14:57 |
| tkajinam | was here to ask that question about cloudkitty-writer :-) | 14:57 |
| rafaelweingartner | =) | 14:57 |
| mrunge | I was mostly here for the loki patch | 14:57 |
| rafaelweingartner | always nice to see new faces here | 14:57 |
| tkajinam | ;-) | 14:58 |
| mrunge | :D | 14:58 |
| rafaelweingartner | thank you all for participating. Have a nice week! | 14:58 |
| mrunge | thank you, likewise | 14:58 |
| rafaelweingartner | #endmeeting | 14:59 |
| opendevmeet | Meeting ended Mon Aug 18 14:59:12 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:59 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cloudkitty/2025/cloudkitty.2025-08-18-14.08.html | 14:59 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cloudkitty/2025/cloudkitty.2025-08-18-14.08.txt | 14:59 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cloudkitty/2025/cloudkitty.2025-08-18-14.08.log.html | 14:59 |
| zigo | What is cloudkitty-writer ?!? | 15:49 |
| zigo | tkajinam: mrunge: ^ | 15:50 |
| zigo | I never heard of it ... | 15:50 |
| mrunge | me neither | 15:50 |
| mrunge | other than earlier today, when Takashi brought it up, and Rafael confirmed it | 15:51 |
| zigo | I just checked, in Debian, it's in the python3-cloudkitty package. | 15:53 |
| mrunge | yeah, should be a cli tool | 15:54 |
| priteau | Sorry I missed the meeting earlier, I had to leave abruptly. | 17:55 |
| priteau | I was asking myself the same question about cloudkitty-writer some months ago. No one really knows what it is, but we think it isn't used anymore. | 17:56 |
| opendevreview | Pedro Henrique Pereira Martins proposed openstack/cloudkitty-tempest-plugin master: Adapt tests to new rating API https://review.opendev.org/c/openstack/cloudkitty-tempest-plugin/+/892382 | 18:04 |
| opendevreview | Merged openstack/cloudkitty-tempest-plugin master: Adapt tests to new rating API https://review.opendev.org/c/openstack/cloudkitty-tempest-plugin/+/892382 | 21:03 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!