*** vint_bra has joined #openstack-telemetry | 00:14 | |
*** chopmann has quit IRC | 00:17 | |
*** gordc has quit IRC | 00:18 | |
*** thorst_ has joined #openstack-telemetry | 00:28 | |
*** gongysh has joined #openstack-telemetry | 00:37 | |
*** thorst_ has quit IRC | 00:45 | |
*** thorst_ has joined #openstack-telemetry | 00:45 | |
*** zhurong has joined #openstack-telemetry | 00:48 | |
*** thorst_ has quit IRC | 00:50 | |
*** gongysh has quit IRC | 00:52 | |
*** thorst_ has joined #openstack-telemetry | 01:19 | |
*** thorst_ has quit IRC | 01:19 | |
*** vint_bra has quit IRC | 01:25 | |
*** chopmann has joined #openstack-telemetry | 01:28 | |
*** zhurong has quit IRC | 01:49 | |
*** zhurong has joined #openstack-telemetry | 01:51 | |
*** lhx__ has joined #openstack-telemetry | 01:53 | |
*** thorst_ has joined #openstack-telemetry | 02:01 | |
*** thorst_ has quit IRC | 02:01 | |
*** chopmann has quit IRC | 02:07 | |
*** jmlowe has joined #openstack-telemetry | 02:08 | |
*** gongysh has joined #openstack-telemetry | 02:11 | |
*** dschultz has quit IRC | 02:15 | |
*** lhx__ has quit IRC | 02:15 | |
*** dschultz has joined #openstack-telemetry | 02:16 | |
*** lhx__ has joined #openstack-telemetry | 02:16 | |
*** links has joined #openstack-telemetry | 02:53 | |
*** larainema has quit IRC | 03:26 | |
*** thorst_ has joined #openstack-telemetry | 03:39 | |
*** zhurong has quit IRC | 03:39 | |
*** thorst_ has quit IRC | 03:40 | |
*** links has quit IRC | 04:16 | |
*** links has joined #openstack-telemetry | 04:18 | |
*** lhx__ has quit IRC | 04:19 | |
*** lhx__ has joined #openstack-telemetry | 04:20 | |
*** r-mibu has quit IRC | 04:33 | |
*** larainema has joined #openstack-telemetry | 04:48 | |
*** jmlowe has quit IRC | 04:51 | |
*** r-mibu has joined #openstack-telemetry | 04:53 | |
*** catinthe_ has quit IRC | 04:54 | |
*** lhx__ has quit IRC | 04:57 | |
*** catintheroof has joined #openstack-telemetry | 05:02 | |
*** sheel has joined #openstack-telemetry | 05:04 | |
*** tlian has quit IRC | 05:13 | |
*** catintheroof has quit IRC | 05:14 | |
*** nadya has joined #openstack-telemetry | 05:14 | |
*** nadya has quit IRC | 05:15 | |
*** Jack_Iv has joined #openstack-telemetry | 05:17 | |
*** Jack_Iv has quit IRC | 05:37 | |
*** thorst_ has joined #openstack-telemetry | 05:40 | |
*** catintheroof has joined #openstack-telemetry | 05:46 | |
*** thorst_ has quit IRC | 05:48 | |
*** zhurong has joined #openstack-telemetry | 05:49 | |
*** lhx__ has joined #openstack-telemetry | 05:53 | |
*** Jack_Iv has joined #openstack-telemetry | 05:59 | |
*** Jack_Iv_ has joined #openstack-telemetry | 06:01 | |
*** adriant has quit IRC | 06:03 | |
*** Jack_Iv has quit IRC | 06:04 | |
*** nadya has joined #openstack-telemetry | 06:17 | |
*** lhx__ has quit IRC | 06:18 | |
*** Jack_Iv_ has quit IRC | 06:19 | |
*** lhx__ has joined #openstack-telemetry | 06:21 | |
*** nadya has quit IRC | 06:26 | |
*** zhurong has quit IRC | 07:03 | |
sileht | jd__, new gate breakage: https://bugs.launchpad.net/ubuntu/+source/mysql-5.7/+bug/1657989 | 07:04 |
---|---|---|
openstack | Launchpad bug 1657989 in mysql-5.7 (Ubuntu) "No more able to create non null datetime column" [Undecided,New] | 07:04 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: DNM: test https://review.openstack.org/423090 | 07:06 |
*** yprokule has joined #openstack-telemetry | 07:10 | |
sileht | jd__, mysql change a warning into a error into minor update ... awesome | 07:14 |
sileht | From 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-telemetry | 07:31 | |
*** Jack_Iv has quit IRC | 07:37 | |
*** pcaruana has joined #openstack-telemetry | 07:43 | |
*** dewanee_ has quit IRC | 07:45 | |
*** dewanee has joined #openstack-telemetry | 07:47 | |
*** nadya has joined #openstack-telemetry | 07:48 | |
*** flwang has quit IRC | 07:51 | |
*** flwang has joined #openstack-telemetry | 07:54 | |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: DNM: test https://review.openstack.org/423090 | 07:58 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: indexer: fix default of new datetime column https://review.openstack.org/423090 | 08:01 |
*** tesseract has joined #openstack-telemetry | 08:05 | |
*** shardy has joined #openstack-telemetry | 08:14 | |
*** larainema has quit IRC | 08:16 | |
jd__ | sileht: it's a pretty good news however that they stop accepting wrong data… | 08:18 |
sileht | jd__, the list is long: https://www.digitalocean.com/community/tutorials/how-to-prepare-for-your-mysql-5-7-upgrade | 08:18 |
sileht | jd__, they are not more able to divide by 0 | 08:18 |
sileht | lol | 08:19 |
jd__ | haha | 08:19 |
jd__ | sileht: the bug might be in Aodh too | 08:19 |
sileht | jd__, we have to set a server_default for this column now | 08:19 |
sileht | jd__, I don't now what to put | 08:19 |
sileht | jd__, because NOW() is not in UTC | 08:19 |
jd__ | we cant' have NOT NULL without default? | 08:19 |
jd__ | ah yes it's aa) | 08:20 |
sileht | jd__, yes | 08:20 |
jd__ | it's dumb | 08:20 |
sileht | it's stupid limitation | 08:20 |
jd__ | i don't know what to say | 08:20 |
jd__ | let's put 1983-10-23 by default | 08:20 |
sileht | the erreur should show up on update/insert | 08:20 |
jd__ | that's my birthday | 08:20 |
sileht | not ALTER/CREATE | 08:20 |
jd__ | :P | 08:20 |
sileht | jd__, or it should reject the ALTER if one row have NULL | 08:23 |
sileht | but in our case we don't have any with NULL... | 08:23 |
sileht | any/any row | 08:23 |
*** shardy is now known as shardy_afk | 08:26 | |
*** afazekas has quit IRC | 08:30 | |
*** liusheng has quit IRC | 08:31 | |
*** liusheng has joined #openstack-telemetry | 08:31 | |
*** afazekas has joined #openstack-telemetry | 08: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-telemetry | 08: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 shardy | 08: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 amusing | 08: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 +1 | 08:57 |
jd__ | yes = +1 | 08:57 |
jd__ | no = -1 | 08: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 now | 09:00 |
*** openstackgerrit has quit IRC | 09:02 | |
jd__ | ok I just saw failure maybe I need to recheck | 09:03 |
*** openstackgerrit has joined #openstack-telemetry | 09:25 | |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: indexer: fix default of new datetime column https://review.openstack.org/423090 | 09:25 |
*** zyun|3 has joined #openstack-telemetry | 09:33 | |
zyun|3 | hi all | 09:33 |
zyun|3 | may I ask a question? | 09:33 |
zyun|3 | I manually delete the ceilometer db, how could I recovery this? | 09:34 |
sileht | restore your backup ? | 09:34 |
zyun|3 | other method? I want the openstack can help rebuild this db | 09:35 |
zyun|3 | is there any method? | 09:35 |
sileht | lol | 09:35 |
*** efoley has joined #openstack-telemetry | 09:35 | |
sileht | it's a database, we store thing because we can't query the past states of your openstack installation | 09:37 |
jd__ | yeah you wont' have any past data though if you wait you'll have new data being filled | 09:38 |
*** yassine has joined #openstack-telemetry | 09:38 | |
*** yassine is now known as Guest55718 | 09:39 | |
*** lhx__ has quit IRC | 09:45 | |
*** thorst_ has joined #openstack-telemetry | 09:46 | |
glb | hello :) | 09:49 |
glb | I think there is a bug in the way stevedore.extension is used | 09:49 |
glb | https://github.com/openstack/ceilometer/blob/master/ceilometer/agent/manager.py#L295 | 09:49 |
glb | the callback should just ignore the extensions it can not load, right ? ^ | 09:50 |
sileht | glb, that depends on the reason of the failure | 09:51 |
*** thorst_ has quit IRC | 09:51 | |
glb | because it crash with: TypeError: Message objects do not support addition. | 09:52 |
glb | for '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 extension | 09:53 |
*** yprokule has quit IRC | 09:54 | |
glb | sileht: so, it's an expected behavior ? | 09:56 |
sileht | glb, yes | 09:57 |
sileht | glb, you have an extension somewhere that raise 'TypeError: Message objects do not support addition.' | 09:57 |
sileht | glb, we just reraise it | 09:57 |
sileht | glb, we could just improve it code to print a message with the extension name that all | 09:58 |
*** nadya has quit IRC | 09:59 | |
glb | thx for the explanation :), yes it will be nice because i've to add some print() for have this kind of information | 09:59 |
*** hfu has quit IRC | 10:00 | |
*** lhx__ has joined #openstack-telemetry | 10:04 | |
*** nijaba has quit IRC | 10:05 | |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/ceilometer: agent: always print the extension name on failure https://review.openstack.org/423167 | 10:08 |
sileht | glb, that should provide the name of the broken extention: https://review.openstack.org/423167 | 10:09 |
*** lhx_ has joined #openstack-telemetry | 10:09 | |
*** lhx__ has quit IRC | 10:10 | |
sileht | jd__, I have no two issue to fix the mysql bug | 10:10 |
sileht | jd__, I don't known of the set a different server_default for mysql and postgres | 10:11 |
glb | sileht, thanks | 10:11 |
sileht | jd__, and the sql migration testing don't work well with custom type and server_default | 10:11 |
*** nijaba has joined #openstack-telemetry | 10:12 | |
*** nijaba has quit IRC | 10:12 | |
*** nijaba has joined #openstack-telemetry | 10: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 IRC | 10:22 | |
lhx_ | jd__, I thought if I just need to change the devstack configure | 10:23 |
lhx_ | ? | 10:23 |
jd__ | lhx_: you have to but… do you think people deploy with devstack? | 10:24 |
openstackgerrit | Sanu M proposed openstack/python-aodhclient: Removes unused utf-8 declaration https://review.openstack.org/423181 | 10:24 |
*** gongysh has joined #openstack-telemetry | 10:28 | |
openstackgerrit | Merged openstack/aodh: api: add auth_mode option https://review.openstack.org/421454 | 10:30 |
lhx_ | jd__, if people deploy without devstack, they can choose to the default package to be installed | 10:30 |
lhx_ | lhx__, does it? | 10:30 |
lhx_ | jd__, sorry ping wrong | 10:31 |
*** dschultz has quit IRC | 10:32 | |
jd__ | lhx_: ok so by default let's say the collector package is not installed anymore by distribution since it's deprecated | 10: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 devstack | 10:34 |
jd__ | lhx_: change the default configuration not-in-devstack? | 10:34 |
jd__ | lhx_: by default everything publishes to the collector | 10: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 devstack | 10:37 |
*** gongysh has quit IRC | 10:37 | |
jd__ | lhx_: that too it should be off by default | 10:37 |
zyun|3 | Hello | 10:39 |
zyun|3 | Where is the log file for ceilometer-dbsync cmd? | 10:39 |
jd__ | zyun|3: on your screen in theory | 10:40 |
zyun|3 | ok | 10:40 |
zyun|3 | and I encounter below error when running this cmd? who can give me some clue? | 10:41 |
*** Jack_Iv has joined #openstack-telemetry | 10:41 | |
zyun|3 | http://paste.openstack.org/show/595758/ | 10:41 |
zyun|3 | Many many thanks | 10:41 |
zyun|3 | what error in my env? | 10:44 |
zyun|3 | who can help me out? Thanks in advance | 10:51 |
lhx_ | jd__, do I just change the pipeline.yaml? | 10:51 |
jd__ | lhx_: is it enough? | 10:52 |
jd__ | lhx_: I think so | 10:52 |
jd__ | zyun|3: what version is this btw? | 10:52 |
lhx_ | jd__, so I just change it with gnocchi://? | 10:53 |
zyun|3 | it is kilo release | 10:53 |
jd__ | zyun|3: did you install another version at some point? it's weird | 10:53 |
jd__ | lhx_: sounds good | 10:53 |
jd__ | lhx_: do you want me to write the patch? :P | 10:53 |
lhx_ | jd__, no no no, lol | 10:54 |
zyun|3 | no, I only has this release | 10:54 |
jd__ | zyun|3: you configured it for mongodb right? | 10:54 |
zyun|3 | yes | 10:54 |
*** zyun|3 has quit IRC | 11:05 | |
sileht | jd__, I don't find a solution for the mysql bug ... | 11:06 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: indexer: fix default of new datetime column https://review.openstack.org/423090 | 11:10 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: indexer: fix datetime with mysql >= 5.7.17 https://review.openstack.org/423090 | 11:11 |
* sileht tries a ugly trick | 11:12 | |
*** Jack_Iv has quit IRC | 11:12 | |
*** Jack_Iv has joined #openstack-telemetry | 11:14 | |
jd__ | sileht: what's the problem? | 11:16 |
sileht | jd__, I can get something working | 11:17 |
sileht | jd__, changing server_default is just a nightmare | 11:17 |
sileht | jd__, server_default is a raw string passed to the server | 11:19 |
sileht | jd__, and mysql need a datetime when postgresql a timestamp | 11:20 |
openstackgerrit | Hanxi Liu proposed openstack/ceilometer: Deprecate collector https://review.openstack.org/413920 | 11:25 |
openstackgerrit | Merged openstack/ceilometer: pipeline: remove tests helper from runtime code https://review.openstack.org/420636 | 11:27 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: indexer: fix datetime with mysql >= 5.7.17 https://review.openstack.org/423090 | 11:32 |
sileht | jd__, I have found for the postgresql pb | 11:32 |
sileht | jd__, for mysql it fails to compare the col type: http://paste.openstack.org/show/595766/ | 11:32 |
*** Jack_Iv has quit IRC | 11:46 | |
*** thorst_ has joined #openstack-telemetry | 11:47 | |
*** yprokule has joined #openstack-telemetry | 11:47 | |
jd__ | sileht: :/ | 11:50 |
jd__ | sileht: can you if based on the engine? | 11:50 |
*** nadya has joined #openstack-telemetry | 11:50 | |
*** thorst_ has quit IRC | 11:52 | |
sileht | jd__, no I haven't found that | 11:52 |
jd__ | sileht: ask zzzeeek maybe | 11:52 |
sileht | jd__, this is due to our custome types | 11:52 |
jd__ | sileht: the datetime one? | 11:53 |
sileht | jd__, yes | 11:53 |
jd__ | hm | 11:53 |
jd__ | maybe we can use a different trick to have fsp=6 | 11:53 |
sileht | jd__, I have found a trick ! | 11:53 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: indexer: fix datetime with mysql >= 5.7.17 https://review.openstack.org/423090 | 11:53 |
jd__ | just now? | 11:53 |
sileht | jd__, if I pass the literal string with miliseconds everybody is happy | 11:53 |
openstackgerrit | Hanxi Liu proposed openstack/ceilometer: Ship YAML file to /usr/share https://review.openstack.org/412309 | 11:55 |
jd__ | sileht: damn… | 11:55 |
sileht | jd__, I think that hide a bug somewhere | 11:56 |
jd__ | sileht: ? | 11:58 |
sileht | jd__, 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-294 | 11:59 |
*** pcaruana has quit IRC | 11:59 | |
jd__ | -_- | 11:59 |
*** hfu has joined #openstack-telemetry | 12:01 | |
sileht | by change adding quote work also for postgres and sqlalchemey server_default | 12:02 |
*** hfu has quit IRC | 12:05 | |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: indexer: fix datetime with mysql >= 5.7.17 https://review.openstack.org/423090 | 12:05 |
sileht | jd__, ready to laught: | 12:06 |
sileht | he supported range is '1000-01-01 00:00:00' to '9999-12-31 23:59:59'. | 12:06 |
*** pcaruana has joined #openstack-telemetry | 12:06 | |
* sileht -> lunch | 12:07 | |
jd__ | why 1000 ? lol | 12:08 |
*** lhx_ has quit IRC | 12:11 | |
openstackgerrit | Merged openstack/ceilometer: gnocchi: prepare removal of gnocchi uuid5 https://review.openstack.org/418426 | 12:14 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: indexer: fix datetime with mysql >= 5.7.17 https://review.openstack.org/423090 | 12:22 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: indexer: fix datetime with mysql >= 5.7.17 https://review.openstack.org/423090 | 12:30 |
*** catinthe_ has joined #openstack-telemetry | 12:32 | |
*** catintheroof has quit IRC | 12:33 | |
*** lhx_ has joined #openstack-telemetry | 12:35 | |
*** shardy is now known as shardy_lunch | 12:36 | |
EmilienM | jd__: 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 |
openstackgerrit | Julien Danjou proposed openstack/gnocchi: Remove py{27,35} from valid tox targets https://review.openstack.org/423248 | 12:43 |
jd__ | EmilienM: you're welcome | 12:43 |
*** thorst_ has joined #openstack-telemetry | 12:47 | |
*** hfu has joined #openstack-telemetry | 12:50 | |
*** hfu has quit IRC | 12:53 | |
*** links has quit IRC | 12:55 | |
*** hfu has joined #openstack-telemetry | 12:56 | |
*** hfu has quit IRC | 12:58 | |
*** hfu has joined #openstack-telemetry | 12:59 | |
*** hfu has quit IRC | 13:01 | |
*** nadya has quit IRC | 13:03 | |
*** hfu has joined #openstack-telemetry | 13:03 | |
*** hfu has quit IRC | 13:04 | |
openstackgerrit | Hanxi Liu proposed openstack/ceilometer: Deprecate collector https://review.openstack.org/413920 | 13:07 |
*** shardy_lunch is now known as shardy | 13:14 | |
*** hfu has joined #openstack-telemetry | 13:21 | |
EmilienM | jd__: so we are not useful? lol | 13:22 |
*** shardy has quit IRC | 13:27 | |
*** shardy has joined #openstack-telemetry | 13:27 | |
*** hfu has quit IRC | 13:32 | |
*** vint_bra has joined #openstack-telemetry | 13:52 | |
*** nicodemus_ has joined #openstack-telemetry | 13:55 | |
openstackgerrit | gordon chung proposed openstack/gnocchi: serialise: simplify array format https://review.openstack.org/422800 | 13:58 |
glb | is there any way to disable some extension like NodesDiscoveryTripleO in ceilometer configuration ? (I found nothing talking about that in doc ...) | 13:58 |
sileht | glb, why do you want to do that ? If the extension doesn't load correctly this is a bug :) | 14:00 |
openstackgerrit | Hanxi Liu proposed openstack/ceilometer: Deprecate collector https://review.openstack.org/413920 | 14:00 |
nicodemus_ | hello | 14:03 |
*** hfu has joined #openstack-telemetry | 14:06 | |
*** dave-mccowan has joined #openstack-telemetry | 14: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 keystone | 14:07 |
nicodemus_ | I was trying to compare it with ceilometer code for example, that has a keystone_client.py | 14:08 |
glb | sileht: 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 |
glb | maybe requirement.txt need to be fixed ^ | 14:09 |
*** Jack_Iv has joined #openstack-telemetry | 14:10 | |
sileht | glb, are your mixing openstack project version ? | 14:10 |
glb | no I try to be consistent in version, and I use venv for each project | 14:12 |
*** fguillot has joined #openstack-telemetry | 14:13 | |
openstackgerrit | gordon chung proposed openstack/ceilometer: polling definition file https://review.openstack.org/405682 | 14:20 |
openstackgerrit | gordon chung proposed openstack/ceilometer: add polling.yaml docs https://review.openstack.org/407711 | 14:21 |
*** tlian has joined #openstack-telemetry | 14:23 | |
*** hfu has quit IRC | 14:30 | |
jd__ | EmilienM: what for? :) | 14:31 |
jd__ | nicodemus_: it's in api-paste.ini that keystone is imported and used | 14:32 |
EmilienM | jd__: I read your candidacy ;-) | 14:33 |
*** catintheroof has joined #openstack-telemetry | 14:33 | |
*** catinthe_ has quit IRC | 14:36 | |
*** Jack_Iv has quit IRC | 14: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_client | 14:38 |
nicodemus_ | that makes sense? I'm just trying to understand | 14:39 |
*** liusheng has quit IRC | 14:40 | |
*** liusheng has joined #openstack-telemetry | 14:40 | |
*** vint_bra has quit IRC | 14:45 | |
*** sheel has quit IRC | 14:47 | |
sileht | nicodemus_, yes | 14:53 |
nicodemus_ | sileht, got it. thanks! | 14:53 |
*** rbak has joined #openstack-telemetry | 14:55 | |
jd__ | EmilienM: the "we" of the TC? no you're completely useless, sorry :) | 14:55 |
jd__ | nicodemus_: exactly | 14:56 |
openstackgerrit | Merged openstack/ceilometer: agent: always print the extension name on failure https://review.openstack.org/423167 | 14:58 |
nicodemus_ | jd__, thanks! | 14:59 |
openstackgerrit | Merged openstack/panko: remove ceilometer/meter specific hbase code https://review.openstack.org/418603 | 15:17 |
lhx_ | jd__, I think we have to choose database(default direct publisher) by default | 15:18 |
lhx_ | https://review.openstack.org/#/c/413920/ | 15:18 |
lhx_ | if not, the tests will fail | 15:21 |
jd__ | lhx_: fix the test then? :) | 15:22 |
lhx_ | jd__, hum..., sure? | 15:23 |
*** gordc has joined #openstack-telemetry | 15:26 | |
openstackgerrit | Merged openstack/ceilometer: Fix same type of simple message error https://review.openstack.org/422590 | 15: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 say | 15:28 |
jd__ | lhx_: you can't remove the database tests but they should not use the collector anyway | 15:29 |
lhx_ | jd__, they use direct publisher(database), not collector | 15:31 |
jd__ | lhx_: so they don't really care about changing the pipeline.yaml? | 15:31 |
*** dschultz has joined #openstack-telemetry | 15:33 | |
lhx_ | jd__, who are "they"? | 15:34 |
jd__ | lhx_: the database tests | 15: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 tests | 15:37 |
*** dschultz has quit IRC | 15:38 | |
lhx_ | jd__, actually they care from my point of view | 15:38 |
openstackgerrit | Merged openstack/gnocchi: indexer: fix datetime with mysql >= 5.7.17 https://review.openstack.org/423090 | 15: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 IRC | 15:48 | |
*** shardy has joined #openstack-telemetry | 15:49 | |
jd__ | lhx_: may the force be with you | 15:57 |
openstackgerrit | Julien Danjou proposed openstack/gnocchi: Remove non-existent -all suffix in tox targets https://review.openstack.org/423343 | 15:57 |
lhx_ | jd__, Star Wars! lol | 15:59 |
*** pcaruana has quit IRC | 16:03 | |
*** joadavis_ has joined #openstack-telemetry | 16:04 | |
*** dave-mccowan has quit IRC | 16:29 | |
*** lhx_ has quit IRC | 16:29 | |
*** lhx_ has joined #openstack-telemetry | 16:30 | |
*** lhx_ has quit IRC | 16:30 | |
*** sudipto has joined #openstack-telemetry | 16:32 | |
*** sudipto_ has joined #openstack-telemetry | 16:32 | |
*** zhangguoqing has quit IRC | 16:36 | |
*** efoley has quit IRC | 16:42 | |
*** nadya has joined #openstack-telemetry | 16:47 | |
jd__ | sileht: one of your patch has a real bug on carbonara as gordc said | 16:54 |
jd__ | just in case you missed | 16:54 |
openstackgerrit | Merged openstack/python-gnocchiclient: Stop always encoding resource https://review.openstack.org/413145 | 16:59 |
gordc | the hell. that merged really quick | 17:01 |
*** fguillot has quit IRC | 17:04 | |
*** glb has quit IRC | 17:05 | |
*** fguillot has joined #openstack-telemetry | 17:20 | |
sileht | jd__, yeah but I wait that most of the patches are merged before doing git-review to not reset the gate queues | 17:29 |
jd__ | k | 17:30 |
openstackgerrit | Merged openstack/aodh: install-guide: remove useless step https://review.openstack.org/421455 | 17:36 |
openstackgerrit | Merged openstack/aodh: Move api-paste file to common code location https://review.openstack.org/421456 | 17:36 |
openstackgerrit | Merged openstack/aodh: Move policy.json out of etc https://review.openstack.org/422220 | 17:36 |
openstackgerrit | Merged openstack/gnocchi: indexer: fix migration script "no_more_slash" https://review.openstack.org/421900 | 17:44 |
openstackgerrit | Merged openstack/gnocchi: rest: returns orignal resource id https://review.openstack.org/417836 | 17:44 |
*** nadya has quit IRC | 17: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 -> created | 17:54 |
nicodemus_ | specificalle, I'd like to provide a custom value to 'original_resource_id' | 17:54 |
nicodemus_ | specifically | 17:54 |
gordc | i 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-telemetry | 18: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 resource | 18: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 ID | 18:09 |
*** dschultz has joined #openstack-telemetry | 18:13 | |
gordc | nicodemus_: 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 |
gordc | it 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 IRC | 18: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 well | 18:21 |
*** nadya has joined #openstack-telemetry | 18:22 | |
*** nadya has quit IRC | 18:22 | |
*** jefrite has quit IRC | 18:40 | |
sileht | nicodemus_, you should not rely on how id are generated by gnocchi that can change over the time | 18:41 |
*** sudipto has quit IRC | 18:41 | |
*** sudipto_ has quit IRC | 18:41 | |
sileht | nicodemus_, I think all API call take the uuid and original_resource_id as id | 18:41 |
sileht | You can use http://localhost/v1/resource/generic/75c44741-cc60-4033-804e-2d3098c7d2e9 or http://localhost/v1/resource/generic/<my_original_resource_id> | 18:42 |
sileht | both should work | 18:42 |
nicodemus_ | sileht, that's good news indeed, if we can query for the original_resource_id | 18:43 |
*** yprokule has quit IRC | 18:45 | |
sileht | nicodemus_, if some API calls doesn't do the conversion, this is perhaps a bug | 18:47 |
openstackgerrit | Merged openstack/gnocchi: add metricd tester for profiling https://review.openstack.org/422446 | 18:48 |
nicodemus_ | sileht, let me check just in case... | 18:48 |
*** jefrite has joined #openstack-telemetry | 18:50 | |
openstackgerrit | Merged openstack/gnocchi: carbonara: prepare datetime for pandas.to_datetime() https://review.openstack.org/421208 | 18:53 |
openstackgerrit | Merged openstack/gnocchi: carbonara: remove a pandas.iteritems() https://review.openstack.org/421516 | 18:53 |
openstackgerrit | Merged openstack/gnocchi: carbonara: handle timestamps from struct with numpy https://review.openstack.org/421570 | 18:54 |
openstackgerrit | Merged openstack/gnocchi: Remove py{27,35} from valid tox targets https://review.openstack.org/423248 | 18:54 |
openstackgerrit | Merged openstack/gnocchi: Remove non-existent -all suffix in tox targets https://review.openstack.org/423343 | 18:54 |
nicodemus_ | sileht, looks like all the api calls do the conversion :) thanks, I'll do it that way | 18:58 |
openstackgerrit | Merged openstack/gnocchi: carbonara: Don't use clean_ts() https://review.openstack.org/421714 | 19:06 |
openstackgerrit | Merged openstack/gnocchi: carbonara: use numpy for serialization https://review.openstack.org/421766 | 19:06 |
openstackgerrit | Merged openstack/gnocchi: carbonara: use numpy for unserialization https://review.openstack.org/421901 | 19:09 |
openstackgerrit | Merged openstack/gnocchi: carbonara: Add benchmark for split() https://review.openstack.org/422496 | 19:11 |
openstackgerrit | Merged openstack/gnocchi: utils: allow ResourceUUID to convert UUID https://review.openstack.org/413014 | 19:12 |
*** Guest55718 has quit IRC | 19:26 | |
*** Jack_Iv has joined #openstack-telemetry | 19:28 | |
*** tesseract has quit IRC | 19:32 | |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: carbonara: add bench for resample https://review.openstack.org/422877 | 19:37 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: carbonara: add merge() benchmark https://review.openstack.org/422637 | 19:37 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: carbonara: don't use groupby for split https://review.openstack.org/422445 | 19:37 |
openstackgerrit | Julien Danjou proposed openstack/gnocchi: carbonara: don't use groupby for split https://review.openstack.org/422445 | 19:49 |
openstackgerrit | Julien Danjou proposed openstack/gnocchi: carbonara: add merge() benchmark https://review.openstack.org/422637 | 19:49 |
openstackgerrit | Julien Danjou proposed openstack/gnocchi: serialise: simplify array format https://review.openstack.org/422800 | 19:50 |
*** catinthe_ has joined #openstack-telemetry | 19:50 | |
*** catintheroof has quit IRC | 19:53 | |
*** Jack_Iv has quit IRC | 19:55 | |
*** Jack_Iv has joined #openstack-telemetry | 19:59 | |
*** jefrite has quit IRC | 20:12 | |
*** jefrite has joined #openstack-telemetry | 20:18 | |
*** jefrite has quit IRC | 20:25 | |
jd__ | sileht: http://logs.openstack.org/96/421096/2/check/gate-python-aodhclient-python27-ubuntu-xenial/3105fb7/console.html | 20:26 |
jd__ | I really think i'm gonna kick oslo out of telemetry sooner or later | 20:26 |
*** jefrite has joined #openstack-telemetry | 20:33 | |
*** dave-mccowan has joined #openstack-telemetry | 20:34 | |
gordc | jd__: stop copying swift | 20:36 |
jd__ | gordc: they got it right :( | 20:36 |
gordc | lol just leave openstack. do it right. :P | 20:37 |
*** gordc has quit IRC | 20:42 | |
openstackgerrit | Julien Danjou proposed openstack/python-aodhclient: Switch to extras usage for test requirements https://review.openstack.org/423515 | 20:44 |
*** jmlowe has joined #openstack-telemetry | 20:48 | |
*** nicodemus_ has quit IRC | 21:02 | |
*** catinthe_ has quit IRC | 21:02 | |
*** catintheroof has joined #openstack-telemetry | 21:03 | |
*** catintheroof has quit IRC | 21:07 | |
openstackgerrit | Merged openstack/gnocchi: carbonara: don't use groupby for split https://review.openstack.org/422445 | 21:13 |
openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: fix compat search on created_by_project_id https://review.openstack.org/423522 | 21:14 |
*** Jack_Iv has quit IRC | 21:17 | |
*** jmlowe has quit IRC | 21:29 | |
openstackgerrit | Pradeep Kilambi proposed openstack/aodh: Add sem-ver flag so pbr generates correct version https://review.openstack.org/423527 | 21:38 |
*** gordc has joined #openstack-telemetry | 21:41 | |
*** thorst_ has quit IRC | 22:02 | |
*** thorst_ has joined #openstack-telemetry | 22:03 | |
*** thorst_ has quit IRC | 22:07 | |
*** shardy has quit IRC | 22:22 | |
*** thorst_ has joined #openstack-telemetry | 22:43 | |
*** dave-mccowan has quit IRC | 22:46 | |
*** thorst_ has quit IRC | 22:47 | |
*** masber has quit IRC | 22:59 | |
*** masber has joined #openstack-telemetry | 23:00 | |
*** thorst_ has joined #openstack-telemetry | 23:15 | |
openstackgerrit | Merged openstack/aodh: Add sem-ver flag so pbr generates correct version https://review.openstack.org/423527 | 23:16 |
*** thorst_ has quit IRC | 23:20 | |
*** gordc has quit IRC | 23:37 | |
*** rbak has quit IRC | 23:49 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!