| 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 | 09:52 |
|---|---|---|
| opendevreview | Merged openstack/cloudkitty master: Update master for stable/2023.2 https://review.opendev.org/c/openstack/cloudkitty/+/895110 | 11:21 |
| opendevreview | Merged openstack/cloudkitty master: Optimizing SQL queries that filter on a time range https://review.opendev.org/c/openstack/cloudkitty/+/865417 | 11:57 |
| opendevreview | Merged openstack/cloudkitty-dashboard master: Update master for stable/2023.2 https://review.opendev.org/c/openstack/cloudkitty-dashboard/+/895111 | 12:18 |
| opendevreview | Merged openstack/cloudkitty master: Remove cloudkitty.conf.sample from docs https://review.opendev.org/c/openstack/cloudkitty/+/895171 | 13:48 |
| opendevreview | Pierre Riteau proposed openstack/cloudkitty master: Clean up release note https://review.opendev.org/c/openstack/cloudkitty/+/895685 | 13:49 |
| opendevreview | Rafael Weingartner proposed openstack/cloudkitty master: Fix a concurrency issue when locking reprocessing tasks https://review.opendev.org/c/openstack/cloudkitty/+/894810 | 13:57 |
| opendevreview | Rafael Weingartner proposed openstack/cloudkitty master: Patch for `use_all_resource_revisions` option https://review.opendev.org/c/openstack/cloudkitty/+/895166 | 14:00 |
| priteau | Hello mkarpiarz | 14:01 |
| rafaelweingartner | #startmeeting cloudkitty | 14:02 |
| opendevmeet | Meeting started Mon Sep 18 14:02:09 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 | Hello guys! | 14:02 |
| rafaelweingartner | Roll count | 14:02 |
| rafaelweingartner | \0 | 14:02 |
| priteau | o/ | 14:02 |
| mkarpiarz | Hi! | 14:03 |
| rafaelweingartner | I am having some issues with my Gnone | 14:04 |
| rafaelweingartner | just a second, I guess I will need to reconnect | 14:04 |
| rafaelweingartner | I am back :) | 14:05 |
| mkarpiarz | OK | 14:05 |
| rafaelweingartner | #topic vPTG meeting | 14:05 |
| rafaelweingartner | Do not forget to register at: https://ptg2023.openinfra.dev/ | 14:05 |
| rafaelweingartner | The date is approaching | 14:06 |
| rafaelweingartner | We decided to pick Monday, the same timeslot as CloudKitty bi-weekly meeting to conduct the vPTG | 14:06 |
| rafaelweingartner | I will schedule the timeslot, as I guess it is possible now | 14:06 |
| rafaelweingartner | Are you guys ok with that? | 14:07 |
| mkarpiarz | I'm all set up and ready for that. :) | 14:08 |
| priteau | Yes, it looks like PTG website is clear now: https://ptg.opendev.org/ptg.html | 14:08 |
| priteau | rafaelweingartner: Do you think we could start at 1300 UTC? | 14:08 |
| rafaelweingartner | It fine by me | 14:09 |
| rafaelweingartner | what about you Mariusz? | 14:09 |
| priteau | Beside I may need to allocate another meeting after. | 14:09 |
| priteau | Are we planning one hour, two hours? | 14:09 |
| rafaelweingartner | I guess, one hour should be enough | 14:10 |
| rafaelweingartner | Last time, I think that we did not use two hours, right? | 14:10 |
| rafaelweingartner | It was a bit over one hour | 14:10 |
| mkarpiarz | Any time works for me. | 14:11 |
| mkarpiarz | And yes, I think we should allocate 2h. | 14:11 |
| priteau | Let's allocate 2 | 14:12 |
| rafaelweingartner | ok | 14:12 |
| rafaelweingartner | what is the channel that the PTG bot listens to? | 14:12 |
| priteau | #openinfra-events | 14:13 |
| rafaelweingartner | thanks | 14:15 |
| rafaelweingartner | done! | 14:15 |
| rafaelweingartner | So, moving on | 14:16 |
| rafaelweingartner | #topic OpenSearch support | 14:16 |
| rafaelweingartner | I guess this is still on hold, right priteau? | 14:16 |
| priteau | Still the same | 14:16 |
| rafaelweingartner | ok | 14:18 |
| rafaelweingartner | #topic Monasca deprecation | 14:18 |
| rafaelweingartner | The same for monasca | 14:18 |
| rafaelweingartner | the goal will be to finally remove it on C release | 14:18 |
| rafaelweingartner | An now, the target reviews | 14:18 |
| rafaelweingartner | #topic Target reviews | 14:18 |
| rafaelweingartner | We have some new patches | 14:18 |
| rafaelweingartner | I just would like to discuss the patch #link https://review.opendev.org/c/openstack/cloudkitty/+/861908 | 14:19 |
| rafaelweingartner | with you guys | 14:19 |
| rafaelweingartner | it was requested a tempest test for it | 14:19 |
| rafaelweingartner | but after investigating, wew noticed that there is no processor test in tempest | 14:19 |
| rafaelweingartner | therefore, we would need to build it from scratch, which seems a good amount of energy for little return now | 14:19 |
| rafaelweingartner | the code being changed has unit tests, therefore, it seems to be ok as is now | 14:20 |
| priteau | I assume it has been tested outside of CI? | 14:22 |
| priteau | I left some minor comments on the reno | 14:22 |
| priteau | OK to approve the patch itself if you are confident it works | 14:23 |
| rafaelweingartner | yes, it has | 14:24 |
| rafaelweingartner | ok, thanks! | 14:25 |
| rafaelweingartner | how do we make a hyperlink in that yml file of the release noteS? | 14:27 |
| priteau | Like this: | 14:27 |
| priteau | `LP#1875223 <https://launchpad.net/bugs/1875223>`__ | 14:28 |
| priteau | `text <url>`__ | 14:28 |
| opendevreview | Rafael Weingartner proposed openstack/cloudkitty master: Use correct metadata for metrics gathered from gnocchi https://review.opendev.org/c/openstack/cloudkitty/+/861908 | 14:29 |
| opendevreview | Rafael Weingartner proposed openstack/cloudkitty master: Fix a concurrency issue when locking reprocessing tasks https://review.opendev.org/c/openstack/cloudkitty/+/894810 | 14:30 |
| rafaelweingartner | I amended the patches you guys reviewed | 14:30 |
| rafaelweingartner | thanks! | 14:31 |
| rafaelweingartner | Before we talk about SQL Alchemy, do you guys want to discuss some other patch? | 14:31 |
| priteau | Are all the other patches listed in the meeting topics ready for review? | 14:32 |
| rafaelweingartner | yes | 14:33 |
| rafaelweingartner | they are | 14:33 |
| priteau | I will try to review a few more | 14:36 |
| rafaelweingartner | ok, thanks! | 14:37 |
| rafaelweingartner | And now, the SQL Alchemy | 14:38 |
| rafaelweingartner | #topic SQLAlchemy 2.x / oslo.db 14.0.0 compatibility | 14:38 |
| rafaelweingartner | I have tried using SQL Alchemy in the past with Pedro, but it broke so many places, that we just gave up | 14:39 |
| rafaelweingartner | maybe with support in oslo.db, it gets a bit easier | 14:39 |
| rafaelweingartner | priteau: the stage is yours. Do you have something to add? Or some experience with it? | 14:39 |
| priteau | Sorry, not much to add except what is in the mailing list discussion | 14:40 |
| opendevreview | Rafael Weingartner proposed openstack/cloudkitty master: Fix a concurrency issue when locking reprocessing tasks https://review.opendev.org/c/openstack/cloudkitty/+/894810 | 14:40 |
| priteau | rafaelweingartner: wheny ou say tried using in the past, do you mean in another project? | 14:40 |
| priteau | cloudkitty is already using oslo.db and sqlalchemy | 14:40 |
| rafaelweingartner | yes | 14:41 |
| rafaelweingartner | Gnocchi | 14:41 |
| rafaelweingartner | I mean, with Olso DB 14 | 14:41 |
| rafaelweingartner | I am not sure if it would break something, and/or if we would need to change much in Cloudkitty to adopt it | 14:41 |
| priteau | I don't think we have a failure trace on Gerrit? Only what zigo said on the mailing list | 14:42 |
| rafaelweingartner | exactly | 14:42 |
| rafaelweingartner | I looked to see if I would find some error, but I could not find any | 14:42 |
| priteau | I suppose we try a patch that depends on https://review.opendev.org/c/openstack/requirements/+/879743 | 14:43 |
| rafaelweingartner | maybe the problem is with this "sqlalchemy-migrate" | 14:44 |
| rafaelweingartner | that is being removed | 14:44 |
| rafaelweingartner | we use it, right? | 14:44 |
| opendevreview | Pierre Riteau proposed openstack/cloudkitty master: Bump oslo.db to 14.0.0 https://review.opendev.org/c/openstack/cloudkitty/+/895692 | 14:44 |
| priteau | Maybe. Let's see | 14:44 |
| priteau | I don't see sqlalchemy-migrate in our repo. We use alembic | 14:45 |
| rafaelweingartner | thanks! | 14:45 |
| rafaelweingartner | ah, so I confused things then | 14:45 |
| priteau | I can try working a bit on this and following what's going on in the community | 14:45 |
| priteau | You are have a lot on your plate | 14:46 |
| rafaelweingartner | Sure, if you need any help then, just let me know | 14:46 |
| rafaelweingartner | thanks for the help =) | 14:46 |
| rafaelweingartner | BTW, that hacking I needed to do because of Shinx | 14:47 |
| rafaelweingartner | do you know how to get people to create a release for https://opendev.org/x/wsme/tags? | 14:47 |
| rafaelweingartner | I pingede the owned of the project, but got no answer | 14:47 |
| rafaelweingartner | the patch has already been merged there | 14:48 |
| priteau | Have you tried stephenfin on IRC? | 14:48 |
| rafaelweingartner | I tried on email | 14:48 |
| rafaelweingartner | I am not a fan of using IRC... | 14:49 |
| priteau | Looks like they are active anyway, many patches updated just a few days ago: https://review.opendev.org/q/project:x/wsme+is:open | 14:49 |
| zigo | priteau: You don't have an error because of the pinning, your patch #895692 will show it ... | 14:49 |
| priteau | They probably will want to merge these patches before releasing | 14:50 |
| priteau | So give it a bit of time. | 14:50 |
| priteau | If no progress in two weeks let's try again | 14:50 |
| zigo | The current plan is to release 15.0.0 with the SQLA 2.x patches reverted. | 14:50 |
| zigo | And then in early C, re-merge ... | 14:50 |
| zigo | (see the chatlog from last release team meeting) | 14:51 |
| priteau | OK, thanks zigo | 14:52 |
| zigo | If you produce SQLA2.x / oslo.db patches not released in Stable, please do track them in the etherpad (sorry, I don't have the link to that SQLA2.x patch tracking etherpad, but it's somewhere...). | 14:53 |
| rafaelweingartner | thanks zigo! | 15:02 |
| rafaelweingartner | priteau and mkarpiarz: is that all for today? | 15:02 |
| priteau | Yes, thank you | 15:02 |
| rafaelweingartner | Thank you guys for participating. Have a nice week. | 15:03 |
| rafaelweingartner | #endmeeting | 15:03 |
| opendevmeet | Meeting ended Mon Sep 18 15:03:14 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:03 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cloudkitty/2023/cloudkitty.2023-09-18-14.02.html | 15:03 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cloudkitty/2023/cloudkitty.2023-09-18-14.02.txt | 15:03 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cloudkitty/2023/cloudkitty.2023-09-18-14.02.log.html | 15:03 |
| zigo | My pleasure. :P | 15:06 |
| 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 | 17:19 |
| 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:36 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!