14:02:58 <rafaelweingartner> #startmeeting cloudkitty
14:02:58 <opendevmeet> Meeting started Mon Aug  8 14:02:58 2022 UTC and is due to finish in 60 minutes.  The chair is rafaelweingartner. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:02:58 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:02:58 <opendevmeet> The meeting name has been set to 'cloudkitty'
14:03:04 <rafaelweingartner> Roll count
14:03:07 <priteau> o/
14:03:09 <priteau> Hello Rafael
14:04:43 <mkarpiarz> Hi!
14:05:33 <priteau> Hello Mariusz
14:06:38 <rafaelweingartner> I guess we can start :)
14:06:41 <rafaelweingartner> #topic Target review of the meeting
14:06:54 <rafaelweingartner> We have the following patches as target for this meeting.
14:07:19 <rafaelweingartner> #link https://review.opendev.org/c/openstack/python-cloudkittyclient/+/839052
14:07:37 <rafaelweingartner> I was waiting for Pierre to review it, and I amended today the review suggestions
14:08:29 <rafaelweingartner> The same is valid for this other patch: #link https://review.opendev.org/c/openstack/python-cloudkittyclient/+/838724
14:09:23 <mkarpiarz> Since I approved this change in the past, I've given my +2 already.
14:10:09 <priteau> Approved
14:12:22 <rafaelweingartner> Thanks!
14:12:47 <rafaelweingartner> Then, we have the following #link https://storyboard.openstack.org/#!/story/2009924
14:12:52 <rafaelweingartner> which was resolved by Pierre
14:13:18 <rafaelweingartner> I reviewed it already. I think it is fine as is. We only need to see if the patch needs/deserves a release note
14:13:22 <priteau> Good point about the release note. I also forget :)
14:13:53 <priteau> I will add one.
14:16:30 <rafaelweingartner> thanks!
14:16:51 <opendevreview> Pierre Riteau proposed openstack/cloudkitty-dashboard master: Remove create_instance override  https://review.opendev.org/c/openstack/cloudkitty-dashboard/+/852401
14:16:57 <priteau> Done
14:18:05 <rafaelweingartner> awesome
14:18:14 <rafaelweingartner> Moving on, we have the following: #link https://review.opendev.org/c/openstack/cloudkitty/+/710500
14:18:32 <rafaelweingartner> the author did not reply our reviews. Therefore, I took over the patch and worked to amend it.
14:20:57 <priteau> I think the reno could be formatted nicer with markup, I can make the change.
14:21:14 <priteau> Could you please create a story and task?
14:22:08 <mkarpiarz> A question related to this change: Has anyone of you managed to get ES 7 to work with CloudKitty?
14:22:51 <rafaelweingartner> We do not use it. So, I am trusting the author of the patch that it works
14:22:56 <priteau> Yes, we use it in several places
14:23:09 <priteau> I think it is 7.10 which is the latest OSS release
14:23:59 <mkarpiarz> I see. I think it would be good for me to test this change out first but when I switch to ES as the backend, I get not data.
14:24:13 <mkarpiarz> It must be something with my setup then.
14:24:19 <mkarpiarz> I'll have another look.
14:24:54 <mkarpiarz> I'm using containerised ES so can easily switch between ES 6 and 7 for testing.
14:25:03 <opendevreview> Pierre Riteau proposed openstack/cloudkitty master: Fix response format 'total' for v2 dataframes API  https://review.opendev.org/c/openstack/cloudkitty/+/710500
14:25:27 <priteau> We use the kolla containerised ES
14:25:54 <mkarpiarz> OK
14:27:02 <mkarpiarz> We can move on now. :)
14:28:33 <opendevreview> Rafael Weingartner proposed openstack/cloudkitty master: Fix response format 'total' for v2 dataframes api  https://review.opendev.org/c/openstack/cloudkitty/+/710500
14:28:55 <rafaelweingartner> I added the story
14:33:46 <priteau> I think you removed my latest changes ;-
14:34:57 <priteau> I will update
14:35:49 <rafaelweingartner> did I?
14:36:21 <priteau> I had uploaded PS1
14:36:24 <priteau> PS10 I mean
14:36:35 <opendevreview> Pierre Riteau proposed openstack/cloudkitty master: Fix response format 'total' for v2 dataframes API  https://review.opendev.org/c/openstack/cloudkitty/+/710500
14:37:00 <rafaelweingartner> I yes, I retrieved it/pulled it, but I forgot to apply
14:40:27 <priteau> I updated
14:41:31 <rafaelweingartner> thanks!
14:41:33 <rafaelweingartner> I merged it
14:41:34 <opendevreview> Merged openstack/cloudkitty-dashboard master: Remove create_instance override  https://review.opendev.org/c/openstack/cloudkitty-dashboard/+/852401
14:43:16 <priteau> I will now cherry-pick this dashboard fix
14:43:54 <mkarpiarz> Sure
14:45:17 <priteau> Please review: https://review.opendev.org/c/openstack/cloudkitty-dashboard/+/852438
14:47:06 <rafaelweingartner> thanks!
14:47:17 <rafaelweingartner> Moving on, we have the other patch opened by Pierre: #link https://review.opendev.org/c/openstack/cloudkitty/+/848957
14:47:46 <rafaelweingartner> I reviewed it, and suggested some validation to help operators to troubleshoot possible misconfigurations
14:48:00 <priteau> I don't have time right now to address your feedback on mutate_map
14:48:12 <priteau> It will need to wait until another day
14:48:23 <rafaelweingartner> no worries
14:48:32 <priteau> Thanks for the review
14:49:03 <rafaelweingartner> At least, that is all from my side.
14:49:05 <rafaelweingartner> #topic Open floor moment
14:49:14 <rafaelweingartner> Do you guys have something extra to Add?
14:49:24 <rafaelweingartner> Or, mayber, somebody from the audience?
14:50:16 <priteau> Once the dashboard fix is backported, we should release another yoga tag
14:50:34 <mkarpiarz> I agree
14:51:05 <mkarpiarz> Will it only be a new release for the Horizon plugin or the whole CloudKitty project?
14:51:36 <priteau> Just dashboard
14:51:43 <mkarpiarz> Perfect!
14:51:52 <priteau> I can prepare the patch if you want
14:52:45 <mkarpiarz> Sure.
14:56:21 <rafaelweingartner> Do you guys have something else to add? Otherwise, I guess, we can close the meeting for today.
14:56:45 <mkarpiarz> Nothing from my side.
14:57:33 <priteau> Let's wrap up
14:58:20 <rafaelweingartner> Thank you guys for participating. Have a nice week.
14:58:26 <rafaelweingartner> #endmeeting