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