| opendevreview | Merged openstack/cloudkitty-dashboard master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/cloudkitty-dashboard/+/917233 | 10:39 |
|---|---|---|
| opendevreview | Merged openstack/cloudkitty master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/cloudkitty/+/917235 | 10:40 |
| opendevreview | Merged openstack/python-cloudkittyclient master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/python-cloudkittyclient/+/917237 | 10:41 |
| opendevreview | Merged openstack/python-cloudkittyclient master: reno: Update master for unmaintained/victoria https://review.opendev.org/c/openstack/python-cloudkittyclient/+/911122 | 10:59 |
| opendevreview | Pedro Henrique Pereira Martins proposed openstack/cloudkitty master: Introduce start and end dates on rating rules https://review.opendev.org/c/openstack/cloudkitty/+/876643 | 11:18 |
| opendevreview | Pedro Henrique Pereira Martins proposed openstack/cloudkitty master: Introduce start and end dates on rating rules https://review.opendev.org/c/openstack/cloudkitty/+/876643 | 11:18 |
| opendevreview | Merged openstack/python-cloudkittyclient master: Fix reprocessing POST request https://review.opendev.org/c/openstack/python-cloudkittyclient/+/915324 | 11:41 |
| opendevreview | Merged openstack/cloudkitty master: Fix API report requests when using opensearch https://review.opendev.org/c/openstack/cloudkitty/+/917068 | 11:48 |
| opendevreview | Merged openstack/cloudkitty-tempest-plugin master: Add stable/2024.1 jobs on master gate https://review.opendev.org/c/openstack/cloudkitty-tempest-plugin/+/917080 | 11:48 |
| opendevreview | Merged openstack/cloudkitty-tempest-plugin master: Update python classifier in setup.cfg https://review.opendev.org/c/openstack/cloudkitty-tempest-plugin/+/904645 | 11:48 |
| opendevreview | Merged openstack/cloudkitty master: Update README content https://review.opendev.org/c/openstack/cloudkitty/+/915895 | 11:48 |
| opendevreview | Pedro Henrique Pereira Martins proposed openstack/cloudkitty master: Introduce start and end dates on rating rules https://review.opendev.org/c/openstack/cloudkitty/+/876643 | 12:42 |
| rafaelweingartner | #startmeeting cloudkitty | 14:00 |
| opendevmeet | Meeting started Mon Apr 29 14:00:01 2024 UTC and is due to finish in 60 minutes. The chair is rafaelweingartner. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| opendevmeet | The meeting name has been set to 'cloudkitty' | 14:00 |
| rafaelweingartner | Hello guys! | 14:00 |
| rafaelweingartner | Roll count | 14:00 |
| seunghunlee | Hello | 14:00 |
| mattcrees | o/ | 14:00 |
| rafaelweingartner | Do you guys know if priteau is comming? | 14:03 |
| priteau | Hello. Sorry I am dealing with a customer issue. | 14:03 |
| rafaelweingartner | ah, no worries :) | 14:03 |
| rafaelweingartner | #topic Storyboard status | 14:04 |
| rafaelweingartner | This is a process that is being done by priteau. I guess, we can sync up about this topic next week | 14:04 |
| rafaelweingartner | The next topic is also to discuss with priteau, so I will just register it here, and we can return to it during our next meeting. | 14:05 |
| rafaelweingartner | #topic Old branches deprecation/deletion | 14:05 |
| rafaelweingartner | The idea is to deprecate and remove branches for EOL versions | 14:06 |
| rafaelweingartner | therefore, we do not need to use time to maintain them anymore | 14:06 |
| mattcrees | Makes sense, I suppose there will be some branches (at least Yoga plus afaik) that we'd be interested in continuing to maintain. | 14:07 |
| rafaelweingartner | exactly | 14:08 |
| seunghunlee | Sounds good | 14:08 |
| rafaelweingartner | Moving on for the target reviews! | 14:13 |
| rafaelweingartner | #topic: Target reviews | 14:13 |
| rafaelweingartner | This one #link https://review.opendev.org/c/openstack/cloudkitty/+/876643, has been in the making for a long time, and it seems to be ready for merging. It would be awesome to get other's people review here. | 14:13 |
| rafaelweingartner | Specially, as it is targeted to 2024.2 | 14:14 |
| mattcrees | That's a big one. I'll add it to my reviews list, but I'm not sure I'll have the time too soon to test it | 14:16 |
| rafaelweingartner | no worries | 14:16 |
| rafaelweingartner | indeed it is a big one | 14:16 |
| rafaelweingartner | it is introducing a new concept to cloudkitty, which is the ability to have start/end date in the rating rules | 14:17 |
| rafaelweingartner | It is a big milestone for us as a community | 14:17 |
| mattcrees | Certainly sounds like a good feature to have! | 14:17 |
| rafaelweingartner | =) | 14:17 |
| seunghunlee | I'll have a look too. But it might take some time as this is a big one | 14:18 |
| priteau | rafaelweingartner: I left comments on that patch | 14:19 |
| seunghunlee | Definitely a good feature to have | 14:19 |
| rafaelweingartner | no worries | 14:19 |
| rafaelweingartner | priteau: yes! | 14:19 |
| rafaelweingartner | Pedro is already taking a look into it | 14:19 |
| rafaelweingartner | Moving on, we have the patch #link https://review.opendev.org/c/openstack/cloudkitty/+/684747, which also seems to be ready. Other revies would also be welcome! | 14:22 |
| mattcrees | Looks like it's ready to go tbh. Are you waiting on a final review from priteau? | 14:26 |
| priteau | I've not looked at this one yet | 14:26 |
| rafaelweingartner | Not necessarelly, I am just inviting others to also review =) | 14:27 |
| rafaelweingartner | just trying to include more contributors into the patches, before we merge them | 14:27 |
| mattcrees | Makes sense :) | 14:29 |
| rafaelweingartner | Moving on, we have the patches #link https://review.opendev.org/c/openstack/cloudkitty/+/916986 and #link https://review.opendev.org/c/openstack/cloudkitty/+/916987 | 14:32 |
| rafaelweingartner | they are the same patches, but targetting different branches | 14:33 |
| rafaelweingartner | why don't we open against master? | 14:33 |
| rafaelweingartner | and then, when it is merged, we push/backport if needed to others | 14:33 |
| rafaelweingartner | It does not seem to be a very important patch to be in releases already released. | 14:33 |
| mattcrees | It was already merged in master: https://review.opendev.org/c/openstack/cloudkitty/+/904433 | 14:33 |
| mattcrees | These are the backports to that | 14:34 |
| rafaelweingartner | I see | 14:34 |
| rafaelweingartner | I guess we can just merge them | 14:35 |
| rafaelweingartner | what do you guys think? | 14:35 |
| priteau | One of them is not passing CI | 14:35 |
| priteau | This is not an important fix, it's mostly to reduce spam in logs | 14:35 |
| rafaelweingartner | the one that was ok, I merged then | 14:36 |
| rafaelweingartner | the other one, seems that we might need to do another patch for that branch | 14:37 |
| rafaelweingartner | to fix CI | 14:37 |
| rafaelweingartner | what do you guys think? | 14:37 |
| mattcrees | At a glance the failures look unrelated to the patch | 14:40 |
| mattcrees | > alembic.script.revision.ResolutionError: No such revision or branch '175fa80908e1' | 14:40 |
| rafaelweingartner | let's see the recheck output | 14:41 |
| rafaelweingartner | and we move on from there | 14:41 |
| rafaelweingartner | Moving on, we have #link https://review.opendev.org/c/openstack/cloudkitty/+/915992 | 14:43 |
| rafaelweingartner | I am not versed in ElasticSearch | 14:44 |
| rafaelweingartner | but it seems something interesting | 14:44 |
| rafaelweingartner | wha tdo you guys think? | 14:44 |
| rafaelweingartner | Is it also supported in OpenSearch? | 14:44 |
| mattcrees | Yes datastreams are in OpenSearch so it would be good to see that get into both backends. | 14:45 |
| rafaelweingartner | Cool | 14:45 |
| rafaelweingartner | Are you guys able to review that patch? | 14:45 |
| rafaelweingartner | and maybe propose this to OpenSearch as well? | 14:45 |
| priteau | We need to decide what we do about the ES driver | 14:46 |
| rafaelweingartner | agree | 14:46 |
| priteau | We said it was deprecated and we would remove it | 14:46 |
| priteau | But maybe zigo is still using it? | 14:46 |
| rafaelweingartner | that is why I was surprised when I saw the patch proposed there | 14:46 |
| priteau | Also in kolla-ansible there was pushback against removing the ES configuration part for cloudkitty | 14:46 |
| priteau | So maybe we should un-deprecate | 14:47 |
| rafaelweingartner | That might be a good patch | 14:47 |
| rafaelweingartner | path* | 14:47 |
| rafaelweingartner | then we have both | 14:47 |
| rafaelweingartner | and they both can evolve apart from each other | 14:47 |
| rafaelweingartner | as they are completly separated drivers/components | 14:48 |
| rafaelweingartner | Moving on, we have the patch #link https://review.opendev.org/c/openstack/cloudkitty/+/915479, where it does not seem to be needed the change proposed. However, we need to see the feedback from the author there | 14:52 |
| rafaelweingartner | And, I guess those were all of the topics covered. | 14:54 |
| rafaelweingartner | Do you guys have something else to add? | 14:54 |
| mattcrees | There was this patch that was missed during the yoga switch to unmaintained. Not urgent just yet, but we'll need it when we try to backport OpenSearch support in KA. https://review.opendev.org/c/openstack/cloudkitty/+/913580 | 14:55 |
| rafaelweingartner | and we already aproved | 14:56 |
| rafaelweingartner | but sadly, those are not merged by us | 14:56 |
| rafaelweingartner | I guess we need to ping Elod there | 14:56 |
| rafaelweingartner | If we want to add that patch into Yoga | 14:56 |
| mattcrees | Oh I see, the unmaintained team needs to do this instead. I forgot that there was a different team for that | 14:57 |
| rafaelweingartner | yes | 14:57 |
| rafaelweingartner | We will need to close guys, but I guess that was it for today | 14:59 |
| rafaelweingartner | Thank you guys for participating. Have a nice week. | 14:59 |
| mattcrees | I don't think there's any other patches we're looking for reviews on. Thanks very much for the quick merges of our recent bugfixes :) | 14:59 |
| rafaelweingartner | =) | 14:59 |
| seunghunlee | Thank you | 14:59 |
| rafaelweingartner | welcome! | 14:59 |
| rafaelweingartner | #endmeeting | 14:59 |
| opendevmeet | Meeting ended Mon Apr 29 14:59:48 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:59 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cloudkitty/2024/cloudkitty.2024-04-29-14.00.html | 14:59 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cloudkitty/2024/cloudkitty.2024-04-29-14.00.txt | 14:59 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cloudkitty/2024/cloudkitty.2024-04-29-14.00.log.html | 14:59 |
| opendevreview | Pedro Henrique Pereira Martins proposed openstack/cloudkitty master: Introduce start and end dates on rating rules https://review.opendev.org/c/openstack/cloudkitty/+/876643 | 15:35 |
| opendevreview | Pedro Henrique Pereira Martins proposed openstack/cloudkitty master: Introduce start and end dates on rating rules https://review.opendev.org/c/openstack/cloudkitty/+/876643 | 17:14 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!