| *** jruano has joined #openstack-ceilometer | 00:07 | |
| *** promulo__ has joined #openstack-ceilometer | 00:10 | |
| *** ddieterly has joined #openstack-ceilometer | 00:13 | |
| *** promulo has quit IRC | 00:14 | |
| *** julim has joined #openstack-ceilometer | 00:17 | |
| *** jruano has quit IRC | 00:18 | |
| *** jruano has joined #openstack-ceilometer | 00:21 | |
| *** zul has joined #openstack-ceilometer | 00:24 | |
| *** pauldix has joined #openstack-ceilometer | 00:24 | |
| *** Kennan2 is now known as Kennan | 00:24 | |
| *** jruano has quit IRC | 00:28 | |
| *** pauldix has quit IRC | 00:58 | |
| *** ankita_w_ has quit IRC | 01:02 | |
| *** Ephur has joined #openstack-ceilometer | 01:04 | |
| *** _cjones_ has quit IRC | 01:14 | |
| *** vishwanathj has joined #openstack-ceilometer | 01:14 | |
| *** Ephur has quit IRC | 01:14 | |
| *** Longgeek has joined #openstack-ceilometer | 01:30 | |
| *** jruano has joined #openstack-ceilometer | 01:38 | |
| *** lan has joined #openstack-ceilometer | 01:40 | |
| *** Ephur has joined #openstack-ceilometer | 01:40 | |
| *** pauldix has joined #openstack-ceilometer | 01:52 | |
| *** Longgeek has quit IRC | 02:14 | |
| *** jruano has quit IRC | 02:22 | |
| *** zhenq has quit IRC | 02:27 | |
| *** Kennan2 has joined #openstack-ceilometer | 02:34 | |
| *** Kennan has quit IRC | 02:36 | |
| *** jruano has joined #openstack-ceilometer | 02:37 | |
| *** Longgeek has joined #openstack-ceilometer | 02:51 | |
| *** jruano has quit IRC | 02:54 | |
| openstackgerrit | ZhiQiang Fan proposed openstack/python-ceilometerclient: fix wrong required option tenant-id https://review.openstack.org/170710 | 03:01 |
|---|---|---|
| *** jruano has joined #openstack-ceilometer | 03:06 | |
| *** amalagon has joined #openstack-ceilometer | 03:09 | |
| *** Longgeek_ has joined #openstack-ceilometer | 03:10 | |
| *** Longgeek has quit IRC | 03:12 | |
| *** jruano has quit IRC | 03:29 | |
| *** ddieterly has quit IRC | 03:29 | |
| *** Longgeek_ has quit IRC | 03:32 | |
| *** sbfox has joined #openstack-ceilometer | 03:32 | |
| *** deepthi has joined #openstack-ceilometer | 03:33 | |
| *** fnaval has quit IRC | 03:47 | |
| *** harlowja is now known as harlowja_away | 03:55 | |
| *** Longgeek has joined #openstack-ceilometer | 04:00 | |
| *** fnaval has joined #openstack-ceilometer | 04:06 | |
| *** pauldix has quit IRC | 04:14 | |
| *** _cjones_ has joined #openstack-ceilometer | 04:15 | |
| *** _cjones_ has quit IRC | 04:19 | |
| *** ddieterly has joined #openstack-ceilometer | 04:30 | |
| *** noye has quit IRC | 04:33 | |
| *** ddieterly has quit IRC | 04:34 | |
| *** ankita_wagh has joined #openstack-ceilometer | 04:37 | |
| *** sbfox has quit IRC | 04:56 | |
| *** cmyster has quit IRC | 04:57 | |
| *** cmyster has joined #openstack-ceilometer | 04:58 | |
| *** cmyster has joined #openstack-ceilometer | 04:58 | |
| *** amalagon has quit IRC | 05:00 | |
| *** yatin has joined #openstack-ceilometer | 05:04 | |
| *** sbfox has joined #openstack-ceilometer | 05:08 | |
| *** cmyster has quit IRC | 05:15 | |
| *** cmyster has joined #openstack-ceilometer | 05:18 | |
| *** ddieterly has joined #openstack-ceilometer | 05:36 | |
| *** ddieterly has quit IRC | 05:41 | |
| *** pauldix has joined #openstack-ceilometer | 06:02 | |
| *** pauldix has quit IRC | 06:07 | |
| *** Longgeek has quit IRC | 06:34 | |
| *** Longgeek has joined #openstack-ceilometer | 06:34 | |
| *** ddieterly has joined #openstack-ceilometer | 06:37 | |
| *** ddieterly has quit IRC | 06:42 | |
| *** ildikov has joined #openstack-ceilometer | 06:51 | |
| openstackgerrit | liusheng proposed openstack/ceilometer: (WIP)Add MySql pagination for resource, meter and sample https://review.openstack.org/171062 | 06:53 |
| *** afazekas has joined #openstack-ceilometer | 06:54 | |
| *** liusheng has joined #openstack-ceilometer | 06:56 | |
| *** eglynn has joined #openstack-ceilometer | 06:57 | |
| *** Ala has joined #openstack-ceilometer | 07:14 | |
| *** ankita_wagh has quit IRC | 07:24 | |
| *** ankita_wagh has joined #openstack-ceilometer | 07:24 | |
| *** fnaval has quit IRC | 07:24 | |
| *** ankita_wagh has quit IRC | 07:29 | |
| *** shardy has joined #openstack-ceilometer | 07:35 | |
| *** ddieterly has joined #openstack-ceilometer | 07:38 | |
| *** yprokule has joined #openstack-ceilometer | 07:41 | |
| *** ddieterly has quit IRC | 07:42 | |
| *** safchain has joined #openstack-ceilometer | 07:46 | |
| *** sbfox has quit IRC | 08:06 | |
| *** zqfan has joined #openstack-ceilometer | 08:35 | |
| *** ddieterly has joined #openstack-ceilometer | 08:38 | |
| *** ddieterly has quit IRC | 08:43 | |
| *** claudiub has joined #openstack-ceilometer | 09:03 | |
| sileht | hi, jd__ | 09:12 |
| jd__ | hello sileht | 09:12 |
| openstackgerrit | Mehdi Abaakouk proposed openstack/gnocchi: Smarter list_resources for history https://review.openstack.org/170385 | 09:13 |
| openstackgerrit | Mehdi Abaakouk proposed openstack/gnocchi: Keep history of updated resource attributes https://review.openstack.org/168405 | 09:13 |
| sileht | jd__, If you have some times to take a look to the history patches | 09:13 |
| jd__ | sileht: yep! | 09:15 |
| sileht | jd__, I have a small question a potential conflict between my patch and the patch that directly use object instead of intermediate dict | 09:17 |
| sileht | jd__, for history I have 'revision' to make row unique | 09:18 |
| sileht | jd__, when I join the resource and the history I use '-1' as revision, but revision is not part of the public API of the object | 09:18 |
| sileht | jd__, should we just skip it in jsonify ? or make it public ? | 09:19 |
| jd__ | sileht: I think we should expose it | 09:21 |
| jd__ | sileht: isn't there some special HTTP headers around that too? | 09:21 |
| sileht | jd__, I don't get it | 09:22 |
| jd__ | sileht: my question? | 09:22 |
| sileht | jd__, yes | 09:22 |
| jd__ | nevermind, I am just wondering if we couldn't expose the version of the object using some HTTP headers such as ETag or the like | 09:23 |
| jd__ | let me google a bit about that | 09:23 |
| sileht | jd__, oh | 09:23 |
| *** Longgeek has quit IRC | 09:23 | |
| jd__ | it would be cool to have ETag support because it would be useful when adding PUT /v1/resource/generic/<uuid> | 09:24 |
| jd__ | we don't have this yet but with ETag it'd be cool so add a If-Match support and all | 09:24 |
| jd__ | I'm just thinking out loud about things to do laaaater :) | 09:25 |
| sileht | that looks a good enhancement | 09:25 |
| jd__ | also it would be cool to have the revision in ETag so you can do | 09:25 |
| jd__ | GET /v1/resource/generic/f00b4r -> ETag: 1 | 09:25 |
| jd__ | then you refresh with GET /v1/resource/generic/f00b4r If-None-Match: 1 | 09:26 |
| jd__ | so you only retrive the object if it changed | 09:26 |
| jd__ | we don't have any caching mechanism yet but if we have later we could use that to retrive or not an object from the indexer | 09:26 |
| *** ddieterly has joined #openstack-ceilometer | 09:39 | |
| sileht | :bn | 09:41 |
| sileht | oups | 09:42 |
| *** ddieterly has quit IRC | 09:44 | |
| *** eglynn has quit IRC | 09:45 | |
| deepthi | all, hello! Just wanted to know if we can retrieve memory utilisation of a vm via ceilometer (similar to cpu_util). Is there any plan for the same ? | 09:50 |
| deepthi | my bad, looks like the code is there in K. | 09:57 |
| *** eglynn has joined #openstack-ceilometer | 10:10 | |
| *** Longgeek has joined #openstack-ceilometer | 10:24 | |
| *** cdent has joined #openstack-ceilometer | 10:26 | |
| *** Longgeek has quit IRC | 10:30 | |
| *** noye has joined #openstack-ceilometer | 10:37 | |
| *** _nadya_ has joined #openstack-ceilometer | 10:41 | |
| *** tobberydberg has joined #openstack-ceilometer | 10:46 | |
| openstackgerrit | Mehdi Abaakouk proposed openstack/gnocchi: Smarter list_resources for history https://review.openstack.org/170385 | 10:56 |
| openstackgerrit | Mehdi Abaakouk proposed openstack/gnocchi: Keep history of updated resource attributes https://review.openstack.org/168405 | 10:56 |
| *** cdent_ has joined #openstack-ceilometer | 11:16 | |
| *** cdent_ has quit IRC | 11:19 | |
| *** cdent has quit IRC | 11:20 | |
| *** cdent has joined #openstack-ceilometer | 11:22 | |
| openstackgerrit | ZhiQiang Fan proposed openstack/ceilometer: limit alarm actions https://review.openstack.org/166154 | 11:28 |
| *** pauldix has joined #openstack-ceilometer | 11:29 | |
| *** deepthi has quit IRC | 11:30 | |
| *** pauldix has quit IRC | 11:34 | |
| *** ddieterly has joined #openstack-ceilometer | 11:40 | |
| openstackgerrit | Divya K Konoor proposed openstack/ceilometer: Adds support for default rule in ceilometer policy.json. https://review.openstack.org/167370 | 11:43 |
| *** ddieterly has quit IRC | 11:45 | |
| *** cmyster has quit IRC | 11:47 | |
| *** pauldix has joined #openstack-ceilometer | 11:50 | |
| *** Longgeek has joined #openstack-ceilometer | 11:53 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ceilometer: Updated from global requirements https://review.openstack.org/170388 | 11:57 |
| openstackgerrit | Thierry Carrez proposed openstack/ceilometer: Open Liberty development https://review.openstack.org/171152 | 12:07 |
| *** _nadya_ has quit IRC | 12:15 | |
| *** alexpilotti has joined #openstack-ceilometer | 12:22 | |
| *** gordc has joined #openstack-ceilometer | 12:28 | |
| *** cmyster has joined #openstack-ceilometer | 12:28 | |
| *** pauldix has quit IRC | 12:32 | |
| *** yatin has quit IRC | 12:38 | |
| *** ddieterly has joined #openstack-ceilometer | 12:41 | |
| *** ddieterly has quit IRC | 12:46 | |
| *** exploreshaifali has joined #openstack-ceilometer | 12:47 | |
| gordc | eglynn: does it make sense to set bug back to triage since it was reverted or do you want to track with another bug? https://bugs.launchpad.net/ceilometer/+bug/1208365 | 12:54 |
| openstack | Launchpad bug 1208365 in Ceilometer "Instance:<flavor> meter should be removed" [Low,Triaged] - Assigned to gordon chung (chungg) | 12:54 |
| *** gentux has quit IRC | 13:00 | |
| *** ildikov has quit IRC | 13:02 | |
| *** ildikov has joined #openstack-ceilometer | 13:03 | |
| ildikov | gordc: sorry again :( | 13:04 |
| eglynn | gordc: a-ha, I get it, sorry ... I assigned that to kilo-rc1 since it was fix-committed in LP | 13:05 |
| eglynn | gordc: ... but fix committed in this case obviously meant that the orginal fix was removed via the revert | 13:05 |
| gordc | ildikov: i don't want to hear it. :P | 13:05 |
| eglynn | gordc: ... so reverting to triaged, but what of the target? | 13:06 |
| eglynn | (i.e. kilo-rc1 no longer makes sense, right?) | 13:06 |
| gordc | eglynn: yeah... i guess we can target liberty-1 | 13:06 |
| eglynn | gordc: agreed, coolness | 13:06 |
| *** openstackgerrit has quit IRC | 13:07 | |
| gordc | or do we want to have it gone in M*? i just want it gone.lol | 13:07 |
| ildikov | gordc: eglynn: we/I need to add it to docco and then the deprecation will be on it's way I guess | 13:07 |
| *** openstackgerrit has joined #openstack-ceilometer | 13:07 | |
| eglynn | ildikov: yeap, coolness | 13:07 |
| ildikov | gordc: eglynn: if we don't plan other "flags" to mark smth as deprecated | 13:07 |
| eglynn | ildikov: just docco / release notes | 13:07 |
| gordc | agreed. | 13:07 |
| ildikov | eglynn: ok | 13:07 |
| ildikov | gordc: BTW do you have a valid query in mind for this instance:<flavor> beauty? | 13:09 |
| *** IBerezovskiy has joined #openstack-ceilometer | 13:10 | |
| gordc | ildikov: hmm.. i'll try it out now. | 13:11 |
| ildikov | gordc: also, the API doc causes much pain for me nowadays, so I suffer for my sins already :) | 13:11 |
| ildikov | gordc: I don't have a running env right now, that is why I hoped you have some ideas how it should look like :) | 13:11 |
| gordc | lol yeah, sorry. i've been testing and i forgot about all the docimpacts i created | 13:11 |
| ildikov | gordc: nah, the API part I realized a while ago, just did not have the time... and docco team has expectations too... | 13:12 |
| *** Ephur_ has joined #openstack-ceilometer | 13:15 | |
| *** jruano has joined #openstack-ceilometer | 13:15 | |
| *** pauldix has joined #openstack-ceilometer | 13:15 | |
| openstackgerrit | Merged openstack/gnocchi: Add gabbi tests to cover the ArchivePoliciesController https://review.openstack.org/169893 | 13:17 |
| *** Ephur has quit IRC | 13:19 | |
| ildikov | eglynn: gordc: one stupid question if you're still around | 13:20 |
| eglynn | shoot | 13:21 |
| ildikov | eglynn: gordc: the default: "" in policy.json is for some backward compatibility? | 13:21 |
| eglynn | ildikov: do you mean for https://bugs.launchpad.net/ceilometer/+bug/1435855 ? | 13:22 |
| openstack | Launchpad bug 1435855 in Ceilometer "Default rule does not work in ceilometer policy.json" [High,In progress] - Assigned to Divya K Konoor (dikonoor) | 13:22 |
| ildikov | so that file looks a bit odd to me right now, but it can be easily because of the lack of my policy knowledge | 13:22 |
| eglynn | ildikov: yeah, the idea is the RBAC doesn't surprise folks by outlawing everything by default for non-admins | 13:22 |
| ildikov | eglynn: yeap, the bug fix for it, as it is what is marked as missing item for rc1 | 13:22 |
| eglynn | ildikov: so we want this to work the same in two different situations | 13:23 |
| eglynn | 1. upgrade to kilo, using old policy.json | 13:23 |
| eglynn | 2. fresh install of kilo, no pre-existing policy.json | 13:23 |
| ildikov | eglynn: do you mean that the "" falls back to admin context? | 13:23 |
| eglynn | ildikov: no, the effect would be similar if the default rule is left out | 13:24 |
| eglynn | ildikov: as the code explictly checks that it's present, before calling the enforcer in a way that requires it to be there | 13:25 |
| *** zhenq has joined #openstack-ceilometer | 13:25 | |
| ildikov | eglynn: does it? | 13:25 |
| eglynn | ildikov: yes see line 53 in https://review.openstack.org/#/c/167370/9/ceilometer/api/rbac.py | 13:26 |
| eglynn | ildikov: if the default is left out, then the RBAC will now iterate through the rules that are present and explicitly enfore only *if present* | 13:26 |
| eglynn | ildikov: which makes an *implicit* default of "allow for non-admins" | 13:27 |
| ildikov | eglynn: ok, now I got what you meant by "requires" | 13:27 |
| eglynn | ildikov: so IIUC specifying that default="" makes the implicit, explicit | 13:28 |
| eglynn | ildikov: ... as this needs to work with pre-existing juno policy.json files | 13:28 |
| eglynn | (esp. in the grenade upgrade testing) | 13:28 |
| eglynn | gotta run, call coming up | 13:28 |
| *** yprokule has quit IRC | 13:28 | |
| ildikov | eglynn: ok, thanks for clarifying | 13:29 |
| *** yprokule has joined #openstack-ceilometer | 13:31 | |
| *** ildikov has quit IRC | 13:35 | |
| openstackgerrit | ZhiQiang Fan proposed openstack/ceilometer: WIP: fix kafka unit test code https://review.openstack.org/171196 | 13:41 |
| *** gcivitella has joined #openstack-ceilometer | 13:43 | |
| openstackgerrit | Divya K Konoor proposed openstack/ceilometer: Adds support for default rule in ceilometer policy.json. https://review.openstack.org/167370 | 13:43 |
| gcivitella | Hi all, are there examples of pushing new metrics to ceilomenter? | 13:46 |
| openstackgerrit | Merged openstack/gnocchi: Adding Gabbi Tests to Metric(s) API https://review.openstack.org/164749 | 13:48 |
| openstackgerrit | ZhiQiang Fan proposed openstack/ceilometer: WIP: fix kafka unit test code https://review.openstack.org/171196 | 13:49 |
| *** ildikov has joined #openstack-ceilometer | 13:51 | |
| openstackgerrit | ZhiQiang Fan proposed openstack/ceilometer: WIP: fix kafka unit test code https://review.openstack.org/171196 | 13:53 |
| *** fawadkhaliq has joined #openstack-ceilometer | 13:54 | |
| gcivitella | I'd like to push custom metrics from vm to ceilometer's api to implement a custom autoscaling policies via Heat templates. | 13:55 |
| *** jaypipes has joined #openstack-ceilometer | 13:59 | |
| gordc | gcivitella: what type of data are you looking to capture from vm? is it something that be gathered from hypervisor level or a notification Nova can provide? | 13:59 |
| *** ddieterly has joined #openstack-ceilometer | 14:01 | |
| *** fawadk has joined #openstack-ceilometer | 14:01 | |
| gcivitella | gordc: at the moment I'm trying to imagine possible use cases of autoscaling (already tested those wich Nova can provide): let think to vms running a custom application that has its own metrics that help decide if the autoscaling group needs more vm | 14:03 |
| gcivitella | I'd like to send those custom metrics to Heat via Ceilometer | 14:04 |
| *** fawadkhaliq has quit IRC | 14:04 | |
| openstackgerrit | Mehdi Abaakouk proposed openstack/gnocchi: Smarter list_resources for history https://review.openstack.org/170385 | 14:05 |
| openstackgerrit | Mehdi Abaakouk proposed openstack/gnocchi: Keep history of updated resource attributes https://review.openstack.org/168405 | 14:05 |
| *** yprokule has quit IRC | 14:05 | |
| *** yprokule has joined #openstack-ceilometer | 14:06 | |
| gcivitella | as I read it should be possible, even for a normal keystone tenant, to define and publish new metrics | 14:06 |
| gordc | gcivitella: i see. you can post messages to Ceilometer's api which would allow you to capture those datapoints if you don't want to publish to the message queue. | 14:07 |
| gordc | http://docs.openstack.org/developer/ceilometer/webapi/v2.html#user-defined-data | 14:07 |
| *** boris-42 has joined #openstack-ceilometer | 14:07 | |
| gordc | i think the consensus recommendation is you publish your data to MQ (if possible) for better scalability | 14:08 |
| openstackgerrit | Merged openstack/ceilometer: Remove 'samples:groupby' from the Capabilities list https://review.openstack.org/170442 | 14:08 |
| *** rbak has joined #openstack-ceilometer | 14:08 | |
| openstackgerrit | Merged openstack/ceilometer: Updated from global requirements https://review.openstack.org/170388 | 14:09 |
| gcivitella | gordc: ok, thanks | 14:09 |
| *** exploreshaifali has quit IRC | 14:10 | |
| openstackgerrit | Merged openstack/ceilometer: Use old name of 'hardware.ipmi.node.temperature' https://review.openstack.org/170379 | 14:12 |
| openstackgerrit | Chris Dent proposed openstack/ceilometer: Further mock adjustments to deal with intermittent failure https://review.openstack.org/170092 | 14:17 |
| *** jruano has quit IRC | 14:23 | |
| *** claudiub has quit IRC | 14:31 | |
| *** ddieterly has quit IRC | 14:31 | |
| *** ddieterly has joined #openstack-ceilometer | 14:31 | |
| *** jruano has joined #openstack-ceilometer | 14:33 | |
| *** jruano has quit IRC | 14:34 | |
| *** _nadya_ has joined #openstack-ceilometer | 14:35 | |
| *** devlaps has quit IRC | 14:38 | |
| *** fnaval has joined #openstack-ceilometer | 14:38 | |
| *** ddieterly has quit IRC | 14:41 | |
| *** ddieterl_ has joined #openstack-ceilometer | 14:42 | |
| *** ddieterl_ has quit IRC | 14:45 | |
| *** ddieterly has joined #openstack-ceilometer | 14:46 | |
| *** edmondsw has joined #openstack-ceilometer | 14:47 | |
| *** ddieterly has quit IRC | 14:47 | |
| edmondsw | eglynn, looking for a second +2 on https://review.openstack.org/#/c/169409/ if you have a moment... very simple change | 14:48 |
| eglynn | edmondsw: looking | 14:48 |
| edmondsw | tx | 14:48 |
| eglynn | edmondsw: so you'll be looking for a ceilo client release also? | 14:49 |
| edmondsw | yes... for that and https://review.openstack.org/#/c/169442/ | 14:49 |
| *** fawadk has quit IRC | 14:54 | |
| *** gentux has joined #openstack-ceilometer | 14:55 | |
| *** ddieterly has joined #openstack-ceilometer | 15:00 | |
| openstackgerrit | Merged openstack/gnocchi: indexer: return resource objects rather than dict https://review.openstack.org/169302 | 15:02 |
| *** claudiub has joined #openstack-ceilometer | 15:24 | |
| *** ddieterly has quit IRC | 15:24 | |
| *** ddieterly has joined #openstack-ceilometer | 15:24 | |
| *** ddieterly has quit IRC | 15:25 | |
| *** jruano has joined #openstack-ceilometer | 15:30 | |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: rest: enhance metric retrieval in controllers https://review.openstack.org/169375 | 15:34 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: rest: do not use expect_error in tests https://review.openstack.org/169376 | 15:34 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: use RDBMS check constraint where available https://review.openstack.org/170544 | 15:34 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: rest: pass the whole metric to aggregation https://review.openstack.org/169380 | 15:34 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: gabbi: merge metric and metrics test files https://review.openstack.org/171254 | 15:34 |
| *** exploreshaifali has joined #openstack-ceilometer | 15:36 | |
| *** ccrouch has quit IRC | 15:36 | |
| *** Ala has quit IRC | 15:36 | |
| *** devlaps has joined #openstack-ceilometer | 15:40 | |
| *** Longgeek has quit IRC | 15:45 | |
| *** exploreshaifali has quit IRC | 15:49 | |
| *** ddieterly has joined #openstack-ceilometer | 15:52 | |
| *** _cjones_ has joined #openstack-ceilometer | 15:54 | |
| *** thomasem has quit IRC | 15:57 | |
| *** alexpilotti has quit IRC | 15:59 | |
| *** ildikov has quit IRC | 16:06 | |
| *** claudiub has quit IRC | 16:08 | |
| *** alexpilotti has joined #openstack-ceilometer | 16:10 | |
| *** fawadkhaliq has joined #openstack-ceilometer | 16:14 | |
| *** afazekas has quit IRC | 16:16 | |
| *** alexpilotti_ has joined #openstack-ceilometer | 16:24 | |
| *** alexpilotti has quit IRC | 16:24 | |
| *** alexpilotti_ is now known as alexpilotti | 16:24 | |
| *** noye has quit IRC | 16:25 | |
| *** fawadkhaliq has quit IRC | 16:34 | |
| *** yatin has joined #openstack-ceilometer | 16:36 | |
| *** devlaps has quit IRC | 16:39 | |
| *** gcivitella has quit IRC | 16:40 | |
| *** ddieterly has quit IRC | 16:45 | |
| *** jruano has quit IRC | 16:45 | |
| *** ddieterly has joined #openstack-ceilometer | 16:45 | |
| openstackgerrit | Pradeep Kilambi proposed openstack/gnocchi: WIP: Add support for ArchivePolicy Rule Endpoints https://review.openstack.org/171288 | 16:46 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: gabbi: merge metric and metrics test files https://review.openstack.org/171254 | 16:52 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: use RDBMS check constraint where available https://review.openstack.org/170544 | 16:52 |
| *** fawadkhaliq has joined #openstack-ceilometer | 16:58 | |
| *** fawadk has joined #openstack-ceilometer | 16:59 | |
| *** fawadkhaliq has quit IRC | 17:03 | |
| *** ankita_wagh has joined #openstack-ceilometer | 17:06 | |
| *** _cjones_ has quit IRC | 17:13 | |
| *** yprokule has quit IRC | 17:14 | |
| *** shardy has quit IRC | 17:17 | |
| *** _cjones_ has joined #openstack-ceilometer | 17:18 | |
| *** claudiub has joined #openstack-ceilometer | 17:19 | |
| *** ddieterly has quit IRC | 17:20 | |
| *** ildikov has joined #openstack-ceilometer | 17:22 | |
| *** sbfox has joined #openstack-ceilometer | 17:23 | |
| *** safchain has quit IRC | 17:24 | |
| *** ddieterly has joined #openstack-ceilometer | 17:30 | |
| *** harlowja_away is now known as harlowja | 17:31 | |
| *** IBerezovskiy has quit IRC | 17:40 | |
| *** yatin has quit IRC | 17:42 | |
| openstackgerrit | Merged openstack/gnocchi: rest: enhance metric retrieval in controllers https://review.openstack.org/169375 | 17:45 |
| openstackgerrit | Merged openstack/gnocchi: rest: do not use expect_error in tests https://review.openstack.org/169376 | 17:45 |
| openstackgerrit | Merged openstack/gnocchi: rest: pass the whole metric to aggregation https://review.openstack.org/169380 | 17:45 |
| *** _nadya_ has quit IRC | 17:50 | |
| *** claudiub has quit IRC | 17:51 | |
| *** fawadk has quit IRC | 17:58 | |
| openstackgerrit | Merged openstack/ceilometer: Tweak authenticate event definition https://review.openstack.org/170193 | 17:58 |
| *** cdent has quit IRC | 17:59 | |
| *** fawadkhaliq has joined #openstack-ceilometer | 18:00 | |
| *** afazekas has joined #openstack-ceilometer | 18:00 | |
| *** jruano has joined #openstack-ceilometer | 18:04 | |
| *** tongli has joined #openstack-ceilometer | 18:08 | |
| openstackgerrit | Merged openstack/python-ceilometerclient: ceilometerclient fails with keystone v3 auth https://review.openstack.org/169409 | 18:21 |
| *** exploreshaifali has joined #openstack-ceilometer | 18:23 | |
| *** afazekas has quit IRC | 18:24 | |
| *** jdandrea has quit IRC | 18:25 | |
| edmondsw | looking for reviews on https://review.openstack.org/#/c/169442/ ... simple change | 18:26 |
| *** jdandrea has joined #openstack-ceilometer | 18:28 | |
| *** _nadya_ has joined #openstack-ceilometer | 18:29 | |
| *** _nadya_ has quit IRC | 18:31 | |
| *** fawadkhaliq has quit IRC | 18:33 | |
| *** fc__ has quit IRC | 18:46 | |
| *** fc__ has joined #openstack-ceilometer | 18:46 | |
| *** eglynn has quit IRC | 18:52 | |
| *** fawadkhaliq has joined #openstack-ceilometer | 19:01 | |
| *** _nadya_ has joined #openstack-ceilometer | 19:02 | |
| *** noye has joined #openstack-ceilometer | 19:09 | |
| *** _nadya_ has quit IRC | 19:10 | |
| *** fc__ has quit IRC | 19:21 | |
| *** _nadya_ has joined #openstack-ceilometer | 19:22 | |
| *** openstackgerrit has quit IRC | 19:22 | |
| *** openstackgerrit has joined #openstack-ceilometer | 19:22 | |
| *** fc__ has joined #openstack-ceilometer | 19:23 | |
| *** harlowja is now known as harlowja_away | 19:31 | |
| *** _nadya_ has quit IRC | 19:32 | |
| *** _nadya_ has joined #openstack-ceilometer | 19:33 | |
| *** ankita_wagh has quit IRC | 19:35 | |
| *** jruano has quit IRC | 19:36 | |
| *** sbfox has quit IRC | 19:36 | |
| *** jruano has joined #openstack-ceilometer | 19:36 | |
| *** harlowja_away is now known as harlowja | 19:37 | |
| openstackgerrit | Matthew Edmonds proposed openstack/python-ceilometerclient: ceilometerclient insecure argument no longer works https://review.openstack.org/169442 | 19:42 |
| *** exploreshaifali has quit IRC | 19:48 | |
| *** ddieterly has quit IRC | 19:48 | |
| *** jruano has quit IRC | 19:52 | |
| *** ddieterly has joined #openstack-ceilometer | 19:54 | |
| *** _nadya_ has quit IRC | 19:54 | |
| *** _cjones_ has quit IRC | 19:55 | |
| openstackgerrit | gordon chung proposed openstack/ceilometer: use oslo.messaging dispatch filter https://review.openstack.org/171347 | 19:55 |
| *** _cjones_ has joined #openstack-ceilometer | 19:55 | |
| edmondsw | ildikov, see gordc's comment in https://review.openstack.org/#/c/167370/ . The author is in India, so if you guys can hash that out before their new day starts... | 19:57 |
| edmondsw | The case where default exists in the policy is tested by UT, and the case where it doesn't is tested by grenade | 19:59 |
| *** ankita_wagh has joined #openstack-ceilometer | 20:00 | |
| gordc | edmondsw: just a heads up, most of the contributors are based in Euro/Asia. | 20:03 |
| gordc | you *might* be able to catch them at this hour. | 20:04 |
| edmondsw | gordc, good to know, thanks | 20:04 |
| gordc | np. thanks for following up on it though. | 20:05 |
| ildikov | hey | 20:05 |
| ildikov | I'm here, just checking the UTs without 'default' to see what's happening | 20:05 |
| edmondsw | hey :) | 20:05 |
| ildikov | however I still think that UTs should test the exact functionality as opposed to fail as a side effect of missing an option in the config file :) | 20:07 |
| ildikov | but gordc is right, I should already be in bed sleeping :) | 20:07 |
| edmondsw | ildikov, are there any existing UTs that test different policy settings, not just the existing policy.json? | 20:08 |
| edmondsw | I'll try to catch you earlier next time :) | 20:08 |
| edmondsw | I don't think there are any UTs that would fail if "default" wasn't in the policy.json... we specifically want to allow it to be there or not be there | 20:09 |
| edmondsw | UT tests the case where it is, and grenade tests the case where it isn't | 20:09 |
| edmondsw | you think we need UTs that test both? | 20:10 |
| ildikov | edmondsw: I think we don't have UTs for policy, but I think we should have | 20:11 |
| edmondsw | ildikov, sounds like something that should have come up long before now :) | 20:11 |
| edmondsw | tackle that in Liberty? | 20:11 |
| ildikov | grenade is a different level of testing, UTs should test the base functiionality as opposed to the E2E flow | 20:12 |
| edmondsw | yeah, I don't disagree with that | 20:12 |
| ildikov | yes, it should've come up, but it didn't as we need 2 +2s on a patch, which is less than the core team :) | 20:12 |
| ildikov | but we can start any time to break this bad habit I hope | 20:13 |
| ildikov | the fact is that tests are always planned to be added, but this never has the priority to actually happen, at least this is my experience | 20:14 |
| edmondsw | this is poor Divya's first attempt at a fix, and we're very close to RC, so while I agree with you in principal I'm not sure I'd pick this particular change to go after that | 20:14 |
| gordc | edmondsw: i believe this is targeted for rc1? just my 2 cents, but i think the test portion can be left for post rc1... that said if we can get it now that'd be great. | 20:14 |
| edmondsw | yes | 20:14 |
| gordc | testing policy portion specifically i mean. | 20:14 |
| ildikov | I think we have time to wait for the author's answer | 20:15 |
| edmondsw | ok, thanks for the discussion... go get some sleep! | 20:15 |
| ildikov | yeap,ve, maybe I'm more supportive, when I'm all relaxed :) | 20:18 |
| gordc | lol | 20:18 |
| ildikov | ... nah, just kidding | 20:18 |
| ildikov | anyway, I will check back on the patch tomorrow morning (European TZ) | 20:19 |
| edmondsw | thanks | 20:20 |
| ildikov | edmondsw: /me thanks for the heads up | 20:22 |
| *** tongli has quit IRC | 20:28 | |
| *** fawadkhaliq has quit IRC | 20:32 | |
| openstackgerrit | gordon chung proposed openstack/ceilometer: use oslo.messaging dispatch filter https://review.openstack.org/171347 | 20:32 |
| *** yuntong has quit IRC | 20:33 | |
| *** gzhai1 has quit IRC | 20:34 | |
| *** llu has quit IRC | 20:36 | |
| *** claudiub has joined #openstack-ceilometer | 20:41 | |
| *** promulo has joined #openstack-ceilometer | 20:43 | |
| *** promulo__ has quit IRC | 20:44 | |
| *** _cjones_ has quit IRC | 20:58 | |
| *** _cjones_ has joined #openstack-ceilometer | 21:16 | |
| *** tobberydberg has quit IRC | 21:17 | |
| *** openstackgerrit has quit IRC | 21:37 | |
| *** openstackgerrit has joined #openstack-ceilometer | 21:37 | |
| openstackgerrit | Komei Shimamura proposed openstack/ceilometer: Fix a variable name mistake https://review.openstack.org/170644 | 21:37 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ceilometer: Updated from global requirements https://review.openstack.org/171395 | 21:51 |
| *** ddieterly has quit IRC | 21:53 | |
| *** sbfox has joined #openstack-ceilometer | 21:54 | |
| *** botang has joined #openstack-ceilometer | 21:54 | |
| *** ddieterly has joined #openstack-ceilometer | 21:55 | |
| *** noye has quit IRC | 21:56 | |
| *** edmondsw has quit IRC | 22:02 | |
| *** rjaiswal has joined #openstack-ceilometer | 22:08 | |
| *** rjaiswal has quit IRC | 22:08 | |
| *** rjaiswal has joined #openstack-ceilometer | 22:08 | |
| openstackgerrit | Pradeep Kilambi proposed openstack/gnocchi: WIP: Add support for ArchivePolicy Rule Endpoints https://review.openstack.org/171288 | 22:10 |
| *** alexpilotti has quit IRC | 22:12 | |
| *** alexpilotti has joined #openstack-ceilometer | 22:13 | |
| *** alexpilotti has quit IRC | 22:14 | |
| *** gordc has quit IRC | 22:20 | |
| *** pauldix has quit IRC | 22:23 | |
| *** promulo__ has joined #openstack-ceilometer | 22:30 | |
| *** promulo has quit IRC | 22:31 | |
| *** rbak has quit IRC | 22:32 | |
| *** boris-42 has quit IRC | 22:38 | |
| *** zhenq has quit IRC | 22:38 | |
| *** joshw42 has joined #openstack-ceilometer | 22:42 | |
| *** alexpilotti has joined #openstack-ceilometer | 22:43 | |
| *** rjaiswal has quit IRC | 22:49 | |
| *** _nadya_ has joined #openstack-ceilometer | 22:55 | |
| *** ankita_wagh has quit IRC | 22:59 | |
| *** _nadya_ has quit IRC | 22:59 | |
| openstackgerrit | Komei Shimamura proposed openstack/ceilometer: Fix a variable name mistake https://review.openstack.org/170644 | 23:04 |
| *** joshw42 has quit IRC | 23:05 | |
| *** ankita_wagh has joined #openstack-ceilometer | 23:09 | |
| *** sbfox has quit IRC | 23:13 | |
| *** ddieterly has quit IRC | 23:28 | |
| *** ankita_w_ has joined #openstack-ceilometer | 23:39 | |
| *** ankita_wagh has quit IRC | 23:42 | |
| *** ankita_wagh has joined #openstack-ceilometer | 23:44 | |
| *** ankita_w_ has quit IRC | 23:47 | |
| *** boris-42 has joined #openstack-ceilometer | 23:54 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!