| mrunge | Hi good afternoon, can we move on with the loki related changes https://review.opendev.org/c/openstack/cloudkitty/+/950868 and following? | 12:55 |
|---|---|---|
| rafaelweingartner | Hello mrunge, to me, it is fine | 13:02 |
| rafaelweingartner | let's see the others | 13:03 |
| rafaelweingartner | our meeting is in about 57min. | 13:03 |
| mrunge | rafaelweingartner: thank you, I saw your +2 | 13:03 |
| mrunge | (many of them) | 13:03 |
| rafaelweingartner | :) | 13:03 |
| mrunge | it looks like either the other cores don't care, or can not test? | 13:04 |
| mrunge | does cloudkitty need more cores? | 13:04 |
| priteau | We care, just limited bandwidth, that's all | 13:29 |
| mrunge | thank you! | 13:33 |
| mrunge | Can you please also look at the related changes (adding tls https://review.opendev.org/c/openstack/cloudkitty/+/961209/3 ) and following? | 13:34 |
| mrunge | another one https://review.opendev.org/c/openstack/cloudkitty/+/961609/2 | 13:35 |
| mrunge | and ultimately https://review.opendev.org/c/openstack/cloudkitty/+/960455 (actually add a CI test for the loki feature) | 13:35 |
| rafaelweingartner | #startmeeting cloudkitty | 14:00 |
| opendevmeet | Meeting started Mon Sep 29 14:00:16 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 | it is nice to see you alll here today :) | 14:00 |
| rafaelweingartner | Roll count | 14:00 |
| rafaelweingartner | \0 | 14:00 |
| jlarriba | o/ | 14:00 |
| rafaelweingartner | I guess we can start | 14:03 |
| rafaelweingartner | #topic vPTG | 14:03 |
| rafaelweingartner | I will schedule the room | 14:03 |
| rafaelweingartner | can I schedule the same time as our usual meeting? | 14:03 |
| jlarriba | thats good for me | 14:03 |
| mrunge | o/ | 14:05 |
| rafaelweingartner | moving on | 14:07 |
| rafaelweingartner | #topic Target reviews | 14:07 |
| rafaelweingartner | #link https://review.opendev.org/c/openstack/cloudkitty/+/960374, is failing because it depends on https://review.opendev.org/c/openstack/devstack-plugin-prometheus/+/959063. We will discuss it when its dependency is merged. | 14:08 |
| rafaelweingartner | Moving on #link https://review.opendev.org/c/openstack/cloudkitty/+/950542. It is an important patch to have merged. Can you guys also review it? | 14:09 |
| rafaelweingartner | Eager to see your reviews there =) | 14:14 |
| rafaelweingartner | the same on this one #link https://review.opendev.org/c/openstack/cloudkitty/+/953827, which is mine. | 14:14 |
| rafaelweingartner | moving on, the patch #link https://review.opendev.org/c/openstack/cloudkitty/+/953762 is interesting to be merged to remove the use of a deprecated method of SLQalchemy. | 14:19 |
| rafaelweingartner | #link https://review.opendev.org/c/openstack/cloudkitty/+/958374 is also cleaning Cloudkitty structure to remove deprecated/unused configurations/code | 14:20 |
| rafaelweingartner | moving on, this #link https://review.opendev.org/c/openstack/cloudkitty/+/954880, is a nice to have for CloudKitty. It would be nice to somebody else to review as well | 14:26 |
| rafaelweingartner | #link https://review.opendev.org/c/openstack/cloudkitty/+/958255, this patch needs fixing to be reviewed and discussed. | 14:27 |
| rafaelweingartner | #link https://review.opendev.org/c/openstack/python-cloudkittyclient/+/953369 this one needs a small amendment to be merged | 14:27 |
| mrunge | sorry, reading back here, was in meetings | 14:28 |
| rafaelweingartner | no worries :) | 14:29 |
| jlarriba | https://review.opendev.org/c/openstack/devstack-plugin-prometheus/+/959063 went under my radar. I added a comment there, I believe this is not needed and the dependant patch can work with Ceilometer metrics | 14:30 |
| jlarriba | since some time ago, Ceilometer has the capability to become a Prometheus exporter | 14:31 |
| mrunge | yes! There is no need to add another dependency, ceilometer has now an own prometheus exporter | 14:31 |
| mrunge | The loki change failed on the gate? | 14:33 |
| priteau | Responding on the openstack-exporter patch | 14:33 |
| priteau | mrunge: rechecked | 14:35 |
| mrunge | thx. I was trying to understand if there was a more generic issue with the patch now. | 14:36 |
| rafaelweingartner | thanks priteau! | 14:36 |
| mrunge | thank you for the comment the openstack-exporter patch. In general, I am all in for adding more test coverage | 14:38 |
| mrunge | I wonder about the delta between ceilometer and openstack-exporter, but that is a side-conversation and not related to the patch itself | 14:38 |
| rafaelweingartner | guys, I have to close early today due to some demands here. Can you guys ping me, if I am needed in some of the patches? | 14:40 |
| mrunge | +1 | 14:41 |
| rafaelweingartner | thank you all for participating. Have a nice week! | 14:42 |
| mrunge | Likewise | 14:42 |
| rafaelweingartner | #endmeeting | 14:42 |
| opendevmeet | Meeting ended Mon Sep 29 14:42:59 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:42 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cloudkitty/2025/cloudkitty.2025-09-29-14.00.html | 14:42 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cloudkitty/2025/cloudkitty.2025-09-29-14.00.txt | 14:42 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cloudkitty/2025/cloudkitty.2025-09-29-14.00.log.html | 14:42 |
| opendevreview | Merged openstack/cloudkitty master: tox: Remove basepython and ineffective ignore_basepython_conflict https://review.opendev.org/c/openstack/cloudkitty/+/958374 | 15:08 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!