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