| *** _cjones_ has quit IRC | 00:16 | |
| *** _cjones_ has joined #openstack-ceilometer | 01:12 | |
| *** _cjones_ has quit IRC | 01:31 | |
| *** liusheng has quit IRC | 01:46 | |
| *** _cjones_ has joined #openstack-ceilometer | 01:51 | |
| *** ddieterl_ has quit IRC | 02:13 | |
| *** _cjones_ has quit IRC | 02:37 | |
| *** ddieterly has joined #openstack-ceilometer | 02:39 | |
| *** Longgeek has joined #openstack-ceilometer | 02:55 | |
| openstackgerrit | Steve Martinelli proposed openstack/ceilometer: Attempt to set user_id for identity events https://review.openstack.org/151524 | 02:56 |
|---|---|---|
| openstackgerrit | Steve Martinelli proposed openstack/ceilometer: Add support for additional identity events https://review.openstack.org/162514 | 03:16 |
| openstackgerrit | Steve Martinelli proposed openstack/ceilometer: Attempt to set user_id for identity events https://review.openstack.org/151524 | 03:17 |
| openstackgerrit | Steve Martinelli proposed openstack/ceilometer: Add support for additional identity events https://review.openstack.org/162514 | 03:17 |
| *** _cjones_ has joined #openstack-ceilometer | 03:44 | |
| *** deepthi has joined #openstack-ceilometer | 03:46 | |
| *** sbfox has joined #openstack-ceilometer | 03:53 | |
| *** ddieterly has quit IRC | 03:54 | |
| *** _cjones_ has quit IRC | 03:55 | |
| *** yatin has joined #openstack-ceilometer | 04:28 | |
| openstackgerrit | ZhiQiang Fan proposed openstack/ceilometer: make field and value attributes mandatory in API Query https://review.openstack.org/162521 | 04:37 |
| *** zqfan-afk has quit IRC | 04:37 | |
| *** sbfox has quit IRC | 04:46 | |
| *** sbfox has joined #openstack-ceilometer | 04:50 | |
| *** ddieterly has joined #openstack-ceilometer | 04:51 | |
| *** _cjones_ has joined #openstack-ceilometer | 04:52 | |
| *** ddieterly has quit IRC | 04:56 | |
| *** _cjones_ has quit IRC | 04:56 | |
| *** exploreshaifali has joined #openstack-ceilometer | 05:00 | |
| *** ddieterly has joined #openstack-ceilometer | 05:52 | |
| *** ddieterly has quit IRC | 05:56 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/155197 | 06:09 |
| *** zqfan-afk has joined #openstack-ceilometer | 06:10 | |
| *** ParsectiX has quit IRC | 06:16 | |
| *** exploreshaifali has quit IRC | 06:39 | |
| *** ddieterly has joined #openstack-ceilometer | 06:53 | |
| *** _cjones_ has joined #openstack-ceilometer | 06:53 | |
| *** ddieterly has quit IRC | 06:57 | |
| *** rwsu has joined #openstack-ceilometer | 06:57 | |
| *** _cjones_ has quit IRC | 06:58 | |
| *** rwsu is now known as rwsu-afk | 06:58 | |
| *** exploreshaifali has joined #openstack-ceilometer | 07:02 | |
| *** afazekas has joined #openstack-ceilometer | 07:09 | |
| *** sbfox has quit IRC | 07:42 | |
| *** Kennan_ has joined #openstack-ceilometer | 07:48 | |
| *** Kennan has quit IRC | 07:49 | |
| *** exploreshaifali has quit IRC | 07:49 | |
| *** ddieterly has joined #openstack-ceilometer | 07:53 | |
| *** _cjones_ has joined #openstack-ceilometer | 07:55 | |
| *** ddieterly has quit IRC | 07:58 | |
| *** _cjones_ has quit IRC | 08:00 | |
| *** nellysmitt has joined #openstack-ceilometer | 08:00 | |
| *** ildikov has joined #openstack-ceilometer | 08:01 | |
| *** safchain has joined #openstack-ceilometer | 08:11 | |
| *** nellysmitt has quit IRC | 08:12 | |
| *** nellysmitt has joined #openstack-ceilometer | 08:12 | |
| openstackgerrit | Julien Danjou proposed stackforge/gnocchi: tests: make authtoken less verbose https://review.openstack.org/162166 | 08:24 |
| openstackgerrit | Julien Danjou proposed stackforge/gnocchi: rest: remove PATCH schemas https://review.openstack.org/162200 | 08:24 |
| *** Ala has joined #openstack-ceilometer | 08:42 | |
| *** ddieterly has joined #openstack-ceilometer | 08:54 | |
| *** _cjones_ has joined #openstack-ceilometer | 08:56 | |
| *** ddieterly has quit IRC | 08:59 | |
| *** briancline has quit IRC | 09:00 | |
| *** _cjones_ has quit IRC | 09:02 | |
| *** dobson has quit IRC | 09:04 | |
| *** karimb has joined #openstack-ceilometer | 09:04 | |
| *** briancline has joined #openstack-ceilometer | 09:06 | |
| *** dobson has joined #openstack-ceilometer | 09:06 | |
| *** eglynn-office has joined #openstack-ceilometer | 09:08 | |
| *** eglynn-office has quit IRC | 09:16 | |
| *** Flcn has joined #openstack-ceilometer | 09:17 | |
| *** eglynn-office has joined #openstack-ceilometer | 09:19 | |
| *** deepthi_d has joined #openstack-ceilometer | 09:25 | |
| *** deepthi has quit IRC | 09:25 | |
| *** karimb has quit IRC | 09:25 | |
| *** karimb has joined #openstack-ceilometer | 09:27 | |
| *** exploreshaifali has joined #openstack-ceilometer | 09:33 | |
| *** ddieterly has joined #openstack-ceilometer | 09:55 | |
| *** yassine_ has joined #openstack-ceilometer | 09:59 | |
| *** _cjones_ has joined #openstack-ceilometer | 09:59 | |
| *** ddieterly has quit IRC | 09:59 | |
| *** _cjones_ has quit IRC | 10:09 | |
| *** ddieterly has joined #openstack-ceilometer | 10:56 | |
| *** rbak has joined #openstack-ceilometer | 10:59 | |
| *** ddieterly has quit IRC | 11:00 | |
| *** _cjones_ has joined #openstack-ceilometer | 11:05 | |
| *** exploreshaifali has quit IRC | 11:16 | |
| *** rbak has quit IRC | 11:17 | |
| *** _cjones_ has quit IRC | 11:38 | |
| *** deepthi_d has quit IRC | 11:46 | |
| *** cdent has joined #openstack-ceilometer | 11:52 | |
| *** ddieterly has joined #openstack-ceilometer | 11:57 | |
| *** ddieterly has quit IRC | 12:01 | |
| *** claudiub has joined #openstack-ceilometer | 12:16 | |
| *** ildikov has quit IRC | 12:30 | |
| *** gordc has joined #openstack-ceilometer | 12:34 | |
| *** _cjones_ has joined #openstack-ceilometer | 12:36 | |
| openstackgerrit | Julien Danjou proposed stackforge/gnocchi: rest: fix exception JSON encoding https://review.openstack.org/162621 | 12:38 |
| *** ildikov has joined #openstack-ceilometer | 12:43 | |
| *** exploreshaifali has joined #openstack-ceilometer | 12:47 | |
| *** thomasem has joined #openstack-ceilometer | 12:50 | |
| *** ccrouch is now known as ccrouch-biab | 12:52 | |
| *** ddieterly has joined #openstack-ceilometer | 12:53 | |
| *** thomasem has quit IRC | 13:01 | |
| *** yatin has quit IRC | 13:01 | |
| *** thomasem has joined #openstack-ceilometer | 13:01 | |
| *** ddieterly has quit IRC | 13:04 | |
| *** _cjones_ has quit IRC | 13:09 | |
| openstackgerrit | Julien Danjou proposed stackforge/gnocchi: rest: fix exception JSON encoding https://review.openstack.org/162621 | 13:11 |
| *** yasu_ has joined #openstack-ceilometer | 13:18 | |
| openstackgerrit | gordon chung proposed openstack/ceilometermiddleware: use pycadf helper module to assign action https://review.openstack.org/157859 | 13:18 |
| *** ccrouch-biab is now known as ccrouch | 13:32 | |
| openstackgerrit | Julien Danjou proposed stackforge/gnocchi: tests: make authtoken less verbose https://review.openstack.org/162166 | 13:32 |
| openstackgerrit | Julien Danjou proposed stackforge/gnocchi: rest: remove PATCH schemas https://review.openstack.org/162200 | 13:32 |
| openstackgerrit | Julien Danjou proposed stackforge/gnocchi: rest: fix exception JSON encoding https://review.openstack.org/162621 | 13:32 |
| openstackgerrit | Julien Danjou proposed stackforge/gnocchi: tests: stop generating scenarios for all indexer https://review.openstack.org/162091 | 13:32 |
| openstackgerrit | Julien Danjou proposed stackforge/gnocchi: rest: return a correct error code if NotImplemented https://review.openstack.org/162090 | 13:32 |
| cdent | lunch | 13:37 |
| *** ddieterly has joined #openstack-ceilometer | 13:37 | |
| *** gzhai_mob has joined #openstack-ceilometer | 13:41 | |
| *** jmatthews has joined #openstack-ceilometer | 13:47 | |
| gzhai_mob | eglynn-office: hi | 13:58 |
| eglynn-office | gzhai_mob: hi | 13:59 |
| gzhai_mob | eglynn-office: Could you pls. have a look at my latest patch? Thanks! | 13:59 |
| eglynn-office | gzhai_mob: sure, will try to get to it today ... thanks! | 14:00 |
| gzhai_mob | eglynn-office: thanks! | 14:00 |
| *** yasu_ has quit IRC | 14:05 | |
| *** _cjones_ has joined #openstack-ceilometer | 14:06 | |
| *** gzhai_mob has quit IRC | 14:07 | |
| *** ddieterly has quit IRC | 14:07 | |
| *** ddieterly has joined #openstack-ceilometer | 14:07 | |
| *** nellysmitt has quit IRC | 14:24 | |
| *** r-daneel has joined #openstack-ceilometer | 14:28 | |
| *** rbak has joined #openstack-ceilometer | 14:37 | |
| *** _cjones_ has quit IRC | 14:39 | |
| *** Flcn has quit IRC | 15:00 | |
| *** Flcn has joined #openstack-ceilometer | 15:01 | |
| openstackgerrit | Julien Danjou proposed stackforge/gnocchi: tests: make authtoken less verbose https://review.openstack.org/162166 | 15:02 |
| openstackgerrit | Julien Danjou proposed stackforge/gnocchi: rest: remove PATCH schemas https://review.openstack.org/162200 | 15:02 |
| openstackgerrit | Julien Danjou proposed stackforge/gnocchi: tests: stop generating scenarios for all indexer https://review.openstack.org/162091 | 15:02 |
| openstackgerrit | Julien Danjou proposed stackforge/gnocchi: rest: return a correct error code if NotImplemented https://review.openstack.org/162090 | 15:02 |
| openstackgerrit | Julien Danjou proposed stackforge/gnocchi: rest: change metric aggregation URL https://review.openstack.org/162649 | 15:02 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ceilometer: Updated from global requirements https://review.openstack.org/161384 | 15:02 |
| *** exploreshaifali has quit IRC | 15:05 | |
| *** prad has joined #openstack-ceilometer | 15:07 | |
| *** nellysmitt has joined #openstack-ceilometer | 15:08 | |
| *** rbak has quit IRC | 15:11 | |
| *** rbak has joined #openstack-ceilometer | 15:17 | |
| openstackgerrit | Rohit Jaiswal proposed openstack/ceilometer: Added option to allow sample expiration more frequently https://review.openstack.org/158519 | 15:22 |
| *** afazekas has quit IRC | 15:26 | |
| *** ildikov has quit IRC | 15:29 | |
| *** _cjones_ has joined #openstack-ceilometer | 15:35 | |
| *** david-lyle_afk is now known as david-lyle | 15:38 | |
| *** rwsu-afk is now known as rwsu | 15:41 | |
| gordc | jd__: gnocchi unit tests keep throwing DuplicateOptError... what am i doing wrong? | 15:43 |
| jd__ | gordc: master? | 15:43 |
| gordc | yep | 15:43 |
| jd__ | gordc: how do you run tox? | 15:43 |
| gordc | "tox -r -epy27" | 15:44 |
| jd__ | gordc: paste me a backtrace | 15:44 |
| *** ildikov has joined #openstack-ceilometer | 15:44 | |
| gordc | http://paste.openstack.org/show/191003/ | 15:45 |
| jd__ | looks like the error prad had in his patch | 15:46 |
| jd__ | I've no idea and I've not seen that in the gate for master yet gordc :/ | 15:47 |
| gordc | yeah... some weird dependency issue? | 15:47 |
| gordc | i see this: {1} gnocchi.tests.test_ceilometer_alarm.TestGnocchiThresholdEvaluate.test_simple_alarm_clear [0.013779s] ... ok | 15:47 |
| cdent | it may be that that is happening because gabbi is messing with config handling too | 15:47 |
| gordc | {3} gnocchi.tests.test_indexer.TestIndexerDriver.test_delete_metric(ceph,null) [0.004191s] ... FAILED | 15:47 |
| gordc | cdent: yeah | 15:47 |
| cdent | and it only shows up when the grouping of tests collides somehow | 15:47 |
| cdent | I think we want: | 15:47 |
| cdent | scratch that, I had to do: | 15:48 |
| cdent | https://github.com/stackforge/gnocchi/blob/master/gnocchi/tests/gabbi/fixtures.py#L62 | 15:48 |
| cdent | to get around what is likely a related problem | 15:48 |
| *** nellysmitt has quit IRC | 15:49 | |
| cdent | and it is not happening on the gate because of the way tests are being distributed | 15:49 |
| cdent | globals are bad! | 15:49 |
| jd__ | the root cause might be that self.conf is a config fixture which is actually a copy of cfg.CONF | 15:49 |
| jd__ | the global one | 15:49 |
| jd__ | IIRC | 15:49 |
| cdent | the gabbi fixture will reset its conf | 15:50 |
| cdent | maybe the base TestCase needs a: https://github.com/stackforge/gnocchi/blob/master/gnocchi/tests/gabbi/fixtures.py#L87 | 15:51 |
| cdent | at: https://github.com/stackforge/gnocchi/blob/master/gnocchi/tests/base.py#L369 | 15:51 |
| * cdent tries it | 15:52 | |
| *** nellysmitt has joined #openstack-ceilometer | 15:52 | |
| gordc | oh wait... it's in gate now | 15:53 |
| gordc | i guess it's related to a new oslo.config? | 15:53 |
| jd__ | haha | 15:53 |
| gordc | or oslo.log? | 15:53 |
| jd__ | gordc: URL? | 15:54 |
| jd__ | got it | 15:55 |
| cdent | oslo log? | 15:55 |
| jd__ | I got the log URL I meant :) | 15:55 |
| cdent | ceilo is using incubated, gnocchi not | 15:56 |
| jd__ | I just warned dhellmann on #openstack-oslo | 15:56 |
| jd__ | now reading git log | 15:56 |
| gordc | jd__: oh sorry. tristan is bothering for lunch.lol | 15:56 |
| jd__ | haha | 15:56 |
| jd__ | classic | 15:56 |
| *** _cjones_ has quit IRC | 16:01 | |
| cdent | jd__, gordc: for reference reseting the conf in teardown makes no difference, I figured it wouldn't but wanted to be sure. poor oslo, can't get a break. | 16:01 |
| *** ildikov has quit IRC | 16:01 | |
| jd__ | so the regression is in oslo.log 1.0.0 | 16:02 |
| jd__ | at least now I know that | 16:02 |
| jd__ | but the diff make no sense | 16:02 |
| jd__ | [17:06:17] <jd__>I bisect the culprit to https://review.openstack.org/#/c/156033/ | 16:08 |
| jd__ | [17:06:20] <jd__>but honestly it makes no sense | 16:08 |
| jd__ | [17:06:28] <jd__>I reverted it and it seems to fix the issue we have | 16:08 |
| jd__ | [17:06:31] <jd__>gordc: could you confirm? | 16:08 |
| jd__ | [17:06:58] <jd__>this might not be a regression but something weird we do that now triggers a bug | 16:08 |
| jd__ | for people not in #openstack-oslo | 16:08 |
| * jd__ heading now | 16:08 | |
| jd__ | +home | 16:08 |
| *** nellysmitt has quit IRC | 16:11 | |
| *** _cjones_ has joined #openstack-ceilometer | 16:14 | |
| *** _cjones_ has quit IRC | 16:14 | |
| *** _cjones_ has joined #openstack-ceilometer | 16:14 | |
| *** jmatthews has quit IRC | 16:14 | |
| *** amalagon has quit IRC | 16:16 | |
| *** amalagon has joined #openstack-ceilometer | 16:17 | |
| *** nellysmitt has joined #openstack-ceilometer | 16:21 | |
| *** amalagon has quit IRC | 16:21 | |
| *** exploreshaifali has joined #openstack-ceilometer | 16:25 | |
| *** Longgeek has quit IRC | 16:29 | |
| linuxhermit | cdent I'm finally back to normal mode, what api should I work on testin gnext | 16:32 |
| linuxhermit | Not sure what you picked up recently :) | 16:32 |
| cdent | I started on /v2/meters in ceilo | 16:32 |
| cdent | and that's all the further I've gotten | 16:33 |
| cdent | and the only gabbi-related patch I've got out that's not merged | 16:33 |
| cdent | so pick what you like: I think you said you want to do something gnocchi next? | 16:33 |
| linuxhermit | yeah that's correct | 16:34 |
| linuxhermit | looking now | 16:34 |
| linuxhermit | is it tox -edocs to get the docs? | 16:37 |
| linuxhermit | I've forgotten how to build the html docs | 16:37 |
| *** sbfox has joined #openstack-ceilometer | 16:38 | |
| cdent | yeah linuxhermit, if you look in tox.ini you can see each of the targets | 16:41 |
| cdent | jd__: shall I do that oslo_log switch? | 16:41 |
| *** fnaval has joined #openstack-ceilometer | 16:42 | |
| gordc | cdent: are you switching ceilometer or gnocchi? | 16:45 |
| cdent | gordc: ceilo | 16:45 |
| gordc | https://review.openstack.org/#/c/147810/ | 16:45 |
| gordc | i get the feeling it requires my patch to drop deprecated middlewarew | 16:46 |
| cdent | feh | 16:46 |
| gordc | dependency madness. | 16:47 |
| cdent | so....I'll do nothing then? | 16:49 |
| gordc | head in the sand. | 16:50 |
| cdent | nnnnnnnngh | 16:51 |
| gordc | why do we have ceilometer stuff in gnocchi anyways? | 16:52 |
| cdent | double nnnnngh | 16:53 |
| cdent | I said as much on the review that introduced it | 16:53 |
| cdent | I think I may have jus fixed it, confirming with a clean env | 16:53 |
| *** ddieterly has quit IRC | 16:53 | |
| *** ddieterly has joined #openstack-ceilometer | 16:54 | |
| gordc | :( it looks really strange... and will probably constantly cause issues like when we tried to extend nova in ceilometer code. | 16:54 |
| linuxhermit | cdent gonna work on metrics | 16:55 |
| cdent | I think it was considered temporary | 16:55 |
| cdent | cool linuxhermit | 16:55 |
| cdent | have fun | 16:55 |
| gordc | cdent: ah i see.. i guess that temporary ends now?... or we wait until we can fix the dependency madness | 16:55 |
| cdent | gordc: I definitely think it is something we should fix before release | 16:55 |
| gordc | cdent: yeah, that'd be good. | 16:56 |
| cdent | gordc: assuming my little fix actualy fixes it, it _only_ fixes the log duplication problem, not the issue with ceilo code in gnocchi | 16:58 |
| cdent | that I think we fix by moving that code to ceilo (or its own repo) | 16:58 |
| *** yatin has joined #openstack-ceilometer | 16:58 | |
| gordc | cdent: ok. will check your fix. | 16:59 |
| cdent | it's beyond me why a ceilometer dispatcher to gnocchi is in the gnocchi code base | 16:59 |
| cdent | feh | 16:59 |
| cdent | doesn't fix it | 16:59 |
| gordc | lol you can't outsmart oslo dependencies. | 17:00 |
| gordc | cdent: i was looking at the dispatcher code too... i guess it was to test but not commit unproven stuff in ceilometer? | 17:01 |
| cdent | repos are cheap, we should have lots of them | 17:01 |
| gordc | cdent: what happens if we just stop trying to implement ceilometer.dispatcher.Base interface.. and just have it as its own obj. | 17:04 |
| gordc | ah never. ceilometer package is use all over. | 17:05 |
| *** _cjones_ has quit IRC | 17:05 | |
| cdent | besides that would just be hacking around the fundamental error? | 17:05 |
| gordc | cdent: yep. i'm all about being super hacky.lol | 17:06 |
| cdent | I think we gotta untangle this for realz, yo | 17:07 |
| cdent | jd__: you back aware yet? | 17:09 |
| *** nellysmitt has quit IRC | 17:11 | |
| *** nellysmitt has joined #openstack-ceilometer | 17:13 | |
| *** Flcn has quit IRC | 17:13 | |
| *** thomasem has quit IRC | 17:14 | |
| *** Flcn has joined #openstack-ceilometer | 17:17 | |
| *** nellysmitt has quit IRC | 17:17 | |
| *** Flcn has quit IRC | 17:22 | |
| *** Ala has quit IRC | 17:22 | |
| *** _cjones_ has joined #openstack-ceilometer | 17:24 | |
| *** safchain has quit IRC | 17:26 | |
| *** yassine_ has quit IRC | 17:26 | |
| *** amalagon has joined #openstack-ceilometer | 17:27 | |
| openstackgerrit | gordon chung proposed stackforge/gnocchi: default pecan debug to false https://review.openstack.org/162698 | 17:28 |
| *** yatin has quit IRC | 17:29 | |
| *** amalagon has quit IRC | 17:31 | |
| jd__ | I'm back | 17:34 |
| jd__ | so we have Ceilometer dispatcher in Gnocchi because it predates the "yeah we are for sure going to use Gnocchi & al" decision | 17:34 |
| jd__ | now we should move it in Ceilometer IMHO | 17:34 |
| gordc | https://www.youtube.com/watch?v=wQKKj_qeOBQ | 17:34 |
| jd__ | until that is done, would be good to switch Ceilo to oslo.log | 17:34 |
| jd__ | I dunnow what's missing | 17:35 |
| jd__ | I can +2A whatever is going to fix that | 17:35 |
| cdent | I'm currently rebasing https://review.openstack.org/#/c/147810/ | 17:35 |
| cdent | there were some conflicts | 17:35 |
| cdent | on the dispatcher: why _not_ put it in its own repo? | 17:36 |
| openstackgerrit | Chris Dent proposed openstack/ceilometer: use oslo.log instead of oslo-incubator code https://review.openstack.org/147810 | 17:37 |
| cdent | jd__, gordc ^ | 17:39 |
| jd__ | cdent: harder to maintain | 17:39 |
| cdent | how so? | 17:40 |
| *** harlowja has joined #openstack-ceilometer | 17:40 | |
| gordc | cdent: i think you'd have to move all the tests to that separate repo | 17:40 |
| jd__ | cdent: LGTM, tests are OK? | 17:41 |
| jd__ | waiting for Jenkins | 17:41 |
| cdent | only locally are they okay | 17:41 |
| cdent | I think the swift problem (whatever that is, still not clear) only shows up in the gate | 17:41 |
| gordc | cdent: i'm hoping that patch miraculously doesn't need my ceilometermiddleware patch | 17:41 |
| jd__ | cdent: bah if anything changes in Ceilometer we'll break it and we'll have to fix it later… not sure it's worth it | 17:41 |
| cdent | jd__ that sounds like you're trying to work around a problem with the architecture rather than fix the architecture. Yeah, sure, there's too much to do we can't fix everything | 17:42 |
| cdent | but in my experience breakings things up into pieces is a _huge_ tool in finding and fixing bugs both in specific details and in the architecture | 17:43 |
| cdent | it also helps manage responsibilities | 17:43 |
| *** harlowja_ has joined #openstack-ceilometer | 17:43 | |
| *** ildikov has joined #openstack-ceilometer | 17:44 | |
| *** harlowja has quit IRC | 17:45 | |
| jd__ | cdent: the architecture is plugin based and we break our plugin signature pretty often | 17:47 |
| jd__ | cdent: so we can fix the architecture by removing the plugin approach but not sure it's gonna please :) | 17:47 |
| cdent | I don't need to do my side of the conversation if you're going to keep making my points for me: if the plugins were in separate repos we wouldn't want to change the signature often and that would be a good thing | 17:48 |
| cdent | make a good signature and stick with it | 17:48 |
| cdent | reify contracts in code and repo boundaries | 17:48 |
| cdent | argh, it would also help make testing faster | 17:51 |
| * cdent twiddles thumbs | 17:51 | |
| openstackgerrit | Chris Dent proposed openstack/ceilometer: use oslo.log instead of oslo-incubator code https://review.openstack.org/147810 | 17:53 |
| cdent | requirements and pep8 failutres ^ | 17:54 |
| cdent | gordc: what was the swift issue we're hoping is magically gone? | 17:55 |
| gordc | cdent: http://logs.openstack.org/10/147810/5/check/check-tempest-dsvm-full/0f43e98/logs/screen-s-proxy.txt.gz | 17:56 |
| cdent | thanks | 17:57 |
| openstackgerrit | Merged openstack/ceilometer: Added option to allow sample expiration more frequently https://review.openstack.org/158519 | 17:57 |
| jd__ | cdent: the more I talk with you the more I realize we think the same, I've just 3 years of Ceilometer behind me that says "this is gonna work" :D | 17:58 |
| jd__ | cdent: +NOT sorry :) | 17:58 |
| jd__ | cdent: the main problem is the lack of man power; it makes much more complicated to change the signature if we need too, more cost, and we don't have a lot of people to work on all of that so we keep things tight | 17:58 |
| jd__ | compromise and pragmatism :/ | 17:59 |
| cdent | Yeah, I know, I just like to maintain that other voice for the sake of...keeping the flame alive? | 17:59 |
| cdent | Also, the fact that I knew nothing about ceilometer (or really even openstack) before I got hired makes hear issues like "lack of man power" as something the bosses should just deal with and not my problem | 18:00 |
| *** sbfox has quit IRC | 18:00 | |
| cdent | this enterprisey style of open source ought to have some kind of benefit to compensate for its sins | 18:00 |
| cdent | which, again, is not particularly pragmatic of me, but meh | 18:01 |
| * cdent is aging hippie | 18:01 | |
| *** sbfox has joined #openstack-ceilometer | 18:13 | |
| *** thomasem has joined #openstack-ceilometer | 18:21 | |
| linuxhermit | cdent does gabbi have a way to store for more than one request? | 18:23 |
| cdent | no, if you want to make a back reference it is only to the most recent request | 18:23 |
| linuxhermit | I know to use $response but can i store that for more than just the next run? | 18:23 |
| linuxhermit | :( okay | 18:23 |
| cdent | I thought about having a full stack, but then the syntax gets all messy | 18:24 |
| cdent | and the tests less and less readable (it's already getting pretty hard) | 18:24 |
| linuxhermit | nod | 18:24 |
| cdent | and there's a bit of bondage and discpline enforcing that gabbi hopes to be able to encourage in a TDD situation | 18:25 |
| cdent | if you can't write the test a particular way, then maybe your api is not doing the right thing... etc | 18:25 |
| cdent | however in the existing world of openstack apis, that's no comfort | 18:25 |
| *** karimb has quit IRC | 18:27 | |
| *** eglynn-officeafk has joined #openstack-ceilometer | 18:27 | |
| linuxhermit | cdent nod I've kinda hit on eof those moments | 18:28 |
| linuxhermit | you have to create measures on a metric, it returns only a 204 | 18:28 |
| *** eglynn-office has quit IRC | 18:28 | |
| *** promulo has joined #openstack-ceilometer | 18:28 | |
| linuxhermit | so i don't have the id for the next request | 18:28 |
| cdent | not location? | 18:28 |
| *** promulo has quit IRC | 18:28 | |
| cdent | s/not/no/ | 18:29 |
| linuxhermit | location like response_headers? | 18:29 |
| cdent | yeah | 18:29 |
| cdent | oh, you mean you want to create several | 18:30 |
| cdent | first one is okay, but then the second...stuck | 18:30 |
| linuxhermit | I might can amke this work | 18:30 |
| linuxhermit | i always forget about location | 18:30 |
| cdent | it's the most important one! | 18:30 |
| cdent | :) | 18:30 |
| cdent | in the earliest version of gabbi, $LOCATION was the only template | 18:31 |
| *** promulo has joined #openstack-ceilometer | 18:31 | |
| linuxhermit | haha | 18:31 |
| *** promulo has quit IRC | 18:31 | |
| cdent | then I found loads of POSTs that were not returning a location: header | 18:31 |
| cdent | after crying, I added some more templates | 18:31 |
| linuxhermit | yeap.. :( | 18:34 |
| linuxhermit | 'test_gabbi_metrics_push_measurements_to_metric' object has no attribute 'location' | 18:34 |
| * linuxhermit cries | 18:34 | |
| cdent | what request is that? | 18:37 |
| *** exploreshaifali has quit IRC | 18:42 | |
| *** _cjones_ has quit IRC | 18:44 | |
| linuxhermit | so it's a request to get measurements after posting them | 18:52 |
| linuxhermit | the post doesn't return anything we can touch really | 18:52 |
| cdent | gordc: what's up with: https://review.openstack.org/#/c/157615/ without it, mysql and postgresql tests are failing for me | 18:53 |
| cdent | linuxhermit: that seems like a bug, check first to see if there isn't one already though, as I did make some | 18:54 |
| cdent | doesn't look like that one though: https://bugs.launchpad.net/gnocchi | 18:54 |
| gordc | cdent: i broke the sql events world with my db refactor patch. | 18:56 |
| linuxhermit | so it's not returning the header is a bug? | 18:56 |
| cdent | no cookies for you gordc | 18:57 |
| cdent | linuxhermit: in my world a post a without a location header in response is a bug | 18:58 |
| cdent | if, in fact, something was created | 18:58 |
| linuxhermit | so just out of curiousity are you writing tests to check that | 18:58 |
| cdent | yeah | 18:58 |
| * cdent finds one | 18:58 | |
| gordc | cdent: jokes on you, cookies were on sale last week and i bought a diabetic amount. | 18:58 |
| cdent | can you save me some thin mints please? | 18:59 |
| gordc | cdent: i had to google that... i can't remember the last time i've seen girl scouts cookies sold door to door | 19:00 |
| cdent | linuxhermit: https://github.com/openstack/ceilometer/blob/master/ceilometer/tests/gabbi/gabbits/alarms.yaml#L36 | 19:00 |
| *** promulo has joined #openstack-ceilometer | 19:00 | |
| linuxhermit | okay I'll add that to my tests as well | 19:00 |
| linuxhermit | I suck | 19:00 |
| cdent | that file probably needs to be tuned up with what we've learned, it's doing an xfail, but has a bunch of useless stuff and has no link to a bug | 19:00 |
| cdent | bah! you're writing tests. no one who writes tests sucks. | 19:01 |
| cdent | gordc sucks though. he sucks because apparently he didn't write (or at least run) enough tests :P :) | 19:02 |
| openstackgerrit | gordon chung proposed openstack/ceilometer: order traits returned within events https://review.openstack.org/157615 | 19:02 |
| cdent | \o/ | 19:02 |
| gordc | lol. i won't lie. that's a very accurate assessment. | 19:03 |
| gordc | ha! there's an apple watch that cost +10,000$ | 19:03 |
| cdent | it's looking like this is going to fail again: https://review.openstack.org/#/c/147810/ | 19:04 |
| cdent | the dsvm jobs are all stalled | 19:04 |
| gordc | :( i don't think my tempest/devstack changes will get in anytime soon tbh. | 19:06 |
| cdent | are they up for review and just not getting eyes, or some other issue? | 19:07 |
| *** Longgeek has joined #openstack-ceilometer | 19:08 | |
| *** _nadya_ has joined #openstack-ceilometer | 19:09 | |
| jd__ | gordc: can you babysit https://review.openstack.org/#/c/147810/ ? | 19:11 |
| jd__ | it's gonna need a fix or a recheck AFAICS | 19:11 |
| cdent | fix | 19:12 |
| *** Longgeek has quit IRC | 19:12 | |
| *** _cjones_ has joined #openstack-ceilometer | 19:12 | |
| linuxhermit | cdent nothing is returning location header :) | 19:14 |
| * cdent glances at jd__ | 19:15 | |
| linuxhermit | :) | 19:15 |
| linuxhermit | I'm gonna file a bug on just the metrics endpoints period, then I might go try to fix it | 19:15 |
| linuxhermit | so for now should I comment out the location check? | 19:15 |
| cdent | What I tend to do is: | 19:16 |
| cdent | * make a test which checks for the location header and xfail it, and add desc with a link to a bug | 19:16 |
| linuxhermit | and fail the test that can't run because we have no endpoing | 19:16 |
| cdent | * make a second test which doesn't check for the location header, so succeeds, and use that as the starting point for wahtever | 19:16 |
| linuxhermit | will do | 19:16 |
| cdent | xfailing a test that uses $LOCATION is bound to cause confusion | 19:17 |
| cdent | so better to xfail the thing that actually has the wrong behavior | 19:17 |
| linuxhermit | you got it | 19:18 |
| linuxhermit | just leave out that other test then and put a todo to add it after we have headers | 19:18 |
| linuxhermit | thanks | 19:18 |
| cdent | gordc: the dependent tempest patch related to this https://review.openstack.org/#/c/155473/ has merged so I've made it recheck | 19:18 |
| cdent | I guess you could use skip: no location! if you felt like it | 19:19 |
| cdent | plenty of different ways | 19:19 |
| cdent | biab, gotta make dinner | 19:23 |
| jd__ | thanks guys | 19:29 |
| jd__ | I want all this fixed when I wake up tomorrow morning or you are all fired! | 19:29 |
| jd__ | :p | 19:29 |
| jd__ | take care see you tomorrow :) | 19:29 |
| linuxhermit | ha | 19:32 |
| linuxhermit | is there a mid cycle for ceilometer | 19:33 |
| *** exploreshaifali has joined #openstack-ceilometer | 19:35 | |
| *** _nadya_ has quit IRC | 19:38 | |
| gordc | sorry in a meeting. but the patch isn't merged yet: https://review.openstack.org/#/c/159184/ | 19:41 |
| gordc | i had to fix it because ceilometer api. | 19:41 |
| *** dhellman_ has joined #openstack-ceilometer | 19:52 | |
| *** dhellman_ has quit IRC | 19:52 | |
| cdent | that's some tangled web gordc | 19:55 |
| gordc | cdent: i wanted to burn down api... i was told no. | 19:58 |
| cdent | maybe we should all just switch to monasca and stacktach? | 19:59 |
| *** pradk has joined #openstack-ceilometer | 19:59 | |
| gordc | lol... i'm not sure what to say. | 20:01 |
| gordc | i think that's the safest comment i can make. | 20:01 |
| cdent | I think you should say: | 20:01 |
| cdent | Yes Chris I will bring you thin mints. | 20:01 |
| gordc | do you not get girl scout cookies in uk? | 20:02 |
| *** pradk has quit IRC | 20:02 | |
| *** prad has quit IRC | 20:02 | |
| cdent | I don't know. The main point, though, is that I don't have any. If you brought me some, I would. | 20:03 |
| * cdent looks in the kitchen for a pale substitute | 20:04 | |
| *** sbfox has quit IRC | 20:05 | |
| *** sbfox has joined #openstack-ceilometer | 20:07 | |
| openstackgerrit | Steve Martinelli proposed openstack/ceilometer: Attempt to set user_id for identity events https://review.openstack.org/151524 | 20:07 |
| gordc | i'm pretty sure you can order online now... (i believe they've entered the digital age) | 20:07 |
| *** claudiub has quit IRC | 20:07 | |
| gordc | err nevermind | 20:08 |
| gordc | it's some digital cookie scam. | 20:09 |
| cdent | gordc: you are correct sir, 155473 hasn't miraculously gotten better, it still fails after that recheck I started | 20:22 |
| gordc | cdent: :( i'm hoping this is right: https://review.openstack.org/#/c/159184/ | 20:23 |
| gordc | sorta just guessing at what the hell ceilometer api is returning. | 20:23 |
| openstackgerrit | Steve Martinelli proposed openstack/ceilometer: Add support for additional identity events https://review.openstack.org/162514 | 20:23 |
| openstackgerrit | Steve Martinelli proposed openstack/ceilometer: Attempt to set user_id for identity events https://review.openstack.org/151524 | 20:27 |
| openstackgerrit | Steve Martinelli proposed openstack/ceilometer: Add support for additional identity events https://review.openstack.org/162514 | 20:27 |
| *** _nadya_ has joined #openstack-ceilometer | 20:27 | |
| cdent | gordc: didn't you paste it somewhere recently and there was a weird space before the second non flattened bit | 20:27 |
| openstackgerrit | Steve Martinelli proposed openstack/ceilometer: Attempt to set user_id for identity events https://review.openstack.org/151524 | 20:27 |
| openstackgerrit | Steve Martinelli proposed openstack/ceilometer: Add support for additional identity events https://review.openstack.org/162514 | 20:27 |
| *** thomasem has quit IRC | 20:28 | |
| cdent | gordc: Imma have to stop thinking about this for the evening. Will catch up in the morning. | 20:29 |
| gordc | cdent: will take a look. i hope not... i really have no idea | 20:30 |
| cdent | maybe you need to use the awesome power of print | 20:30 |
| cdent | good luck | 20:30 |
| *** cdent has quit IRC | 20:30 | |
| gordc | cdent: i think space came from when i ran it through formatting in jsonlint... it thought "hey, there's a stupid colon in the middle of your key, let's throw a space in there" | 20:37 |
| *** _nadya_ has quit IRC | 20:38 | |
| *** xmltok has joined #openstack-ceilometer | 20:41 | |
| *** Flcn has joined #openstack-ceilometer | 20:41 | |
| openstackgerrit | Merged openstack/ceilometer: add option to store raw notification https://review.openstack.org/153790 | 20:45 |
| *** sbfox has quit IRC | 20:45 | |
| *** sbfox has joined #openstack-ceilometer | 20:50 | |
| openstackgerrit | Steve Martinelli proposed openstack/ceilometer: Fix formatting error in licence https://review.openstack.org/162755 | 20:52 |
| *** claudiub has joined #openstack-ceilometer | 20:53 | |
| openstackgerrit | gordon chung proposed openstack/ceilometer: mysql doesn't understand intersect https://review.openstack.org/157574 | 20:56 |
| *** thomasem has joined #openstack-ceilometer | 20:57 | |
| *** thomasem has quit IRC | 20:57 | |
| *** ddieterly has quit IRC | 20:57 | |
| *** thomasem has joined #openstack-ceilometer | 20:58 | |
| *** ddieterly has joined #openstack-ceilometer | 20:59 | |
| *** ddieterl_ has joined #openstack-ceilometer | 21:01 | |
| *** ddieterly has quit IRC | 21:04 | |
| *** prad has joined #openstack-ceilometer | 21:05 | |
| *** sbfox has quit IRC | 21:43 | |
| *** sbfox has joined #openstack-ceilometer | 22:04 | |
| *** gordc has quit IRC | 22:05 | |
| *** promulo__ has joined #openstack-ceilometer | 22:15 | |
| *** promulo has quit IRC | 22:15 | |
| *** theanalyst has quit IRC | 22:21 | |
| *** openstack has joined #openstack-ceilometer | 22:24 | |
| *** prad has quit IRC | 22:27 | |
| *** Flcn has quit IRC | 22:31 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ceilometer: Updated from global requirements https://review.openstack.org/161384 | 22:41 |
| *** exploreshaifali has quit IRC | 22:41 | |
| *** rbak has quit IRC | 22:47 | |
| *** ddieterl_ has quit IRC | 23:00 | |
| ccrouch | anyone know if folks are using gabbit outside of ceilometer testing? | 23:06 |
| ccrouch | and if so which projects? | 23:06 |
| *** openstack has joined #openstack-ceilometer | 23:24 | |
| *** _nadya_ has joined #openstack-ceilometer | 23:38 | |
| *** _nadya_ has quit IRC | 23:43 | |
| *** _cjones_ has quit IRC | 23:45 | |
| *** _cjones_ has joined #openstack-ceilometer | 23:48 | |
| *** promulo has joined #openstack-ceilometer | 23:57 | |
| *** promulo__ has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!