| *** matsuhashi has joined #openstack-ceilometer | 00:25 | |
| *** liusheng has quit IRC | 01:13 | |
| *** nosnos has joined #openstack-ceilometer | 01:36 | |
| *** flwang has quit IRC | 01:39 | |
| openstackgerrit | liusheng proposed a change to openstack/python-ceilometerclient: Reduce redundant parameter of some commands in CLI https://review.openstack.org/66776 | 01:52 |
|---|---|---|
| *** liusheng has joined #openstack-ceilometer | 01:53 | |
| *** xianghui has joined #openstack-ceilometer | 02:02 | |
| *** xianghui has quit IRC | 02:08 | |
| *** xianghui has joined #openstack-ceilometer | 02:10 | |
| *** flwang has joined #openstack-ceilometer | 02:18 | |
| *** matsuhashi has quit IRC | 02:55 | |
| *** matsuhashi has joined #openstack-ceilometer | 03:21 | |
| *** nati_ueno has joined #openstack-ceilometer | 03:23 | |
| *** ekarlso has quit IRC | 03:48 | |
| *** ekarlso has joined #openstack-ceilometer | 03:48 | |
| *** terriyu has joined #openstack-ceilometer | 04:00 | |
| *** terriyu has quit IRC | 04:01 | |
| *** terriyu has joined #openstack-ceilometer | 04:02 | |
| *** Akshik has joined #openstack-ceilometer | 04:21 | |
| *** terriyu has quit IRC | 04:35 | |
| *** nati_ueno has quit IRC | 04:46 | |
| *** nati_ueno has joined #openstack-ceilometer | 04:47 | |
| *** nati_ueno has quit IRC | 04:51 | |
| *** nati_ueno has joined #openstack-ceilometer | 04:52 | |
| *** nosnos_ has joined #openstack-ceilometer | 05:16 | |
| *** nosnos has quit IRC | 05:19 | |
| *** Akshik has quit IRC | 05:20 | |
| *** Akshik has joined #openstack-ceilometer | 05:20 | |
| *** Akshik_ has joined #openstack-ceilometer | 05:25 | |
| *** Akshik has quit IRC | 05:26 | |
| *** Akshik_ has quit IRC | 05:33 | |
| *** Akshik has joined #openstack-ceilometer | 05:35 | |
| *** Akshik has quit IRC | 05:41 | |
| *** Akshik has joined #openstack-ceilometer | 05:46 | |
| *** sdake_ has quit IRC | 05:47 | |
| *** Akshik has quit IRC | 05:53 | |
| *** Akshik_ has joined #openstack-ceilometer | 05:54 | |
| *** Akshik_ has quit IRC | 05:57 | |
| *** Akshik_ has joined #openstack-ceilometer | 05:57 | |
| *** mihgen has joined #openstack-ceilometer | 06:00 | |
| *** matsuhas_ has joined #openstack-ceilometer | 06:00 | |
| *** matsuhashi has quit IRC | 06:01 | |
| *** flwang has quit IRC | 06:01 | |
| openstackgerrit | Jenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/76762 | 06:05 |
| *** flwang has joined #openstack-ceilometer | 06:09 | |
| *** nati_ueno has quit IRC | 06:14 | |
| openstackgerrit | liusheng proposed a change to openstack/ceilometer: Avoid query samples before time of earliest sample https://review.openstack.org/82048 | 06:16 |
| *** nosnos has joined #openstack-ceilometer | 06:19 | |
| *** nosnos_ has quit IRC | 06:22 | |
| *** ildikov_ has quit IRC | 06:26 | |
| *** matsuhas_ has quit IRC | 06:31 | |
| *** matsuhashi has joined #openstack-ceilometer | 06:31 | |
| openstackgerrit | liusheng proposed a change to openstack/ceilometer: Avoid query samples before time of earliest sample https://review.openstack.org/82048 | 06:32 |
| *** AMike has joined #openstack-ceilometer | 06:34 | |
| *** AMike has joined #openstack-ceilometer | 06:34 | |
| *** saju_m has joined #openstack-ceilometer | 06:35 | |
| *** saju_m has quit IRC | 06:59 | |
| *** saju_m has joined #openstack-ceilometer | 07:12 | |
| *** ildikov_ has joined #openstack-ceilometer | 07:13 | |
| *** saju_m has quit IRC | 07:17 | |
| *** saju_m has joined #openstack-ceilometer | 07:17 | |
| *** mihgen_ has joined #openstack-ceilometer | 07:18 | |
| *** Akshik__ has joined #openstack-ceilometer | 07:19 | |
| *** Ruetobas has quit IRC | 07:19 | |
| *** nprivalova has quit IRC | 07:19 | |
| *** AMike has quit IRC | 07:19 | |
| *** Akshik_ has quit IRC | 07:19 | |
| *** mihgen has quit IRC | 07:20 | |
| *** liusheng has quit IRC | 07:20 | |
| *** mihgen_ is now known as mihgen | 07:20 | |
| *** liusheng has joined #openstack-ceilometer | 07:20 | |
| *** saju_m has quit IRC | 07:22 | |
| *** Akshik__ has quit IRC | 07:23 | |
| *** Ruetobas has joined #openstack-ceilometer | 07:23 | |
| *** saju_m has joined #openstack-ceilometer | 07:24 | |
| *** Akshik has joined #openstack-ceilometer | 07:24 | |
| *** flwang has quit IRC | 07:28 | |
| *** saju_m has quit IRC | 07:30 | |
| *** saju_m has joined #openstack-ceilometer | 07:31 | |
| *** AMike has joined #openstack-ceilometer | 07:32 | |
| *** AMike has joined #openstack-ceilometer | 07:32 | |
| *** flwang has joined #openstack-ceilometer | 07:33 | |
| *** nprivalova has joined #openstack-ceilometer | 07:33 | |
| *** Akshik has quit IRC | 07:36 | |
| *** AMike has quit IRC | 07:36 | |
| *** Akshik has joined #openstack-ceilometer | 07:56 | |
| *** bogdando has joined #openstack-ceilometer | 08:26 | |
| *** Akshik has quit IRC | 08:36 | |
| *** renlt has joined #openstack-ceilometer | 08:42 | |
| *** Akshik has joined #openstack-ceilometer | 08:48 | |
| *** mihgen has quit IRC | 08:50 | |
| *** AMike has joined #openstack-ceilometer | 08:52 | |
| *** AMike has joined #openstack-ceilometer | 08:52 | |
| *** AMike has quit IRC | 08:52 | |
| *** ilyashakhat has quit IRC | 08:55 | |
| *** nacim has joined #openstack-ceilometer | 09:04 | |
| *** yassine has joined #openstack-ceilometer | 09:05 | |
| *** flwang has quit IRC | 09:20 | |
| *** ilyashakhat has joined #openstack-ceilometer | 09:20 | |
| *** liusheng has quit IRC | 09:21 | |
| *** liusheng has joined #openstack-ceilometer | 09:21 | |
| openstackgerrit | Michal Jastrzebski proposed a change to openstack/python-ceilometerclient: Implementation of statistics aggregators https://review.openstack.org/80499 | 09:26 |
| *** nacim has quit IRC | 09:28 | |
| *** inc0 has joined #openstack-ceilometer | 09:28 | |
| *** mihgen has joined #openstack-ceilometer | 09:29 | |
| *** Akshik has quit IRC | 09:30 | |
| *** renlt has quit IRC | 09:35 | |
| *** nacim has joined #openstack-ceilometer | 09:36 | |
| *** admin0 has joined #openstack-ceilometer | 09:39 | |
| admin0 | hey guys | 09:40 |
| *** AMike has joined #openstack-ceilometer | 09:42 | |
| *** AMike has joined #openstack-ceilometer | 09:42 | |
| inc0 | hello | 09:42 |
| *** flwang has joined #openstack-ceilometer | 10:14 | |
| *** Akshik has joined #openstack-ceilometer | 10:16 | |
| *** Akshik has quit IRC | 10:23 | |
| *** Akshik has joined #openstack-ceilometer | 10:24 | |
| *** matsuhashi has quit IRC | 10:33 | |
| *** matsuhas_ has joined #openstack-ceilometer | 10:35 | |
| *** eglynn has joined #openstack-ceilometer | 10:49 | |
| *** yassine has quit IRC | 10:52 | |
| eglynn | jd__: boo! :( ... "Unfortunately, your submission was not chosen to be part of the official agenda this time around ... Quo vadis Ceilometer" | 11:04 |
| eglynn | jd__: ... better luck with your other ceilo proposal? | 11:04 |
| *** matsuhas_ has quit IRC | 11:07 | |
| *** admin0 has quit IRC | 11:20 | |
| *** admin0 has joined #openstack-ceilometer | 11:21 | |
| *** xianghui has quit IRC | 11:22 | |
| *** Akshik has quit IRC | 11:22 | |
| *** Akshik_ has joined #openstack-ceilometer | 11:23 | |
| *** nosnos has quit IRC | 11:23 | |
| *** Akshik_ has quit IRC | 11:28 | |
| openstackgerrit | Uros Jovanovic proposed a change to openstack/python-ceilometerclient: Adds alarm time constraint support to ceilometer CLI https://review.openstack.org/78356 | 11:38 |
| openstackgerrit | Jenkins proposed a change to openstack/ceilometer: Updated from global requirements https://review.openstack.org/77639 | 11:47 |
| *** matsuhashi has joined #openstack-ceilometer | 11:53 | |
| *** nosnos has joined #openstack-ceilometer | 11:54 | |
| openstackgerrit | Jenkins proposed a change to openstack/pycadf: Updated from global requirements https://review.openstack.org/79693 | 11:54 |
| inc0 | eglynn, 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 |
| inc0 | I mean line :48 | 11:58 |
| eglynn | inc0: the suggestion was just to change the format of the --aggregate option, not the output | 11:59 |
| eglynn | inc0: ... so basically --aggregate 'cardinality<-resource_id' instead of --aggregate 'cardinality::resource_id' as '::' is already used by the CLI to mean something else | 11:59 |
| inc0 | ah, sorry, I asked for line 77, yes, I've fixed :48 | 12:00 |
| inc0 | and now I see what you meant in :77 | 12:00 |
| eglynn | inc0: a-ha, yes ... so in the case where I call say: ceilometer statistics --aggregate count --aggregate 'cardinality<-resource_id' ... | 12:01 |
| eglynn | inc0: ... the other standard aggregates (min, max, sum, avg) won't be present in the response | 12:02 |
| eglynn | inc0: ... so doesn't seem helpful to include in the displayed fields | 12:02 |
| inc0 | hmm...I see. Ok, makes sense. How about removing standard aggregates from help and leave all the usual columns? | 12:04 |
| inc0 | and if someone chooses to add non-usual aggregate like stddev just append this column? | 12:05 |
| jd__ | eglynn: nothing I proposed was accepted | 12:05 |
| jd__ | this summit is going to be boring | 12:05 |
| *** yassine has joined #openstack-ceilometer | 12:05 | |
| eglynn | jd__: yeah, they don't what they're missing ;) | 12:05 |
| eglynn | don't *know | 12:06 |
| eglynn | inc0: would that leave potentially blank colums in the output? | 12:06 |
| inc0 | hmm, it might | 12:07 |
| inc0 | ok, I'll remove them | 12:08 |
| eglynn | inc0: ... would something liek this work? | 12:08 |
| eglynn | fields = ['period', 'period_start', 'period_end'] | 12:09 |
| eglynn | aggregate_fields = [a.func for a in aggregates] | 12:09 |
| eglynn | fields.append(aggregate_fields) | 12:09 |
| eglynn | fields.append(['duration', 'duration_start', 'duration_end']) | 12:09 |
| inc0 | well, I'd do it a bit differently to maintain DRY | 12:09 |
| inc0 | but right, I'll get the same outpu | 12:10 |
| eglynn | inc0: ... fair nuffski ... of course with the above you'd need an if not aggregates: fields.append(['count', 'min', 'max', 'sum', 'avg']) | 12:10 |
| inc0 | t | 12:10 |
| eglynn | inc0: ... cool, thanks! | 12:10 |
| inc0 | do we want to keep aggregates like sum, avg? | 12:10 |
| inc0 | in client? | 12:11 |
| eglynn | inc0: "keep" in what sense | 12:11 |
| eglynn | inc0: ... I'm not sure what you mean | 12:11 |
| inc0 | in help aggtegates and statistics | 12:11 |
| inc0 | are very simular | 12:11 |
| inc0 | I mean aggtegates is basically statistics with few additional fields | 12:11 |
| inc0 | look at line :32 and :33 | 12:12 |
| eglynn | inc0: ah yes, I see ... so in my view, statistics are basically some collection of aggregates | 12:12 |
| eglynn | inc0: ... those can be "standard aggregates", i.e. ['max', 'min', 'avg', 'sum', 'count'], given by default | 12:13 |
| eglynn | inc0: ... of else "selectable aggregates", i.e. ['max', 'min', 'sum', 'avg', 'count', 'stddev', 'cardinality'], requested explicitly | 12:13 |
| eglynn | *or else | 12:13 |
| *** matsuhashi has quit IRC | 12:14 | |
| eglynn | inc: so instead of STATISTICS and AGGREGATES, better to define say STANDARD_AGGREGATES and SELECTABLE_AGGREATES | 12:14 |
| eglynn | (on lines 32 & 33) | 12:14 |
| *** yassine has quit IRC | 12:15 | |
| inc0 | and full list of aggregates should be standard.extend(selectable( | 12:16 |
| *** matsuhas_ has joined #openstack-ceilometer | 12:17 | |
| *** yassine has joined #openstack-ceilometer | 12:17 | |
| inc0 | I can see statistics are used in line 316 in alarms | 12:17 |
| inc0 | I'll check if you could use other aggregates there as well | 12:18 |
| eglynn | inc0: yeah there's some cross-over between standard and selectable | 12:20 |
| eglynn | inc0: ... in the sense that you get *all* the standard aggregates back by default if not aggregates explicitly requested | 12:20 |
| eglynn | inc0: ... but if you elect to select explicitly, then you only get the standard aggregates and/or [stddev, cardinality] that you've explicitly asked for | 12:20 |
| *** promulo has quit IRC | 12:24 | |
| *** _nadya__ has joined #openstack-ceilometer | 12:39 | |
| eglynn | interesting blogpost on scaling issues encountered by CERN with the load imposed on nova-api by the ceilo compuet agent | 12:50 |
| eglynn | *compute | 12:50 |
| eglynn | http://openstack-in-production.blogspot.com.es/2014/03/cern-cloud-architecture-update-for.html | 12: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 on | 12:53 |
| eglynn | ... maybe a grab-bag of incremental scalability improvements | 12:53 |
| openstackgerrit | A change was merged to openstack/ceilometer: Fix hardware pollster to inspect multiple resources https://review.openstack.org/82135 | 12:54 |
| _nadya__ | yep...we already have several investigation about performance and summit is a good place to summarize it | 12:55 |
| *** thomasem has joined #openstack-ceilometer | 12: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 though | 12: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: hehe | 12:59 |
| *** alexpilotti has quit IRC | 13:00 | |
| *** yassine has quit IRC | 13:08 | |
| *** gordc has joined #openstack-ceilometer | 13:10 | |
| ildikov_ | _nadya__: hi. do you have a few seconds for me? :) | 13:10 |
| *** matsuhas_ has quit IRC | 13:12 | |
| *** matsuhashi has joined #openstack-ceilometer | 13:13 | |
| _nadya__ | ildikov_: hi! yes | 13:14 |
| *** Wangpan has quit IRC | 13:15 | |
| ildikov_ | _nadya__: here is this bug: https://bugs.launchpad.net/ceilometer/+bug/1294663 | 13:16 |
| ildikov_ | _nadya__: I've had a really quick research on google last week to figure out if HBase supports orderby or not | 13:17 |
| *** matsuhashi has quit IRC | 13:17 | |
| ildikov_ | _nadya__: I found a sort_colums option | 13:18 |
| *** julim has joined #openstack-ceilometer | 13:18 | |
| ildikov_ | _nadya__: I'm not sure that it would be the one or not | 13:19 |
| *** promulo has joined #openstack-ceilometer | 13: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 topic | 13:19 |
| *** Wangpan has joined #openstack-ceilometer | 13:21 | |
| *** yassine has joined #openstack-ceilometer | 13: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 check | 13:23 |
| *** Wangpan has quit IRC | 13:24 | |
| *** nosnos has quit IRC | 13:24 | |
| *** Wangpan has joined #openstack-ceilometer | 13:24 | |
| *** nosnos has joined #openstack-ceilometer | 13:25 | |
| ildikov_ | _nadya__: IIRC, it returns the samples in the order of recording | 13:25 |
| _nadya__ | ildikov_: for each sample we use it's timestamp in key_name, so it should be considered during processing | 13: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 ascending | 13:28 |
| *** Wangpan has quit IRC | 13:28 | |
| ildikov_ | _nadya__: the other drivers return the samples in descending order based on the timestamp | 13:29 |
| *** nosnos has quit IRC | 13: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 this | 13:30 |
| _nadya__ | ildikov_: it's related to key-names format. I's better to avoid any additional sorting | 13: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 compatible | 13:32 |
| ildikov_ | _nadya__: I just tried to find a documented support for this, if there is any other, than the key-names format | 13: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 drivers | 13:34 |
| ildikov_ | _nadya__: we have some migration code for other drivers too | 13:34 |
| *** Wangpan has joined #openstack-ceilometer | 13:35 | |
| ildikov_ | _nadya__: I have no better idea | 13: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" version | 13: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 currently | 13:39 |
| ildikov_ | _nadya__: so from this PoV, I agree that it should be fixed now, at the beginning of the HBase story | 13: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 comments | 13:41 |
| _nadya__ | ildikov_: ok, I think we have a common vision now | 13:42 |
| *** Wangpan has quit IRC | 13:42 | |
| ildikov_ | _nadya__: yes, thanks for your time | 13:42 |
| *** Wangpan has joined #openstack-ceilometer | 13:50 | |
| *** nosnos has joined #openstack-ceilometer | 13:58 | |
| *** _nadya__ has quit IRC | 13:59 | |
| *** matsuhashi has joined #openstack-ceilometer | 13:59 | |
| *** nosnos has quit IRC | 14:00 | |
| *** nosnos has joined #openstack-ceilometer | 14:01 | |
| *** nosnos has quit IRC | 14:05 | |
| *** nosnos has joined #openstack-ceilometer | 14:11 | |
| *** nosnos has quit IRC | 14:14 | |
| *** matsuhashi has quit IRC | 14:16 | |
| *** rwsu has joined #openstack-ceilometer | 14:17 | |
| openstackgerrit | Michal Jastrzebski proposed a change to openstack/python-ceilometerclient: Implementation of statistics aggregators https://review.openstack.org/80499 | 14:18 |
| *** jpich has joined #openstack-ceilometer | 14:22 | |
| *** yassine has quit IRC | 14:25 | |
| jpich | eglynn: 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/1292937 | 14:26 |
| openstackgerrit | Stéphane Albert proposed a change to openstack/python-ceilometerclient: Statistics groupby handling improvement https://review.openstack.org/80256 | 14:26 |
| eglynn | jpich: looking now | 14:26 |
| jpich | eglynn: Thank you! | 14:27 |
| *** anamalagon has quit IRC | 14:28 | |
| *** matsuhashi has joined #openstack-ceilometer | 14:32 | |
| *** yassine has joined #openstack-ceilometer | 14:37 | |
| eglynn | jpich: my attempt to clarify ... https://bugs.launchpad.net/horizon/+bug/1292937/comments/5 | 14:37 |
| eglynn | jpich: s/Period/Periodic/ | 14:39 |
| *** saju_m has quit IRC | 14:39 | |
| jpich | eglynn: Thank you! I missed your s// comment and was scratching my head :-) I think that should be helpful to the translators, thank you | 14:42 |
| eglynn | jpich: np! (... /me rarely types anything without fat-fingering at least once) | 14:43 |
| jpich | eglynn: A variant of https://en.wikipedia.org/wiki/Muphry%27s_law here :-) | 14:47 |
| eglynn | jpich: LOL :) ... I love it, "The Iron Law of Nitpicking" | 14:48 |
| *** terriyu has joined #openstack-ceilometer | 14:50 | |
| *** admin0 has quit IRC | 15:03 | |
| *** admin0 has joined #openstack-ceilometer | 15:04 | |
| *** shadower has joined #openstack-ceilometer | 15:04 | |
| *** dhellmann_ is now known as dhellmann | 15:05 | |
| *** jmckind has joined #openstack-ceilometer | 15:06 | |
| *** prad has joined #openstack-ceilometer | 15:20 | |
| *** prad has left #openstack-ceilometer | 15:21 | |
| *** inc0 has quit IRC | 15:21 | |
| *** prad has joined #openstack-ceilometer | 15:21 | |
| *** inc0 has joined #openstack-ceilometer | 15:22 | |
| *** matsuhashi has quit IRC | 15:25 | |
| *** nati_ueno has joined #openstack-ceilometer | 15:26 | |
| openstackgerrit | Michal Jastrzebski proposed a change to openstack/python-ceilometerclient: Implementation of statistics aggregators https://review.openstack.org/80499 | 15:27 |
| *** matsuhas_ has joined #openstack-ceilometer | 15:28 | |
| *** packet has joined #openstack-ceilometer | 15:29 | |
| openstackgerrit | Nadya Privalova proposed a change to openstack/ceilometer: Hardcode the logging level in swift_middleware https://review.openstack.org/82526 | 15:31 |
| *** matsuhas_ has quit IRC | 15:33 | |
| openstackgerrit | A change was merged to openstack/ceilometer: Install global lazy _() https://review.openstack.org/70824 | 15:33 |
| *** inc0 has quit IRC | 15:39 | |
| openstackgerrit | Stéphane Albert proposed a change to openstack/python-ceilometerclient: Statistics groupby handling improvement https://review.openstack.org/80256 | 15:39 |
| *** inc0 has joined #openstack-ceilometer | 15:42 | |
| openstackgerrit | gordon chung proposed a change to openstack/ceilometer: fix create_or_update logic to avoid rollbacks https://review.openstack.org/81969 | 15:43 |
| *** _cjones_ has joined #openstack-ceilometer | 15:47 | |
| *** Ruetobas has quit IRC | 16:01 | |
| *** Ruetobas has joined #openstack-ceilometer | 16:03 | |
| *** Ruetobas has quit IRC | 16:07 | |
| *** Ruetobas has joined #openstack-ceilometer | 16:11 | |
| *** mihgen has quit IRC | 16:18 | |
| *** sdake_ has joined #openstack-ceilometer | 16:18 | |
| *** sdake_ has joined #openstack-ceilometer | 16:18 | |
| *** nati_ueno has quit IRC | 16:35 | |
| openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: Fix validation error for invalid field name in simple query https://review.openstack.org/82110 | 16:43 |
| *** packet has quit IRC | 16:46 | |
| *** shakayumi has joined #openstack-ceilometer | 16:51 | |
| *** shakayumi has quit IRC | 16:51 | |
| *** prad has quit IRC | 16:52 | |
| *** jpich has quit IRC | 16:53 | |
| *** admin0 has quit IRC | 16:58 | |
| *** shakayumi has joined #openstack-ceilometer | 17:00 | |
| *** shakayumi has quit IRC | 17:01 | |
| *** inc0 has quit IRC | 17:02 | |
| *** inc0 has joined #openstack-ceilometer | 17:08 | |
| *** sayalilunkad has joined #openstack-ceilometer | 17:09 | |
| *** prad has joined #openstack-ceilometer | 17:14 | |
| *** admin0 has joined #openstack-ceilometer | 17:15 | |
| *** inc0 has quit IRC | 17:15 | |
| *** nati_ueno has joined #openstack-ceilometer | 17:17 | |
| *** yassine has quit IRC | 17:21 | |
| *** prad has quit IRC | 17:21 | |
| *** eglynn has quit IRC | 17:23 | |
| *** mihgen has joined #openstack-ceilometer | 17:24 | |
| *** prad has joined #openstack-ceilometer | 17:24 | |
| *** nacim has quit IRC | 17:26 | |
| *** ildikov_ has quit IRC | 17:29 | |
| *** bada has joined #openstack-ceilometer | 17:29 | |
| *** leakypipes is now known as jaypipes | 17:33 | |
| *** prad has quit IRC | 17:34 | |
| *** admin0 has quit IRC | 17:38 | |
| *** _nadya_ has joined #openstack-ceilometer | 17:40 | |
| *** prad has joined #openstack-ceilometer | 17:45 | |
| *** piyushmasrani has quit IRC | 17:46 | |
| *** _nadya_ has quit IRC | 17:59 | |
| *** terriyu has quit IRC | 17:59 | |
| *** jmckind has quit IRC | 18:09 | |
| *** jmckind has joined #openstack-ceilometer | 18:10 | |
| *** creiht has joined #openstack-ceilometer | 18:11 | |
| creiht | howdy | 18:11 |
| creiht | I'm trying to better understand the issues around: https://bugs.launchpad.net/ceilometer/+bug/1294789?comments=all | 18:12 |
| *** zul has quit IRC | 18:13 | |
| creiht | anyone available to discuss how we can make the logging with swift better? | 18:13 |
| *** zul has joined #openstack-ceilometer | 18:16 | |
| *** sayalilunkad has quit IRC | 18:19 | |
| *** prad has quit IRC | 18:20 | |
| gordc | creiht: go for it. :) | 18:25 |
| creiht | gordc: well I'm having difficulty groking what is the root of the issue | 18:26 |
| gordc | creiht: as i understand it, swift logging level is by default set to debug. | 18:27 |
| creiht | what exactly about using the ceilometer middleware with swift makes ceilometer log extra stuff | 18:27 |
| creiht | gordc: if it isn't set in the config, yes it defaults to the debug | 18:27 |
| gordc | creiht: 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-ceilometer | 18:28 | |
| creiht | hah | 18:28 |
| *** mihgen has quit IRC | 18:29 | |
| creiht | gordc: I guess that's why I'm having trouble figuring out what the real issue is | 18:29 |
| creiht | is 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 |
| gordc | creiht: 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 |
| creiht | gordc: If that is an acceptable solution, I'm cool with that as well, just seems like debug info might be good for tracking down issues | 18:30 |
| openstackgerrit | Peter Portante proposed a change to openstack/ceilometer: Use swob instead of webob in swift unit tests https://review.openstack.org/82572 | 18:31 |
| gordc | creiht: 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 |
| creiht | heh | 18:31 |
| creiht | well 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 |
| gordc | creiht: to each their own :) i guess it doesn't make sense to have ceilometer logs flooding swift logs. | 18:32 |
| creiht | gordc: but if there is anything that I can do to help, let me know | 18:32 |
| gordc | creiht: ah, yeah i don't think the oslo.log stuff is relevant, i'm pretty sure it'll happen regardless. | 18:32 |
| creiht | right | 18:32 |
| gordc | creiht: cool cool. thanks for following up. | 18:32 |
| creiht | gordc: do you see though how the level should be able to be set in the config without having to make any changes? | 18:34 |
| creiht | and not have to hard code it? | 18:34 |
| openstackgerrit | Peter Portante proposed a change to openstack/ceilometer: Use swob instead of webob in swift unit tests https://review.openstack.org/82572 | 18:35 |
| gordc | creiht: here? https://github.com/openstack/swift/blob/master/etc/proxy-server.conf-sample#L41 | 18:35 |
| creiht | gordc: well that sets it for the proxy server | 18:36 |
| creiht | you should also be able to set the same variable in your middleware section | 18:36 |
| creiht | and if that is the config that is read in and set to swift.common.utils.get_logger | 18:36 |
| creiht | it will set that level | 18:36 |
| gordc | creiht: like this: https://review.openstack.org/#/c/82526/1/ceilometer/objectstore/swift_middleware.py? or something different? | 18:37 |
| gordc | the above would be reading in the conf file and changing the log_level to another level (just for ceilometer middleware) | 18:38 |
| creiht | right, but what I'm saying is that you should be able to actually set it in the conf and not have to change th code | 18:38 |
| creiht | just sec | 18:38 |
| creiht | let me paste something | 18:38 |
| *** _nadya_ has joined #openstack-ceilometer | 18:39 | |
| gordc | creiht: but if we change the conf file, it'll affect swift as well no? | 18:39 |
| creiht | gordc: well the swift conf has a section specifically for the middleware right? | 18:39 |
| openstackgerrit | Peter Portante proposed a change to openstack/ceilometer: Use swob instead of webob in swift unit tests https://review.openstack.org/82572 | 18:40 |
| * creiht is trying to find an example | 18:41 | |
| gordc | creiht: 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 |
| creiht | gordc: http://paste.openstack.org/show/74175/ | 18:44 |
| creiht | in the swift proxy-server.conf, there is a section that looks like that | 18:44 |
| creiht | that's the paste.deploy config | 18:45 |
| creiht | adding the log_level there *should* in theory pass on through to the swift get_logger func | 18:45 |
| gordc | creiht: cool. yeah, that's what i told nprivalova to add. didn't realise it by default would override the one defined in proxy-server.conf | 18:46 |
| creiht | right | 18:46 |
| creiht | it is supposed to work that way | 18:46 |
| creiht | if it doesn't, let me know and we can work through it | 18:46 |
| gordc | creiht: i guess no code change is needed... just need to update devstack then. | 18:46 |
| creiht | yeah | 18:47 |
| gordc | creiht: i'll comment on patch... thanks for clarifying | 18:47 |
| creiht | np | 18:47 |
| creiht | I would try testing it first to be sure it works :) | 18:47 |
| creiht | but going that route still also allows the deployer to configure what level they want | 18:47 |
| gordc | i'll give it a try. you can probably mark invalid for the swift portion of bug. | 18:48 |
| creiht | k | 18:49 |
| *** prad has left #openstack-ceilometer | 18:51 | |
| creiht | gordc: I'll hang out here a bit longer if you have any questions | 18:52 |
| creiht | for future questions/issues it is pretty easy to talk to anyone in #openstack-swift | 18:52 |
| gordc | creiht: cool cool. | 18:52 |
| gordc | creiht: 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 |
| creiht | hrm | 18:54 |
| creiht | It has been a while since I messed with devstack, but ctrl+c *should* work | 18:54 |
| creiht | did you try hitting it again? | 18:54 |
| gordc | creiht: yeah, i hammer away at it. :) | 18:56 |
| creiht | heheh | 18:56 |
| gordc | creiht: maybe a new bug in devstack... i do remember it was fixed at one point | 18:56 |
| creiht | I guess it is about time to spin up a devstack instance again | 18:56 |
| gordc | creiht: ok. give it a try if you have time. it might just be my environment acting up. | 18:57 |
| creiht | gordc: I may have to set one up anyways to look how at how logging is setup | 18:58 |
| creiht | since they also seem to be complaining about the double logging | 19:00 |
| *** _nadya_ has quit IRC | 19:06 | |
| *** llu has quit IRC | 19:14 | |
| *** llu has joined #openstack-ceilometer | 19:15 | |
| *** ildikov_ has joined #openstack-ceilometer | 20:09 | |
| *** prad has joined #openstack-ceilometer | 20:11 | |
| *** prad has quit IRC | 20:18 | |
| *** prad has joined #openstack-ceilometer | 20:21 | |
| *** prad has quit IRC | 20:28 | |
| *** prad has joined #openstack-ceilometer | 20:31 | |
| *** ddutta has joined #openstack-ceilometer | 20:32 | |
| *** ddutta has quit IRC | 20:42 | |
| *** ddutta has joined #openstack-ceilometer | 20:43 | |
| *** luis_ has joined #openstack-ceilometer | 20:57 | |
| *** luisg has quit IRC | 20:59 | |
| *** bogdando has quit IRC | 21:06 | |
| *** bogdando has joined #openstack-ceilometer | 21:08 | |
| *** luis_ has quit IRC | 21:11 | |
| *** ruhe has left #openstack-ceilometer | 21:11 | |
| *** julim has quit IRC | 21:35 | |
| *** nati_ueno has quit IRC | 21:47 | |
| *** nati_ueno has joined #openstack-ceilometer | 21:48 | |
| *** thomasem has quit IRC | 21:50 | |
| *** alexpilotti has joined #openstack-ceilometer | 21:54 | |
| *** promulo has quit IRC | 22:02 | |
| *** ddutta has quit IRC | 22:09 | |
| *** promulo has joined #openstack-ceilometer | 22:31 | |
| *** prad has quit IRC | 22:35 | |
| *** jmckind has quit IRC | 22:42 | |
| *** gordc has quit IRC | 22:54 | |
| *** promulo has quit IRC | 23:04 | |
| *** ddutta has joined #openstack-ceilometer | 23:05 | |
| openstackgerrit | Joe Gordon proposed a change to openstack/ceilometer: Disable oslo.messaging and oslo-incubator debug logs https://review.openstack.org/82671 | 23:35 |
| openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Ensure idempotency of cardinality reduction in mongo https://review.openstack.org/82672 | 23:40 |
| *** promulo has joined #openstack-ceilometer | 23:45 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!