14:00:05 <rafaelweingartner> #startmeeting cloudkitty
14:00:05 <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:05 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:00:05 <opendevmeet> The meeting name has been set to 'cloudkitty'
14:00:08 <rafaelweingartner> hello guys!
14:00:13 <rafaelweingartner> Roll count
14:00:58 <priteau> o/
14:02:02 <rafaelweingartner> \o
14:02:06 <mkarpiarz> Hi!
14:02:26 <rafaelweingartner> Awesome! You all are here today!
14:02:38 <rafaelweingartner> #topic vPTG meeting
14:02:50 <rafaelweingartner> Please, register at: https://ptg2023.openinfra.dev/
14:02:59 <rafaelweingartner> we     scheduled at room bexar for 13:00 UTC and 14:00 UTC - October 23-27, 2023
14:03:16 <rafaelweingartner> We just need to create the Etherpad for the vPTG
14:03:30 <rafaelweingartner> is the timeslot fine for everybody?
14:03:52 <mkarpiarz> It's fine for me. :)
14:04:33 <priteau> All good for me. It's in my calendar.
14:06:54 <rafaelweingartner> awesome!
14:08:21 <rafaelweingartner> #topic OpenSearch support
14:08:25 <rafaelweingartner> do we have updates on this matter?
14:09:53 <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:10:11 <priteau> I will try to prepare this before PTG
14:10:20 <rafaelweingartner> that would be great!
14:10:22 <rafaelweingartner> thanks for the update
14:13:44 <rafaelweingartner> #topic Monasca deprecation
14:13:47 <rafaelweingartner> this one is on my plate
14:13:57 <rafaelweingartner> and I will conduct this one to be released under C release
14:14:12 <mkarpiarz> OK
14:15:32 <rafaelweingartner> #topic Target reviews
14:15:45 <rafaelweingartner> We have the following patch, #link https://review.opendev.org/c/openstack/cloudkitty/+/861786
14:15:50 <rafaelweingartner> which seems to be ready for merging
14:15:59 <rafaelweingartner> do you guys have an opinion on it?
14:16:23 <rafaelweingartner> once it is merged, we can move on with #link https://review.opendev.org/c/openstack/cloudkitty/+/862824
14:19:06 <priteau> I am leaving some comments
14:19:24 <mkarpiarz> I would like to try 861786 out in my test environment first.
14:20:08 <mkarpiarz> I should have time this week.
14:21:41 <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:22:30 <opendevreview> Rafael Weingartner proposed openstack/cloudkitty master: Add groupby options by different timeframes  https://review.opendev.org/c/openstack/cloudkitty/+/861786
14:23:24 <opendevreview> Rafael Weingartner proposed openstack/cloudkitty master: Add groupby options by different timeframes  https://review.opendev.org/c/openstack/cloudkitty/+/861786
14:23:51 <rafaelweingartner> to maitain compatibility
14:24:12 <rafaelweingartner> cloudkitty already had an implementation that was grouping elements hourly via the groupby time option
14:24:29 <priteau> OK, I see.
14:24:34 <rafaelweingartner> "time: to group data by hourly"
14:24:48 <rafaelweingartner> then, when adding this feature, I added the option to group by day, week, month, and year
14:25:03 <rafaelweingartner> and the idea was to use the structure that was already there
14:25:15 <rafaelweingartner> ideally, I would prefer time-<the time we want>
14:25:25 <rafaelweingartner> bu the "time" option to group hourly was already there
14:27:52 <rafaelweingartner> priteau: I do not understand what you mean with "This should be with the testscenarios import."
14:27:53 <priteau> Thanks for explaining.
14:27:58 <rafaelweingartner> is it about ordering?
14:28:02 <priteau> Yes, just ordering
14:28:09 <rafaelweingartner> I mean, ordering of the imports
14:28:23 <priteau> Normally the third party lib imports should be before any local (cloudkitty) imports
14:28:50 <priteau> You should have three blocks: 1) python stdlib import 2) third party libraries 3) cloudkitty
14:28:59 <rafaelweingartner> ok
14:29:01 <rafaelweingartner> ok
14:29:03 <rafaelweingartner> I see
14:29:34 <opendevreview> Rafael Weingartner proposed openstack/cloudkitty master: Add groupby options by different timeframes  https://review.opendev.org/c/openstack/cloudkitty/+/861786
14:29:38 <rafaelweingartner> done
14:31:46 <rafaelweingartner> moving on to #link https://review.opendev.org/c/openstack/cloudkitty/+/861908
14:31:49 <rafaelweingartner> it seems ready
14:32:07 <rafaelweingartner> there was a request for tempest, but we do not have a tempest structure to run tests on CloudKitty processor
14:32:17 <rafaelweingartner> therefore, it felt like a lot of working for such as small gain now
14:32:27 <rafaelweingartner> the code that is changed has unit tests, and they seem to be enough
14:32:41 <rafaelweingartner> specially, now that Gnocchi has already launched the fix a few releases ago
14:35:01 <priteau> I just suggested a small reno change, otherwise looks good
14:35:19 <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:32 <rafaelweingartner> I amended the patch
14:35:35 <rafaelweingartner> thanks!
14:37:09 <rafaelweingartner> moving on, this is an interesting one #link https://review.opendev.org/c/openstack/cloudkitty/+/895204
14:37:22 <rafaelweingartner> that patch adds the support to InfluxDB V2 to CloudKitty
14:37:40 <rafaelweingartner> it might be interesting for you guys, and for us as a community to have it in the upcomming releases
14:39:39 <rafaelweingartner> the reviewing is probably going to take a while, so we can have this review in async mode =)
14:40:26 <priteau> I will need to read it fully first
14:40:36 <rafaelweingartner> thanks!
14:41:03 <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:47 <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:43:39 <priteau> I made a quick first pass of 895166, more complete review needed later
14:46:15 <opendevreview> Rafael Weingartner proposed openstack/cloudkitty master: Patch for `use_all_resource_revisions` option  https://review.opendev.org/c/openstack/cloudkitty/+/895166
14:47:45 <opendevreview> Rafael Weingartner proposed openstack/cloudkitty master: Optimize CloudKitty reprocessing process  https://review.opendev.org/c/openstack/cloudkitty/+/894812
14:48:14 <rafaelweingartner> thanks!!
14:49:09 <rafaelweingartner> and, last but not least. #topic SQLAlchemy 2.x / oslo.db 14.0.0 compatibility
14:49:19 <rafaelweingartner> I guess priteau said that he was going to take upon this one
14:49:25 <rafaelweingartner> do we have an update on this topic?
14:49:29 <priteau> Sorry, not yet
14:50:51 <opendevreview> Rafael Weingartner proposed openstack/cloudkitty master: Optimize CloudKitty reprocessing process  https://review.opendev.org/c/openstack/cloudkitty/+/894812
14:51:03 <rafaelweingartner> no worries
14:51:06 <rafaelweingartner> thanks for the update
14:51:13 <rafaelweingartner> I guess that is all from my side
14:51:21 <rafaelweingartner> do you guys have anything else, before we close today?
14:51:46 <priteau> Nothing else from me
14:52:29 <mkarpiarz> Nothing from my side.
14:52:34 <rafaelweingartner> Thank you guys for participating. Have a nice week.
14:52:37 <rafaelweingartner> #endmeeting