Monday, 2023-02-06

rafaelweingartnerHello guys!14:02
rafaelweingartner#startmeeting cloudkitty14:02
opendevmeetMeeting started Mon Feb  6 14:02:24 2023 UTC and is due to finish in 60 minutes.  The chair is rafaelweingartner. Information about MeetBot at http://wiki.debian.org/MeetBot.14:02
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:02
opendevmeetThe meeting name has been set to 'cloudkitty'14:02
rafaelweingartnerRoll count14:02
rafaelweingartner\O14:02
mkarpiarzHi!14:02
priteauo/14:04
rafaelweingartner#topic add CloudKitty API reference docs (https://docs.openstack.org/cloudkitty/latest/api-reference/index.html) to https://docs.openstack.org/api-ref/14:04
rafaelweingartnerI guess that everythin is ok with this front, right?14:05
rafaelweingartnerAfter the last check we did last week14:05
priteaumkarpiarz was asking whether we backport14:06
rafaelweingartnerDo we need?14:07
mkarpiarzYes, CloudKitty API refs are available at https://docs.openstack.org/api-ref/rating/14:07
mkarpiarzwhich is the main thing I wanted to archive.14:08
priteauAs I said by email earlier, we can wait until A is release14:08
rafaelweingartnerI agree with that14:08
mkarpiarzCool, then we can consider this task over and done. :)14:09
priteauThank you mkarpiarz14:09
rafaelweingartneryes14:09
rafaelweingartner#topic Target reviews14:10
rafaelweingartner#link: https://review.opendev.org/c/openstack/cloudkitty-specs/+/866198 (14:11
rafaelweingartnerthe spec, what do you guys think about it?14:11
rafaelweingartnerCan we move on with it?14:11
priteauI just left some comments14:11
priteauIt's a nice proposal but we could make some small changes. Please check comments.14:11
rafaelweingartnerAwesome14:12
rafaelweingartnerthanks!14:12
rafaelweingartnerAnd, now, what about the patches:14:14
rafaelweingartner#link https://review.opendev.org/c/openstack/cloudkitty/+/86282414:14
priteauLeft a comment14:17
rafaelweingartnerThanks!14:19
rafaelweingartner#link https://review.opendev.org/c/openstack/cloudkitty/+/861908, this one I have a good news14:20
rafaelweingartnerGnocchi community is going to cut a new release14:21
mkarpiarzNice!14:21
rafaelweingartnerwhich is gong to be important for this one14:22
rafaelweingartnerI have worked with them durign the past weeks14:22
rafaelweingartnerThen, we have this one #link: https://review.opendev.org/c/openstack/cloudkitty/+/86178614:24
rafaelweingartneralso an intersting one to proceed with14:24
mkarpiarzOK, I'll have a look at this one.14:24
mkarpiarzThis looks like a good idea but will try the implementation out first before giving it my +2.14:27
priteaumkarpiarz: Any reason you went with time, time-d, time-w instead of hour, day, week?14:27
mkarpiarzrafaelweingartner: ^ :)14:28
priteauSorry, that was a question for rafaelweingartner14:28
rafaelweingartnerNo reason14:29
rafaelweingartnerI just wanted to be compatible with the "time" notation14:30
rafaelweingartnerthat was already there14:30
rafaelweingartnerin CloudKitty14:30
priteauWe already have group_by time?14:31
rafaelweingartneryes14:31
rafaelweingartnerbut only hourly14:31
priteauMaybe I misunderstand, but the unit tests suggest that API support: "time", "time-d", "time-w", "time-m", "time-y"14:33
priteauBut commit/release note suggest other names14:33
rafaelweingartnerhmm14:34
rafaelweingartnerI might be expressing my self wrong in the commit message then14:34
priteauI left a few comments, let's follow up on Gerrit14:36
rafaelweingartnerthanks!14:36
rafaelweingartnerDo you guys have other patches to be added to this list?14:38
mkarpiarzNot at this moment.14:38
priteauI am hoping to have OpenSearch soon14:39
priteauActively working on it14:39
mkarpiarzAwesome!14:40
priteauAnything else?14:47
priteaurafaelweingartner: Are you still here?14:48
rafaelweingartneryes, I am14:51
rafaelweingartnersorry I had to answer some call here14:51
rafaelweingartnerpriteau: so you are working on the OpenSearch support14:51
rafaelweingartnerare you doing for v2?14:52
priteauyes, since v1 is compatible already14:52
rafaelweingartnercool14:52
rafaelweingartnerIf you need any help, just le me know14:53
priteauThanks14:58
rafaelweingartnerDo you guys have something else do add?15:01
mkarpiarzNope15:01
rafaelweingartnerThank you guys for participating. Have a nice week.15:02
mkarpiarzThanks!15:03
rafaelweingartner#endmeeting15:04
opendevmeetMeeting ended Mon Feb  6 15:04:02 2023 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:04
opendevmeetMinutes:        https://meetings.opendev.org/meetings/cloudkitty/2023/cloudkitty.2023-02-06-14.02.html15:04
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/cloudkitty/2023/cloudkitty.2023-02-06-14.02.txt15:04
opendevmeetLog:            https://meetings.opendev.org/meetings/cloudkitty/2023/cloudkitty.2023-02-06-14.02.log.html15:04
opendevreviewPedro Henrique Pereira Martins proposed openstack/cloudkitty master: Allows multiple rating types for same metric for gnocchi  https://review.opendev.org/c/openstack/cloudkitty/+/86180616:28
opendevreviewPedro Henrique Pereira Martins proposed openstack/cloudkitty-specs master: Add spec to add audit and ttl in the rating rules  https://review.opendev.org/c/openstack/cloudkitty-specs/+/86619817:39
opendevreviewRafael Weingartner proposed openstack/cloudkitty master: Use correct metadata for metrics gathered from gnocchi  https://review.opendev.org/c/openstack/cloudkitty/+/86190818:20
opendevreviewRafael Weingartner proposed openstack/cloudkitty master: Add description option to a rating metric definition  https://review.opendev.org/c/openstack/cloudkitty/+/86282420:40

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!