| *** vishalmanchanda has joined #cloudkitty | 03:59 | |
| *** sboyron has joined #cloudkitty | 06:41 | |
| *** mkarpiarz has joined #cloudkitty | 11:57 | |
| *** jopdorp has quit IRC | 13:27 | |
| *** gmann has quit IRC | 13:27 | |
| *** vishalmanchanda has quit IRC | 13:27 | |
| *** mnaser has quit IRC | 13:27 | |
| *** mnaser has joined #cloudkitty | 13:28 | |
| *** jopdorp has joined #cloudkitty | 13:29 | |
| *** vishalmanchanda has joined #cloudkitty | 13:29 | |
| *** gmann has joined #cloudkitty | 13:31 | |
| *** rafaelweingartne has joined #cloudkitty | 13:55 | |
| rafaelweingartne | #startmeeting cloudkitty | 14:00 |
|---|---|---|
| openstack | Meeting started Mon Feb 8 14:00:10 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 | Roll count | 14:00 |
| mkarpiarz | Hi! | 14:00 |
| priteau | o/ | 14:01 |
| rafaelweingartne | #topic Review priorities | 14:05 |
| *** openstack changes topic to "Review priorities (Meeting topic: cloudkitty)" | 14:05 | |
| rafaelweingartne | I do not have any review priority for today's meeting | 14:05 |
| rafaelweingartne | do you guys have any? | 14:05 |
| mkarpiarz | Any thoughts on https://review.opendev.org/c/openstack/cloudkitty/+/774084? | 14:06 |
| rafaelweingartne | I have not seen this one | 14:06 |
| mkarpiarz | (it's a WIP but feel free to comment on the idea) | 14:06 |
| rafaelweingartne | I will note it here to review | 14:07 |
| priteau | Hi mkarpiarz. I haven't had the chance to look at it yet, slowly getting back to reviews. | 14:07 |
| mkarpiarz | Sure, no problem. Whenever you guys have some spare time. :) | 14:08 |
| mkarpiarz | It's a simple change that basically adds another "if" to the mutator() function. | 14:08 |
| mkarpiarz | I think we could do with cleaning this function up so it's not just a bunch of ifs | 14:09 |
| mkarpiarz | but for now I just want to add the functionality the simplest way possible. | 14:10 |
| mkarpiarz | Also, there is this idea: https://storyboard.openstack.org/#!/story/2008584 | 14:10 |
| mkarpiarz | It might be useful to have a "keystone" module (seperate from the Keystone fetcher) | 14:12 |
| mkarpiarz | that could do things like translation of project names to UUIDs | 14:13 |
| priteau | The main question is whether we want the complexity to be in CloudKitty or in the deployment tool | 14:13 |
| priteau | Do keep in mind that you'll need to deal with user domains and project domains | 14:14 |
| mkarpiarz | Do you mean situations in which the rating user/project is in a different domain than the tenant storing metrics? | 14:16 |
| mkarpiarz | We are setting all the required auth parameters when configuring the Monasca collector so we can always use these credentials. | 14:19 |
| priteau | I mean to do a lookup against keystone v3 you'll have to handle domain information as well | 14:19 |
| priteau | And potentially the metrics user could be under a different domain than the cloudkitty service user? | 14:20 |
| priteau | Not that I am against the idea, just letting you know about the potential complexity :) | 14:21 |
| mkarpiarz | OK, so what do you think would the best way for interacting with external services be? | 14:23 |
| priteau | What kind of external service do you have in mind? | 14:25 |
| mkarpiarz | I'm asking a general question because one can imagine a usecase where users would want to call not only Keysone but maybe some other APIs. | 14:25 |
| mkarpiarz | Let's limit the question to REST-based HTTP APIs. :) | 14:26 |
| priteau | Are we still talking about fetching some information to replace what's in the metrics file? | 14:27 |
| priteau | I would be OK with some code to handle a project name instead of ID but I don't think we want to encourage cloudkitty to interact with various external APIs unless there is a strong use case for it | 14:28 |
| *** rafaelweingartne has quit IRC | 14:29 | |
| *** rafaelweingartne has joined #cloudkitty | 14:29 | |
| mkarpiarz | I see. It's a fair point. | 14:29 |
| rafaelweingartne | Sorry, I lost the connection | 14:30 |
| mkarpiarz | Just to clarify that the idea behind translation of project names to UUIDs is affecting the query | 14:33 |
| priteau | Hum, I thought it was just to get around the forced_project_id key in metrics.yml | 14:34 |
| mkarpiarz | that we run against the database with metrics and is not going to affect the metrics.yml file. | 14:34 |
| mkarpiarz | "external services" could come in when you have other parameters that you want to define in metrics.yml and then tell Cloudkitty how to translate these into elements of the query. | 14:36 |
| mkarpiarz | In this case you can think of Keystone as an "external service". :) | 14:36 |
| mkarpiarz | Apologis if the Story description was not clear but the idea is to introduce the `forced_project_name` parameter | 14:37 |
| mkarpiarz | that would then be translated by Cloudkitty to the project UUID. | 14:38 |
| priteau | If these elements in metrics.yml can be determined at deployment time and don't tend to change, I would prefer if it was the job of the deployment system to figure out the right parameters | 14:39 |
| priteau | except for keystone project name -> id which is a very common use case | 14:40 |
| mkarpiarz | OK, so a different approach maybe: | 14:41 |
| mkarpiarz | We can tell collectors to add metadata parameters to metrics, right? | 14:42 |
| mkarpiarz | Instead of authenticating against Keystone from Cloudkitty we could ask Monasca to add the "project_name" parameter to each measurement. | 14:44 |
| mkarpiarz | Then we'd only need to adjust the query that Cloudkitty sends to, say, Influx so that it includes the metadata parameter. | 14:45 |
| rafaelweingartne | That is possible already in the sumary V2 endpoint | 14:45 |
| rafaelweingartne | one can customize the projection/output of the summary request | 14:46 |
| mkarpiarz | Right, but doesn't the /v2/summary endpoint work on data that's been processed to quantities and rates? | 14:47 |
| mkarpiarz | (so stuff that is in the "cloudkitty" database and not in, say, the "monasca" one) | 14:48 |
| rafaelweingartne | it works on the influx database | 14:49 |
| mkarpiarz | Ah, I didn't know that! | 14:49 |
| rafaelweingartne | I mean the endpoint consumes it, and it would normally only return the quantity and price | 14:49 |
| rafaelweingartne | but I extended it to allow us to project more | 14:49 |
| rafaelweingartne | this allows one to create richer reports | 14:49 |
| rafaelweingartne | using the metadata that we have stored in Influx | 14:49 |
| rafaelweingartne | #link https://docs.openstack.org/cloudkitty/latest/api-reference/v2/index.html?expanded=get-a-rating-summary-detail#get-a-rating-summary | 14:50 |
| mkarpiarz | Interesting, thanks! | 14:51 |
| mkarpiarz | I'll have a look then. | 14:52 |
| mkarpiarz | (into what the /v2/summary endpoint can do) | 14:52 |
| rafaelweingartne | Guys, I guess we will have to end the meeting, as we are running out of time | 14:53 |
| mkarpiarz | I don't want to keep you guys on this tangent so feel free to move to the next item. | 14:53 |
| rafaelweingartne | We do not have any other defined topics | 14:54 |
| rafaelweingartne | #topic AOB | 14:54 |
| *** openstack changes topic to "AOB (Meeting topic: cloudkitty)" | 14:54 | |
| mkarpiarz | Haha, good tming. ;) | 14:54 |
| rafaelweingartne | Now, I open for general questions and topics that people might have. | 14:54 |
| rafaelweingartne | I will wait for another 5min. before ending the meeting | 14:54 |
| mkarpiarz | Nothing on my side. | 14:54 |
| priteau | Nothing special from me either | 14:56 |
| rafaelweingartne | Thank you guys for participating. Have a nice week. | 14:58 |
| rafaelweingartne | #endmeeting | 14:58 |
| *** openstack changes topic to "Rating component for OpenStack https://wiki.openstack.org/wiki/CloudKitty" | 14:58 | |
| openstack | Meeting ended Mon Feb 8 14:58:52 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:58 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/cloudkitty/2021/cloudkitty.2021-02-08-14.00.html | 14:58 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/cloudkitty/2021/cloudkitty.2021-02-08-14.00.txt | 14:58 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/cloudkitty/2021/cloudkitty.2021-02-08-14.00.log.html | 14:58 |
| *** rafaelweingartne has quit IRC | 14:59 | |
| *** vishalmanchanda has quit IRC | 15:45 | |
| openstackgerrit | Mariusz Karpiarz proposed openstack/cloudkitty-dashboard master: Unstack series on the "Cost Per Service Per Hour" graph https://review.opendev.org/c/openstack/cloudkitty-dashboard/+/773680 | 16:11 |
| openstackgerrit | Rafael Weingartner proposed openstack/cloudkitty master: Introduce "response_format" for the V2 summary API https://review.opendev.org/c/openstack/cloudkitty/+/774520 | 17:14 |
| openstackgerrit | Merged openstack/cloudkitty-specs master: Deprecate `state` field and propose `last_processed_at` field https://review.opendev.org/c/openstack/cloudkitty-specs/+/771513 | 17:27 |
| openstackgerrit | Rafael Weingartner proposed openstack/cloudkitty master: Introduce "response_format" for the V2 summary API https://review.opendev.org/c/openstack/cloudkitty/+/774520 | 18:38 |
| *** sboyron has quit IRC | 22:46 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!