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