| opendevreview | OpenStack Release Bot proposed openstack/cloudkitty-dashboard master: Update master for stable/2025.2 https://review.opendev.org/c/openstack/cloudkitty-dashboard/+/961189 | 10:00 |
|---|---|---|
| opendevreview | OpenStack Release Bot proposed openstack/cloudkitty master: Update master for stable/2025.2 https://review.opendev.org/c/openstack/cloudkitty/+/961198 | 10:07 |
| opendevreview | Juan Larriba proposed openstack/cloudkitty master: Support TLS connections to Loki storage https://review.opendev.org/c/openstack/cloudkitty/+/961209 | 10:42 |
| opendevreview | Juan Larriba proposed openstack/cloudkitty master: Support TLS connections to Loki storage https://review.opendev.org/c/openstack/cloudkitty/+/961209 | 10:43 |
| opendevreview | Merged openstack/cloudkitty-dashboard master: Update master for stable/2025.2 https://review.opendev.org/c/openstack/cloudkitty-dashboard/+/961189 | 12:04 |
| opendevreview | Merged openstack/cloudkitty master: Update master for stable/2025.2 https://review.opendev.org/c/openstack/cloudkitty/+/961198 | 12:08 |
| opendevreview | Merged openstack/python-cloudkittyclient master: Add support to rating rules with start and end https://review.opendev.org/c/openstack/python-cloudkittyclient/+/958175 | 13:48 |
| rafaelweingartner | #startmeeting cloudkitty | 14:00 |
| opendevmeet | Meeting started Mon Sep 15 14:00:19 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 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| opendevmeet | The meeting name has been set to 'cloudkitty' | 14:00 |
| rafaelweingartner | Hello guys! | 14:00 |
| rafaelweingartner | Roll count | 14:00 |
| rafaelweingartner | \o | 14:00 |
| priteau | o/ | 14:03 |
| rafaelweingartner | Nice to see you Priteau! | 14:04 |
| priteau | Sorry, I have had conflicts with another meeting for many weeks now | 14:04 |
| priteau | It finished early today | 14:05 |
| rafaelweingartner | no worries :) | 14:05 |
| rafaelweingartner | glad you are here now | 14:05 |
| rafaelweingartner | so let's start | 14:06 |
| rafaelweingartner | #topic vPTG | 14:06 |
| opendevreview | Merged openstack/cloudkitty master: Fix FileBackend in Python 3 https://review.opendev.org/c/openstack/cloudkitty/+/957435 | 14:06 |
| rafaelweingartner | I have registered the team, and we just need to book the room. However, I think that the room registration is not open yet. | 14:06 |
| jlarriba | o/, sorry im late :) | 14:06 |
| rafaelweingartner | no worries, we just started =) | 14:06 |
| rafaelweingartner | as soon as the room registration is open, I will do so in our usual vPTG time slot | 14:07 |
| rafaelweingartner | which is basically, our usual meeting time | 14:07 |
| rafaelweingartner | is that ok for you guys? | 14:07 |
| priteau | October 27? | 14:07 |
| rafaelweingartner | yes | 14:09 |
| priteau | I might be on holiday, unsure. But if I am working I will make myself available. | 14:09 |
| opendevreview | Merged openstack/cloudkitty master: Remove unused [state] options https://review.opendev.org/c/openstack/cloudkitty/+/957436 | 14:09 |
| rafaelweingartner | ok | 14:09 |
| opendevreview | Merged openstack/cloudkitty master: Drop unused BaseIOBackend https://review.opendev.org/c/openstack/cloudkitty/+/957437 | 14:09 |
| jlarriba | i am eager to see you guys again there | 14:09 |
| rafaelweingartner | me too | 14:09 |
| rafaelweingartner | so, moving on | 14:11 |
| rafaelweingartner | #topic Target reviews | 14:11 |
| rafaelweingartner | #link https://review.opendev.org/c/openstack/cloudkitty/+/960374 | 14:11 |
| rafaelweingartner | is an interesting one, I just asked the author to remove one extra space in the code | 14:12 |
| rafaelweingartner | other reviews would be welcome in this one, and in the others that we will go through today | 14:12 |
| rafaelweingartner | #link https://review.opendev.org/c/openstack/cloudkitty/+/950542, this one Pedro and I reviewed. IF you are ok with it, priteau: I guess we can merge it | 14:15 |
| priteau | Is this a bug fix that we would backport? If yes, can you add a release note? | 14:17 |
| rafaelweingartner | hmm | 14:17 |
| rafaelweingartner | I think it would be an interesting patch to backport | 14:17 |
| rafaelweingartner | I am not the owner of the patch, but I guess we can ask the author to create a reno release notes | 14:17 |
| priteau | Also the Git author should be updated from "root <root@cloudkitty.dev.rc.nectar.org.au>" ;-) | 14:18 |
| priteau | I will respond | 14:18 |
| rafaelweingartner | I did not get it | 14:18 |
| rafaelweingartner | now, I see what you mean | 14:20 |
| rafaelweingartner | moving on, we have #link https://review.opendev.org/c/openstack/cloudkitty/+/960032 | 14:20 |
| rafaelweingartner | it is changing a default for the cloudkitty threads | 14:20 |
| rafaelweingartner | It changes a default configuration, I do not know what should be our policies. However, the change seem to make sense | 14:21 |
| rafaelweingartner | We never use those configs as default | 14:21 |
| priteau | That's a good one | 14:22 |
| rafaelweingartner | cool | 14:22 |
| rafaelweingartner | can you do the honours then? | 14:23 |
| priteau | We are sometimes working with nodes that have 192 cores | 14:23 |
| rafaelweingartner | exactly | 14:23 |
| priteau | That would be 184320 threads? | 14:23 |
| rafaelweingartner | a lot:) | 14:24 |
| priteau | Approved | 14:24 |
| rafaelweingartner | thanks! | 14:25 |
| rafaelweingartner | moving on #link https://review.opendev.org/c/openstack/cloudkitty/+/950868, we have this one | 14:25 |
| rafaelweingartner | we are fine with it | 14:25 |
| rafaelweingartner | therefore, we are just waiting your feedback | 14:25 |
| priteau | I will review in more depth soon. It's going to be for the G release anyway | 14:26 |
| rafaelweingartner | ok | 14:26 |
| priteau | An important one is https://review.opendev.org/c/openstack/cloudkitty/+/961242 | 14:26 |
| priteau | Cherry pick for 2025.2 | 14:26 |
| priteau | I thought it would be included in 2025.2 directly, I didn't realise you had not updated the release SHA | 14:27 |
| rafaelweingartner | I merged some of those | 14:27 |
| rafaelweingartner | I also did not paid attention to this one | 14:27 |
| priteau | We should have updated https://review.opendev.org/c/openstack/releases/+/960069 as soon as it merged. | 14:27 |
| rafaelweingartner | ok | 14:28 |
| priteau | The cleanup patches merged to master recently should probably have waited until after feature freeze ended too | 14:28 |
| rafaelweingartner | did not it finish? | 14:29 |
| priteau | Sorry, I see they just merged this afternoon | 14:29 |
| priteau | Git history confused me | 14:29 |
| rafaelweingartner | I mean, the feature freeze was already over | 14:29 |
| rafaelweingartner | I merged today only | 14:29 |
| priteau | Yeah, as I said Git history confused me just now, sorry. | 14:29 |
| rafaelweingartner | :) | 14:30 |
| priteau | I know some projects don't start merging to master right after RC1 because they want to keep the master branch stable in case they need to make some important fixes + backports | 14:30 |
| rafaelweingartner | I see | 14:30 |
| rafaelweingartner | but in our case is more or less controlled, right | 14:30 |
| rafaelweingartner | Moving one, we also have #link https://review.opendev.org/c/openstack/cloudkitty/+/953827 | 14:33 |
| rafaelweingartner | It is a patch proposed by me, which is an interesting feature to be used to enable people to configure methods/expressions to skip persisting some of the processed rating points | 14:33 |
| rafaelweingartner | moving one, we have this other small patch, which is interesting as well to have #link https://review.opendev.org/c/openstack/cloudkitty/+/953762 | 14:40 |
| rafaelweingartner | on a similar line, we have #link https://review.opendev.org/c/openstack/cloudkitty/+/958374 and #link https://review.opendev.org/c/openstack/cloudkitty/+/954880 | 14:40 |
| rafaelweingartner | as we are reaching our final minutes. Instead of going over the other patches, I would like to open for the open floor, for people to discuss/comment things that they find important. | 14:45 |
| jlarriba | i dont know if you noticed, but we submitted a patch to add the Loki testing to the CI: https://review.opendev.org/c/openstack/cloudkitty/+/960455 | 14:48 |
| jlarriba | which might be very useful to be able to ensure that everything works as expected | 14:48 |
| priteau | rafaelweingartner: Should we make an RC2 to include the security patch? | 14:50 |
| rafaelweingartner | I would say so | 15:00 |
| rafaelweingartner | jlarriba: yes, I did notice it | 15:00 |
| jlarriba | ack | 15:01 |
| priteau | rafaelweingartner: will you propose a release patch? | 15:02 |
| rafaelweingartner | sure I can | 15:02 |
| rafaelweingartner | I took some note here to do | 15:02 |
| rafaelweingartner | I will just wait for the patches to get merged | 15:02 |
| rafaelweingartner | I am not sure if they were already all merged | 15:03 |
| priteau | Someone is asking if we can backport the thread count patch | 15:03 |
| priteau | https://review.opendev.org/c/openstack/cloudkitty/+/960032 | 15:03 |
| rafaelweingartner | maybe | 15:03 |
| priteau | Maybe we backport it but don't try to include it in the rc2? | 15:03 |
| rafaelweingartner | ok | 15:04 |
| priteau | It could have bad effects that we haven't noticed yet | 15:04 |
| rafaelweingartner | exactly | 15:04 |
| rafaelweingartner | and it is not that important | 15:04 |
| priteau | For the security fix, you need to wait until it has merged | 15:04 |
| rafaelweingartner | someone can always override the defautls | 15:04 |
| priteau | true | 15:05 |
| rafaelweingartner | and that is basically it =) | 15:05 |
| rafaelweingartner | great job guys! | 15:05 |
| rafaelweingartner | thank you all for participating. Have a nice week! | 15:05 |
| rafaelweingartner | #endmeeting | 15:07 |
| opendevmeet | Meeting ended Mon Sep 15 15:07:09 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:07 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cloudkitty/2025/cloudkitty.2025-09-15-14.00.html | 15:07 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cloudkitty/2025/cloudkitty.2025-09-15-14.00.txt | 15:07 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cloudkitty/2025/cloudkitty.2025-09-15-14.00.log.html | 15:07 |
| opendevreview | Merged openstack/cloudkitty master: Lower down max_threads https://review.opendev.org/c/openstack/cloudkitty/+/960032 | 15:47 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!