Wednesday, 2015-04-08

*** zhenq has joined #openstack-ceilometer00:07
*** Kennan2 has quit IRC00:08
*** Kennan has joined #openstack-ceilometer00:08
*** promulo has joined #openstack-ceilometer00:09
*** promulo__ has quit IRC00:12
*** jaypipes has quit IRC00:27
*** claudiub has quit IRC00:43
*** ankita_wagh has quit IRC00:43
*** ddieterly has joined #openstack-ceilometer00:46
*** Longgeek has joined #openstack-ceilometer00:50
*** Longgeek has quit IRC01:07
*** _cjones_ has quit IRC01:08
*** Longgeek has joined #openstack-ceilometer01:08
*** ddieterly has quit IRC01:10
*** ddieterly has joined #openstack-ceilometer01:24
*** gzhai_mob has joined #openstack-ceilometer01:27
*** gzhai_mob has quit IRC01:29
*** sbfox has joined #openstack-ceilometer01:31
*** llu has joined #openstack-ceilometer01:35
*** sbfox has quit IRC01:51
*** swamireddy has quit IRC02:00
*** swamireddy has joined #openstack-ceilometer02:00
*** boris-42 has quit IRC02:02
*** adam_g has quit IRC02:02
*** jroll has quit IRC02:02
*** boris-42 has joined #openstack-ceilometer02:05
*** jroll has joined #openstack-ceilometer02:07
*** adam_g has joined #openstack-ceilometer02:07
*** adam_g has quit IRC02:07
*** adam_g has joined #openstack-ceilometer02:07
*** jroll has quit IRC02:08
*** jroll has joined #openstack-ceilometer02:08
*** julim has quit IRC02:12
*** botang has quit IRC02:12
*** harlowja is now known as harlowja_away02:22
*** sbfox has joined #openstack-ceilometer02:26
*** jasonamyers_ has joined #openstack-ceilometer02:28
openstackgerritKomei Shimamura proposed openstack/ceilometer: Fix a variable name mistake  https://review.openstack.org/17064402:32
*** fnaval has quit IRC02:47
*** fnaval has joined #openstack-ceilometer02:54
*** sbfox has quit IRC03:04
*** julim has joined #openstack-ceilometer03:08
*** boris-42 has quit IRC03:18
*** zhenq has quit IRC03:19
openstackgerritZhiQiang Fan proposed openstack/ceilometer: WIP: ugly fix: pymongo 3.0 breaks ci gate  https://review.openstack.org/17145803:25
*** ddieterly has quit IRC03:52
*** rwsu has quit IRC04:07
*** _cjones_ has joined #openstack-ceilometer04:19
*** _cjones_ has quit IRC04:19
*** _cjones_ has joined #openstack-ceilometer04:20
*** _cjones_ has quit IRC04:28
*** amalagon has joined #openstack-ceilometer04:29
*** llu has quit IRC04:29
*** llu has joined #openstack-ceilometer04:30
*** deepthi has joined #openstack-ceilometer04:31
*** sbfox has joined #openstack-ceilometer04:35
*** ddieterly has joined #openstack-ceilometer04:52
*** ddieterly has quit IRC04:58
*** fnaval has quit IRC05:06
*** botang has joined #openstack-ceilometer05:13
*** julim has quit IRC05:15
*** botang has quit IRC05:18
*** yatin has joined #openstack-ceilometer05:24
*** changbl has joined #openstack-ceilometer05:31
*** jasonamyers_ has quit IRC05:33
*** ddieterly has joined #openstack-ceilometer05:54
*** ildikov has quit IRC05:54
*** ddieterly has quit IRC05:58
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Smarter list_resources for history  https://review.openstack.org/17038506:36
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Keep history of updated resource attributes  https://review.openstack.org/16840506:36
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Don't touch the sqla orm object  https://review.openstack.org/17148006:36
*** cmyster has quit IRC06:38
*** dikonoor has joined #openstack-ceilometer06:46
*** yprokule has joined #openstack-ceilometer06:47
*** Ala has joined #openstack-ceilometer06:54
*** ddieterly has joined #openstack-ceilometer06:55
*** ddieterly has quit IRC06:59
dikonoorI have a patch that has gone through reviews, but i would like to get some cores attention to this, could you please help? >> https://review.openstack.org/#/c/167370/07:00
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Smarter list_resources for history  https://review.openstack.org/17038507:03
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Keep history of updated resource attributes  https://review.openstack.org/16840507:03
*** ildikov has joined #openstack-ceilometer07:11
*** sbfox has quit IRC07:12
*** shardy has joined #openstack-ceilometer07:13
*** _nadya_ has joined #openstack-ceilometer07:17
*** afazekas has joined #openstack-ceilometer07:23
*** _nadya_ has quit IRC07:28
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Smarter list_resources for history  https://review.openstack.org/17038507:33
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Keep history of updated resource attributes  https://review.openstack.org/16840507:33
*** _nadya_ has joined #openstack-ceilometer07:35
openstackgerritJulien Danjou proposed openstack/gnocchi: sqlalchemy: use RDBMS check constraint where available  https://review.openstack.org/17054407:37
*** _nadya_ has quit IRC07:51
*** safchain has joined #openstack-ceilometer07:51
*** ildikov has quit IRC07:53
*** ddieterly has joined #openstack-ceilometer07:55
*** eglynn has joined #openstack-ceilometer07:59
*** ddieterly has quit IRC08:00
*** ildikov has joined #openstack-ceilometer08:09
*** IBerezovskiy has joined #openstack-ceilometer08:12
*** llu has quit IRC08:25
*** llu has joined #openstack-ceilometer08:25
*** ildikov has quit IRC08:41
*** amalagon has quit IRC08:48
*** _nadya_ has joined #openstack-ceilometer08:55
*** ddieterly has joined #openstack-ceilometer08:56
*** ildikov has joined #openstack-ceilometer08:57
*** ddieterly has quit IRC09:01
*** theanalyst has quit IRC09:04
*** theanalyst has joined #openstack-ceilometer09:07
dikonoorildikov : This is about https://review.openstack.org/#/c/167370/. Did you get a chance to sleep on it ? :)09:12
dikonooreglynn : Could you also take a look at https://review.openstack.org/#/c/167370/ and check if it's good to go ?09:13
ildikovdikonoor: I still prefer the way of adding tests09:14
eglynndikonoor: on a call right now, but will look again soon after09:14
ildikovdikonoor: this is not against you, I annoy others too with this testing topic... :)09:16
dikonoorildikov:  :) I notice that the ceilometer api tests use the pecan.testing module and all the current api tests are written in such a way that the moment I add any rule (for eg. get_meters: "rule:role_admin") , other test cases break.I think it happens because none of them set any header values with role and other data. Am I right about this observation ?09:18
*** _nadya_ has quit IRC09:18
dikonoorildikov: I see some test cases under ceilometer/tests/api/v2/test_acl_scenarios.py for access related stuff but don't think they really hit the policy.json09:21
ildikovdikonoor: I messed up policy.json yesterday, but the tests did not care about it...09:23
ildikovdikonoor: I mean locally09:23
dikonoorildikov: well.that's probably because of the bug in the old code.09:24
dikonoorildikov : Once the proposed changes go in , that might not be the case. The existing policy.json neither has a default rule nor has any entry for a specific rest api . Due to this the current code never hits the policy.json validation09:25
ildikovdikonoor: yeap, true, will check with your patch too09:25
dikonoorildikov: With the proposed changes, if the rule for a specific rest api is not found, it defaults to the default rule and starts to check for permissions09:27
dikonoorildikov : If I were to set the default rule to something specific (I have left it as "") ..say like rule : "role:admin" , then all the existing api test cases will fail..Because the test cases don't pass in role information09:28
ildikovdikonoor: well, role:admin as default would be too strict anyway09:29
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Smarter list_resources for history  https://review.openstack.org/17038509:29
ildikovdikonoor: well, what's missing is the real unit tests that fix the changed functionality and to check the api tests, which are more functional tests, but the restructirng is still an ongoing process and will finally happen at the beginning of Liberty09:30
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Smarter list_resources for history  https://review.openstack.org/17038509:31
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Keep history of updated resource attributes  https://review.openstack.org/16840509:31
dikonoorildikov : If I have to write few test cases around the changes I have proposed , I should ideally check for things like role , ownership , "!" and so on.. That would mean that I add some rules in the policy.json . However , any addition in policy.json would mean breaking existing test cases09:32
ildikovdikonoor: for instance the unit tests looks like thi sin Nova: https://github.com/openstack/nova/blob/master/nova/tests/unit/test_policy.py09:32
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Smarter list_resources for history  https://review.openstack.org/17038509:34
dikonoorildikov : This would mean writing the entire test suite for ceilometer api permissions for the small bug fix I made :)09:40
dikonoorildikov : But thanks for pointing me to the test_policy.py09:40
*** _nadya_ has joined #openstack-ceilometer09:53
*** ddieterly has joined #openstack-ceilometer09:57
*** ddieterly has quit IRC10:01
*** htruta has quit IRC10:05
*** htruta has joined #openstack-ceilometer10:09
*** cdent has joined #openstack-ceilometer10:13
*** elynn has joined #openstack-ceilometer10:26
dikonooreglynn : Please take a look at https://review.openstack.org/#/c/167370/ , when you get a chance10:28
*** cdent has quit IRC10:30
eglynndikonoor: what's the tl;dr on the discussion above with ildikov on providing a test for this?10:31
eglynndikonoor: conclusion is that this is too disruptive to accomodation within the current testing framework?10:31
eglynnildikov: do we have a rough consensus to land this patch for kilo-rc1 and then subsequently revisit the testing logic?10:32
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Smarter list_resources for history  https://review.openstack.org/17038510:34
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Keep history of updated resource attributes  https://review.openstack.org/16840510:34
dikonooregylnn : we haven't arrived at a consensus yet.. It will be better if the testing logic is revisited subsequently..10:35
ildikoveglynn: dikonoor: I'm not against of fixing this in a subsequent patch (series)10:36
eglynnildikov: that sounds pragmatic, thanks! :)10:37
eglynndikonoor: k, let's get this patch landed! ... thank you for your effeorts on this :)10:38
eglynn*efforts10:38
dikonoorildikov : That can be done : ) I can open another LP bug to get the test cases fixed10:39
ildikoveglynn: I guess the gate is still affected by the pymongo bump though...10:39
ildikovdikonoor: cool, sounds like a plan, thanks :)10:40
ildikoveglynn: thanks for the administration10:41
eglynnildikov: LOL on the administration10:42
ildikoveglynn: I guess it's a reflex thing, you're too experienced in this :)10:44
dikonoorildikov , egylnn : Thanks very much Folks ..that was my first OpenStack contribution..phew !10:45
ildikovdikonoor: welcome on board! :)10:46
dikonoorildikov , egylnn : Thanks , as promised , I will open a LP bug for the test cases and add you guys as reviewers10:47
ildikovdikonoor: cool, thanks10:50
*** claudiub has joined #openstack-ceilometer10:51
*** f13o has joined #openstack-ceilometer10:52
openstackgerritMerged openstack/gnocchi: Don't touch the sqla orm object  https://review.openstack.org/17148010:53
*** ddieterly has joined #openstack-ceilometer10:58
*** _nadya_ has quit IRC10:59
*** dikonoor has quit IRC11:00
*** ddieterly has quit IRC11:02
*** claudiub has quit IRC11:06
*** noye has joined #openstack-ceilometer11:20
*** noye has quit IRC11:28
*** pauldix has joined #openstack-ceilometer11:51
*** cdent has joined #openstack-ceilometer11:56
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Smarter list_resources for history  https://review.openstack.org/17038512:01
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Keep history of updated resource attributes  https://review.openstack.org/16840512:01
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Avoid query metric object twice from indexer  https://review.openstack.org/17159012:01
*** claudiub has joined #openstack-ceilometer12:02
*** deepthi has quit IRC12:06
*** cmyster has joined #openstack-ceilometer12:19
*** cmyster has joined #openstack-ceilometer12:19
*** boris-42 has joined #openstack-ceilometer12:23
*** jdandrea has quit IRC12:23
*** pauldix has quit IRC12:32
*** cmyster has quit IRC12:35
*** gordc has joined #openstack-ceilometer12:36
*** jdandrea has joined #openstack-ceilometer12:38
silehtjd__, I have added the tests ^^12:38
silehtjd__, I'm preparing a new review with some docs/examples12:38
jd__sileht: cool12:38
jd__I saw the tests for the timespam12:39
*** jasonamyers_ has joined #openstack-ceilometer12:39
silehtjd__, I have finished to fight with sqlalchemy to make my union query working with the 'orm joinedquery'12:39
* jd__ papats sileht 12:39
jd__joy of ORM :)12:40
*** jasonamyers has quit IRC12:40
*** prad has quit IRC12:41
*** jasonamyers_ has quit IRC12:42
*** jasonamyers has joined #openstack-ceilometer12:42
*** _nadya_ has joined #openstack-ceilometer12:42
openstackgerritMerged openstack/gnocchi: gabbi: merge metric and metrics test files  https://review.openstack.org/17125412:45
*** yatin has quit IRC12:46
*** jdandrea has quit IRC12:46
*** jdandrea has joined #openstack-ceilometer12:51
openstackgerritJulien Danjou proposed openstack/gnocchi: indexer: add support for filtering by duration  https://review.openstack.org/17160612:58
*** ddieterly has joined #openstack-ceilometer12:59
*** cmyster has joined #openstack-ceilometer13:02
*** cmyster has joined #openstack-ceilometer13:02
*** ddieterly has quit IRC13:04
*** cdent has quit IRC13:11
*** julim has joined #openstack-ceilometer13:11
*** edmondsw has joined #openstack-ceilometer13:12
*** jmank has joined #openstack-ceilometer13:12
openstackgerritJulien Danjou proposed openstack/gnocchi: Move JSON encoding routing in its own module  https://review.openstack.org/17161613:14
*** pauldix has joined #openstack-ceilometer13:15
*** jmanko has quit IRC13:21
*** jmank has quit IRC13:21
*** jmank has joined #openstack-ceilometer13:21
silehtjd__, any reason to have PATCH resource that return 200 with no data ?13:22
*** zul has quit IRC13:22
*** zul has joined #openstack-ceilometer13:22
silehtjd__, it's return 204, but why do we not return 200 with the new resource data ?13:23
jd__sileht: to avoid fetching the data once updated I'd say13:23
*** Longgeek has quit IRC13:23
jd__though we may have it available so I don't know otherwise13:24
silehtjd__, the indexer return the full resource13:24
jd__yeah so that's not the reason13:24
*** cdent has joined #openstack-ceilometer13:24
jd__sileht: there might be no reason!13:24
silehtjd__, ok I will send something, so13:25
silehtjd__, I need your https://review.openstack.org/#/c/171606/ to add the same thing for lifetime13:26
jd__hint hint cdent ^^^^13:26
cdentpardon what jd__ ?13:27
jd__cdent: you should review that patch :-)13:27
* cdent has been doing paperwork :(13:28
jd__I feel your pain cdent13:30
*** thomasem has joined #openstack-ceilometer13:34
cdentjd__: interesting test failure on that review13:34
cdenthttp://logs.openstack.org/06/171606/1/check/gate-gnocchi-python27/f87b647/console.html#_2015-04-08_13_16_03_44113:35
*** zhenq has joined #openstack-ceilometer13:35
*** thomasem has quit IRC13:36
*** thomasem has joined #openstack-ceilometer13:41
*** jaypipes has joined #openstack-ceilometer13:48
*** noye has joined #openstack-ceilometer13:49
*** ddieterly has joined #openstack-ceilometer13:52
*** jmatthew_ has joined #openstack-ceilometer13:59
gordcildikov: i forgot this yesterday but these are the equivalent functions: http://paste.openstack.org/show/200439/14:00
ildikovgordc: cool, thank you very much!14:02
gordcnp... re: meter-list there isn't one because well, the meter shouldn't reallly exist on its own.14:03
ildikovgordc: and these are supported on all back ends now, right?14:03
ildikovthe meter-list is not an issue, I realized it too after14:04
ildikovif the deprecation will be marked, then it will be totally fine to not have this in meter-list14:04
ildikov... I mean when we finally remove it14:04
gordcthere's patches for mongo and sql... elasticsearch is not applicable.14:05
ildikovgordc: hbase14:06
ildikov?14:06
gordcit's not available in hbase... because hbase is not close to feature parity14:07
gordci'd actually like to know if anyone is using it... because i'm not even sure it works.14:08
ildikovok, so it is acceptable that it will not be available there14:08
gordci personally think so but we can reach out to community on that?14:09
ildikovI'm not against your viewpoint, I just ask these many questions to be sure we don't miss anything major14:10
*** exploreshaifali has joined #openstack-ceilometer14:10
ildikovI don't think either that it would be widely used, but maybe it worth a question14:10
gordcildikov: thanks for keeping me honest.lol14:11
ildikovgordc: LOL :), you're welcome14:11
ildikovgordc: BTW, just like all our back end drivers it is also marked as production ready14:12
gordchbase?14:12
ildikovyes14:13
gordcerrr... we should probably look into that. DinaBelova, _nadya_, could probably correct me but i don't think even mirantis is using hbase... just playing with it.14:14
ildikovgordc: I think I owe you some beers or smth because of the backward compat stuff and other reminders... :)14:15
gordci won't turn down beers.14:15
ildikovok :)14:16
*** ddieterly has quit IRC14:18
*** ddieterly has joined #openstack-ceilometer14:18
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Smarter list_resources for history  https://review.openstack.org/17038514:19
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Keep history of updated resource attributes  https://review.openstack.org/16840514:19
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Avoid query metric object twice from indexer  https://review.openstack.org/17159014:19
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Allow comparaison of timestamp field to None  https://review.openstack.org/17166914:19
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Add history documentation  https://review.openstack.org/17167014:19
gordcsileht: was there a recent oslo.messaging patch that resolved tihs? https://bugs.launchpad.net/ceilometer/+bug/139370814:21
openstackLaunchpad bug 1393708 in Ceilometer "ceilometer collector can not reconnect to rabbitmq after restarting rabbitmq" [Undecided,New]14:21
ildikovgordc: anyway, I will update the docs with the deprecation info and then we will see what to do with the hbase case. does it sound ok?14:22
gordcildikov: sounds good to me.14:23
silehtgordc, I don't known, the reconnection code was a really crap before14:23
ildikovgordc: cool, tnx14:23
silehtgordc, but the kilo version have improve a lot that14:23
gordcsileht: hmm... i'll try simulating it.14:24
_nadya_gordc: yep, you're right14:26
_nadya_gordc: what's the question about hbase :)?14:27
silehtgordc, the error message looks like a issue with the ssl version negociation14:27
jd__sileht: I'm pondering about removing the PreciseTimestamp sometime…14:27
silehtjd__, the precise timestamp can be useful for ETAG14:28
*** prad has joined #openstack-ceilometer14:28
*** rbak has joined #openstack-ceilometer14:29
silehtjd__, because 'revision' is not present for the latest version and allows an attackers to known the number of records for your databases14:29
jd__sileht: ok but can we use that http://dev.mysql.com/doc/refman/5.6/en/fractional-seconds.html instead?14:29
silehtjd__, and 'id+lifetime_from' make a resource history row unique, so it can be used for etag14:30
gordc_nadya_: i was just wondering since mirantis did most/all contributions on hbase, if you actually test against it actively? because i have no idea if it's really production ready.14:30
silehtjd__, that looks good, but I don't remember why I have not used that for ceilometer14:31
jd__sileht: I think that's because we support old MySQL version14:31
silehtjd__, mariadb support that since 5.314:33
jd__sileht: ok I'll work on that14:34
silehtjd__, https://mariadb.com/kb/en/mariadb/microseconds-in-mariadb/14:34
_nadya_gordc: yep, we did some tests, but it was a release ago14:35
gordc_nadya_: ah cool cool. so i guess it's still a viable backend? except it has less functionality.14:38
_nadya_gordc: I believe yes14:38
gordc_nadya_: kk14:42
ildikov_nadya_: gordc: as for the Kilo release it still does not sound good that it is not really tested :(14:42
ildikovsame for DB2, even if it is similar to Mongo...14:43
_nadya_ildikov: gordc: we will discuss it today with the team14:43
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Add history documentation  https://review.openstack.org/17167014:45
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Allow comparaison of timestamp field to None  https://review.openstack.org/17166914:45
ildikov_nadya_: cool, sounds good, please let us know about the outcome :)14:46
gordcildikov: yeah, the db2 stuff is really sketchy to me. it's definitely something we should follow up on.14:46
gordc_nadya_: thanks!14:46
ildikovgordc: yeap, we have so many issues with back ends, would be nice to have some priority on clarifying the current state in the next cycle14:48
gordcildikov: agreed14:49
gordcwho understands this english? " Accept the default write concern, acknowledged writes, instead of setting safe=True."14:50
ildikovLOL :)14:50
*** claudiub has quit IRC14:50
ildikovyou definitely need some background to understand it...14:51
gordchttp://api.mongodb.org/python/current/changelog.html?highlight=safe14:51
gordci can't tell if that's a real sentence and i'm stilling waking up or if it makes no sense at all.lol14:52
cdentit makes no sense14:53
ildikovgordc: this explains the things better: http://emptysqua.re/blog/pymongos-new-default-safe-writes/14:54
gordccdent: yeah... ... ... lol14:54
gordcildikov: ah cool.14:55
ildikovgordc: I still think that we should limit the version in requirements...14:55
ildikovI rechecked ZhiQiang's WIP patch to see if it fails with the same issue or a different one as it fails randomly for me locally...14:57
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Avoid query metric object twice from indexer  https://review.openstack.org/17159015:00
*** jaypipes has quit IRC15:03
gordcildikov: we could do something similar to keystone and check if safe is valid attr in MongoClient.15:03
gordcif not we can just not pass it in?15:03
*** jaypipes has joined #openstack-ceilometer15:05
ildikovgordc: sounds good to me15:06
*** jaypipes has quit IRC15:06
ildikovgordc: the only issue is that two tests fail with the w=1 setting on the gate currently...15:06
*** exploreshaifali has quit IRC15:08
ildikovgordc: logs: http://logs.openstack.org/58/171458/1/check/gate-ceilometer-python27/f7d5bb0/console.html15:09
silehtcdent, jd__ gordc I try to find a better name for the 'lifetime_to/from' columns in my gnocchi history patch15:12
silehtto ensure people make the difference between lifetime_to/from and started_at/ended_at15:12
cdentsileht: can you explain to me in more words what it is supposed to represent, then maybe I can think of a different term?15:12
gordcildikov: dang pymongo.15:13
silehtcdent, lifetime_to/from is related to the history15:14
silehtcdent, attributes of the resource was that between lifetime_to/from15:14
cdentis it a value that is per revision of the resource, or per the entire history?15:14
silehtcdent, per revision15:15
gordcildikov: there's this patch https://review.openstack.org/#/c/171438/15:15
silehtcdent, lifetime_to/from is handled by gnocchi , started_at/ended_at is user defined15:15
ildikovgordc: agreed15:15
* cdent nods and thinks15:15
ildikovgordc: cool, I saw the fixes on the stable branches already15:16
ildikovgordc: I think it's the best for the master too15:16
* cdent looks at code15:16
gordcsileht: me thinking too.15:16
cdentsileht: is the reason for having a lifetime_to to avoid looking at the next revision in the history? presumably the window of a lifetime is from when this one changed to when the next one (forward in time) changed15:20
*** _cjones_ has joined #openstack-ceilometer15:20
silehtcdent, yes15:20
cdentanother question: What we expecting someone to want to do with this info?15:20
gordcinception/death?15:21
gordcwe need our namer of things15:21
* cdent plays the trombone of doom15:22
silehtcdent, for example: a user want to know all the instances that have the flavor 'mini' during the last month, with how many times15:22
cdentah15:22
cdentso if I resize my instance multiple times during the last month, that's going to make for resource history messiness...got it15:23
* cdent continues thinking15:23
cdentwhat about instead of a to and from a start and duration?15:24
silehtcdent, you have an example of this query here: https://review.openstack.org/#/c/171670/2/doc/source/rest.yaml15:24
*** _cjones_ has quit IRC15:24
silehtline 23615:24
* cdent nods15:25
cdentso duration is semantically awkward for those kinds of queries15:25
silehtyes15:25
silehtit's simpler to store utcnow(), than computing thing with duration15:26
* cdent nods15:26
cdentso the very long form of the names are:15:26
cdentresource revision start time15:27
cdentresourc revision end time15:27
silehtyes :)15:27
cdentis there anything wrong with revision_start and revision_end?15:27
cdentI find that a bit more comprehensible than "lifetime"15:27
silehtno that looks good15:27
cdent\o/15:28
silehtjd__, gordc wdyt ? ^15:28
silehtlets change to that, that looks better than lifetime and see what happend15:30
silehtcdent, thx15:31
gordcsileht: revision seems ok to me.15:31
cdentyou're welcome, we got there in the end15:31
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Smarter list_resources for history  https://review.openstack.org/17038515:34
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Keep history of updated resource attributes  https://review.openstack.org/16840515:34
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Add history documentation  https://review.openstack.org/17167015:34
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Allow comparison of timestamp field to None  https://review.openstack.org/17166915:34
*** claudiub has joined #openstack-ceilometer15:36
*** rjaiswal has joined #openstack-ceilometer15:37
*** rwsu has joined #openstack-ceilometer15:37
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Smarter list_resources for history  https://review.openstack.org/17038515:38
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Keep history of updated resource attributes  https://review.openstack.org/16840515:38
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Add history documentation  https://review.openstack.org/17167015:38
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Allow comparison of timestamp field to None  https://review.openstack.org/17166915:38
*** afazekas has quit IRC15:43
*** ildikov has quit IRC15:44
*** ozialien has joined #openstack-ceilometer15:44
*** ozialien has quit IRC15:47
*** Ephur_ is now known as Ephur15:48
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Smarter list_resources for history  https://review.openstack.org/17038515:51
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Keep history of updated resource attributes  https://review.openstack.org/16840515:51
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Add history documentation  https://review.openstack.org/17167015:51
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Allow comparison of timestamp field to None  https://review.openstack.org/17166915:51
openstackgerritgordon chung proposed openstack/ceilometer-specs: Enable Liberty specs  https://review.openstack.org/16832115:54
*** rjaiswal has quit IRC15:55
*** rjaiswal has joined #openstack-ceilometer15:55
openstackgerritgordon chung proposed openstack/ceilometer-specs: Enable Liberty specs  https://review.openstack.org/16832115:55
*** _cjones_ has joined #openstack-ceilometer15:57
*** ozialien has joined #openstack-ceilometer15:59
*** ozialien has quit IRC16:01
*** theanalyst has quit IRC16:04
*** ozialien has joined #openstack-ceilometer16:05
*** ozialien has quit IRC16:06
*** theanalyst has joined #openstack-ceilometer16:07
*** julim has quit IRC16:14
*** ozialien has joined #openstack-ceilometer16:19
*** ozialien has quit IRC16:23
*** Longgeek has joined #openstack-ceilometer16:25
*** jruano has joined #openstack-ceilometer16:25
*** claudiub has quit IRC16:28
*** Longgeek has quit IRC16:30
*** rjaiswal has quit IRC16:34
*** ankita_wagh has joined #openstack-ceilometer16:48
*** julim has joined #openstack-ceilometer16:54
openstackgerritgordon chung proposed openstack/ceilometer-specs: Enable Liberty specs  https://review.openstack.org/16832117:00
*** rbak has quit IRC17:02
*** claudiub has joined #openstack-ceilometer17:05
*** _cjones_ has quit IRC17:06
*** _cjones_ has joined #openstack-ceilometer17:07
*** drjones has joined #openstack-ceilometer17:09
*** _cjones_ has quit IRC17:10
*** yatin has joined #openstack-ceilometer17:10
*** ddieterly has quit IRC17:17
*** safchain has quit IRC17:18
*** ddieterly has joined #openstack-ceilometer17:18
*** megh has joined #openstack-ceilometer17:19
*** megh_ has joined #openstack-ceilometer17:20
*** megh has quit IRC17:20
*** harlowja_away is now known as harlowja17:21
*** ozialien has joined #openstack-ceilometer17:22
*** megh_ has quit IRC17:22
*** megh has joined #openstack-ceilometer17:22
cdentgordc: do we want to moot specs such as "extract alarming to its own repo" or "extract polling agent to its own repo" (even if just for sake of discussion)?17:23
*** megh has quit IRC17:23
*** megh has joined #openstack-ceilometer17:23
*** IBerezovskiy has quit IRC17:24
gordccdent: what do you mean by 'moot'17:24
cdentbring to the meeting to talk about17:24
*** megh has quit IRC17:24
*** megh_ has joined #openstack-ceilometer17:24
cdentconsider for our consideration17:24
cdentthat kind of thing17:24
*** ildikov has joined #openstack-ceilometer17:25
gordccdent: ah, yeah, it's probably best to give a heads up on that.17:25
*** megh_ has quit IRC17:26
*** megh has joined #openstack-ceilometer17:26
gordccdent: i'd assume there will be some people that would want to discuss how/what we split.17:26
cdentgordc: I think it should happen, but I don't know if I want to go to the trouble of writing the specs if people are opposed in principle17:26
*** claudiub has quit IRC17:26
gordccdent: hahah. i'm going to assume we'll need to write something detailed to explain what 'extract alarming' exactly entails.17:27
*** ddieterly has quit IRC17:28
cdenttrue17:28
*** ddieterly has joined #openstack-ceilometer17:28
ildikovdo we have an exact plan regarding what to extract and why, etc?17:30
*** exploreshaifali has joined #openstack-ceilometer17:31
cdentildikov: not yet, no17:31
cdentI guess my question is: do we want to make a plan in advance of specs, or use specs as the place where discussion happens17:32
ildikovcdent: where did the question come from?17:32
ildikovI guess specs should be the place or maybe a plan spec in advance of specs can happen :)17:33
cdentjust before you joined I said: gordc: do we want to moot specs such as "extract alarming to its own repo" or "extract polling agent to its own repo" (even if just for sake of discussion)?17:33
ildikovyeap, I quickly checked the logs, that is why I asked if we or you have a plan or desired state to reach?17:34
*** yatin has quit IRC17:35
cdentthe desired state is for each of the several unique value propositions provided by the current ceilometer to be serviced by decoupled pieces17:35
cdentthis would make them easier to maintain and improve and also allow greater diversity in the ecosystem: they could evolve independently more easily17:35
*** drjones has quit IRC17:36
gordccdent: we'd probably need some strategy on how to do testing between the separated services.17:36
ildikovwell, as for the pieces it's more architectural question than repo question I think, but well maybe from one side this direction could still help17:38
cdentthe different repos help reinforce what ought to be strong architectural boundaries. it's sort of social cue or clue on how to manage stuff.17:39
*** _cjones_ has joined #openstack-ceilometer17:39
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Smarter list_resources for history  https://review.openstack.org/17038517:40
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Keep history of updated resource attributes  https://review.openstack.org/16840517:40
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Add history documentation  https://review.openstack.org/17167017:40
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Allow comparison of timestamp field to None  https://review.openstack.org/17166917:40
edmondswgetting gate failures for icehouse and juno on https://review.openstack.org/#/c/169442/ ... any ideas here?17:41
edmondswI see the following in the logs... but if that was the problem, why wouldn't we have hit this before?17:41
edmondswpkg_resources.ContextualVersionConflict: (python-ceilometerclient 1.0.13.post24 (/opt/stack/new/python-ceilometerclient), Requirement.parse('python-ceilometerclient<=1.0.12,>=1.0.6'), set(['ceilometer']))17:41
edmondsweglynn, gordc, ^^ any ideas?17:42
*** jruano has quit IRC17:43
ildikovcdent: hmm, maybe you're right, but it would be better to see an architecture diagram with proposed boundaries as repos to have the same big picture17:43
*** srsakhamuri has quit IRC17:44
cdentildikov: yeah, some diagrams would be useful. I haven't had a chance to think about it deeply enough, yet, but plan to17:45
ildikovedmondsw: we had issues with heat and the latest ceilo client, maybe a restriction has just landed as requirements patches landed today on stable branches because of the latest pymongo client17:45
edmondswildikov, tx... so what can I do about that?17:46
ildikovcdent: cool, it can worth some time on the summit too, if you have diagrams to sketch on a whiteboard17:47
* cdent nods17:47
ildikovcdent: but we will see what we will have room for...17:47
ildikovedmondsw: hmm, as we are talking about the gate maybe the infra guys can give you a better answer than me17:49
*** jruano has joined #openstack-ceilometer17:49
edmondswildikov, yeah... headed there17:49
ildikovedmondsw: cool, tnx, I will try to follow the discussion to be prepared if it happens again in the future17:51
*** ddieterly has quit IRC17:52
*** _nadya_ has quit IRC17:53
*** ddieterly has joined #openstack-ceilometer17:53
*** yprokule has quit IRC18:04
*** jruano has quit IRC18:13
edmondsweglynn, ildikov, the infra team is telling me and gordc they think the problem on https://review.openstack.org/#/c/169442/ is caused by not having stable branches for ceilometerclient18:17
edmondswhttp://specs.openstack.org/openstack/openstack-specs/specs/library-stable-branches.html18:17
ildikovedmondsw: yeap, I saw, thanks for heads up18:18
edmondswdhellmann ^18:18
edmondswwhat do we do about that?18:18
dhellmannedmondsw: someone with permission needs to create a stable branch at the appropriate commit in the repo18:19
edmondswyeah... which isn't me, so... Guess I'm asking WHO is going to do something about that :)18:19
openstackgerritJulien Danjou proposed openstack/gnocchi: indexer: add support for filtering by duration  https://review.openstack.org/17160618:20
openstackgerritJulien Danjou proposed openstack/gnocchi: sqlalchemy: use precise DATETIME rather than DECIMAL in MySQL  https://review.openstack.org/17176418:20
edmondswand what do we do about my patch and others in the meantime... find a way to bypass that check in the gate, or will they just wait?18:21
dhellmannedmondsw: I don't know if I have permission, and I'm travelling this week so I wouldn't want to do it and disappear from conference wifi if there was an issue. gordc or eglynn should be able to take care of it18:21
edmondswdhellmann, tx18:22
ildikovedmondsw: as much as I know eglynn is our stable branch expert on board18:22
eglynnedmondsw: I can look into this tomorrow, I need to run out the office door very shortly (after 7pm here)18:23
edmondsweglynn, tx18:23
edmondswalways another thing for the todo list :)18:24
*** yeungp has joined #openstack-ceilometer18:24
gordcsorry, i don't have permission either18:25
*** _nadya_ has joined #openstack-ceilometer18:25
dhellmanngordc: I added you to https://review.openstack.org/#/admin/groups/533,members so you should now. Do you know how to use gerrit to create the branch?18:28
*** boris-42 has quit IRC18:28
dhellmanneglynn: fyi ^^18:28
*** jruano has joined #openstack-ceilometer18:29
gordcdhellmann: nope... follow the steps in the link you gave?18:29
dhellmanngordc: fill out the form at https://review.openstack.org/#/admin/projects/openstack/python-ceilometerclient,branches -- be careful, though! :-)18:30
dhellmanngordc: the branch name should be stable/something, probably stable/kilo at this point but you might need more branches for older releases18:31
dhellmanngordc: you'll want to find the sha from the tag for the version meant to be used with kilo18:31
gordcdhellmann: i don't have a form on that link.18:33
*** _nadya_ has quit IRC18:34
gordcedmondsw: is this critical? or we ok with letting eglynn handle it later?18:34
*** jmatthew_ has quit IRC18:39
edmondswgordc, I think it can wait until tomorrow18:39
gordccool cool18:39
edmondswthen once that's in we need to talk about cutting 1.0.14 to pick up it and the other recent changes18:40
dhellmanngordc: hrm. I wonder if I gave permission to the wrong gordon. What is your gerrit id?18:40
dhellmanngordc: gord@live.ca?18:40
gordcyep. chungg18:41
gordcdhellmann: i can see i'm part of ceilometer-stable-maint18:41
dhellmanngordc: ah, you may need to be in https://review.openstack.org/#/admin/groups/151,members instead -- jd__ can you help with permissions for gordc to make a stable branch of ceilometer-client?18:42
dhellmannI'm not in that group, so I can't add anyone18:42
jd__dhellmann: sure18:42
jd__you're in gordc18:42
dhellmannjd__: thanks18:43
edmondswildikov, you mentioned "requirements patches landed today on stable branches because of the latest pymongo client"... could that be causing the py27 errors on https://review.openstack.org/#/c/167370/ ?18:44
gordcjd__: cool cool.18:44
ildikovedmondsw: yes, we still have the issue on master18:44
* gordc sees same list on https://review.openstack.org/#/admin/projects/openstack/python-ceilometerclient,branches as before.18:45
edmondswildikov, have a bug number?18:45
gordcdhellmann, i'm assuming i'm doing something wrong. will check with eglynn tomorrow18:45
ildikovedmondsw: one sec, need to fing the links18:45
gordchttps://bugs.launchpad.net/ceilometer/+bug/144139318:46
openstackLaunchpad bug 1441393 in Keystone juno "Keystone and Ceilometer unit tests fail with pymongo 3.0" [High,In progress] - Assigned to Ihar Hrachyshka (ihar-hrachyshka)18:46
gordcedmondsw: pymongo3 broke the world18:46
openstackgerritJulien Danjou proposed openstack/gnocchi: Move JSON encoding routing in its own module  https://review.openstack.org/17161618:46
*** eglynn has quit IRC18:46
edmondswgordc, tx ... ildikov ^18:47
ildikovedmondsw: requirements patch on master: https://review.openstack.org/#/c/17143818:47
ildikovgordc: thanks :) I was already too optimistic and closed that tab already :)18:48
*** _nadya_ has joined #openstack-ceilometer18:49
ildikovedmondsw: the plan is to limit pymongo for kilo for now18:49
edmondswworks for me18:50
ildikovedmondsw: you mean locally?18:51
*** _nadya_ has quit IRC18:52
edmondswildikov, no, meant "that sounds good"18:52
edmondswsorry for the confusion18:52
ildikovedmondsw: a-ha, ok, I've just seen that you rechecked that patch I think that is why I associated to the tests :)18:53
edmondswwanted to associate the bugs with it for dikonoor doesn't have to look those up herself18:54
ildikovcool, got it18:54
*** exploreshaifali has quit IRC18:58
*** Ala has quit IRC19:01
openstackgerritPradeep Kilambi proposed openstack/gnocchi: WIP: Add support for ArchivePolicy Rule Endpoints  https://review.openstack.org/17128819:01
*** rbak has joined #openstack-ceilometer19:02
jd__sileht: HAHAHA we have MySQL 5.5 on the gate which does not support decimal with timestamp19:03
*** megh has quit IRC19:04
* jd__ is going berserk19:04
* cdent gives jd__ a cookie19:04
openstackgerritgordon chung proposed openstack/ceilometer-specs: Enable Liberty specs  https://review.openstack.org/16832119:08
openstackgerritgordon chung proposed openstack/ceilometer: use oslo.messaging dispatch filter  https://review.openstack.org/17134719:11
cdentthat's it for me19:11
cdentnight19:11
*** cdent has quit IRC19:11
*** fnaval has joined #openstack-ceilometer19:23
*** eglynn has joined #openstack-ceilometer19:25
*** jruano has quit IRC19:40
*** _cjones_ has quit IRC19:50
*** jruano has joined #openstack-ceilometer19:53
*** ddieterly has quit IRC19:58
*** ozialien has quit IRC19:59
*** _cjones_ has joined #openstack-ceilometer20:04
*** ddieterly has joined #openstack-ceilometer20:04
*** megh has joined #openstack-ceilometer20:05
*** megh has quit IRC20:05
*** megh has joined #openstack-ceilometer20:05
*** _nadya_ has joined #openstack-ceilometer20:09
*** _nadya_ has quit IRC20:12
*** msly has joined #openstack-ceilometer20:17
mslyanyone have additional comments on this bug? https://bugs.launchpad.net/ceilometer/+bug/142158420:18
openstackLaunchpad bug 1421584 in Ceilometer "cpu_util volume won't report right for cpu over commit case" [Medium,Triaged] - Assigned to Ilya Tyaptin (ityaptin)20:18
*** prad has quit IRC20:19
gordcmsly: sorry, haven't looked at it lately, putting out gate fires.20:20
gordcwill look in a bit.20:20
*** prad has joined #openstack-ceilometer20:30
*** rbak has quit IRC20:33
*** rbak has joined #openstack-ceilometer20:34
openstackgerritgordon chung proposed openstack/ceilometer: WIP: ugly fix: pymongo 3.0 breaks ci gate  https://review.openstack.org/17145820:36
*** rbak has quit IRC20:36
openstackgerritgordon chung proposed openstack/ceilometer: pymongo 3.0 breaks ci gate  https://review.openstack.org/17145820:36
*** rbak has joined #openstack-ceilometer20:36
*** rbak has quit IRC20:36
openstackgerritKomei Shimamura proposed openstack/ceilometer: Fix a variable name mistake  https://review.openstack.org/17064420:37
*** eglynn has quit IRC20:42
gordcmsly: do you have a solution?20:44
*** sbfox has joined #openstack-ceilometer20:47
*** exploreshaifali has joined #openstack-ceilometer20:55
*** sbfox has quit IRC21:00
*** ddieterly has quit IRC21:02
*** ddieterly has joined #openstack-ceilometer21:02
*** jruano has quit IRC21:03
mslyI do not... I tried digging into libvirt to see if I could get the vm util from it, but couldn't find it21:05
*** jruano has joined #openstack-ceilometer21:06
*** megh has quit IRC21:07
gordcmsly: yeah... i need to look into this further but from what i've been told, the only real way is to have an agent on each guest.21:07
gordcthere is apparently a libvirt agent but only works based a list of conditions (ie. must be linux, might not be installed, etc...)21:08
mslygordc: I have thought about that but.. that doesn't seem like a solution the community would accept to me21:08
gordcmsly: agreed. especially if the only thing you get is cpu_util.21:09
gordcmsly: we could multiple the value by number of cpus? but i think that gives only a rough estimate... and i haven't really tried it to verify if that's even valid.21:10
mslygordc: The hard part about generating a formula for it with the current info we can get is figuring out when it is 'overcommitting'21:12
*** edmondsw has quit IRC21:15
gordcmsly: yeah... tbh, i'm not a libvirt expert so i'll need to chat more with Nova contact. maybe there is some obvious option we're missing.21:16
*** ddieterly has quit IRC21:30
openstackgerritJulien Danjou proposed openstack/gnocchi: indexer: add support for filtering by duration  https://review.openstack.org/17160621:33
*** megh has joined #openstack-ceilometer21:35
openstackgerritRohit Jaiswal proposed openstack/ceilometer-specs: configuration via data store  https://review.openstack.org/17182621:39
openstackgerritRohit Jaiswal proposed openstack/ceilometer-specs: configuration via data store  https://review.openstack.org/17182621:40
*** noye has quit IRC21:43
*** noye has joined #openstack-ceilometer21:47
openstackgerritRohit Jaiswal proposed openstack/ceilometer-specs: configuration via data store  https://review.openstack.org/17182621:49
*** exploreshaifali has quit IRC21:57
*** exploreshaifali has joined #openstack-ceilometer22:02
*** noye has quit IRC22:03
*** noye has joined #openstack-ceilometer22:03
*** noye has quit IRC22:13
*** jruano has quit IRC22:20
*** prad has quit IRC22:27
openstackgerritRohit Jaiswal proposed openstack/ceilometer-specs: configuration via data store  https://review.openstack.org/17182622:28
*** zhenq has quit IRC22:31
*** exploreshaifali has quit IRC22:32
*** alexpilotti has quit IRC22:39
*** Challa has joined #openstack-ceilometer22:40
*** Challa has left #openstack-ceilometer22:44
*** gordc has quit IRC22:45
*** challa has joined #openstack-ceilometer22:45
*** noye has joined #openstack-ceilometer22:47
ankita_waghHi, Did anyone face any issues in running tox for juno release on centos 7 ?22:49
*** sbfox has joined #openstack-ceilometer22:54
challaHi,  i am having issues with PartitionCoordination when i enable HA for Central agent. Members of group: set([])  & My subset: [] extract_my_subset  are always empty, can some point me if my  zookeeper configuration is indeed correct?22:56
*** noye has quit IRC22:58
challa[coordination] backend_url = kazoo://10.22.152.47:2181 heartbeat = 1.023:01
*** sbfox has quit IRC23:09
*** pauldix has quit IRC23:16
*** sbfox has joined #openstack-ceilometer23:32
*** Ephur has quit IRC23:34
*** yeungp has quit IRC23:38
*** sbfox has quit IRC23:43
*** sbfox has joined #openstack-ceilometer23:46
openstackgerritKomei Shimamura proposed openstack/ceilometer: Fix a variable name mistake  https://review.openstack.org/17064423:55

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