Friday, 2017-01-20

*** vint_bra has joined #openstack-telemetry00:14
*** chopmann has quit IRC00:17
*** gordc has quit IRC00:18
*** thorst_ has joined #openstack-telemetry00:28
*** gongysh has joined #openstack-telemetry00:37
*** thorst_ has quit IRC00:45
*** thorst_ has joined #openstack-telemetry00:45
*** zhurong has joined #openstack-telemetry00:48
*** thorst_ has quit IRC00:50
*** gongysh has quit IRC00:52
*** thorst_ has joined #openstack-telemetry01:19
*** thorst_ has quit IRC01:19
*** vint_bra has quit IRC01:25
*** chopmann has joined #openstack-telemetry01:28
*** zhurong has quit IRC01:49
*** zhurong has joined #openstack-telemetry01:51
*** lhx__ has joined #openstack-telemetry01:53
*** thorst_ has joined #openstack-telemetry02:01
*** thorst_ has quit IRC02:01
*** chopmann has quit IRC02:07
*** jmlowe has joined #openstack-telemetry02:08
*** gongysh has joined #openstack-telemetry02:11
*** dschultz has quit IRC02:15
*** lhx__ has quit IRC02:15
*** dschultz has joined #openstack-telemetry02:16
*** lhx__ has joined #openstack-telemetry02:16
*** links has joined #openstack-telemetry02:53
*** larainema has quit IRC03:26
*** thorst_ has joined #openstack-telemetry03:39
*** zhurong has quit IRC03:39
*** thorst_ has quit IRC03:40
*** links has quit IRC04:16
*** links has joined #openstack-telemetry04:18
*** lhx__ has quit IRC04:19
*** lhx__ has joined #openstack-telemetry04:20
*** r-mibu has quit IRC04:33
*** larainema has joined #openstack-telemetry04:48
*** jmlowe has quit IRC04:51
*** r-mibu has joined #openstack-telemetry04:53
*** catinthe_ has quit IRC04:54
*** lhx__ has quit IRC04:57
*** catintheroof has joined #openstack-telemetry05:02
*** sheel has joined #openstack-telemetry05:04
*** tlian has quit IRC05:13
*** catintheroof has quit IRC05:14
*** nadya has joined #openstack-telemetry05:14
*** nadya has quit IRC05:15
*** Jack_Iv has joined #openstack-telemetry05:17
*** Jack_Iv has quit IRC05:37
*** thorst_ has joined #openstack-telemetry05:40
*** catintheroof has joined #openstack-telemetry05:46
*** thorst_ has quit IRC05:48
*** zhurong has joined #openstack-telemetry05:49
*** lhx__ has joined #openstack-telemetry05:53
*** Jack_Iv has joined #openstack-telemetry05:59
*** Jack_Iv_ has joined #openstack-telemetry06:01
*** adriant has quit IRC06:03
*** Jack_Iv has quit IRC06:04
*** nadya has joined #openstack-telemetry06:17
*** lhx__ has quit IRC06:18
*** Jack_Iv_ has quit IRC06:19
*** lhx__ has joined #openstack-telemetry06:21
*** nadya has quit IRC06:26
*** zhurong has quit IRC07:03
silehtjd__, new gate breakage: https://bugs.launchpad.net/ubuntu/+source/mysql-5.7/+bug/165798907:04
openstackLaunchpad bug 1657989 in mysql-5.7 (Ubuntu) "No more able to create non null datetime column" [Undecided,New]07:04
openstackgerritMehdi Abaakouk (sileht) proposed openstack/gnocchi: DNM: test  https://review.openstack.org/42309007:06
*** yprokule has joined #openstack-telemetry07:10
silehtjd__, mysql change a warning into a error into minor update ... awesome07:14
silehtFrom the releasenote: An ALTER TABLE operation failed with an error rather than a warning when adding a DATE or DATETIME column under these conditions: a) the column was NOT NULL and no default value was supplied; b) strict SQL mode was enabled and NO_ZERO_DATE SQL mode was not enabled; c) the table was not empty. (Bug #16888677)07:14
*** Jack_Iv has joined #openstack-telemetry07:31
*** Jack_Iv has quit IRC07:37
*** pcaruana has joined #openstack-telemetry07:43
*** dewanee_ has quit IRC07:45
*** dewanee has joined #openstack-telemetry07:47
*** nadya has joined #openstack-telemetry07:48
*** flwang has quit IRC07:51
*** flwang has joined #openstack-telemetry07:54
openstackgerritMehdi Abaakouk (sileht) proposed openstack/gnocchi: DNM: test  https://review.openstack.org/42309007:58
openstackgerritMehdi Abaakouk (sileht) proposed openstack/gnocchi: indexer: fix default of new datetime column  https://review.openstack.org/42309008:01
*** tesseract has joined #openstack-telemetry08:05
*** shardy has joined #openstack-telemetry08:14
*** larainema has quit IRC08:16
jd__sileht: it's a pretty good news however that they stop accepting wrong data…08:18
silehtjd__, the list is long: https://www.digitalocean.com/community/tutorials/how-to-prepare-for-your-mysql-5-7-upgrade08:18
silehtjd__, they are not more able to divide by 008:18
silehtlol08:19
jd__haha08:19
jd__sileht: the bug might be in Aodh too08:19
silehtjd__, we have to set a server_default for this column now08:19
silehtjd__, I don't now what to put08:19
silehtjd__, because NOW() is not in UTC08:19
jd__we cant' have NOT NULL without default?08:19
jd__ah yes it's aa)08:20
silehtjd__, yes08:20
jd__it's dumb08:20
silehtit's stupid limitation08:20
jd__i don't know what to say08:20
jd__let's put 1983-10-23 by default08:20
silehtthe erreur should show up on update/insert08:20
jd__that's my birthday08:20
silehtnot ALTER/CREATE08:20
jd__:P08:20
silehtjd__, or it should reject the ALTER if one row have NULL08:23
silehtbut in our case we don't have any with NULL...08:23
silehtany/any row08:23
*** shardy is now known as shardy_afk08:26
*** afazekas has quit IRC08:30
*** liusheng has quit IRC08:31
*** liusheng has joined #openstack-telemetry08:31
*** afazekas has joined #openstack-telemetry08:35
lhx__jd__, could you give some tips about your last comment? :)08:36
lhx__https://review.openstack.org/#/c/413920/08:36
jd__lhx__: does the default install of Ceilometer works without collector?08:37
jd__lhx__: (with your patch)08:37
*** zhangguoqing has joined #openstack-telemetry08:38
lhx__jd__, sileht has tested for it, seems worked well.08:38
jd__lhx__: you did not answer my question :(08:46
lhx__jd__, do you mean in the patch?08:48
jd__lhx__: I mean, with your patch, does the default install of Ceilometer runs ans works without collector (* by default *) ?08:48
*** shardy_afk is now known as shardy08:52
lhx__jd__, the patch just added a warning, and the defualt install without collector works :)08:53
jd__lhx__: so by default Ceilometer is installed without the collector?08:53
jd__the answer to my question is just a yes or no and you are not able to answer, that is amusing08:54
jd__:)08:54
lhx__jd__, sorry for that, it's no :)08:55
jd__lhx__: then until it's "yes" I'll -1 your patch ;-)08:55
lhx__jd__, I guess you meant +108:57
jd__yes = +108:57
jd__no = -108:57
jd__:)08:57
jd__did Heat get fixed btw?08:58
lhx__jd__, I will make sure about that to get the honor +1 :)09:00
lhx__jd__, the gate seems to be ok now09:00
*** openstackgerrit has quit IRC09:02
jd__ok I just saw failure maybe I need to recheck09:03
*** openstackgerrit has joined #openstack-telemetry09:25
openstackgerritMehdi Abaakouk (sileht) proposed openstack/gnocchi: indexer: fix default of new datetime column  https://review.openstack.org/42309009:25
*** zyun|3 has joined #openstack-telemetry09:33
zyun|3hi all09:33
zyun|3may I ask a question?09:33
zyun|3I manually delete the ceilometer db, how could I recovery this?09:34
silehtrestore your backup ?09:34
zyun|3other method? I want the openstack can help rebuild this db09:35
zyun|3is there any method?09:35
silehtlol09:35
*** efoley has joined #openstack-telemetry09:35
silehtit's a database, we store thing because we can't query the past states of your openstack installation09:37
jd__yeah you wont' have any past data though if you wait you'll have new data being filled09:38
*** yassine has joined #openstack-telemetry09:38
*** yassine is now known as Guest5571809:39
*** lhx__ has quit IRC09:45
*** thorst_ has joined #openstack-telemetry09:46
glbhello :)09:49
glb I think there is a bug in the way stevedore.extension is used09:49
glbhttps://github.com/openstack/ceilometer/blob/master/ceilometer/agent/manager.py#L29509:49
glbthe callback should just ignore the extensions it can not load, right ? ^09:50
silehtglb, that depends on the reason of the failure09:51
*** thorst_ has quit IRC09:51
glbbecause it crash with: TypeError: Message objects do not support addition.09:52
glbfor 'tripleo_overcloud_nodes' (I don't know what it is...)  but if I use the default stevedore callback it just run over and not load the extension09:53
*** yprokule has quit IRC09:54
glbsileht: so, it's an expected behavior ?09:56
silehtglb, yes09:57
silehtglb, you have an extension somewhere that raise 'TypeError: Message objects do not support addition.'09:57
silehtglb, we just reraise it09:57
silehtglb, we could just improve it code to print a message with the extension name that all09:58
*** nadya has quit IRC09:59
glbthx for the explanation :), yes it will be nice because i've to add some print() for have this kind of information09:59
*** hfu has quit IRC10:00
*** lhx__ has joined #openstack-telemetry10:04
*** nijaba has quit IRC10:05
openstackgerritMehdi Abaakouk (sileht) proposed openstack/ceilometer: agent: always print the extension name on failure  https://review.openstack.org/42316710:08
silehtglb, that should provide the name of the broken extention: https://review.openstack.org/42316710:09
*** lhx_ has joined #openstack-telemetry10:09
*** lhx__ has quit IRC10:10
silehtjd__, I have no two issue to fix the mysql bug10:10
silehtjd__, I don't known of the set a different server_default for mysql and postgres10:11
glbsileht, thanks10:11
silehtjd__, and the sql migration testing don't work well with custom type and server_default10:11
*** nijaba has joined #openstack-telemetry10:12
*** nijaba has quit IRC10:12
*** nijaba has joined #openstack-telemetry10:12
jd__sileht: :(10:12
lhx_jd__, was what you said default install on devstack?10:21
jd__lhx_: sorry?10:21
*** gongysh has quit IRC10:22
lhx_jd__, I thought if I just need to change the devstack configure10:23
lhx_?10:23
jd__lhx_: you have to but… do you think people deploy with devstack?10:24
openstackgerritSanu M proposed openstack/python-aodhclient: Removes unused utf-8 declaration  https://review.openstack.org/42318110:24
*** gongysh has joined #openstack-telemetry10:28
openstackgerritMerged openstack/aodh: api: add auth_mode option  https://review.openstack.org/42145410:30
lhx_jd__, if people deploy without devstack, they can choose to the default package to be installed10:30
lhx_lhx__, does it?10:30
lhx_jd__, sorry ping wrong10:31
*** dschultz has quit IRC10:32
jd__lhx_: ok so by default let's say the collector package is not installed anymore by distribution since it's deprecated10:33
jd__lhx_: does Ceilometer work in this case?10:33
lhx_jd__, I'm not clear what I should do in addition to changing the default configuration of devstack10:34
jd__lhx_: change the default configuration not-in-devstack?10:34
jd__lhx_: by default everything publishes to the collector10:36
jd__lhx_: and then you say to users "do not install the collector"?10:36
jd__if i'm a user I'm going to be "WTF?"10:36
jd__:)10:36
lhx_jd__, I meant to change enable_service ceilometer-collector in devstack10:37
*** gongysh has quit IRC10:37
jd__lhx_: that too it should be off by default10:37
zyun|3Hello10:39
zyun|3Where is the log file for ceilometer-dbsync cmd?10:39
jd__zyun|3: on your screen in theory10:40
zyun|3ok10:40
zyun|3and I encounter below error when running this cmd? who can give me some clue?10:41
*** Jack_Iv has joined #openstack-telemetry10:41
zyun|3http://paste.openstack.org/show/595758/10:41
zyun|3Many many thanks10:41
zyun|3what error in my env?10:44
zyun|3who can help me out? Thanks in advance10:51
lhx_jd__, do I just change the pipeline.yaml?10:51
jd__lhx_: is it enough?10:52
jd__lhx_: I think so10:52
jd__zyun|3: what version is this btw?10:52
lhx_jd__, so I just change it with gnocchi://?10:53
zyun|3it is kilo release10:53
jd__zyun|3: did you install another version at some point? it's weird10:53
jd__lhx_: sounds good10:53
jd__lhx_: do you want me to write the patch? :P10:53
lhx_jd__, no no no, lol10:54
zyun|3no, I only has this release10:54
jd__zyun|3: you configured it for mongodb right?10:54
zyun|3yes10:54
*** zyun|3 has quit IRC11:05
silehtjd__, I don't find a solution for the mysql bug ...11:06
openstackgerritMehdi Abaakouk (sileht) proposed openstack/gnocchi: indexer: fix default of new datetime column  https://review.openstack.org/42309011:10
openstackgerritMehdi Abaakouk (sileht) proposed openstack/gnocchi: indexer: fix datetime with mysql >= 5.7.17  https://review.openstack.org/42309011:11
* sileht tries a ugly trick11:12
*** Jack_Iv has quit IRC11:12
*** Jack_Iv has joined #openstack-telemetry11:14
jd__sileht: what's the problem?11:16
silehtjd__, I can get something working11:17
silehtjd__, changing server_default is just a nightmare11:17
silehtjd__, server_default is a raw string passed to the server11:19
silehtjd__, and mysql need a datetime when postgresql a timestamp11:20
openstackgerritHanxi Liu proposed openstack/ceilometer: Deprecate collector  https://review.openstack.org/41392011:25
openstackgerritMerged openstack/ceilometer: pipeline: remove tests helper from runtime code  https://review.openstack.org/42063611:27
openstackgerritMehdi Abaakouk (sileht) proposed openstack/gnocchi: indexer: fix datetime with mysql >= 5.7.17  https://review.openstack.org/42309011:32
silehtjd__, I have found for the postgresql pb11:32
silehtjd__, for mysql it fails to compare the col type: http://paste.openstack.org/show/595766/11:32
*** Jack_Iv has quit IRC11:46
*** thorst_ has joined #openstack-telemetry11:47
*** yprokule has joined #openstack-telemetry11:47
jd__sileht: :/11:50
jd__sileht: can you if based on the engine?11:50
*** nadya has joined #openstack-telemetry11:50
*** thorst_ has quit IRC11:52
silehtjd__, no I haven't found that11:52
jd__sileht: ask zzzeeek maybe11:52
silehtjd__, this is due to our custome types11:52
jd__sileht: the datetime one?11:53
silehtjd__, yes11:53
jd__hm11:53
jd__maybe we can use a different trick to have fsp=611:53
silehtjd__, I have found a trick !11:53
openstackgerritMehdi Abaakouk (sileht) proposed openstack/gnocchi: indexer: fix datetime with mysql >= 5.7.17  https://review.openstack.org/42309011:53
jd__just now?11:53
silehtjd__, if I pass the literal string with miliseconds everybody is happy11:53
openstackgerritHanxi Liu proposed openstack/ceilometer: Ship YAML file to /usr/share  https://review.openstack.org/41230911:55
jd__sileht: damn…11:55
silehtjd__, I think that hide a bug somewhere11:56
jd__sileht: ?11:58
silehtjd__, alembic is adding extra quote for string instead of using the sqla column type: https://bitbucket.org/zzzeek/alembic/src/376837484acc80a6cbc1cda9125935c05a94ee0f/alembic/ddl/mysql.py?at=master&fileviewer=file-view-default#mysql.py-29411:59
*** pcaruana has quit IRC11:59
jd__-_-11:59
*** hfu has joined #openstack-telemetry12:01
silehtby change adding quote work also for postgres and sqlalchemey server_default12:02
*** hfu has quit IRC12:05
openstackgerritMehdi Abaakouk (sileht) proposed openstack/gnocchi: indexer: fix datetime with mysql >= 5.7.17  https://review.openstack.org/42309012:05
silehtjd__, ready to laught:12:06
silehthe supported range is '1000-01-01 00:00:00' to '9999-12-31 23:59:59'.12:06
*** pcaruana has joined #openstack-telemetry12:06
* sileht -> lunch12:07
jd__why 1000 ? lol12:08
*** lhx_ has quit IRC12:11
openstackgerritMerged openstack/ceilometer: gnocchi: prepare removal of gnocchi uuid5  https://review.openstack.org/41842612:14
openstackgerritMehdi Abaakouk (sileht) proposed openstack/gnocchi: indexer: fix datetime with mysql >= 5.7.17  https://review.openstack.org/42309012:22
openstackgerritMehdi Abaakouk (sileht) proposed openstack/gnocchi: indexer: fix datetime with mysql >= 5.7.17  https://review.openstack.org/42309012:30
*** catinthe_ has joined #openstack-telemetry12:32
*** catintheroof has quit IRC12:33
*** lhx_ has joined #openstack-telemetry12:35
*** shardy is now known as shardy_lunch12:36
EmilienMjd__: thanks for your help on the Wsgi Goal - I'm on PTO today but i'll update the wording this week-end, thanks again!12:42
openstackgerritJulien Danjou proposed openstack/gnocchi: Remove py{27,35} from valid tox targets  https://review.openstack.org/42324812:43
jd__EmilienM: you're welcome12:43
*** thorst_ has joined #openstack-telemetry12:47
*** hfu has joined #openstack-telemetry12:50
*** hfu has quit IRC12:53
*** links has quit IRC12:55
*** hfu has joined #openstack-telemetry12:56
*** hfu has quit IRC12:58
*** hfu has joined #openstack-telemetry12:59
*** hfu has quit IRC13:01
*** nadya has quit IRC13:03
*** hfu has joined #openstack-telemetry13:03
*** hfu has quit IRC13:04
openstackgerritHanxi Liu proposed openstack/ceilometer: Deprecate collector  https://review.openstack.org/41392013:07
*** shardy_lunch is now known as shardy13:14
*** hfu has joined #openstack-telemetry13:21
EmilienMjd__: so we are not useful? lol13:22
*** shardy has quit IRC13:27
*** shardy has joined #openstack-telemetry13:27
*** hfu has quit IRC13:32
*** vint_bra has joined #openstack-telemetry13:52
*** nicodemus_ has joined #openstack-telemetry13:55
openstackgerritgordon chung proposed openstack/gnocchi: serialise: simplify array format  https://review.openstack.org/42280013:58
glbis there any way to disable some extension like NodesDiscoveryTripleO in ceilometer configuration ? (I found nothing talking about that in doc ...)13:58
silehtglb, why do you want to do that ? If the extension doesn't load correctly this is a bug :)14:00
openstackgerritHanxi Liu proposed openstack/ceilometer: Deprecate collector  https://review.openstack.org/41392014:00
nicodemus_hello14:03
*** hfu has joined #openstack-telemetry14:06
*** dave-mccowan has joined #openstack-telemetry14:07
nicodemus_I have a question: how does gnocchi handle keystone authentication? I mean, I was looking at the code (stable/3.0) but couldn't find any import referencing keystone14:07
nicodemus_I was trying to compare it with ceilometer code for example, that has a keystone_client.py14:08
glbsileht: because I don't need this extension, I only use ceilometer for swift object storage, and I find my issue is related to novaclient because in requirement.txt it's set to "> 2.x.x" so it have install the latest (ocata version) and it's not compatible :)14:08
glbmaybe requirement.txt need to be fixed ^14:09
*** Jack_Iv has joined #openstack-telemetry14:10
silehtglb, are your mixing openstack project version ?14:10
glbno I try to be consistent in version, and I use venv for each project14:12
*** fguillot has joined #openstack-telemetry14:13
openstackgerritgordon chung proposed openstack/ceilometer: polling definition file  https://review.openstack.org/40568214:20
openstackgerritgordon chung proposed openstack/ceilometer: add polling.yaml docs  https://review.openstack.org/40771114:21
*** tlian has joined #openstack-telemetry14:23
*** hfu has quit IRC14:30
jd__EmilienM: what for? :)14:31
jd__nicodemus_: it's in api-paste.ini that keystone is imported and used14:32
EmilienMjd__: I read your candidacy ;-)14:33
*** catintheroof has joined #openstack-telemetry14:33
*** catinthe_ has quit IRC14:36
*** Jack_Iv has quit IRC14:37
nicodemus_jd__, oh, I see. So, unlike ceilometer since gnocchi only needs to authenticate it imports keystone_authtoken and that's it, right? Ceilometer probably needs to do other stuff with keystone and for that needs a keystone_client14:38
nicodemus_that makes sense? I'm just trying to understand14:39
*** liusheng has quit IRC14:40
*** liusheng has joined #openstack-telemetry14:40
*** vint_bra has quit IRC14:45
*** sheel has quit IRC14:47
silehtnicodemus_, yes14:53
nicodemus_sileht, got it. thanks!14:53
*** rbak has joined #openstack-telemetry14:55
jd__EmilienM: the "we" of the TC? no you're completely useless, sorry :)14:55
jd__nicodemus_: exactly14:56
openstackgerritMerged openstack/ceilometer: agent: always print the extension name on failure  https://review.openstack.org/42316714:58
nicodemus_jd__, thanks!14:59
openstackgerritMerged openstack/panko: remove ceilometer/meter specific hbase code  https://review.openstack.org/41860315:17
lhx_jd__, I think we have to choose database(default direct publisher) by default15:18
lhx_https://review.openstack.org/#/c/413920/15:18
lhx_if not, the tests will fail15:21
jd__lhx_: fix the test then? :)15:22
lhx_jd__, hum..., sure?15:23
*** gordc has joined #openstack-telemetry15:26
openstackgerritMerged openstack/ceilometer: Fix same type of simple message error  https://review.openstack.org/42259015:26
lhx_jd__, but must we change them with gnocchi://?15:27
lhx_if so, we may remove all the database tests?15:28
jd__lhx_: whatever is the easiest I'd say15:28
jd__lhx_: you can't remove the database tests but they should not use the collector anyway15:29
lhx_jd__, they use direct publisher(database), not collector15:31
jd__lhx_: so they don't really care about changing the pipeline.yaml?15:31
*** dschultz has joined #openstack-telemetry15:33
lhx_jd__, who are "they"?15:34
jd__lhx_: the database tests15:34
lhx_jd__, before I change into database://, many tests failed. but now it get through normally. I think the database connect with publisher normally in the tests15:37
*** dschultz has quit IRC15:38
lhx_jd__, actually they care from my point of view15:38
openstackgerritMerged openstack/gnocchi: indexer: fix datetime with mysql >= 5.7.17  https://review.openstack.org/42309015:39
jd__lhx_: we don't want people to use the database:// by default so …15:39
jd__just fix the test in another way :)15:39
lhx_jd__, I will have a try. :(15:44
*** shardy has quit IRC15:48
*** shardy has joined #openstack-telemetry15:49
jd__lhx_: may the force be with you15:57
openstackgerritJulien Danjou proposed openstack/gnocchi: Remove non-existent -all suffix in tox targets  https://review.openstack.org/42334315:57
lhx_jd__, Star Wars! lol15:59
*** pcaruana has quit IRC16:03
*** joadavis_ has joined #openstack-telemetry16:04
*** dave-mccowan has quit IRC16:29
*** lhx_ has quit IRC16:29
*** lhx_ has joined #openstack-telemetry16:30
*** lhx_ has quit IRC16:30
*** sudipto has joined #openstack-telemetry16:32
*** sudipto_ has joined #openstack-telemetry16:32
*** zhangguoqing has quit IRC16:36
*** efoley has quit IRC16:42
*** nadya has joined #openstack-telemetry16:47
jd__sileht: one of your patch has a real bug on carbonara as gordc said16:54
jd__just in case you missed16:54
openstackgerritMerged openstack/python-gnocchiclient: Stop always encoding resource  https://review.openstack.org/41314516:59
gordcthe hell. that merged really quick17:01
*** fguillot has quit IRC17:04
*** glb has quit IRC17:05
*** fguillot has joined #openstack-telemetry17:20
silehtjd__, yeah but I wait that most of the patches are merged before doing git-review to not reset the gate queues17:29
jd__k17:30
openstackgerritMerged openstack/aodh: install-guide: remove useless step  https://review.openstack.org/42145517:36
openstackgerritMerged openstack/aodh: Move api-paste file to common code location  https://review.openstack.org/42145617:36
openstackgerritMerged openstack/aodh: Move policy.json out of etc  https://review.openstack.org/42222017:36
openstackgerritMerged openstack/gnocchi: indexer: fix migration script "no_more_slash"  https://review.openstack.org/42190017:44
openstackgerritMerged openstack/gnocchi: rest: returns orignal resource id  https://review.openstack.org/41783617:44
*** nadya has quit IRC17:49
nicodemus_question: when a resource is rceted in gnocchi, there are certain values for the resource that belong to generic, while others are metadata belonging to the resource-type. Is it possible to force a specific value that belongs to generic?17:53
nicodemus_rceted -> created17:54
nicodemus_specificalle, I'd like to provide a custom value to 'original_resource_id'17:54
nicodemus_specifically17:54
gordci don't know if you want to be hacking original_resource_id. its value relates directly to resource_id. if you change one, you need to change the other or it won't be accessible.17:59
*** nadya has joined #openstack-telemetry18:02
nicodemus_gordc, do both always have the same value?18:03
nicodemus_gordc, to rephrase... our use case is the following: we need to create an instance_disk resource and we only know about the external ID (which will be the original_resource_id)18:07
nicodemus_therefore, we don't know what ID gnocchi will auto-generate for the resulting resource18:08
nicodemus_in that case, we cannot associate the instance_disk resource with its instance, except querying gnocchi every time and searching for the original_resource_id to get the actual ID18:09
*** dschultz has joined #openstack-telemetry18:13
gordcnicodemus_: sorry, have to step out. but they are not necessarily same value. if the id you used to create resource isn't a uuid, it will translate it into one and put that in id and the resource id you passed in will be put into orignial_resource_id.18:18
gordcit should allow you to just search for it using the id you created with though. don't really understand your issue.18:18
*** nadya has quit IRC18:19
nicodemus_the thing is that we'd like to avoid having to search each time, since the id is not a hash we cannot know which one it is unless we search...18:20
nicodemus_if we knew how gnocchi translates the non-uuid into an uuid, we could make the translation in our side as well18:21
*** nadya has joined #openstack-telemetry18:22
*** nadya has quit IRC18:22
*** jefrite has quit IRC18:40
silehtnicodemus_, you should not rely on how id are generated by gnocchi that can change over the time18:41
*** sudipto has quit IRC18:41
*** sudipto_ has quit IRC18:41
silehtnicodemus_, I think all API call take the uuid and original_resource_id as id18:41
silehtYou can use http://localhost/v1/resource/generic/75c44741-cc60-4033-804e-2d3098c7d2e9 or http://localhost/v1/resource/generic/<my_original_resource_id>18:42
silehtboth should work18:42
nicodemus_sileht, that's good news indeed, if we can query for the original_resource_id18:43
*** yprokule has quit IRC18:45
silehtnicodemus_, if some API calls doesn't do the conversion, this is perhaps a bug18:47
openstackgerritMerged openstack/gnocchi: add metricd tester for profiling  https://review.openstack.org/42244618:48
nicodemus_sileht, let me check just in case...18:48
*** jefrite has joined #openstack-telemetry18:50
openstackgerritMerged openstack/gnocchi: carbonara: prepare datetime for pandas.to_datetime()  https://review.openstack.org/42120818:53
openstackgerritMerged openstack/gnocchi: carbonara: remove a pandas.iteritems()  https://review.openstack.org/42151618:53
openstackgerritMerged openstack/gnocchi: carbonara: handle timestamps from struct with numpy  https://review.openstack.org/42157018:54
openstackgerritMerged openstack/gnocchi: Remove py{27,35} from valid tox targets  https://review.openstack.org/42324818:54
openstackgerritMerged openstack/gnocchi: Remove non-existent -all suffix in tox targets  https://review.openstack.org/42334318:54
nicodemus_sileht, looks like all the api calls do the conversion :) thanks, I'll do it that way18:58
openstackgerritMerged openstack/gnocchi: carbonara: Don't use clean_ts()  https://review.openstack.org/42171419:06
openstackgerritMerged openstack/gnocchi: carbonara: use numpy for serialization  https://review.openstack.org/42176619:06
openstackgerritMerged openstack/gnocchi: carbonara: use numpy for unserialization  https://review.openstack.org/42190119:09
openstackgerritMerged openstack/gnocchi: carbonara: Add benchmark for split()  https://review.openstack.org/42249619:11
openstackgerritMerged openstack/gnocchi: utils: allow ResourceUUID to convert UUID  https://review.openstack.org/41301419:12
*** Guest55718 has quit IRC19:26
*** Jack_Iv has joined #openstack-telemetry19:28
*** tesseract has quit IRC19:32
openstackgerritMehdi Abaakouk (sileht) proposed openstack/gnocchi: carbonara: add bench for resample  https://review.openstack.org/42287719:37
openstackgerritMehdi Abaakouk (sileht) proposed openstack/gnocchi: carbonara: add merge() benchmark  https://review.openstack.org/42263719:37
openstackgerritMehdi Abaakouk (sileht) proposed openstack/gnocchi: carbonara: don't use groupby for split  https://review.openstack.org/42244519:37
openstackgerritJulien Danjou proposed openstack/gnocchi: carbonara: don't use groupby for split  https://review.openstack.org/42244519:49
openstackgerritJulien Danjou proposed openstack/gnocchi: carbonara: add merge() benchmark  https://review.openstack.org/42263719:49
openstackgerritJulien Danjou proposed openstack/gnocchi: serialise: simplify array format  https://review.openstack.org/42280019:50
*** catinthe_ has joined #openstack-telemetry19:50
*** catintheroof has quit IRC19:53
*** Jack_Iv has quit IRC19:55
*** Jack_Iv has joined #openstack-telemetry19:59
*** jefrite has quit IRC20:12
*** jefrite has joined #openstack-telemetry20:18
*** jefrite has quit IRC20:25
jd__sileht: http://logs.openstack.org/96/421096/2/check/gate-python-aodhclient-python27-ubuntu-xenial/3105fb7/console.html20:26
jd__I really think i'm gonna kick oslo out of telemetry sooner or later20:26
*** jefrite has joined #openstack-telemetry20:33
*** dave-mccowan has joined #openstack-telemetry20:34
gordcjd__: stop copying swift20:36
jd__gordc: they got it right :(20:36
gordclol just leave openstack. do it right. :P20:37
*** gordc has quit IRC20:42
openstackgerritJulien Danjou proposed openstack/python-aodhclient: Switch to extras usage for test requirements  https://review.openstack.org/42351520:44
*** jmlowe has joined #openstack-telemetry20:48
*** nicodemus_ has quit IRC21:02
*** catinthe_ has quit IRC21:02
*** catintheroof has joined #openstack-telemetry21:03
*** catintheroof has quit IRC21:07
openstackgerritMerged openstack/gnocchi: carbonara: don't use groupby for split  https://review.openstack.org/42244521:13
openstackgerritJulien Danjou proposed openstack/gnocchi: sqlalchemy: fix compat search on created_by_project_id  https://review.openstack.org/42352221:14
*** Jack_Iv has quit IRC21:17
*** jmlowe has quit IRC21:29
openstackgerritPradeep Kilambi proposed openstack/aodh: Add sem-ver flag so pbr generates correct version  https://review.openstack.org/42352721:38
*** gordc has joined #openstack-telemetry21:41
*** thorst_ has quit IRC22:02
*** thorst_ has joined #openstack-telemetry22:03
*** thorst_ has quit IRC22:07
*** shardy has quit IRC22:22
*** thorst_ has joined #openstack-telemetry22:43
*** dave-mccowan has quit IRC22:46
*** thorst_ has quit IRC22:47
*** masber has quit IRC22:59
*** masber has joined #openstack-telemetry23:00
*** thorst_ has joined #openstack-telemetry23:15
openstackgerritMerged openstack/aodh: Add sem-ver flag so pbr generates correct version  https://review.openstack.org/42352723:16
*** thorst_ has quit IRC23:20
*** gordc has quit IRC23:37
*** rbak has quit IRC23:49

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