| *** r-mibu has quit IRC | 00:05 | |
| *** harlowja_ has joined #openstack-telemetry | 00:11 | |
| *** harlowja has quit IRC | 00:15 | |
| *** r-mibu has joined #openstack-telemetry | 00:20 | |
| *** sheeprine has quit IRC | 00:22 | |
| *** nadya has joined #openstack-telemetry | 00:22 | |
| *** diogogmt has quit IRC | 00:23 | |
| *** sheeprine has joined #openstack-telemetry | 00:26 | |
| *** nadya has quit IRC | 00:27 | |
| *** diogogmt has joined #openstack-telemetry | 00:27 | |
| *** thorst_ has joined #openstack-telemetry | 00:27 | |
| *** donghao has quit IRC | 00:29 | |
| *** julim has joined #openstack-telemetry | 00:35 | |
| *** diogogmt has quit IRC | 00:36 | |
| *** julim has quit IRC | 00:49 | |
| openstackgerrit | liusheng proposed openstack/aodh: Clean unrelated error of two tests https://review.openstack.org/315396 | 00:58 |
|---|---|---|
| *** cheneydc has joined #openstack-telemetry | 00:59 | |
| *** Kevin_Zheng has joined #openstack-telemetry | 01:03 | |
| openstackgerrit | liusheng proposed openstack/aodh: Update the home-page with developer documentation https://review.openstack.org/318395 | 01:11 |
| *** diogogmt has joined #openstack-telemetry | 01:14 | |
| *** thorst_ has quit IRC | 01:42 | |
| *** thorst_ has joined #openstack-telemetry | 01:42 | |
| *** thorst_ has quit IRC | 01:51 | |
| *** ljxiash has joined #openstack-telemetry | 01:54 | |
| *** pradk has joined #openstack-telemetry | 01:59 | |
| openstackgerrit | Merged openstack/python-ceilometerclient: Update the home-page with developer documentation https://review.openstack.org/317231 | 02:02 |
| *** pradk_ has joined #openstack-telemetry | 02:06 | |
| *** liamji has joined #openstack-telemetry | 02:09 | |
| *** vishwanathj has quit IRC | 02:09 | |
| openstackgerrit | gordon chung proposed openstack/gnocchi: enable pagination when querying metrics https://review.openstack.org/318409 | 02:10 |
| *** pradk has quit IRC | 02:10 | |
| *** pradk_ is now known as pradk | 02:10 | |
| *** vinaypotluri has quit IRC | 02:10 | |
| *** nadya has joined #openstack-telemetry | 02:23 | |
| *** llu has joined #openstack-telemetry | 02:25 | |
| *** nadya has quit IRC | 02:27 | |
| *** links has joined #openstack-telemetry | 02:35 | |
| *** thorst_ has joined #openstack-telemetry | 02:48 | |
| *** thorst_ has quit IRC | 02:57 | |
| *** vinaypotluri has joined #openstack-telemetry | 03:04 | |
| openstackgerrit | liyuanzhen proposed openstack/python-aodhclient: Add explanation for how to get meter name https://review.openstack.org/317928 | 03:05 |
| *** ljxiash has quit IRC | 03:41 | |
| *** thorst_ has joined #openstack-telemetry | 03:54 | |
| *** ljxiash has joined #openstack-telemetry | 03:56 | |
| *** ljxiash has quit IRC | 04:00 | |
| *** links has quit IRC | 04:00 | |
| *** thorst_ has quit IRC | 04:01 | |
| *** chlong has joined #openstack-telemetry | 04:04 | |
| *** flwang has quit IRC | 04:04 | |
| *** flwang has joined #openstack-telemetry | 04:11 | |
| *** links has joined #openstack-telemetry | 04:16 | |
| *** nadya has joined #openstack-telemetry | 04:23 | |
| *** nadya has quit IRC | 04:28 | |
| *** ljxiash has joined #openstack-telemetry | 04:53 | |
| *** thorst_ has joined #openstack-telemetry | 04:59 | |
| *** flwang1 has quit IRC | 05:00 | |
| *** thorst_ has quit IRC | 05:06 | |
| *** cheneydc has quit IRC | 05:07 | |
| openstackgerrit | Merged openstack/python-aodhclient: Update the home-page with developer documentation https://review.openstack.org/317225 | 05:07 |
| *** cheneydc has joined #openstack-telemetry | 05:10 | |
| *** chlong has quit IRC | 05:17 | |
| *** nadya has joined #openstack-telemetry | 05:20 | |
| *** yprokule has joined #openstack-telemetry | 05:23 | |
| *** chlong has joined #openstack-telemetry | 05:30 | |
| *** apoorv has joined #openstack-telemetry | 05:57 | |
| *** thorst_ has joined #openstack-telemetry | 06:04 | |
| *** chlong has quit IRC | 06:05 | |
| *** nadya has quit IRC | 06:10 | |
| *** thorst_ has quit IRC | 06:10 | |
| *** nadya has joined #openstack-telemetry | 06:10 | |
| *** rcernin has joined #openstack-telemetry | 06:13 | |
| *** nadya has quit IRC | 06:15 | |
| *** chlong has joined #openstack-telemetry | 06:22 | |
| *** flwang1 has joined #openstack-telemetry | 06:25 | |
| *** tomoiaga has joined #openstack-telemetry | 06:39 | |
| *** nadya has joined #openstack-telemetry | 06:39 | |
| *** belmoreira has joined #openstack-telemetry | 06:45 | |
| *** jmccarthy has left #openstack-telemetry | 06:50 | |
| *** LanceHaig has joined #openstack-telemetry | 06:51 | |
| *** chlong has quit IRC | 06:54 | |
| *** nadya has quit IRC | 07:01 | |
| *** nadya has joined #openstack-telemetry | 07:01 | |
| *** nadya has quit IRC | 07:06 | |
| *** jmccarthy has joined #openstack-telemetry | 07:06 | |
| *** jmccarthy has left #openstack-telemetry | 07:07 | |
| *** thorst_ has joined #openstack-telemetry | 07:08 | |
| *** thorst_ has quit IRC | 07:15 | |
| *** flwang1 has quit IRC | 07:20 | |
| *** donghao has joined #openstack-telemetry | 07:30 | |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Add grenade plugin https://review.openstack.org/306329 | 07:37 |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: devstack: remove support for old devstack ceph plugin https://review.openstack.org/315994 | 07:37 |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: gate: remove old job conf https://review.openstack.org/315909 | 07:37 |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: Make tempest tests compatible with keystone v3 https://review.openstack.org/315991 | 07:37 |
| *** flwang1 has joined #openstack-telemetry | 07:39 | |
| *** fwdit has joined #openstack-telemetry | 07:40 | |
| *** vinaypotluri has quit IRC | 07:40 | |
| *** shardy has joined #openstack-telemetry | 08:05 | |
| jd__ | I have an UUID collision in the test | 08:13 |
| jd__ | how unlikely | 08:14 |
| *** thorst_ has joined #openstack-telemetry | 08:14 | |
| sileht | jd__, you should play loto today | 08:15 |
| jd__ | I'm suspicious… | 08:16 |
| jd__ | is it like really possible? | 08:16 |
| jd__ | http://logs.openstack.org/07/305307/25/check/gate-gnocchi-tox-db-py34-postgresql/6dd2710/console.html#_2016-05-18_16_36_18_217 | 08:17 |
| jd__ | i'll recheck, we'll see | 08:19 |
| sileht | jd__, 1/10^36 possibilities ? | 08:19 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: retry on PostgreSQL catalog errors too https://review.openstack.org/316909 | 08:19 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: avoid deadlock on list_metrics() https://review.openstack.org/317893 | 08:19 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: retry on deadlocks in get_resource() https://review.openstack.org/318044 | 08:19 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: retry on deadlock for create_metric() https://review.openstack.org/317546 | 08:19 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: tests: create common resources at class init time https://review.openstack.org/315960 | 08:19 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: tests: remove database synchronization code on upgrade https://review.openstack.org/315961 | 08:19 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: tests: move custom agg setup code in the tests using it https://review.openstack.org/315958 | 08:19 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: retry on deadlock for create_resource_type() https://review.openstack.org/318051 | 08:19 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: tests: remove skip_archive_policies_creation https://review.openstack.org/315959 | 08:19 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: add retry on deadlock for delete_resource() https://review.openstack.org/315957 | 08:19 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: set max_retries & all when retrying https://review.openstack.org/315940 | 08:19 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: tests: do not create legacy resources https://review.openstack.org/305307 | 08:19 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: retry on deadlock for create_resource() https://review.openstack.org/316365 | 08:19 |
| *** thorst_ has quit IRC | 08:21 | |
| *** nadya has joined #openstack-telemetry | 08:29 | |
| *** nadya has quit IRC | 08:34 | |
| *** liusheng has quit IRC | 08:34 | |
| *** liusheng has joined #openstack-telemetry | 08:35 | |
| *** nadya has joined #openstack-telemetry | 08:39 | |
| *** yassine has joined #openstack-telemetry | 08:40 | |
| *** yassine is now known as Guest86061 | 08:40 | |
| *** ljxiash has quit IRC | 08:42 | |
| *** ljxiash has joined #openstack-telemetry | 08:42 | |
| EmilienM | jd__, sileht: hi guys, could you please review this backport? https://review.openstack.org/#/c/318503/ | 08:45 |
| jd__ | EmilienM: sure | 08:45 |
| jd__ | EmilienM: so that solved your problem in the end? cool | 08:46 |
| jd__ | approved | 08:46 |
| EmilienM | jd__: yeah ! | 08:46 |
| EmilienM | awesome ! | 08:46 |
| EmilienM | jd__: http://logs.openstack.org/20/316520/5/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/4359dc4/console.html#_2016-05-19_08_05_21_855 | 08:46 |
| EmilienM | we have tempest tests working again for Ceilo | 08:46 |
| jd__ | great | 08:47 |
| *** ljxiash has quit IRC | 08:47 | |
| *** cdent has joined #openstack-telemetry | 08:48 | |
| openstackgerrit | He Xin proposed openstack/python-ceilometerclient: Correct the usage of the method assertEqual() https://review.openstack.org/317930 | 08:54 |
| EmilienM | jd__: ubuntu guys are asking when you plan to release a new ceilometer/mitaka | 08:56 |
| jd__ | I think there's a review for that EmilienM and that it does not depend on us but on the release team | 08:57 |
| jd__ | ah no that's for liberty | 08:58 |
| jd__ | then, I don't know, we could do anytime now | 08:58 |
| jd__ | EmilienM: I can do it after your fix is landed | 08:58 |
| EmilienM | ok | 08:58 |
| EmilienM | jd__: another thing | 08:58 |
| EmilienM | jd__: https://github.com/openstack/tempest/commit/e4796f8de446aaaafa83902d1fb2d613331436cf | 08:58 |
| EmilienM | this famous patch | 08:58 |
| EmilienM | I think it broke our gate on Liberty because there is no longer tests in Tempest for ceilometer | 08:58 |
| EmilienM | and the in-tree plugin was introduced after Liberty IIRC | 08:59 |
| openstackgerrit | Merged openstack/aodh: Update the home-page with developer documentation https://review.openstack.org/318395 | 08:59 |
| jd__ | likely indeed EmilienM | 08:59 |
| EmilienM | I'm not sure about that | 08:59 |
| EmilienM | yeah I'm sure now, I haven't found any tempest in ceilo/liberty | 08:59 |
| jd__ | that's a problem due to the old tempest design indeed | 08:59 |
| jd__ | I'm not sure what the solution is | 08:59 |
| jd__ | revert this patch until liberty is EOL? | 08:59 |
| EmilienM | I'm going to propose it | 09:00 |
| EmilienM | we can also disable testing for ceilomter in our liberty gate | 09:01 |
| EmilienM | but well... | 09:01 |
| EmilienM | disabling testing is not really what we want... | 09:01 |
| jd__ | agreed | 09:01 |
| EmilienM | jd__: https://review.openstack.org/#/c/318519/ | 09:04 |
| *** LanceHaig has quit IRC | 09:13 | |
| *** LanceHaig has joined #openstack-telemetry | 09:13 | |
| openstackgerrit | He Xin proposed openstack/python-ceilometerclient: Correct the usage of the method assertEqual() https://review.openstack.org/317930 | 09:16 |
| *** thorst_ has joined #openstack-telemetry | 09:19 | |
| *** thorst_ has quit IRC | 09:26 | |
| openstackgerrit | Merged openstack/gnocchi: devstack: remove support for old devstack ceph plugin https://review.openstack.org/315994 | 09:26 |
| *** zqfan has joined #openstack-telemetry | 09:30 | |
| openstackgerrit | Merged openstack/gnocchi: doc: add a glossary https://review.openstack.org/315113 | 09:46 |
| openstackgerrit | liusheng proposed openstack/aodh: Add a tool for migrating alarms data from NoSQL to SQL https://review.openstack.org/309915 | 09:47 |
| *** donghao has quit IRC | 09:48 | |
| *** cheneydc has quit IRC | 10:00 | |
| *** Novegin has joined #openstack-telemetry | 10:15 | |
| Novegin | Hi | 10:15 |
| Novegin | Any one knows how to delay the instances creation | 10:16 |
| *** cdent has quit IRC | 10:21 | |
| openstackgerrit | liyuanzhen proposed openstack/python-aodhclient: Add unit test to validate composite alarm args https://review.openstack.org/318558 | 10:37 |
| *** apoorv has quit IRC | 10:47 | |
| *** apoorv has joined #openstack-telemetry | 10:53 | |
| *** cdent has joined #openstack-telemetry | 10:55 | |
| *** thorst_ has joined #openstack-telemetry | 11:27 | |
| *** cdent has quit IRC | 11:38 | |
| *** gordc has joined #openstack-telemetry | 11:41 | |
| *** fwdit has quit IRC | 11:45 | |
| *** fwdit has joined #openstack-telemetry | 11:46 | |
| *** Novegin has quit IRC | 11:47 | |
| *** liamji has quit IRC | 12:00 | |
| *** cdent has joined #openstack-telemetry | 12:05 | |
| *** ljxiash has joined #openstack-telemetry | 12:15 | |
| jd__ | sileht: looks like ceph is broken on gnocchi stable/2.1, but it looks like a devstack issue (lol), you know something about that? | 12:17 |
| *** ljxiash has quit IRC | 12:19 | |
| *** apoorv has quit IRC | 12:25 | |
| sileht | jd__, do you have a link ? | 12:30 |
| jd__ | sure | 12:30 |
| jd__ | sileht: http://logs.openstack.org/81/317081/1/check/gate-gnocchi-dsvm-functional-ceph-mysql/fa7e6fb/ | 12:30 |
| jd__ | from https://review.openstack.org/#/c/317081/ | 12:30 |
| sileht | jd__, I have missed to backport some change to stable branches before jobs switch | 12:31 |
| *** julim has joined #openstack-telemetry | 12:34 | |
| openstackgerrit | Merged openstack/aodh: Clean unrelated error of two tests https://review.openstack.org/315396 | 12:34 |
| sileht | jd__, https://review.openstack.org/#/c/318616/ | 12:34 |
| *** links has quit IRC | 12:36 | |
| jd__ | sileht: ok, we miss the cherry-pick tag though :] | 12:37 |
| jd__ | sileht: i'll edit | 12:38 |
| *** _prad_ has joined #openstack-telemetry | 12:40 | |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: retry on PostgreSQL catalog errors too https://review.openstack.org/316909 | 12:44 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: avoid deadlock on list_metrics() https://review.openstack.org/317893 | 12:44 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: retry on deadlocks in get_resource() https://review.openstack.org/318044 | 12:44 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: retry on deadlock for create_metric() https://review.openstack.org/317546 | 12:44 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: tests: create common resources at class init time https://review.openstack.org/315960 | 12:44 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: tests: remove database synchronization code on upgrade https://review.openstack.org/315961 | 12:44 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: retry on deadlock for create_resource_type() https://review.openstack.org/318051 | 12:44 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: tests: move custom agg setup code in the tests using it https://review.openstack.org/315958 | 12:44 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: tests: remove skip_archive_policies_creation https://review.openstack.org/315959 | 12:44 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: add retry on deadlock for delete_resource() https://review.openstack.org/315957 | 12:44 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: set max_retries & all when retrying https://review.openstack.org/315940 | 12:44 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: tests: do not create legacy resources https://review.openstack.org/305307 | 12:44 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: retry on deadlock for create_resource() https://review.openstack.org/316365 | 12:44 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: no fail if resources and type are deleted under our feet https://review.openstack.org/318637 | 12:44 |
| jd__ | sileht: you'll like https://review.openstack.org/318637 | 12:44 |
| jd__ | I'm not sure, but it seems to be MySQL specific, probably a transaction thing that differs with PostgreSQL | 12:45 |
| openstackgerrit | liusheng proposed openstack/aodh: Add a tool for migrating alarms data from NoSQL to SQL https://review.openstack.org/309915 | 12:45 |
| gordc | jd__: is there a reason why metric name isn't mandatory in gnocchi? | 12:46 |
| jd__ | gordc: because there's no reason to make it mandatory | 12:47 |
| jd__ | not sure it's a reason :p | 12:47 |
| sileht | jd__, yeah deleting a table are not done in an transaction with mysl | 12:47 |
| * jd__ throws table | 12:47 | |
| gordc | jd__: :) but we have a unique constraint on name+resource_id | 12:47 |
| jd__ | gordc: and? | 12:49 |
| jd__ | you either attach it to a resource or you don't, that's what it means | 12:50 |
| gordc | jd__: i believe oslo.db pagination will fail if you try to sort against a column with NULL in it. | 12:52 |
| gordc | let me find the error | 12:52 |
| jd__ | gordc: fix oslo.db? | 12:52 |
| jd__ | :P | 12:52 |
| gordc | nah. that's too much work. | 12:53 |
| gordc | actually it might be sqlalchemy. | 12:55 |
| gordc | it will fail here https://github.com/openstack/oslo.db/blob/master/oslo_db/sqlalchemy/utils.py#L168-L170 | 12:55 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: tests: create common resources at class init time https://review.openstack.org/315960 | 12:57 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: tests: remove database synchronization code on upgrade https://review.openstack.org/315961 | 12:57 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: tests: remove skip_archive_policies_creation https://review.openstack.org/315959 | 12:57 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: tests: do not create legacy resources https://review.openstack.org/305307 | 12:57 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: add missing constraint delete_resource_type() https://review.openstack.org/318643 | 12:57 |
| *** ljxiash has joined #openstack-telemetry | 13:06 | |
| jd__ | sileht: I finally understand what INFO alembic.runtime.migration [-] Will assume non-transactional DDL. means | 13:09 |
| jd__ | :) | 13:09 |
| jd__ | gordc: yeah so let's fix it | 13:09 |
| jd__ | gordc: wanna help seriously? | 13:09 |
| sileht | jd__, do you plan the replace the message to something clearer ? poor database detected | 13:09 |
| jd__ | sileht: "crazy bastard" | 13:09 |
| jd__ | YOLO database detected | 13:10 |
| openstackgerrit | Julien Danjou proposed openstack/aodh: api: deprecate and disable combination alarms https://review.openstack.org/318052 | 13:11 |
| gordc | jd__: i can fix it. but the question is "is None greater than or less than a non-None value" | 13:12 |
| *** dave-mccowan has joined #openstack-telemetry | 13:12 | |
| gordc | i think that's why sqlalchemy just throws an error when you try x > None | 13:12 |
| *** peristeri has joined #openstack-telemetry | 13:15 | |
| *** nadya has quit IRC | 13:16 | |
| jd__ | gordc: sounds like a philosophical issue | 13:17 |
| jd__ | is the absence of something bigger than anything? | 13:17 |
| *** diogogmt has quit IRC | 13:17 | |
| sileht | gordc, that was working before | 13:17 |
| openstackgerrit | Merged openstack/gnocchi: Backlog status not accurate when batching https://review.openstack.org/318305 | 13:18 |
| gordc | *shrugs* i think sql returns null as smaller than non-null | 13:18 |
| sileht | gordc, you can choice the behavior with "nullsfirst" and "nullslast" | 13:18 |
| *** nadya has joined #openstack-telemetry | 13:20 | |
| gordc | sileht: if you use pagination code in oslo.db, sqlalchemy will throw https://github.com/zzzeek/sqlalchemy/blob/732c613eeb890e7b7cbd04750468dac584151a31/lib/sqlalchemy/sql/default_comparator.py#L53-L55 | 13:20 |
| sileht | I have written the fixes | 13:20 |
| gordc | if it compares something >|< None | 13:20 |
| gordc | don't have the exact error since it's on my machine at home... | 13:21 |
| *** rbak has quit IRC | 13:22 | |
| sileht | gordc, looks like I have written broken code one more times x) | 13:22 |
| gordc | :) i think it might just avoid your code. | 13:23 |
| gordc | i can see the nullslast stuff. | 13:23 |
| sileht | gordc, the fact is that marker have to be a non-null column | 13:23 |
| sileht | gordc, otherwise the whole sort code will not work as expected | 13:23 |
| sileht | gordc, and a uniq column too obviously | 13:24 |
| gordc | sileht: ack. i think the only marker in metric model is id | 13:24 |
| gordc | and i can't write a gabbi test for it. (unless there's a way to save multiple responses in variables?) | 13:25 |
| gordc | cdent: ^ | 13:25 |
| cdent | gordc: que? | 13:25 |
| gordc | can i save multiple responses in gabbi? not just the previous | 13:26 |
| cdent | not really (and on purpose), but there are often ways to accomplish what you want to do. what data are you trying to access? | 13:27 |
| cdent | (if what you're trying to do is get the next page of a set paginated results, then the current page should have a 'next' link, but it sounds like you're trying to do something more complex?) | 13:28 |
| gordc | cdent: i want to create 5 different metrics (5 separate posts) and on the 6th request, i want ot compare response to the ids in previous 5 responses | 13:28 |
| gordc | we didnt' need to do this for resources because we could actually set the id in resources (not so in metrics) | 13:29 |
| cdent | this might not work as you want, but i'd post and get 5 times to confirm each one, and then on the 6th request check the `len` of results | 13:30 |
| cdent | so in the sixth check you are checking that they are all there, but not what they are | 13:30 |
| gordc | cdent: oh. yeah, i want to check the order on 6th req | 13:30 |
| gordc | forgot to mention that. :) | 13:31 |
| cdent | check length and the value of the last one in the list? | 13:31 |
| cdent | (which you presumably know from the previous request?) | 13:31 |
| cdent | if you've got code in progress, show me, maybe I can think of something? | 13:31 |
| openstackgerrit | Julien Danjou proposed openstack/aodh: Replace overtest by pifpaf https://review.openstack.org/318664 | 13:32 |
| gordc | ... i don't want to show you because then it's obvious my code is (arguably) broken | 13:32 |
| gordc | let's say, hypothetically, it's this: https://review.openstack.org/#/c/318409/1/gnocchi/tests/gabbi/gabbits/pagination.yaml | 13:33 |
| openstackgerrit | gordon chung proposed openstack/python-gnocchiclient: add support for update archive policy https://review.openstack.org/307366 | 13:34 |
| *** BigWillie has joined #openstack-telemetry | 13:37 | |
| *** rbak has joined #openstack-telemetry | 13:38 | |
| *** vint_bra has joined #openstack-telemetry | 13:40 | |
| *** ametts has joined #openstack-telemetry | 13:52 | |
| cdent | gordc: sorry, I went away for a minute | 13:55 |
| *** belmoreira has quit IRC | 13:55 | |
| cdent | first off you could move the repeated request_headers into a defaults: , if you wanted | 13:55 |
| *** belmoreira has joined #openstack-telemetry | 13:56 | |
| cdent | and there's some new fancy that you might like | 13:56 |
| cdent | instead of method: post\nurl: /foo/bar you can do POST: /foo/bar | 13:57 |
| cdent | gordc: what's missing from what you've got there? | 13:57 |
| jd__ | sileht: I really hit a mystery with PG | 14:02 |
| jd__ | sileht: I get http://paste.openstack.org/show/497718/ which says that "test_indexer" resource type is already there… except that there is only one test using that as a name for a resource type | 14:04 |
| jd__ | so how that can happen, I'm circumspect | 14:04 |
| *** KrishR has joined #openstack-telemetry | 14:05 | |
| *** vint_bra1 has joined #openstack-telemetry | 14:08 | |
| *** liamji has joined #openstack-telemetry | 14:10 | |
| sileht | jd__, do you known how I can checkout the whole topic from gerrit ? | 14:10 |
| jd__ | sileht: git review -d <id of review> ? | 14:11 |
| sileht | jd__, or what is the top patch of your big branch ? | 14:11 |
| jd__ | sileht: git review -d 315960 | 14:11 |
| sileht | jd__, this one is the HEAD ? https://review.openstack.org/#/c/315958/ | 14:11 |
| sileht | jd__, thx | 14:11 |
| *** vint_bra has quit IRC | 14:12 | |
| jd__ | sileht: while tox -e py27-postgresql-file; do true; done | 14:12 |
| jd__ | :-) | 14:12 |
| jd__ | it'll crash at some point | 14:12 |
| sileht | jd__, I catch it | 14:18 |
| jd__ | cool | 14:20 |
| sileht | jd__, take a looks to https://review.openstack.org/#/c/318051/4 | 14:20 |
| jd__ | sileht: oh fuck yeah | 14:21 |
| jd__ | you're good at that! | 14:21 |
| jd__ | sileht: which is directly in line with https://twitter.com/juldanjou/status/733298684058800128 … | 14:21 |
| jd__ | sileht: I guess you wondered the same | 14:22 |
| *** fwdit has quit IRC | 14:23 | |
| sileht | jd__, thx | 14:24 |
| *** belmoreira has quit IRC | 14:27 | |
| gordc | cdent: nothing's technically missing. it's just that i'm currently sorting on metric.name and in theory that isn't something that you can sort on. (since it's nullable) | 14:28 |
| cdent | gordc: are you trying to test the api with these tests, or are you trying to test something elsewhere in the stack? | 14:29 |
| gordc | both? testing pagination works and you can configure it via api | 14:30 |
| *** diogogmt has joined #openstack-telemetry | 14:33 | |
| *** belmoreira has joined #openstack-telemetry | 14:38 | |
| sileht | jd__, I was looking at what alexis was looking about ceilometer maturity indicator: https://www.openstack.org/software/releases/mitaka/components/ceilometer | 14:38 |
| jd__ | this is lolilol | 14:38 |
| sileht | jd__, some line are set to No, but when you click on details, ceilometer have the tag ... | 14:38 |
| gordc | sileht: yeah, i saw that. it's broken on a lot of projects... | 14:39 |
| sileht | awesome... | 14:40 |
| gordc | i think we should be 7 of 8 | 14:40 |
| jd__ | how is that maintained? | 14:41 |
| gordc | it's suppose to be 'tags'... 'tags' are amazing things... | 14:43 |
| sileht | gordc, you can make 'cloud tags' with them | 14:43 |
| sileht | cloud is so amazing | 14:45 |
| gordc | sileht: it fixes everything. no more war after cloud is ready | 14:46 |
| *** EZPZ has joined #openstack-telemetry | 14:47 | |
| jd__ | sileht: turns out map_and_create…() fails a lot even with retry=10 | 14:53 |
| jd__ | I'm worried | 14:53 |
| sileht | jd__, with PG, delete/create tables lock the whole table resource | 14:54 |
| sileht | jd__, while all other operation lock only a row | 14:54 |
| openstackgerrit | Julien Danjou proposed openstack/aodh: Replace overtest by pifpaf https://review.openstack.org/318664 | 14:55 |
| *** cdent has quit IRC | 14:55 | |
| jd__ | Gonna go to 20 retry with less time | 14:55 |
| *** julim has quit IRC | 14:56 | |
| sileht | jd__, in real life this API should not be public I think due to the huge impact on the BD | 14:56 |
| jd__ | true | 14:56 |
| *** julim has joined #openstack-telemetry | 14:56 | |
| jd__ | it's admin by default anyway, and the uniq name of resource types implies it's not by-user | 14:56 |
| sileht | jd__, if we want something that can be public, we should rewrite the model to not have one table per resource_type that will be less performant | 14:57 |
| jd__ | yeah I don't think it's a good idea | 14:57 |
| sileht | jd__, and will looks like ceilometer metadata model... | 14:57 |
| jd__ | i like what we have :) people can run their own Gnocchi then… | 14:58 |
| *** tomoiaga has quit IRC | 15:02 | |
| *** shardy has quit IRC | 15:07 | |
| *** vinaypotluri has joined #openstack-telemetry | 15:08 | |
| sileht | jd__, I got an idea to workaround the fact that we don't have transaction for DDL | 15:10 |
| jd__ | sileht: 20 retries over 17s and I still got a failure with that transaction -_- | 15:11 |
| jd__ | I am going to start preferring MySQL lol | 15:11 |
| *** links has joined #openstack-telemetry | 15:11 | |
| sileht | jd__, I would add a state for resource_type | 15:11 |
| sileht | jd__, with two values CREATING, CREATED | 15:12 |
| jd__ | are you suggesting we implement a transactional DDL for MySQL? | 15:12 |
| jd__ | SURE WHY NOT | 15:12 |
| sileht | jd__, that will ensure we will not have race between map_and_creates/get_classes | 15:14 |
| *** cdent has joined #openstack-telemetry | 15:15 | |
| sileht | jd__, because list_resource_types should not return resource_type in creation | 15:15 |
| sileht | jd__, currently it does, because the resource_type creation cannot be atomic, sqlalchemy doesn't allow that | 15:16 |
| sileht | jd__, if we do this we can release the process lock that sqlalchemy mappers during resource_type creation/deletion | 15:18 |
| sileht | that/that protect | 15:18 |
| jd__ | sileht: sounds like a good idea I guess | 15:21 |
| sileht | jd__, code is always more readable | 15:22 |
| *** links has quit IRC | 15:26 | |
| *** belmoreira has quit IRC | 15:28 | |
| *** ljxiash has quit IRC | 15:35 | |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: retry on PostgreSQL catalog errors too https://review.openstack.org/316909 | 15:43 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: tests: create common resources at class init time https://review.openstack.org/315960 | 15:43 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: no fail if resources and type are deleted under our feet https://review.openstack.org/318637 | 15:43 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: add missing constraint delete_resource_type() https://review.openstack.org/318643 | 15:43 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: retry on deadlock for create_resource_type() https://review.openstack.org/318051 | 15:43 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: tests: remove skip_archive_policies_creation https://review.openstack.org/315959 | 15:43 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: tests: do not create legacy resources https://review.openstack.org/305307 | 15:43 |
| *** stupidni` is now known as stupidnic | 15:46 | |
| jd__ | sileht: finally won over PG :) you need to reset the connection so I picked a new one and now it's solid | 15:47 |
| sileht | jd__, ... | 15:48 |
| jd__ | would be cool to have session support for create_all() then | 15:48 |
| jd__ | not sure if it's somewhere | 15:48 |
| sileht | jd__, session support will never work for mysql, DDL are non transactionnal | 15:53 |
| sileht | non/not | 15:53 |
| jd__ | yeah but with PG it would | 15:53 |
| sileht | jd__, yeah | 15:53 |
| sileht | jd__, my idea will workaround the issue and perhaps will remove the lock that protect sqlalchemt mapper | 15:54 |
| sileht | jd__, less lock contention should reduce the deadlock occurence | 15:54 |
| jd__ | cool | 15:57 |
| *** nadya has quit IRC | 16:00 | |
| *** julim has quit IRC | 16:02 | |
| *** nadya has joined #openstack-telemetry | 16:03 | |
| *** rcernin has quit IRC | 16:04 | |
| *** nadya has quit IRC | 16:13 | |
| *** vishwanathj has joined #openstack-telemetry | 16:20 | |
| *** lsmola has quit IRC | 16:24 | |
| *** peristeri has quit IRC | 16:26 | |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: WIP: track resource_type creation/deletion state https://review.openstack.org/318768 | 16:28 |
| sileht | jd__, I haven't tested but that the idea: https://review.openstack.org/318768 | 16:29 |
| *** adrian_otto has joined #openstack-telemetry | 16:37 | |
| *** vint_bra has joined #openstack-telemetry | 16:42 | |
| *** vint_bra1 has quit IRC | 16:43 | |
| *** vint_bra1 has joined #openstack-telemetry | 16:43 | |
| *** vint_bra has quit IRC | 16:47 | |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: retry on PostgreSQL catalog errors too https://review.openstack.org/316909 | 16:47 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: tests: create common resources at class init time https://review.openstack.org/315960 | 16:47 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: no fail if resources and type are deleted under our feet https://review.openstack.org/318637 | 16:47 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: add missing constraint delete_resource_type() https://review.openstack.org/318643 | 16:47 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: tests: remove skip_archive_policies_creation https://review.openstack.org/315959 | 16:47 |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: tests: do not create legacy resources https://review.openstack.org/305307 | 16:47 |
| *** vint_bra1 has quit IRC | 16:50 | |
| *** yprokule has quit IRC | 16:56 | |
| *** r-daneel has joined #openstack-telemetry | 16:58 | |
| *** julim has joined #openstack-telemetry | 17:10 | |
| *** nadya has joined #openstack-telemetry | 17:23 | |
| *** liamji has quit IRC | 17:30 | |
| *** jwcroppe_ has quit IRC | 17:33 | |
| *** r-daneel has quit IRC | 17:44 | |
| *** Guest86061 has quit IRC | 17:47 | |
| *** jwcroppe has joined #openstack-telemetry | 17:51 | |
| *** nadya has quit IRC | 18:10 | |
| *** nadya has joined #openstack-telemetry | 18:18 | |
| *** j_king has quit IRC | 18:26 | |
| *** kfox1111 has quit IRC | 18:26 | |
| *** ityaptin has quit IRC | 18:26 | |
| *** j_king has joined #openstack-telemetry | 18:26 | |
| *** kfox1111 has joined #openstack-telemetry | 18:26 | |
| *** ityaptin has joined #openstack-telemetry | 18:26 | |
| *** llu has quit IRC | 18:31 | |
| *** llu has joined #openstack-telemetry | 18:34 | |
| *** efoley has joined #openstack-telemetry | 18:38 | |
| *** rcernin has joined #openstack-telemetry | 18:47 | |
| *** renatoarmani has joined #openstack-telemetry | 18:53 | |
| *** renatoarmani has quit IRC | 18:54 | |
| *** diogogmt has quit IRC | 18:55 | |
| *** nadya has quit IRC | 18:57 | |
| *** diogogmt has joined #openstack-telemetry | 19:12 | |
| *** diogogmt has quit IRC | 19:15 | |
| *** adrian_otto has quit IRC | 19:26 | |
| *** jwcroppe has quit IRC | 19:28 | |
| *** cdent has quit IRC | 19:28 | |
| *** jwcroppe has joined #openstack-telemetry | 19:30 | |
| *** nadya has joined #openstack-telemetry | 19:33 | |
| *** diogogmt has joined #openstack-telemetry | 19:37 | |
| *** rbak has quit IRC | 19:38 | |
| openstackgerrit | Merged openstack/gnocchi: gate: remove old job conf https://review.openstack.org/315909 | 19:51 |
| *** rbak has joined #openstack-telemetry | 19:51 | |
| *** efoley has quit IRC | 19:53 | |
| *** diogogmt has quit IRC | 19:55 | |
| *** diogogmt has joined #openstack-telemetry | 20:04 | |
| *** diogogmt has quit IRC | 20:08 | |
| *** diogogmt has joined #openstack-telemetry | 20:09 | |
| *** diogogmt has quit IRC | 20:25 | |
| *** flwang1 has quit IRC | 20:26 | |
| *** diogogmt has joined #openstack-telemetry | 20:29 | |
| *** julim has quit IRC | 20:42 | |
| *** julim has joined #openstack-telemetry | 20:42 | |
| *** adrian_otto has joined #openstack-telemetry | 20:52 | |
| *** zqfan has quit IRC | 20:53 | |
| *** BigWillie has quit IRC | 20:58 | |
| *** julim has quit IRC | 21:20 | |
| *** thorst_ has quit IRC | 21:23 | |
| *** flwang1 has joined #openstack-telemetry | 21:24 | |
| *** thorst_ has joined #openstack-telemetry | 21:25 | |
| *** thorst_ has quit IRC | 21:29 | |
| *** imcsk8 has quit IRC | 21:34 | |
| *** imcsk8 has joined #openstack-telemetry | 21:37 | |
| *** EZPZ has quit IRC | 21:44 | |
| *** thorst_ has joined #openstack-telemetry | 21:45 | |
| *** thorst_ has quit IRC | 21:49 | |
| *** diogogmt has quit IRC | 21:50 | |
| *** adrian_otto has quit IRC | 22:04 | |
| flwang | gordc: ping | 22:06 |
| flwang | jd__: ping any luck you guys around? | 22:06 |
| *** nadya has quit IRC | 22:07 | |
| gordc | flwang: whatsup? | 22:07 |
| flwang | i'm trying to figure out how to maintain customized pollsters | 22:07 |
| gordc | open source them ;) | 22:07 |
| gordc | flwang: did something break? | 22:07 |
| flwang | gordc: nope, you know, the operations guys always don't want to keep a private pkg | 22:08 |
| flwang | means we don't want to maintain a private ceilometer repo/pkg just for some customized pollsters | 22:09 |
| flwang | then we would like to keep it in a central repo and install the repo separately | 22:09 |
| flwang | with that way, can the customzied pollsters work well ? | 22:09 |
| gordc | i'm not entirely sure what you're asking. | 22:10 |
| gordc | you're asking how make sure your custom pollsters aren't broken? | 22:10 |
| gordc | flwang: have you seen what powervm folks did with their pollsters?https://review.openstack.org/#/q/project:openstack/ceilometer-powervm | 22:15 |
| flwang | gordc: yep, that's what i'm looking for, thanks, looking now.. | 22:16 |
| flwang | gordc: so it's a powervm pollster for ceilometer? | 22:17 |
| gordc | yeah, they just wrote their own compute pollsters for powervm | 22:18 |
| *** diogogmt has joined #openstack-telemetry | 22:19 | |
| flwang | gordc: cool, thanks | 22:21 |
| gordc | flwang: np. thorst is the person who manages it i think so you can ask him how easy/hard it is. | 22:22 |
| gordc | we can streamline it if needed | 22:22 |
| *** rcernin has quit IRC | 22:23 | |
| flwang | gordc: i'm looking for some support like nova scheduler filter | 22:24 |
| gordc | flwang: hmm.. it should work. in theory you just need to add entry_point | 22:32 |
| flwang | gordc: yep,i guess so | 22:32 |
| flwang | gordc: just like this, right? | 22:33 |
| *** diogogmt has quit IRC | 22:56 | |
| *** ametts has quit IRC | 23:04 | |
| *** adrian_otto has joined #openstack-telemetry | 23:19 | |
| *** gordc has quit IRC | 23:26 | |
| *** imcsk8 has quit IRC | 23:27 | |
| *** imcsk8 has joined #openstack-telemetry | 23:27 | |
| openstackgerrit | gordon chung proposed openstack/gnocchi: fix details filter for measures report https://review.openstack.org/318976 | 23:32 |
| *** zqfan has joined #openstack-telemetry | 23:41 | |
| *** adrian_otto has quit IRC | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!