| openstackgerrit | Rohit Jaiswal proposed openstack/ceilometer: Fix catch-all event def to exclude !event https://review.openstack.org/187397 | 00:03 |
|---|---|---|
| *** ddieterly has joined #openstack-ceilometer | 00:11 | |
| *** botang has joined #openstack-ceilometer | 00:32 | |
| *** robsparker has joined #openstack-ceilometer | 00:44 | |
| *** robsparker has quit IRC | 00:49 | |
| *** _cjones_ has quit IRC | 00:52 | |
| *** promulo_ has joined #openstack-ceilometer | 00:54 | |
| *** promulo has quit IRC | 00:57 | |
| *** gordc has joined #openstack-ceilometer | 00:59 | |
| *** sputnik13 has quit IRC | 01:02 | |
| lan | hi gordc | 01:20 |
| lan | gordc, for this patch https://review.openstack.org/#/c/185295/ | 01:21 |
| lan | gordc, the error only throws in the first cycle, and the rgw related pollsters should self-disabled | 01:21 |
| lan | gordc, and we need "endpoint" parameter to create the client, so it may be hard to create the client on init() : https://github.com/openstack/ceilometer/blob/master/ceilometer/objectstore/rgw.py#L94 | 01:24 |
| *** chinmay_g has quit IRC | 01:35 | |
| *** boris-42 has quit IRC | 01:35 | |
| *** rex_lee has quit IRC | 01:36 | |
| *** serverascode has quit IRC | 01:36 | |
| *** rjaiswal has joined #openstack-ceilometer | 01:36 | |
| rjaiswal | Any idea why this test fails on CI but no locally with latest review - | 01:36 |
| rjaiswal | 2015-06-02 01:11:53.644 | FAIL: ceilometer.tests.api.v2.test_alarm_scenarios.TestAlarms.test_post_invalid_alarm_input_enabled_int(mongodb) 2015-06-02 01:11:53.644 | tags: worker-6 2015-06-02 01:11:53.644 | ---------------------------------------------------------------------- 2015-06-02 01:11:53.644 | Empty attachments: 2015-06-02 01:11:53.645 | stderr 2015-06-02 01:11:53.645 | stdout 2015-06-02 01:11:53.645 | 20 | 01:36 |
| rjaiswal | 01:11:53.645 | Traceback (most recent call last): 2015-06-02 01:11:53.645 | File "ceilometer/tests/base.py", line 97, in skip_if_not_implemented 2015-06-02 01:11:53.645 | return func(*args, **kwargs) 2015-06-02 01:11:53.646 | File "ceilometer/tests/api/v2/test_alarm_scenarios.py", line 785, in test_post_invalid_alarm_input_enabled_int 2015-06-02 01:11:53.646 | resp.json['error_message']['faultstring']) 201 | 01:37 |
| rjaiswal | Here is the review - https://review.openstack.org/#/c/187397/ | 01:37 |
| *** chinmay_g has joined #openstack-ceilometer | 01:37 | |
| gordc | lan: hmm... i think it`d be better to be in init still but i see your point | 01:39 |
| gordc | rjaiswal: i`m pretty sure something changed in wsme library that broke us | 01:40 |
| gordc | lan: updated review | 01:41 |
| rjaiswal | gordc: thanks, need to file a bug then in CM | 01:41 |
| rjaiswal | ? | 01:41 |
| rjaiswal | for the broken unit tests | 01:42 |
| gordc | rjaiswal: what does CM stand for? | 01:42 |
| rjaiswal | CeiloMeter | 01:42 |
| gordc | rjaiswal: oh :) | 01:42 |
| rjaiswal | :) | 01:43 |
| gordc | rjaiswal: sure, file a bug. we can link wsme too | 01:43 |
| *** serverascode has joined #openstack-ceilometer | 01:43 | |
| rjaiswal | i mean shud we file a bug in ceilometer launchpad | 01:43 |
| rjaiswal | gordc: ok | 01:43 |
| lan | gordc, thanks for your vote : ) | 01:43 |
| gordc | thanks rjaiswal | 01:43 |
| gordc | lan: no problem. | 01:43 |
| rjaiswal | thanks gordc | 01:45 |
| *** robsparker has joined #openstack-ceilometer | 01:45 | |
| rjaiswal | yep, confirmed its an external problem since my other reviews also started failing CI | 01:47 |
| gordc | rjaiswal: cool cool | 01:48 |
| *** robsparker has quit IRC | 01:49 | |
| rjaiswal | gordc: here is the bug report - https://bugs.launchpad.net/wsme/+bug/1460871 | 01:52 |
| openstack | Launchpad bug 1460871 in WSME "Ceilometer alarm unit test failures" [Undecided,New] | 01:52 |
| gordc | rjaiswal: awesome. marked as high | 01:53 |
| rjaiswal | gordc: cool, thanks | 01:53 |
| *** rex_lee has joined #openstack-ceilometer | 01:55 | |
| *** boris-42 has joined #openstack-ceilometer | 01:55 | |
| *** sputnik13 has joined #openstack-ceilometer | 01:59 | |
| *** alexpilotti has quit IRC | 02:00 | |
| *** liusheng has quit IRC | 02:02 | |
| *** rjaiswal has quit IRC | 02:02 | |
| *** sputnik13 has quit IRC | 02:03 | |
| *** pauldix has quit IRC | 02:04 | |
| openstackgerrit | ZhiQiang Fan proposed openstack/ceilometer: adjust alarm post ut code to adapt to upstream wsme https://review.openstack.org/187425 | 02:11 |
| *** liusheng has joined #openstack-ceilometer | 02:22 | |
| *** shakamunyi has joined #openstack-ceilometer | 02:26 | |
| openstackgerrit | ZhiQiang Fan proposed openstack/ceilometer: adjust alarm post ut code to adapt to upstream wsme https://review.openstack.org/187425 | 02:31 |
| *** shakamunyi has quit IRC | 02:33 | |
| *** robsparker has joined #openstack-ceilometer | 02:46 | |
| *** ddieterly has quit IRC | 02:48 | |
| *** robsparker has quit IRC | 02:51 | |
| *** gordc has quit IRC | 03:37 | |
| *** deepthi has joined #openstack-ceilometer | 04:03 | |
| *** haigang has joined #openstack-ceilometer | 04:19 | |
| *** ddieterly has joined #openstack-ceilometer | 04:19 | |
| *** fawadkhaliq has joined #openstack-ceilometer | 04:21 | |
| *** pauldix has joined #openstack-ceilometer | 04:23 | |
| *** ddieterly has quit IRC | 04:23 | |
| *** david-lyle has quit IRC | 04:24 | |
| *** david-lyle has joined #openstack-ceilometer | 04:25 | |
| *** exploreshaifali has joined #openstack-ceilometer | 04:26 | |
| *** pauldix has quit IRC | 04:28 | |
| *** exploreshaifali has quit IRC | 04:51 | |
| openstackgerrit | Fawad Khaliq proposed openstack/ceilometer-specs: PLUMgrid network monitoring driver spec https://review.openstack.org/186353 | 04:52 |
| *** changbl_ has quit IRC | 04:52 | |
| *** Longgeek has joined #openstack-ceilometer | 05:07 | |
| *** nosnos has joined #openstack-ceilometer | 05:12 | |
| *** yprokule has joined #openstack-ceilometer | 05:12 | |
| *** yprokule has quit IRC | 05:13 | |
| *** Longgeek has quit IRC | 05:13 | |
| *** yprokule has joined #openstack-ceilometer | 05:14 | |
| *** ddieterly has joined #openstack-ceilometer | 05:20 | |
| *** ddieterly has quit IRC | 05:25 | |
| *** tobasco has joined #openstack-ceilometer | 05:25 | |
| *** _cjones_ has joined #openstack-ceilometer | 05:28 | |
| *** nosnos has quit IRC | 05:37 | |
| *** Longgeek has joined #openstack-ceilometer | 05:52 | |
| *** Kennan2 has joined #openstack-ceilometer | 06:06 | |
| *** Kennan has quit IRC | 06:06 | |
| *** ddieterly has joined #openstack-ceilometer | 06:21 | |
| *** ddieterly has quit IRC | 06:25 | |
| *** _cjones_ has quit IRC | 06:30 | |
| *** ildikov has quit IRC | 06:36 | |
| *** belmoreira has joined #openstack-ceilometer | 06:38 | |
| *** safchain has joined #openstack-ceilometer | 06:51 | |
| openstackgerrit | Rohit Jaiswal proposed openstack/ceilometer: Fix catch-all event def to exclude !event https://review.openstack.org/187397 | 07:03 |
| *** haigang has quit IRC | 07:11 | |
| *** haigang has joined #openstack-ceilometer | 07:12 | |
| *** yassine has joined #openstack-ceilometer | 07:14 | |
| *** yassine has quit IRC | 07:14 | |
| *** yassine has joined #openstack-ceilometer | 07:17 | |
| *** yassine has quit IRC | 07:17 | |
| *** yassine has joined #openstack-ceilometer | 07:18 | |
| *** ddieterly has joined #openstack-ceilometer | 07:21 | |
| *** ddieterly has quit IRC | 07:26 | |
| *** ildikov has joined #openstack-ceilometer | 07:27 | |
| *** fawadkhaliq has quit IRC | 07:29 | |
| *** Kennan has joined #openstack-ceilometer | 07:39 | |
| *** Kennan2 has quit IRC | 07:39 | |
| *** Ala has joined #openstack-ceilometer | 07:52 | |
| *** ddieterly has joined #openstack-ceilometer | 08:22 | |
| *** cmyster has joined #openstack-ceilometer | 08:23 | |
| *** ddieterly has quit IRC | 08:27 | |
| openstackgerrit | Julien Danjou proposed openstack/ceilometer-specs: Split Ceilometer alarming https://review.openstack.org/184307 | 08:40 |
| *** promulo_ has quit IRC | 08:40 | |
| *** kbyrne has joined #openstack-ceilometer | 08:53 | |
| *** cdent has joined #openstack-ceilometer | 09:21 | |
| *** fawadkhaliq has joined #openstack-ceilometer | 09:22 | |
| *** ddieterly has joined #openstack-ceilometer | 09:23 | |
| *** ddieterly has quit IRC | 09:27 | |
| *** haigang has quit IRC | 09:50 | |
| *** pauldix has joined #openstack-ceilometer | 09:50 | |
| *** gordc has joined #openstack-ceilometer | 09:51 | |
| *** pauldix has quit IRC | 09:54 | |
| *** _nadya_ has joined #openstack-ceilometer | 10:02 | |
| *** gordc has quit IRC | 10:03 | |
| *** _nadya_ has quit IRC | 10:04 | |
| *** _nadya_ has joined #openstack-ceilometer | 10:04 | |
| *** haigang has joined #openstack-ceilometer | 10:08 | |
| *** _nadya_ has quit IRC | 10:16 | |
| *** robsparker has joined #openstack-ceilometer | 10:17 | |
| *** yassine has quit IRC | 10:19 | |
| *** robsparker has quit IRC | 10:22 | |
| *** haigang has quit IRC | 10:22 | |
| *** ddieterly has joined #openstack-ceilometer | 10:24 | |
| *** haigang has joined #openstack-ceilometer | 10:28 | |
| *** ddieterly has quit IRC | 10:29 | |
| *** dikonoor has joined #openstack-ceilometer | 10:34 | |
| *** yatin has joined #openstack-ceilometer | 10:36 | |
| *** dikonoo has joined #openstack-ceilometer | 10:36 | |
| *** dikonoor has quit IRC | 10:39 | |
| *** alexpilotti has joined #openstack-ceilometer | 10:46 | |
| *** fawadkhaliq has quit IRC | 10:56 | |
| *** Longgeek_ has joined #openstack-ceilometer | 11:01 | |
| *** Longgeek has quit IRC | 11:01 | |
| *** tobasco has quit IRC | 11:17 | |
| *** ddieterly has joined #openstack-ceilometer | 11:24 | |
| *** triggerz has joined #openstack-ceilometer | 11:27 | |
| *** ddieterly has quit IRC | 11:29 | |
| *** robsparker has joined #openstack-ceilometer | 11:32 | |
| *** robsparker has quit IRC | 11:37 | |
| *** ildikov has quit IRC | 11:41 | |
| *** deepthi has quit IRC | 11:42 | |
| *** samueldmq has joined #openstack-ceilometer | 11:45 | |
| samueldmq | morning | 11:46 |
| samueldmq | could someone from ceilometer check if this bug is valid/invalid in your side ? https://bugs.launchpad.net/ceilometer/+bug/1458945 | 11:46 |
| openstack | Launchpad bug 1458945 in murano "Use graduated oslo.policy instead of oslo-incubator code" [High,Confirmed] - Assigned to Ekaterina Chernova (efedorova) | 11:46 |
| *** ddieterly has joined #openstack-ceilometer | 11:56 | |
| *** ildikov has joined #openstack-ceilometer | 11:56 | |
| *** pauldix has joined #openstack-ceilometer | 12:03 | |
| *** haigang has quit IRC | 12:06 | |
| *** haigang has joined #openstack-ceilometer | 12:07 | |
| *** ildikov has quit IRC | 12:12 | |
| *** cmyster has quit IRC | 12:12 | |
| *** cmyster has joined #openstack-ceilometer | 12:13 | |
| *** ildikov has joined #openstack-ceilometer | 12:27 | |
| *** shardy_ has joined #openstack-ceilometer | 12:32 | |
| *** shardy has quit IRC | 12:34 | |
| *** shardy_ has quit IRC | 12:38 | |
| *** shardy has joined #openstack-ceilometer | 12:39 | |
| cdent | samueldmq: ceilometer switched to oslo_policy back in march: https://review.openstack.org/#/c/162881/ | 12:43 |
| cdent | is that what you wanted to know? | 12:43 |
| jasonamyers | FYI Im gonna up date the spec this afternoon | 12:43 |
| * cdent fistbumps jasonamyers | 12:44 | |
| jasonamyers | hey welcome back cdent | 12:44 |
| jasonamyers | how was you break | 12:45 |
| cdent | It was nice to have a break but I spent it with a cold | 12:45 |
| jasonamyers | :( | 12:45 |
| *** ddieterly has quit IRC | 12:47 | |
| *** gordc has joined #openstack-ceilometer | 12:54 | |
| *** ildikov has quit IRC | 12:55 | |
| *** ildikov has joined #openstack-ceilometer | 12:56 | |
| *** yatin has quit IRC | 12:59 | |
| *** yassine has joined #openstack-ceilometer | 13:01 | |
| *** yassine has quit IRC | 13:01 | |
| *** robsparker has joined #openstack-ceilometer | 13:01 | |
| *** yassine has joined #openstack-ceilometer | 13:02 | |
| samueldmq | cdent, yes! great thanks | 13:03 |
| cdent | samueldmq: I went ahead and updated the bug to say that a fix was released | 13:03 |
| samueldmq | cdent, thanks for updating the bug report as well | 13:03 |
| samueldmq | cdent, :) | 13:03 |
| *** ddieterly has joined #openstack-ceilometer | 13:12 | |
| *** Ephur has joined #openstack-ceilometer | 13:13 | |
| *** julim has joined #openstack-ceilometer | 13:17 | |
| *** robsparker has quit IRC | 13:17 | |
| *** Ephur has quit IRC | 13:19 | |
| *** gordc has quit IRC | 13:22 | |
| *** Ephur has joined #openstack-ceilometer | 13:23 | |
| *** kaliya has joined #openstack-ceilometer | 13:30 | |
| *** pauldix has quit IRC | 13:32 | |
| *** _nadya_ has joined #openstack-ceilometer | 13:36 | |
| *** claudiub has joined #openstack-ceilometer | 13:39 | |
| *** ErickHeinrich has joined #openstack-ceilometer | 13:41 | |
| *** ErickHeinrich has quit IRC | 13:41 | |
| *** robsparker has joined #openstack-ceilometer | 13:46 | |
| *** ddieterly has quit IRC | 13:54 | |
| *** rbak has joined #openstack-ceilometer | 13:55 | |
| *** gordc has joined #openstack-ceilometer | 14:00 | |
| *** julim has quit IRC | 14:02 | |
| *** pauldix has joined #openstack-ceilometer | 14:03 | |
| *** julim has joined #openstack-ceilometer | 14:03 | |
| prad | hi ildikov, when you get a chance please glance through https://review.openstack.org/#/c/187339/ | 14:15 |
| ildikov | prad: sure, thanks for the pointer, I have too many meetings these days, but I will try to get to it today | 14:15 |
| prad | ildikov, cool, ty | 14:16 |
| ildikov | prad: np | 14:17 |
| *** boris-42 has quit IRC | 14:18 | |
| kaliya | hi guys, I'm hitting "RuntimeError: No ceilometer.dispatcher extensions found" which prevents me to have ceilometer working... Anything known in upstream? Maybe oslo? | 14:23 |
| *** fawadkhaliq has joined #openstack-ceilometer | 14:26 | |
| *** ddieterly has joined #openstack-ceilometer | 14:34 | |
| openstackgerrit | Merged openstack/ceilometer: adjust alarm post ut code to adapt to upstream wsme https://review.openstack.org/187425 | 14:37 |
| *** gordc_ has joined #openstack-ceilometer | 14:39 | |
| *** gordc has quit IRC | 14:41 | |
| *** _nadya_ has quit IRC | 14:41 | |
| *** jruano has joined #openstack-ceilometer | 14:46 | |
| openstackgerrit | Rohit Jaiswal proposed openstack/ceilometer: Migrate script to alter trait_text value to Text type https://review.openstack.org/178435 | 14:47 |
| openstackgerrit | Rohit Jaiswal proposed openstack/ceilometer: Fix catch-all event def to exclude !event https://review.openstack.org/187397 | 14:47 |
| *** henrique_ has quit IRC | 14:48 | |
| *** htruta has joined #openstack-ceilometer | 14:48 | |
| *** rjaiswal has joined #openstack-ceilometer | 14:48 | |
| *** robsparker has quit IRC | 14:49 | |
| *** exploreshaifali has joined #openstack-ceilometer | 14:51 | |
| *** exploreshaifali has quit IRC | 14:52 | |
| *** robspark_ has joined #openstack-ceilometer | 14:53 | |
| *** lan_ has joined #openstack-ceilometer | 14:58 | |
| *** lan_ has quit IRC | 15:02 | |
| rjaiswal | https://review.openstack.org/#/c/184655/ | 15:06 |
| rjaiswal | Looking to more reviews | 15:06 |
| rjaiswal | for https://review.openstack.org/#/c/184655/ | 15:07 |
| cdent | rjaiswal: I posted a question on that review just now. | 15:16 |
| *** changbl_ has joined #openstack-ceilometer | 15:20 | |
| rjaiswal | thanks cdent | 15:34 |
| openstackgerrit | Chris Dent proposed openstack/ceilometer: Move eventlet using commands into own directory https://review.openstack.org/179054 | 15:36 |
| *** barra204 has joined #openstack-ceilometer | 15:36 | |
| *** shakamunyi has joined #openstack-ceilometer | 15:36 | |
| *** boris-42 has joined #openstack-ceilometer | 15:37 | |
| *** gordc_ is now known as gordc | 15:42 | |
| *** openstackgerrit has quit IRC | 15:42 | |
| *** openstackgerrit has joined #openstack-ceilometer | 15:43 | |
| rjaiswal | cdent: posted a reply to your comment on https://review.openstack.org/#/c/184655/ | 15:46 |
| *** robspark_ has quit IRC | 15:47 | |
| *** _cjones_ has joined #openstack-ceilometer | 15:51 | |
| gordc | rjaiswal: did you ever try doing performance check against: https://review.openstack.org/#/c/178435/ | 16:00 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ceilometer: Updated from global requirements https://review.openstack.org/178332 | 16:04 |
| *** robsparker has joined #openstack-ceilometer | 16:05 | |
| openstackgerrit | Chris Dent proposed openstack/ceilometer: WSME version >=0.7 correctly returns a 405 https://review.openstack.org/187649 | 16:09 |
| rjaiswal | gordc: no | 16:10 |
| *** ildikov has quit IRC | 16:11 | |
| *** fawadkhaliq has quit IRC | 16:12 | |
| *** belmoreira has quit IRC | 16:12 | |
| *** robsparker has quit IRC | 16:16 | |
| *** robspark_ has joined #openstack-ceilometer | 16:20 | |
| rjaiswal | gordc: i believe we want to check the performance of inserting and retrieving large strings in trait_text | 16:20 |
| *** gordc has quit IRC | 16:21 | |
| *** gordc has joined #openstack-ceilometer | 16:21 | |
| *** Ala has quit IRC | 16:23 | |
| *** sputnik13 has joined #openstack-ceilometer | 16:26 | |
| *** yassine has quit IRC | 16:29 | |
| gordc | rjaiswal: hmm.. yeah my only concern was that it would slow things down dramatically. but i guess that would make it consistent with other backends | 16:32 |
| *** liusheng has quit IRC | 16:33 | |
| *** liusheng has joined #openstack-ceilometer | 16:33 | |
| *** robspark_ has quit IRC | 16:37 | |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: Remove AvoidDictInterface https://review.openstack.org/187669 | 16:38 |
| *** robsparker has joined #openstack-ceilometer | 16:40 | |
| *** robsparker has quit IRC | 16:41 | |
| *** safchain has quit IRC | 16:42 | |
| *** dscottsf has joined #openstack-ceilometer | 16:44 | |
| rjaiswal | gordc: How feasible is separating out big_text values into trait_bigtext? | 16:45 |
| gordc | rjaiswal: it'd probably involve an additional level of redirection... we'd have to figure out the length of each string... i'm not sure the gains would justify the investment | 16:48 |
| rjaiswal | gordc: or we can insert to big_text when trait_text insert fails on a specifc error | 16:48 |
| *** robsparker has joined #openstack-ceilometer | 16:50 | |
| *** dscottsf has quit IRC | 16:52 | |
| gordc | rjaiswal: yes that is also an alternative... there's also logic required to know how to filter | 16:52 |
| gordc | when someone queries for certain values | 16:53 |
| gordc | biab. grabbing quick lunch | 16:53 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ceilometer: Updated from global requirements https://review.openstack.org/178332 | 16:56 |
| *** ddieterly has quit IRC | 17:00 | |
| *** ddieterly has joined #openstack-ceilometer | 17:03 | |
| *** ildikov has joined #openstack-ceilometer | 17:05 | |
| *** pauldix has quit IRC | 17:08 | |
| *** pauldix has joined #openstack-ceilometer | 17:22 | |
| rjaiswal | gordc: If performance is a concern, Can we change the model definition - TraitText: value from Column(Text) to Column(String(255)), to make it consistent at least with the migrate script definition? - to make sure our internal model is consistent and make it explicit that we dont support string traits longer than 255 chars | 17:27 |
| gordc | rjaiswal: sure. that is also an option... probably the least invasive. | 17:31 |
| gordc | that said, the error would still throw no? if you gave it a bigger value? | 17:31 |
| *** shardy_ has joined #openstack-ceilometer | 17:33 | |
| *** rjaiswal has quit IRC | 17:33 | |
| openstackgerrit | gordon chung proposed openstack/ceilometer: test https://review.openstack.org/187693 | 17:34 |
| *** shardy has quit IRC | 17:34 | |
| *** shardy_ has quit IRC | 17:38 | |
| openstackgerrit | Chris Dent proposed openstack/ceilometer: Cause gabbi to skip on no storage sooner https://review.openstack.org/187694 | 17:38 |
| *** shardy has joined #openstack-ceilometer | 17:39 | |
| cdent | bbl | 17:40 |
| *** cdent has quit IRC | 17:40 | |
| *** _nadya_ has joined #openstack-ceilometer | 17:40 | |
| *** ildikov has quit IRC | 17:42 | |
| *** yprokule has quit IRC | 17:51 | |
| *** dikonoo has quit IRC | 17:52 | |
| *** ildikov has joined #openstack-ceilometer | 17:55 | |
| *** rjaiswal has joined #openstack-ceilometer | 18:01 | |
| *** edmondsw has joined #openstack-ceilometer | 18:02 | |
| *** jwcroppe has joined #openstack-ceilometer | 18:04 | |
| *** openstackgerrit has quit IRC | 18:09 | |
| *** openstackgerrit has joined #openstack-ceilometer | 18:10 | |
| *** _nadya_ has quit IRC | 18:10 | |
| *** promulo has joined #openstack-ceilometer | 18:14 | |
| ityaptin | gordc: Hi! I've made research about MongoDB aggregation and answered you in the spec https://review.openstack.org/#/c/168349. Could you take a look, please? | 18:16 |
| rjaiswal | gordc: might have missed it, so reposting - If performance is a concern, Can we change the model definition - TraitText: value from Column(Text) to Column(String(255)), to make it consistent at least with the migrate script definition? - to make sure our internal model is consistent and make it explicit that we dont support string traits longer than 255 chars.. | 18:16 |
| rjaiswal | for https://review.openstack.org/#/c/178435/ | 18:18 |
| gordc | rjaiswal: hmm. might not have sent. | 18:18 |
| gordc | 1:31:23 PM gordc rjaiswal: sure. that is also an option... probably the least invasive. 1:31:39 PM gordc that said, the error would still throw no? if you gave it a bigger value? | 18:18 |
| gordc | ityaptin: sure. i'll take a look later today | 18:18 |
| ityaptin | gordc: Thanks! | 18:19 |
| rjaiswal | gordc: yes, the error will still be there | 18:20 |
| rjaiswal | gordc: but it can be addressed either as a not supported or a new feature request.. | 18:21 |
| rjaiswal | i think | 18:21 |
| gordc | rjaiswal: hmm.. i guess we can create another bug for it if we go that route... probably makes sense to either cropped everything that comes in... or make it Text | 18:21 |
| rjaiswal | gordc:sorry, missed your messages, was out for a while. | 18:21 |
| rjaiswal | gordc: yes, agree with that, so i will change this https://review.openstack.org/178435 to fix the model to be String(255) | 18:24 |
| gordc | rjaiswal: will you crop the messages that we receive too? to avoid errors being thrown? | 18:25 |
| gordc | i think the main point of the bug is to avoid the errors | 18:25 |
| rjaiswal | sure, you mean suppress the errors | 18:26 |
| rjaiswal | we can log it | 18:26 |
| rjaiswal | and not throw it back | 18:26 |
| gordc | rjaiswal: right | 18:27 |
| gordc | let | 18:27 |
| gordc | let's try that for now... we can switch to Text if someone actually wants it. | 18:28 |
| rjaiswal | sure | 18:28 |
| rjaiswal | Thanks gordc | 18:28 |
| gordc | rjaiswal: awesome. thanks to you. | 18:28 |
| *** rjaiswal has quit IRC | 18:35 | |
| *** jwcroppe has quit IRC | 18:43 | |
| *** alexpilotti has quit IRC | 18:52 | |
| openstackgerrit | Merged openstack/ceilometer: remove useless log message https://review.openstack.org/186658 | 18:55 |
| *** openstackgerrit has quit IRC | 18:56 | |
| *** openstackgerrit has joined #openstack-ceilometer | 18:56 | |
| *** belmoreira has joined #openstack-ceilometer | 19:20 | |
| *** cdent has joined #openstack-ceilometer | 19:21 | |
| cdent | gordc and others, qa folks want some additional ceilo oriented review on https://review.openstack.org/#/c/182083/ | 19:46 |
| gordc | cdent: sure. | 19:47 |
| cdent | i think basically to confirm that someone besides me wants it | 19:49 |
| gordc | cdent: what if you're the only one? | 19:49 |
| *** belmoreira has quit IRC | 19:49 | |
| cdent | then I will cut you | 19:50 |
| gordc | daymmmnnn that escalated real quick | 19:51 |
| gordc | good thing i wasn't drinking anything when i read that. | 19:52 |
| cdent | no point waitin around | 19:52 |
| * cdent sharpens his knife | 19:52 | |
| gordc | i like your can do attitude. | 19:52 |
| cdent | \o/ | 19:53 |
| gordc | that's why i nominated you as core... start jabbing | 19:53 |
| *** ddaskal has joined #openstack-ceilometer | 19:54 | |
| * cdent lives to serve | 19:54 | |
| * cdent retires for the evening | 19:56 | |
| *** cdent has quit IRC | 19:56 | |
| *** samueldmq has quit IRC | 20:05 | |
| *** ddaskal has quit IRC | 20:08 | |
| *** openstackgerrit has quit IRC | 20:10 | |
| *** openstackgerrit has joined #openstack-ceilometer | 20:10 | |
| *** ddaskal has joined #openstack-ceilometer | 20:12 | |
| *** ddaskal has quit IRC | 20:28 | |
| *** julim has quit IRC | 20:32 | |
| *** ddaskal has joined #openstack-ceilometer | 20:45 | |
| *** robsparker has quit IRC | 21:01 | |
| *** shakamunyi_ has joined #openstack-ceilometer | 21:03 | |
| *** shakamunyi__ has joined #openstack-ceilometer | 21:04 | |
| *** shakamunyi__ is now known as tylerdurden | 21:05 | |
| *** tylerdurden has quit IRC | 21:08 | |
| *** shakamunyi_ has quit IRC | 21:08 | |
| *** robsparker has joined #openstack-ceilometer | 21:10 | |
| *** shardy has quit IRC | 21:13 | |
| *** leomazzo has joined #openstack-ceilometer | 21:17 | |
| openstackgerrit | Rohit Jaiswal proposed openstack/ceilometer: TraitText value restricted to max length 255 https://review.openstack.org/178435 | 21:20 |
| *** leomazzo has quit IRC | 21:22 | |
| openstackgerrit | Rohit Jaiswal proposed openstack/ceilometer: TraitText value restricted to max length 255 https://review.openstack.org/178435 | 21:31 |
| *** jruano has quit IRC | 21:32 | |
| *** rjaiswal has joined #openstack-ceilometer | 21:34 | |
| rjaiswal | https://bugs.launchpad.net/ceilometer/+bug/1460808 | 21:34 |
| openstack | Launchpad bug 1460808 in Ceilometer "Excluding an event in event_definitions doesnt prevent the event from being excluded" [Undecided,In progress] - Assigned to Rohit Jaiswal (rohit-jaiswal-3) | 21:34 |
| *** claudiub has quit IRC | 21:45 | |
| *** edmondsw has quit IRC | 21:55 | |
| *** fawadkhaliq has joined #openstack-ceilometer | 21:57 | |
| *** primechuck has quit IRC | 22:04 | |
| *** jasonamyers has quit IRC | 22:05 | |
| *** karolyn has joined #openstack-ceilometer | 22:13 | |
| *** ddieterly has quit IRC | 22:30 | |
| *** gordc has quit IRC | 22:32 | |
| *** robsparker has quit IRC | 22:39 | |
| *** rbak has quit IRC | 22:42 | |
| *** jasonamyers has joined #openstack-ceilometer | 22:43 | |
| *** ddaskal has quit IRC | 22:50 | |
| *** liusheng has quit IRC | 22:55 | |
| *** liusheng has joined #openstack-ceilometer | 22:56 | |
| openstackgerrit | Rohit Jaiswal proposed openstack/ceilometer: TraitText value restricted to max length 255 https://review.openstack.org/178435 | 23:08 |
| openstackgerrit | Rohit Jaiswal proposed openstack/ceilometer: TraitText value restricted to max length 255 https://review.openstack.org/178435 | 23:11 |
| *** jeraldv has joined #openstack-ceilometer | 23:15 | |
| *** changbl_ has quit IRC | 23:30 | |
| *** pauldix has quit IRC | 23:31 | |
| openstackgerrit | Rohit Jaiswal proposed openstack/ceilometer: Handle database failures on api startup https://review.openstack.org/184655 | 23:35 |
| *** jeraldv has quit IRC | 23:39 | |
| openstackgerrit | Rohit Jaiswal proposed openstack/ceilometer: Handle database failures on api startup https://review.openstack.org/184655 | 23:42 |
| *** yfujioka has joined #openstack-ceilometer | 23:55 | |
| *** barra204 has quit IRC | 23:57 | |
| *** shakamunyi has quit IRC | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!