14:01:15 <rafaelweingartner> #startmeeting cloudkitty
14:01:15 <opendevmeet> Meeting started Mon Oct 16 14:01:15 2023 UTC and is due to finish in 60 minutes.  The chair is rafaelweingartner. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:01:15 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:01:15 <opendevmeet> The meeting name has been set to 'cloudkitty'
14:01:19 <rafaelweingartner> Roll count
14:01:21 <rafaelweingartner> \O
14:01:34 <priteau> o/
14:02:34 <mkarpiarz> Hi!
14:02:47 <rafaelweingartner> Awesome to see you all here today!
14:03:29 <rafaelweingartner> #topic vPTG meeting
14:03:40 <rafaelweingartner> We already scheduled the data, it is going ot be next week!
14:03:55 <rafaelweingartner> we scheduled the room bexar for 13:00 UTC and 14:00 UTC - October 23, 2023
14:03:57 <priteau> Looking forward.
14:04:16 <rafaelweingartner> We need now to create the Etherpad for the vPTG
14:04:28 <rafaelweingartner> I will try to do it today, or to re-use the auto generated one
14:08:07 <mkarpiarz> Thanks!
14:08:07 <rafaelweingartner> #topic OpenSearch support
14:08:12 <rafaelweingartner> we already started reviewing the patch
14:08:27 <rafaelweingartner> maybe, we will need to apply some effort to reduce the amount of replicated code there
14:08:31 <rafaelweingartner> but it is looking promissing
14:08:35 <rafaelweingartner> thanks priteau!
14:09:12 <rafaelweingartner> and, of course, thanks to Matt! =)
14:09:14 <priteau> Unfortunately I thought the migration issue were resolved, but I was able to reproduce them again :/
14:09:23 <priteau> I will try to find some time this week to investigate more
14:09:32 <rafaelweingartner> thanks you!
14:11:20 <rafaelweingartner> Moving on
14:11:22 <rafaelweingartner> #topic
14:11:27 <rafaelweingartner> #topic Monasca deprecation
14:11:37 <rafaelweingartner> that is still on my plate, I will try to do it until the end ot this month
14:12:13 <priteau> Thanks!
14:12:48 <rafaelweingartner> moving on
14:12:49 <rafaelweingartner> #topic Target reviews
14:13:24 <rafaelweingartner> We have the patch #link https://review.opendev.org/c/openstack/cloudkitty/+/861786, which has been approved by Mariusz already
14:13:26 <rafaelweingartner> thank you1
14:13:51 <rafaelweingartner> It was raised some interesting points there, but those points exist before the patch
14:14:13 <mkarpiarz> Absolutely, let's merge the change first.
14:14:22 <rafaelweingartner> I am not sure if we need to handle them, and how to handle them in a consistent fashion
14:16:34 <mkarpiarz> Agreed
14:18:41 <rafaelweingartner> thanks!
14:19:05 <rafaelweingartner> if we merge that, we can then proceed with #link https://review.opendev.org/c/openstack/cloudkitty/+/862824
14:20:16 <rafaelweingartner> if priteau is fine with it, we can move on then, and merge it after this meeting
14:20:34 <priteau> I have a comment actually
14:20:40 <priteau> Posted on Gerrit
14:23:48 <opendevreview> Rafael Weingartner proposed openstack/cloudkitty master: Add groupby options by different timeframes  https://review.opendev.org/c/openstack/cloudkitty/+/861786
14:24:11 <rafaelweingartner> patch updated
14:25:34 <priteau> Otherwise in principle it looks ok, but I haven't had the chance to test it
14:25:51 <rafaelweingartner> We use it in production for over a year now
14:25:54 <rafaelweingartner> =)
14:28:11 <priteau> I believe you, but sometimes because of iterations on Gerrit the merged patch might be different from what you run in prod
14:28:26 <priteau> I think this may be what happened to reprocessing (to be discussed later)
14:28:37 <rafaelweingartner> exactly
14:28:44 <rafaelweingartner> but that is the same for everything else
14:29:45 <priteau> Of course. It's just the bigger the patch, the more chances there are to sneak a bug in. Anyway, I will approve
14:30:17 <rafaelweingartner> I aggree, that is why we need to check things in detail, like we did with this patch
14:33:33 <rafaelweingartner> Anyways, moving on
14:33:48 <rafaelweingartner> We have the path #link https://review.opendev.org/c/openstack/cloudkitty/+/895204, which is the introduction to the InfluxDB V2
14:34:06 <rafaelweingartner> Some reviewings were already done, but if you guys can, it might be good to have other people testing it
14:34:23 <rafaelweingartner> Also, we use this for over a year in production, but it might be interesting for other people to also use/test it
14:35:56 <mkarpiarz> Yes, this is the patch I'll be testing next.
14:36:18 <rafaelweingartner> thanks!
14:36:28 <rafaelweingartner> your help is much appreciatted =)
14:38:30 <mkarpiarz> I'm glad! :)
14:39:03 <rafaelweingartner> Besides that, we also have these patches that are smaller #link https://review.opendev.org/c/openstack/cloudkitty/+/894812 and #link https://review.opendev.org/c/openstack/cloudkitty/+/895166
14:39:08 <rafaelweingartner> they are smaller, but important
14:39:24 <rafaelweingartner> it might be interesting to review them and work to merge them for C release as well
14:44:12 <mkarpiarz> OK
14:44:24 <priteau> I will look at them soon.
14:45:14 <rafaelweingartner> thanks!
14:45:27 <rafaelweingartner> moving on, we have the last topic
14:45:31 <rafaelweingartner> #topic SQLAlchemy 2.x / oslo.db 14.0.0 compatibility
14:45:36 <rafaelweingartner> I guess this one is with Pierre
14:45:46 <rafaelweingartner> if you need some help, just let us know
14:47:49 <rafaelweingartner> priteau: do you have an update on this topic?
14:47:59 <priteau> Not yet, OpenSearch is the first priority
14:48:10 <priteau> SQLAlchemy will be a longer term goal for C release
14:48:59 <rafaelweingartner> I see
14:49:05 <rafaelweingartner> thanks, for the update
14:49:12 <rafaelweingartner> well.. that was all from my side
14:49:16 <rafaelweingartner> do you guys have something else to add?
14:50:16 <mkarpiarz> I also run into this error but I think this is because my test environment is still mostly on Zed:
14:50:23 <mkarpiarz> > This version of MariaDB doesn't yet support 'GET_LOCK in cluster (WSREP_ON=ON)'
14:50:40 <rafaelweingartner> hmm
14:50:56 <rafaelweingartner> I guess that is related to the DB, but not CloudKitty side
14:51:03 <rafaelweingartner> We have never seen this before
14:51:09 <priteau> I haven't seen this either.
14:51:36 <priteau> rafaelweingartner: I have a few issues to raise
14:52:07 <rafaelweingartner> sure
14:52:52 <priteau> First, I discovered that yoga Zuul gate is broken because of lower-constraints
14:53:18 <priteau> The simplest approach is to drop the job like we did in Zed: https://review.opendev.org/c/openstack/cloudkitty/+/898226
14:53:20 <rafaelweingartner> I saw the patch
14:53:28 <priteau> mkarpiarz: could you please review?
14:54:30 <priteau> I also submitted a couple of fixes discovered while trying to use the reprocessing feature on yoga
14:54:44 <mkarpiarz> Sure, I'll approve this one and merge.
14:54:47 <priteau> Error 500 while updating scope: https://review.opendev.org/c/openstack/cloudkitty/+/898217
14:55:04 <priteau> Error 500 while retrieving reprocessing tasks: https://review.opendev.org/c/openstack/cloudkitty/+/898207
14:55:30 <rafaelweingartner> yes, I reviewed them
14:55:40 <rafaelweingartner> they were alreayd fixed in upstream versions
14:55:48 <priteau> Thanks rafaelweingartner. This is mostly for mkarpiarz for a second +2.
14:55:54 <priteau> Related to the last one, I would also like to change the client so it sends the lowercase order.
14:55:58 <priteau> What do you think?
14:56:07 <mkarpiarz> Unless you only want my +2 and would prefer someone else have a look before we merge.
14:56:33 <rafaelweingartner> I agree, we do not use it with the client
14:56:45 <rafaelweingartner> I mean, we are using the API directly, that is why we never had this issue before
14:56:50 <priteau> I see
14:57:31 <priteau> Also I should note that I didn't manage to successfully use the reprocessing feature. It worked for some timestamps but other raised an exception in the storage backend (elasticsearch)
14:57:50 <priteau> I ended up scripting HTTP calls to delete the Elasticsearch documents directly.
14:57:52 <rafaelweingartner> what error?
14:58:11 <priteau> I don't have it available now but I will share it on the bug tracker
14:58:25 <rafaelweingartner> ok, thanks
14:58:30 <rafaelweingartner> maybe, it has a relation with https://review.opendev.org/c/openstack/cloudkitty/+/894812
14:59:03 <rafaelweingartner> we had some issues in InfluxDB, as it was not designed to work nicely with a bunch of deletes, and then we created a batch delete process in the reprocessing feature
14:59:45 <priteau> I don't think it was the same issue
14:59:53 <rafaelweingartner> I see
15:01:21 <priteau> I will share more information before the PTG
15:01:39 <rafaelweingartner> Ok, thanks!
15:01:42 <priteau> mkarpiarz: Also (not from me): https://review.opendev.org/c/openstack/cloudkitty/+/896159
15:02:13 <rafaelweingartner> thanks
15:02:16 <rafaelweingartner> I forgot that one =)
15:03:29 <rafaelweingartner> Is that all folks?
15:06:09 <mkarpiarz> Thanks for the reminder, priteau!
15:06:35 <mkarpiarz> That'll be all from my side.
15:06:42 <rafaelweingartner> Thanks
15:06:47 <rafaelweingartner> I will have to close
15:06:48 <rafaelweingartner> Thank you guys for participating. Have a nice week.
15:06:52 <rafaelweingartner> #endmeeting