Monday, 2021-03-08

*** vishalmanchanda has joined #cloudkitty03:00
*** sboyron has joined #cloudkitty08:12
*** fresta has quit IRC10:04
*** fresta has joined #cloudkitty10:15
*** fresta has quit IRC11:08
*** fresta has joined #cloudkitty11:21
*** rafaelweingartne has joined #cloudkitty13:54
rafaelweingartne#startmeeting cloudkitty14:01
openstackMeeting started Mon Mar  8 14:01:00 2021 UTC and is due to finish in 60 minutes.  The chair is rafaelweingartne. Information about MeetBot at http://wiki.debian.org/MeetBot.14:01
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:01
*** openstack changes topic to " (Meeting topic: cloudkitty)"14:01
openstackThe meeting name has been set to 'cloudkitty'14:01
rafaelweingartneRoll count14:01
priteauo/14:01
mkarpiarzHi!14:02
jopdorp_o/14:02
*** mnaser has quit IRC14:03
*** mnaser has joined #cloudkitty14:03
*** mnaser has quit IRC14:03
*** mnaser has joined #cloudkitty14:03
rafaelweingartne#topic Review priorities14:04
*** openstack changes topic to "Review priorities (Meeting topic: cloudkitty)"14:04
rafaelweingartneDo you guys have some review priorities to ask for?14:04
jopdorp_none for me14:04
mkarpiarzJust FYI that I'm testing https://review.opendev.org/c/openstack/cloudkitty/+/77892214:05
rafaelweingartneI create a new patch to allow the admin to use multiple filters of the same type ('project_id') in the summary GET API. It would be interesting to get some opinions there.14:05
rafaelweingartnemkarpiarz: Thanks!14:05
mkarpiarzSure thing.14:06
mkarpiarzAnything else worth looking into?14:08
rafaelweingartneNot from my side14:09
rafaelweingartnemaybe we have a release coming, but I need to take a look into the process14:09
rafaelweingartnepierre: have you ever closed a Cloudkitty release before?14:09
priteauYes, it's easy14:10
priteauJust need to tag an RC1 of the various components14:10
rafaelweingartneI see14:10
rafaelweingartnedo we get a ping to execute this process?14:10
priteauI think so, the release management team will propose something14:11
priteauhttps://docs.openstack.org/project-team-guide/release-management.html14:11
rafaelweingartneOk, thanks14:11
priteauBut I will monitor once we are near14:11
rafaelweingartneok, thanks for the help14:12
rafaelweingartneSo, moving on14:13
rafaelweingartne#topic AOB14:14
*** openstack changes topic to "AOB (Meeting topic: cloudkitty)"14:14
rafaelweingartneNow, I open for general questions and topics that people might have.14:14
mkarpiarzNothing from my side.14:14
priteauThis week is feature freeze so we should wrap up patches14:15
priteau(although our PTL can grant exceptions)14:15
priteauWhich patches do we think are in a good state for Wallaby?14:15
priteaumkarpiarz: Is NOTNUMBOOL still WIP?14:16
mkarpiarzYeah, I need to add docs.14:17
rafaelweingartnethis one seems to be ready as well: https://review.opendev.org/c/openstack/cloudkitty/+/77452014:18
rafaelweingartnepriteau: I did not understand your comment in: https://review.opendev.org/c/openstack/cloudkitty/+/77463414:19
rafaelweingartnecould you clarify it?14:20
priteauWell, it's my fault. You originally suggested something like last_processed_timestamp14:20
priteauAnd I asked to change it to last_processed_at to make it similar to timestamps in other OpenStack DBs14:20
priteauBut it's a different meaning: it doesn't really track when the last period was processed, but which one14:21
priteaui.e. if you stop cloudkitty and restart it several hours later, this value will be set to times from the past (until it progressively syncs with present)14:22
priteauDo you see what I mean?14:22
rafaelweingartneyes, I see14:24
rafaelweingartneand yes, that is what happens14:24
rafaelweingartneI do not see the problem with this name though14:24
priteauWell, last_processed_at could make you think that it is the time the period is actually processed14:25
priteauMaybe just call it "last_processed"?14:25
rafaelweingartneHmm, I see14:26
rafaelweingartneI would like to see an "at" or "timestamp" wording14:26
rafaelweingartneotherwise, just reading, we do not know what the type of the field is14:26
rafaelweingartneWhat do others think?14:27
priteauSo last_processed_timestamp or last_processed_time?14:27
priteauLike you originally suggested14:27
rafaelweingartnemkarpiarz?14:28
mkarpiarzNot sure about this one.14:29
rafaelweingartnepriteau: if we decide to change? What do we do? I mean, we create a new spec to amend the one that has been merged?14:29
priteauIt's not a formal process. If we all agree on the code review, we can change the spec later.14:30
rafaelweingartneI see14:31
rafaelweingartneif you all agree, I can change to last_processed_timestamp or last_processed_time, then14:32
priteauIt would be good to understand if it's the time of the start or end of the last period14:33
rafaelweingartneit is the last timestamp that has been processed14:33
rafaelweingartneI will do the change then14:37
rafaelweingartneand we can discuss afterwards in the patch14:37
priteauThanks14:37
mkarpiarzOK14:38
rafaelweingartneIf we do no have anything else14:45
rafaelweingartneI will close the meeting in the next 5 minutes14:46
rafaelweingartneThank you guys for participating. Have a nice week.14:46
priteauThanks rafaelweingartne!14:46
mkarpiarzThank you all!14:47
rafaelweingartne#endmeeting14:51
*** openstack changes topic to "Rating component for OpenStack https://wiki.openstack.org/wiki/CloudKitty"14:51
openstackMeeting ended Mon Mar  8 14:51:32 2021 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:51
openstackMinutes:        http://eavesdrop.openstack.org/meetings/cloudkitty/2021/cloudkitty.2021-03-08-14.01.html14:51
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/cloudkitty/2021/cloudkitty.2021-03-08-14.01.txt14:51
openstackLog:            http://eavesdrop.openstack.org/meetings/cloudkitty/2021/cloudkitty.2021-03-08-14.01.log.html14:51
*** rafaelweingartne has quit IRC14:51
*** gmann is now known as gmann_afk16:01
*** gmann_afk is now known as gmann19:32
*** vishalmanchanda has quit IRC20:47

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