| rafaelweingartner | #startmeeting cloudkitty | 14:00 |
|---|---|---|
| opendevmeet | Meeting started Mon Oct 2 14:00: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:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| opendevmeet | The meeting name has been set to 'cloudkitty' | 14:00 |
| rafaelweingartner | hello guys! | 14:00 |
| rafaelweingartner | Roll count | 14:00 |
| priteau | o/ | 14:00 |
| rafaelweingartner | \o | 14:02 |
| mkarpiarz | Hi! | 14:02 |
| rafaelweingartner | Awesome! You all are here today! | 14:02 |
| rafaelweingartner | #topic vPTG meeting | 14:02 |
| rafaelweingartner | Please, register at: https://ptg2023.openinfra.dev/ | 14:02 |
| rafaelweingartner | we scheduled at room bexar for 13:00 UTC and 14:00 UTC - October 23-27, 2023 | 14:02 |
| rafaelweingartner | We just need to create the Etherpad for the vPTG | 14:03 |
| rafaelweingartner | is the timeslot fine for everybody? | 14:03 |
| mkarpiarz | It's fine for me. :) | 14:03 |
| priteau | All good for me. It's in my calendar. | 14:04 |
| rafaelweingartner | awesome! | 14:06 |
| rafaelweingartner | #topic OpenSearch support | 14:08 |
| rafaelweingartner | do we have updates on this matter? | 14:08 |
| priteau | My colleague Matt Crees was not able to replicate the migration issues I saw, so I need to test again. If I cannot replicate them, I suppose it will be good to go. | 14:09 |
| priteau | I will try to prepare this before PTG | 14:10 |
| rafaelweingartner | that would be great! | 14:10 |
| rafaelweingartner | thanks for the update | 14:10 |
| rafaelweingartner | #topic Monasca deprecation | 14:13 |
| rafaelweingartner | this one is on my plate | 14:13 |
| rafaelweingartner | and I will conduct this one to be released under C release | 14:13 |
| mkarpiarz | OK | 14:14 |
| rafaelweingartner | #topic Target reviews | 14:15 |
| rafaelweingartner | We have the following patch, #link https://review.opendev.org/c/openstack/cloudkitty/+/861786 | 14:15 |
| rafaelweingartner | which seems to be ready for merging | 14:15 |
| rafaelweingartner | do you guys have an opinion on it? | 14:15 |
| rafaelweingartner | once it is merged, we can move on with #link https://review.opendev.org/c/openstack/cloudkitty/+/862824 | 14:16 |
| priteau | I am leaving some comments | 14:19 |
| mkarpiarz | I would like to try 861786 out in my test environment first. | 14:19 |
| mkarpiarz | I should have time this week. | 14:20 |
| priteau | It still don't understand why we needed to use these time-h, time-d, etc. options rather than groupby: hour, groupby: day. Maybe I need to test it too, to understand how it works. | 14:21 |
| opendevreview | Rafael Weingartner proposed openstack/cloudkitty master: Add groupby options by different timeframes https://review.opendev.org/c/openstack/cloudkitty/+/861786 | 14:22 |
| opendevreview | Rafael Weingartner proposed openstack/cloudkitty master: Add groupby options by different timeframes https://review.opendev.org/c/openstack/cloudkitty/+/861786 | 14:23 |
| rafaelweingartner | to maitain compatibility | 14:23 |
| rafaelweingartner | cloudkitty already had an implementation that was grouping elements hourly via the groupby time option | 14:24 |
| priteau | OK, I see. | 14:24 |
| rafaelweingartner | "time: to group data by hourly" | 14:24 |
| rafaelweingartner | then, when adding this feature, I added the option to group by day, week, month, and year | 14:24 |
| rafaelweingartner | and the idea was to use the structure that was already there | 14:25 |
| rafaelweingartner | ideally, I would prefer time-<the time we want> | 14:25 |
| rafaelweingartner | bu the "time" option to group hourly was already there | 14:25 |
| rafaelweingartner | priteau: I do not understand what you mean with "This should be with the testscenarios import." | 14:27 |
| priteau | Thanks for explaining. | 14:27 |
| rafaelweingartner | is it about ordering? | 14:27 |
| priteau | Yes, just ordering | 14:28 |
| rafaelweingartner | I mean, ordering of the imports | 14:28 |
| priteau | Normally the third party lib imports should be before any local (cloudkitty) imports | 14:28 |
| priteau | You should have three blocks: 1) python stdlib import 2) third party libraries 3) cloudkitty | 14:28 |
| rafaelweingartner | ok | 14:28 |
| rafaelweingartner | ok | 14:29 |
| rafaelweingartner | I see | 14:29 |
| opendevreview | Rafael Weingartner proposed openstack/cloudkitty master: Add groupby options by different timeframes https://review.opendev.org/c/openstack/cloudkitty/+/861786 | 14:29 |
| rafaelweingartner | done | 14:29 |
| rafaelweingartner | moving on to #link https://review.opendev.org/c/openstack/cloudkitty/+/861908 | 14:31 |
| rafaelweingartner | it seems ready | 14:31 |
| rafaelweingartner | there was a request for tempest, but we do not have a tempest structure to run tests on CloudKitty processor | 14:32 |
| rafaelweingartner | therefore, it felt like a lot of working for such as small gain now | 14:32 |
| rafaelweingartner | the code that is changed has unit tests, and they seem to be enough | 14:32 |
| rafaelweingartner | specially, now that Gnocchi has already launched the fix a few releases ago | 14:32 |
| priteau | I just suggested a small reno change, otherwise looks good | 14:35 |
| opendevreview | Rafael Weingartner proposed openstack/cloudkitty master: Use correct metadata for metrics gathered from gnocchi https://review.opendev.org/c/openstack/cloudkitty/+/861908 | 14:35 |
| rafaelweingartner | I amended the patch | 14:35 |
| rafaelweingartner | thanks! | 14:35 |
| rafaelweingartner | moving on, this is an interesting one #link https://review.opendev.org/c/openstack/cloudkitty/+/895204 | 14:37 |
| rafaelweingartner | that patch adds the support to InfluxDB V2 to CloudKitty | 14:37 |
| rafaelweingartner | it might be interesting for you guys, and for us as a community to have it in the upcomming releases | 14:37 |
| rafaelweingartner | the reviewing is probably going to take a while, so we can have this review in async mode =) | 14:39 |
| priteau | I will need to read it fully first | 14:40 |
| rafaelweingartner | thanks! | 14:40 |
| rafaelweingartner | besides that, we have the following #link https://review.opendev.org/c/openstack/cloudkitty/+/895166, which is also important. It is a fix for a feature with Gnocchi backend. This one is important for us to have it in in the upcomming release. | 14:41 |
| rafaelweingartner | We also have two other optimizations that are interesting to merge as soon as possible. #link https://review.opendev.org/c/openstack/cloudkitty/+/894810 and #link https://review.opendev.org/c/openstack/cloudkitty/+/894812. | 14:41 |
| priteau | I made a quick first pass of 895166, more complete review needed later | 14:43 |
| opendevreview | Rafael Weingartner proposed openstack/cloudkitty master: Patch for `use_all_resource_revisions` option https://review.opendev.org/c/openstack/cloudkitty/+/895166 | 14:46 |
| opendevreview | Rafael Weingartner proposed openstack/cloudkitty master: Optimize CloudKitty reprocessing process https://review.opendev.org/c/openstack/cloudkitty/+/894812 | 14:47 |
| rafaelweingartner | thanks!! | 14:48 |
| rafaelweingartner | and, last but not least. #topic SQLAlchemy 2.x / oslo.db 14.0.0 compatibility | 14:49 |
| rafaelweingartner | I guess priteau said that he was going to take upon this one | 14:49 |
| rafaelweingartner | do we have an update on this topic? | 14:49 |
| priteau | Sorry, not yet | 14:49 |
| opendevreview | Rafael Weingartner proposed openstack/cloudkitty master: Optimize CloudKitty reprocessing process https://review.opendev.org/c/openstack/cloudkitty/+/894812 | 14:50 |
| rafaelweingartner | no worries | 14:51 |
| rafaelweingartner | thanks for the update | 14:51 |
| rafaelweingartner | I guess that is all from my side | 14:51 |
| rafaelweingartner | do you guys have anything else, before we close today? | 14:51 |
| priteau | Nothing else from me | 14:51 |
| mkarpiarz | Nothing from my side. | 14:52 |
| rafaelweingartner | Thank you guys for participating. Have a nice week. | 14:52 |
| rafaelweingartner | #endmeeting | 14:52 |
| opendevmeet | Meeting ended Mon Oct 2 14:52:37 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:52 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cloudkitty/2023/cloudkitty.2023-10-02-14.00.html | 14:52 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cloudkitty/2023/cloudkitty.2023-10-02-14.00.txt | 14:52 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cloudkitty/2023/cloudkitty.2023-10-02-14.00.log.html | 14:52 |
| opendevreview | Merged openstack/cloudkitty master: Fix a concurrency issue when locking reprocessing tasks https://review.opendev.org/c/openstack/cloudkitty/+/894810 | 15:36 |
| opendevreview | Merged openstack/cloudkitty master: Use correct metadata for metrics gathered from gnocchi https://review.opendev.org/c/openstack/cloudkitty/+/861908 | 17:28 |
| opendevreview | Pedro Henrique Pereira Martins proposed openstack/cloudkitty master: Add support to influx v2 as storage backend https://review.opendev.org/c/openstack/cloudkitty/+/895204 | 18:05 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!