14:01:39 #startmeeting cloudkitty 14:01:39 Meeting started Mon May 27 14:01:39 2024 UTC and is due to finish in 60 minutes. The chair is rafaelweingartner. Information about MeetBot at http://wiki.debian.org/MeetBot. 14:01:39 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 14:01:39 The meeting name has been set to 'cloudkitty' 14:01:43 Hello guys! 14:01:46 Roll count 14:01:48 \o 14:01:52 Hi! 14:02:03 Hello 14:02:10 mkarpiarz: I thought you might be on holiday 14:03:18 Ah, yes, it's a bank holiday in the UK but I'm here today. :) 14:05:01 I see 14:05:07 awesome that you are here :) 14:05:22 :) 14:05:28 #topic Old branches deprecation/deletion 14:05:40 This one is one me. I forgot to address it this past weeks 14:05:48 I will try to add to my list and not forget about it 14:06:02 sorry for that 14:08:01 rafaelweingartner: Would you like me to create the release patch to drop these branches? I can do it if you are busy. 14:08:13 yes 14:08:15 that is the idea 14:08:33 if you can, that would be awesome! 14:10:34 I will check if I can do it. 14:10:43 You will still need to give your +1 14:11:09 OK 14:11:28 sure, I will 14:12:46 #topic Target reviews 14:13:06 We have this one, #link https://review.opendev.org/c/openstack/cloudkitty/+/917445 , which is a cherry-pick and seems to be fine for merging 14:13:28 mkarpiarz: can you do the honours? 14:13:49 Sure 14:14:05 Same for https://review.opendev.org/c/openstack/cloudkitty/+/917446 and https://review.opendev.org/c/openstack/cloudkitty/+/917447 14:14:18 rafaelweingartner: you gave W+1 but without Code=Review+2 first 14:14:52 probably selected the wrong options there 14:15:33 fixed 14:16:31 Thanks both 14:17:34 moving on, we have #link https://review.opendev.org/c/openstack/cloudkitty/+/876643. 14:17:50 This one has been in the making for a while, and Pedro has added the documentation requested by priteau 14:19:05 The change still looks good to me. :) 14:19:16 I will do another review this week. I was out of the office last week 14:20:42 ok, thanks for the reviews! 14:21:44 That patch is a new step to CloudKitty regarding rating rules management 14:23:13 So is backward compatibility of rules without start and end date still an issue we need to discuss? 14:24:48 for us it is not an issue, because one can always set a start date in the past. However, if needed, I can discuss with pedro. For instance, to allow somebody to create a rating rule without start, which would mean that it is valid since eer 14:24:51 ever* 14:25:07 this is the same as setting a start timestamp in the distant past, for instance 14:27:29 I'm fine with either but it would be nice if we came to a broader agreement, not just majority. 14:27:39 (i.e. 2/3) 14:27:56 I still think backward compatibility is a nice thing, but I need to test a bit more how it behaves currently 14:28:35 I am afraid I have to leave for school pickup 14:28:58 ok 14:29:01 we can close for today then 14:29:14 and discuss this further in our next meeting 14:29:20 mkarpiarz: Thanks for the quick review of https://review.opendev.org/c/openstack/cloudkitty/+/919751 14:29:33 Matt is back tomorrow and will respond to your comment 14:29:46 And update the patch if required 14:29:46 OK 14:29:53 Thanks! 14:30:07 Good bye! 14:30:12 bye 14:30:15 Bye! 14:30:21 Thank you guys for participating. Have a nice week. 14:30:24 #endmeeting