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