| rafaelweingartner | Hello Guys! | 14:00 |
|---|---|---|
| rafaelweingartner | #startmeeting cloudkitty | 14:01 |
| opendevmeet | Meeting started Mon Jan 10 14:01:29 2022 UTC and is due to finish in 60 minutes. The chair is rafaelweingartner. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:01 |
| opendevmeet | The meeting name has been set to 'cloudkitty' | 14:01 |
| rafaelweingartner | Roll count | 14:01 |
| opendevreview | Merged openstack/python-cloudkittyclient master: Re-add python 3.6/3.7 in classifier https://review.opendev.org/c/openstack/python-cloudkittyclient/+/821634 | 14:01 |
| mkarpiarz | Hi! | 14:02 |
| priteau | o/ | 14:02 |
| opendevreview | Rafael Weingartner proposed openstack/cloudkitty-specs master: Propose reprocessing API https://review.opendev.org/c/openstack/cloudkitty-specs/+/791245 | 14:06 |
| rafaelweingartner | #topic Target review of meeting | 14:06 |
| rafaelweingartner | We have the following patches as target for this meeting. | 14:06 |
| rafaelweingartner | #link https://review.opendev.org/c/openstack/cloudkitty-specs/+/791245 #link https://review.opendev.org/c/openstack/cloudkitty/+/775359 #link https://review.opendev.org/c/openstack/cloudkitty/+/817275 #link https://review.opendev.org/c/openstack/cloudkitty/+/799207 | 14:06 |
| rafaelweingartner | about the first one, https://review.opendev.org/c/openstack/cloudkitty-specs/+/791245, I amended the commit as Pierre's requests | 14:07 |
| rafaelweingartner | Do you guys think that this spec can be merged? | 14:07 |
| priteau | I think you missed some occurrences of "reprocess" | 14:08 |
| rafaelweingartner | hmm | 14:09 |
| rafaelweingartner | which one? | 14:09 |
| opendevreview | Rafael Weingartner proposed openstack/cloudkitty-specs master: Propose reprocessing API https://review.opendev.org/c/openstack/cloudkitty-specs/+/791245 | 14:09 |
| mkarpiarz | Looks good to me. | 14:09 |
| priteau | - `GET` `/v2/task/reprocess/<scope_id>` -- to retrieve the | 14:09 |
| rafaelweingartner | ah ok | 14:09 |
| rafaelweingartner | so you think that one should also be plural | 14:09 |
| opendevreview | Rafael Weingartner proposed openstack/cloudkitty-specs master: Propose reprocessing API https://review.opendev.org/c/openstack/cloudkitty-specs/+/791245 | 14:10 |
| priteau | Yes so it's the same URL pattern everywhere | 14:10 |
| rafaelweingartner | I see | 14:11 |
| rafaelweingartner | done | 14:11 |
| priteau | Taking one last look, we can discuss the next patch in the meantime | 14:13 |
| rafaelweingartner | Sure, the next one is https://review.opendev.org/c/openstack/cloudkitty/+/799207, which will require some changes after the spec is merged | 14:14 |
| rafaelweingartner | therefore, I will skip it for our next meeting then | 14:15 |
| rafaelweingartner | After that, we have the following: https://review.opendev.org/c/openstack/cloudkitty/+/817275 | 14:15 |
| rafaelweingartner | it is more or less approved, I just felt the need for some use cases where that change is interesting/needed/useful | 14:15 |
| priteau | Sorry, I haven't had time to work on an example | 14:15 |
| priteau | Do you think it could be a follow-up patch? | 14:15 |
| rafaelweingartner | Actually, something describing the use case in the commit message would be enough to me | 14:16 |
| rafaelweingartner | so we can understand where it is used and why the change is being introduced | 14:16 |
| rafaelweingartner | otherwise, it is a bit dificult to understand why the changes are being added | 14:16 |
| priteau | OK, I will try to add this information | 14:17 |
| priteau | For the specs, do you mind if I push an updated version fixing formatting? Some parts don't render correctly | 14:17 |
| rafaelweingartner | No, I would not mind | 14:19 |
| rafaelweingartner | go ahead | 14:19 |
| rafaelweingartner | :) | 14:19 |
| opendevreview | Pierre Riteau proposed openstack/cloudkitty master: Support customising Prometheus queries https://review.opendev.org/c/openstack/cloudkitty/+/817275 | 14:20 |
| priteau | Added justification in commit msg ^^ | 14:20 |
| rafaelweingartner | thanks | 14:24 |
| rafaelweingartner | I just gave a +2 | 14:24 |
| opendevreview | Pierre Riteau proposed openstack/cloudkitty-specs master: Propose reprocessing API https://review.opendev.org/c/openstack/cloudkitty-specs/+/791245 | 14:28 |
| priteau | Just formatting fixes to make the rendered html more readable | 14:30 |
| priteau | Gave a +2 | 14:30 |
| priteau | Are you still around? | 14:35 |
| mkarpiarz | I'm here. | 14:37 |
| rafaelweingartner | I am | 14:38 |
| priteau | Should we move to the next patch? | 14:38 |
| rafaelweingartner | yes | 14:38 |
| rafaelweingartner | the last one is: https://review.opendev.org/c/openstack/cloudkitty/+/775359 | 14:38 |
| rafaelweingartner | I have reviewed that patch, but the author did not respond | 14:39 |
| rafaelweingartner | I am not sure how we should proceed with these situations | 14:39 |
| priteau | Can we try a direct email? Maybe they're not monitoring Gerrit | 14:39 |
| rafaelweingartner | I will do that then | 14:40 |
| priteau | I assume the author is a fellow Frenchman, I am happy to volunteer | 14:40 |
| rafaelweingartner | ah | 14:41 |
| rafaelweingartner | ok | 14:41 |
| rafaelweingartner | if you can take the task, it is great :) | 14:41 |
| rafaelweingartner | and that was the last patch of our list | 14:42 |
| rafaelweingartner | do you guys have something else that we need to discuss? | 14:42 |
| mkarpiarz | Just a quick question about: https://review.opendev.org/c/openstack/cloudkitty-dashboard/+/822865 | 14:43 |
| mkarpiarz | OK to abandon this one? | 14:44 |
| rafaelweingartner | I would say so | 14:44 |
| priteau | Of course | 14:45 |
| mkarpiarz | Thanks. Done. | 14:46 |
| priteau | Something else: https://review.opendev.org/c/openstack/cloudkitty-tempest-plugin/+/822475 was failing its train job. I just tried a recheck, if it doesn't work I will propose a patch to remove the job | 14:46 |
| rafaelweingartner | ok | 14:46 |
| mkarpiarz | Just to let you guys know that I'm testing https://review.opendev.org/c/openstack/cloudkitty/+/799207. | 14:47 |
| mkarpiarz | I run into an issue which might be caused by my setup so I'm now trying a new environment. | 14:48 |
| rafaelweingartner | Thanks Marius! | 14:48 |
| rafaelweingartner | I guess that we can close for today, right? | 14:49 |
| mkarpiarz | I think so. | 14:49 |
| rafaelweingartner | Thank you guys for participating. Have a nice week. | 14:52 |
| rafaelweingartner | #endmeeting | 14:52 |
| opendevmeet | Meeting ended Mon Jan 10 14:52:08 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:52 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cloudkitty/2022/cloudkitty.2022-01-10-14.01.html | 14:52 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cloudkitty/2022/cloudkitty.2022-01-10-14.01.txt | 14:52 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cloudkitty/2022/cloudkitty.2022-01-10-14.01.log.html | 14:52 |
| mkarpiarz | Thank you! | 14:52 |
| priteau | Thank you both | 14:52 |
| opendevreview | Merged openstack/cloudkitty-specs master: Propose reprocessing API https://review.opendev.org/c/openstack/cloudkitty-specs/+/791245 | 14:55 |
| opendevreview | Merged openstack/cloudkitty master: Support customising Prometheus queries https://review.opendev.org/c/openstack/cloudkitty/+/817275 | 15:51 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!