| opendevreview | Matt Crees proposed openstack/cloudkitty master: Fix v1 summary/total with ES/OS storage backend https://review.opendev.org/c/openstack/cloudkitty/+/919751 | 12:34 |
|---|---|---|
| rafaelweingartner | #startmeeting cloudkitty | 14:00 |
| opendevmeet | Meeting started Mon Jun 10 14:00:08 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 |
| rafaelweingartner | \) | 14:00 |
| rafaelweingartner | \o | 14:00 |
| priteau | o/ | 14:01 |
| seunghunlee | Hello | 14:02 |
| rafaelweingartner | #topic Old branches deprecation/deletion | 14:04 |
| rafaelweingartner | this one is on my plate, and I forgot to address it. Sorry for that guys | 14:04 |
| rafaelweingartner | I will try to address this before our next meeting | 14:04 |
| priteau | Zuul checks are passing now so we don't really have to drop them. | 14:06 |
| rafaelweingartner | yes | 14:06 |
| rafaelweingartner | I approved them | 14:06 |
| priteau | We could wait for the branches to EOL on their expected time | 14:06 |
| rafaelweingartner | but nonetheless, we will need to plan this removal, right? | 14:06 |
| rafaelweingartner | ah, I see | 14:06 |
| rafaelweingartner | so, if we do nothing | 14:06 |
| priteau | We could just wait for the release team to EOL them | 14:06 |
| rafaelweingartner | they get removed automatically? | 14:07 |
| priteau | The release team will decide on EOL for Victoria, I don't know when that is expected. | 14:07 |
| rafaelweingartner | I see | 14:07 |
| priteau | See https://releases.openstack.org | 14:07 |
| priteau | Ussuri and Train were EOL in February | 14:07 |
| rafaelweingartner | I see, thanks for the information | 14:08 |
| rafaelweingartner | Moving on, #topic Target reviews | 14:11 |
| rafaelweingartner | #link https://review.opendev.org/c/openstack/cloudkitty/+/911125 and #link https://review.opendev.org/c/openstack/cloudkitty/+/911119 are the patches that were failing, and now are fine. We already approved. However, they are not merged yet. This happens because we do not have permissions to merge those branches. | 14:12 |
| priteau | I have added Elod as a reviewer | 14:12 |
| rafaelweingartner | moving on, we have our long standing patch to add start/end timestamps to rating rules. | 14:16 |
| rafaelweingartner | We still need to find a consensus #link https://review.opendev.org/c/openstack/cloudkitty/+/876643 | 14:16 |
| rafaelweingartner | priteau: do you think that we need more aditions/tests/discussions on it? | 14:16 |
| priteau | Sorry rafaelweingartner, I have been on travel and no progress from me on reviewing this patch :( | 14:17 |
| rafaelweingartner | ok, no worries | 14:18 |
| rafaelweingartner | we can wait to our next meeting to discuss it back | 14:19 |
| rafaelweingartner | moving on, #link https://review.opendev.org/c/openstack/cloudkitty/+/915479 | 14:21 |
| rafaelweingartner | It is a patch that I reviewed, but the author the not answer | 14:21 |
| rafaelweingartner | I am not sure if the patch is needed | 14:21 |
| priteau | I agree with your comment rafaelweingartner | 14:24 |
| rafaelweingartner | do you think that we should abandon it? | 14:26 |
| rafaelweingartner | or, wait for a little longer before doing it | 14:27 |
| priteau | I don't think so, but there must be some other way to fix it? | 14:27 |
| rafaelweingartner | I agree | 14:27 |
| priteau | I would like to understand more the problem. It talks about Python 3.12, is the result returned by the API different on this version? | 14:27 |
| rafaelweingartner | Let's wait a little longer then | 14:27 |
| rafaelweingartner | not that I know of | 14:28 |
| rafaelweingartner | the author does not provide much details | 14:28 |
| priteau | openstack-tox-py312 is SUCCESS without this patch too | 14:28 |
| rafaelweingartner | yes | 14:29 |
| rafaelweingartner | So, moving on, we have #link https://review.opendev.org/c/openstack/cloudkitty/+/915992 | 14:31 |
| rafaelweingartner | which relates to the topic of un-deprecating ElasticSearch | 14:31 |
| rafaelweingartner | what do you think about it? | 14:31 |
| priteau | zigo: Are you around? | 14:32 |
| priteau | rafaelweingartner: do you think you could send an email to the list regarding ElasticSearch and potential users? | 14:33 |
| rafaelweingartner | yes | 14:33 |
| rafaelweingartner | I will add this to my plate | 14:33 |
| rafaelweingartner | and then, with the answer for this query, we can discuss this topic further | 14:33 |
| opendevreview | Pierre Riteau proposed openstack/cloudkitty master: Bump hacking minimum version to 6.1.0 https://review.opendev.org/c/openstack/cloudkitty/+/920943 | 14:35 |
| rafaelweingartner | And, that was all of the topics I had here | 14:37 |
| rafaelweingartner | do you guys have something else to add? | 14:37 |
| priteau | The patch I just updated may be ready for review now | 14:38 |
| priteau | I don't know yet if it passes all checks | 14:39 |
| rafaelweingartner | ok | 14:39 |
| rafaelweingartner | When it passes, I will review it then | 14:39 |
| priteau | Thank you | 14:40 |
| rafaelweingartner | if there is nothing else, we can close for today, then | 14:42 |
| priteau | Nothing else from me | 14:43 |
| rafaelweingartner | Thank you guys for participating. Have a nice week. | 14:43 |
| rafaelweingartner | #endmeeting | 14:43 |
| opendevmeet | Meeting ended Mon Jun 10 14:43:38 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:43 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cloudkitty/2024/cloudkitty.2024-06-10-14.00.html | 14:43 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cloudkitty/2024/cloudkitty.2024-06-10-14.00.txt | 14:43 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cloudkitty/2024/cloudkitty.2024-06-10-14.00.log.html | 14:43 |
| seunghunlee | Thanks | 14:43 |
| zigo | priteau: Yes. | 15:01 |
| zigo | What about this patch? Why isn't it getting any review? | 15:02 |
| priteau | zigo: Hello. Are you planning to keep using Elasticsearch (rather than OpenSearch) as cloudkitty backend long term? | 15:02 |
| zigo | Yes, because opensearch is unfortunately lagging behind and not adding features as fast as elasticsearch. | 15:03 |
| zigo | It doesn't feel like opensearch is as well maintained. | 15:03 |
| zigo | That's what my colleagues told me. | 15:04 |
| priteau | We deprecated the Elasticsearch driver and ultimately wanted to remove it. But if it is still being used, we can keep maintaining it. | 15:04 |
| priteau | Which version are you using? | 15:04 |
| zigo | Of OpenStack you mean or Elasticsearch ? | 15:04 |
| priteau | ES | 15:05 |
| zigo | I'm getting the information, hang on. | 15:06 |
| zigo | What I know already for sure, is that they have no intention to use OpenSearch... | 15:07 |
| zigo | If the ElasticSearch driver was to be removed, they told me they would use the OpenSearch one, still connecting to ES, because it's the same exact calls, and it would continue working, I've just been told. | 15:07 |
| priteau | So the new OpenSearch driver is tested against OpenSearch 2.x, which I believe is similar to Elasticsearch 8.x | 15:08 |
| priteau | While the existing Elasticsearch driver is really only tested against Elasticsearch 6.x / 7.x | 15:09 |
| zigo | We're on ES 8.13.2 and we will soon switch to 8.14. | 15:10 |
| zigo | They are closely following upstream upgrades. | 15:10 |
| priteau | And the cloudkitty Elasticsearch driver works against these versions? That's good to know | 15:11 |
| zigo | Yes it does/ | 15:12 |
| zigo | Do you need us to extend our patch to also do it for OpenSearch ? | 15:13 |
| priteau | That would be ideal, if OpenSearch also supports it | 15:14 |
| zigo | My colleague will look into it. | 15:22 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!