| *** jopdorp has quit IRC | 00:18 | |
| *** jopdorp_ has joined #cloudkitty | 00:18 | |
| *** vishalmanchanda has joined #cloudkitty | 06:09 | |
| *** sboyron has joined #cloudkitty | 08:39 | |
| *** sboyron has quit IRC | 12:16 | |
| *** sboyron has joined #cloudkitty | 12:17 | |
| *** mkarpiarz has joined #cloudkitty | 13:27 | |
| *** rafaelweingartne has joined #cloudkitty | 13:55 | |
| rafaelweingartne | #startmeeting cloudkitty | 14:00 |
|---|---|---|
| openstack | Meeting started Mon Jan 25 14:00:38 2021 UTC and is due to finish in 60 minutes. The chair is rafaelweingartne. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| *** openstack changes topic to " (Meeting topic: cloudkitty)" | 14:00 | |
| openstack | The meeting name has been set to 'cloudkitty' | 14:00 |
| rafaelweingartne | Hello guys | 14:00 |
| rafaelweingartne | Roll count | 14:00 |
| priteau | o/ | 14:01 |
| mkarpiarz | Hi! | 14:03 |
| rafaelweingartne | :) | 14:03 |
| rafaelweingartne | #topic Review priorities | 14:05 |
| *** openstack changes topic to "Review priorities (Meeting topic: cloudkitty)" | 14:05 | |
| rafaelweingartne | Do you guys have some review priorities? | 14:05 |
| rafaelweingartne | At least, from my side, I do not have any so far | 14:05 |
| priteau | I'll look at the policy patch, it's one of the priorities for this cycle. | 14:06 |
| mkarpiarz | I can do testing of changes to the v2 API once I wrap my head around how v2 API works. :) | 14:07 |
| rafaelweingartne | I am mistaken, I would have some priorities | 14:08 |
| rafaelweingartne | #link https://review.opendev.org/c/openstack/cloudkitty-specs/+/770928 | 14:08 |
| rafaelweingartne | #link https://review.opendev.org/c/openstack/cloudkitty-specs/+/771513 | 14:08 |
| rafaelweingartne | Did you have time to check them? | 14:08 |
| mkarpiarz | Yeah, good shout on this one, rafaelweingartne. | 14:09 |
| rafaelweingartne | As soon as I get your feedback on them there, and if we are ok with them. I could create the patches | 14:09 |
| rafaelweingartne | and maybe we can add these to Wallaby release as well | 14:09 |
| mkarpiarz | There would need to be a change in database migration between releases, I imagine. | 14:11 |
| rafaelweingartne | exactly | 14:11 |
| rafaelweingartne | and specially for the state field deprecation | 14:12 |
| mkarpiarz | I'm totally onboard with this change and I feel like the name you suggested describes this column way better. | 14:13 |
| mkarpiarz | (but I've already given my +1 to this one) | 14:14 |
| priteau | I need to check the code but I think the last_processed_date is OK, although I'll have comments | 14:15 |
| rafaelweingartne | Thanks for the review in both of them | 14:15 |
| rafaelweingartne | priteau: I am open to suggestions | 14:15 |
| priteau | The other one I am not so sure, I was hoping the fetcher could do something automatic | 14:15 |
| rafaelweingartne | the last_processed_date was just what I came up with | 14:15 |
| rafaelweingartne | priteau: we discussed that, but it is not so simple | 14:16 |
| rafaelweingartne | There are some implication if we do that automatically, and we could always miss data, or create a situation that the system could be exploited | 14:17 |
| mkarpiarz | It might be better to have an option to manually set projects as inactive rather than no such option at all. | 14:20 |
| priteau | OK, looking at the Gnocchi fetcher code, it is based on resource.list(). I thought that it was looking at metrics. | 14:22 |
| priteau | Your approach may be appropriate then | 14:24 |
| rafaelweingartne | Thank you guys for your reviews! | 14:27 |
| rafaelweingartne | I will open for general questions and topics that people might have. | 14:28 |
| rafaelweingartne | #topic AOB | 14:28 |
| *** openstack changes topic to "AOB (Meeting topic: cloudkitty)" | 14:28 | |
| priteau | Do you want to discuss the mutators? | 14:29 |
| rafaelweingartne | If you guys want, we do have time for it | 14:30 |
| mkarpiarz | Sure. Is there anything you want to discuss? | 14:31 |
| mkarpiarz | There are no changes yet but I belive we've agreed to going the "function mappings" route for now. | 14:32 |
| mkarpiarz | Unless you guys are comfortable with using eval() with custom strings for defining mutators. | 14:33 |
| priteau | I think it's best to avoid if we can get away with something well defined | 14:33 |
| mkarpiarz | I agree. | 14:34 |
| priteau | Even for ease of use. Get the function a bit wrong and you get exceptions in CloudKitty | 14:35 |
| mkarpiarz | However mapping allowed strings to allowed mutators is already done in the code. | 14:36 |
| mkarpiarz | Granted, we could move all the mutators to a dict and add more of them, especially the negating mutator. | 14:36 |
| mkarpiarz | Or maybe we should let users chain basic mutators together so they can create more complex functions this way? | 14:38 |
| rafaelweingartne | Do we have use cases for that? | 14:39 |
| rafaelweingartne | If not, I would not go down that road | 14:39 |
| mkarpiarz | Well, the main usecase is the Prometheus one. | 14:41 |
| mkarpiarz | Where 0 in openstack_nova_server_status denotes instances that are active. | 14:42 |
| rafaelweingartne | hmm for chained mutators? | 14:43 |
| mkarpiarz | Well, if we let users chain mutator they could chain NOT with the NUMBOOL for this. :) | 14:44 |
| rafaelweingartne | I mean, is there a current use case for Prometheus where one needs to retrieve some data, and then pass it through different mutators? | 14:44 |
| rafaelweingartne | I see | 14:44 |
| mkarpiarz | But the same can easier if we introduce a "NOT NUMBOOL" mutator | 14:46 |
| mkarpiarz | which returns 1 when the measurement is a zero and 0 otherwise. | 14:48 |
| rafaelweingartne | I see | 14:49 |
| rafaelweingartne | we do not actually use mutators, but I can see where you are using it | 14:49 |
| mkarpiarz | But then what if someone wants to bill shut down instances but not these in error state? | 14:49 |
| rafaelweingartne | We use the dynamic pollster for that | 14:50 |
| rafaelweingartne | we map the states to either 0 or 1 | 14:50 |
| rafaelweingartne | and then we create a metric on Gnocchi with these, and this metric that has either 0 or 1 is rated by Cloudkitty | 14:50 |
| rafaelweingartne | we do this for octavia, VPN, and other resources as well | 14:50 |
| mkarpiarz | Ah, nice! | 14:51 |
| rafaelweingartne | That is why we do not use the mutators | 14:51 |
| rafaelweingartne | CloudKitty receives that data already prepared and processed, and it only has to rate it | 14:51 |
| rafaelweingartne | We do know that Ceilometer removed the binary that collection pollsters, but the dynamic pollsters is generic :) | 14:52 |
| mkarpiarz | I've no idea if it's possible to tell Prometheus to convert values it returns to bools before submitting them but I'll check. | 14:52 |
| rafaelweingartne | therefore, the operators create their own using YAML | 14:52 |
| rafaelweingartne | data collection pollsters* | 14:53 |
| rafaelweingartne | guys, I gotta go now | 14:56 |
| rafaelweingartne | Thank you guys for participating. Have a nice week. | 14:56 |
| priteau | Thanks both | 14:56 |
| priteau | Bye! | 14:56 |
| rafaelweingartne | If you need anything from me, just ping | 14:56 |
| rafaelweingartne | #endmeeting | 14:56 |
| mkarpiarz | Thanks and bye! | 14:56 |
| *** openstack changes topic to "Rating component for OpenStack https://wiki.openstack.org/wiki/CloudKitty" | 14:56 | |
| openstack | Meeting ended Mon Jan 25 14:56:54 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:56 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/cloudkitty/2021/cloudkitty.2021-01-25-14.00.html | 14:56 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/cloudkitty/2021/cloudkitty.2021-01-25-14.00.txt | 14:56 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/cloudkitty/2021/cloudkitty.2021-01-25-14.00.log.html | 14:56 |
| *** rafaelweingartne has quit IRC | 14:57 | |
| *** vishalmanchanda has quit IRC | 15:18 | |
| openstackgerrit | Rafael Weingartner proposed openstack/cloudkitty-specs master: Add "active" column to storage scope, and API to manage it https://review.opendev.org/c/openstack/cloudkitty-specs/+/770928 | 16:21 |
| openstackgerrit | Rafael Weingartner proposed openstack/cloudkitty-specs master: Deprecate `state` field and propose `last_processed_date` field https://review.opendev.org/c/openstack/cloudkitty-specs/+/771513 | 16:24 |
| *** sboyron has quit IRC | 17:56 | |
| *** sboyron has joined #cloudkitty | 17:57 | |
| *** irclogbot_0 has quit IRC | 18:46 | |
| *** irclogbot_3 has joined #cloudkitty | 18:49 | |
| *** sboyron has quit IRC | 21:52 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!