| *** mrunge_ is now known as mrunge | 06:53 | |
| niux | Hey o/! Following the fix I submitted: https://review.opendev.org/c/openstack/cloudkitty/+/962653 | 12:03 |
|---|---|---|
| niux | 1) How can I add a test in Tempest? Is there any documentation or good practices I should follow? Should I open a Story in the cloudkitty-tempest-plugin project? | 12:03 |
| niux | 2) Would it be fine to merge the patch even if the test isn’t implemented yet? | 12:03 |
| rafaelweingartner | #startmeeting cloudkitty | 14:02 |
| opendevmeet | Meeting started Mon Oct 13 14:02:24 2025 UTC and is due to finish in 60 minutes. The chair is rafaelweingartner. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:02 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:02 |
| opendevmeet | The meeting name has been set to 'cloudkitty' | 14:02 |
| rafaelweingartner | Hello guys! | 14:02 |
| rafaelweingartner | Roll count | 14:02 |
| rafaelweingartner | \o | 14:02 |
| priteau | o/ | 14:05 |
| rafaelweingartner | nice to see you here | 14:07 |
| rafaelweingartner | Let's start | 14:10 |
| rafaelweingartner | #topic vPTG | 14:10 |
| rafaelweingartner | everything seems to be ready, room is booked, date is scheduled | 14:10 |
| rafaelweingartner | I also already create an initial scratch for the etherpad | 14:10 |
| rafaelweingartner | I will then amend with some more information about releases and information | 14:10 |
| priteau | Monday at 14UTC? | 14:12 |
| priteau | *Monday October 27* | 14:13 |
| rafaelweingartner | yes | 14:13 |
| rafaelweingartner | that is how we normally do it, in our usual meeting time, right? | 14:13 |
| priteau | Yes | 14:13 |
| rafaelweingartner | is that timeslot fine? | 14:15 |
| rafaelweingartner | or do we need some change? | 14:15 |
| rafaelweingartner | as nobody mentioned anything before, I just maintained the usual that we use | 14:15 |
| priteau | It's OK for me for 30 minutes. I might need to leave after that but Matt should be around for one hour. | 14:16 |
| rafaelweingartner | I see | 14:16 |
| rafaelweingartner | I guess that would be fine | 14:16 |
| rafaelweingartner | let's move on. If needed we can adjust the day and timeslot | 14:20 |
| rafaelweingartner | let me know | 14:20 |
| rafaelweingartner | #Target reviews | 14:21 |
| rafaelweingartner | #link https://review.opendev.org/c/openstack/cloudkitty/+/963621, this one is an important one I guess | 14:21 |
| rafaelweingartner | to fix our pipelines | 14:21 |
| rafaelweingartner | can you guys review it | 14:21 |
| rafaelweingartner | and approve if you think that one is fine to be merged | 14:22 |
| priteau | Done | 14:23 |
| rafaelweingartner | awesome! | 14:23 |
| rafaelweingartner | thanks | 14:23 |
| rafaelweingartner | if everything is fine there, I guess we could merge it | 14:26 |
| priteau | Gerrit issue, my vote didn't post | 14:27 |
| priteau | Merging now | 14:27 |
| rafaelweingartner | thanks! | 14:28 |
| rafaelweingartner | then, we have this one #link https://review.opendev.org/c/openstack/cloudkitty/+/960374, which I also think we can merge | 14:28 |
| rafaelweingartner | I executed a recheck in this one #link https://review.opendev.org/c/openstack/cloudkitty/+/950542 | 14:31 |
| priteau | Just a note on recheck, the community has been asked to avoid "bare rechecks", i.e. rechecks without a reason | 14:34 |
| rafaelweingartner | yes, but I just did it because the patch that is going to fix is going to be merged | 14:37 |
| rafaelweingartner | you, now, we always should add a reson? | 14:38 |
| rafaelweingartner | I mean, a written one? | 14:38 |
| mrunge | that is my understanding | 14:38 |
| mrunge | like recheck opendev.org 404 | 14:38 |
| mrunge | (just making something up) | 14:38 |
| mrunge | it would help to group issues by reason | 14:39 |
| priteau | I don't always do a thorough investigation of the issue, but at least which job failed, and ideally why | 14:39 |
| rafaelweingartner | I see | 14:39 |
| rafaelweingartner | I will do that next time then | 14:39 |
| mrunge | The request not to do bare rechecks comes up about once every year | 14:40 |
| rafaelweingartner | :) | 14:40 |
| rafaelweingartner | yes | 14:40 |
| rafaelweingartner | I have see | 14:40 |
| rafaelweingartner | seen* | 14:41 |
| mrunge | Any thoughts on https://review.opendev.org/c/openstack/cloudkitty/+/961609 ? It helps spreading out requests to the loki storage, where the query window is smaller than 30 days, i.e. you can not search for data for a whole month | 14:44 |
| mrunge | some deployments try to limit the search window to a couple of days or a week, which is of course problematic for cloudkitty | 14:45 |
| priteau | Left a comment to address | 14:47 |
| mrunge | thank you! | 14:48 |
| mrunge | Will get it addressed asap | 14:48 |
| rafaelweingartner | thanks! | 14:48 |
| rafaelweingartner | Then, #link https://review.opendev.org/c/openstack/cloudkitty/+/953827, it would be interesting to get some feedback here as well | 14:50 |
| priteau | Still not too keen on the eval(). I have to think about it. | 14:52 |
| rafaelweingartner | ok | 14:52 |
| rafaelweingartner | I wait for you to mature | 14:53 |
| rafaelweingartner | but this is somethign common that we use in other systems | 14:53 |
| rafaelweingartner | remember, it is a config on the server side done by the operator | 14:53 |
| priteau | Yes, I understand. | 14:55 |
| rafaelweingartner | Do you guys have something else to add to the meeting? | 14:59 |
| rafaelweingartner | we still have 10min, and the other patches I guess we can discuss in our next meeting | 14:59 |
| priteau | I have to leave for another meeting | 15:00 |
| rafaelweingartner | ok | 15:03 |
| rafaelweingartner | thanks! | 15:03 |
| rafaelweingartner | thank you all for participating. Have a nice week! | 15:04 |
| rafaelweingartner | #endmeeting | 15:04 |
| rafaelweingartner | #endmeeting | 15:04 |
| rafaelweingartner | #endmeeting | 15:04 |
| opendevmeet | Meeting ended Mon Oct 13 15:04:18 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:04 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cloudkitty/2025/cloudkitty.2025-10-13-14.02.html | 15:04 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cloudkitty/2025/cloudkitty.2025-10-13-14.02.txt | 15:04 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cloudkitty/2025/cloudkitty.2025-10-13-14.02.log.html | 15:04 |
| opendevreview | Merged openstack/cloudkitty master: Add pyproject.toml https://review.opendev.org/c/openstack/cloudkitty/+/963621 | 15:26 |
| opendevreview | Merged openstack/cloudkitty master: Support TLS connections to Loki storage https://review.opendev.org/c/openstack/cloudkitty/+/961209 | 15:26 |
| opendevreview | Merged openstack/cloudkitty master: Adds CI Testing for Prometheus https://review.opendev.org/c/openstack/cloudkitty/+/960374 | 15:38 |
| opendevreview | Matt Crees proposed openstack/cloudkitty master: Fix the Flux query issue when no groupby used https://review.opendev.org/c/openstack/cloudkitty/+/950542 | 16:57 |
| opendevreview | Juan Larriba proposed openstack/cloudkitty master: Shard the requests to Loki https://review.opendev.org/c/openstack/cloudkitty/+/961609 | 20:46 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!