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