| opendevreview | Pierre Riteau proposed openstack/cloudkitty master: Add requests to requirements https://review.opendev.org/c/openstack/cloudkitty/+/909368 | 08:32 |
|---|---|---|
| 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:51 |
| rafaelweingartner | #startmeeting cloudkitty | 14:00 |
| opendevmeet | Meeting started Mon Feb 19 14:00:32 2024 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:03 |
| rafaelweingartner | How are you mkarpiarz? | 14:04 |
| priteau | Hello rafaelweingartner | 14:05 |
| rafaelweingartner | Hello Priteau! | 14:05 |
| mkarpiarz | I'm great, thanks for asking! How are you guys? | 14:05 |
| priteau | Hi mkarpiarz | 14:05 |
| rafaelweingartner | I am fine here as well =) | 14:06 |
| rafaelweingartner | #topic Cloudkitty vPTG meeting | 14:06 |
| rafaelweingartner | I have registered our team for the meeting | 14:06 |
| rafaelweingartner | we just need to schedule the date and time for it now. However, the process to pick the time slot is not available yet | 14:07 |
| rafaelweingartner | anyways, when we have the opportunity, can we schedule for Monday? The same time that we have our usual meetings? | 14:08 |
| opendevreview | Pierre Riteau proposed openstack/cloudkitty master: Add support to InfluxDB v2 as storage backend https://review.opendev.org/c/openstack/cloudkitty/+/895204 | 14:08 |
| priteau | That would be April 8? | 14:08 |
| mkarpiarz | This sounds good to me. :) | 14:08 |
| rafaelweingartner | priteau: I guess so | 14:09 |
| priteau | That would work for me. | 14:09 |
| rafaelweingartner | awesome! | 14:10 |
| rafaelweingartner | does it work for you Mariusz? | 14:10 |
| mkarpiarz | Yes | 14:10 |
| rafaelweingartner | Ah, you already answered! | 14:10 |
| rafaelweingartner | =) | 14:10 |
| rafaelweingartner | Moving on, Pedro is woring on the SLURP testing process. | 14:11 |
| rafaelweingartner | #topic SLURP support | 14:11 |
| rafaelweingartner | To move on with the implementation, we need to merge patch #link https://review.opendev.org/c/openstack/cloudkitty/+/908785?usp=search | 14:11 |
| rafaelweingartner | Can you guys review it? And, if everything is ok, merge it? | 14:11 |
| mkarpiarz | The change looks good to me. | 14:12 |
| priteau | I had not seen this patch, taking a quick look | 14:13 |
| mkarpiarz | priteau: Feel free to merge when you are done. | 14:14 |
| priteau | Two of my colleagues are seeing issues with devstack too, but when CLOUDKITTY_USE_MOD_WSGI=true | 14:15 |
| rafaelweingartner | This one is fixing for situations when it is CLOUDKITTY_USE_MOD_WSGI=false | 14:16 |
| rafaelweingartner | which is the case for the Granade tests | 14:16 |
| rafaelweingartner | Thanks for the merge priteau! | 14:17 |
| priteau | We will investigate the devstack issues we are seeing. I am not seeing any problem in CI | 14:21 |
| rafaelweingartner | I see | 14:21 |
| rafaelweingartner | if you need any help, let us know | 14:21 |
| priteau | Thanks | 14:23 |
| rafaelweingartner | Moving on | 14:23 |
| rafaelweingartner | #topic Monasca deprecation | 14:23 |
| rafaelweingartner | priteau has approved the patch | 14:23 |
| rafaelweingartner | can you review it mkarpiarz? | 14:23 |
| rafaelweingartner | then, if you are ok with it, you can merge it =) | 14:24 |
| mkarpiarz | Done | 14:24 |
| rafaelweingartner | thanks! | 14:25 |
| rafaelweingartner | moving on, to our standard reviews | 14:25 |
| rafaelweingartner | #topic Target reviews | 14:25 |
| rafaelweingartner | #link https://review.opendev.org/c/openstack/cloudkitty/+/876643, this is the start/end timestamp for rating rules | 14:25 |
| rafaelweingartner | they are only depending on priteau | 14:26 |
| rafaelweingartner | what do you think? | 14:26 |
| rafaelweingartner | after we merge it, we can merge #link https://review.opendev.org/c/openstack/cloudkitty-tempest-plugin/+/892382 | 14:26 |
| priteau | I haven't reached this one yet ;( | 14:27 |
| rafaelweingartner | I see | 14:28 |
| rafaelweingartner | If you can try to check this week? Or maybe, we merge it, and then we work in any improvement if needed afterwards | 14:28 |
| priteau | I will try next week | 14:28 |
| priteau | Sorry, this week | 14:28 |
| rafaelweingartner | ok | 14:29 |
| rafaelweingartner | thanks! | 14:29 |
| rafaelweingartner | #link https://review.opendev.org/c/openstack/cloudkitty/+/895204, this is the InfluxDB patch. Did you check it mkarpiarz? | 14:30 |
| rafaelweingartner | Can we merge it? Or, maybe, you need/want some time to review it? | 14:30 |
| opendevreview | Pierre Riteau proposed openstack/cloudkitty master: Add support to InfluxDB v2 as storage backend https://review.opendev.org/c/openstack/cloudkitty/+/895204 | 14:31 |
| mkarpiarz | So I have the revision before the latest changes in my test environment and see no issues. | 14:31 |
| priteau | The latest changes are purely cosmetic | 14:31 |
| priteau | No real code change | 14:31 |
| mkarpiarz | Ah, OK, the in this case I'm happy to give this one my +2. :) | 14:32 |
| mkarpiarz | Do you also want me to merge this change? | 14:32 |
| priteau | I will merge it once CI has completed | 14:33 |
| priteau | I had broken pep8 by mistake | 14:33 |
| mkarpiarz | OK | 14:33 |
| rafaelweingartner | besides those, do you guys have something else to add? | 14:34 |
| rafaelweingartner | that seems to be all from my side here | 14:34 |
| mkarpiarz | Nothing from my side. | 14:35 |
| priteau | A small patch I submitted this morning: https://review.opendev.org/c/openstack/cloudkitty/+/909368 | 14:37 |
| priteau | Rather trivial | 14:37 |
| rafaelweingartner | yes, I already gae my +2 | 14:38 |
| priteau | Is there a reason you removed your W+1 mkarpiarz? | 14:41 |
| mkarpiarz | Sorry, did this mess up Zuul? | 14:42 |
| priteau | It's going through check jobs again | 14:42 |
| priteau | And it won't merge until someone adds Workflow+1 again | 14:43 |
| mkarpiarz | It looked to me like Zuul didn't vote but I just missed its +1. | 14:43 |
| mkarpiarz | Right, OK, my W+1 is back. | 14:43 |
| priteau | Thanks | 14:44 |
| priteau | That's all from me | 14:44 |
| rafaelweingartner | thanks | 14:46 |
| rafaelweingartner | that is all from me as well | 14:46 |
| rafaelweingartner | Thank you guys for participating. Have a nice week. | 14:46 |
| rafaelweingartner | #endmeeting | 14:46 |
| opendevmeet | Meeting ended Mon Feb 19 14:46:41 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:46 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cloudkitty/2024/cloudkitty.2024-02-19-14.00.html | 14:46 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cloudkitty/2024/cloudkitty.2024-02-19-14.00.txt | 14:46 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cloudkitty/2024/cloudkitty.2024-02-19-14.00.log.html | 14:46 |
| opendevreview | Merged openstack/cloudkitty master: Fix devstack runprocess for cloudkitty api https://review.opendev.org/c/openstack/cloudkitty/+/908785 | 15:01 |
| opendevreview | Merged openstack/cloudkitty master: Removal of Monasca fetcher and collector https://review.opendev.org/c/openstack/cloudkitty/+/908833 | 15:12 |
| opendevreview | Merged openstack/cloudkitty master: Add support to InfluxDB v2 as storage backend https://review.opendev.org/c/openstack/cloudkitty/+/895204 | 16:03 |
| opendevreview | Merged openstack/cloudkitty master: Add requests to requirements https://review.opendev.org/c/openstack/cloudkitty/+/909368 | 16:03 |
| opendevreview | Pedro Henrique Pereira Martins proposed openstack/cloudkitty master: Add grenade tests https://review.opendev.org/c/openstack/cloudkitty/+/908749 | 17:20 |
| opendevreview | Pedro Henrique Pereira Martins proposed openstack/cloudkitty master: Add grenade tests https://review.opendev.org/c/openstack/cloudkitty/+/908749 | 17:49 |
| opendevreview | Pedro Henrique Pereira Martins proposed openstack/cloudkitty master: Add grenade tests https://review.opendev.org/c/openstack/cloudkitty/+/908749 | 19:34 |
| opendevreview | Pedro Henrique Pereira Martins proposed openstack/cloudkitty master: Add grenade tests https://review.opendev.org/c/openstack/cloudkitty/+/908749 | 20:23 |
| opendevreview | Pedro Henrique Pereira Martins proposed openstack/cloudkitty master: Add grenade tests https://review.opendev.org/c/openstack/cloudkitty/+/908749 | 21:09 |
| opendevreview | Pedro Henrique Pereira Martins proposed openstack/cloudkitty master: Add grenade tests https://review.opendev.org/c/openstack/cloudkitty/+/908749 | 22:41 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!