14:01:49 <rafaelweingartner> #startmeeting cloudkitty
14:01:49 <opendevmeet> Meeting started Mon Dec 12 14:01:49 2022 UTC and is due to finish in 60 minutes.  The chair is rafaelweingartner. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:01:49 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:01:49 <opendevmeet> The meeting name has been set to 'cloudkitty'
14:01:53 <rafaelweingartner> Roll count
14:01:58 <rafaelweingartner> \O
14:02:00 <priteau> o/
14:02:28 <mkarpiarz> Hi!
14:03:37 <rafaelweingartner> # Topic OpenSearch support
14:04:50 <rafaelweingartner> We already merged the notice, and Pierre is proposing a change in DevStack, which I guess was already merged.
14:04:54 <priteau> #topic OpenSearch support
14:05:24 <priteau> That's just step 0 for OpenSearch support really. The hard work is still to be done.
14:05:33 <priteau> Which is compatibility with v2
14:05:52 <priteau> I don't have time this week to look into it. Early next year.
14:06:27 <rafaelweingartner> I see
14:06:44 <rafaelweingartner> but the process has started. I mean, the notice, and the change in Devstack
14:07:40 <rafaelweingartner> Now, we are missing this item, "Crete the OpenSearch option in the Storage backend and deprecate elastic search", right?
14:08:02 <priteau> Yes, create a new storage driver specifically for opensearch v2
14:08:10 <rafaelweingartner> Ok
14:08:17 <priteau> Which, if I understand correctly, would be compatible with elasticsearch v8 too
14:08:39 <priteau> But we probably won't be testing it
14:08:43 <rafaelweingartner> I see
14:09:43 <rafaelweingartner> Do we need anything else regarding OpenSearch?
14:10:52 <priteau> Not right now, but we need to be aware that Kolla Ansible will release Zed this week, with OpenSearch v2 replacing Elasticsearch v7, which will break CloudKitty until we can add support.
14:11:06 <priteau> Actually I should add a release note in Kolla Ansible itself about it.
14:11:17 <priteau> So we will want to backport support to Zed once we have it.
14:11:31 <rafaelweingartner> Hmm
14:11:51 <rafaelweingartner> I see
14:12:26 <rafaelweingartner> thanks for that!
14:12:33 <rafaelweingartner> If you need my help, let me know
14:12:54 <priteau> Thanks
14:13:05 <rafaelweingartner> Moving on then
14:13:09 <rafaelweingartner> #topic Monasca deprecation
14:13:17 <rafaelweingartner> #link https://review.opendev.org/c/openstack/cloudkitty/+/866140
14:13:24 <rafaelweingartner> we need to merge the patch with the notice
14:13:37 <priteau> I've given my +2.
14:14:01 <mkarpiarz> Ah, sorry, I missed that one!
14:14:10 <rafaelweingartner> mkarpiarz: can you do the honors?
14:14:25 <mkarpiarz> Reviewed and merged. :)
14:14:32 <rafaelweingartner> thanks!
14:14:50 <rafaelweingartner> Moving on to the next topic
14:14:54 <rafaelweingartner> #topic add CloudKitty API reference docs
14:15:06 <rafaelweingartner> mkarpiarz: What is the status of that?
14:15:55 <mkarpiarz> First we need to merge https://review.opendev.org/c/openstack/cloudkitty/+/867122
14:16:53 <mkarpiarz> With API docs in api-ref/source we can then update a different repo and docs should be added to the landing page.
14:17:04 <mkarpiarz> At least this is my understanding. :)
14:17:26 <rafaelweingartner> cool
14:17:30 <rafaelweingartner> let's see
14:17:41 <rafaelweingartner> I think it is good to merge. I gave my +2 there
14:18:49 <opendevreview> Merged openstack/cloudkitty master: Add deprecation notice for Monasca  https://review.opendev.org/c/openstack/cloudkitty/+/866140
14:18:57 <mkarpiarz> Thanks!
14:19:24 <mkarpiarz> In case you are curious, this is the repo I'm talking about: https://opendev.org/openstack/openstack-manuals/src/branch/master/www/project-data/latest.yaml#L452-L461
14:20:44 <priteau> Looking now
14:20:49 <rafaelweingartner> I see, thanks
14:21:36 <priteau> No change in generated docs, looks good.
14:22:36 <mkarpiarz> Yes, we are simply moving API refs to a separate directory.
14:23:48 <mkarpiarz> Like I said, it would be good to convert v1 refs to the format used by v2 refs but this is a completely separate task.
14:23:55 <priteau> What's next?
14:24:41 <mkarpiarz> My understanding is we need to add "has_api_ref: true" here:
14:24:43 <mkarpiarz> https://opendev.org/openstack/openstack-manuals/src/branch/master/www/project-data/latest.yaml#L452-L461
14:25:23 <mkarpiarz> "has_user_guide: true" is already there and this is why CloudKitty shows up here: https://docs.openstack.org/zed/user/
14:25:58 <rafaelweingartner> cool
14:27:09 <rafaelweingartner> mkarpiarz: are you going to move on with that, and then we sync up next meeting?
14:27:49 <mkarpiarz> Yes, this week I want to try to add our API refs to "openstack-manuals".
14:28:04 <rafaelweingartner> cool, thanks
14:28:07 <priteau> The patch should merge in about 30 minutes
14:28:13 <rafaelweingartner> So, moving on.
14:28:16 <rafaelweingartner> #topic Target reviews
14:28:41 <rafaelweingartner> We have a new spec. I hava reviewed it, and it seems ok. Have you guys had time to review it?
14:28:52 <rafaelweingartner> #link https://review.opendev.org/c/openstack/cloudkitty-specs/+/866198
14:29:12 <priteau> I am afraid I did not, sorry.
14:29:38 <mkarpiarz> Me neither. :(
14:30:50 <rafaelweingartner> Ok, it is a long one to read in the meeting
14:31:05 <rafaelweingartner> but if you guys can try to check it today or tomorrow, it would be interesting for us
14:31:18 <rafaelweingartner> it is the first spec we see in quite some time
14:33:19 <priteau> Do we know where the author is from?
14:33:53 <rafaelweingartner> yes, I know him
14:34:21 <rafaelweingartner> He is a colleague of mine
14:35:37 <priteau> You guys are always in stealth mode :D
14:35:58 <priteau> OK, I will add to my todo list to read it
14:36:04 <rafaelweingartner> =)
14:38:26 <rafaelweingartner> Ok, so moving on
14:38:30 <mkarpiarz> I'll have a read too. Probably not tomorrow though.
14:38:35 <rafaelweingartner> we have the patch #link https://review.opendev.org/c/openstack/cloudkitty/+/862824
14:38:40 <mkarpiarz> *not today
14:38:59 <rafaelweingartner> Have you guys had the opportunity to check it?
14:39:33 <mkarpiarz> Please check out my last comment.
14:40:14 <mkarpiarz> The feature seems to be working as expected and it adds description to each record.
14:40:27 <priteau> There is the issue that we will duplicate the description to each record. We should at least warn the operator about this, it could bloat the database.
14:41:02 <rafaelweingartner> yes, it could
14:41:19 <rafaelweingartner> However, that already happens with two other fields
14:41:35 <rafaelweingartner> Can you suggest that there? Then, we have a story line for this topic
14:41:53 <rafaelweingartner> and, maybe, we can look for a better way of handling these metadata
14:43:09 <priteau> Why can't we store this in SQL DB? Or even in metrics.yml?
14:43:27 <priteau> The API can read the DB when it processes the response
14:43:27 <rafaelweingartner> that is something that Pedro proposed :)
14:43:44 <rafaelweingartner> but then we would need to change not just for the description, but also for other metadata, such as unit
14:44:24 <priteau> There are things that are as they are, for historical reasons. We don't have to keep doing the same mistake ;-)
14:44:44 <rafaelweingartner> I could work on this improvement/optimization, but ideally it is something on top of those patches to separate the context, and to maybe apply it for other things, such as "metadata"
14:45:14 <rafaelweingartner> priteau: I agree with you
14:46:52 <rafaelweingartner> So, moving on. #link https://review.opendev.org/c/openstack/cloudkitty/+/861806
14:47:05 <rafaelweingartner> Mariusz has approved it, but still need one more
14:48:05 <rafaelweingartner> Also, this one #link https://review.opendev.org/c/openstack/cloudkitty/+/864269, It is provides an optimization for Gnocchi fecther, which would be a nice one to have
14:48:24 <rafaelweingartner> It is an optimization to control the memory foor print of CloudKitty processor when loading scopes to process
14:50:33 <priteau> On the first you left a comment on the commit message.
14:51:54 <rafaelweingartner> What do you mean?
14:52:11 <rafaelweingartner> I see
14:52:16 <rafaelweingartner> that must have slipped through
14:52:32 <priteau> This: https://review.opendev.org/c/openstack/cloudkitty/+/861806/2/COMMIT_MSG
14:52:39 <priteau> Was that a proposal to rephrase?
14:52:52 <rafaelweingartner> I am pinging pedro now
14:52:55 <priteau> Not very important, but anyway I need a bit more time to read through the code.
14:53:14 <rafaelweingartner> yes, the text was a bit hard to read. So, I tried helping him
14:55:46 <priteau> I will review both patches, hopefully before the next meeting.
14:56:38 <priteau> Should we discuss next meeting now?
14:57:11 <rafaelweingartner> sure
14:57:13 <rafaelweingartner> I guess so
14:57:17 <rafaelweingartner> we are running out of time
14:57:41 <rafaelweingartner> Are you guys going to be working on 26 of December?
14:57:57 <mkarpiarz> Oh, a very good question!
14:58:07 <priteau> Certainly not ;-)
14:58:14 <mkarpiarz> Looks like I'm off. :)
14:58:18 <rafaelweingartner> I see
14:58:26 <opendevreview> Pedro Henrique Pereira Martins proposed openstack/cloudkitty master: Allows multiple rating types for same metric for gnocchi  https://review.opendev.org/c/openstack/cloudkitty/+/861806
14:58:28 <priteau> I'll be back on January 3
14:58:28 <rafaelweingartner> so, we skip the next one then?
14:58:44 <priteau> Yes, I think we should meet next on the 9th
14:59:01 <rafaelweingartner> ok
14:59:08 <rafaelweingartner> let's do that then
14:59:43 <mkarpiarz> Sure
15:00:00 <rafaelweingartner> Do you guys have something else to add?
15:00:25 <priteau> Merry Christmas :)
15:01:08 <mkarpiarz> :D
15:01:59 <rafaelweingartner> I wish you guys a merry Christmas and Happy New Year. See you next year then =)
15:02:20 <rafaelweingartner> Thank you guys for participating. Have a nice week.
15:02:22 <priteau> Thanks. Good bye guys.
15:02:36 <mkarpiarz> Thanks and all the best to both of you. :)
15:02:44 <rafaelweingartner> It has been a pleasure to work this year with you guys. I hope we can keep the good work next year.
15:02:54 <rafaelweingartner> #endmeeting