| _0230220-joek | hello folks, | 10:11 |
|---|---|---|
| _0230220-joek | have a problem with cloudkitty on startup. Cloudkitty gets a error code 400 while instantiating a mapping. Destination is a opensearch (version 2.3) cluster over elasticsearch storage driver. Can anyone help? | 10:13 |
| opendevreview | Rafael Weingartner proposed openstack/cloudkitty master: Add groupby options by different timeframes https://review.opendev.org/c/openstack/cloudkitty/+/861786 | 11:16 |
| _0230220-joek | Can anyone tell me if cloudkitty is basically compliant to opensearch? Kolla or kolla-ansible deploys with zed release a opensearch cluster. | 11:28 |
| priteau | Hello _0230220-joek. Currently CloudKitty is not compatible with OpenSearch 2.x. | 13:50 |
| priteau | Elasticsearch was replaced by OpenSearch late in the Zed cycle. | 13:50 |
| priteau | I am working on compatibility which we will backport | 13:50 |
| rafaelweingartner | Hello guys! | 14:02 |
| rafaelweingartner | #startmeeting cloudkitty | 14:02 |
| 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 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:02 |
| opendevmeet | The meeting name has been set to 'cloudkitty' | 14:02 |
| rafaelweingartner | Roll Count | 14:02 |
| rafaelweingartner | \O | 14:02 |
| mkarpiarz | Hi! | 14:02 |
| rafaelweingartner | priteau: are you going to join us today? | 14:06 |
| priteau | l | 14:07 |
| priteau | o/ | 14:07 |
| rafaelweingartner | :) | 14:07 |
| priteau | Sorry, doing too many things at once | 14:07 |
| rafaelweingartner | no worries | 14:08 |
| rafaelweingartner | let's start with the reviews | 14:08 |
| rafaelweingartner | #topic Target review of the meeting | 14:08 |
| 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:08 |
| rafaelweingartner | I guess we are almost ready to merge it, right? | 14:09 |
| priteau | I need to take another look at it, but haven't got time recently. | 14:09 |
| priteau | It's too late for A cycle anyway… | 14:09 |
| 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:10 |
| priteau | Then I will aim to approve it next week at the latest. | 14:11 |
| rafaelweingartner | awesome | 14:11 |
| mkarpiarz | Yeah, I think we all agree we want this feature in but there are a few details we are still discussing. | 14:11 |
| rafaelweingartner | Thanks! | 14:11 |
| mkarpiarz | Maybe we should merge the blueprint and move discussion to patches implementing the ideas? | 14:12 |
| rafaelweingartner | hmm, if the discussion is about implementation yes | 14:12 |
| rafaelweingartner | However, if the discussion is about design and feature scope, then it should be done in the spec, right? | 14:13 |
| mkarpiarz | Ah, I see. | 14:13 |
| mkarpiarz | Anyway, I think we can wait for Pierre to have another look here. | 14:14 |
| rafaelweingartner | agreed | 14:14 |
| 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 |
| rafaelweingartner | I am plan to update that patch this week | 14:15 |
| mkarpiarz | OK, I'll keep an eye on this one and get it into my test environemnt after your update. | 14:16 |
| rafaelweingartner | thanks! | 14:17 |
| rafaelweingartner | We then have this one #link https://review.opendev.org/c/openstack/cloudkitty/+/862824, which seems to be ready | 14:17 |
| rafaelweingartner | What do you guys think? | 14:17 |
| mkarpiarz | Sure, I can +2 this one again. | 14:18 |
| rafaelweingartner | thanks! | 14:19 |
| priteau | kb -> kB | 14:21 |
| priteau | in the docs | 14:22 |
| 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:22 |
| rafaelweingartner | I see | 14:23 |
| rafaelweingartner | no worries, I can fix that for you | 14:23 |
| rafaelweingartner | as soon as I am done with the Gabbi tests on the other patch | 14:23 |
| priteau | Thanks. I will +2 once refreshed | 14:24 |
| rafaelweingartner | that is the change, right: "up to 64 KB of size" | 14:24 |
| rafaelweingartner | When I saw your description, I uderstood that you wanted to add a space | 14:24 |
| rafaelweingartner | I did not understand that you want to change the kb to kB | 14:24 |
| priteau | The space was a nit, the real fix was the b -> B | 14:25 |
| priteau | b = bit | 14:25 |
| priteau | B = byte | 14:25 |
| priteau | 8 times more ;-) | 14:25 |
| rafaelweingartner | exactly | 14:25 |
| rafaelweingartner | thanks | 14:25 |
| rafaelweingartner | Then, movin on to #link: https://review.opendev.org/c/openstack/cloudkitty/+/861908 | 14:26 |
| rafaelweingartner | For this one I need to add tempest tests | 14:26 |
| rafaelweingartner | After that one, we have the following #link: https://review.opendev.org/c/openstack/cloudkitty/+/865417 | 14:30 |
| rafaelweingartner | it seems ready to merge | 14:30 |
| rafaelweingartner | mkarpiarz: do you have any concerns there? | 14:32 |
| mkarpiarz | Sure | 14:33 |
| rafaelweingartner | Thanks! | 14:34 |
| rafaelweingartner | Moing on with the optimizations, we have #link: https://review.opendev.org/c/openstack/cloudkitty/+/872035 | 14:34 |
| rafaelweingartner | which is another optimiaztion that we did for Gnocchi fetcher | 14:34 |
| mkarpiarz | I haven't looked at this one yet. | 14:36 |
| rafaelweingartner | no worries, but this is an interesting one to have to speed Gnocchi fetchers | 14:37 |
| mkarpiarz | OK | 14:39 |
| priteau | Sorry but 865417 is not ready | 14:40 |
| priteau | mkarpiarz can you please remove your W+1 | 14:40 |
| rafaelweingartner | isn't it? | 14:40 |
| rafaelweingartner | Ah | 14:41 |
| rafaelweingartner | I have not read the release notes | 14:41 |
| rafaelweingartner | :( | 14:41 |
| rafaelweingartner | sorry for that mistake | 14:41 |
| priteau | No a big problem. Let's update and merge again. | 14:42 |
| mkarpiarz | OK | 14:42 |
| rafaelweingartner | Good catch | 14:42 |
| 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 |
| rafaelweingartner | have you reached a conclusion about it? | 14:45 |
| rafaelweingartner | I mean, have you had time to finish the review there | 14:47 |
| priteau | I am not really familiar enough with Gnocchi to be honest | 14:48 |
| rafaelweingartner | no worries, but in theory it should not be limite to Gnocchi | 14:50 |
| rafaelweingartner | even though it was implemented for it first, we can validate the idea, and maybe apply to others | 14:51 |
| rafaelweingartner | that is all from my side guys | 14:53 |
| rafaelweingartner | Do you have something else to add? | 14:54 |
| mkarpiarz | Nothing from my side. | 14:54 |
| priteau | As mentioned earlier, I've been working on the OpenSearch support | 14:55 |
| priteau | I got a bootstrap working but still need to test the migration | 14:55 |
| rafaelweingartner | I see | 14:56 |
| rafaelweingartner | is your intention to migrate the data via CloudKitty? | 14:56 |
| rafaelweingartner | Or some process that operators do ? | 14:56 |
| priteau | No, it would be done by Kolla Ansible | 14:56 |
| priteau | But need to check that it works fine | 14:57 |
| priteau | It has only been tested for logging data, not CK | 14:57 |
| rafaelweingartner | ah, I see | 14:57 |
| priteau | This week is crazy but I will try to go back to it ASAP | 14:58 |
| rafaelweingartner | ok, no worries | 14:59 |
| rafaelweingartner | and if you need a hand, just let me know | 14:59 |
| rafaelweingartner | I guess that is all for today | 15:00 |
| rafaelweingartner | Thank you guys for participating. Have a nice week. | 15:00 |
| rafaelweingartner | #endmeeting | 15:01 |
| opendevmeet | Meeting ended Mon Feb 20 15:01:07 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:01 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cloudkitty/2023/cloudkitty.2023-02-20-14.02.html | 15:01 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cloudkitty/2023/cloudkitty.2023-02-20-14.02.txt | 15:01 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cloudkitty/2023/cloudkitty.2023-02-20-14.02.log.html | 15:01 |
| *** gmann is now known as gmann_afk | 18:37 | |
| *** gmann_afk is now known as gmann | 18:51 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!