14:00:05 <rafaelweingartner> #startmeeting cloudkitty
14:00:05 <opendevmeet> Meeting started Mon Apr  3 14:00:05 2023 UTC and is due to finish in 60 minutes.  The chair is rafaelweingartner. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:00:05 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:00:05 <opendevmeet> The meeting name has been set to 'cloudkitty'
14:00:22 <rafaelweingartner> Roll count
14:00:24 <rafaelweingartner> \O
14:03:41 <priteau> o/
14:04:36 <priteau> There was a message from Mariusz sent on another channel
14:04:38 <priteau> "I may not be able to join our meeting today."
14:04:40 <priteau> "I've approved as many patches as I can - the rest I'll have to try out in my test environment this week."
14:04:49 <rafaelweingartner> ah ok
14:05:00 <rafaelweingartner> I guess we can start then?
14:05:05 <priteau> Yes.
14:05:10 <rafaelweingartner> #topic vPTG meeting
14:05:27 <rafaelweingartner> I just had our vPTG meeting last week, on Friday. Thank you guys for participating.
14:05:58 <rafaelweingartner> I will be writing up an e-mail to summarize our discussions and the direction we will be working under B release of OpenStack this week.
14:06:55 <priteau> Thank you for leading the meeting.
14:09:19 <rafaelweingartner> Welcome! It has been a pleasure to be working with you guys for all of these years!
14:09:32 <rafaelweingartner> So, moving on to the target reviews
14:09:34 <rafaelweingartner> #topic Target reviews
14:09:52 <rafaelweingartner> #link https://review.opendev.org/c/openstack/cloudkitty/+/861786, this patch was missing Gabbi tests, which I created.
14:09:56 <rafaelweingartner> Can you guys reconsider it?
14:10:14 <priteau> I was reviewing patches before the meeting but didn't reach this one yet
14:10:27 <rafaelweingartner> ah ok
14:10:41 <priteau> I will check this week, we need Mariusz's vote too
14:10:49 <rafaelweingartner> yes
14:10:51 <rafaelweingartner> ok
14:11:31 <rafaelweingartner> Moiving on, we have #link https://review.opendev.org/c/openstack/cloudkitty/+/861908, and #link https://review.opendev.org/c/openstack/cloudkitty/+/861806. These two patches will require some work, one requires some additions to the docs. And the other requires some tempest tests.
14:13:48 <rafaelweingartner> Then, we have #link https://review.opendev.org/c/openstack/cloudkitty/+/865435, which seems to be ready for merging. What do you think?
14:15:44 <priteau> I thought we deprecated SQL storage?
14:19:40 <priteau> We can still merge it of course.
14:19:40 <priteau> Approved
14:24:16 <rafaelweingartner> hmm
14:24:21 <rafaelweingartner> it seems that I got disconnected
14:24:25 <rafaelweingartner> are you guys still there?
14:24:46 <priteau> I am here.
14:25:56 <rafaelweingartner> so, we were discussing the V1 API
14:26:06 <rafaelweingartner> I asked if there was a deprecation/removal note in the docs
14:26:13 <rafaelweingartner> then, it seems that I got disconnected
14:27:19 <rafaelweingartner> Anyways, moving on, we have the patch #link: https://review.opendev.org/c/openstack/cloudkitty/+/862824, which also seem to be ready for merging.
14:27:53 <rafaelweingartner> It is related to the field vs. tag situation in Influx, but we will address that in a different patch, which will affect all attributes that are persisted today as fields in InfluxDB
14:31:35 <rafaelweingartner> We also have the patch #link: https://review.opendev.org/c/openstack/cloudkitty/+/877525, which is already reviewed by priteau. Therefore, we only need Mariusz blessing there to proceed with merging.
14:32:13 <rafaelweingartner> And finally, but not least, we have the patch #link https://review.opendev.org/c/openstack/cloudkitty/+/877537, which is the final procees of the `state` field deprecation/removal.
14:32:30 <rafaelweingartner> That is all of the patch list I had here. Do you guys have something else to add?
14:35:20 <opendevreview> Rafael Weingartner proposed openstack/cloudkitty master: Add description option to a rating metric definition  https://review.opendev.org/c/openstack/cloudkitty/+/862824
14:35:43 <priteau> Did we decide what to do about "rating types"?
14:35:54 <priteau> Are we putting it in the docs?
14:36:30 <opendevreview> Rafael Weingartner proposed openstack/cloudkitty master: Remove `state` field from API  https://review.opendev.org/c/openstack/cloudkitty/+/877537
14:36:45 <rafaelweingartner> yes, I just talked with Pedro
14:36:54 <rafaelweingartner> we will be writing up a documentation to explain these concepts
14:37:04 <rafaelweingartner> and then, in the review, we can discuss the better/proper term to use
14:40:52 <priteau> Thank you
14:42:14 <rafaelweingartner> Welcome! Do you guys have anything ot add?
14:42:23 <rafaelweingartner> Otherwise, we might be able to close a bit earlier today
14:49:45 <priteau> Nothing else.
14:49:56 <rafaelweingartner> Then, thank you guys for participating. Have a nice week.
14:50:01 <rafaelweingartner> #endmeeting