| opendevreview | Merged openstack/cloudkitty master: Add detail about reprocessing all scope IDs in reprocessing API https://review.opendev.org/c/openstack/cloudkitty/+/877829 | 10:19 |
|---|---|---|
| opendevreview | Merged openstack/cloudkitty master: Fix formatting in release note https://review.opendev.org/c/openstack/cloudkitty/+/879164 | 13:17 |
| opendevreview | Rafael Weingartner proposed openstack/cloudkitty master: Remove `state` field from API https://review.opendev.org/c/openstack/cloudkitty/+/877537 | 13:27 |
| opendevreview | Pierre Riteau proposed openstack/cloudkitty master: Optimize Gnocchi fetcher processing time https://review.opendev.org/c/openstack/cloudkitty/+/872035 | 13:27 |
| opendevreview | Rafael Weingartner proposed openstack/cloudkitty master: Improve reprocessing task documentation https://review.opendev.org/c/openstack/cloudkitty/+/877525 | 13:29 |
| opendevreview | Rafael Weingartner proposed openstack/cloudkitty master: Remove `state` field from API https://review.opendev.org/c/openstack/cloudkitty/+/877537 | 13:42 |
| rafaelweingartner | Hello guys! | 14:00 |
| rafaelweingartner | #startmeeting cloudkitty | 14:00 |
| opendevmeet | Meeting started Mon Apr 3 14:00:05 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 |
| rafaelweingartner | \O | 14:00 |
| priteau | o/ | 14:03 |
| priteau | There was a message from Mariusz sent on another channel | 14:04 |
| priteau | "I may not be able to join our meeting today." | 14:04 |
| priteau | "I've approved as many patches as I can - the rest I'll have to try out in my test environment this week." | 14:04 |
| rafaelweingartner | ah ok | 14:04 |
| rafaelweingartner | I guess we can start then? | 14:05 |
| priteau | Yes. | 14:05 |
| rafaelweingartner | #topic vPTG meeting | 14:05 |
| rafaelweingartner | I just had our vPTG meeting last week, on Friday. Thank you guys for participating. | 14:05 |
| rafaelweingartner | I will be writing up an e-mail to summarize our discussions and the direction we will be working under B release of OpenStack this week. | 14:05 |
| priteau | Thank you for leading the meeting. | 14:06 |
| rafaelweingartner | Welcome! It has been a pleasure to be working with you guys for all of these years! | 14:09 |
| rafaelweingartner | So, moving on to the target reviews | 14:09 |
| rafaelweingartner | #topic Target reviews | 14:09 |
| rafaelweingartner | #link https://review.opendev.org/c/openstack/cloudkitty/+/861786, this patch was missing Gabbi tests, which I created. | 14:09 |
| rafaelweingartner | Can you guys reconsider it? | 14:09 |
| priteau | I was reviewing patches before the meeting but didn't reach this one yet | 14:10 |
| rafaelweingartner | ah ok | 14:10 |
| priteau | I will check this week, we need Mariusz's vote too | 14:10 |
| rafaelweingartner | yes | 14:10 |
| rafaelweingartner | ok | 14:10 |
| rafaelweingartner | Moiving on, we have #link https://review.opendev.org/c/openstack/cloudkitty/+/861908, and #link https://review.opendev.org/c/openstack/cloudkitty/+/861806. These two patches will require some work, one requires some additions to the docs. And the other requires some tempest tests. | 14:11 |
| rafaelweingartner | Then, we have #link https://review.opendev.org/c/openstack/cloudkitty/+/865435, which seems to be ready for merging. What do you think? | 14:13 |
| priteau | I thought we deprecated SQL storage? | 14:15 |
| priteau | We can still merge it of course. | 14:19 |
| priteau | Approved | 14:19 |
| rafaelweingartner | hmm | 14:24 |
| rafaelweingartner | it seems that I got disconnected | 14:24 |
| rafaelweingartner | are you guys still there? | 14:24 |
| priteau | I am here. | 14:24 |
| rafaelweingartner | so, we were discussing the V1 API | 14:25 |
| rafaelweingartner | I asked if there was a deprecation/removal note in the docs | 14:26 |
| rafaelweingartner | then, it seems that I got disconnected | 14:26 |
| rafaelweingartner | Anyways, moving on, we have the patch #link: https://review.opendev.org/c/openstack/cloudkitty/+/862824, which also seem to be ready for merging. | 14:27 |
| rafaelweingartner | It is related to the field vs. tag situation in Influx, but we will address that in a different patch, which will affect all attributes that are persisted today as fields in InfluxDB | 14:27 |
| rafaelweingartner | We also have the patch #link: https://review.opendev.org/c/openstack/cloudkitty/+/877525, which is already reviewed by priteau. Therefore, we only need Mariusz blessing there to proceed with merging. | 14:31 |
| rafaelweingartner | And finally, but not least, we have the patch #link https://review.opendev.org/c/openstack/cloudkitty/+/877537, which is the final procees of the `state` field deprecation/removal. | 14:32 |
| rafaelweingartner | That is all of the patch list I had here. Do you guys have something else to add? | 14:32 |
| opendevreview | Rafael Weingartner proposed openstack/cloudkitty master: Add description option to a rating metric definition https://review.opendev.org/c/openstack/cloudkitty/+/862824 | 14:35 |
| priteau | Did we decide what to do about "rating types"? | 14:35 |
| priteau | Are we putting it in the docs? | 14:35 |
| opendevreview | Rafael Weingartner proposed openstack/cloudkitty master: Remove `state` field from API https://review.opendev.org/c/openstack/cloudkitty/+/877537 | 14:36 |
| rafaelweingartner | yes, I just talked with Pedro | 14:36 |
| rafaelweingartner | we will be writing up a documentation to explain these concepts | 14:36 |
| rafaelweingartner | and then, in the review, we can discuss the better/proper term to use | 14:37 |
| priteau | Thank you | 14:40 |
| rafaelweingartner | Welcome! Do you guys have anything ot add? | 14:42 |
| rafaelweingartner | Otherwise, we might be able to close a bit earlier today | 14:42 |
| priteau | Nothing else. | 14:49 |
| rafaelweingartner | Then, thank you guys for participating. Have a nice week. | 14:49 |
| rafaelweingartner | #endmeeting | 14:50 |
| opendevmeet | Meeting ended Mon Apr 3 14:50:01 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:50 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cloudkitty/2023/cloudkitty.2023-04-03-14.00.html | 14:50 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cloudkitty/2023/cloudkitty.2023-04-03-14.00.txt | 14:50 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cloudkitty/2023/cloudkitty.2023-04-03-14.00.log.html | 14:50 |
| opendevreview | Merged openstack/cloudkitty master: Optimize Gnocchi fetcher processing time https://review.opendev.org/c/openstack/cloudkitty/+/872035 | 14:59 |
| opendevreview | Merged openstack/cloudkitty master: Create indexes to allow SQL rewrites and optimizations https://review.opendev.org/c/openstack/cloudkitty/+/865435 | 15:00 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!