| opendevreview | Matt Crees proposed openstack/cloudkitty master: Install OpenSearch 2.5.0 using packages https://review.opendev.org/c/openstack/cloudkitty/+/873101 | 10:52 |
|---|---|---|
| rafaelweingartner | #startmeeting cloudkitty | 14:00 |
| opendevmeet | Meeting started Mon Apr 17 14:00:42 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 | Hello guys! | 14:00 |
| rafaelweingartner | Roll count | 14:00 |
| mkarpiarz | Hi! | 14:01 |
| priteau | o/ | 14:02 |
| priteau | rafaelweingartner: Let's start? | 14:07 |
| rafaelweingartner | #topic vPTG meeting | 14:07 |
| rafaelweingartner | Thank you all for participating in the vPTG meeting! We had an interesting session regarding the next steps we will be taken into CloudKitty, and I hope we can make it better every new release! | 14:07 |
| rafaelweingartner | Again, thank you all for participating. | 14:07 |
| rafaelweingartner | #topic Target reviews | 14:11 |
| rafaelweingartner | #link https://review.opendev.org/c/openstack/cloudkitty/+/861786, this one seems ready for merging. What do you guys think? | 14:11 |
| priteau | Can you remind me why you went with time-d/time-w, but also month/year | 14:12 |
| rafaelweingartner | Not a specific reason | 14:13 |
| rafaelweingartner | one is what we expose in the API, and the other is just the attribute name in the backend | 14:14 |
| rafaelweingartner | would you like to change the pattern? | 14:14 |
| priteau | Wouldn't it make more sense to use "daily", "weekly" or "day", "week" | 14:16 |
| priteau | Also your commit message doesn't match your reno | 14:16 |
| priteau | Commit message says "year", reno says "time-y" | 14:16 |
| priteau | etc. | 14:16 |
| priteau | Can you please rework? | 14:16 |
| rafaelweingartner | Ok, thanks! | 14:16 |
| rafaelweingartner | I will do that | 14:16 |
| rafaelweingartner | Anyways., moving on | 14:18 |
| rafaelweingartner | #link https://review.opendev.org/c/openstack/cloudkitty/+/862824 seems ready for merging. What do you guys think? | 14:18 |
| mkarpiarz | I added my +2 back (to the latest patch). | 14:20 |
| rafaelweingartner | thanks! | 14:21 |
| priteau | Minor nit on reno, I will fix it myself | 14:21 |
| rafaelweingartner | ok | 14:22 |
| opendevreview | Pierre Riteau proposed openstack/cloudkitty master: Add description option to a rating metric definition https://review.opendev.org/c/openstack/cloudkitty/+/862824 | 14:22 |
| priteau | And kb wasn't fixed | 14:23 |
| opendevreview | Pierre Riteau proposed openstack/cloudkitty master: Add description option to a rating metric definition https://review.opendev.org/c/openstack/cloudkitty/+/862824 | 14:24 |
| priteau | I have to be afk for a few minutes | 14:24 |
| mkarpiarz | OK | 14:24 |
| rafaelweingartner | ok | 14:25 |
| rafaelweingartner | thanks! | 14:25 |
| rafaelweingartner | #link https://review.opendev.org/c/openstack/cloudkitty/+/861908 and #link https://review.opendev.org/c/openstack/cloudkitty/+/861806 need some extra work from our side. Therefore, no need to review them again for now. | 14:26 |
| priteau | Back | 14:26 |
| rafaelweingartner | thanks for the patches there! | 14:27 |
| rafaelweingartner | I thought we fixed the KB thing, but we left it lower case | 14:27 |
| rafaelweingartner | thanks for fixing that | 14:27 |
| mkarpiarz | Is there a reason we've not merged 861806 yet? | 14:29 |
| rafaelweingartner | good question | 14:29 |
| rafaelweingartner | I might have mixed it with some other patch | 14:29 |
| rafaelweingartner | Looking at it, it should have already been merged | 14:30 |
| mkarpiarz | Let's do it then. :) | 14:31 |
| priteau | Well I only gave my +2 today | 14:31 |
| mkarpiarz | Ah! | 14:31 |
| rafaelweingartner | ah no | 14:32 |
| rafaelweingartner | wait | 14:32 |
| rafaelweingartner | there is a reason | 14:32 |
| rafaelweingartner | that one needs the introduction page that Pierre requested because of the expression "rating type" | 14:33 |
| priteau | Yes, I saw it wasn't added yet. If you are still planning to do so please do | 14:33 |
| rafaelweingartner | yes, we are | 14:34 |
| rafaelweingartner | So, moving one we have the following that might be ready for merging: #link https://review.opendev.org/c/openstack/cloudkitty/+/877525 is a simple one that can be merged. What do you guys think? | 14:34 |
| priteau | Merging | 14:35 |
| rafaelweingartner | thanks | 14:35 |
| mkarpiarz | Ah, yes, I should have re-applied my +2 here as well. :) | 14:36 |
| rafaelweingartner | and last, but not least: #link https://review.opendev.org/c/openstack/cloudkitty/+/877537 can be merged. We just need another core review approval. | 14:36 |
| priteau | mkarpiarz: Please check this one and merge if you're happy | 14:36 |
| rafaelweingartner | and that basically finishes my list here. Do you guys have something else to add? | 14:37 |
| priteau | Just letting you know that we are making progress with OpenSearch v2 | 14:37 |
| priteau | But still needs more work | 14:37 |
| rafaelweingartner | ok | 14:37 |
| rafaelweingartner | thanks! | 14:37 |
| mkarpiarz | Awesome! | 14:38 |
| mkarpiarz | I believe I already added OpenSearch to my test environemnt so you guys just let us know when we can start testing your changes. | 14:40 |
| priteau | Sure, thanks | 14:40 |
| priteau | Something else to discuss: there was a call for participation to the physical PTG in Vancouver. I assume we are not participating? | 14:40 |
| rafaelweingartner | I would say not | 14:42 |
| rafaelweingartner | Unless, if it is mandatory | 14:42 |
| priteau | No it isn't | 14:43 |
| rafaelweingartner | Now, I open for general questions and topics that people might have. | 14:43 |
| rafaelweingartner | #topic Open floor moment | 14:43 |
| rafaelweingartner | Do you guys have something else to add? | 14:43 |
| mkarpiarz | Nothing from my side. | 14:44 |
| priteau | Nothing else. | 14:44 |
| rafaelweingartner | Thank you guys for participating. Have a nice week. | 14:46 |
| rafaelweingartner | #endmeeting | 14:46 |
| opendevmeet | Meeting ended Mon Apr 17 14:46:10 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:46 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cloudkitty/2023/cloudkitty.2023-04-17-14.00.html | 14:46 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cloudkitty/2023/cloudkitty.2023-04-17-14.00.txt | 14:46 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cloudkitty/2023/cloudkitty.2023-04-17-14.00.log.html | 14:46 |
| opendevreview | Merged openstack/cloudkitty master: Improve reprocessing task documentation https://review.opendev.org/c/openstack/cloudkitty/+/877525 | 15:07 |
| opendevreview | Merged openstack/cloudkitty master: Remove `state` field from API https://review.opendev.org/c/openstack/cloudkitty/+/877537 | 15:25 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!