14:02:05 <rafaelweingartner> #startmeeting cloudkitty
14:02:05 <opendevmeet> Meeting started Mon Feb 20 14:02: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:02:05 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:02:05 <opendevmeet> The meeting name has been set to 'cloudkitty'
14:02:13 <rafaelweingartner> Roll Count
14:02:14 <rafaelweingartner> \O
14:02:20 <mkarpiarz> Hi!
14:06:40 <rafaelweingartner> priteau: are you going to join us today?
14:07:45 <priteau> l
14:07:48 <priteau> o/
14:07:52 <rafaelweingartner> :)
14:07:56 <priteau> Sorry, doing too many things at once
14:08:04 <rafaelweingartner> no worries
14:08:23 <rafaelweingartner> let's start with the reviews
14:08:25 <rafaelweingartner> #topic Target review of the meeting
14:08:52 <rafaelweingartner> #link https://review.opendev.org/c/openstack/cloudkitty-specs/+/866198, it is that spec to add the support of start/end date to rating rules
14:09:02 <rafaelweingartner> I guess we are almost ready to merge it, right?
14:09:33 <priteau> I need to take another look at it, but haven't got time recently.
14:09:45 <priteau> It's too late for A cycle anyway…
14:10:46 <rafaelweingartner> yes, exactly. We do not mean to add it to A release, but just so we have a basic understanding if that would be an interesting addition to CloudKitty
14:11:32 <priteau> Then I will aim to approve it next week at the latest.
14:11:55 <rafaelweingartner> awesome
14:11:56 <mkarpiarz> Yeah, I think we all agree we want this feature in but there are a few details we are still discussing.
14:11:57 <rafaelweingartner> Thanks!
14:12:28 <mkarpiarz> Maybe we should merge the blueprint and move discussion to patches implementing the ideas?
14:12:47 <rafaelweingartner> hmm, if the discussion is about implementation yes
14:13:08 <rafaelweingartner> However, if the discussion is about design and feature scope, then it should be done in the spec, right?
14:13:23 <mkarpiarz> Ah, I see.
14:14:14 <mkarpiarz> Anyway, I think we can wait for Pierre to have another look here.
14:14:28 <rafaelweingartner> agreed
14:15:16 <rafaelweingartner> So, moving on to the patches. I have the following #link https://review.opendev.org/c/openstack/cloudkitty/+/861786, which I need to create Gabbi tests
14:15:28 <rafaelweingartner> I am plan to update that patch this week
14:16:51 <mkarpiarz> OK, I'll keep an eye on this one and get it into my test environemnt after your update.
14:17:02 <rafaelweingartner> thanks!
14:17:40 <rafaelweingartner> We then have this one #link https://review.opendev.org/c/openstack/cloudkitty/+/862824, which seems to be ready
14:17:47 <rafaelweingartner> What do you guys think?
14:18:52 <mkarpiarz> Sure, I can +2 this one again.
14:19:33 <rafaelweingartner> thanks!
14:21:49 <priteau> kb -> kB
14:22:23 <priteau> in the docs
14:22:33 <priteau> I just tried to change it but it depends on not current version of https://review.opendev.org/c/openstack/cloudkitty/+/861786/5
14:23:36 <rafaelweingartner> I see
14:23:41 <rafaelweingartner> no worries, I can fix that for you
14:23:54 <rafaelweingartner> as soon as I am done with the Gabbi tests on the other patch
14:24:15 <priteau> Thanks. I will +2 once refreshed
14:24:20 <rafaelweingartner> that is the change, right: "up to 64 KB of size"
14:24:37 <rafaelweingartner> When I saw your description, I uderstood that you wanted to add a space
14:24:55 <rafaelweingartner> I did not understand that you want to change the kb to kB
14:25:02 <priteau> The space was a nit, the real fix was the b -> B
14:25:08 <priteau> b = bit
14:25:14 <priteau> B = byte
14:25:22 <priteau> 8 times more ;-)
14:25:50 <rafaelweingartner> exactly
14:25:57 <rafaelweingartner> thanks
14:26:19 <rafaelweingartner> Then, movin on to #link: https://review.opendev.org/c/openstack/cloudkitty/+/861908
14:26:32 <rafaelweingartner> For this one I need to add tempest tests
14:30:18 <rafaelweingartner> After that one, we have the following #link: https://review.opendev.org/c/openstack/cloudkitty/+/865417
14:30:30 <rafaelweingartner> it seems ready to merge
14:32:40 <rafaelweingartner> mkarpiarz: do you have any concerns there?
14:33:24 <mkarpiarz> Sure
14:34:21 <rafaelweingartner> Thanks!
14:34:34 <rafaelweingartner> Moing on with the optimizations, we have #link: https://review.opendev.org/c/openstack/cloudkitty/+/872035
14:34:46 <rafaelweingartner> which is another optimiaztion that we did for Gnocchi fetcher
14:36:51 <mkarpiarz> I haven't looked at this one yet.
14:37:14 <rafaelweingartner> no worries, but this is an interesting one to have to speed Gnocchi fetchers
14:39:40 <mkarpiarz> OK
14:40:00 <priteau> Sorry but 865417 is not ready
14:40:07 <priteau> mkarpiarz can you please remove your W+1
14:40:20 <rafaelweingartner> isn't it?
14:41:22 <rafaelweingartner> Ah
14:41:29 <rafaelweingartner> I have not read the release notes
14:41:32 <rafaelweingartner> :(
14:41:38 <rafaelweingartner> sorry for that mistake
14:42:19 <priteau> No a big problem. Let's update and merge again.
14:42:26 <mkarpiarz> OK
14:42:33 <rafaelweingartner> Good catch
14:45:33 <rafaelweingartner> And, last,but not least, the last patch I had here in my list #link: https://review.opendev.org/c/openstack/cloudkitty/+/861806
14:45:53 <rafaelweingartner> have you reached a conclusion about it?
14:47:06 <rafaelweingartner> I mean, have you had time to finish the review there
14:48:55 <priteau> I am not really familiar enough with Gnocchi to be honest
14:50:55 <rafaelweingartner> no worries, but in theory it should not be limite to Gnocchi
14:51:14 <rafaelweingartner> even though it was implemented for it first, we can validate the idea, and maybe apply to others
14:53:55 <rafaelweingartner> that is all from my side guys
14:54:01 <rafaelweingartner> Do you have something else to add?
14:54:33 <mkarpiarz> Nothing from my side.
14:55:25 <priteau> As mentioned earlier, I've been working on the OpenSearch support
14:55:49 <priteau> I got a bootstrap working but still need to test the migration
14:56:09 <rafaelweingartner> I see
14:56:35 <rafaelweingartner> is your intention to migrate the data via CloudKitty?
14:56:46 <rafaelweingartner> Or some process that operators do ?
14:56:56 <priteau> No, it would be done by Kolla Ansible
14:57:01 <priteau> But need to check that it works fine
14:57:14 <priteau> It has only been tested for logging data, not CK
14:57:52 <rafaelweingartner> ah, I see
14:58:39 <priteau> This week is crazy but I will try to go back to it ASAP
14:59:25 <rafaelweingartner> ok, no worries
14:59:33 <rafaelweingartner> and if you need a hand, just let me know
15:00:55 <rafaelweingartner> I guess that is all for today
15:00:58 <rafaelweingartner> Thank you guys for participating. Have a nice week.
15:01:07 <rafaelweingartner> #endmeeting