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