| opendevreview | Merged openstack/cloudkitty master: Fix Loki storage delete with None end parameter https://review.opendev.org/c/openstack/cloudkitty/+/970606 | 12:23 |
|---|---|---|
| rafaelweingartner | #startmeeting cloudkitty | 13:59 |
| opendevmeet | Meeting started Mon Jan 5 13:59:24 2026 UTC and is due to finish in 60 minutes. The chair is rafaelweingartner. Information about MeetBot at http://wiki.debian.org/MeetBot. | 13:59 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 13:59 |
| opendevmeet | The meeting name has been set to 'cloudkitty' | 13:59 |
| rafaelweingartner | Hello guys! | 13:59 |
| jwysogla | hello | 13:59 |
| rafaelweingartner | Happy new year! | 14:00 |
| jwysogla | Yes, Happy new year to all you too! | 14:01 |
| rafaelweingartner | Roll count | 14:02 |
| rafaelweingartner | \o | 14:02 |
| jwysogla | o/ | 14:02 |
| jwysogla | I believe Juan is still out | 14:03 |
| priteau | o/ | 14:03 |
| rafaelweingartner | #topic Target reviews | 14:04 |
| rafaelweingartner | #link https://review.opendev.org/c/openstack/cloudkitty/+/953827, we have this one that has been in the making for a while. Your feedbacks are welcome there. | 14:04 |
| mattcrees | o/ | 14:07 |
| priteau | rafaelweingartner: I was off for a couple of weeks. I will try to review during the next 2 weeks | 14:08 |
| rafaelweingartner | ok, thanks! | 14:08 |
| rafaelweingartner | Moving on, we have #link https://review.opendev.org/c/openstack/cloudkitty/+/969934, which I just did a recheck. Let's see if the tests pass | 14:12 |
| opendevreview | Merged openstack/cloudkitty master: Ignore sample config file created in doc build https://review.opendev.org/c/openstack/cloudkitty/+/971627 | 14:13 |
| rafaelweingartner | moving on, we have #link https://review.opendev.org/c/openstack/cloudkitty/+/965897. priteau: are you fine with it now? | 14:16 |
| priteau | Yes, +2 | 14:20 |
| rafaelweingartner | thanks! | 14:20 |
| rafaelweingartner | merged | 14:20 |
| rafaelweingartner | moving on, we have #link https://review.opendev.org/c/openstack/cloudkitty/+/958204, I am not sure on how to porceed here. The patch propsoses somethign to a code that I am not sure if it is even used anymore in production. | 14:21 |
| rafaelweingartner | what do you guys think? | 14:21 |
| jwysogla | If it's stated somewhere, that the v1 driver is deprecated, then it seems strange to me to add new config options to it. | 14:24 |
| jwysogla | oh, it's not completely new, he's just modifying code for an existing one | 14:25 |
| rafaelweingartner | exactly | 14:27 |
| rafaelweingartner | but this code/api should be probably remove some time | 14:27 |
| mattcrees | The change looks fine, but I agree with v1 being deprecated there doesn't seem much point in merging new features to it | 14:30 |
| mattcrees | Do we have any planned timeline for removing this backend? | 14:30 |
| rafaelweingartner | not yet | 14:31 |
| rafaelweingartner | should we discuss this in during our next meetings? | 14:31 |
| mattcrees | Sure. | 14:32 |
| jwysogla | +1 | 14:32 |
| rafaelweingartner | ok, I will add a note for us to mature the topic with time | 14:32 |
| jwysogla | Code looks good to me. It just depends whether something should be merged into v1 driver. | 14:32 |
| rafaelweingartner | exactly | 14:33 |
| mattcrees | I think if the backend will stick around for a while yet, albeit deprecated, it doesn't hurt to merge this patch. It shouldn't change any behaviour, just validate existing behaviour. | 14:33 |
| jwysogla | yes | 14:34 |
| rafaelweingartner | I agree | 14:34 |
| mattcrees | I'll give it a closer review and +2 if it all looks good then :) | 14:35 |
| rafaelweingartner | ok, thanks matt! | 14:36 |
| rafaelweingartner | and last, but not least, we have the following new spec #link https://review.opendev.org/c/openstack/cloudkitty-specs/+/966753. I hae not read it. However, I think that we should try to target it for this up comming release | 14:37 |
| jwysogla | I was about to mention it :) | 14:38 |
| mattcrees | I've not had the time to read this yet. I'll take a look in the coming weeks | 14:39 |
| rafaelweingartner | Thanks Matt! | 14:40 |
| jwysogla | thank you | 14:40 |
| rafaelweingartner | and that is all from my side. Do you guys have something else to add? | 14:42 |
| rafaelweingartner | I guess we can close today then | 14:48 |
| mattcrees | There's nothing else from me. Thanks for hosting :) | 14:51 |
| rafaelweingartner | thank you all for participating. Have a nice week! | 14:51 |
| rafaelweingartner | #endmeeting | 14:52 |
| opendevmeet | Meeting ended Mon Jan 5 14:52:04 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:52 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cloudkitty/2026/cloudkitty.2026-01-05-13.59.html | 14:52 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cloudkitty/2026/cloudkitty.2026-01-05-13.59.txt | 14:52 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cloudkitty/2026/cloudkitty.2026-01-05-13.59.log.html | 14:52 |
| opendevreview | Merged openstack/cloudkitty master: Use project_id as part of the stream instead of part of the search https://review.opendev.org/c/openstack/cloudkitty/+/965897 | 15:33 |
| opendevreview | Takashi Kajinami proposed openstack/cloudkitty master: Remove old hacking check for oslo namespace package https://review.opendev.org/c/openstack/cloudkitty/+/972220 | 16:22 |
| opendevreview | Takashi Kajinami proposed openstack/cloudkitty master: Drop redundant hacking check for xrange https://review.opendev.org/c/openstack/cloudkitty/+/972221 | 16:22 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!