Monday, 2014-03-24

*** matsuhashi has joined #openstack-ceilometer00:25
*** liusheng has quit IRC01:13
*** nosnos has joined #openstack-ceilometer01:36
*** flwang has quit IRC01:39
openstackgerritliusheng proposed a change to openstack/python-ceilometerclient: Reduce redundant parameter of some commands in CLI  https://review.openstack.org/6677601:52
*** liusheng has joined #openstack-ceilometer01:53
*** xianghui has joined #openstack-ceilometer02:02
*** xianghui has quit IRC02:08
*** xianghui has joined #openstack-ceilometer02:10
*** flwang has joined #openstack-ceilometer02:18
*** matsuhashi has quit IRC02:55
*** matsuhashi has joined #openstack-ceilometer03:21
*** nati_ueno has joined #openstack-ceilometer03:23
*** ekarlso has quit IRC03:48
*** ekarlso has joined #openstack-ceilometer03:48
*** terriyu has joined #openstack-ceilometer04:00
*** terriyu has quit IRC04:01
*** terriyu has joined #openstack-ceilometer04:02
*** Akshik has joined #openstack-ceilometer04:21
*** terriyu has quit IRC04:35
*** nati_ueno has quit IRC04:46
*** nati_ueno has joined #openstack-ceilometer04:47
*** nati_ueno has quit IRC04:51
*** nati_ueno has joined #openstack-ceilometer04:52
*** nosnos_ has joined #openstack-ceilometer05:16
*** nosnos has quit IRC05:19
*** Akshik has quit IRC05:20
*** Akshik has joined #openstack-ceilometer05:20
*** Akshik_ has joined #openstack-ceilometer05:25
*** Akshik has quit IRC05:26
*** Akshik_ has quit IRC05:33
*** Akshik has joined #openstack-ceilometer05:35
*** Akshik has quit IRC05:41
*** Akshik has joined #openstack-ceilometer05:46
*** sdake_ has quit IRC05:47
*** Akshik has quit IRC05:53
*** Akshik_ has joined #openstack-ceilometer05:54
*** Akshik_ has quit IRC05:57
*** Akshik_ has joined #openstack-ceilometer05:57
*** mihgen has joined #openstack-ceilometer06:00
*** matsuhas_ has joined #openstack-ceilometer06:00
*** matsuhashi has quit IRC06:01
*** flwang has quit IRC06:01
openstackgerritJenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex  https://review.openstack.org/7676206:05
*** flwang has joined #openstack-ceilometer06:09
*** nati_ueno has quit IRC06:14
openstackgerritliusheng proposed a change to openstack/ceilometer: Avoid query samples before time of earliest sample  https://review.openstack.org/8204806:16
*** nosnos has joined #openstack-ceilometer06:19
*** nosnos_ has quit IRC06:22
*** ildikov_ has quit IRC06:26
*** matsuhas_ has quit IRC06:31
*** matsuhashi has joined #openstack-ceilometer06:31
openstackgerritliusheng proposed a change to openstack/ceilometer: Avoid query samples before time of earliest sample  https://review.openstack.org/8204806:32
*** AMike has joined #openstack-ceilometer06:34
*** AMike has joined #openstack-ceilometer06:34
*** saju_m has joined #openstack-ceilometer06:35
*** saju_m has quit IRC06:59
*** saju_m has joined #openstack-ceilometer07:12
*** ildikov_ has joined #openstack-ceilometer07:13
*** saju_m has quit IRC07:17
*** saju_m has joined #openstack-ceilometer07:17
*** mihgen_ has joined #openstack-ceilometer07:18
*** Akshik__ has joined #openstack-ceilometer07:19
*** Ruetobas has quit IRC07:19
*** nprivalova has quit IRC07:19
*** AMike has quit IRC07:19
*** Akshik_ has quit IRC07:19
*** mihgen has quit IRC07:20
*** liusheng has quit IRC07:20
*** mihgen_ is now known as mihgen07:20
*** liusheng has joined #openstack-ceilometer07:20
*** saju_m has quit IRC07:22
*** Akshik__ has quit IRC07:23
*** Ruetobas has joined #openstack-ceilometer07:23
*** saju_m has joined #openstack-ceilometer07:24
*** Akshik has joined #openstack-ceilometer07:24
*** flwang has quit IRC07:28
*** saju_m has quit IRC07:30
*** saju_m has joined #openstack-ceilometer07:31
*** AMike has joined #openstack-ceilometer07:32
*** AMike has joined #openstack-ceilometer07:32
*** flwang has joined #openstack-ceilometer07:33
*** nprivalova has joined #openstack-ceilometer07:33
*** Akshik has quit IRC07:36
*** AMike has quit IRC07:36
*** Akshik has joined #openstack-ceilometer07:56
*** bogdando has joined #openstack-ceilometer08:26
*** Akshik has quit IRC08:36
*** renlt has joined #openstack-ceilometer08:42
*** Akshik has joined #openstack-ceilometer08:48
*** mihgen has quit IRC08:50
*** AMike has joined #openstack-ceilometer08:52
*** AMike has joined #openstack-ceilometer08:52
*** AMike has quit IRC08:52
*** ilyashakhat has quit IRC08:55
*** nacim has joined #openstack-ceilometer09:04
*** yassine has joined #openstack-ceilometer09:05
*** flwang has quit IRC09:20
*** ilyashakhat has joined #openstack-ceilometer09:20
*** liusheng has quit IRC09:21
*** liusheng has joined #openstack-ceilometer09:21
openstackgerritMichal Jastrzebski proposed a change to openstack/python-ceilometerclient: Implementation of statistics aggregators  https://review.openstack.org/8049909:26
*** nacim has quit IRC09:28
*** inc0 has joined #openstack-ceilometer09:28
*** mihgen has joined #openstack-ceilometer09:29
*** Akshik has quit IRC09:30
*** renlt has quit IRC09:35
*** nacim has joined #openstack-ceilometer09:36
*** admin0 has joined #openstack-ceilometer09:39
admin0hey guys09:40
*** AMike has joined #openstack-ceilometer09:42
*** AMike has joined #openstack-ceilometer09:42
inc0hello09:42
*** flwang has joined #openstack-ceilometer10:14
*** Akshik has joined #openstack-ceilometer10:16
*** Akshik has quit IRC10:23
*** Akshik has joined #openstack-ceilometer10:24
*** matsuhashi has quit IRC10:33
*** matsuhas_ has joined #openstack-ceilometer10:35
*** eglynn has joined #openstack-ceilometer10:49
*** yassine has quit IRC10:52
eglynnjd__: boo! :( ... "Unfortunately, your submission was not chosen to be part of the official agenda this time around ... Quo vadis Ceilometer"11:04
eglynnjd__: ... better luck with your other ceilo proposal?11:04
*** matsuhas_ has quit IRC11:07
*** admin0 has quit IRC11:20
*** admin0 has joined #openstack-ceilometer11:21
*** xianghui has quit IRC11:22
*** Akshik has quit IRC11:22
*** Akshik_ has joined #openstack-ceilometer11:23
*** nosnos has quit IRC11:23
*** Akshik_ has quit IRC11:28
openstackgerritUros Jovanovic proposed a change to openstack/python-ceilometerclient: Adds alarm time constraint support to ceilometer CLI  https://review.openstack.org/7835611:38
openstackgerritJenkins proposed a change to openstack/ceilometer: Updated from global requirements  https://review.openstack.org/7763911:47
*** matsuhashi has joined #openstack-ceilometer11:53
*** nosnos has joined #openstack-ceilometer11:54
openstackgerritJenkins proposed a change to openstack/pycadf: Updated from global requirements  https://review.openstack.org/7969311:54
inc0eglynn, hi, about your comment here -> https://review.openstack.org/#/c/80499/8/ceilometerclient/v2/shell.py are we sure that we'd like to change output of this call?11:58
inc0I mean line :4811:58
eglynninc0: the suggestion was just to change the format of the --aggregate option, not the output11:59
eglynninc0: ... so basically --aggregate 'cardinality<-resource_id' instead of --aggregate 'cardinality::resource_id' as '::' is already used by the CLI to mean something else11:59
inc0ah, sorry, I asked for line 77, yes, I've fixed :4812:00
inc0and now I see what you meant in :7712:00
eglynninc0: a-ha, yes ... so in the case where I call say: ceilometer statistics --aggregate count --aggregate 'cardinality<-resource_id' ...12:01
eglynninc0: ... the other standard aggregates (min, max, sum, avg) won't be present in the response12:02
eglynninc0: ... so doesn't seem helpful to include in the displayed fields12:02
inc0hmm...I see. Ok, makes sense. How about removing standard aggregates from help and leave all the usual columns?12:04
inc0and if someone chooses to add non-usual aggregate like stddev just append this column?12:05
jd__eglynn: nothing I proposed was accepted12:05
jd__this summit is going to be boring12:05
*** yassine has joined #openstack-ceilometer12:05
eglynnjd__: yeah, they don't what they're missing ;)12:05
eglynndon't *know12:06
eglynninc0: would that leave potentially blank colums in the output?12:06
inc0hmm, it might12:07
inc0ok, I'll remove them12:08
eglynninc0: ... would something liek this work?12:08
eglynnfields = ['period', 'period_start', 'period_end']12:09
eglynnaggregate_fields = [a.func for a in aggregates]12:09
eglynnfields.append(aggregate_fields)12:09
eglynnfields.append(['duration', 'duration_start', 'duration_end'])12:09
inc0well, I'd do it a bit differently to maintain DRY12:09
inc0but right, I'll get the same outpu12:10
eglynninc0: ... fair nuffski ... of course with the above you'd need an if not aggregates: fields.append(['count', 'min', 'max', 'sum', 'avg'])12:10
inc0t12:10
eglynninc0: ... cool, thanks!12:10
inc0do we want to keep aggregates like sum, avg?12:10
inc0in client?12:11
eglynninc0: "keep" in what sense12:11
eglynninc0: ... I'm not sure what you mean12:11
inc0in help aggtegates and statistics12:11
inc0are very simular12:11
inc0I mean aggtegates is basically statistics with few additional fields12:11
inc0look at line :32 and :3312:12
eglynninc0: ah yes, I see ... so in my view, statistics are basically some collection of aggregates12:12
eglynninc0: ... those can be "standard aggregates", i.e. ['max', 'min', 'avg', 'sum', 'count'], given by default12:13
eglynninc0: ... of else "selectable aggregates", i.e. ['max', 'min', 'sum', 'avg', 'count', 'stddev', 'cardinality'], requested explicitly12:13
eglynn*or else12:13
*** matsuhashi has quit IRC12:14
eglynninc: so instead of STATISTICS and AGGREGATES, better to define say STANDARD_AGGREGATES and SELECTABLE_AGGREATES12:14
eglynn(on lines 32 & 33)12:14
*** yassine has quit IRC12:15
inc0and full list of aggregates should be standard.extend(selectable(12:16
*** matsuhas_ has joined #openstack-ceilometer12:17
*** yassine has joined #openstack-ceilometer12:17
inc0I can see statistics are used in line 316 in alarms12:17
inc0I'll check if you could use other aggregates there as well12:18
eglynninc0: yeah there's some cross-over between standard and selectable12:20
eglynninc0: ... in the sense that you get *all* the standard aggregates back by default if not aggregates explicitly requested12:20
eglynninc0: ... but if you elect to select explicitly, then you only get the standard aggregates and/or [stddev, cardinality] that you've explicitly asked for12:20
*** promulo has quit IRC12:24
*** _nadya__ has joined #openstack-ceilometer12:39
eglynninteresting blogpost on scaling issues encountered by CERN with the load imposed on nova-api by the ceilo compuet agent12:50
eglynn*compute12:50
eglynnhttp://openstack-in-production.blogspot.com.es/2014/03/cern-cloud-architecture-update-for.html12:50
eglynn... /me wonders do we really need to hit nova-api on every polling cycle?12:51
eglynn... could we call virt.inspector.inspect_instances first and only hit nova-api if a previously unknown instance is reported by the hypervisor?12:51
eglynn(... would result in stale resource metadata being metered, but not sure if that's the end of the world?)12:51
eglynn... /me feels a design summit proposal coming on12:53
eglynn... maybe a grab-bag of incremental scalability improvements12:53
openstackgerritA change was merged to openstack/ceilometer: Fix hardware pollster to inspect multiple resources  https://review.openstack.org/8213512:54
_nadya__yep...we already have several investigation about performance and summit is a good place to summarize it12:55
*** thomasem has joined #openstack-ceilometer12:55
_nadya__eglynn: did you have time to read tempest-ceilometer thread?12:56
eglynn_nadya__: I was travelling last week, so only catching up with email backlog now ... will read shortly though12:56
_nadya__eglynn: it's great that you've returned back :)12:57
eglynn_nadya__: yeah, I nearly got lost in some random pub in Brno ;)12:58
eglynn... luckily the Red Hat motto is: "never leave a man behind on the battlefield, or the pub" ;)12:58
_nadya__eglynn: hehe12:59
*** alexpilotti has quit IRC13:00
*** yassine has quit IRC13:08
*** gordc has joined #openstack-ceilometer13:10
ildikov__nadya__: hi. do you have a few seconds for me? :)13:10
*** matsuhas_ has quit IRC13:12
*** matsuhashi has joined #openstack-ceilometer13:13
_nadya__ildikov_: hi! yes13:14
*** Wangpan has quit IRC13:15
ildikov__nadya__: here is this bug: https://bugs.launchpad.net/ceilometer/+bug/129466313:16
ildikov__nadya__: I've had a really quick research on google last week to figure out if HBase supports orderby or not13:17
*** matsuhashi has quit IRC13:17
ildikov__nadya__: I found a sort_colums option13:18
*** julim has joined #openstack-ceilometer13:18
ildikov__nadya__: I'm not sure that it would be the one or not13:19
*** promulo has joined #openstack-ceilometer13:19
ildikov__nadya__: I do not have HBase, therefore I cannot try, so I thought that it is more effective, if I ask your opinion about this topic13:19
*** Wangpan has joined #openstack-ceilometer13:21
*** yassine has joined #openstack-ceilometer13:21
_nadya__ildikov_: HBase stores "keys" in alphabetic order. So if we want "ordering by" it's better to consruct keys in the needed way. I will take a look on this bug, ok? I suspect that from HBase we return samples in ASC order, need to check13:23
*** Wangpan has quit IRC13:24
*** nosnos has quit IRC13:24
*** Wangpan has joined #openstack-ceilometer13:24
*** nosnos has joined #openstack-ceilometer13:25
ildikov__nadya__: IIRC, it returns the samples in the order of recording13:25
_nadya__ildikov_: for each sample we use it's timestamp in key_name, so it should be considered during processing13:25
_nadya__ildikov_: yes, it is. and how it should be? we should consider time of sample-creation, right? not recording?13:26
ildikov__nadya__: so the current order is ascending13:28
*** Wangpan has quit IRC13:28
ildikov__nadya__: the other drivers return the samples in descending order based on the timestamp13:29
*** nosnos has quit IRC13:29
_nadya__ildikov_: I will assign it on myself, ok?13:29
ildikov__nadya__: sure, you have more experience, than I do, that is why I wanted to ask your opinion about this13:30
_nadya__ildikov_: it's related to key-names format. I's better to avoid any additional sorting13:31
ildikov__nadya__: we can also mark this as will be implemented later, or something like this, I do not know how much trouble we would have with maknig this backward compatible13:32
ildikov__nadya__: I just tried to find a documented support for this, if there is any other, than the key-names format13:32
_nadya__ildikov_: I can't imagine how it may be backward compatible :)13:34
ildikov__nadya__: but if there is no dynamic support, then maybe it is good to once have the default order by synchronized with the other drivers13:34
ildikov__nadya__: we have some migration code for other drivers too13:34
*** Wangpan has joined #openstack-ceilometer13:35
ildikov__nadya__: I have no better idea13:35
ildikov__nadya__: somehow it should be handled that before this change it was ordered in ascending and after it in descending order or maybe it's still Monday so I should not think about HBase bugs, when I do not deeply know HBase :)13:36
_nadya__ildikov_: hm...I don't think we should think about this stuff regarding HBase. First of all, it is not usable now even in testing. We are working on this but have a lot of issues in happybase library. So it's better to put more efforts to make it work rather to make it compatible with "broken" version13:38
ildikov__nadya__: cool, that is exactly, why I wanted to ask your opinion, because I was not 100% sure, how much it is working currently13:39
ildikov__nadya__: so from this PoV, I agree that it should be fixed now, at the beginning of the HBase story13:40
_nadya__eglynn: ildikov_: jd__: sileht: gordc: JFYI we have a critical bug https://bugs.launchpad.net/ceilometer/+bug/1294789 . I want to make it swift's one. If you have time, please take a look on comments13:41
_nadya__ildikov_: ok, I think we have a common vision now13:42
*** Wangpan has quit IRC13:42
ildikov__nadya__: yes, thanks for your time13:42
*** Wangpan has joined #openstack-ceilometer13:50
*** nosnos has joined #openstack-ceilometer13:58
*** _nadya__ has quit IRC13:59
*** matsuhashi has joined #openstack-ceilometer13:59
*** nosnos has quit IRC14:00
*** nosnos has joined #openstack-ceilometer14:01
*** nosnos has quit IRC14:05
*** nosnos has joined #openstack-ceilometer14:11
*** nosnos has quit IRC14:14
*** matsuhashi has quit IRC14:16
*** rwsu has joined #openstack-ceilometer14:17
openstackgerritMichal Jastrzebski proposed a change to openstack/python-ceilometerclient: Implementation of statistics aggregators  https://review.openstack.org/8049914:18
*** jpich has joined #openstack-ceilometer14:22
*** yassine has quit IRC14:25
jpicheglynn: Hello o/ Could you or one of the Ceilometer folks help us to clarify a Ceilometer string for the translators, if you have a chance? -> see https://bugs.launchpad.net/horizon/+bug/129293714:26
openstackgerritSt├ęphane Albert proposed a change to openstack/python-ceilometerclient: Statistics groupby handling improvement  https://review.openstack.org/8025614:26
eglynnjpich: looking now14:26
jpicheglynn: Thank you!14:27
*** anamalagon has quit IRC14:28
*** matsuhashi has joined #openstack-ceilometer14:32
*** yassine has joined #openstack-ceilometer14:37
eglynnjpich: my attempt to clarify ... https://bugs.launchpad.net/horizon/+bug/1292937/comments/514:37
eglynnjpich: s/Period/Periodic/14:39
*** saju_m has quit IRC14:39
jpicheglynn: Thank you! I missed your s// comment and was scratching my head :-) I think that should be helpful to the translators, thank you14:42
eglynnjpich: np! (... /me rarely types anything without fat-fingering at least once)14:43
jpicheglynn: A variant of https://en.wikipedia.org/wiki/Muphry%27s_law here :-)14:47
eglynnjpich: LOL :) ... I love it, "The Iron Law of Nitpicking"14:48
*** terriyu has joined #openstack-ceilometer14:50
*** admin0 has quit IRC15:03
*** admin0 has joined #openstack-ceilometer15:04
*** shadower has joined #openstack-ceilometer15:04
*** dhellmann_ is now known as dhellmann15:05
*** jmckind has joined #openstack-ceilometer15:06
*** prad has joined #openstack-ceilometer15:20
*** prad has left #openstack-ceilometer15:21
*** inc0 has quit IRC15:21
*** prad has joined #openstack-ceilometer15:21
*** inc0 has joined #openstack-ceilometer15:22
*** matsuhashi has quit IRC15:25
*** nati_ueno has joined #openstack-ceilometer15:26
openstackgerritMichal Jastrzebski proposed a change to openstack/python-ceilometerclient: Implementation of statistics aggregators  https://review.openstack.org/8049915:27
*** matsuhas_ has joined #openstack-ceilometer15:28
*** packet has joined #openstack-ceilometer15:29
openstackgerritNadya Privalova proposed a change to openstack/ceilometer: Hardcode the logging level in swift_middleware  https://review.openstack.org/8252615:31
*** matsuhas_ has quit IRC15:33
openstackgerritA change was merged to openstack/ceilometer: Install global lazy _()  https://review.openstack.org/7082415:33
*** inc0 has quit IRC15:39
openstackgerritSt├ęphane Albert proposed a change to openstack/python-ceilometerclient: Statistics groupby handling improvement  https://review.openstack.org/8025615:39
*** inc0 has joined #openstack-ceilometer15:42
openstackgerritgordon chung proposed a change to openstack/ceilometer: fix create_or_update logic to avoid rollbacks  https://review.openstack.org/8196915:43
*** _cjones_ has joined #openstack-ceilometer15:47
*** Ruetobas has quit IRC16:01
*** Ruetobas has joined #openstack-ceilometer16:03
*** Ruetobas has quit IRC16:07
*** Ruetobas has joined #openstack-ceilometer16:11
*** mihgen has quit IRC16:18
*** sdake_ has joined #openstack-ceilometer16:18
*** sdake_ has joined #openstack-ceilometer16:18
*** nati_ueno has quit IRC16:35
openstackgerritIldiko Vancsa proposed a change to openstack/ceilometer: Fix validation error for invalid field name in simple query  https://review.openstack.org/8211016:43
*** packet has quit IRC16:46
*** shakayumi has joined #openstack-ceilometer16:51
*** shakayumi has quit IRC16:51
*** prad has quit IRC16:52
*** jpich has quit IRC16:53
*** admin0 has quit IRC16:58
*** shakayumi has joined #openstack-ceilometer17:00
*** shakayumi has quit IRC17:01
*** inc0 has quit IRC17:02
*** inc0 has joined #openstack-ceilometer17:08
*** sayalilunkad has joined #openstack-ceilometer17:09
*** prad has joined #openstack-ceilometer17:14
*** admin0 has joined #openstack-ceilometer17:15
*** inc0 has quit IRC17:15
*** nati_ueno has joined #openstack-ceilometer17:17
*** yassine has quit IRC17:21
*** prad has quit IRC17:21
*** eglynn has quit IRC17:23
*** mihgen has joined #openstack-ceilometer17:24
*** prad has joined #openstack-ceilometer17:24
*** nacim has quit IRC17:26
*** ildikov_ has quit IRC17:29
*** bada has joined #openstack-ceilometer17:29
*** leakypipes is now known as jaypipes17:33
*** prad has quit IRC17:34
*** admin0 has quit IRC17:38
*** _nadya_ has joined #openstack-ceilometer17:40
*** prad has joined #openstack-ceilometer17:45
*** piyushmasrani has quit IRC17:46
*** _nadya_ has quit IRC17:59
*** terriyu has quit IRC17:59
*** jmckind has quit IRC18:09
*** jmckind has joined #openstack-ceilometer18:10
*** creiht has joined #openstack-ceilometer18:11
creihthowdy18:11
creihtI'm trying to better understand the issues around: https://bugs.launchpad.net/ceilometer/+bug/1294789?comments=all18:12
*** zul has quit IRC18:13
creihtanyone available to discuss how we can make the logging with swift better?18:13
*** zul has joined #openstack-ceilometer18:16
*** sayalilunkad has quit IRC18:19
*** prad has quit IRC18:20
gordccreiht: go for it. :)18:25
creihtgordc: well I'm having difficulty groking what is the root of the issue18:26
gordccreiht: as i understand it, swift logging level is by default set to debug.18:27
creihtwhat exactly about using the ceilometer middleware with swift makes ceilometer log extra stuff18:27
creihtgordc: if it isn't set in the config, yes it defaults to the debug18:27
gordccreiht: i wouldn't say it extra stuff. it's just stuff unrelated to swift (i'm not sure the bug is CRITICAL or logs 'useless' info, but that's debatable).18:28
*** prad has joined #openstack-ceilometer18:28
creihthah18:28
*** mihgen has quit IRC18:29
creihtgordc: I guess that's why I'm having trouble figuring out what the real issue is18:29
creihtis the real issue more that the way they have swift setup for devstack so that it is logging to both console and syslog?18:30
gordccreiht: iiuc, they just don't want ceilometer logs apart of swift logs... so i think setting ceilometer loglevel to a higher level than debug will hide our debug level logs from swift logs.18:30
creihtgordc: If that is an acceptable solution, I'm cool with that as well, just seems like debug info might be good for tracking down issues18:30
openstackgerritPeter Portante proposed a change to openstack/ceilometer: Use swob instead of webob in swift unit tests  https://review.openstack.org/8257218:31
gordccreiht: i'm not sure if they have an issue with swift logs in syslog. the bug just mentions the ceilometer logs are 'useless'18:31
creihtheh18:31
creihtwell I just want to make sure we have a constructive conversation rather than, oh "if swift used oslo.log we wouldn't have any issues"18:31
gordccreiht: to each their own :) i guess it doesn't make sense to have ceilometer logs flooding swift logs.18:32
creihtgordc: but if there is anything that I can do to help, let me know18:32
gordccreiht: ah, yeah i don't think the oslo.log stuff is relevant, i'm pretty sure it'll happen regardless.18:32
creihtright18:32
gordccreiht: cool cool. thanks for following up.18:32
creihtgordc: do you see though how the level should be able to be set in the config without having to make any changes?18:34
creihtand not have to hard code it?18:34
openstackgerritPeter Portante proposed a change to openstack/ceilometer: Use swob instead of webob in swift unit tests  https://review.openstack.org/8257218:35
gordccreiht: here? https://github.com/openstack/swift/blob/master/etc/proxy-server.conf-sample#L4118:35
creihtgordc: well that sets it for the proxy server18:36
creihtyou should also be able to set the same variable in your middleware section18:36
creihtand if that is the config that is read in and set to swift.common.utils.get_logger18:36
creihtit will set that level18:36
gordccreiht: like this: https://review.openstack.org/#/c/82526/1/ceilometer/objectstore/swift_middleware.py? or something different?18:37
gordcthe above would be reading in the conf file and changing the log_level to another level (just for ceilometer middleware)18:38
creihtright, but what I'm saying is that you should be able to actually set it in the conf and not have to change th code18:38
creihtjust sec18:38
creihtlet me paste something18:38
*** _nadya_ has joined #openstack-ceilometer18:39
gordccreiht: but if we change the conf file, it'll affect swift as well no?18:39
creihtgordc: well the swift conf has a section specifically for the middleware right?18:39
openstackgerritPeter Portante proposed a change to openstack/ceilometer: Use swob instead of webob in swift unit tests  https://review.openstack.org/8257218:40
* creiht is trying to find an example18:41
gordccreiht: i'm proabably to focussed on a single solution... you mean passing in parameters to the filter? like this: https://github.com/openstack/swift/blob/master/etc/proxy-server.conf-sample#L204?18:41
creihtgordc: http://paste.openstack.org/show/74175/18:44
creihtin the swift proxy-server.conf, there is a section that looks like that18:44
creihtthat's the paste.deploy config18:45
creihtadding the log_level there *should* in theory pass on through to the swift get_logger func18:45
gordccreiht: cool. yeah, that's what i told nprivalova to add. didn't realise it by default would override the one defined in proxy-server.conf18:46
creihtright18:46
creihtit is supposed to work that way18:46
creihtif it doesn't, let me know and we can work through it18:46
gordccreiht: i guess no code change is needed... just need to update devstack then.18:46
creihtyeah18:47
gordccreiht: i'll comment on patch... thanks for clarifying18:47
creihtnp18:47
creihtI would try testing it first to be sure it works :)18:47
creihtbut going that route still also allows the deployer to configure what level they want18:47
gordci'll give it a try. you can probably mark invalid for the swift portion of bug.18:48
creihtk18:49
*** prad has left #openstack-ceilometer18:51
creihtgordc: I'll hang out here a bit longer if you have any questions18:52
creihtfor future questions/issues it is pretty easy to talk to anyone in #openstack-swift18:52
gordccreiht: cool cool.18:52
gordccreiht: i guess as a side topic, is there an easy way to restart swift service in devstack? ctrl+c doesn't seem to work... i keep having to kill process manually.18:53
creihthrm18:54
creihtIt has been a while since I messed with devstack, but ctrl+c *should* work18:54
creihtdid you try hitting it again?18:54
gordccreiht: yeah, i hammer away at it. :)18:56
creihtheheh18:56
gordccreiht: maybe a new bug in devstack... i do remember it was fixed at one point18:56
creihtI guess it is about time to spin up a devstack instance again18:56
gordccreiht: ok. give it a try if you have time. it might just be my environment acting up.18:57
creihtgordc: I may have to set one up anyways to look how at how logging is setup18:58
creihtsince they also seem to be complaining about the double logging19:00
*** _nadya_ has quit IRC19:06
*** llu has quit IRC19:14
*** llu has joined #openstack-ceilometer19:15
*** ildikov_ has joined #openstack-ceilometer20:09
*** prad has joined #openstack-ceilometer20:11
*** prad has quit IRC20:18
*** prad has joined #openstack-ceilometer20:21
*** prad has quit IRC20:28
*** prad has joined #openstack-ceilometer20:31
*** ddutta has joined #openstack-ceilometer20:32
*** ddutta has quit IRC20:42
*** ddutta has joined #openstack-ceilometer20:43
*** luis_ has joined #openstack-ceilometer20:57
*** luisg has quit IRC20:59
*** bogdando has quit IRC21:06
*** bogdando has joined #openstack-ceilometer21:08
*** luis_ has quit IRC21:11
*** ruhe has left #openstack-ceilometer21:11
*** julim has quit IRC21:35
*** nati_ueno has quit IRC21:47
*** nati_ueno has joined #openstack-ceilometer21:48
*** thomasem has quit IRC21:50
*** alexpilotti has joined #openstack-ceilometer21:54
*** promulo has quit IRC22:02
*** ddutta has quit IRC22:09
*** promulo has joined #openstack-ceilometer22:31
*** prad has quit IRC22:35
*** jmckind has quit IRC22:42
*** gordc has quit IRC22:54
*** promulo has quit IRC23:04
*** ddutta has joined #openstack-ceilometer23:05
openstackgerritJoe Gordon proposed a change to openstack/ceilometer: Disable oslo.messaging and oslo-incubator debug logs  https://review.openstack.org/8267123:35
openstackgerritEoghan Glynn proposed a change to openstack/ceilometer: Ensure idempotency of cardinality reduction in mongo  https://review.openstack.org/8267223:40
*** promulo has joined #openstack-ceilometer23:45

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