| *** rbowen has joined #openstack-ceilometer | 00:00 | |
| *** raymondr has quit IRC | 00:01 | |
| *** julim has quit IRC | 00:09 | |
| *** fnaval has joined #openstack-ceilometer | 00:12 | |
| *** isviridov has quit IRC | 00:20 | |
| *** matsuhashi has joined #openstack-ceilometer | 00:22 | |
| *** isviridov has joined #openstack-ceilometer | 00:22 | |
| *** jdonalds has quit IRC | 00:29 | |
| *** r0j4z0 has joined #openstack-ceilometer | 00:36 | |
| *** rbowen has quit IRC | 00:48 | |
| *** _cjones_ has quit IRC | 00:48 | |
| *** _cjones_ has joined #openstack-ceilometer | 00:48 | |
| *** nati_ueno has quit IRC | 00:50 | |
| *** _cjones_ has quit IRC | 00:53 | |
| *** liusheng has quit IRC | 00:58 | |
| *** raymondr has joined #openstack-ceilometer | 01:03 | |
| *** fnaval has quit IRC | 01:05 | |
| *** fnaval has joined #openstack-ceilometer | 01:05 | |
| *** raymondr has quit IRC | 01:08 | |
| *** shakamunyi has quit IRC | 01:16 | |
| *** nati_ueno has joined #openstack-ceilometer | 01:21 | |
| *** liusheng has joined #openstack-ceilometer | 01:28 | |
| liusheng | morning all :) | 01:28 |
|---|---|---|
| *** nati_ueno has quit IRC | 01:32 | |
| *** nati_ueno has joined #openstack-ceilometer | 01:42 | |
| *** nati_ueno has quit IRC | 01:42 | |
| *** nosnos has joined #openstack-ceilometer | 01:45 | |
| *** Ruetobas has quit IRC | 02:06 | |
| *** Ruetobas has joined #openstack-ceilometer | 02:12 | |
| *** jdonalds has joined #openstack-ceilometer | 02:14 | |
| *** Ruetobas has quit IRC | 02:16 | |
| *** Ruetobas has joined #openstack-ceilometer | 02:23 | |
| *** raymondr_ has joined #openstack-ceilometer | 02:36 | |
| *** jdonalds has quit IRC | 02:39 | |
| *** liusheng has quit IRC | 02:48 | |
| *** matsuhashi has quit IRC | 03:00 | |
| *** matsuhashi has joined #openstack-ceilometer | 03:04 | |
| openstackgerrit | ZhiQiang Fan proposed a change to openstack/ceilometer: Ignore the generated file ceilometer.conf.sample https://review.openstack.org/93878 | 03:16 |
| *** nosnos has quit IRC | 03:33 | |
| *** matsuhashi has quit IRC | 03:34 | |
| *** matsuhashi has joined #openstack-ceilometer | 03:35 | |
| *** matsuhashi has quit IRC | 03:39 | |
| *** shakamunyi has joined #openstack-ceilometer | 03:49 | |
| *** matsuhashi has joined #openstack-ceilometer | 03:56 | |
| *** liusheng has joined #openstack-ceilometer | 03:57 | |
| *** shakamunyi has quit IRC | 04:05 | |
| *** renlt has joined #openstack-ceilometer | 04:18 | |
| *** psharma has joined #openstack-ceilometer | 04:18 | |
| *** raymondr_ has quit IRC | 04:27 | |
| *** jdonalds has joined #openstack-ceilometer | 04:31 | |
| openstackgerrit | ZhiQiang Fan proposed a change to openstack/ceilometer: Enable filter alarms by their type https://review.openstack.org/93658 | 04:33 |
| *** raymondr has joined #openstack-ceilometer | 04:39 | |
| *** jdonalds has quit IRC | 04:40 | |
| *** raymondr has quit IRC | 04:42 | |
| *** nosnos has joined #openstack-ceilometer | 04:45 | |
| *** jdonalds has joined #openstack-ceilometer | 04:53 | |
| *** xiej has joined #openstack-ceilometer | 04:56 | |
| *** xiej has quit IRC | 04:58 | |
| *** jdonalds has quit IRC | 04:59 | |
| *** jdonalds has joined #openstack-ceilometer | 05:02 | |
| *** shakamunyi has joined #openstack-ceilometer | 05:03 | |
| openstackgerrit | A change was merged to openstack/ceilometer: Update the copyright date in doc https://review.openstack.org/96195 | 05:04 |
| *** jdonalds has quit IRC | 05:06 | |
| *** shakamunyi has quit IRC | 05:14 | |
| *** fnaval_ has joined #openstack-ceilometer | 05:34 | |
| *** fnaval has quit IRC | 05:36 | |
| *** shakamunyi has joined #openstack-ceilometer | 05:40 | |
| *** raymondr has joined #openstack-ceilometer | 05:43 | |
| openstackgerrit | A change was merged to openstack/python-ceilometerclient: fixed several pep8 issues https://review.openstack.org/93515 | 05:46 |
| *** raymondr has quit IRC | 05:47 | |
| *** shakamunyi has quit IRC | 05:50 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/88506 | 06:01 |
| *** fnaval_ has quit IRC | 06:07 | |
| *** ildikov has quit IRC | 06:13 | |
| *** _nadya_ has joined #openstack-ceilometer | 06:20 | |
| *** fnaval has joined #openstack-ceilometer | 06:23 | |
| *** _nadya_ has quit IRC | 06:25 | |
| *** inc0 has joined #openstack-ceilometer | 06:27 | |
| *** ildikov has joined #openstack-ceilometer | 06:40 | |
| *** fnaval has quit IRC | 06:42 | |
| openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: Fix doc gate job false success https://review.openstack.org/96258 | 06:46 |
| *** shakamunyi has joined #openstack-ceilometer | 06:47 | |
| *** shakamunyi has quit IRC | 06:59 | |
| *** fnaval has joined #openstack-ceilometer | 07:00 | |
| *** _nadya_ has joined #openstack-ceilometer | 07:01 | |
| *** jdonalds has joined #openstack-ceilometer | 07:03 | |
| *** fnaval has quit IRC | 07:05 | |
| *** _nadya_ has quit IRC | 07:18 | |
| *** shakamunyi has joined #openstack-ceilometer | 07:26 | |
| openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: Fix doc gate job false success https://review.openstack.org/96258 | 07:29 |
| *** shakamunyi has quit IRC | 07:36 | |
| *** r0j4z0 has quit IRC | 07:37 | |
| *** zqfan has quit IRC | 07:41 | |
| *** raymondr has joined #openstack-ceilometer | 07:44 | |
| *** raymondr has quit IRC | 07:48 | |
| openstackgerrit | liusheng proposed a change to openstack/ceilometer: Improve the timestamp validation of ceilometer API https://review.openstack.org/92786 | 07:50 |
| *** zhikunliu has joined #openstack-ceilometer | 07:55 | |
| *** matsuhashi has quit IRC | 07:58 | |
| *** matsuhashi has joined #openstack-ceilometer | 07:59 | |
| *** matsuhas_ has joined #openstack-ceilometer | 08:02 | |
| *** zqfan has joined #openstack-ceilometer | 08:03 | |
| *** shakamunyi has joined #openstack-ceilometer | 08:03 | |
| *** matsuhashi has quit IRC | 08:04 | |
| *** cdent has joined #openstack-ceilometer | 08:12 | |
| *** shakamunyi has quit IRC | 08:14 | |
| *** cdent has quit IRC | 08:16 | |
| *** eglynn has joined #openstack-ceilometer | 08:23 | |
| *** cdent has joined #openstack-ceilometer | 08:23 | |
| *** r0j4z0 has joined #openstack-ceilometer | 08:40 | |
| *** shakamunyi has joined #openstack-ceilometer | 08:41 | |
| *** raymondr has joined #openstack-ceilometer | 08:43 | |
| *** raymondr has quit IRC | 08:47 | |
| *** shakamunyi has quit IRC | 08:52 | |
| *** matsuhas_ has quit IRC | 08:53 | |
| *** matsuhashi has joined #openstack-ceilometer | 08:58 | |
| openstackgerrit | A change was merged to openstack/ceilometer: Ignore the generated file ceilometer.conf.sample https://review.openstack.org/93878 | 08:59 |
| *** inc0_ has joined #openstack-ceilometer | 09:00 | |
| *** inc0 has quit IRC | 09:02 | |
| *** zqfan has quit IRC | 09:07 | |
| *** zqfan has joined #openstack-ceilometer | 09:08 | |
| *** renlt has quit IRC | 09:12 | |
| *** fnaval has joined #openstack-ceilometer | 09:18 | |
| *** shakamunyi has joined #openstack-ceilometer | 09:19 | |
| *** pnascimento has joined #openstack-ceilometer | 09:21 | |
| DinaBelova | jd__, o/ Should be the swift container created automatically by gnocchi - or this shuold be done manually? | 09:24 |
| *** zhikunliu has quit IRC | 09:30 | |
| *** shakamunyi has quit IRC | 09:31 | |
| *** matsuhashi has quit IRC | 09:33 | |
| *** matsuhashi has joined #openstack-ceilometer | 09:33 | |
| *** matsuhashi has quit IRC | 09:35 | |
| *** matsuhashi has joined #openstack-ceilometer | 09:36 | |
| DinaBelova | jd__ - I simply have strange situation trying to create entity in the gnocchi and I have 404 not found for the Swift container :D the funny fact is I see the id of the container in the request to swift, that makes me really suspicious) | 09:36 |
| *** fnaval has quit IRC | 09:38 | |
| openstackgerrit | Christian Berendt proposed a change to openstack/ceilometer: use logger instead of LOG as name for the Logger object attribute https://review.openstack.org/95666 | 09:42 |
| DinaBelova | amalagon - did you try this thing? | 09:43 |
| *** shakamunyi has joined #openstack-ceilometer | 09:58 | |
| *** idegtiarov has joined #openstack-ceilometer | 10:00 | |
| *** nosnos has quit IRC | 10:02 | |
| *** nosnos has joined #openstack-ceilometer | 10:05 | |
| *** shakamunyi has quit IRC | 10:05 | |
| *** isviridov has quit IRC | 10:10 | |
| *** isviridov has joined #openstack-ceilometer | 10:18 | |
| openstackgerrit | Igor Degtiarov proposed a change to openstack/ceilometer-specs: Hbase-events-feature specs https://review.openstack.org/96417 | 10:35 |
| openstackgerrit | Igor Degtiarov proposed a change to openstack/ceilometer-specs: Hbase-events-feature specs https://review.openstack.org/96417 | 10:38 |
| * cdent gets lunch and runs errands | 10:40 | |
| *** matsuhashi has quit IRC | 10:41 | |
| *** nosnos has quit IRC | 10:41 | |
| jd__ | DinaBelova: do you have more information on your issue? | 10:41 |
| jd__ | DinaBelova: the container is created by gnocchi.storage.swift in __init__ IIRC | 10:42 |
| DinaBelova | jd__ - http://paste.openstack.org/show/wlJ0bXoIArMk3xWUEcrN/ | 10:42 |
| DinaBelova | jd__ - I used this script to ping the API http://paste.openstack.org/show/9Rv4k9rk4Q4RygzCcCdG/ | 10:43 |
| jd__ | DinaBelova: hum I think it's a Swift issue, like it looks like your admin account does not exist | 10:45 |
| jd__ | the user exists in tempauth, but the swift account is not created | 10:45 |
| jd__ | something like that | 10:45 |
| *** nosnos has joined #openstack-ceilometer | 10:45 | |
| jd__ | checks that it works with the 'swift' client CLI first :) | 10:45 |
| DinaBelova | okay, will try to look on this. | 10:45 |
| jd__ | s/checks/check/ | 10:45 |
| DinaBelova | yes, it works) | 10:45 |
| jd__ | you can create a container with the same parameter you provided to Gnocchi? | 10:46 |
| DinaBelova | well, I tried listing | 10:46 |
| DinaBelova | as I'm not quite experienced in swift :d | 10:46 |
| jd__ | check if it does the same requests, and also use TEMPAUTH_ | 10:46 |
| jd__ | ok :) | 10:46 |
| *** nosnos has quit IRC | 10:46 | |
| *** nosnos has joined #openstack-ceilometer | 10:47 | |
| *** nosnos has quit IRC | 10:48 | |
| *** cdent_ has joined #openstack-ceilometer | 10:50 | |
| *** cdent has quit IRC | 10:50 | |
| *** cdent_ is now known as cdent | 10:50 | |
| openstackgerrit | Igor Degtiarov proposed a change to openstack/ceilometer-specs: Hbase-events-feature specs https://review.openstack.org/96417 | 10:57 |
| DinaBelova | jd__ - I've restarted devstack, this error is gone) | 11:02 |
| DinaBelova | I have new one :D | 11:02 |
| DinaBelova | jd__ - btw - it there a db-sync for gnocchi or not yet? | 11:03 |
| openstackgerrit | liusheng proposed a change to openstack/ceilometer: Improve the timestamp validation of ceilometer API https://review.openstack.org/92786 | 11:07 |
| jd__ | no db sync yet | 11:11 |
| jd__ | what's your error DinaBelova ? | 11:11 |
| *** IvanBerezovskiy has joined #openstack-ceilometer | 11:11 | |
| DinaBelova | no tables in my sql db)) | 11:12 |
| DinaBelova | that's why I asked about the db-sync) | 11:12 |
| jd__ | haha yeah that's something missing I guess | 11:12 |
| jd__ | there's a upgrade() method in the driver | 11:12 |
| jd__ | but I didn't write an entry point for that I guess | 11:12 |
| jd__ | he you got your first patch | 11:12 |
| jd__ | :D | 11:12 |
| DinaBelova | jd__, yes, sure :D | 11:14 |
| DinaBelova | will do :D | 11:14 |
| openstackgerrit | Igor Degtiarov proposed a change to openstack/ceilometer-specs: Hbase-events-feature specs https://review.openstack.org/96417 | 11:14 |
| *** pnascimento has quit IRC | 11:14 | |
| *** prad_ has joined #openstack-ceilometer | 11:17 | |
| *** liusheng has quit IRC | 11:17 | |
| *** prad__ has joined #openstack-ceilometer | 11:22 | |
| *** prad_ has quit IRC | 11:22 | |
| openstackgerrit | Igor Degtiarov proposed a change to openstack/ceilometer-specs: Hbase-events-feature specs https://review.openstack.org/96417 | 11:26 |
| amalagon | /away | 11:29 |
| amalagon | sorry, ignore the above | 11:30 |
| *** prad__ has quit IRC | 11:38 | |
| openstackgerrit | Mitsuru Kanabuchi proposed a change to openstack/ceilometer: Add the function of deleting alarm history https://review.openstack.org/87869 | 11:44 |
| *** zqfan_ has joined #openstack-ceilometer | 11:46 | |
| DinaBelova | jd__ - https://review.openstack.org/#/c/96426/1 | 11:47 |
| Alexei_987 | stupid question: def make_timestamp_range(start, end, start_timestamp_op=None, end_timestamp_op=None) Why do we need timestamp_op ?? | 11:50 |
| jd__ | DinaBelova: cool I'll check that, don't worry your patch will fail, I still have CI issues to fix | 11:50 |
| DinaBelova | okay, got it | 11:50 |
| DinaBelova | I've tried it on my testing env - after its running I have sqlite db and entity recorded | 11:51 |
| eglynn | Alexei_987: ... possibly to distinguish between <= and <? | 11:54 |
| Alexei_987 | eglynn: it doesn't make sense | 11:54 |
| Alexei_987 | eglynn: just like comparing float == float | 11:54 |
| DinaBelova | eglynn, o/ I guess I have bad-bad English)) I'm about is/are question in the https://review.openstack.org/#/c/96417/ :) | 11:54 |
| Alexei_987 | date in milliseconds would never be equal | 11:55 |
| Alexei_987 | so it just adds additional code complexity without any reason | 11:55 |
| eglynn | Alexei_987: ... never be equal, or unlikely to be equal? | 11:56 |
| eglynn | Alexei_987: ... do we actually have millisecond granularity on all timestamps? | 11:56 |
| openstackgerrit | Mitsuru Kanabuchi proposed a change to openstack/ceilometer: Split bunch of samples into multiple messages https://review.openstack.org/94155 | 11:56 |
| Alexei_987 | eglynn: in most backends we do | 11:57 |
| Alexei_987 | eglynn: basically when you query an API there is no way to know that you have sample with date exactly equal | 11:57 |
| Alexei_987 | so IMO no need to support <= operator | 11:57 |
| Alexei_987 | eglynn: < would be enough for everybody | 11:58 |
| *** dmsimard has left #openstack-ceilometer | 11:58 | |
| *** prad_ has joined #openstack-ceilometer | 12:00 | |
| amalagon | hey eglynn thanks for the comments :) | 12:02 |
| eglynn | Alexei_987: ... do we support such a distinction still in the API? | 12:02 |
| eglynn | Alexei_987: ... so even if it's unlikely to make a difference in practice whether the user chooses < or <= | 12:02 |
| eglynn | Alexei_987: ... seems wrong to ignore their preference, or? | 12:03 |
| Alexei_987 | eglynn: actually I care more about driver level | 12:03 |
| Alexei_987 | eglynn: if user wants <= we can simply subtract 1 second/millisecond from his query and use < | 12:03 |
| eglynn | Alexei_987: ... the point is that we *allow* the API caller currently to use <= | 12:04 |
| Alexei_987 | eglynn: yes :) | 12:05 |
| eglynn | Alexei_987: ... how to force then to use < and subtract 1 milli as you suggest? | 12:05 |
| Alexei_987 | eglynn: they can use <= in the API but in driver we can use < with -1 second | 12:06 |
| eglynn | Alexei_987: ... and is the code simplification sufficient to justify leaking that into the API? | 12:06 |
| Alexei_987 | eglynn: ok nevermind that :) | 12:06 |
| Alexei_987 | eglynn: I just don't like this part of the code | 12:06 |
| Alexei_987 | eglynn: but i'm ok with it being around | 12:07 |
| eglynn | Alexei_987: I hear ya ... I just don't think it's important enough to worry about | 12:07 |
| *** prad_ has quit IRC | 12:14 | |
| *** prad_ has joined #openstack-ceilometer | 12:15 | |
| r0j4z0 | hi | 12:15 |
| r0j4z0 | i think i found a bug on ceilometer | 12:16 |
| r0j4z0 | when requesting trait information | 12:16 |
| r0j4z0 | of an event | 12:16 |
| eglynn | r0j4z0: can you file a report in launchpad for that bug? | 12:20 |
| r0j4z0 | im trying to do it now im firstimer... | 12:21 |
| r0j4z0 | :D | 12:21 |
| DinaBelova | r0j4z0, yep - please just create account first ))) | 12:21 |
| DinaBelova | then click 'Report a bug' in the right column on the https://launchpad.net/ceilometer | 12:21 |
| DinaBelova | Summary is short line like 'my-bug-about-this' | 12:22 |
| r0j4z0 | ok :D i try | 12:22 |
| DinaBelova | and then it'll be a field for the extra information - you may place there traces, symptoms, etc. | 12:22 |
| r0j4z0 | im not sure if is a bug | 12:24 |
| r0j4z0 | can you try it ? is on icehouse | 12:24 |
| r0j4z0 | when i try to get a integer trait | 12:24 |
| r0j4z0 | i get this error: WARNING (http:173) Request returned failure status. | 12:24 |
| r0j4z0 | HTTPBadRequest (HTTP 400) ERROR Invalid input for field/attribute value. Value: '20'. Wrong type. Expected '<type 'unicode'>', got '<type 'long'>' | 12:24 |
| r0j4z0 | but if i get a string trait it works fine | 12:25 |
| openstackgerrit | Igor Degtiarov proposed a change to openstack/ceilometer-specs: Hbase-events-feature specs https://review.openstack.org/96417 | 12:29 |
| *** fnaval has joined #openstack-ceilometer | 12:30 | |
| r0j4z0 | https://bugs.launchpad.net/ceilometer/+bug/1324523 | 12:34 |
| r0j4z0 | :D | 12:34 |
| *** cdent has quit IRC | 12:39 | |
| *** eglynn has quit IRC | 12:39 | |
| *** zqfan_ has quit IRC | 12:40 | |
| amalagon | o/ DinaBelova | 12:40 |
| amalagon | the Mismatch error strikes again in Jenkins | 12:41 |
| *** gordc has joined #openstack-ceilometer | 12:46 | |
| *** raymondr has joined #openstack-ceilometer | 12:51 | |
| *** cdent has joined #openstack-ceilometer | 12:51 | |
| *** raymondr has quit IRC | 12:52 | |
| *** _nadya_ has joined #openstack-ceilometer | 12:58 | |
| DinaBelova | amalagon, it looks like issues Julien has mentioned earlier) | 12:59 |
| *** fnaval_ has joined #openstack-ceilometer | 13:01 | |
| *** rbowen has joined #openstack-ceilometer | 13:03 | |
| *** fnaval has quit IRC | 13:03 | |
| *** eglynn has joined #openstack-ceilometer | 13:06 | |
| *** thomasem has joined #openstack-ceilometer | 13:06 | |
| *** cmart has joined #openstack-ceilometer | 13:08 | |
| *** jaypipes has quit IRC | 13:10 | |
| *** jdob has joined #openstack-ceilometer | 13:11 | |
| *** ddieterly has joined #openstack-ceilometer | 13:13 | |
| *** eglynn is now known as eglynn|afk | 13:14 | |
| *** eglynn|afk is now known as eglynn_afk | 13:14 | |
| *** eglynn_afk is now known as eglynn-away | 13:14 | |
| *** prad_ has quit IRC | 13:15 | |
| *** jaypipes has joined #openstack-ceilometer | 13:16 | |
| *** shakamunyi has joined #openstack-ceilometer | 13:16 | |
| *** shakayumi has joined #openstack-ceilometer | 13:19 | |
| *** julim has joined #openstack-ceilometer | 13:21 | |
| *** shakamunyi has quit IRC | 13:21 | |
| cdent | cmart you around? | 13:24 |
| cmart | cdent: hi! yes, I'm here | 13:25 |
| cdent | can you remind me of that query that was giving you grief? | 13:25 |
| cmart | sure: https://github.com/openstack/python-ceilometerclient/blob/master/ceilometerclient/v2/options.py#L84 | 13:25 |
| cdent | yeah, I remember that, but what was the string you were passing that led you there? | 13:26 |
| cmart | "state=string::insufficient data" | 13:26 |
| cmart | the error was that the pattern doesn´t allow whitespaces.. | 13:26 |
| cmart | I got confused by the ^ right at the beginning of the last group | 13:27 |
| * cdent nods | 13:27 | |
| cmart | I solved it by removing the ¬v and the whitespace right next to the ^ symbol | 13:28 |
| cmart | ([^\t\n\r\f]+) | 13:28 |
| cmart | (I also removed the restrictions related to commas and "-", because we could send strings with that kind of characters | 13:28 |
| cmart | I guess that once I upload my patch, we can discuss it appropriately :) | 13:29 |
| *** fnaval_ has quit IRC | 13:30 | |
| *** eglynn-away is now known as eglynn | 13:32 | |
| cdent | split_by_op seems over specified to me | 13:32 |
| cdent | it ought to be doable as just one much simpler line | 13:32 |
| * cdent tries something | 13:32 | |
| *** shakayumi has quit IRC | 13:34 | |
| cmart | OK! | 13:36 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/python-ceilometerclient: Updated from global requirements https://review.openstack.org/91238 | 13:47 |
| *** shakayumi has joined #openstack-ceilometer | 13:48 | |
| *** prad_ has joined #openstack-ceilometer | 13:49 | |
| *** prad_ has quit IRC | 13:53 | |
| *** fnaval has joined #openstack-ceilometer | 13:55 | |
| *** raymondr has joined #openstack-ceilometer | 13:58 | |
| *** prad_ has joined #openstack-ceilometer | 14:00 | |
| *** prad__ has joined #openstack-ceilometer | 14:03 | |
| *** prad_ has quit IRC | 14:03 | |
| *** prad__ is now known as prad_ | 14:04 | |
| *** ildikov_ has joined #openstack-ceilometer | 14:05 | |
| cdent | cmart I think I have a patch which covers your needs as well as addressing my concerns with the regex | 14:06 |
| *** ildikov has quit IRC | 14:06 | |
| * cdent runs many tests | 14:07 | |
| *** inc0_ has quit IRC | 14:09 | |
| *** psharma has quit IRC | 14:14 | |
| *** yfujioka has joined #openstack-ceilometer | 14:19 | |
| *** ityaptin has joined #openstack-ceilometer | 14:19 | |
| idegtiarov | eglynn: Eoghan it was some strange happened with my answer for your comment, I have added it separately below in comments. | 14:20 |
| DinaBelova | idegtiarov - just appeared :-\ | 14:20 |
| DinaBelova | hehe | 14:21 |
| DinaBelova | funny Gerrit) | 14:21 |
| DinaBelova | eglynn, anyway - I'm writing two specs to the qa-specs now | 14:21 |
| DinaBelova | for the ceilometer | 14:21 |
| DinaBelova | based on two existing BPs | 14:21 |
| DinaBelova | events feature will be the part of scenario testing | 14:21 |
| DinaBelova | like alarms | 14:21 |
| idegtiarov | thanks Dina, while I try to write that it will be a part of spec you are going to prepare! You have already write about it! :)) | 14:23 |
| openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: Fix doc gate job false success https://review.openstack.org/96258 | 14:24 |
| eglynn | DinaBelova: gremlins in the gerrit machine stealing comments? ;) | 14:24 |
| DinaBelova | idegtiarov, ;) | 14:24 |
| DinaBelova | eglynn, hehe | 14:24 |
| openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: Fix doc gate job false success https://review.openstack.org/96258 | 14:24 |
| DinaBelova | my little secret) | 14:24 |
| *** jdonalds has quit IRC | 14:26 | |
| *** ildikov_ has quit IRC | 14:26 | |
| *** ildikov has joined #openstack-ceilometer | 14:26 | |
| cdent | anyone: is there a way to indicate in a commit that the current patch obsoletes a pending but not merged patch? | 14:28 |
| cdent | i've got a changed to ceilometerclient that makes this one redundant: https://review.openstack.org/#/c/92074/ | 14:28 |
| cdent | cmart did you make a bug for your issue? | 14:30 |
| cmart | I have a bug related to that | 14:30 |
| cmart | but I could create a bug for that, sure | 14:31 |
| nsaje | cdent: I'd simply -1 the redundant one and post an explanation in the comment | 14:31 |
| cdent | cmart unless it is necessary to have a bug for the record, I can just submit the patch for review | 14:32 |
| cdent | nsaje: thanks | 14:32 |
| eglynn | cdent: normally you could -1 the patch and reference the new patch in the comment | 14:32 |
| eglynn | cdent: ... however if the pre-existing patch already has a core +2 | 14:32 |
| cdent | yeah, in this case it does... | 14:33 |
| eglynn | cdent: ... why not just rebase your patch on top of that? | 14:33 |
| eglynn | cdent: ... assuming your patch does something similar AND more | 14:33 |
| cdent | it changes all the same code, but in a rather different way | 14:33 |
| eglynn | cdent: ... a-ha so splitting out the "AND more" bit into a separate patch rebased on the original patch isn't feasible? | 14:34 |
| cdent | I _think_ rebasing it against the existing will obscure the point | 14:34 |
| eglynn | cdent: ... is the original wrong, or just incomplete? | 14:34 |
| cdent | and the +2 patch removes many of the "security" protections in the original code | 14:35 |
| cdent | so if I were king I'd -1 the existing thing... | 14:35 |
| nsaje | cdent: you don't need to be a king to -1 a patch :-D | 14:35 |
| cdent | not even for one day? | 14:36 |
| cdent | ;) | 14:36 |
| eglynn | cdent: ... k, in that case -1 the original patch and just clearly state it's deficiencies | 14:36 |
| eglynn | cdent: ... and add a link to your more complete / more correct patch | 14:36 |
| nsaje | cdent: you can be king for the next 20ms! | 14:37 |
| *** nati_ueno has joined #openstack-ceilometer | 14:37 | |
| cdent | hmmm, actually I think I will rebase to the existing, because parts of it are better | 14:38 |
| eglynn | cdent: ... OK definitely better to rebase if that's the case | 14:38 |
| *** yfujioka has quit IRC | 14:39 | |
| cdent | what's the procedure there, my rebase foo is weak | 14:39 |
| eglynn | cdent: ... "nicer" from an etiquette PoV as well :) | 14:39 |
| eglynn | cdent: procedure from a git PoV? | 14:40 |
| cdent | yes | 14:40 |
| cdent | i know how to rebase in general | 14:40 |
| cdent | but rebasing to the thing sitting on gerrit | 14:40 |
| cdent | ...? | 14:40 |
| eglynn | cdent: well create a fresh branch first | 14:40 |
| eglynn | cdent: then ... $ git fetch https://review.openstack.org/openstack/python-ceilometerclient refs/changes/74/92074/1 && git format-patch -1 --stdout FETCH_HEAD > original | 14:40 |
| eglynn | cdent: $ git am original | 14:41 |
| eglynn | cdent: is your patch already up on gerrit? | 14:41 |
| cdent | no not yet | 14:41 |
| eglynn | cdent: ... if not: $ git format-patch -1 -o my_patch.patch # <=== on the old branch | 14:42 |
| eglynn | cdent: ... switch back to new branch | 14:42 |
| eglynn | cdent: $ git am my_patch.patch --reject | 14:42 |
| eglynn | cdent: $ find . -name "*.rej" # <=== manually fix up these rejected hunks | 14:42 |
| eglynn | cdent: $ git add $(git status | grep modified | awk -F: '{print $2}') | 14:44 |
| eglynn | cdent: $ git am --resolved | 14:44 |
| cdent | hmmm | 14:44 |
| eglynn | cdent: prolly a zillion other ways of doing it in git | 14:44 |
| nsaje | that is some serious work, shouldn't that be on a wiki somewhere? | 14:44 |
| cdent | that's what I was thinking | 14:44 |
| cdent | I'm pretty sure it ought to be possible to use the rebase command itself to do much of that work (after fetching from the remote) | 14:45 |
| eglynn | cdent: ... as I said, many ways of skinning a cat in git | 14:46 |
| cdent | indeed | 14:46 |
| cdent | so easy to love and hate git | 14:46 |
| eglynn | cdent: ... the above is not meant to be prescriptive | 14:47 |
| * cdent nods | 14:47 | |
| eglynn | cdent: ... just a quick'n'dirty off-the-top-of-my-head way of getting you to where you want to be with that patch | 14:47 |
| *** fabiog has joined #openstack-ceilometer | 14:51 | |
| *** r0j4z0 has quit IRC | 14:52 | |
| *** fabiog has quit IRC | 14:53 | |
| *** idegtiarov has quit IRC | 14:53 | |
| openstackgerrit | Pradeep Kilambi proposed a change to openstack/ceilometer-specs: Spec for metering LBaaS https://review.openstack.org/94958 | 15:01 |
| *** jmckind has joined #openstack-ceilometer | 15:11 | |
| openstackgerrit | gordon chung proposed a change to openstack/ceilometer: improve expirer performance for sql backend https://review.openstack.org/96489 | 15:11 |
| *** gordc-mobile has joined #openstack-ceilometer | 15:22 | |
| *** jergerber has joined #openstack-ceilometer | 15:23 | |
| *** nati_ueno has quit IRC | 15:27 | |
| *** rbowen has quit IRC | 15:31 | |
| *** rbowen has joined #openstack-ceilometer | 15:32 | |
| *** cmart has quit IRC | 15:34 | |
| *** promulo has quit IRC | 15:43 | |
| *** shakayumi is now known as shakamunyi | 15:59 | |
| *** Ruetobas has quit IRC | 16:01 | |
| *** llu-laptop has joined #openstack-ceilometer | 16:02 | |
| *** fabiog has joined #openstack-ceilometer | 16:02 | |
| aviau | Hey | 16:02 |
| eglynn | aviau: hey | 16:02 |
| eglynn | aviau: so you mentioned https://review.openstack.org/#/c/87526/5/reference/programs.yaml | 16:02 |
| *** Ruetobas has joined #openstack-ceilometer | 16:03 | |
| eglynn | aviau: ... you think that specifically *excludes* monitoring from the telemetry program? | 16:03 |
| eglynn | aviau: ... that wasn't really the intention | 16:03 |
| *** gordc-mobile has quit IRC | 16:03 | |
| aviau | No, but I would like to see it included. | 16:03 |
| ildikov | aviau: it is still an open question that should it be included or it should be a separate service/project | 16:04 |
| ildikov | aviau: it was also mentioned in ATL that maybe it could be started under the Ceilometer umbrella and then we will see | 16:05 |
| * eglynn doesn't want to get rat-holed in a long discussion about the mission statement ... | 16:05 | |
| llu-laptop | ildikov: but it looks like hp-mon and ceilometer have a lot in common | 16:05 |
| eglynn | ... but it was intended not to close off future possibilities | 16:05 |
| ildikov | eglynn: sorry, agreed | 16:05 |
| eglynn | llu-laptop: yeap hp-mon looks like it's got some potential all right | 16:06 |
| ildikov | llu-laptop: I would separate billing related stuff and monitoring from each other | 16:06 |
| * eglynn is especially interested in hearing about how hp-mon will be proven in production in the HP cloud over the next 6 months or so | 16:07 | |
| fabiog | ildikov: I agree, even if the data can be somehow shared. So have a common storage strategy can be beneficial for the two projects | 16:07 |
| nealph | I think the hp-mon folks are of the opinion that the two should co-exist (what ildikov ^^^ noted). | 16:07 |
| llu-laptop | ildikov: agreed, billing and monitoring should be separated, but architecturally, many common things | 16:08 |
| *** Ruetobas has quit IRC | 16:08 | |
| eglynn | aviau: ... so it seems like someone from outside HP getting involved with hp-mon would be useful | 16:08 |
| eglynn | aviau: ... in the sense of community building etc. | 16:08 |
| nealph | eglynn:they're open to that, absolutely. I think that's the idea of moving to stackforge. | 16:09 |
| eglynn | nealph: yeap, exactly ... co-exist and maybe over time begin to share some elements | 16:09 |
| ildikov | for me it is a bit difficult to imagine how the separation will work, if we do both in the exact same project, but we will se what we can share and how it could work | 16:09 |
| ildikov | s/se/see | 16:09 |
| eglynn | ildikov: for example they use kafka as a highly performant/scalable message bus, IIUC | 16:10 |
| nealph | ildikov:well, we'll probably have to make it work internally...we're the guinea pigs? :) | 16:10 |
| aviau | One thing is certain. I think there has to be a separated service. Do you agree on this? | 16:10 |
| eglynn | ildikov: ... it would rock if kafka could be layered under the oslo.messaging layer | 16:10 |
| fabiog | aviau +1 | 16:11 |
| ildikov | eglynn: that's ok, but it is far from doing monitoring inside Ceilometer | 16:11 |
| eglynn | aviau: "separated service" ... what exactly do you mean by that? | 16:11 |
| ildikov | nealph: hmm, I like guineapigs, but I do not like to be one usually ;) | 16:12 |
| *** nati_ueno has joined #openstack-ceilometer | 16:12 | |
| *** fnaval has quit IRC | 16:12 | |
| aviau | I think MONaaS needs a separate API and endpoint. | 16:12 |
| eglynn | ildikov: "if we do both in the exact same project" ==> there's the ceilometer project and then there's the telemetry program | 16:12 |
| eglynn | aviau: so multiple APIs can be exposed by a single service, and multiple services can colloborate to provide a single API | 16:13 |
| ildikov | eglynn: a-ha, ok, then let's say that under the Telemetry umbrella and we use only those things in common, like kafka, that really makes sense | 16:13 |
| *** Ruetobas has joined #openstack-ceilometer | 16:14 | |
| fabiog | eglynn: are you advocating to have a telemetry program, a telemetry-common project and a ceilometer and monaas projects? | 16:14 |
| eglynn | ildikov: ... well it could be kafka, it could be we also leverage an influxDB integration that they do as part of the startegy to have a "pure" opensource version | 16:14 |
| eglynn | ildikov: (... i.e a version minus the vertica DB) | 16:15 |
| eglynn | fabiog: well, we already have a telemetry program | 16:15 |
| ildikov | eglynn: sure, that looks doable, but I wouldn't say it is necessary from the beginning | 16:15 |
| *** Titilambert has joined #openstack-ceilometer | 16:16 | |
| eglynn | fabiog: ... I'm just saying there are possibilities there for two projects to collaborate | 16:16 |
| Titilambert | hello | 16:16 |
| eglynn | Titilambert: hello | 16:16 |
| Titilambert | are you in meeting ? | 16:16 |
| ildikov | eglynn: I haven't checked the DB part of the HP solution yet, so it's an open question for me | 16:16 |
| eglynn | Titilambert: are we in #openstack-meeting do you mean? | 16:17 |
| Titilambert | is the ceilometer weekly meeting ? | 16:17 |
| eglynn | Titilambert: nope that meeting ending ~16 mins ago | 16:17 |
| aviau | We continue the meeting here. | 16:17 |
| Titilambert | oki | 16:17 |
| eglynn | Titilambert: that was on 1500-1600UTC | 16:17 |
| _nadya_ | ildikov: db part is interesting. I see only vertica on picture | 16:18 |
| Titilambert | eglynn: Do you think integrate HP solution as monitoring as a service ? | 16:18 |
| *** vrovachev has quit IRC | 16:19 | |
| eglynn | Titilambert: ... we're just on the learning phase, figuring out what it is, and riffing on possibilities | 16:19 |
| ildikov | eglynn: so monitoring and billing are different things, but if one DB solution can efficiently handle both, then I'm not against, we will see | 16:19 |
| aviau | ildikov, +1 | 16:19 |
| cdent | the time dimension seems much different to me | 16:19 |
| gordc | ildikov: i don't think we're looking at one db solution. we're already planning to split meter/alarm/event data into their own db. | 16:19 |
| *** IvanBerezovskiy has left #openstack-ceilometer | 16:20 | |
| eglynn | Titilambert: ... as we agreed at summit, our focus for the Juno cycle is all about solidifying the ceilometer core | 16:20 |
| eglynn | Titilambert: ... so this is more future-oriented discussion | 16:20 |
| Titilambert | eglynn: ok | 16:20 |
| ildikov | gordc: yeap sure, then let's call it shared DB solution for any of these areas, if poss | 16:20 |
| eglynn | Titilambert: ... with an eye to the K summit and beyond | 16:20 |
| Titilambert | eglynn: I see | 16:21 |
| eglynn | _nadya_: AFAIK it requires vertica right now | 16:21 |
| ildikov | gordc: I just reflected to what eglynn wrote, I'm a great fan of separation of responsibilities, like what you mentioned about the "own dbs" | 16:21 |
| eglynn | _nadya_: ... and there's a community edition of vertica with a limitation on total data size | 16:21 |
| eglynn | _nadya_: ... but it may well make sense for them to also have a "pure opensource" metric DB as an option | 16:22 |
| aviau | I dont think you guys are taking the right approach to implementing MONaaS. I think we should start from scratch and evaluate the integration of HP components as we go. | 16:22 |
| gordc | ildikov: yeah, i think separating data to different targets makes more sense... (you can always target the data to point to the same db if you want). | 16:23 |
| ildikov | _nadya_: maybe that will be improved also, I did not have time to deeply check the doc and collect the info all together that I saw or heard nowadays :( | 16:23 |
| gordc | aviau: who are 'you guys'? the HP design? Stacktach? | 16:23 |
| eglynn | aviau: before starting from scratch, IMO it makes sense to evaluate the prior art | 16:23 |
| ildikov | gordc: yeap, exactly, I think we're on the same page | 16:24 |
| gordc | ildikov: :) cool cool. | 16:24 |
| openstackgerrit | Chris Dent proposed a change to openstack/python-ceilometerclient: Refactor split_by_op for greater encapsulation https://review.openstack.org/96514 | 16:25 |
| ildikov | gordc: I'm just trying to not to be a really bad cop here only a simple bad one ;) | 16:25 |
| Titilambert | hum... | 16:25 |
| *** nati_ueno has quit IRC | 16:26 | |
| *** isviridov is now known as isviridov|away | 16:26 | |
| Titilambert | I guess nobody wants to reinvent Nagios, right ? | 16:26 |
| gordc | ildikov: i'll take the really bad cop role. | 16:26 |
| eglynn | aviau: ... as stated a few times already, our focus for the Juno cycle is all about solidifying the ceilometer core | 16:27 |
| ildikov | gordc: cool, tnx :) | 16:28 |
| aviau | eglynn, Right now the blueprint targets 'next' and not Juno. | 16:28 |
| openstackgerrit | Chris Dent proposed a change to openstack/python-ceilometerclient: Refactor split_by_op for greater encapsulation https://review.openstack.org/96514 | 16:28 |
| openstackgerrit | Chris Dent proposed a change to openstack/python-ceilometerclient: Enable special characters in query keys and values https://review.openstack.org/92074 | 16:28 |
| ildikov | aviau: do you mean that we should start to implement MONaaS from scratch independently from the HP one and then we will see? | 16:29 |
| ildikov | aviau: ... or I misunderstood what you wrote above about the not right approach? | 16:30 |
| *** llu-laptop has quit IRC | 16:31 | |
| eglynn | aviau: ... so my recommendation would be to get familiar with hp-mon and figure out where the gaps are (as opposed to assuming a start from a complete green-field) | 16:32 |
| aviau | ildikov, I would start "from kind of scratch" and use hp's solution during development. | 16:34 |
| _nadya_ | aviau: what's your plan? are you interested in integration in ceilometer or you consider variants? whats the priority? | 16:34 |
| Titilambert | eglynn: you're right, is just the hp solution is not a small project | 16:35 |
| aviau | _nadya_, To integrate with Ceilometer is not my priority. It would be to cover the use cases defined in the proposition. | 16:36 |
| aviau | Wether it is integrated or not. | 16:36 |
| aviau | In the future, integration will become important. | 16:36 |
| eglynn | Titilambert: sure, it's not small ... question is whether starting independently from scratch makes sense | 16:37 |
| ildikov | aviau: you can start it as a completely separate thing, if you would like to or join to the HP one if that is poss | 16:37 |
| *** promulo has joined #openstack-ceilometer | 16:38 | |
| Titilambert | eglynn: you're right! I think if we start from scratch, we have to know and USE HP solution's ideas and maybe some part of the code | 16:39 |
| ildikov | aviau: ... and then we can see how this under the Telemetry umbrella thing can work out, if you have something that's working | 16:39 |
| eglynn | aviau: ... if you want to benefit from the "multiplier effect" of open source development, it makes sense I think to consider the existing prior art | 16:40 |
| nealph | aviau:I'll echo again...I think the hp-mon folks would welcome contributions from all corners. | 16:40 |
| ildikov | nealph: +1 | 16:41 |
| eglynn | nealph: ... that's also the impression I got from conversations at summit | 16:41 |
| aviau | eglynn, This is correct. I will work on collaborating with what has already been done. | 16:44 |
| Titilambert | eglynn: oki, so first step: enter into HPcloud-mon, second step: check use cases with HPcloud-mon, then new meeting ? | 16:44 |
| *** _nadya_ has quit IRC | 16:44 | |
| eglynn | aviau, Titilambert: cool, sounds like a plan | 16:44 |
| eglynn | aviau, Titilambert: discuss again when, maybe in a week or two? | 16:45 |
| ildikov | Titilambert: new meeting about use cases, or? | 16:45 |
| aviau | I would like to have some time to dive in HP's solutions. Maybe two weeks? | 16:46 |
| Titilambert | ildikov: about our research about HPcloudmon | 16:46 |
| Titilambert | ildikov: about our researches about HPcloudmon | 16:46 |
| ildikov | Titilambert: hmm, ok, it can't hurt :) | 16:47 |
| Titilambert | eglynn: about the meeting, I don't know, I don't have time during the next weeks | 16:47 |
| Titilambert | ildikov: why ? | 16:48 |
| Titilambert | ildikov: oups, i read "it can hurt" :D | 16:48 |
| eglynn | aviau, Titilambert: cool, let's aim to talk again in circa two weeks (... modulo Titilambert's time constraints) | 16:48 |
| ildikov | Titilambert: :) | 16:48 |
| eglynn | right-o gentlepersons, getting late for me ... | 16:49 |
| eglynn | ... I'm gonna have to drop off IRC very soon | 16:49 |
| eglynn | ... have a good evening all, and thanks for the discussion | 16:49 |
| Titilambert | eglynn: maybe we have to get HP people here | 16:49 |
| aviau | Have a good day :) | 16:50 |
| eglynn | Titilambert: ... yeap I'll reach out to them about that | 16:50 |
| aviau | As far as I know, roland-hochmuth was supposed to be present. | 16:50 |
| Titilambert | eglynn: cool | 16:50 |
| ildikov | all right guys, then see you soon around :) | 16:51 |
| fabiog | guys, can anyone review https://review.openstack.org/#/c/95270/ please? | 16:51 |
| fabiog | is to update the client to KS v3 API | 16:52 |
| *** zqfan_ has joined #openstack-ceilometer | 16:53 | |
| *** dhandy has joined #openstack-ceilometer | 16:54 | |
| ildikov | nealph: still around? | 16:55 |
| nealph | yep... | 16:57 |
| ildikov | I will try to dig into the new Gnocchi design soon and participate from the pipeline side in the integration | 16:58 |
| ildikov | will you have some time later on to think about the dynamic config a bit? | 16:58 |
| *** dhandy has quit IRC | 16:59 | |
| nealph | absolutely...have just been pushing that agenda "in-house" and will have some more time to devote next week. | 16:59 |
| ildikov | cool, sounds good | 17:00 |
| *** shardy is now known as shardy_afk | 17:01 | |
| nealph | aviau:Titilambert:ping me or fabio...we work closely with those hp-mon guys and I know they're glad to kvetch on any topic. | 17:01 |
| nealph | any topic you might have questions on, that is. | 17:02 |
| nealph | ooo, "kvetch" not what I meant to imply. "chat" is the better word. | 17:02 |
| ildikov | nealph: ping then, when you will have some time, I will also ping if I have some info that we can start with | 17:02 |
| *** eglynn has quit IRC | 17:03 | |
| openstackgerrit | Chris Dent proposed a change to openstack/python-ceilometerclient: Refactor split_by_op for greater encapsulation https://review.openstack.org/96514 | 17:04 |
| *** DrBacchus has joined #openstack-ceilometer | 17:05 | |
| *** _cjones_ has joined #openstack-ceilometer | 17:06 | |
| nealph | ildikov:perfect...will do. | 17:08 |
| ildikov | nealph: cool, tnx | 17:08 |
| *** rbowen has quit IRC | 17:10 | |
| ildikov | nealph: we should start with some specs anyway, so I will try to collect as many info as I can about Gnocchi, hopefully we can figure the dynamic config design in parallel with that implementation and also we can have some brainstorming about the basics as a starting point | 17:10 |
| *** fnaval has joined #openstack-ceilometer | 17:10 | |
| ildikov | nealph: ok, I will not annoy you more for today, I think it's the best time to say that my office day is ended :) | 17:10 |
| nealph | ildikov: "you don't gotta go home, but you can't stay here." :) | 17:11 |
| ildikov | nealph: LOL :) | 17:13 |
| ildikov | nealph: BTW, what's your TZ? | 17:13 |
| *** underyx is now known as Underyx|off | 17:14 | |
| *** _nadya_ has joined #openstack-ceilometer | 17:14 | |
| *** DrBacchus is now known as rbowen | 17:17 | |
| nealph | ildikov: I'm in US Mountain. almost lunch. :D | 17:21 |
| ildikov | nealph: ah, ok :D | 17:21 |
| ildikov | nealph: I will keep it in mind :) | 17:21 |
| ildikov | nealph: ... and also bon apetite :) | 17:22 |
| *** jergerber has quit IRC | 17:23 | |
| ildikov | nealph: it's 19:30 here, I should drop the notebook out of the window to be sure that I will not work more today ;) | 17:23 |
| *** _nadya_ has quit IRC | 17:25 | |
| * cdent retires | 17:30 | |
| *** cdent has quit IRC | 17:30 | |
| *** ildikov has quit IRC | 17:35 | |
| *** nati_ueno has joined #openstack-ceilometer | 17:37 | |
| *** ityaptin has quit IRC | 17:38 | |
| *** fnaval has quit IRC | 17:42 | |
| *** cmart has joined #openstack-ceilometer | 17:55 | |
| *** _nadya_ has joined #openstack-ceilometer | 17:55 | |
| *** openstackgerrit has quit IRC | 17:57 | |
| *** fabiog has quit IRC | 18:17 | |
| *** raymondr has quit IRC | 18:20 | |
| *** _nadya_ has quit IRC | 18:20 | |
| *** _nadya_ has joined #openstack-ceilometer | 18:20 | |
| *** raymondr_ has joined #openstack-ceilometer | 18:20 | |
| *** ildikov has joined #openstack-ceilometer | 18:35 | |
| *** dmsimard has joined #openstack-ceilometer | 18:53 | |
| dmsimard | Any clue if it's possible to have the Swift ceilometer middleware talk to the Ceilometer API instead of the Bus/Database backend ? | 18:55 |
| *** zqfan_ has quit IRC | 19:01 | |
| *** openstackgerrit has joined #openstack-ceilometer | 19:07 | |
| *** fnaval has joined #openstack-ceilometer | 19:08 | |
| *** openstackgerrit has quit IRC | 19:13 | |
| *** nati_ueno has quit IRC | 19:16 | |
| dmsimard | If this yet does not exist, would it make sense to implement something like this ? Otherwise the swift-proxy servers need access to the message bus. The swift-proxy are likely to be public-facing while the bus is likely isolated. | 19:17 |
| *** nati_ueno has joined #openstack-ceilometer | 19:28 | |
| DinaBelova | ildikov, eglynn, gordc - please take a look on the https://review.openstack.org/#/c/87249/ - it's j1, probably we may keep more attention to it) Quite old change and really nice feature) | 19:28 |
| gordc | dmsimard: have you tried using udp publisher instead? i assume it's possible since it leverages the pipeline. | 19:29 |
| gordc | DinaBelova: will take a look after meeting. | 19:29 |
| DinaBelova | gordc, thanks a lot! | 19:30 |
| dmsimard | gordc: That's what I am looking at as I type this, if the file or udp publishers can help me. | 19:32 |
| gordc | dmsimard: cool cool. the udp functionality might be broken but give it a try. | 19:33 |
| dmsimard | gordc: Trying to figure out how to use them right now. I know they're there but can't find how to use them in the docs :( | 19:34 |
| *** openstackgerrit has joined #openstack-ceilometer | 19:34 | |
| gordc | dmsimard: on a call... you'll need to edit pipeline.yaml and change sink from rpc to udp | 19:35 |
| *** nati_ueno has quit IRC | 19:36 | |
| *** nati_ueno has joined #openstack-ceilometer | 19:36 | |
| openstackgerrit | Christian Berendt proposed a change to openstack/ceilometer-specs: fixed typos found by RETF rules https://review.openstack.org/96565 | 19:42 |
| *** _nadya_ has quit IRC | 19:48 | |
| *** nati_ueno has quit IRC | 19:54 | |
| *** nati_ueno has joined #openstack-ceilometer | 19:57 | |
| openstackgerrit | Chris Dent proposed a change to openstack/python-ceilometerclient: Refactor split_by_op for greater encapsulation https://review.openstack.org/96514 | 19:58 |
| dmsimard | gordc: It doesn't seem like I can get the collector to listen on the UDP port. | 19:59 |
| *** fnaval has quit IRC | 20:00 | |
| *** nati_ueno has quit IRC | 20:01 | |
| *** nati_ueno has joined #openstack-ceilometer | 20:03 | |
| *** fnaval has joined #openstack-ceilometer | 20:05 | |
| openstackgerrit | Christian Berendt proposed a change to openstack/ceilometer-specs: Fixed typos found by RETF rules https://review.openstack.org/96565 | 20:09 |
| gordc | dmsimard: working against master/ | 20:11 |
| gordc | ? | 20:11 |
| dmsimard | I can do either icehouse or havana. | 20:12 |
| dmsimard | The UDP settings are in both available in the settings | 20:13 |
| *** nati_ueno has quit IRC | 20:15 | |
| openstackgerrit | Chris Dent proposed a change to openstack/python-ceilometerclient: Refactor split_by_op for greater encapsulation https://review.openstack.org/96514 | 20:17 |
| -openstackstatus- NOTICE: Gerrit is going offline to correct an issue with a recent project rename. ETA 20:45 UTC. | 20:19 | |
| *** ChanServ changes topic to "Gerrit is going offline to correct an issue with a recent project rename. ETA 20:45 UTC." | 20:19 | |
| gordc | dmsimard: hmm. i don't recall any issues with udp in icehouse. | 20:22 |
| gordc | i should think you should be able to just add udp://host:port as a publisher and it would work... if not, maybe open a bug. | 20:22 |
| *** jergerber has joined #openstack-ceilometer | 20:22 | |
| *** openstackgerrit has quit IRC | 20:25 | |
| dmsimard | gordc: perhaps the collector end of my config is wrong. I do see in the logs that it loads the udp config (ex: DEBUG ceilometer.openstack.common.service [-] collector.udp_port = 4952 ) | 20:28 |
| dmsimard | gordc: But there is nothing listening on UDP 4952 | 20:28 |
| *** eglynn has joined #openstack-ceilometer | 20:31 | |
| *** ChanServ changes topic to "OpenStack Telemetry (Ceilometer) | https://wiki.openstack.org/wiki/Ceilometer" | 20:38 | |
| -openstackstatus- NOTICE: Gerrit is back online | 20:38 | |
| prad_ | eglynn, gordc, if you can take a look at https://review.openstack.org/#/c/94958/ should have all the changes discussed and ready to go | 20:49 |
| *** shakamunyi has quit IRC | 20:51 | |
| *** jmckind has quit IRC | 20:52 | |
| *** shakamunyi has joined #openstack-ceilometer | 20:52 | |
| *** jdob has quit IRC | 20:54 | |
| *** thomasem has quit IRC | 20:58 | |
| cmart | Alexei_987, are you there? | 21:01 |
| Alexei_987 | cmart: Hi yes | 21:02 |
| cmart | Hello.. you are alexei kornienko, right? The one that comment the patch I just submitted, right? | 21:02 |
| Alexei_987 | cmart: Yes :) | 21:03 |
| Alexei_987 | cmart: change itself is not bad | 21:03 |
| eglynn | prad_: comments inline | 21:03 |
| eglynn | prad_: thanks! | 21:03 |
| Alexei_987 | cmart: just need to check with cores/PTL that it's acceptable | 21:03 |
| Alexei_987 | eglynn: could you please check this one when you'll have time: https://review.openstack.org/#/c/96582/ | 21:03 |
| cmart | Alexei_987: +1 to that. In my defense, I "joined" ceilometer after the summit :S | 21:04 |
| Alexei_987 | eglynn: do we plan to add new features to v2 api? | 21:04 |
| eglynn | Alexei_987: well v2 is on a "soft deprecation path" in my mind | 21:04 |
| *** promulo has quit IRC | 21:05 | |
| eglynn | Alexei_987: ... which means that while it hasn't been *officially* deprecated | 21:05 |
| eglynn | Alexei_987: ... we should concentrate on our innovation on v3 | 21:05 |
| Alexei_987 | eglynn: since we have a good progress in improving v2 performance maybe it will be reasonable to extend it? until we'll have a production ready v3? | 21:06 |
| Alexei_987 | so we could release 2.1 or something like that | 21:07 |
| cmart | Alexei_987: .. and my implementation it's hitting performance right in the chin :( | 21:07 |
| eglynn | Alexei_987: ... yep, *soft* deprecation in my view means we can extend the API on a case-by-case basis | 21:08 |
| eglynn | Alexei_987: looking at https://review.openstack.org/#/c/96582/ now | 21:08 |
| Alexei_987 | cmart: it's no so bad as you say | 21:08 |
| Alexei_987 | cmart: I can only say for sqlalchemy + mongodb but this 2 looks ok | 21:08 |
| Alexei_987 | cmart: except for minor pep8 issues | 21:09 |
| Alexei_987 | cmart: in memory filtering is not really cool but since you already have a filtered query it should not iterate too many results | 21:10 |
| eglynn | cmart: so no postgresql special-casing \o/ :( | 21:10 |
| eglynn | cmart: ... I jest :) | 21:10 |
| cmart | Alexei_987: Glad to hear that :) (or not) | 21:10 |
| cmart | haha | 21:10 |
| Alexei_987 | cmart: the only thing I could add is that would should now allow to use meter filter *only* | 21:10 |
| eglynn | cmart: ... no no postgresql special-casing == goodness :) | 21:10 |
| Alexei_987 | cmart: it might be reasonable to fail query and require some additional filters | 21:11 |
| Alexei_987 | to avoid read-all case | 21:11 |
| Alexei_987 | for sqla | 21:12 |
| cmart | I think that the filtering options are waay too many.. and the schema ,IMHRookieO, doesn't help.. | 21:12 |
| eglynn | cmart: ... looks sane, -1 purely for lack of test coverage | 21:13 |
| cmart | eglynn: I couldn't find unit tests on that.. Only scenario tests... | 21:14 |
| Alexei_987 | these are the ones you need | 21:14 |
| *** shakamunyi has quit IRC | 21:14 | |
| Alexei_987 | 1 test covers all backends | 21:14 |
| eglynn | cmart: scenario tests are good enough for sqlalchemy and mongo (runs against sqlite and real mongodb) | 21:15 |
| Alexei_987 | cmart: left you a small note on sqla | 21:15 |
| cmart | eglynn: test_post_invalid_alarm_query at test_alarm_scenarios.py file seems to be a good case to replicate and change | 21:16 |
| cmart | Alexei_987: which tess? | 21:16 |
| cmart | *test? | 21:16 |
| cmart | OK! | 21:16 |
| cmart | Alexei_987:eglynn: Any tips to make the tests run "faster"? Config options, extra params, any?? | 21:18 |
| Alexei_987 | cmart: move /tmp on tmpfs | 21:18 |
| Alexei_987 | cmart: helps a lot :) | 21:18 |
| eglynn | cmart: you can also run tests selectively | 21:20 |
| cmart | Alexei_987: (linux rookie): Do you mean move a /tmp folder that is being created inside ceilometer folder to a new created folder called tmpfs? | 21:20 |
| Alexei_987 | no... | 21:20 |
| cmart | eglynn: I usually run them by activating a virtual env and then using testtools | 21:20 |
| eglynn | cmart: ... $ tox -epy27 -- package.subpackage.module # <=== for example | 21:20 |
| Alexei_987 | tmpfs /tmp tmpfs size=4G 0 0 | 21:21 |
| eglynn | (^^^ runs only that individual module's worth of tests) | 21:21 |
| Alexei_987 | you can put such record in your /etc/fstab | 21:21 |
| Alexei_987 | and your system wide /tmp folder will stay in RAM | 21:21 |
| Alexei_987 | ceilometer tests make heavy use of this folder so it helps a lot | 21:22 |
| cmart | Alexei_987: got it! | 21:22 |
| Alexei_987 | 4G is the size for /tmp | 21:22 |
| Alexei_987 | you can put your own value here | 21:23 |
| cmart | great! | 21:23 |
| *** julim has quit IRC | 21:24 | |
| cmart | Alexie_987: eglynn: thanks for the help!!! | 21:25 |
| * Alexei_987 thinking about changing nick to Alex_987 :) | 21:26 | |
| eglynn | Alexei_987: nah, wouldn't suit you ;) | 21:28 |
| eglynn | Alexei_987: ... the 'ei' adds cultural background :) | 21:28 |
| dmsimard | gordc: I can get the file publisher to work fine. Got to see how to have it read by the collector, now. | 21:28 |
| Alexei_987 | eglynn: actually my friends often call me alex | 21:28 |
| *** cmart has quit IRC | 21:29 | |
| eglynn | Alexei_987: ... a-ha, I thought Alex was more a British-ism | 21:29 |
| *** thomasem has joined #openstack-ceilometer | 21:29 | |
| *** thomasem has quit IRC | 21:30 | |
| *** nati_ueno has joined #openstack-ceilometer | 21:31 | |
| *** thomasem has joined #openstack-ceilometer | 21:31 | |
| Alexei_987 | eglynn: just in case you have good review mood now could you please also check this one: https://review.openstack.org/#/c/95613/ | 21:34 |
| *** openstackgerrit has joined #openstack-ceilometer | 21:34 | |
| gordc | dmsimard: the file publisher isn't meant to connect to collector. you'd need to work your own magic there. | 21:34 |
| eglynn | Alexei_987: ... sure, looking now | 21:34 |
| dmsimard | gordc: Yeah, okay, I can deal with that - but how would you insert the data into ceilometer ? | 21:34 |
| gordc | dmsimard: when you say 'ceilometer' you mean the database? | 21:35 |
| dmsimard | gordc: Yeah, I'm using a mongodb backend. | 21:36 |
| gordc | dmsimard: using the file publisher? you'd need to write some script to parse and insert it.... i hadn't really thought about it. | 21:38 |
| dmsimard | gordc: Hmm, could these be posted as is to the metering API endpoint ? Like with http://docs.openstack.org/developer/ceilometer/webapi/v2.html#post--v2-meters-(meter_name) | 21:38 |
| gordc | dmsimard: seems like it could work. | 21:39 |
| gordc | not sure how well that'd scale. | 21:39 |
| *** promulo has joined #openstack-ceilometer | 21:41 | |
| *** Underyx|off is now known as underyx | 21:44 | |
| *** openstackgerrit has quit IRC | 21:47 | |
| *** raymondr_ has quit IRC | 21:56 | |
| *** jergerber has quit IRC | 22:02 | |
| *** shakamunyi has joined #openstack-ceilometer | 22:11 | |
| *** r0j4z0 has joined #openstack-ceilometer | 22:13 | |
| *** gordc has left #openstack-ceilometer | 22:21 | |
| *** shakamunyi has quit IRC | 22:22 | |
| *** nati_ueno has quit IRC | 22:22 | |
| *** shakayumi has joined #openstack-ceilometer | 22:22 | |
| *** yjiang5 is now known as yjiang5_away | 22:23 | |
| *** isviridov|away is now known as isviridov | 22:24 | |
| *** fnaval has quit IRC | 22:37 | |
| *** prad_ has quit IRC | 22:54 | |
| *** nati_ueno has joined #openstack-ceilometer | 22:56 | |
| *** raymondr_ has joined #openstack-ceilometer | 23:06 | |
| *** fnaval has joined #openstack-ceilometer | 23:13 | |
| *** ddieterly has quit IRC | 23:27 | |
| *** fnaval has quit IRC | 23:42 | |
| *** ddieterly has joined #openstack-ceilometer | 23:51 | |
| *** zhikunliu has joined #openstack-ceilometer | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!