| opendevreview | Pierre Riteau proposed openstack/cloudkitty master: Introduce start and end dates on rating rules https://review.opendev.org/c/openstack/cloudkitty/+/876643 | 13:55 |
|---|---|---|
| rafaelweingartner | #startmeeting cloudkitty | 14:00 |
| opendevmeet | Meeting started Mon Mar 4 14:00:25 2024 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 | \0 | 14:00 |
| mattcrees | Hello o/ | 14:00 |
| priteau | o/ | 14:01 |
| rafaelweingartner | hello mkarpiarz! | 14:03 |
| mkarpiarz | Hi! | 14:03 |
| rafaelweingartner | Nice to see you all here! | 14:03 |
| rafaelweingartner | To start, we have the following | 14:04 |
| rafaelweingartner | #topic Cloudkitty vPTG meeting (April 8-12, 2024) | 14:04 |
| rafaelweingartner | We signed up, but we sill need to register for a time slot | 14:04 |
| rafaelweingartner | We will select the same day when our meetings happen (Monday, April 8th), and the same timeslot as well. That is fine for everybody, I guess, right? | 14:04 |
| priteau | It is fine me so far. | 14:05 |
| priteau | The booking system is now available. | 14:05 |
| mkarpiarz | Yes, this is fine for me as well. | 14:05 |
| mattcrees | Right now it looks clear for me too | 14:06 |
| rafaelweingartner | ok, thanks! | 14:06 |
| rafaelweingartner | I will register the time slot then | 14:06 |
| priteau | Are we using the default Etherpad link? | 14:08 |
| rafaelweingartner | yes | 14:08 |
| rafaelweingartner | that is what we normally do | 14:09 |
| priteau | https://ptg.opendev.org/etherpads.html now links to Meetpad by default, but you can access the Etherpad by changing the URL: meetpad -> etherpad | 14:09 |
| priteau | #link https://etherpad.opendev.org/p/apr2024-ptg-cloudkitty | 14:09 |
| rafaelweingartner | thanks | 14:09 |
| priteau | Last Etherpad for reference: https://etherpad.opendev.org/p/oct2023-ptg-cloudkitty | 14:10 |
| rafaelweingartner | Moving on, we have the following urgent topic | 14:11 |
| rafaelweingartner | #topic SQLAlchemy 2.x / oslo.db 15.0.0 compatibility | 14:11 |
| rafaelweingartner | I guess priteau already prepared the patch | 14:11 |
| priteau | Actually, mattcrees did, so kudos to him. | 14:12 |
| rafaelweingartner | awesome! | 14:12 |
| rafaelweingartner | thanks | 14:12 |
| mattcrees | https://review.opendev.org/c/openstack/cloudkitty/+/910336 pretty confident in this patch, it passes unit tests and tempest testing in devstack | 14:12 |
| rafaelweingartner | awesome, thank you | 14:12 |
| rafaelweingartner | I will review after the meeting | 14:12 |
| priteau | It looks like a big patch in Gerrit, but if you review it with `git show/diff -w` (ignoring whitespace) it looks much smaller. | 14:13 |
| priteau | Patch stat: 14 files changed, 604 insertions(+), 659 deletions(-) | 14:13 |
| priteau | Without whitespace: 14 files changed, 111 insertions(+), 166 deletions(-) | 14:13 |
| priteau | That's because a lot of existing code is indented to move under a `with` statement | 14:14 |
| mattcrees | Almost all the changes are identical too, it is essentially replacing a shared session with individual session creations. | 14:14 |
| mkarpiarz | Sounds good! | 14:14 |
| priteau | We won't be able to merge anything else until this is one is merged, since without it the gate is broken. | 14:15 |
| priteau | That is due to oslo.db 15.0.0 being added to upper constraints late last month. | 14:15 |
| rafaelweingartner | I see | 14:16 |
| rafaelweingartner | I already went over the patch, and it seems ok | 14:16 |
| mkarpiarz | It looks good to me as well, so I'll merge it. | 14:17 |
| rafaelweingartner | ok | 14:17 |
| rafaelweingartner | thanks | 14:17 |
| rafaelweingartner | We then have the following topic | 14:18 |
| rafaelweingartner | #topic Deadline for cycle highlights on March 8, 2024 | 14:18 |
| rafaelweingartner | We need to generate the highlights, right? | 14:18 |
| rafaelweingartner | What do you guys consider as the major highlights of this cycle? | 14:18 |
| priteau | Add groupby options by different timeframes | 14:19 |
| rafaelweingartner | Add support to InfluxDB v2 as storage backend | 14:19 |
| priteau | Add OpenSearch as a v2 storage backend | 14:19 |
| priteau | (even though this one was backported already) | 14:20 |
| priteau | Add description option to a rating metric definition | 14:20 |
| rafaelweingartner | Removal of Monasca fetcher and collector? | 14:20 |
| priteau | Yes, Monasca definitely | 14:20 |
| rafaelweingartner | I will add the highlights then | 14:20 |
| priteau | I think that's all for now, the rest is mostly bug fixes / internal improvements. | 14:20 |
| rafaelweingartner | thank you guys for an awesome cycle. | 14:20 |
| priteau | Potentially the start/end dates if we manage to merge it | 14:20 |
| rafaelweingartner | yes | 14:21 |
| rafaelweingartner | Movin on, we have SLURP | 14:21 |
| rafaelweingartner | #topic SLURP support | 14:21 |
| rafaelweingartner | tha support is being added via https://review.opendev.org/c/openstack/cloudkitty/+/908749 | 14:21 |
| rafaelweingartner | #link https://review.opendev.org/c/openstack/cloudkitty/+/908749 | 14:22 |
| priteau | It's really SLURP testing, right? | 14:23 |
| rafaelweingartner | yes | 14:23 |
| priteau | Actually, is this just for regular upgrade testing for now? | 14:24 |
| priteau | I am looking at the code and I don't see anything SLURP specific | 14:25 |
| mkarpiarz | I think we should merge this change since there is no risk of it breaking anything. :) | 14:25 |
| mkarpiarz | I believe this is prerequisites for Grenade to work. | 14:25 |
| mkarpiarz | And SLURP will be the next step. | 14:26 |
| priteau | Agreed. I just left some minor comments but happy to merge once fixed. | 14:26 |
| mkarpiarz | Correct me if I'm wrong! | 14:26 |
| rafaelweingartner | thanks! | 14:26 |
| rafaelweingartner | Then, we have the reviews | 14:27 |
| rafaelweingartner | #topic Target reviews | 14:27 |
| rafaelweingartner | #link https://review.opendev.org/c/openstack/cloudkitty/+/876643 | 14:27 |
| rafaelweingartner | I guess it was only missing priteau reviews | 14:27 |
| priteau | I am in the process of reviewing. I made various small edits. | 14:28 |
| rafaelweingartner | thanks | 14:28 |
| priteau | I will try to finish today | 14:28 |
| priteau | Technically we are beyond feature freeze but nobody depends on us so it would be easy to get an exception | 14:29 |
| mkarpiarz | I agree | 14:31 |
| rafaelweingartner | agree | 14:31 |
| rafaelweingartner | that was all I had prepared for today. Do you guys have something else to add? | 14:32 |
| mkarpiarz | Not me. | 14:33 |
| priteau | I think that's all for me. | 14:33 |
| mattcrees | Nothing else from me. | 14:33 |
| mkarpiarz | Sorry guys, I have to go now. | 14:45 |
| rafaelweingartner | Thank you guys for participating. Have a nice week. | 14:49 |
| rafaelweingartner | #endmeeting | 14:49 |
| opendevmeet | Meeting ended Mon Mar 4 14:49:54 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:49 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cloudkitty/2024/cloudkitty.2024-03-04-14.00.html | 14:49 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cloudkitty/2024/cloudkitty.2024-03-04-14.00.txt | 14:49 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cloudkitty/2024/cloudkitty.2024-03-04-14.00.log.html | 14:49 |
| opendevreview | Merged openstack/cloudkitty master: Replace usage of LegacyEngineFacade https://review.opendev.org/c/openstack/cloudkitty/+/910336 | 15:02 |
| opendevreview | Pedro Henrique Pereira Martins proposed openstack/cloudkitty master: Introduce start and end dates on rating rules https://review.opendev.org/c/openstack/cloudkitty/+/876643 | 15:39 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!