14:01:29 <rafaelweingartner> #startmeeting cloudkitty
14:01:29 <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:29 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:01:29 <opendevmeet> The meeting name has been set to 'cloudkitty'
14:01:37 <rafaelweingartner> Roll count
14:01:44 <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:02:15 <mkarpiarz> Hi!
14:02:50 <priteau> o/
14:06:09 <opendevreview> Rafael Weingartner proposed openstack/cloudkitty-specs master: Propose reprocessing API  https://review.opendev.org/c/openstack/cloudkitty-specs/+/791245
14:06:21 <rafaelweingartner> #topic Target review of meeting
14:06:25 <rafaelweingartner> We have the following patches as target for this meeting.
14:06:31 <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:07:19 <rafaelweingartner> about the first one, https://review.opendev.org/c/openstack/cloudkitty-specs/+/791245, I amended the commit as Pierre's requests
14:07:55 <rafaelweingartner> Do you guys think that this spec can be merged?
14:08:06 <priteau> I think you missed some occurrences of "reprocess"
14:09:05 <rafaelweingartner> hmm
14:09:07 <rafaelweingartner> which one?
14:09:08 <opendevreview> Rafael Weingartner proposed openstack/cloudkitty-specs master: Propose reprocessing API  https://review.opendev.org/c/openstack/cloudkitty-specs/+/791245
14:09:23 <mkarpiarz> Looks good to me.
14:09:25 <priteau> - `GET` `/v2/task/reprocess/<scope_id>` -- to retrieve the
14:09:44 <rafaelweingartner> ah ok
14:09:53 <rafaelweingartner> so you think that one should also be plural
14:10:34 <opendevreview> Rafael Weingartner proposed openstack/cloudkitty-specs master: Propose reprocessing API  https://review.opendev.org/c/openstack/cloudkitty-specs/+/791245
14:10:49 <priteau> Yes so it's the same URL pattern everywhere
14:11:05 <rafaelweingartner> I see
14:11:07 <rafaelweingartner> done
14:13:58 <priteau> Taking one last look, we can discuss the next patch in the meantime
14:14:33 <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:15:01 <rafaelweingartner> therefore, I will skip it for our next meeting then
14:15:19 <rafaelweingartner> After that, we have the following: https://review.opendev.org/c/openstack/cloudkitty/+/817275
14:15:37 <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:48 <priteau> Sorry, I haven't had time to work on an example
14:15:54 <priteau> Do you think it could be a follow-up patch?
14:16:14 <rafaelweingartner> Actually, something describing the use case in the commit message would be enough to me
14:16:25 <rafaelweingartner> so we can understand where it is used and why the change is being introduced
14:16:41 <rafaelweingartner> otherwise, it is a bit dificult to understand why the changes are being added
14:17:03 <priteau> OK, I will try to add this information
14:17:31 <priteau> For the specs, do you mind if I push an updated version fixing formatting? Some parts don't render correctly
14:19:41 <rafaelweingartner> No, I would not mind
14:19:45 <rafaelweingartner> go ahead
14:19:58 <rafaelweingartner> :)
14:20:35 <opendevreview> Pierre Riteau proposed openstack/cloudkitty master: Support customising Prometheus queries  https://review.opendev.org/c/openstack/cloudkitty/+/817275
14:20:47 <priteau> Added justification in commit msg ^^
14:24:02 <rafaelweingartner> thanks
14:24:07 <rafaelweingartner> I just gave a +2
14:28:47 <opendevreview> Pierre Riteau proposed openstack/cloudkitty-specs master: Propose reprocessing API  https://review.opendev.org/c/openstack/cloudkitty-specs/+/791245
14:30:02 <priteau> Just formatting fixes to make the rendered html more readable
14:30:41 <priteau> Gave a +2
14:35:18 <priteau> Are you still around?
14:37:25 <mkarpiarz> I'm here.
14:38:19 <rafaelweingartner> I am
14:38:42 <priteau> Should we move to the next patch?
14:38:46 <rafaelweingartner> yes
14:38:58 <rafaelweingartner> the last one is: https://review.opendev.org/c/openstack/cloudkitty/+/775359
14:39:26 <rafaelweingartner> I have reviewed that patch, but the author did not respond
14:39:39 <rafaelweingartner> I am not sure how we should proceed with these situations
14:39:49 <priteau> Can we try a direct email? Maybe they're not monitoring Gerrit
14:40:28 <rafaelweingartner> I will do that then
14:40:52 <priteau> I assume the author is a fellow Frenchman, I am happy to volunteer
14:41:10 <rafaelweingartner> ah
14:41:11 <rafaelweingartner> ok
14:41:24 <rafaelweingartner> if you can take the task, it is great :)
14:42:35 <rafaelweingartner> and that was the last patch of our list
14:42:44 <rafaelweingartner> do you guys have something else that we need to discuss?
14:43:46 <mkarpiarz> Just a quick question about: https://review.opendev.org/c/openstack/cloudkitty-dashboard/+/822865
14:44:00 <mkarpiarz> OK to abandon this one?
14:44:51 <rafaelweingartner> I would say so
14:45:33 <priteau> Of course
14:46:34 <mkarpiarz> Thanks. Done.
14:46: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:56 <rafaelweingartner> ok
14:47:46 <mkarpiarz> Just to let you guys know that I'm testing https://review.opendev.org/c/openstack/cloudkitty/+/799207.
14:48:38 <mkarpiarz> I run into an issue which might be caused by my setup so I'm now trying a new environment.
14:48:42 <rafaelweingartner> Thanks Marius!
14:49:33 <rafaelweingartner> I guess that we can close for today, right?
14:49:58 <mkarpiarz> I think so.
14:52:00 <rafaelweingartner> Thank you guys for participating. Have a nice week.
14:52:08 <rafaelweingartner> #endmeeting