14:00:42 <rafaelweingartner> #startmeeting cloudkitty 14:00:42 <opendevmeet> Meeting started Mon Apr 17 14:00:42 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:42 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 14:00:42 <opendevmeet> The meeting name has been set to 'cloudkitty' 14:00:46 <rafaelweingartner> Hello guys! 14:00:50 <rafaelweingartner> Roll count 14:01:04 <mkarpiarz> Hi! 14:02:17 <priteau> o/ 14:07:37 <priteau> rafaelweingartner: Let's start? 14:07:41 <rafaelweingartner> #topic vPTG meeting 14:07:48 <rafaelweingartner> Thank you all for participating in the vPTG meeting! We had an interesting session regarding the next steps we will be taken into CloudKitty, and I hope we can make it better every new release! 14:07:57 <rafaelweingartner> Again, thank you all for participating. 14:11:11 <rafaelweingartner> #topic Target reviews 14:11:16 <rafaelweingartner> #link https://review.opendev.org/c/openstack/cloudkitty/+/861786, this one seems ready for merging. What do you guys think? 14:12:39 <priteau> Can you remind me why you went with time-d/time-w, but also month/year 14:13:39 <rafaelweingartner> Not a specific reason 14:14:12 <rafaelweingartner> one is what we expose in the API, and the other is just the attribute name in the backend 14:14:19 <rafaelweingartner> would you like to change the pattern? 14:16:09 <priteau> Wouldn't it make more sense to use "daily", "weekly" or "day", "week" 14:16:19 <priteau> Also your commit message doesn't match your reno 14:16:36 <priteau> Commit message says "year", reno says "time-y" 14:16:40 <priteau> etc. 14:16:44 <priteau> Can you please rework? 14:16:51 <rafaelweingartner> Ok, thanks! 14:16:55 <rafaelweingartner> I will do that 14:18:00 <rafaelweingartner> Anyways., moving on 14:18:01 <rafaelweingartner> #link https://review.opendev.org/c/openstack/cloudkitty/+/862824 seems ready for merging. What do you guys think? 14:20:34 <mkarpiarz> I added my +2 back (to the latest patch). 14:21:46 <rafaelweingartner> thanks! 14:21:59 <priteau> Minor nit on reno, I will fix it myself 14:22:10 <rafaelweingartner> ok 14:22:13 <opendevreview> Pierre Riteau proposed openstack/cloudkitty master: Add description option to a rating metric definition https://review.opendev.org/c/openstack/cloudkitty/+/862824 14:23:23 <priteau> And kb wasn't fixed 14:24:04 <opendevreview> Pierre Riteau proposed openstack/cloudkitty master: Add description option to a rating metric definition https://review.opendev.org/c/openstack/cloudkitty/+/862824 14:24:43 <priteau> I have to be afk for a few minutes 14:24:54 <mkarpiarz> OK 14:25:41 <rafaelweingartner> ok 14:25:42 <rafaelweingartner> thanks! 14:26:07 <rafaelweingartner> #link https://review.opendev.org/c/openstack/cloudkitty/+/861908 and #link https://review.opendev.org/c/openstack/cloudkitty/+/861806 need some extra work from our side. Therefore, no need to review them again for now. 14:26:46 <priteau> Back 14:27:02 <rafaelweingartner> thanks for the patches there! 14:27:13 <rafaelweingartner> I thought we fixed the KB thing, but we left it lower case 14:27:17 <rafaelweingartner> thanks for fixing that 14:29:12 <mkarpiarz> Is there a reason we've not merged 861806 yet? 14:29:47 <rafaelweingartner> good question 14:29:54 <rafaelweingartner> I might have mixed it with some other patch 14:30:03 <rafaelweingartner> Looking at it, it should have already been merged 14:31:03 <mkarpiarz> Let's do it then. :) 14:31:27 <priteau> Well I only gave my +2 today 14:31:44 <mkarpiarz> Ah! 14:32:47 <rafaelweingartner> ah no 14:32:48 <rafaelweingartner> wait 14:32:50 <rafaelweingartner> there is a reason 14:33:10 <rafaelweingartner> that one needs the introduction page that Pierre requested because of the expression "rating type" 14:33:40 <priteau> Yes, I saw it wasn't added yet. If you are still planning to do so please do 14:34:06 <rafaelweingartner> yes, we are 14:34:28 <rafaelweingartner> So, moving one we have the following that might be ready for merging: #link https://review.opendev.org/c/openstack/cloudkitty/+/877525 is a simple one that can be merged. What do you guys think? 14:35:43 <priteau> Merging 14:35:54 <rafaelweingartner> thanks 14:36:27 <mkarpiarz> Ah, yes, I should have re-applied my +2 here as well. :) 14:36:36 <rafaelweingartner> and last, but not least: #link https://review.opendev.org/c/openstack/cloudkitty/+/877537 can be merged. We just need another core review approval. 14:36:59 <priteau> mkarpiarz: Please check this one and merge if you're happy 14:37:11 <rafaelweingartner> and that basically finishes my list here. Do you guys have something else to add? 14:37:35 <priteau> Just letting you know that we are making progress with OpenSearch v2 14:37:43 <priteau> But still needs more work 14:37:52 <rafaelweingartner> ok 14:37:53 <rafaelweingartner> thanks! 14:38:06 <mkarpiarz> Awesome! 14:40:12 <mkarpiarz> I believe I already added OpenSearch to my test environemnt so you guys just let us know when we can start testing your changes. 14:40:22 <priteau> Sure, thanks 14:40:45 <priteau> Something else to discuss: there was a call for participation to the physical PTG in Vancouver. I assume we are not participating? 14:42:04 <rafaelweingartner> I would say not 14:42:11 <rafaelweingartner> Unless, if it is mandatory 14:43:04 <priteau> No it isn't 14:43:30 <rafaelweingartner> Now, I open for general questions and topics that people might have. 14:43:34 <rafaelweingartner> #topic Open floor moment 14:43:41 <rafaelweingartner> Do you guys have something else to add? 14:44:04 <mkarpiarz> Nothing from my side. 14:44:57 <priteau> Nothing else. 14:46:07 <rafaelweingartner> Thank you guys for participating. Have a nice week. 14:46:10 <rafaelweingartner> #endmeeting