| rafaelweingartner | Hello guys! | 13:56 |
|---|---|---|
| priteau | Hello | 13:58 |
| rafaelweingartner | #startmeeting cloudkitty | 14:00 |
| opendevmeet | Meeting started Mon Jan 23 14:00:14 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 |
| 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 | Roll count | 14:00 |
| mkarpiarz | Hi! | 14:00 |
| rafaelweingartner | \O | 14:00 |
| priteau | o/ | 14:00 |
| rafaelweingartner | #topic add CloudKitty API reference docs | 14:01 |
| rafaelweingartner | mkarpiarz: are we missing something to finish this item? | 14:01 |
| mkarpiarz | Something is still missing and I'm not sure what. | 14:04 |
| mkarpiarz | `tox -e api-ref` works just fine when run locally | 14:04 |
| mkarpiarz | And yet, https://docs.openstack.org/api-ref/rating/index.html is not generated. | 14:05 |
| mkarpiarz | There is one unmerged change here: https://review.opendev.org/c/openstack/openstack-manuals/+/867357 | 14:07 |
| rafaelweingartner | that seems to be the issue | 14:07 |
| priteau | It already failed a few days ago | 14:08 |
| priteau | https://zuul.opendev.org/t/openstack/build/57c9416614b94930bf618e8c737e624b | 14:08 |
| priteau | We may need help from the maintainers of this repository | 14:08 |
| rafaelweingartner | I would say so | 14:09 |
| rafaelweingartner | it seems that the job is expecting something to exist | 14:09 |
| rafaelweingartner | maybe it is another step that we need to take | 14:10 |
| mkarpiarz | Maintainers keep referring to this page: https://docs.openstack.org/doc-contrib-guide/api-guides.html | 14:10 |
| mkarpiarz | Please let me know if I missed a step! | 14:10 |
| mkarpiarz | My understanding is this patch should have triggered a build of API docs: https://review.opendev.org/c/openstack/project-config/+/867651 | 14:11 |
| rafaelweingartner | mkarpiarz: can you contact the maitainer of the system to see what is missing? | 14:11 |
| mkarpiarz | I've done this already and, like I said, was told to follow the docs page. | 14:12 |
| mkarpiarz | From my perspective all requirements are in place and the problem is somewhere in the plumbing. | 14:13 |
| mkarpiarz | Until I'm told what is missing exactly, I can't progres any further. | 14:14 |
| priteau | Maybe something is broken in their pipeline. | 14:15 |
| priteau | Have you tried talking with them directly on IRC? | 14:15 |
| rafaelweingartner | Or, maybe, the trigger has not been executed | 14:16 |
| rafaelweingartner | those definitions are the project pipelines executed by Zuul | 14:16 |
| rafaelweingartner | they might only be executed during merges or maybe tags | 14:17 |
| rafaelweingartner | we did not do any of those since that was merged, right? | 14:17 |
| priteau | I don't think so | 14:18 |
| rafaelweingartner | lets see if we have something to merge today then | 14:19 |
| mkarpiarz | OK | 14:19 |
| rafaelweingartner | Mariusz thanks for the effort on this one so far ! | 14:20 |
| rafaelweingartner | #topic Target review of the meeting | 14:21 |
| rafaelweingartner | Starting the the spec: https://review.opendev.org/c/openstack/cloudkitty-specs/+/866198 | 14:22 |
| rafaelweingartner | I see that Mariusz has already approved. Have you had time to take a look into the proposal priteau? | 14:22 |
| priteau | Sorry I haven't had the chance to look at it, it's been a busy period. I can make some time this week. | 14:24 |
| rafaelweingartner | ok | 14:26 |
| rafaelweingartner | thanks! | 14:26 |
| rafaelweingartner | that would be a very interesting feature for us to have in CloudKitty | 14:27 |
| rafaelweingartner | Lets wait for you then | 14:27 |
| rafaelweingartner | So, moving on. The patch #link https://review.opendev.org/c/openstack/cloudkitty/+/864269 | 14:28 |
| rafaelweingartner | I amended it as suggested by priteau, and Mariusz has given his blessing | 14:28 |
| rafaelweingartner | Can we move on with its merge? | 14:28 |
| priteau | I will make a small cosmetic change | 14:29 |
| priteau | Faster than asking you to update | 14:29 |
| rafaelweingartner | ok | 14:30 |
| rafaelweingartner | thanks | 14:30 |
| rafaelweingartner | sorry for missing these details | 14:30 |
| opendevreview | Pierre Riteau proposed openstack/cloudkitty master: Optimize Gnocchi fetcher https://review.opendev.org/c/openstack/cloudkitty/+/864269 | 14:30 |
| rafaelweingartner | Thanks! | 14:31 |
| rafaelweingartner | Moving on, we have this other patch, #link: https://review.opendev.org/c/openstack/cloudkitty/+/862824 | 14:32 |
| rafaelweingartner | mkarpiarz: would you like to do the honors on this one: https://review.opendev.org/c/openstack/cloudkitty/+/864269? | 14:33 |
| mkarpiarz | Done. | 14:34 |
| rafaelweingartner | The patch #link https://review.opendev.org/c/openstack/cloudkitty/+/862824 was approved by Mariusz, and we had that discussion regarding the optimization of the attributes stored in the backend. | 14:34 |
| priteau | Looking now | 14:36 |
| rafaelweingartner | awesome! | 14:39 |
| rafaelweingartner | thanks! | 14:39 |
| rafaelweingartner | I will amend those comments after the meeting then | 14:39 |
| rafaelweingartner | After that, we have this new patch #link: https://review.opendev.org/c/openstack/cloudkitty/+/871178 | 14:40 |
| rafaelweingartner | It seems to be ready to be merged | 14:40 |
| rafaelweingartner | it is a small patch | 14:40 |
| rafaelweingartner | To be that patch is ready to go | 14:46 |
| rafaelweingartner | Let's see if you guys can review it | 14:46 |
| rafaelweingartner | We also have these others #link https://review.opendev.org/c/openstack/cloudkitty/+/861786 and #link https://review.opendev.org/c/openstack/cloudkitty/+/861806, which would be interesting to have a review round to see if we can move on with them | 14:47 |
| rafaelweingartner | The same goes for #link: https://review.opendev.org/c/openstack/cloudkitty/+/864578 | 14:47 |
| priteau | Approved the RPC patch | 14:48 |
| rafaelweingartner | the patch #link https://review.opendev.org/c/openstack/cloudkitty/+/864578 is approved by Mariusz already | 14:48 |
| priteau | I am merging PyScripts now. We should backport too. | 14:48 |
| rafaelweingartner | awesome | 14:49 |
| rafaelweingartner | thanks! | 14:49 |
| rafaelweingartner | Having gone through the target reviews, I open now for the general discussions momment | 14:53 |
| rafaelweingartner | #topic Open Floor moment | 14:53 |
| rafaelweingartner | Do you guys have something extra that we need to discuss? | 14:54 |
| mkarpiarz | Nothing from my side | 14:55 |
| rafaelweingartner | Thank you guys for participating. Have a nice week. | 14:58 |
| rafaelweingartner | #endmeeting | 14:58 |
| opendevmeet | Meeting ended Mon Jan 23 14:58:16 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:58 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cloudkitty/2023/cloudkitty.2023-01-23-14.00.html | 14:58 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cloudkitty/2023/cloudkitty.2023-01-23-14.00.txt | 14:58 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cloudkitty/2023/cloudkitty.2023-01-23-14.00.log.html | 14:58 |
| rafaelweingartner | If you need me in some of the patches, just ping me | 14:58 |
| rafaelweingartner | :) | 14:58 |
| opendevreview | Merged openstack/cloudkitty master: Use new get_rpc_client API from oslo.messaging https://review.opendev.org/c/openstack/cloudkitty/+/871178 | 15:45 |
| opendevreview | Merged openstack/cloudkitty master: Fix PyScripts processing https://review.opendev.org/c/openstack/cloudkitty/+/864578 | 15:45 |
| opendevreview | Merged openstack/cloudkitty master: Optimize Gnocchi fetcher https://review.opendev.org/c/openstack/cloudkitty/+/864269 | 15:49 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!