| *** r-daneel has joined #openstack-ceilometer | 00:09 | |
| openstackgerrit | Pradeep Kilambi proposed openstack/ceilometer: Move profiler meters to yaml https://review.openstack.org/213832 | 00:15 |
|---|---|---|
| pradk | can someone fast approve this ^^ .. its just a rebase on conflict. already had a +A | 00:18 |
| openstackgerrit | Merged openstack/ceilometer: Fix description for "Inapt spelling of 'MongoDB'" https://review.openstack.org/217029 | 00:24 |
| *** rjaiswal has quit IRC | 00:40 | |
| *** _cjones_ has quit IRC | 01:05 | |
| *** thorst has quit IRC | 01:09 | |
| openstackgerrit | Merged openstack/ceilometer: Add validation tests for arithmetic, string and prefix expressions https://review.openstack.org/210619 | 01:18 |
| openstackgerrit | Ashwin Agate proposed openstack/ceilometer: Ceilometer resource table columns are not indexed https://review.openstack.org/217453 | 01:23 |
| *** r-daneel has quit IRC | 01:35 | |
| openstackgerrit | gordon chung proposed openstack/ceilometer: support custom metadata https://review.openstack.org/215348 | 01:57 |
| *** summer has joined #openstack-ceilometer | 02:17 | |
| *** summer has quit IRC | 02:18 | |
| *** summerWendy has joined #openstack-ceilometer | 02:18 | |
| openstackgerrit | Merged openstack/aodh: Fix args for get_notification_listener() https://review.openstack.org/216945 | 02:25 |
| *** ankita_wagh has joined #openstack-ceilometer | 02:27 | |
| openstackgerrit | Merged openstack/aodh: Create conf directory during devstack install phase https://review.openstack.org/217082 | 02:32 |
| *** summerWendy has quit IRC | 02:58 | |
| *** r-mibu has quit IRC | 03:18 | |
| openstackgerrit | liusheng proposed openstack/ceilometer: Keep the instance_type meta from polling and notification consistent https://review.openstack.org/217469 | 03:21 |
| *** liusheng has joined #openstack-ceilometer | 03:22 | |
| *** r-mibu has joined #openstack-ceilometer | 03:23 | |
| openstackgerrit | Ashwin Agate proposed openstack/ceilometer: Ceilometer resource table columns are not indexed https://review.openstack.org/217453 | 03:36 |
| *** deepthi has joined #openstack-ceilometer | 03:42 | |
| *** dikonoor has joined #openstack-ceilometer | 03:42 | |
| *** ankita_w_ has joined #openstack-ceilometer | 03:56 | |
| *** ankita_wagh has quit IRC | 03:58 | |
| openstackgerrit | Merged openstack/ceilometer: Change json path's to start with $. for consistency https://review.openstack.org/216330 | 04:08 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ceilometer: Updated from global requirements https://review.openstack.org/216964 | 04:13 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ceilometermiddleware: Updated from global requirements https://review.openstack.org/212245 | 04:13 |
| *** ankita_w_ has quit IRC | 04:14 | |
| openstackgerrit | Rohit Jaiswal proposed openstack/ceilometer: Fixed identity trust event types https://review.openstack.org/216979 | 04:23 |
| *** ankita_wagh has joined #openstack-ceilometer | 04:28 | |
| *** ankita_wagh has quit IRC | 04:31 | |
| *** ankita_wagh has joined #openstack-ceilometer | 04:31 | |
| *** summer has joined #openstack-ceilometer | 04:38 | |
| *** links has joined #openstack-ceilometer | 04:38 | |
| openstackgerrit | Divya K Konoor proposed openstack/ceilometer: Control Events RBAC from policy.json https://review.openstack.org/211494 | 04:41 |
| openstackgerrit | Divya K Konoor proposed openstack/ceilometer: Events RBAC needs scoped token https://review.openstack.org/212557 | 04:41 |
| *** changbl has quit IRC | 04:46 | |
| openstackgerrit | Roman Dobosz proposed openstack/ceilometer: Versioned object for Meter and Sample model https://review.openstack.org/190521 | 04:59 |
| *** Guest53419 has quit IRC | 05:01 | |
| *** zeus has joined #openstack-ceilometer | 05:03 | |
| *** zeus is now known as Guest22530 | 05:04 | |
| *** yprokule has joined #openstack-ceilometer | 05:09 | |
| *** openstackgerrit has quit IRC | 05:16 | |
| *** openstackgerrit has joined #openstack-ceilometer | 05:16 | |
| *** rjaiswal has joined #openstack-ceilometer | 05:27 | |
| *** ildikov has quit IRC | 05:34 | |
| openstackgerrit | Ashwin Agate proposed openstack/ceilometer: Ceilometer resource table columns are not indexed https://review.openstack.org/217453 | 05:35 |
| *** agatea has joined #openstack-ceilometer | 05:48 | |
| *** ankita_w_ has joined #openstack-ceilometer | 05:51 | |
| *** ankita_wagh has quit IRC | 05:55 | |
| *** afazekas_ has joined #openstack-ceilometer | 06:06 | |
| *** fawadkhaliq has joined #openstack-ceilometer | 06:21 | |
| *** ankita_w_ has quit IRC | 06:23 | |
| *** ankita_wagh has joined #openstack-ceilometer | 06:25 | |
| *** ankita_wagh has quit IRC | 06:27 | |
| *** ankita_wagh has joined #openstack-ceilometer | 06:28 | |
| openstackgerrit | liusheng proposed openstack/ceilometer: Keep the instance_type meta from polling and notification consistent https://review.openstack.org/217469 | 06:28 |
| *** ankita_w_ has joined #openstack-ceilometer | 06:46 | |
| *** ankita_wagh has quit IRC | 06:49 | |
| *** ankita_w_ has quit IRC | 06:50 | |
| openstackgerrit | Merged openstack/gnocchi: Create conf directory during install phase https://review.openstack.org/217083 | 06:51 |
| *** ildikov has joined #openstack-ceilometer | 06:59 | |
| *** cdent has joined #openstack-ceilometer | 07:04 | |
| *** tobasco_ is now known as tobasco | 07:14 | |
| cdent | sileht can you confirm this is cool: https://review.openstack.org/#/c/215313/ ? thanks | 07:20 |
| sileht | cdent, yes it's cool :) | 07:21 |
| cdent | thanks | 07:22 |
| cdent | I figured if I make enough infra changes in one single week I can make everything explode and we can all go live on the beach | 07:22 |
| *** agatea has quit IRC | 07:23 | |
| sileht | cdent, do new some lib releases to add some pepper for debugging people if you want to really appreciate the beach | 07:25 |
| cdent | :) | 07:25 |
| openstackgerrit | Roman Dobosz proposed openstack/ceilometer: Versioned object for Event model https://review.openstack.org/181338 | 07:27 |
| *** agatea has joined #openstack-ceilometer | 07:27 | |
| *** agatea has quit IRC | 07:29 | |
| *** agatea has joined #openstack-ceilometer | 07:30 | |
| *** shardy has joined #openstack-ceilometer | 07:31 | |
| *** agatea has quit IRC | 07:35 | |
| *** rjaiswal has quit IRC | 07:40 | |
| *** agatea has joined #openstack-ceilometer | 07:44 | |
| *** yassine has joined #openstack-ceilometer | 07:49 | |
| *** ildikov has quit IRC | 07:59 | |
| *** jaosorior has joined #openstack-ceilometer | 08:02 | |
| *** Ala has joined #openstack-ceilometer | 08:07 | |
| *** boris-42 has quit IRC | 08:10 | |
| *** ildikov has joined #openstack-ceilometer | 08:12 | |
| *** summer has quit IRC | 08:13 | |
| *** summer has joined #openstack-ceilometer | 08:17 | |
| *** shardy_ has joined #openstack-ceilometer | 08:51 | |
| *** shardy has quit IRC | 08:52 | |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/ceilometer: WIP: gnocchi: use events to crud resources https://review.openstack.org/217017 | 08:53 |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/ceilometer: gnocchi: cleanup instance resource definition https://review.openstack.org/217583 | 08:53 |
| *** shardy_ has quit IRC | 08:56 | |
| *** shardy has joined #openstack-ceilometer | 08:57 | |
| sileht | cdent, jd__ I wonder if a instance disk should be considered as new kind of gnocchi resource | 08:58 |
| cdent | we’re going to have this problem a lot | 08:59 |
| sileht | for example we receive this metric from ceilometer: disk.device.read.requests.rate | 08:59 |
| sileht | with an id taht looks like: 1880d27c-678f-45bf-98ce-12e21e52a1c7-vda | 08:59 |
| cdent | I kind of think of it as a rather fundamental limitation in gnocchi | 08:59 |
| cdent | that kind of id is why I did the uuid5 thing | 08:59 |
| sileht | I see two solutions: | 08:59 |
| sileht | * I create the gnocchi metric vda_disk.device.read.requests.rate on instance 1880d27c-678f-45bf-98ce-12e21e52a1c7 | 09:00 |
| sileht | * I create the gnocchi metric disk.device.read.requests.rate on *instance-disk" 1880d27c-678f-45bf-98ce-12e21e52a1c7-vda | 09:00 |
| cdent | Is “vda” static? | 09:03 |
| cdent | or is there also vdb c etc | 09:03 |
| sileht | you have vdb/vbc/hdd/... | 09:04 |
| cdent | that should be an attribute of a resource then, shouldn’t it? | 09:06 |
| cdent | not in an id of a resource, nor in a metric name | 09:06 |
| sileht | cdent, the instance can have many disk | 09:07 |
| cdent | the metric is disk.device.read.requests.rate, the resource is the disk | 09:07 |
| sileht | ok so solution 2 is more logic | 09:07 |
| cdent | but the resource id should not contain the disk name | 09:07 |
| cdent | (internally) | 09:07 |
| cdent | it should be metadata/attribute | 09:08 |
| sileht | cdent, that you patch job ;) | 09:08 |
| cdent | still has to get into the metadata somehow | 09:08 |
| sileht | cdent, about attributes I need to check if I can have the list of disk from the instance.create notification | 09:08 |
| sileht | we have the same issue for network interfaces attached to the vm | 09:08 |
| * cdent nods | 09:09 | |
| jd__ | sileht: (how) do you link the disk and instance? | 09:27 |
| sileht | jd__, just a instance_id attribute on the gnocchi resource instsance_disk | 09:27 |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Add instance_disk and instance_network resources https://review.openstack.org/217597 | 09:28 |
| jd__ | sileht: sounds good | 09:28 |
| sileht | something like that: https://review.openstack.org/#/c/217597/1/gnocchi/rest/__init__.py | 09:28 |
| sileht | the hardest part is to create the resource correclty from the samples and/or from the events | 09:29 |
| *** nadya has joined #openstack-ceilometer | 09:41 | |
| cdent | biab | 09:46 |
| *** cdent has quit IRC | 09:46 | |
| *** ildikov has quit IRC | 09:51 | |
| *** cdent has joined #openstack-ceilometer | 09:56 | |
| *** nadya has quit IRC | 09:58 | |
| *** summer has quit IRC | 10:02 | |
| openstackgerrit | Ilya Tyaptin proposed openstack/ceilometer: [ES] Implement event time to live https://review.openstack.org/200573 | 10:04 |
| *** ildikov has joined #openstack-ceilometer | 10:05 | |
| *** Kennan has quit IRC | 10:38 | |
| *** lan_ has quit IRC | 10:39 | |
| *** lan_ has joined #openstack-ceilometer | 10:52 | |
| *** Kennan has joined #openstack-ceilometer | 10:55 | |
| openstackgerrit | gordon chung proposed openstack/ceilometer: allow configurable pipeline partitioning https://review.openstack.org/214813 | 10:56 |
| cdent | shardy can I direct your attention, or the attention of some other heat person to: http://lists.openstack.org/pipermail/openstack-dev/2015-August/072958.html | 11:03 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: Only exclude Alembic 0.8.1 https://review.openstack.org/217639 | 11:12 |
| cdent | jd__, sileht: any ideas on this: http://logs.openstack.org/13/215313/6/gate/gate-gnocchi-python27/ff32aa7/testr_results.html.gz | 11:12 |
| jd__ | cdent: a call to process_measures() forgotten? | 11:14 |
| jd__ | I imagine it's a random failure | 11:15 |
| cdent | that's my guess too | 11:15 |
| * cdent inspects | 11:15 | |
| cdent | ah, process_measures is called before the request is called | 11:18 |
| cdent | that's a bit backwards | 11:18 |
| *** thorst has joined #openstack-ceilometer | 11:27 | |
| *** thorst1 has joined #openstack-ceilometer | 11:29 | |
| *** thorst has quit IRC | 11:32 | |
| openstackgerrit | Chris Dent proposed openstack/gnocchi: In test_rest run process_measures after a request https://review.openstack.org/217650 | 11:33 |
| openstackgerrit | gordon chung proposed openstack/ceilometer: support custom metadata https://review.openstack.org/215348 | 11:43 |
| *** gordc has joined #openstack-ceilometer | 11:45 | |
| openstackgerrit | Lianhao Lu proposed openstack/ceilometer: Added snmp declarative hardware pollster https://review.openstack.org/217667 | 11:46 |
| gordc | llu: i guess i may need to ask for FFE now :) ^ | 11:49 |
| gordc | dikonoor: https://review.openstack.org/#/c/211494/ | 11:53 |
| shardy | cdent: thanks, I'll reply shortly - have you raised a bug and marked it as affecting all those projects? | 11:53 |
| gordc | after that patch, can you describe what the behaviour will be? | 11:53 |
| cdent | not yet, but will do shardy | 11:54 |
| dikonoor | gordc:hi..checking | 11:54 |
| shardy | cdent: we probably need to chat to stevebaker as IIRC he set up most of our gate jobs, particularly the integration/functional tests | 11:54 |
| shardy | cdent: Ok, thanks - that's probably the easiest way to coordinate things IMO | 11:54 |
| cdent | I've also just sent a mail to you and some other relevant folk, just so it is covered, so it will be redandant | 11:54 |
| cdent | gordc: pardon what? | 11:55 |
| dikonoor | gordc: This removes the hard-coded rule that says only admins can view events. Instead , it uses the roles specified in the new rules to determine who should be allowed to access events | 11:55 |
| cdent | gordc: nm | 11:55 |
| gordc | cdent: quoi? | 11:55 |
| gordc | ok. | 11:55 |
| cdent | I thought you were asking me something when you were not | 11:56 |
| gordc | ah cool cool | 11:56 |
| *** deepthi has quit IRC | 11:56 | |
| dikonoor | gordc: So, if you want only admins to see your events, the rules should look like it is in this >> https://review.openstack.org/#/c/211494/16/etc/ceilometer/policy.json.sample | 11:56 |
| gordc | dikonoor: ok. so basically we can define which roles can see all events | 11:57 |
| dikonoor | gordc: right | 11:57 |
| gordc | ok. that's a bit more flexible. | 11:57 |
| dikonoor | gordc: and there are two different rules defined - one for get all events (index) and the other for get one event (show) | 11:57 |
| llu | gordc: yes, looks like I need to ask for FFE, my bad | 11:58 |
| gordc | dikonoor: if we want to create it so results are limited per role as we said, how would that happen? what's the rule for non-admin see only it's own events, and admin see it's own project? | 11:58 |
| gordc | or is that not what rules are for? | 11:58 |
| gordc | llu: no worries, good to have feature | 11:59 |
| gordc | llu: are there more patches? | 11:59 |
| dikonoor | gordc: this changeset is not for that.. this is just to give flexibility on which roles of users can make these rest calls | 12:00 |
| dikonoor | gordc: After this is done, I will create another patch which will add a check in the events.py code that says if non-admin user, then show only events for that user | 12:00 |
| gordc | dikonoor: understood. ok. so the actually filtering is not actually policy related. | 12:01 |
| dikonoor | gordc: and if admin user , then show events for that user's project + all events that don't have project infor | 12:01 |
| llu | gordc: yes, there will be 1 or 2 more patches. I'm struggling to have each separate patches not breaking the CM, also writing unit test cases | 12:01 |
| dikonoor | gordc: but that needs support that we discussed about yesterday | 12:01 |
| dikonoor | gordc: support to run a db query that says project_id= abc or project_id=null | 12:01 |
| llu | but the remaining patches should be much smaller and simple compared to the first one | 12:02 |
| gordc | dikonoor: sure. i'll try helping out. that's why i'm asking whether the policy side of things is done. (i'm not a policy expert) | 12:02 |
| gordc | llu: good to know :) | 12:02 |
| *** idegtiarov has quit IRC | 12:08 | |
| openstackgerrit | liusheng proposed openstack/ceilometer: Fix requeue process on event handling error https://review.openstack.org/217677 | 12:09 |
| gordc | cdent: jd__: added a comment https://review.openstack.org/#/c/214813/ not sure what we should default the pipeline processing queue value to | 12:10 |
| *** idegtiarov has joined #openstack-ceilometer | 12:12 | |
| dikonoor | gordc: I think i can submit another patch that is specifically for non-admin users being able to see only events that they have generated | 12:12 |
| *** jasonamyers has joined #openstack-ceilometer | 12:12 | |
| dikonoor | gordc: and the last pending one would be for admin users which will depend on your changes | 12:13 |
| gordc | dikonoor: that works for me. | 12:13 |
| gordc | please work on non-admin route for now | 12:13 |
| dikonoor | gordc: also , do you think https://review.openstack.org/#/c/211494/ can be approved ? :) | 12:13 |
| gordc | dikonoor: it needs two... but i can review today | 12:14 |
| dikonoor | gordc: ok..thanks.. I have noticed that when you give a tick, another tick soon follows ;-P | 12:15 |
| dikonoor | gordc: for the non-admin part , I am planning to add a trait filter for user_id filter somewhere here >> https://github.com/openstack/ceilometer/blob/master/ceilometer/api/controllers/v2/events.py#L167 | 12:16 |
| dikonoor | something like this: | 12:17 |
| dikonoor | traits_filter.append({"key": "project_id", | 12:17 |
| dikonoor | trait_type: project_id, | 12:17 |
| dikonoor | "op": 'eq'}) | 12:17 |
| dikonoor | traits_filter.append({"key": "user_id", | 12:17 |
| dikonoor | trait_type: user_id, | 12:17 |
| dikonoor | "op": 'eq'}) | 12:17 |
| dikonoor | gordc: Do you have any suggestions on any better way? | 12:17 |
| gordc | dikonoor: i have two accounts. it's just me, two times :) | 12:17 |
| cdent | sugar, the influxdb gate job just merged before the corresponding code did, maybe due to a ever so slight typo in the depends-on clause. jd__, sileht | 12:18 |
| dikonoor | gordc: :) I should have guessed | 12:18 |
| gordc | dikonoor: i think that makes sense.^^ | 12:18 |
| dikonoor | gordc: ok..then i will proceed with the changes | 12:19 |
| gordc | dikonoor: if operator changes policy json to allow non-admin access... what happens to results? are they still filtered? | 12:19 |
| gordc | can they say, non-admin users can see everything in policy.json? | 12:20 |
| dikonoor | gordc: yes..thats exactly how non-admin users get to see events | 12:20 |
| dikonoor | gordc: with the current logic (ie without my changes) , only admins can see events | 12:20 |
| gordc | dikonoor: so policy just defines who can access api endpoint, it doesn't describe what content they can see. | 12:21 |
| dikonoor | gordc: right .with the flexibility to define which roles can access events, we have the option to let admins and non-admins access events | 12:21 |
| dikonoor | gordc: once they access , then we add filters so that they see restricted stuff | 12:22 |
| *** haigang has joined #openstack-ceilometer | 12:22 | |
| gordc | dikonoor: ah, got it. i was under impression that policy defined the filtering as well. so it just defines role access to endpoint | 12:22 |
| gordc | cool cool. it makes more sense now | 12:23 |
| *** haigang has quit IRC | 12:26 | |
| openstackgerrit | gordon chung proposed openstack/ceilometer: make telemetry sample payloads dictionaries https://review.openstack.org/213851 | 12:30 |
| *** openstackgerrit has quit IRC | 12:31 | |
| *** openstackgerrit has joined #openstack-ceilometer | 12:31 | |
| *** edmondsw has joined #openstack-ceilometer | 12:32 | |
| openstackgerrit | Merged openstack/gnocchi: Only exclude Alembic 0.8.1 https://review.openstack.org/217639 | 12:34 |
| dikonoor | gordc: Thanks for the review and approval. The one pending is https://review.openstack.org/#/c/211509/ | 12:58 |
| *** changbl has joined #openstack-ceilometer | 13:06 | |
| *** I has joined #openstack-ceilometer | 13:06 | |
| *** I is now known as Guest62749 | 13:07 | |
| openstackgerrit | Merged openstack/gnocchi: Rudimentary support for influxdb in devstack plugin https://review.openstack.org/215313 | 13:08 |
| cdent | lets see what exciting ways this goes boom | 13:11 |
| gordc | dikonoor: i'm not sure the value of that patch but i can +1.. but i'd like more opinions | 13:12 |
| dikonoor | gordc: This patch is required to be able to do the filtering that we discussed few min back.. This patch ensures that user_id and project_id is set for all events and that can be used for filtering | 13:13 |
| dikonoor | gordc: gordc: without this patch , the events are not going to have user/project information and filtering based on user_id for non-admins and based on project_id for admins wouldn't make any sense | 13:15 |
| gordc | :) | 13:15 |
| gordc | understood | 13:15 |
| *** cmyster has quit IRC | 13:18 | |
| *** afazekas_ has quit IRC | 13:26 | |
| *** julim has joined #openstack-ceilometer | 13:28 | |
| *** jasonamyers has quit IRC | 13:33 | |
| *** Guest62749 has quit IRC | 13:39 | |
| *** ityaptin has quit IRC | 13:41 | |
| cdent | dikonoor: I can't tell if the comment you just made about the rbac stuff is saying "yes, the traits dont' need to be covered here" or "oh, yeah, they do" | 13:42 |
| dikonoor | cdent: :) Let me fix that then.. | 13:42 |
| dikonoor | cdent : thanks for pointing that out | 13:43 |
| cdent | I can sort of guess, but for clarity and all that ... | 13:43 |
| *** ityaptin has joined #openstack-ceilometer | 13:43 | |
| dikonoor | cdent : I read the question again and I am little confused. Could you explain the question again ? :-| | 13:58 |
| dikonoor | cdent : I didn't follow the question - I would guess that traits are not private-to-users or roles but I don't actually know. | 13:58 |
| *** exploreshaifali has joined #openstack-ceilometer | 13:59 | |
| cdent | liusheng's original question is: do we need an enforce statement on the get_all method at line 233 | 13:59 |
| cdent | MY followup is to say : I suspect event types are not private | 14:01 |
| cdent | although I said traits it wasn't what I meant :) | 14:01 |
| gordc | cdent: yep | 14:01 |
| gordc | event_types and trait_types are just keys... i think the idea is to restrict values. | 14:01 |
| gordc | dikonoor: correct? | 14:01 |
| *** Kennan has quit IRC | 14:02 | |
| dikonoor | gordc: right | 14:02 |
| cdent | cool, then I reckon we're good to go, yeah? | 14:02 |
| dikonoor | cdent : gordc: yeah..all good ! | 14:03 |
| cdent | unless somebody objects I'll make it go | 14:03 |
| *** Kennan has joined #openstack-ceilometer | 14:03 | |
| *** jwcroppe has joined #openstack-ceilometer | 14:03 | |
| *** fawadkhaliq has quit IRC | 14:04 | |
| gordc | cdent: none here. /me hopes it doesn't break | 14:04 |
| *** fawadkhaliq has joined #openstack-ceilometer | 14:05 | |
| *** fawadkhaliq has quit IRC | 14:08 | |
| *** ildikov has quit IRC | 14:10 | |
| *** rbak has joined #openstack-ceilometer | 14:11 | |
| *** eglynn_ has quit IRC | 14:11 | |
| *** jasonamyers has joined #openstack-ceilometer | 14:14 | |
| openstackgerrit | Chris Dent proposed openstack/gnocchi: In test_rest run process_measures after a request https://review.openstack.org/217650 | 14:15 |
| *** links has quit IRC | 14:37 | |
| *** yprokule has quit IRC | 14:48 | |
| *** ildikov has joined #openstack-ceilometer | 14:53 | |
| *** afazekas_ has joined #openstack-ceilometer | 14:54 | |
| *** jaypipes has joined #openstack-ceilometer | 14:55 | |
| *** eandersson has joined #openstack-ceilometer | 14:58 | |
| *** maubry has joined #openstack-ceilometer | 15:00 | |
| eandersson | How would I go about to disable snmp / discovery (ceilometer.hardware.discovery) in Ceilometer/Kilo? | 15:00 |
| *** fguillot has joined #openstack-ceilometer | 15:00 | |
| eandersson | It is erroring out, so I rather just disable it for now. | 15:01 |
| gordc | eandersson: we're about to start a meeting at openstack-meeting so you might not get a reply for next while. sorry. | 15:02 |
| eandersson | no worries =] | 15:02 |
| eandersson | no rush | 15:02 |
| *** eglynn has joined #openstack-ceilometer | 15:02 | |
| openstackgerrit | Lianhao Lu proposed openstack/ceilometer: Added snmp declarative hardware pollster https://review.openstack.org/217667 | 15:02 |
| *** fawadkhaliq has joined #openstack-ceilometer | 15:06 | |
| *** fawadkhaliq has quit IRC | 15:10 | |
| *** exploreshaifali has quit IRC | 15:22 | |
| *** links has joined #openstack-ceilometer | 15:22 | |
| *** afazekas_ has quit IRC | 15:27 | |
| *** idegtiarov_ has joined #openstack-ceilometer | 15:30 | |
| *** afazekas_ has joined #openstack-ceilometer | 15:35 | |
| *** changbl has quit IRC | 15:37 | |
| *** agatea has quit IRC | 15:43 | |
| *** jruano has joined #openstack-ceilometer | 15:43 | |
| *** changbl has joined #openstack-ceilometer | 15:44 | |
| openstackgerrit | David Wahlstrom proposed openstack/aodh: Add storage documentation https://review.openstack.org/217771 | 15:49 |
| *** afazekas_ has quit IRC | 15:50 | |
| *** changbl has quit IRC | 15:51 | |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Add instance_disk and instance_network resources https://review.openstack.org/217597 | 15:55 |
| cdent | sileht: i'm seeing a fair bit of integration test failure today, for example here: https://review.openstack.org/#/c/217650/ | 15:55 |
| sileht | cdent, https://review.openstack.org/#/c/215604/ fixes that | 15:56 |
| cdent | ossum | 15:56 |
| *** IBerezovskiy has quit IRC | 15:59 | |
| *** IBerezovskiy has joined #openstack-ceilometer | 15:59 | |
| jasonamyers | So I wanted to chat about the midcycle, mostly around the idea of should we do in person or virtual | 15:59 |
| jasonamyers | I kept pushing for in person, but travel to dublin might not work for many (judging based on response to prior attempts at organizing a date etc) | 16:00 |
| jasonamyers | would we rather do a virtual again this time? | 16:00 |
| ildikov | jasonamyers: from the perspective of one was cancelled and then the next one almost I would say virtual seems more bullet proof | 16:00 |
| jasonamyers | I think it was helpful last time | 16:00 |
| ildikov | I'm on the side of face to face too though! | 16:01 |
| gordc | jasonamyers: can we expense coffee makers if we do virtual | 16:01 |
| jasonamyers | ildikov yeah :( I want to make sure we do something | 16:01 |
| gordc | it's still cheaper than hotel/flight | 16:01 |
| jasonamyers | that depends on your megalo corps expense policies | 16:01 |
| cdent | If we're going to have low attendance in tokyo (because of distance, expense etc) (are we?) then having an in person midcycle seems more important/useful | 16:02 |
| jasonamyers | I just wanna make sure we do something for a midcycle | 16:02 |
| jasonamyers | how all is going to Tokyo | 16:02 |
| jasonamyers | s/how/who | 16:02 |
| gordc | we will probably have low attendance from places outside asia | 16:02 |
| idegtiarov_ | gordc, cdent, llu, liusheng could you please take a look at that spec https://review.openstack.org/#/c/162167 do we still have any unacceptable points there | 16:02 |
| sileht | cdent, I run the job on my review to ensure we didn't break something since tuesday | 16:02 |
| gordc | i think our biggest hurdle is we have small contributor base and probably the most distributed as well. | 16:03 |
| cdent | jasonamyers: I'm going I don't know who else | 16:03 |
| ildikov | I'm going too | 16:03 |
| idegtiarov_ | it seems that main point were already discussed and approved, don't they? | 16:03 |
| jasonamyers | we don't have that many people so if the main ones of you are going to tokyo then virtual will be fine | 16:03 |
| jasonamyers | jd_ and gordc are you going to tokyo? | 16:04 |
| gordc | idegtiarov_: will take a look after liberty-3. | 16:04 |
| gordc | sorry, got a few bigger items currently. | 16:04 |
| idegtiarov_ | gordc, got it, thankd | 16:04 |
| jd__ | jasonamyers: はい | 16:04 |
| gordc | jasonamyers: i'll be there. | 16:05 |
| idegtiarov_ | s/thankd/thanks | 16:05 |
| jasonamyers | so sounds like many of the people from last time will be there | 16:05 |
| jd__ | jasonamyers: don't tell that like it ain't a party | 16:05 |
| jasonamyers | I was selfishly wanting in person because I'm not goign to tokyo | 16:05 |
| gordc | jasonamyers: i think we all want in person. | 16:06 |
| jasonamyers | jd__ they had to shut us down last time :P | 16:07 |
| openstackgerrit | Merged openstack/gnocchi: In test_rest run process_measures after a request https://review.openstack.org/217650 | 16:07 |
| jd__ | jasonamyers: hehe | 16:07 |
| jasonamyers | okay I'll make sure to keep the funding open for it | 16:07 |
| gordc | i just don't know where or when... December is pretty much a writeoff for most people | 16:07 |
| gordc | so it's much harder to do midcycles in these 6months | 16:07 |
| jasonamyers | and I'm flexible on all of that with Jan kinda being the cut off for me | 16:08 |
| sileht | cdent, jd__ do you think we should remove ceilometer from the "integrated gate queue" ? | 16:08 |
| openstackgerrit | Chris Dent proposed openstack/gnocchi: Turn on influxdb in gate_hook https://review.openstack.org/217782 | 16:08 |
| jasonamyers | so late Nov - end of JAN is my budget window | 16:08 |
| *** afazekas_ has joined #openstack-ceilometer | 16:08 | |
| jd__ | sileht: yes | 16:08 |
| gordc | sileht: so we can merge faster? | 16:08 |
| sileht | gordc, yes | 16:08 |
| gordc | lol | 16:08 |
| gordc | if heat wasn't dependent on us, i think we could | 16:09 |
| jd__ | heat is going to depend on Aodh which is not in integrated AFAIK | 16:09 |
| jd__ | :-> | 16:09 |
| gordc | jasonamyers: i'm most available in january... but i have no idea of others. | 16:10 |
| *** changbl has joined #openstack-ceilometer | 16:10 | |
| gordc | jasonamyers: also we have a strong contingent of red hat/mirantis so they're all dependent on same pool of money | 16:11 |
| gordc | jd__: but they populate ceilometer for their tests. | 16:11 |
| jasonamyers | nod | 16:11 |
| gordc | that said, i guess we don't know when things break heat since we don't gate on that part. | 16:12 |
| jasonamyers | that's why we picked ireland originally I thought was to help with some of that | 16:12 |
| jasonamyers | so I'll send an email with what I can provide location wise and date wise | 16:12 |
| jasonamyers | hopefully that will spark more conversation and commitment | 16:13 |
| gordc | jasonamyers: yeah. no idea what the budget for mirantis/red hat is... don't know my own.lol | 16:13 |
| gordc | jasonamyers: sounds like a good idea | 16:13 |
| jasonamyers | it's a short hope from the summit to end of JAN after all the holidays and vacation just wanna make sure we don't missing getting together in some manner | 16:13 |
| *** fawadkhaliq has joined #openstack-ceilometer | 16:13 | |
| jasonamyers | s/hope/hop | 16:13 |
| gordc | jasonamyers: agreed | 16:14 |
| cdent | Something I wanted to bring up in the meeting was my comment at the end of this: https://review.openstack.org/#/c/216243/ | 16:14 |
| cdent | (versioned objects) | 16:14 |
| *** jasonamyers has quit IRC | 16:16 | |
| openstackgerrit | Merged openstack/ceilometer: remove jsonpath-rw requirement https://review.openstack.org/216794 | 16:18 |
| gordc | cdent: your comment or the reply? | 16:19 |
| cdent | both | 16:20 |
| gordc | based on reply, it seems like it's not very valid. | 16:20 |
| gordc | s/valid/useful/ | 16:20 |
| cdent | that's the impression I got to | 16:20 |
| cdent | o | 16:20 |
| openstackgerrit | gordon chung proposed openstack/ceilometer: make telemetry sample payloads dictionaries https://review.openstack.org/213851 | 16:21 |
| gordc | it seems like we just need to review how useful it is when we send notification from aodh -> heat | 16:21 |
| cdent | which is why I brought it up | 16:21 |
| cdent | if it's useless why bother | 16:21 |
| cdent | we don't send a distributed object to heat | 16:21 |
| cdent | we send a dict | 16:21 |
| gordc | yeah. | 16:21 |
| cdent | I think | 16:21 |
| gordc | .. | 16:21 |
| gordc | if it's oslo.messaging it's all serialised to regardless | 16:22 |
| gordc | s/to// | 16:22 |
| gordc | i think it might be helpful to have version to define payload but i don't know if versionedobjects is required | 16:23 |
| cdent | my hope is that someone besides me will join the review to comment on which way it should go | 16:25 |
| gordc | cdent: mailinglist? '[versionedobjects][ceilometer] explain why versionedobjects is useful' | 16:25 |
| *** jaosorior has quit IRC | 16:25 | |
| cdent | if you're volunteering, then sure :) | 16:25 |
| gordc | ... fine! | 16:26 |
| cdent | sweet | 16:26 |
| *** exploreshaifali has joined #openstack-ceilometer | 16:26 | |
| gordc | yeah, right now it seems like those implementing are aware of the benefits of versionedobjects but not versionedobjects+ceilometer | 16:26 |
| *** _cjones_ has joined #openstack-ceilometer | 16:26 | |
| gordc | or maybe i'm just not aware | 16:26 |
| cdent | yeah it is very hard to tell who is the ignorant one: me, you, them, someone else | 16:27 |
| *** afazekas_ has quit IRC | 16:27 | |
| gordc | whoa whoa! no need to point fingers at me. | 16:28 |
| * cdent points | 16:28 | |
| cdent | everywhere | 16:28 |
| pradk | ildikov, gordc, llu, i have the initial draft of declarative meters docs https://review.openstack.org/#/c/217780/ .. lemme know your comments | 16:37 |
| *** Ala has quit IRC | 16:38 | |
| *** boris-42 has joined #openstack-ceilometer | 16:39 | |
| pradk | gordc, hmm not sure whats taking so long for profiler meter patch to merge its in queue for 14 hr 25 min and still not picked up.. is gate down? | 16:41 |
| cdent | i'm hearing numbers > 15 hours pradk | 16:41 |
| cdent | does zuul have it? | 16:41 |
| pradk | awesome | 16:41 |
| pradk | yea its queued | 16:41 |
| gordc | pradk: it's... something | 16:41 |
| pradk | thats helpful thx gordc ;) | 16:42 |
| gordc | lol it's the gate before code freeze. i'm pretty sure it's going to hit 24hrs | 16:43 |
| gordc | we all have short memories... what was it last time? | 16:43 |
| ildikov | pradk: cool, tnx! | 16:43 |
| cdent | I guess the issue for me is I thought code freeze was further away than it actually is. When is it? | 16:43 |
| gordc | sept 1-4 | 16:44 |
| cdent | when does m open? | 16:44 |
| gordc | https://wiki.openstack.org/wiki/Liberty_Release_Schedule | 16:45 |
| cdent | thank you | 16:45 |
| *** links has quit IRC | 16:45 | |
| gordc | pretty much whenever we declare rc1 | 16:45 |
| * cdent needs to work on his Gord C is a big jerk ptl self-nomination message | 16:46 | |
| gordc | we have a few weeks usually to go nuts with rc and then when we cap rc1 we'll have mitaka open too | 16:46 |
| gordc | cdent: you're suppose to tell them something new | 16:46 |
| gordc | everyone knows i have an iron fist | 16:46 |
| cdent | s/a big jerk/a realy big jerk/ | 16:46 |
| *** eglynn has quit IRC | 16:50 | |
| *** afazekas_ has joined #openstack-ceilometer | 16:52 | |
| *** harlowja has joined #openstack-ceilometer | 16:55 | |
| *** Guest22530 has quit IRC | 17:04 | |
| *** afazekas_ has quit IRC | 17:07 | |
| *** afazekas_ has joined #openstack-ceilometer | 17:09 | |
| cdent | if/when do we plan a gnocchi client? | 17:12 |
| jaypipes | cdent: neat :) | 17:12 |
| cdent | jaypipes: that's all sileht's work, he picked up on gabbi very quickly | 17:13 |
| *** afazekas_ has quit IRC | 17:15 | |
| *** afazekas_ has joined #openstack-ceilometer | 17:27 | |
| *** ankita_wagh has joined #openstack-ceilometer | 17:41 | |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/ceilometer: gnocchi: quote the resource_id in url https://review.openstack.org/216628 | 17:43 |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/ceilometer: gnocchi: quote the resource_id in url https://review.openstack.org/216628 | 17:46 |
| *** afazekas_ has quit IRC | 17:48 | |
| *** exploreshaifali has quit IRC | 17:48 | |
| openstackgerrit | Chris Dent proposed openstack/gnocchi: devstack: explicit path in run_process calls https://review.openstack.org/217825 | 17:49 |
| *** kfox1111 has joined #openstack-ceilometer | 17:57 | |
| kfox1111 | so... just upgraded to kilo. how do I switch ceilometer-compute-agent to use keystone internal rather then public services? | 17:58 |
| cdent | sileht: if you're still around can you check this one: https://review.openstack.org/#/c/217782/ | 18:01 |
| cdent | without that the influxdb experimental job doesn't really do anything | 18:01 |
| sileht | cdent, done | 18:01 |
| cdent | thanks | 18:01 |
| *** Kennan has quit IRC | 18:03 | |
| *** Kennan has joined #openstack-ceilometer | 18:03 | |
| *** cdent has quit IRC | 18:08 | |
| gordc | kfox1111: what's keystone internal? vs keystone public? the endpoint urls? | 18:11 |
| *** dikonoor has quit IRC | 18:11 | |
| *** yassine has quit IRC | 18:25 | |
| kfox1111 | yeah. it was publicURL/internalURL. | 18:28 |
| gordc | you should be able to just set os_endpoint_type = internalURL in [service_credentials] section of conf file | 18:35 |
| openstackgerrit | Merged openstack/gnocchi: Turn on influxdb in gate_hook https://review.openstack.org/217782 | 18:39 |
| *** exploreshaifali has joined #openstack-ceilometer | 18:40 | |
| openstackgerrit | Divya K Konoor proposed openstack/ceilometer: Control Events RBAC from policy.json https://review.openstack.org/211494 | 18:42 |
| openstackgerrit | Divya K Konoor proposed openstack/ceilometer: Events RBAC needs scoped token https://review.openstack.org/212557 | 18:42 |
| openstackgerrit | Divya K Konoor proposed openstack/ceilometer: WIP Event filtering for non-admin users https://review.openstack.org/217844 | 18:42 |
| *** ankita_wagh has quit IRC | 18:54 | |
| *** jruano has quit IRC | 18:54 | |
| *** jruano has joined #openstack-ceilometer | 19:00 | |
| *** fawadkhaliq has quit IRC | 19:35 | |
| openstackgerrit | Merged openstack/aodh: Add storage documentation https://review.openstack.org/217771 | 19:37 |
| openstackgerrit | Merged openstack/ceilometer: Create conf directory during devstack install phase https://review.openstack.org/217081 | 19:45 |
| gordc | pradk: you're very lucky... merge conflict | 19:49 |
| pradk | damn | 19:49 |
| gordc | i appreciate how much you censored yourself. | 19:49 |
| gordc | i can sub in the appropriate words | 19:49 |
| pradk | lol k i'll rebase again | 19:50 |
| gordc | i think your patch merge conflict with your $. patch 16 hours ago.lol! | 19:51 |
| pradk | ah i noticed "skipped" at some point in zuul this morning.. i should have guessed | 19:56 |
| *** jruano has quit IRC | 19:57 | |
| gordc | pradk: i noticed that too.. i just figured it was just a random restart... now we know. if it's 'skipped' you should probably do something | 19:58 |
| pradk | yea | 19:59 |
| openstackgerrit | Pradeep Kilambi proposed openstack/ceilometer: Move profiler meters to yaml https://review.openstack.org/213832 | 20:00 |
| pradk | gordc, k tox tests pass after resolving conflict ^^ .. lets see 5th is the charm ;) | 20:01 |
| pradk | oh and if you could please fast approve ^^ | 20:02 |
| gordc | done | 20:04 |
| pradk | ty sir | 20:05 |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Add instance_disk and instance_network resources https://review.openstack.org/217597 | 20:05 |
| *** fawadkhaliq has joined #openstack-ceilometer | 20:06 | |
| openstackgerrit | gordon chung proposed openstack/ceilometer: support custom metadata https://review.openstack.org/215348 | 20:16 |
| *** exploreshaifali has quit IRC | 20:30 | |
| *** jruano has joined #openstack-ceilometer | 20:37 | |
| *** exploreshaifali has joined #openstack-ceilometer | 20:39 | |
| *** exploreshaifali has quit IRC | 20:45 | |
| *** ildikov has quit IRC | 20:50 | |
| *** julim has quit IRC | 20:50 | |
| *** julim has joined #openstack-ceilometer | 20:54 | |
| openstackgerrit | gordon chung proposed openstack/ceilometer: fix metadata for compute cpu notifications https://review.openstack.org/217906 | 20:54 |
| *** ankita_wagh has joined #openstack-ceilometer | 20:57 | |
| *** fawadkhaliq has quit IRC | 21:00 | |
| openstackgerrit | Merged openstack/ceilometer: Fix description for "Incorrect spelling of a word" https://review.openstack.org/216986 | 21:01 |
| *** julim has quit IRC | 21:07 | |
| *** stevebaker has quit IRC | 21:11 | |
| *** ankita_w_ has joined #openstack-ceilometer | 21:13 | |
| *** thorst1 has quit IRC | 21:13 | |
| *** ankita_wagh has quit IRC | 21:16 | |
| *** idegtiarov_ has quit IRC | 21:41 | |
| *** ankita_wagh has joined #openstack-ceilometer | 21:44 | |
| *** thorst1 has joined #openstack-ceilometer | 21:44 | |
| *** ankita_w_ has quit IRC | 21:47 | |
| vishwanathj | david-lyle, lhcheng, I am looking for guidance on how to approach fixing the bug https://bugs.launchpad.net/horizon/+bug/1484723 before I assign it to myself | 22:07 |
| openstack | Launchpad bug 1484723 in OpenStack Dashboard (Horizon) "Form errors does not propogate to the user" [High,Confirmed] | 22:07 |
| *** jruano has quit IRC | 22:08 | |
| david-lyle | vishwanathj: I haven't dug into that deeply, but I think the exception is being caught/not caught by the wrong handler | 22:09 |
| david-lyle | could be that django/forms is handling it | 22:09 |
| vishwanathj | david-lyle, looks like I posted in the ceilometer channel instead of the horizon channel :( , should I repost in the horizon channel to continue this discussion | 22:24 |
| david-lyle | vishwanathj: would probably be better | 22:24 |
| david-lyle | :) | 22:24 |
| david-lyle | lhcheng is in there | 22:24 |
| vishwanathj | ok | 22:24 |
| *** gordc has quit IRC | 22:27 | |
| *** Ephur has quit IRC | 22:36 | |
| *** edmondsw has quit IRC | 22:51 | |
| *** jwcroppe has quit IRC | 23:00 | |
| *** rbak has quit IRC | 23:02 | |
| *** maubry has quit IRC | 23:02 | |
| *** liusheng has quit IRC | 23:02 | |
| *** liusheng has joined #openstack-ceilometer | 23:03 | |
| *** jwcroppe_ has joined #openstack-ceilometer | 23:03 | |
| *** jruano has joined #openstack-ceilometer | 23:09 | |
| *** jwcroppe_ has quit IRC | 23:09 | |
| *** jwcroppe has joined #openstack-ceilometer | 23:10 | |
| *** jwcroppe has quit IRC | 23:14 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ceilometer: Updated from global requirements https://review.openstack.org/216964 | 23:20 |
| *** jaypipes has quit IRC | 23:31 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!