| *** mrunge_ is now known as mrunge | 01:05 | |
| opendevreview | Jaromír Wysoglad proposed openstack/cloudkitty master: Add Aetos collector with Keystone authentication https://review.opendev.org/c/openstack/cloudkitty/+/978198 | 07:57 |
|---|---|---|
| opendevreview | Merged openstack/cloudkitty master: Fix CloudKitty log formatting https://review.opendev.org/c/openstack/cloudkitty/+/977234 | 11:10 |
| opendevreview | Rafael Weingartner proposed openstack/cloudkitty master: Address CloudKitty memory leak during collection processing. https://review.opendev.org/c/openstack/cloudkitty/+/975224 | 12:48 |
| rafaelweingartner | #startmeeting cloudkitty | 14:00 |
| opendevmeet | Meeting started Mon Mar 2 14:00:08 2026 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 |
| jwysogla | o/ | 14:00 |
| jlarriba | o/ | 14:00 |
| opendevreview | Jaromír Wysoglad proposed openstack/cloudkitty master: Add Aetos collector with Keystone authentication https://review.opendev.org/c/openstack/cloudkitty/+/978198 | 14:01 |
| mrunge | o/ | 14:02 |
| rafaelweingartner | awesome to see you all here ! | 14:03 |
| rafaelweingartner | #topic vPTG | 14:03 |
| rafaelweingartner | I have already registered our team, and we just need to sign for a room | 14:04 |
| rafaelweingartner | I am not sure if the registration of the room is already open | 14:04 |
| mattcrees | o/ | 14:04 |
| rafaelweingartner | also, do not forget to register for the event at #link https://ptg.openinfra.org/ | 14:04 |
| jlarriba | i dont think so, i can only see last editions rooms for now | 14:04 |
| mrunge | I have not seen any note about registering rooms yet. But thank you for proposing rafael! | 14:04 |
| rafaelweingartner | thanks! | 14:05 |
| rafaelweingartner | moving on... | 14:06 |
| rafaelweingartner | #topic Target reviews | 14:06 |
| rafaelweingartner | #link https://review.opendev.org/c/openstack/cloudkitty/+/975224, is a patch that we shoudl try to get in this cycle | 14:07 |
| rafaelweingartner | what do you guys think? | 14:07 |
| rafaelweingartner | should we merge it? | 14:07 |
| rafaelweingartner | Or, do you guys think that it needs more time? | 14:07 |
| mrunge | yes please, definitely, Let's get it merged | 14:07 |
| mrunge | it removes eventlet from cloudkitty | 14:07 |
| mattcrees | This does look worth getting in. I'll give it a look now | 14:08 |
| jwysogla | +1 from me to merge it this cycle | 14:08 |
| mrunge | eventlet does not work with python 3.12, or if it does: you are lucky | 14:08 |
| rafaelweingartner | it does work... | 14:08 |
| rafaelweingartner | but we have seen memory leak issues | 14:08 |
| rafaelweingartner | mattcrees: would you mind doing the honours and merging it if you think that it is fine? | 14:11 |
| mattcrees | Sure thing :) | 14:12 |
| rafaelweingartner | thanks!! =) | 14:14 |
| rafaelweingartner | Moving on then... | 14:16 |
| rafaelweingartner | #link https://review.opendev.org/c/openstack/cloudkitty-dashboard/+/973486 is also, another patch that might make sense to be merged | 14:16 |
| jlarriba | yeah, i think it makes the horizon plugin more informative | 14:17 |
| rafaelweingartner | Moving on, we also have #link https://review.opendev.org/c/openstack/cloudkitty-dashboard/+/973215, which seems to be ready for merging as well. | 14:22 |
| rafaelweingartner | mattcrees: I will implement the new configuration later | 14:22 |
| mattcrees | Thanks | 14:22 |
| rafaelweingartner | thanks for the reviw | 14:22 |
| mattcrees | From the screenshots it looks good, but I'd like to test those dashboard enhancements before approving :) | 14:23 |
| rafaelweingartner | thanks Matt! | 14:24 |
| rafaelweingartner | #link https://review.opendev.org/c/openstack/cloudkitty-dashboard/+/958033, also seems to make sense to merge | 14:24 |
| mrunge | ...3215 is pretty straightforward to me | 14:24 |
| rafaelweingartner | I agree | 14:24 |
| mattcrees | I agree, it looks good. I've set that off to merge. | 14:24 |
| mrunge | https://review.opendev.org/c/openstack/cloudkitty-dashboard/+/973215 I meant, it already has a +2, and changes 2 lines of code | 14:24 |
| mrunge | https://review.opendev.org/c/openstack/cloudkitty-dashboard/+/958033 is out there for a while, I'd love to see it merged | 14:27 |
| rafaelweingartner | me too | 14:27 |
| mattcrees | I'm happy to merge 958033 too, will set it off now | 14:28 |
| mrunge | https://review.opendev.org/c/openstack/cloudkitty/+/977611 is also pretty straightforward and has a +2 | 14:29 |
| rafaelweingartner | I got kicked out | 14:29 |
| mrunge | I just linked https://review.opendev.org/c/openstack/cloudkitty/+/977611 , which is imho good to go a well | 14:30 |
| rafaelweingartner | I see | 14:31 |
| mrunge | that's probably all that you missed rafaelweingartner | 14:32 |
| rafaelweingartner | thanks! | 14:37 |
| rafaelweingartner | Moving on | 14:37 |
| rafaelweingartner | #topic New Core reviewers | 14:37 |
| rafaelweingartner_ | I got kicked out again :) | 14:39 |
| rafaelweingartner_ | did you guys received my last message? | 14:39 |
| mrunge | you got kicked out | 14:39 |
| mrunge | but wrote about new core reviewers | 14:40 |
| mrunge | before | 14:40 |
| rafaelweingartner_ | yes | 14:40 |
| mrunge | yes! I'd love to see more people helping out | 14:40 |
| jlarriba | you just opened the topic and said nothing else | 14:40 |
| rafaelweingartner_ | Juan and Jaromir have been consistenly contributing to cloudkitty for a while already, with both doing reviews and code contributions, like bug fixes and features. | 14:40 |
| mrunge | ++ | 14:41 |
| rafaelweingartner_ | I would like to propose them as CR | 14:41 |
| rafaelweingartner_ | What do you guys think? | 14:41 |
| opendevreview | Merged openstack/cloudkitty master: Drop redundant get_string_type https://review.opendev.org/c/openstack/cloudkitty/+/972290 | 14:41 |
| rafaelweingartner_ | mattcrees: what do you think? It has been a while that we do not add new CRs to the team. | 14:41 |
| opendevreview | Merged openstack/cloudkitty master: Remove unused Makefile for doc https://review.opendev.org/c/openstack/cloudkitty/+/977523 | 14:41 |
| mrunge | I would be very happy to see both of them as additional cores | 14:42 |
| mrunge | this is not meant as critics to existing cores at all | 14:42 |
| mattcrees | That's sounds like a great idea to me, Juan and Jaromir have both been very helpful already. I think they've been contributing long enough to prove consistency too | 14:42 |
| rafaelweingartner_ | awesome! | 14:43 |
| rafaelweingartner_ | do we have some announcement to make? | 14:43 |
| rafaelweingartner_ | I see some projects post in the mailing list, but I am not sure if that is mandatory | 14:43 |
| mattcrees | Will help to keep the pace of merging up too, when us existing cores get busy periods ;) | 14:43 |
| mrunge | there is no requirement | 14:43 |
| opendevreview | Merged openstack/cloudkitty-dashboard master: Fix exception handling in hashmap IndexView https://review.opendev.org/c/openstack/cloudkitty-dashboard/+/973215 | 14:43 |
| opendevreview | Merged openstack/cloudkitty-dashboard master: Upgrade admin/rating panel to v2 of the CloudKitty API https://review.opendev.org/c/openstack/cloudkitty-dashboard/+/958033 | 14:43 |
| jwysogla | Depends if you want people outside of this meeting to know that there are new cores. | 14:44 |
| mattcrees | Yeah it's not required, but it's nice to do. | 14:44 |
| jwysogla | But as said, it's not mandatory | 14:44 |
| mrunge | it is nice to have, but no hard requirement | 14:44 |
| rafaelweingartner_ | ok, I will write something today then | 14:44 |
| mrunge | thank you! | 14:44 |
| rafaelweingartner_ | and add them as CRs into cloudkitty later | 14:44 |
| rafaelweingartner_ | thanks! | 14:44 |
| rafaelweingartner_ | and that is all from my side | 14:44 |
| rafaelweingartner_ | do you guys have something else to add? | 14:44 |
| mrunge | nope, thank you very much rafaelweingartner_ | 14:45 |
| jwysogla | Nothing from me. | 14:45 |
| mattcrees | Nothing from me | 14:45 |
| rafaelweingartner_ | ok | 14:45 |
| rafaelweingartner_ | thank you all :) | 14:45 |
| rafaelweingartner_ | Thank you for participating. Have a nice week. | 14:46 |
| rafaelweingartner_ | #endmeeting | 14:46 |
| jlarriba | thank you very much! | 14:46 |
| rafaelweingartner_ | #endmeeting | 14:46 |
| mattcrees | Thanks for hosting Rafael :) | 14:46 |
| rafaelweingartner_ | funny, I cannot close the meeting :) | 14:46 |
| rafaelweingartner_ | #startmeeting cloudkitty | 14:46 |
| opendevmeet | rafaelweingartner_: Error: Can't start another meeting, one is in progress. Use #endmeeting first. | 14:46 |
| rafaelweingartner_ | #endmeeting | 14:46 |
| rafaelweingartner_ | #endmeeting | 14:46 |
| mrunge | fun | 14:46 |
| jwysogla | might be because of your nick? | 14:46 |
| mrunge | #endmeeting | 14:46 |
| rafaelweingartner_ | ah | 14:46 |
| rafaelweingartner_ | yes | 14:46 |
| rafaelweingartner_ | my nick is different | 14:47 |
| *** rafaelweingartner_ is now known as NewName | 14:48 | |
| *** NewName is now known as rafaelweingartner | 14:49 | |
| rafaelweingartner | #endmeeting | 14:49 |
| opendevmeet | Meeting ended Mon Mar 2 14:49:06 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:49 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cloudkitty/2026/cloudkitty.2026-03-02-14.00.html | 14:49 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cloudkitty/2026/cloudkitty.2026-03-02-14.00.txt | 14:49 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cloudkitty/2026/cloudkitty.2026-03-02-14.00.log.html | 14:49 |
| rafaelweingartner | now it works! | 14:49 |
| mrunge | awesome | 14:49 |
| rafaelweingartner | see yah! | 14:49 |
| opendevreview | Merged openstack/cloudkitty master: Do not use f-string in logging message https://review.opendev.org/c/openstack/cloudkitty/+/977611 | 15:34 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!