Thursday, 2014-08-28

*** Qiming has joined #openstack-ceilometer00:08
*** _cjones_ has quit IRC00:09
*** _cjones_ has joined #openstack-ceilometer00:10
*** sdake_ has quit IRC00:12
*** terriyu has quit IRC00:13
*** _cjones_ has quit IRC00:14
*** rbowen has quit IRC00:17
*** r-daneel has quit IRC00:17
*** dmsimard_away is now known as dmsimard00:22
*** Qiming has quit IRC00:29
*** dmsimard is now known as dmsimard_away00:42
*** sbfox has joined #openstack-ceilometer00:56
openstackgerritRen Qiaowei proposed a change to openstack/ceilometer: XenAPI support: Update measurements documentation  https://review.openstack.org/11487400:56
*** renatoarmani has quit IRC00:59
*** sdake_ has joined #openstack-ceilometer01:05
*** sbfox has quit IRC01:13
*** sbfox has joined #openstack-ceilometer01:21
*** sbfox has quit IRC01:22
*** sdake_ has quit IRC01:31
*** ccrouch1 has quit IRC01:36
*** Qiming has joined #openstack-ceilometer01:41
*** sdake_ has joined #openstack-ceilometer01:46
*** thomasem_ has quit IRC01:48
*** nosnos has joined #openstack-ceilometer01:49
*** sbfox has joined #openstack-ceilometer01:52
*** yasukun has joined #openstack-ceilometer02:00
*** openstackgerrit has quit IRC02:02
*** openstackgerrit has joined #openstack-ceilometer02:02
openstackgerritA change was merged to openstack/ceilometer: XenAPI support: List the instances  https://review.openstack.org/11224402:05
*** nsaje has joined #openstack-ceilometer02:13
openstackgerritOpenStack Proposal Bot proposed a change to openstack/ceilometer: Updated from global requirements  https://review.openstack.org/11009602:20
*** sbfox has quit IRC02:27
*** _nadya_ has joined #openstack-ceilometer02:32
*** Yanyanhu has joined #openstack-ceilometer02:43
*** _nadya_ has quit IRC02:45
*** yasukun has quit IRC02:53
*** yasukun has joined #openstack-ceilometer02:54
*** _nadya_ has joined #openstack-ceilometer03:04
*** sdake_ has quit IRC03:05
*** sdake_ has joined #openstack-ceilometer03:06
openstackgerritA change was merged to openstack/ceilometer: Permit usage of notifications for metering  https://review.openstack.org/8022503:19
*** nosnos has quit IRC03:26
*** nosnos has joined #openstack-ceilometer03:27
*** _nadya_ has quit IRC03:27
*** Longgeek has joined #openstack-ceilometer03:31
*** nosnos has quit IRC03:32
*** sbfox has joined #openstack-ceilometer03:56
*** mitz_ has quit IRC03:59
*** mitz has joined #openstack-ceilometer03:59
*** sbfox has quit IRC04:09
*** sbfox has joined #openstack-ceilometer04:21
*** nosnos has joined #openstack-ceilometer04:27
openstackgerritRikimaru Honjo proposed a change to openstack/ceilometer: Add validate alarm_actions schema in alarm API  https://review.openstack.org/10733004:33
*** stevelle has quit IRC04:41
openstackgerritSrinivas Sakhamuri proposed a change to openstack/ceilometer: Get meters query performance improvements using index  https://review.openstack.org/11674804:49
*** Longgeek_ has joined #openstack-ceilometer04:52
*** Longgeek has quit IRC04:56
*** idegtiarov has joined #openstack-ceilometer05:05
*** stevelle_ has joined #openstack-ceilometer05:10
*** asalkeld has quit IRC05:19
*** stevelle_ is now known as stevelle05:23
openstackgerritOpenStack Proposal Bot proposed a change to openstack/ceilometer: Updated from global requirements  https://review.openstack.org/11009605:34
*** asalkeld has joined #openstack-ceilometer05:37
*** sbfox has quit IRC05:38
*** _nadya_ has joined #openstack-ceilometer05:53
*** _nadya_ has quit IRC06:01
*** k4n0 has joined #openstack-ceilometer06:03
*** taplax has joined #openstack-ceilometer06:05
openstackgerritOpenStack Proposal Bot proposed a change to openstack/ceilometer: Imported Translations from Transifex  https://review.openstack.org/11550806:09
openstackgerritRen Qiaowei proposed a change to openstack/ceilometer: XenAPI support: Update measurements documentation  https://review.openstack.org/11487406:10
*** ildikov has quit IRC06:32
*** ildikov has joined #openstack-ceilometer06:54
*** harlowja_ is now known as harlowja_away07:14
*** _nadya_ has joined #openstack-ceilometer07:15
openstackgerritNejc Saje proposed a change to openstack/python-ceilometerclient: Revamp documentation, add module references  https://review.openstack.org/11714007:35
*** arezmerita has joined #openstack-ceilometer07:38
*** asalkeld has quit IRC07:51
*** IvanBerezovskiy has joined #openstack-ceilometer07:55
*** yasukun has quit IRC08:00
*** Qiming_ has joined #openstack-ceilometer08:02
*** Qiming has quit IRC08:05
*** asalkeld has joined #openstack-ceilometer08:05
openstackgerritZhai, Edwin proposed a change to openstack/ceilometer: Add IPMI support  https://review.openstack.org/11584508:11
openstackgerritIgor Degtiarov proposed a change to openstack/ceilometer: Adding comparison operators in query for event traits  https://review.openstack.org/11258208:11
idegtiarovildikov: hi! could you look on cr https://review.openstack.org/#/c/115277/ which implements some new functions that are needed for migration in HBase08:17
*** boris-42 has quit IRC08:21
openstackgerritNejc Saje proposed a change to openstack/python-ceilometerclient: Revamp documentation, add module references  https://review.openstack.org/11714008:22
*** yassine has joined #openstack-ceilometer08:27
ildikovidegtiarov: is there any reason for this tiny function is in a separate patch from migration?08:31
*** boris-42 has joined #openstack-ceilometer08:31
ildikovidegtiarov: IIUC it is needed by the migration patch08:31
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Alarm: Use stevedore to load the service class  https://review.openstack.org/11719808:35
DinaBelovansaje, o/08:37
nsajeDinaBelova: o/08:37
DinaBelovawe've tried two compute agents08:38
nsajeand?08:38
DinaBelovaon the compute node08:38
DinaBelovahehe08:38
DinaBelovafirst polling was ok (they were in the group, etc)08:38
DinaBelovawe could see them in the group08:38
DinaBelovain the ZK08:38
DinaBelovathe second one was without errors but then they were 'members of the gorup []'08:38
DinaBelovaand third polling cycle - ToozError08:39
DinaBelovaas they were not in the group08:39
nsajehmm08:39
nsajeso apparently tooz zookeeper backend has issues08:40
idegtiarovildikov: actually there is long story, i have  proposed a cr with redesigning separator in HBase, but before it could land we need migration in HBase, and ityaptin  who implements migration ask for this change to use functions that are added here08:40
DinaBelovansaje, even smth really strange - http://paste2.org/Eyxz33Dh08:41
DinaBelovathese logs are followed one by one08:41
DinaBelovaone cycle justempty subset08:42
DinaBelovaone cycle error08:42
DinaBelovaand so far and so on08:42
nsajeyeah, it's the intended behaviour that the subset is empty if coordination backend fails08:43
DinaBelovaa-ha, ok08:43
nsajeso as to avoid duplicates08:43
nsajeit's weird that there's no exception text coming from tooz08:43
nsajejust a toozerror08:44
*** asalkeld has quit IRC08:45
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Allow to requeue a sample  https://review.openstack.org/11222508:45
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Document pipeline publishers configuration  https://review.openstack.org/11491908:45
nsajemodify https://github.com/stackforge/tooz/blob/master/tooz/drivers/zookeeper.py#L128 to wrap the exception itself, not string08:45
nsajeso you'll see what's the error in kazoo08:45
nsajeor better yet, use08:46
*** nosnos has quit IRC08:47
*** nosnos has joined #openstack-ceilometer08:47
ildikovidegtiarov: I was just wondering that if these functions are used now only in the migartion patch, then why don't merge these two08:48
*** flwang1 has joined #openstack-ceilometer08:48
nsajeDinaBelova: http://paste.openstack.org/show/101558/08:49
*** nosnos has quit IRC08:52
DinaBelovansaje, ehm, they are ok for now 0_008:55
DinaBelovansaje, that is strange08:55
DinaBelovaI don't actually like this unpredictable behaviour 0_008:56
DinaBelovansaje, hehe, I know why currently they are ok :D for some reason no disk meters were collected08:58
*** asalkeld has joined #openstack-ceilometer08:58
DinaBelovaand nothing went worng08:58
DinaBelovawrong*08:58
idegtiarovildikov:  you mean why them haven't been merged yet or why doesn't  Ilya  just use them in his patch?  :)09:03
ildikovidegtiarov: I meant that why this patch wasn't merged into Ilya's patch, as it is really does not have any context in this way09:06
DinaBelovansaje, yeah, compute agents stopped to poll :)09:10
openstackgerritling-yun proposed a change to openstack/ceilometer: ignore .idea folder in ceilometer  https://review.openstack.org/11745909:11
idegtiarovildikov: probably  he doesn't want to use functions that were proposed in other patch before this patch is merged09:11
ildikovidegtiarov: you mean the redesign separator patch?09:12
idegtiarovildikov:  yep09:12
nsajeDinaBelova: why was that?09:13
DinaBelova I don't know :D09:13
DinaBelovalet me restart them once more :)09:13
*** alexpilotti has joined #openstack-ceilometer09:18
ildikovidegtiarov: I understand that part, it's just looks weird now, as it is without context, I mean there are three tiny functions without any test or usage example and I have to dig into the migration patch anyway to see where these functions are used09:23
*** nosnos has joined #openstack-ceilometer09:24
ildikovidegtiarov: so I kinda agree with jd__'s comment there09:24
*** Qiming__ has joined #openstack-ceilometer09:24
idegtiarovildikov:  k, thanks for your time09:26
DinaBelovansaje, one more question09:28
DinaBelovais that the normal situation when one compute agent on the compute node polls all the VMs and other does nothing?09:28
*** Qiming_ has quit IRC09:28
nsajeDinaBelova: no, what are the group memberships reported?09:31
ildikovidegtiarov: np, I will try to get to the comparsion operator patch a bit later today09:31
DinaBelovansaje, oh, sorry, let me change the question09:31
idegtiarovildikov: thank you very much. hope this time it could be landed09:31
DinaBelovaone compute agents polls one set of meters (for all the VMs) and other polls other (for all the VMs as well)09:31
DinaBelovansaje ^^09:32
nsajeDinaBelova: nope, that's not the way it's supposed to work :-) they should divide the VMs, not the meters09:32
DinaBelovahehe09:33
DinaBelovansaje, well :D09:33
nsajeDinaBelova: there's nothing in the code that would limit which meters are processed09:34
nsajeDinaBelova: do you have different pollsters configured?09:34
DinaBelovano09:34
DinaBelovathey use the same file09:34
nsajeDinaBelova: and is the group membership correctly reported?09:34
DinaBelova:D:D:D09:34
DinaBelovait looks like that09:34
DinaBelovascreen was crashed09:34
DinaBelovaand lost the long buffer I configured09:35
DinaBelovaI'll increase it now and will look09:35
ildikovidegtiarov: hope so too09:35
nsajeDinaBelova: when I tested it single-node, I got with the first agent 'debug: my subset [Server1,Server4]' and on the other 'debug: my subset [Server2, Server3]'09:36
DinaBelovaok, I'll try it once more :d09:36
DinaBelova:D09:36
DinaBelova(wall)09:39
DinaBelovaNo group id defined, returning the whole iterable.09:39
* DinaBelova is going to go and die09:39
DinaBelovahey duplicated the work09:40
DinaBelovathey*09:40
DinaBelovawtf09:41
DinaBelovathey behave strange!09:41
nsajeDinaBelova: you don't have workload_partitioning option enabled in the [compute] section of ceilometer config09:41
nsajeDinaBelova: compute isn't partitioned by default09:41
DinaBelovansaje, a-ha, yeah, I did that on the wrong compute09:42
DinaBelovansaje, okay09:45
DinaBelovaso the result09:45
*** yassine has quit IRC09:55
DinaBelovaon the multimode with multiple compute agents on one node they share the load by VMs (with all meters per VM)09:57
nsajegreat!09:59
nsajethat's how it's supposed to work!10:00
DinaBelovansaje, am I right that Neutron (that has the default discovery) should be polled by central agents ok by your change?10:00
DinaBelovansaje, ok10:00
DinaBelovacool-cool10:00
nsajelet's see, neutron10:00
*** Yanyanhu has quit IRC10:00
DinaBelovalike ip.floating, etc10:00
nsaje(I'm just in the process of figuring out how best to migrate everytihng to discovery model)10:00
nsajeno, ip.floating won't work yet10:00
DinaBelovaehm, why?10:00
*** isviridov is now known as isviridov_awa10:00
*** isviridov_awa is now known as isviridov_awat10:00
nsajeah10:00
*** isviridov_awat is now known as isviridov_away10:00
nsajelet me check10:01
DinaBelovaok10:01
nsajebecause ip.floating polls nova, not neutron10:01
nsaje(I don't really know why this isn't in compute agent)10:01
DinaBelovao_010:01
nsajeDinaBelova: https://github.com/openstack/ceilometer/blob/master/ceilometer/network/floatingip.py#L3410:02
DinaBelovansaje, well10:02
DinaBelovawhat can I poll via central agent?!10:02
DinaBelovafor that thing to work?10:03
nsajeDinaBelova: gimme a sec10:03
DinaBelovansaje, ok :)10:03
nsajeDinaBelova: you're not going to like this :) just the ones that have 'discovery:' param specified https://github.com/openstack/ceilometer/blob/master/etc/ceilometer/pipeline.yaml10:04
nsajeDinaBelova: others don't use discoveries yet10:05
DinaBelova>_<10:05
DinaBelovahehe10:05
DinaBelovavpnaas, lbaas and fwaas10:05
nsajeDinaBelova: but I think your tests on compute indicate that at least the code is working as intended10:05
DinaBelovamagical thing10:05
nsajeDinaBelova: (as it's literally the same code, just different entry point)10:05
DinaBelovawell, yeah10:05
nsajeDinaBelova: even if the final functionality isn't yet there for the central agent, because it'll take additional patches10:06
*** Qiming__ has quit IRC10:08
DinaBelovansaje, so you're planning to give more patches *before* your change10:08
DinaBelovato allow discovery model everywhere10:08
nsajeDinaBelova: yes, though there is no code dependency between the patches10:08
DinaBelovansaje, yeah, but his discovery thing needs to go first10:09
DinaBelovathis*10:09
nsajeDinaBelova: yep, workin' on it10:09
DinaBelovaokay10:09
*** nosnos has quit IRC10:20
*** nosnos has joined #openstack-ceilometer10:21
*** nosnos has quit IRC10:25
DinaBelovansaje, btw - do I understand write that compute agents works only because you've added some code to the compute.discovery?10:28
DinaBelovaand that's it?10:28
nsajeDinaBelova: yes10:29
DinaBelovaa-ha10:29
nsajeDinaBelova: well, not exactly, it would work even if I wouldn't add that code10:29
DinaBelovawith sharing the load?10:29
nsajeDinaBelova: yes, but only if there was a single node10:29
DinaBelovaa-ha10:30
nsajeDinaBelova: the added code in compute discovery makes sure that compute agents are grouped by node10:30
DinaBelovaokay, gotca10:30
DinaBelovaokay, gotcha*10:30
openstackgerritIgor Degtiarov proposed a change to openstack/ceilometer: [HBase] Catch AlreadyExists error in Connection upgrade  https://review.openstack.org/11350410:31
openstackgerritIgor Degtiarov proposed a change to openstack/ceilometer: [HBase] Add migration script for new row separate design  https://review.openstack.org/11561510:32
*** nosnos has joined #openstack-ceilometer10:34
*** isviridov_away is now known as isviridov10:37
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Allow to requeue a sample  https://review.openstack.org/11222510:43
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Document pipeline publishers configuration  https://review.openstack.org/11491910:43
nsajeDinaBelova: do you still have multinode devstack up?10:54
DinaBelovayep10:54
DinaBelovansaje ^^10:54
nsajeDinaBelova: can you execute 'keystone endpoint-get --service compute' for me?10:55
nsajeDinaBelova: I'd like to see what that outputs on a multinode setup10:55
DinaBelovansaje - http://paste2.org/5sKtdyjP10:56
DinaBelovastrange a little bit10:56
DinaBelovaas we're having 3 compute nodes actually10:56
nsajeDinaBelova: yeah10:56
DinaBelovansaje, btw, it'll be the same output is this will be processed from other compute node10:58
DinaBelovajust keystone endpoint, and that's it10:58
nsajeDinaBelova: I guess I don't understand keystone enough10:58
* DinaBelova as well10:58
nsajeyou do have nova-api running on each node right?10:59
DinaBelovano, only onthe controller10:59
DinaBelovaah11:00
DinaBelovathat's actually normal :)11:00
nsajeok11:00
DinaBelovaone controller two computes11:00
DinaBelovaI may run one more api service11:00
DinaBelovaand look on what will happen11:00
nsajeso IIUC, you have a controller node and two compute nodes11:01
nsajeand then ceilometer-agent-compute on each of those compute nodes?11:01
DinaBelovayep, and controller might be compute as well11:01
DinaBelovayes11:01
nsajewhich then polls the controller node about its compute node11:01
DinaBelovansaje, why should it?11:02
nsajewell, it polls nova API11:02
nsajeso it polls the controller node, right?11:02
DinaBelovaah, yeah11:02
DinaBelovabut it polls only for some of the meters11:02
nsajeyes11:03
DinaBelovayep11:03
nsaje(only for the meters for VMs on its own compute node)11:03
DinaBelova(nod)11:03
DinaBelovasorry, need to go afk for a moment11:03
nsajethis was an easy way to 'scale' compute, but don't you think the partitioning code obsoletes that?11:03
nsajeperhaps we could even merge central&compute back together and to the scaling via the partitioning code11:04
nsaje(just thinking out loud)11:04
*** ildikov has quit IRC11:05
*** ildikov has joined #openstack-ceilometer11:07
*** ildikov has quit IRC11:12
*** nosnos has quit IRC11:18
*** nosnos has joined #openstack-ceilometer11:20
*** Qiming has joined #openstack-ceilometer11:24
DinaBelovansaje, well, I guess that currently that will make sense - just set local_instances as discover for this *agent* and that's it11:26
DinaBelovaalthough we need more eyes here :S11:27
nsajeDinaBelova: that's K- effort anyway, running out of time for juno11:27
DinaBelovayeah, for sure11:27
DinaBelovanow we need to understand what to do with the discovery thing :)11:28
*** flwang1 has quit IRC11:57
*** cdent has joined #openstack-ceilometer11:59
*** _nadya_ has quit IRC12:02
*** _nadya_ has joined #openstack-ceilometer12:02
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Alarm: Use stevedore to load the service class  https://review.openstack.org/11719812:02
*** foexle has joined #openstack-ceilometer12:11
foexlehi guys, i've a short question about the ceilometer-expirer. I don't find any documentations about it so i need to ask :) ... how i understood the process should look at all schemas > TTL and delete it correct ?12:13
foexleif i've ttl = -1 all schemas will be store anytime12:13
*** erecio has joined #openstack-ceilometer12:18
*** cdent has quit IRC12:19
*** nosnos has quit IRC12:22
*** nosnos has joined #openstack-ceilometer12:23
*** _nadya_ has quit IRC12:26
nsajefoexle: yes12:27
*** nosnos has quit IRC12:27
*** sdake_ has quit IRC12:28
foexlensaje: ok thx for your answer, but the expirer is running noch over 30 min and my database size is the same12:28
foexlettl is set >012:28
nsajewhich DB are you using? not all DB backends support it12:28
foexlemongodb12:29
foexlei dont see any deletes in mongostat12:29
nsajefoexle: mongodb should work, you're right12:29
nsajefoexle: let me check, there were some changes to the services, perhaps some bug creeped in12:30
foexlensaje: Version: 2013.2.3-0ubuntu1~cloud012:30
foexlensaje: mongodb: 2.4.812:31
*** julim has joined #openstack-ceilometer12:33
*** sdake_ has joined #openstack-ceilometer12:34
*** julim has quit IRC12:35
*** gordc has joined #openstack-ceilometer12:37
*** julim has joined #openstack-ceilometer12:39
nsajefoexle: in this case it's not some fresh bug, you're using havana. I don't have enough mongo-foo to help you out right now, sorry. Take a look at the code https://github.com/openstack/ceilometer/blob/2013.2/ceilometer/storage/impl_mongodb.py#L425 or file a bug report12:39
*** urulama has joined #openstack-ceilometer12:39
*** Yanyanhu has joined #openstack-ceilometer12:43
*** urulama has left #openstack-ceilometer12:48
*** asalkeld has quit IRC12:49
*** sdake_ has quit IRC12:54
*** sdake_ has joined #openstack-ceilometer12:55
*** rbowen has joined #openstack-ceilometer12:57
*** itisit has joined #openstack-ceilometer12:59
foexlensaje: all right, thanks :)13:00
*** urulama has joined #openstack-ceilometer13:04
*** urulama has left #openstack-ceilometer13:04
*** jaypipes has joined #openstack-ceilometer13:09
*** k4n0 has quit IRC13:10
*** sdake_ has quit IRC13:11
openstackgerritEric Pendergrass proposed a change to openstack/python-ceilometerclient: Verify alarm found before modifying  https://review.openstack.org/11034513:14
*** cdent has joined #openstack-ceilometer13:32
openstackgerritA change was merged to stackforge/gnocchi: swift: avoid doing a HEAD to check container presence  https://review.openstack.org/11627613:37
*** joesavak has joined #openstack-ceilometer13:42
*** rbak has quit IRC13:43
*** jsavak has joined #openstack-ceilometer13:44
openstackgerritJulien Danjou proposed a change to stackforge/gnocchi: storage: implement archiving policy  https://review.openstack.org/11355513:46
openstackgerritJulien Danjou proposed a change to stackforge/gnocchi: storage: add support for creating storage policies  https://review.openstack.org/11666113:46
openstackgerritJulien Danjou proposed a change to stackforge/gnocchi: indexer: make entities be resources  https://review.openstack.org/11724613:46
*** joesavak has quit IRC13:48
openstackgerritRikimaru Honjo proposed a change to openstack/ceilometer: Add validate alarm_actions schema in alarm API  https://review.openstack.org/10733013:51
*** thomasem has joined #openstack-ceilometer13:56
*** idegtiarov has left #openstack-ceilometer13:59
*** idegtiarov has joined #openstack-ceilometer13:59
*** pradk has joined #openstack-ceilometer13:59
openstackgerritJulien Danjou proposed a change to stackforge/gnocchi: indexer: use sqlalchemy-utils for UUID type  https://review.openstack.org/11751914:00
*** taplax has quit IRC14:01
*** r-daneel has joined #openstack-ceilometer14:05
*** rbak has joined #openstack-ceilometer14:06
*** claudiub has joined #openstack-ceilometer14:13
*** sdake_ has joined #openstack-ceilometer14:13
*** dmsimard_away is now known as dmsimard14:15
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Allow to requeue a sample  https://review.openstack.org/11222514:17
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Document pipeline publishers configuration  https://review.openstack.org/11491914:17
htrutahello, everyone! I want to have an alarm that is triggered when I get some kind of trend. For example, an alarm triggered when the cpu utilization growth is similar to a linear function. Is there any way to do this kind of job? I took a quick look on the code and saw that new evaluators can be created. is it a possible way?14:17
*** dmsimard has left #openstack-ceilometer14:18
*** raildo has joined #openstack-ceilometer14:21
openstackgerritPiyush Masrani proposed a change to openstack/ceilometer-specs: VMware Vsphere Declarative polling spec  https://review.openstack.org/10513014:24
*** Longgeek_ has quit IRC14:24
*** _nadya_ has joined #openstack-ceilometer14:25
*** Qiming has quit IRC14:30
openstackgerritPhil Neal proposed a change to openstack/ceilometer: Document the standard for PaaS service notifications  https://review.openstack.org/11339614:32
*** sdake_ has quit IRC14:33
*** dragondm has joined #openstack-ceilometer14:40
*** _nadya_ has quit IRC14:40
*** fabiog has joined #openstack-ceilometer14:50
*** ildikov has joined #openstack-ceilometer14:51
*** shengjiemin has joined #openstack-ceilometer14:53
*** _nadya_ has joined #openstack-ceilometer14:58
*** thomasem has quit IRC14:59
*** nealph has joined #openstack-ceilometer14:59
*** thomasem has joined #openstack-ceilometer15:02
*** yassine has joined #openstack-ceilometer15:06
*** underyx|off is now known as underyx15:06
*** jsavak has quit IRC15:07
*** sdake_ has joined #openstack-ceilometer15:07
*** joesavak has joined #openstack-ceilometer15:07
*** joesavak has quit IRC15:07
*** sbfox has joined #openstack-ceilometer15:31
*** sbfox has quit IRC15:31
*** IvanBerezovskiy has left #openstack-ceilometer15:33
*** foexle has quit IRC15:35
*** sdake_ has quit IRC15:35
*** Yanyanhu has quit IRC15:38
*** nealph has quit IRC15:40
*** _cjones_ has joined #openstack-ceilometer15:46
*** KurtRao has joined #openstack-ceilometer15:59
jd__ok SQLAlchemy expert, here's my problem http://paste.openstack.org/show/101806/16:00
ildikovfabiog: but devstack creates the url with v2.0 IIRC16:00
gordci blame all of your for tacking on an extra 30mins to meeting.lol16:00
jd__:D16:01
nsajehaha16:01
fabiogildikov: so the Keystone client only accepts V3 calls in port 3535716:01
ildikovgordc: business as usual :)16:01
jd__gonna post my problem on Twitter so I can relax and wait for people to write my code16:01
cdentjd__ that paste makes no sense16:02
ildikovfabiog: ok, so that is where the problem originates16:02
gordcjd__: i was going to type you're going to get a bunch of people trolling you if you post to twitter16:02
fabiogildikov: yes, basically if you are using port 35357 you have to have a V3 compliant request16:03
fabiogbecause that is the admin port16:03
fabiogport 5000 supports both V3 and V2 (it is both the internal an public port)16:03
gordcjd__: you can just define a relationship no?16:03
*** nealph has joined #openstack-ceilometer16:03
fabiogildikov: if this is not acceptable, we need to ask the Keystone guys to enable V2 on 35357 on devstack, because the current default is V3 only16:04
ildikovfabiog: I should have thought as I didn't find any other code related issues... :)16:04
fabiogildikov: AFAIK, the only service that uses that port is Horizon and services can use it to vaildate tokens16:04
fabioggordc: can we close the bug then?16:05
cdentjd__: a_name = relationship('A', backref='name') # might be close to what you want16:06
*** erecio has quit IRC16:06
cdentBut I'm not sure because your desired SQL is confused16:06
gordccdent: that returns entire A obj.. not just attribute... although i guess it works still.16:07
jd__what's confusing?16:08
cdentyou've got two from16:08
jd__AFAIU backref adds an attribute on A, not on B16:09
cdentyeah, backref not needed16:09
gordcfabiog: just to clarify, what scenario causes the error to show up? if you use v2 against 35357?16:09
cdentjd__ can you point to real code?16:09
gordcjd__: that's right.16:09
fabioggordc: exactly16:10
fabiogonly that16:10
jd__cdent: not sure that would be less confusing :))16:10
gordchave we always been pointing to 35357? or is this more a custom configuration issue16:11
gordcfabiog: ^16:11
cdentjd__ I'm only asking because I don't really understand what you are trying to do, so thought some context might help16:11
jd__cdent: let me try16:11
openstackgerritJulien Danjou proposed a change to stackforge/gnocchi: storage: implement archiving policy  https://review.openstack.org/11355516:11
openstackgerritJulien Danjou proposed a change to stackforge/gnocchi: indexer: make entities be resources  https://review.openstack.org/11724616:11
ildikovgordc: devstack deploy the system with that port16:11
openstackgerritJulien Danjou proposed a change to stackforge/gnocchi: indexer: handle archive policies  https://review.openstack.org/11755116:11
jd__cdent: https://review.openstack.org/#/c/117551/16:11
jd__cdent: in the test you can see I pass the name of the archive policy rather than the id, I'd like the ORM to automatically resolve it to the archive policy id16:12
gordcildikov: hmm.. but why would it point to 35357 if it only works for v316:12
fabioggordc: I think we have always pointed to 5000, isn't that the standard port in the Ceilo conf?16:12
gordcfabiog: to my knowledge yes16:12
*** erecio has joined #openstack-ceilometer16:13
fabioggordc: if Ildikov thinks that there is a need for V2 on 35357 we can open a ticket ot Keystone to accept V2 on that port16:13
fabiogmy undestanding is that is a conf change16:13
fabiogs/undestanding/understanding16:13
ildikovgordc: TBH, I don't know, I thought it was used always, but now it has issues because of the versions16:14
gordcyeah if the default conf settings make sense maybe the fix is to add doc16:14
*** foexle has joined #openstack-ceilometer16:14
openstackgerritNejc Saje proposed a change to openstack/ceilometer: Central agent work-load partitioning  https://review.openstack.org/11354916:15
openstackgerritNejc Saje proposed a change to openstack/ceilometer: Switch partitioned alarm evaluation to a hash-based approach  https://review.openstack.org/11523716:15
cdentthanks jd__, reading16:17
fabioggordc: I would close the bug, because ceilometer is using port 35357 for the token validation only and that is correct. If you use the Ceilometer Client you have to use the port 5000 because that is the non admin one. Maybe this needs to be made clear in the docs16:17
fabioggordc: and in devstack everything is working fine16:17
fabiogildikov: do you agree?16:18
*** isviridov is now known as isviridov_away16:18
gordcfabiog: i'd agree with that... doesn't seem like it's a backward compatibilty issue either... just a configuration error16:18
ildikovfabiog: gordc: but I also do not get why 35357 is working only with v3..., but what fabiog suggests is fine with me16:19
fabiogildikov: can you please clarify this point with the Keystone folks?16:19
gordcjd__: http://docs.sqlalchemy.org/en/rel_0_7/orm/extensions/hybrid.html#join-dependent-relationship-hybrid16:19
fabiogI don't want to close the bug if you are not happy with the resolution16:20
gordcjd__: i think that's the way if you don't want ot access attribute through object.16:20
jd__gordc: found that but that's jeberish16:20
gordclol16:20
fabioggordc: is not even a conf issue, is really a doc issue. We should let know that Ceilo client users should use port 5000 and not 3535716:21
ildikovfabiog: I'm not 100% sure that it would worth the effort with the Keystone folks, so we can close this bug, but it should be mentioned in the docs that how it should be used16:21
fabiogildikov: I actually ping a colleage and he gave me the explanation16:21
fabiogildikov: 35357 is the port for V3 only because it was introduced with V3 and 5000 is the legacy V2 port that supports also V316:22
ildikovfabiog: I will add a note about this in the Admin guide, as I'm writing the related section now16:22
fabiogildikov: he claims that when (if) we all move to V3, 5000 will be deprecated16:22
gordcjd__: you'll probably need to save it through objec then... ie. B.A.name16:22
jd__gordc: my problem is more assignation16:23
ildikovfabiog: a-ha, ok, got it, actually I didn't really follow the port evolution within Keystone :)16:23
gordci've idea how to do it without more googling16:23
fabiogildikov: you don;t know how much fun you missed ;-)16:23
ildikovfabiog: LOL :)16:23
gordcno idea*16:24
ildikovfabiog: and also thanks for clarifying the issue behind this "bug"16:25
fabiogildikov: no problem16:27
cdentjd__ another option is to a) make the relationship as described above, create the archive object (in the session) by name, assign it to A, and then flush the session and let sqlA figured it out16:28
fabiogildikov: are you going to follow up on the docs side please?16:28
fabioggordc; can we close the bug then?16:28
jd__cdent: yeah that's what I wanted to avoid but I'll do that in the end I guess16:28
cdentwhat's bad about that?16:28
openstackgerritJames E. Blair proposed a change to openstack/ceilometer: Enhance compute diskio tests to handle multi instance  https://review.openstack.org/11499116:29
cdenta = A(name='foo'); B.A=a; session.add(A), session.add(B); flush # <- very much guesstimate16:29
gordcfabiog: i'm ok closing. just add a note i guess. we can always reopen if there's a debate.16:30
cdentif your foreign keys are set up right, it ought to come out correct16:30
*** jaypipes has quit IRC16:30
fabioggordc: thanks16:30
*** idegtiarov has quit IRC16:32
* gordc grabbing lunch16:33
*** _nadya_ has quit IRC16:34
*** nealph has quit IRC16:35
*** sbfox has joined #openstack-ceilometer16:39
*** _nadya_ has joined #openstack-ceilometer16:40
*** KurtRao has quit IRC16:45
*** KurtRao has joined #openstack-ceilometer16:46
*** jaypipes has joined #openstack-ceilometer16:46
*** _nadya_ has quit IRC16:49
*** fabiog has quit IRC16:50
*** yassine has quit IRC16:51
*** lsmola has quit IRC16:56
*** sbfox has quit IRC16:57
*** sbfox has joined #openstack-ceilometer17:00
*** harlowja_away is now known as harlowja_17:03
*** harlowja_ is now known as harlowja_away17:04
*** harlowja_away is now known as harlowja_17:09
*** sdake_ has joined #openstack-ceilometer17:14
*** erecio has quit IRC17:17
*** erecio has joined #openstack-ceilometer17:20
*** sdake_ has quit IRC17:22
*** _nadya_ has joined #openstack-ceilometer17:29
pradkgordc, ping17:31
gordcpradk: whatsup?17:34
pradkgordc, so looks like this got kicked out of the gate https://review.openstack.org/#/c/114991/ ..  oslo common got updated in ceilometer since this patch?17:34
*** nsaje has quit IRC17:35
gordcpradk: yeah... it's going to be merge hell for next few weeks i think17:35
pradkgordc, so instead of from ceilometer.openstack.common.fixture import mockpatch i need to do .. from oslotest import mockpatch ?17:36
gordci guess they're manually cleaning up gate since it's backed up so much.17:36
*** sdake_ has joined #openstack-ceilometer17:37
*** zul has quit IRC17:37
*** nsaje has joined #openstack-ceilometer17:37
*** shakamunyi has joined #openstack-ceilometer17:41
*** lynxbat has joined #openstack-ceilometer17:43
gordcpradk: whoops. didn't see that last message17:44
gordcyeah. that's the correct fix17:44
*** sbfox has quit IRC17:45
pradkgordc, cool17:45
*** lynxbat has quit IRC17:45
*** lynxbat has joined #openstack-ceilometer17:46
*** zul has joined #openstack-ceilometer17:50
*** _nadya_ has quit IRC17:57
*** sdake_ has quit IRC17:57
zhiyanhi folks, when you ok, could you pls talk a look on this sample one: https://review.openstack.org/#/c/116672/ thanks!18:00
*** _nadya_ has joined #openstack-ceilometer18:06
*** sbfox has joined #openstack-ceilometer18:06
*** sbfox1 has joined #openstack-ceilometer18:07
*** sbfox has quit IRC18:11
*** sdake has quit IRC18:12
*** thomasem has quit IRC18:18
*** sbfox has joined #openstack-ceilometer18:21
*** sbfox1 has quit IRC18:21
*** thomasem has joined #openstack-ceilometer18:24
openstackgerritA change was merged to openstack/ceilometer: XenAPI support: Changes for cpu_util  https://review.openstack.org/11245318:33
openstackgerritA change was merged to openstack/ceilometer: In-code comments should start with `#`, not with `"""`  https://review.openstack.org/11563318:34
*** thomasem has quit IRC18:37
openstackgerritA change was merged to openstack/python-ceilometerclient: Check if the alarm has time constraints field before displaying  https://review.openstack.org/9571318:40
*** sdake_ has joined #openstack-ceilometer18:40
*** thomasem has joined #openstack-ceilometer18:41
*** sdake_ has quit IRC18:42
*** sdake has joined #openstack-ceilometer18:42
*** sdake has joined #openstack-ceilometer18:42
*** lynxbat has quit IRC18:49
*** sbfox has quit IRC18:49
*** cdent has quit IRC18:49
*** _nadya_ has quit IRC18:54
*** sdake has quit IRC18:56
*** sbfox has joined #openstack-ceilometer18:57
*** lynxbat has joined #openstack-ceilometer18:59
*** sdake has joined #openstack-ceilometer19:02
*** sdake has joined #openstack-ceilometer19:02
*** lynxbat has quit IRC19:05
*** lynxbat has joined #openstack-ceilometer19:07
*** sbfox has quit IRC19:08
*** sbfox1 has joined #openstack-ceilometer19:08
*** sdake has quit IRC19:08
*** lynxbat has quit IRC19:09
*** sdake has joined #openstack-ceilometer19:09
*** sdake has joined #openstack-ceilometer19:09
*** chuck_ has joined #openstack-ceilometer19:14
*** zul has quit IRC19:16
*** chuck_ has quit IRC19:20
*** zul has joined #openstack-ceilometer19:20
*** sdake_ has joined #openstack-ceilometer19:23
*** openstackgerrit has quit IRC19:31
*** openstackgerrit has joined #openstack-ceilometer19:32
*** sdake_ has quit IRC19:36
*** sdake_ has joined #openstack-ceilometer19:37
*** sdake_ has quit IRC19:38
*** sdake_ has joined #openstack-ceilometer19:38
*** claudiub has quit IRC19:42
openstackgerritA change was merged to openstack/ceilometer: add documentation for setting up api pipeline  https://review.openstack.org/11691719:58
openstackgerritA change was merged to openstack/ceilometer: Remove oslo middleware  https://review.openstack.org/11568119:59
*** sbfox1 has quit IRC20:08
*** lynxbat has joined #openstack-ceilometer20:08
*** flwang1 has joined #openstack-ceilometer20:13
*** sbfox has joined #openstack-ceilometer20:13
*** flwang1 has quit IRC20:16
*** Dafna has quit IRC20:23
*** mspreitz has joined #openstack-ceilometer20:28
mspreitzIn recent DevStack installs, I am seeing ceilometer alarm evaluator fail to start , with this complaint: AttributeError: 'module' object has no attribute 'import_object'20:28
mspreitzHas anybody else tried a recent install of HEAD?20:28
gordcmspreitz: there's a bug open for this: https://bugs.launchpad.net/ceilometer/+bug/136200620:29
gordcerr.. although it points to wrong patch. right one -> https://review.openstack.org/#/c/117198/20:30
*** _slo has joined #openstack-ceilometer20:30
*** foexle has quit IRC20:32
mspreitzgordc: what about https://review.openstack.org/#/c/116940/ ?20:32
*** promulo_ has quit IRC20:32
gordcmspreitz: it's an option... but we use stevedore in ceilometer and the current patch makes it fall more inline with 'ceilometer mantra'20:33
_sloDoes Ceilometer meter how much data was written within some Swift user account?20:33
gordc_slo: not for user specifically. you'd have to filter by user to get that data. http://docs.openstack.org/developer/ceilometer/measurements.html#object-storage-swift20:35
mspreitzgordc: by current patch you mean https://review.openstack.org/#/c/117198/ right?20:35
gordcmspreitz: yes20:35
mspreitzgordc: thanks20:36
gordcnp20:36
_slogordc: since Ceph is also now somehow integrated with Open Stack, does Celiometer also works with RadosGW object store within Ceph?20:37
gordc_slo: not sure... not familir with Rados gateway... the actual incoming/outgoing bytes meters is built using a wsgi middleware filter...20:42
_sloaha ok, possible not then, since I thing Ceph goes with FastCGI what matters pipes (aka WSGI)20:42
gordc_slo: i see.. i guess it would require a different approach then.20:43
_sloyes, probably20:43
*** sbfox has quit IRC20:43
*** sbfox has joined #openstack-ceilometer20:43
_slosince Ceph is now integrated with openstack, does anyone know is there some middleware actual written , as is for Swift, that is actual usable?20:44
openstackgerritPhil Neal proposed a change to openstack/ceilometer: Document the standard for PaaS service notifications  https://review.openstack.org/11339620:47
*** jaypipes has quit IRC20:50
gordc_slo: may want to ask during non-american hours... a lot of the ceilometer devs are based in the americas20:51
stevelleanyone able to confirm the date of the v2 REST API becoming stable?  Was it 04 Apr 2013 (Grizzly)?20:57
*** thomasem has quit IRC21:02
*** mspreitz has quit IRC21:04
openstackgerritgordon chung proposed a change to openstack/ceilometer: Enhance compute diskio tests to handle multi instance  https://review.openstack.org/11499121:06
*** rbowen has quit IRC21:06
*** foexle has joined #openstack-ceilometer21:15
*** dhellmann is now known as dhellmann_21:17
openstackgerritA change was merged to openstack/ceilometer: XenAPI support: Memory Usage  https://review.openstack.org/11486721:19
openstackgerritPradeep Kilambi proposed a change to openstack/ceilometer: Enhance compute diskio tests to handle multi instance  https://review.openstack.org/11499121:25
*** flwang_ has joined #openstack-ceilometer21:28
*** flwang_ has quit IRC21:29
*** _slo has quit IRC21:35
*** alexpilotti has quit IRC21:47
*** sbfox has quit IRC21:48
*** sbfox has joined #openstack-ceilometer21:48
*** _nadya_ has joined #openstack-ceilometer21:54
*** erecio has quit IRC21:57
*** _nadya_ has quit IRC21:59
*** asalkeld has joined #openstack-ceilometer22:02
*** pradk has quit IRC22:12
*** itisit has quit IRC22:13
openstackgerritgordon chung proposed a change to openstack/ceilometer: normalise resource data  https://review.openstack.org/11131322:25
openstackgerritgordon chung proposed a change to openstack/ceilometer: switch to sqlalchemy core  https://review.openstack.org/11352422:26
*** gordc has quit IRC22:26
*** sbfox has quit IRC22:44
*** sbfox1 has joined #openstack-ceilometer22:44
*** sbfox1 has quit IRC22:44
*** sbfox has joined #openstack-ceilometer22:44
openstackgerritA change was merged to openstack/ceilometer: update requirements  https://review.openstack.org/11631122:58
*** rbak has quit IRC23:01
*** asalkeld has quit IRC23:01
*** lynxbat has quit IRC23:01
*** lynxbat has joined #openstack-ceilometer23:03
*** lynxbat has joined #openstack-ceilometer23:04
*** asalkeld has joined #openstack-ceilometer23:09
*** sbfox has quit IRC23:25
*** shakamunyi has quit IRC23:33
*** KurtRao has quit IRC23:41
*** lynxbat has quit IRC23:42
*** lynxbat has joined #openstack-ceilometer23:46
*** openstackgerrit has quit IRC23:46
*** openstackgerrit has joined #openstack-ceilometer23:47

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!