| rafaelweingartner | #startmeeting cloudkitty | 14:00 |
|---|---|---|
| opendevmeet | Meeting started Mon May 12 14:00:20 2025 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 |
| rafaelweingartner | \o | 14:00 |
| priteau | o/ | 14:01 |
| MattCrees[m] | o/ | 14:03 |
| rafaelweingartner | nice to see you guys here :) | 14:06 |
| rafaelweingartner | #topic Target reviews | 14:06 |
| rafaelweingartner | #link https://review.opendev.org/c/openstack/cloudkitty/+/947908, this one is in the making for a long time, and it was ready, but now some tests started breaking | 14:07 |
| rafaelweingartner | we do not clearly understand why yet | 14:07 |
| rafaelweingartner | sorry, wrong comment on this one :) | 14:08 |
| rafaelweingartner | it was meant to another patch. | 14:08 |
| rafaelweingartner | This patch #947908, is one that priteau requested some changes, and then the tests started failing | 14:08 |
| priteau | Good news is that the Prometheus plugin for DevStack exists elsewhere | 14:12 |
| rafaelweingartner | exactly | 14:12 |
| priteau | And indeed, bad news is that the jobs are failing. Might be a more general issue in CloudKitty following the reopening of the master branch? | 14:12 |
| rafaelweingartner | I am not sure | 14:13 |
| rafaelweingartner | maybe those issues regarding library updates | 14:13 |
| rafaelweingartner | I just faced some issues in Gnocchi | 14:13 |
| rafaelweingartner | Last week when opening a patch, due to Gabbits upgrade and oslo.middleware upgrade as well | 14:13 |
| rafaelweingartner | maybe it is something similar | 14:13 |
| MattCrees[m] | Details: b'Internal Server Error'... Not a very helpful of error, looks like it will need more in-depth debugging. | 14:14 |
| MattCrees[m] | I would guess some dependency has changed recently, given all the projects have been making releases for Flamingo | 14:14 |
| priteau | The logs show 500 internal server error when tempest tries to contact CK | 14:14 |
| priteau | e.g. | 14:14 |
| priteau | 10.0.17.190 - - [29/Apr/2025:11:40:45 +0000] "POST /rating/v1/collector/mappings/ HTTP/1.1" 500 231 "-" "python-urllib3/1.26.20" | 14:14 |
| priteau | 10.0.17.190 - - [29/Apr/2025:11:40:45 +0000] "GET /rating/v1/collector/mappings/ HTTP/1.1" 500 231 "-" "python-urllib3/1.26.20" | 14:14 |
| priteau | https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_66e/openstack/66e14448cada460fad8913caa7cc78f7/controller/logs/screen-ck-api.txt | 14:15 |
| priteau | failed to open python file /opt/stack/data/venv/bin/cloudkitty-api | 14:15 |
| rafaelweingartner | gabbits change the way they integrate/work with wsgi | 14:15 |
| priteau | Not sure it is gabbit in this case, maybe some change in devstack | 14:16 |
| rafaelweingartner | they were causing an issue that Gnocchi-api was not working. I did not look into the root cause, and just pinned Gabbits version | 14:16 |
| priteau | Someone should try running devstack manually | 14:17 |
| rafaelweingartner | yes | 14:17 |
| priteau | I will test it locally | 14:20 |
| rafaelweingartner | thanks for taking this one priteau! | 14:21 |
| rafaelweingartner | Moving on, we have #link https://review.opendev.org/c/openstack/cloudkitty/+/876643, which it the patch that has been in the making for a while | 14:23 |
| rafaelweingartner | it has some test failures which seem to be related to the previous one that we discussed | 14:24 |
| priteau | Let's focus on fixing the general issue first, and come back to this one soon | 14:25 |
| mattcrees_ | I have started testing this patch myself, but I still need to spend more time on it. Haven't found any issues with how it functions yet though | 14:25 |
| rafaelweingartner | no worries | 14:27 |
| rafaelweingartner | if you have questions/doubts or any other need for help, just let me know | 14:27 |
| mattcrees_ | will do :) | 14:29 |
| rafaelweingartner | besides those, I noticed this one that I do not follow: #link https://review.opendev.org/c/openstack/cloudkitty/+/948480. | 14:30 |
| rafaelweingartner | Do you guys what this one is about? | 14:30 |
| priteau | It's a minor change, I think we can approve it if gmaan is OK with it | 14:31 |
| mattcrees_ | The same kind of change has been made in many projects so it looks fine to me | 14:32 |
| rafaelweingartner | ok, thanks | 14:33 |
| mattcrees_ | Actually, it looks like there's disagreements in the placement patch. We should probably wait to see how that pans out: https://review.opendev.org/c/openstack/placement/+/941792/comment/2cf097e3_61949354/ | 14:34 |
| rafaelweingartner | I did not understand the context | 14:34 |
| rafaelweingartner | that is why I did not manifested before | 14:34 |
| priteau | It's related to metadata about OpenStack governance | 14:36 |
| priteau | I think we can ignore it until gmaan provides a new opinion | 14:36 |
| rafaelweingartner | ok | 14:37 |
| rafaelweingartner | Moving on, we have #link https://review.opendev.org/c/openstack/cloudkitty/+/946330 | 14:39 |
| rafaelweingartner | which seems some basic info being added to CloudKitty doces | 14:39 |
| priteau | Jobs failed but the logs are gone now | 14:40 |
| priteau | I have rechecked | 14:40 |
| rafaelweingartner | me too :) | 14:40 |
| rafaelweingartner | let's see the output and then we can vote/evalute the patch | 14:41 |
| rafaelweingartner | and last, but not least, we have #link https://review.opendev.org/c/openstack/cloudkitty/+/946336 | 14:43 |
| rafaelweingartner | I am not sure if this is needed, it has been a while that I do not install from scratch, but I would expect for these tables to be created by the system automaticaly already | 14:43 |
| rafaelweingartner | I would need to test this one indepth to understand what happened | 14:44 |
| priteau | I had not realised we had a bug there | 14:44 |
| rafaelweingartner | me neither | 14:45 |
| priteau | It doesn't look right though, it is creating the cloudkitty_storage_states table, which should already exist | 14:45 |
| rafaelweingartner | and we use this feature quite a lot | 14:45 |
| priteau | I think it's an operator issue from zigo | 14:46 |
| priteau | The reprocessing was added in cloudkitty/storage_state/alembic/versions/9feccd32_create_reprocessing_scheduler.py | 14:46 |
| priteau | We have two separate DB migrations | 14:46 |
| priteau | Maybe a ran just one | 14:47 |
| zigo | o/ | 14:47 |
| zigo | It's a mistake, I should abandon this patch, I believe. | 14:47 |
| rafaelweingartner | =) | 14:48 |
| zigo | These are created when doing init-storage. | 14:48 |
| zigo | My init-storage was simply crashing ... | 14:48 |
| * zigo just abandonned the patch. | 14:48 | |
| priteau | Maybe we are missing something in our grenade tests though | 14:48 |
| rafaelweingartner | what do you mean? | 14:49 |
| zigo | Well, I don't really understand why this has been separated form the normal migrations, though. | 14:49 |
| priteau | zigo: Neither do we :) Historical decisions from Objectif Libre | 14:49 |
| rafaelweingartner | exactly | 14:50 |
| rafaelweingartner | we just followed the process already in-place :) | 14:50 |
| zigo | Maybe just done when creating the v2 storage ? | 14:50 |
| zigo | It'd be probably a good idea to "repair" the current state, but nothing urgent, IMO. | 14:50 |
| priteau | rafaelweingartner: I mean that in grenade code, the upgrade_cloudkitty_database function only calls $CLOUDKITTY_BIN_DIR/cloudkitty-dbsync upgrade | 14:51 |
| priteau | Maybe it needs to call the cloudkitty-storage-init binary too | 14:51 |
| rafaelweingartner | I see | 14:51 |
| zigo | Indeed. | 14:55 |
| rafaelweingartner | And that is basically all from my side | 14:56 |
| rafaelweingartner | do you guys have something else to add? | 14:57 |
| priteau | Nothing from me | 14:58 |
| mattcrees_ | Nothing from me either | 14:58 |
| rafaelweingartner | Ok | 14:59 |
| rafaelweingartner | thank you all for participating. Have a nice week! | 14:59 |
| mattcrees_ | Thanks rafaelweingartner :) | 14:59 |
| priteau | Thanks | 15:00 |
| rafaelweingartner | #endmeeting | 15:00 |
| opendevmeet | Meeting ended Mon May 12 15:00:04 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:00 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cloudkitty/2025/cloudkitty.2025-05-12-14.00.html | 15:00 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cloudkitty/2025/cloudkitty.2025-05-12-14.00.txt | 15:00 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cloudkitty/2025/cloudkitty.2025-05-12-14.00.log.html | 15:00 |
| gmaan | priteau: on 948480. I am +1. this change needed as it is linked to the non exist TC tag framework link. | 15:23 |
| priteau | Thanks gmaan! | 15:30 |
| priteau | gmaan: Should we still link to the svg though? It doesn't appear to render correctly: https://governance.openstack.org/tc/badges/cloudkitty.svg | 15:31 |
| gmaan | priteau: yeah, that is one of the other thing we should fix also, current change only remove the non exist link from project side and somepoint we should check if those svg links can be directly point to the actual one instead of one more click | 15:33 |
| gmaan | currently, you can go to actual valid links 1. OpenStack official project 2. OpenSSF best practices badge from svg file | 15:33 |
| opendevreview | Pierre Riteau proposed openstack/cloudkitty master: [WIP] Fix wrong name of cloudkitty-api WSGI script https://review.opendev.org/c/openstack/cloudkitty/+/949526 | 16:49 |
| opendevreview | Merged openstack/cloudkitty master: Remove tags from README https://review.opendev.org/c/openstack/cloudkitty/+/948480 | 17:05 |
| opendevreview | Pierre Riteau proposed openstack/cloudkitty master: Add wsgi module to cloudkitty https://review.opendev.org/c/openstack/cloudkitty/+/949526 | 21:16 |
| opendevreview | Pierre Riteau proposed openstack/cloudkitty master: Add wsgi module to cloudkitty and remove wsgi script https://review.opendev.org/c/openstack/cloudkitty/+/949526 | 21:36 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
