| *** annegentle has quit IRC | 00:10 | |
| *** flwang has joined #openstack-metering | 00:11 | |
| *** annegentle has joined #openstack-metering | 00:12 | |
| *** raghuvrao has quit IRC | 00:23 | |
| *** ekarlso has quit IRC | 00:23 | |
| *** swann has quit IRC | 00:23 | |
| *** swann has joined #openstack-metering | 00:24 | |
| *** ekarlso has joined #openstack-metering | 00:29 | |
| *** thomasm has joined #openstack-metering | 00:40 | |
| *** annegentle has quit IRC | 00:43 | |
| *** thomasm has quit IRC | 00:43 | |
| *** annegentle has joined #openstack-metering | 00:44 | |
| *** shang has joined #openstack-metering | 01:18 | |
| *** herndon has joined #openstack-metering | 01:43 | |
| *** raghuvrao has joined #openstack-metering | 02:17 | |
| *** sungju has joined #openstack-metering | 02:18 | |
| *** sungju has quit IRC | 02:18 | |
| openstackgerrit | Fei Long Wang proposed a change to openstack/ceilometer: Incorrect error code impacted by debug level https://review.openstack.org/47775 | 02:34 |
|---|---|---|
| *** taplax has quit IRC | 02:41 | |
| *** annegentle has quit IRC | 03:05 | |
| *** SergeyLukjanov has joined #openstack-metering | 03:06 | |
| *** annegentle has joined #openstack-metering | 03:08 | |
| *** evanjfraser has joined #openstack-metering | 03:13 | |
| *** herndon has quit IRC | 03:23 | |
| *** raghuvrao has quit IRC | 03:43 | |
| *** raghuvrao has joined #openstack-metering | 03:44 | |
| *** raghuvrao has quit IRC | 03:44 | |
| *** raghuvrao has joined #openstack-metering | 03:44 | |
| *** SergeyLukjanov has quit IRC | 04:17 | |
| *** annegentle has quit IRC | 04:49 | |
| *** annegentle has joined #openstack-metering | 04:53 | |
| *** evanjfraser has quit IRC | 05:22 | |
| *** taplax has joined #openstack-metering | 06:02 | |
| openstackgerrit | Jenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/47083 | 06:02 |
| *** bogdando has joined #openstack-metering | 07:19 | |
| *** SergeyLukjanov has joined #openstack-metering | 07:32 | |
| *** shardy_afk is now known as shardy | 08:10 | |
| *** Alexei_987 has joined #openstack-metering | 08:11 | |
| *** eglynn has joined #openstack-metering | 08:13 | |
| eglynn | jd__: good morning | 08:24 |
| jd__ | hi eglynn | 08:24 |
| eglynn | jd__: is there anything outstanding needed for https://review.openstack.org/44751 to land? | 08:24 |
| eglynn | (got a couple of +2s ... you'd also +2'd the previous patch version, latest only differs by a few trivial changes) | 08:24 |
| jd__ | eglynn: should be good enough then, I'll take a look ASAP | 08:25 |
| eglynn | jd__: thank you sir! | 08:25 |
| *** boris-42 has joined #openstack-metering | 08:41 | |
| *** ruhe has joined #openstack-metering | 09:29 | |
| *** Ruetobas has quit IRC | 09:50 | |
| *** Ruetobas has joined #openstack-metering | 09:53 | |
| *** zaneb has joined #openstack-metering | 09:53 | |
| *** Ruetobas has quit IRC | 09:57 | |
| *** Ruetobas has joined #openstack-metering | 09:58 | |
| *** zigo_ has joined #openstack-metering | 10:04 | |
| *** ruhe has quit IRC | 10:05 | |
| *** zigo has quit IRC | 10:05 | |
| jd__ | sileht: what do we need for https://review.openstack.org/#/c/46980/ to pass Jenkins? | 10:05 |
| *** ruhe has joined #openstack-metering | 10:11 | |
| *** ruhe has quit IRC | 10:12 | |
| sileht | jd__, https://github.com/dreamhost/sphinxcontrib-pecanwsme/pull/5 | 10:27 |
| openstackgerrit | A change was merged to openstack/ceilometer: Simple alarm partitioning protocol based on AMQP fanout RPC https://review.openstack.org/44751 | 10:29 |
| *** SergeyLukjanov has quit IRC | 10:36 | |
| *** maksimov has joined #openstack-metering | 10:47 | |
| *** ruhe has joined #openstack-metering | 10:50 | |
| *** boris-42 has quit IRC | 10:51 | |
| *** shang has quit IRC | 10:54 | |
| sileht | jd__, do we need to wait the next ceilometer meeting to release the new ceilometerclient ? because two heat patches are waiting the next version to be merged | 10:55 |
| *** SergeyLukjanov has joined #openstack-metering | 11:05 | |
| jd__ | sileht: no we can release whenever we want | 11:19 |
| jd__ | sileht: I imagine the patch needed for Heat are already in? can you take a look at https://review.openstack.org/#/q/status:open+project:openstack/python-ceilometerclient,n,z so I can release with these patches if they are ready | 11:20 |
| *** Shaan7 has quit IRC | 11:24 | |
| *** Shaan7 has joined #openstack-metering | 11:25 | |
| *** ruhe has quit IRC | 11:27 | |
| sileht | jd__, arf no, this one is missing https://review.openstack.org/#/c/46707/ | 11:32 |
| *** ruhe has joined #openstack-metering | 11:46 | |
| jd__ | eglynn: https://review.openstack.org/#/c/46707/ :) | 11:47 |
| eglynn | jd__: looking now | 11:48 |
| *** maksimov has quit IRC | 12:01 | |
| *** litong has joined #openstack-metering | 12:13 | |
| *** boris-42 has joined #openstack-metering | 12:13 | |
| openstackgerrit | litong01 proposed a change to openstack/ceilometer: add more test cases to improve the test code coverage #1 https://review.openstack.org/47608 | 12:15 |
| openstackgerrit | litong01 proposed a change to openstack/ceilometer: add more test cases to improve the test code coverage #1 https://review.openstack.org/47608 | 12:24 |
| *** thomasm has joined #openstack-metering | 12:24 | |
| sileht | jd__, I have answer to your remark on https://review.openstack.org/#/c/46048/ | 12:30 |
| jd__ | sileht: ah right I missed the other arguments on split_path | 12:32 |
| *** bpokorny has quit IRC | 12:32 | |
| sileht | jd__, I have prepared a other patch I will upload it (when github rework :( ) | 12:33 |
| jd__ | github? | 12:33 |
| jd__ | split_path(req.path, 3, 4, True) should be better I think | 12:33 |
| sileht | jd__, my patch do that | 12:33 |
| jd__ | sileht: use https://git.openstack.org/ ? | 12:33 |
| sileht | jd__, I have put 2, 4, because only account is mandatory | 12:35 |
| sileht | you can don't have the container when you request the stat on the account | 12:35 |
| jd__ | sileht: yeah fair enough :) | 12:36 |
| sileht | and this request work fine | 12:36 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Don't publish samples if resource_id in missing https://review.openstack.org/46048 | 12:36 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Print the exception of samples publishing https://review.openstack.org/46049 | 12:36 |
| thomasm | jd__, Thanks for the review! =] Silly MySQL. | 12:39 |
| jd__ | sileht: I think you can squash those patches | 12:39 |
| sileht | jd__, yes I have done my change in the wrong one, too :( | 12:39 |
| jd__ | thomasm: I still don't understand how people continue to use this… | 12:40 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Don't publish samples if resource_id in missing https://review.openstack.org/46048 | 12:49 |
| thomasm | jd__, MySQL? I have no idea. Often, I figure it's just convenient - what people are used to. | 12:49 |
| *** ruhe has quit IRC | 12:50 | |
| jd__ | thomasm: that's what I think too, but it's sad :( | 12:50 |
| thomasm | So? I guess I do have some idea. :) | 12:50 |
| jd__ | sileht: alarm created by Heat are created using the user token? | 12:52 |
| eglynn | sileht: one other thought on the python-ceiloclient patch for the new alarm format ... | 12:52 |
| eglynn | sileht: so name, meter_name, threshold etc. are now required args for alarm-update | 12:53 |
| jd__ | or eglynn for above question ^ | 12:53 |
| *** bpokorny has joined #openstack-metering | 12:53 | |
| sileht | eglynn, yes it's required | 12:53 |
| eglynn | jd__: hmmm, I don't actually know for sure ... I thought they were associated with the heat user | 12:54 |
| eglynn | (even if these field not being changed) | 12:54 |
| eglynn | sileht: ^^^ is that fall-out from the re-org of the @arg decorators in the ceiloclient.v2.shell? | 12:54 |
| eglynn | or a reflection of the new alarm API PUT semantics? | 12:54 |
| eglynn | (or a mixture of both?) | 12:54 |
| sileht | eglynn, both | 12:55 |
| eglynn | sileht: just thinking that in the past, an alarm-update was implemented as a GET, merge in new fields, then a PUT | 12:55 |
| eglynn | sileht: I removed that logic here ... https://github.com/openstack/python-ceilometerclient/commit/52e3f6ff | 12:55 |
| eglynn | sileht: but for convenience might be an idea to re-instate | 12:55 |
| eglynn | sileht: so that the update CLI usage can be limited to the fields actually being changed | 12:56 |
| jd__ | might be a good idea indeed | 12:56 |
| sileht | eglynn, I see the idea | 12:56 |
| eglynn | prolly have to somewhat ditch the @common_alarm_arguments idea | 12:57 |
| eglynn | (since the required fields will be different for create & update) | 12:57 |
| *** sandywalsh has quit IRC | 12:57 | |
| eglynn | pity, that's kinda neat ... | 12:57 |
| jd__ | sileht: do you have an answer to my question "who creats and owns Heat alarms created for autoscaling?" | 12:57 |
| sileht | jd__, not yet, I think with the heat user, I'm checking | 12:58 |
| jd__ | sileht: thanks | 12:58 |
| *** herndon has joined #openstack-metering | 12:59 | |
| *** fc__ has quit IRC | 13:01 | |
| thomasm | jd__, Agreed. I think the major thing MySQL has on, like, Postgres are the clustering features. Apparently they've got some pretty slick in-memory clusters and what-not. | 13:03 |
| thomasm | If one wanted to use it | 13:04 |
| *** sandywalsh has joined #openstack-metering | 13:10 | |
| *** julim has joined #openstack-metering | 13:12 | |
| *** julim has quit IRC | 13:13 | |
| openstackgerrit | A change was merged to openstack/python-ceilometerclient: Use the new alarm format https://review.openstack.org/46707 | 13:13 |
| *** julim has joined #openstack-metering | 13:17 | |
| *** gordc has joined #openstack-metering | 13:18 | |
| *** shang has joined #openstack-metering | 13:26 | |
| *** changbl has quit IRC | 13:29 | |
| *** SergeyLukjanov has quit IRC | 13:29 | |
| *** maksimov has joined #openstack-metering | 13:31 | |
| *** SergeyLukjanov has joined #openstack-metering | 13:34 | |
| *** fnaval__ has quit IRC | 13:35 | |
| *** fnaval_ has joined #openstack-metering | 13:55 | |
| *** fc__ has joined #openstack-metering | 13:56 | |
| *** herndon has quit IRC | 14:08 | |
| *** ruhe has joined #openstack-metering | 14:19 | |
| openstackgerrit | litong01 proposed a change to openstack/ceilometer: add more test cases to improve the test code coverage #2 https://review.openstack.org/47629 | 14:22 |
| openstackgerrit | Vladislav Kuzmin proposed a change to openstack/ceilometer: Replace tests.base part4 https://review.openstack.org/47145 | 14:27 |
| openstackgerrit | Vladislav Kuzmin proposed a change to openstack/ceilometer: Replace tests.base part3 https://review.openstack.org/47144 | 14:27 |
| openstackgerrit | Vladislav Kuzmin proposed a change to openstack/ceilometer: Replace tests.base part6 https://review.openstack.org/47147 | 14:27 |
| openstackgerrit | Vladislav Kuzmin proposed a change to openstack/ceilometer: Replace tests.base part5 https://review.openstack.org/47146 | 14:27 |
| openstackgerrit | Vladislav Kuzmin proposed a change to openstack/ceilometer: Update oslo in ceilometer part1 https://review.openstack.org/46476 | 14:27 |
| openstackgerrit | Vladislav Kuzmin proposed a change to openstack/ceilometer: Replace tests.base part1 https://review.openstack.org/46480 | 14:27 |
| openstackgerrit | Vladislav Kuzmin proposed a change to openstack/ceilometer: Replace tests.base part2 https://review.openstack.org/47143 | 14:27 |
| openstackgerrit | Vladislav Kuzmin proposed a change to openstack/ceilometer: Update oslo in ceilometer part2 https://review.openstack.org/47142 | 14:27 |
| openstackgerrit | litong01 proposed a change to openstack/ceilometer: add more test cases to improve the test code coverage #2 https://review.openstack.org/47629 | 14:36 |
| thomasm | Still needing an approval on my patch, if anyone has a little bit to look at it =] : https://review.openstack.org/#/c/46100/ | 14:42 |
| *** tasdomas has joined #openstack-metering | 14:57 | |
| *** tasdomas has quit IRC | 14:58 | |
| *** tasdomas has joined #openstack-metering | 14:59 | |
| *** changbl has joined #openstack-metering | 15:05 | |
| sileht | jd__, heat use the user credentials to create the alarm | 15:05 |
| jd__ | sileht: ok | 15:06 |
| sileht | jd__, but to use autoscaling you must be admin for the haproxy stuffs | 15:06 |
| *** shang has quit IRC | 15:07 | |
| sileht | jd__, I don't know if neutron-lbass object in heat has been merged, but with this lb everybody can do autoscaling I guess | 15:07 |
| *** maksimov has quit IRC | 15:14 | |
| openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: service: check that timestamps are almost equals https://review.openstack.org/47857 | 15:30 |
| thomasm | eglynn, ping | 15:37 |
| eglynn | thomasm: pong | 15:37 |
| thomasm | eglynn, Regarding our conversation about a backport for https://bugs.launchpad.net/ceilometer/+bug/1208547: So, I set up grizzly devstack and it seemed to be returning the latest stored resource metadata, the only one that wasn't is hbase, which would require a schema change (strictly prohibited). I'm thinking we wouldn't really be helping much, if at all, by applying this fix to grizzly. The only concern I have there is t | 15:40 |
| thomasm | he potential for messages to get out of order due to the nature of message queueing. | 15:40 |
| openstackgerrit | litong01 proposed a change to openstack/ceilometer: add more test cases to improve the test code coverage #2 https://review.openstack.org/47629 | 15:41 |
| eglynn | thomasm: can you explain the out-of-order concern? | 15:42 |
| *** zul has quit IRC | 15:43 | |
| thomasm | eglynn, AMQP doesn't guarantee the order messages are delivered in unless it's a direct queue, also if any re-queueing happens, it would definitely be delivered out of order. | 15:43 |
| thomasm | So, you could grab the latest stored, but that may be an older message than one that was stored before it. | 15:44 |
| thomasm | latest stored being based on an assumption that natural ordering lends itself to LIFO in the case of these drivers (which appears to be the case since I don't see any ordering) | 15:44 |
| thomasm | Also, if there's significant infrastructure in between the queue and the recipient, messages could get out of order due to the nature of networking retries and what-not. | 15:45 |
| thomasm | So, in order to avoid those problems, we must order by payload timestamp. | 15:46 |
| eglynn | thomasm: a-ha, OK I think I see what you mean | 15:46 |
| dragondm | multiple collectors could also save messages out of order. | 15:46 |
| eglynn | thomasm: so is that a concern related to backporting the original fix, or a concern about the completeness of the original fix? | 15:46 |
| thomasm | yep ^^ | 15:46 |
| thomasm | eglynn, well the original fix addresses the messages being out of order. I made sure to sort. | 15:47 |
| thomasm | eglynn, So, it's addressing backporting the original fix | 15:47 |
| eglynn | thomasm: ok, so could that sorting be split off into a self-contained patch for back-porting? | 15:47 |
| eglynn | thomasm: (that didn't require the problematic schema change) | 15:48 |
| eglynn | thomasm: (but still addressed the out-of-order issue) | 15:48 |
| thomasm | eglynn, I believe so. | 15:48 |
| *** zul has joined #openstack-metering | 15:48 | |
| eglynn | thomasm: that sounds like it would be useful | 15:48 |
| eglynn | thomasm: from a release mgmt point of view, it might be best to open a separate bug targeted at stable/grizzy | 15:49 |
| eglynn | thomasm: (as opposed to treating it as a limited backport of the original bug) | 15:49 |
| thomasm | eglynn, Okay, great. Let me do that. Out of curiosity, what are we suspecting the lifetime of these releases will be anyway? | 15:50 |
| eglynn | thomasm: well upstream ... stable is usually maintained for one release cycle by the stable-maint group, then it's passed over to the vulnerability mgmt team for "security fixes only" type of support | 15:52 |
| eglynn | thomasm: so in that sense stable/essex is EoL'd as is stable/diablo | 15:53 |
| eglynn | thomasm: however, downstream is likely to be different | 15:53 |
| eglynn | thomasm: i.e. distro packagers will maintain longer term support than one release cycle | 15:53 |
| thomasm | eglynn, I see | 15:54 |
| thomasm | eglynn, Okay, cool. Thanks. =] | 15:54 |
| *** terriyu has joined #openstack-metering | 15:59 | |
| *** litong has quit IRC | 16:00 | |
| *** Ruetobas has quit IRC | 16:01 | |
| thomasm | gordc, Thanks! =] | 16:01 |
| *** maksimov has joined #openstack-metering | 16:02 | |
| *** Ruetobas has joined #openstack-metering | 16:03 | |
| openstackgerrit | Yuuichi Fujioka proposed a change to openstack/python-ceilometerclient: Fix order of sample list https://review.openstack.org/47331 | 16:03 |
| gordc | thomasm: np | 16:03 |
| *** Ruetobas has quit IRC | 16:08 | |
| *** Ruetobas has joined #openstack-metering | 16:14 | |
| *** boris-42 has quit IRC | 16:19 | |
| *** maksimov has quit IRC | 16:24 | |
| *** julim has quit IRC | 16:29 | |
| *** julim has joined #openstack-metering | 16:42 | |
| *** SergeyLukjanov has quit IRC | 16:56 | |
| *** Alexei_987 has quit IRC | 16:56 | |
| openstackgerrit | A change was merged to openstack/ceilometer: Fix for timestamp precision in SQLAlchemy https://review.openstack.org/46100 | 16:59 |
| *** eglynn has quit IRC | 17:00 | |
| openstackgerrit | A change was merged to openstack/python-ceilometerclient: Replace OpenStack LLC with OpenStack Foundation https://review.openstack.org/47449 | 17:02 |
| openstackgerrit | A change was merged to openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/47083 | 17:04 |
| *** SergeyLukjanov has joined #openstack-metering | 17:31 | |
| *** maksimov has joined #openstack-metering | 17:31 | |
| *** julim has quit IRC | 17:35 | |
| *** maksimov has quit IRC | 17:35 | |
| *** eglynn has joined #openstack-metering | 17:36 | |
| *** ruhe has quit IRC | 17:38 | |
| *** eglynn has quit IRC | 17:49 | |
| *** zbitter has joined #openstack-metering | 17:50 | |
| *** boris-42 has joined #openstack-metering | 17:50 | |
| *** zaneb has quit IRC | 17:53 | |
| *** maksimov has joined #openstack-metering | 17:59 | |
| *** ruhe has joined #openstack-metering | 18:03 | |
| *** ruhe has quit IRC | 18:03 | |
| openstackgerrit | Terri Yu proposed a change to openstack/ceilometer: Remove vim configuration comment in audit.py https://review.openstack.org/47894 | 18:16 |
| *** eglynn has joined #openstack-metering | 18:16 | |
| openstackgerrit | Terri Yu proposed a change to openstack/ceilometer: Remove extraneous vim configuration comments https://review.openstack.org/47894 | 18:48 |
| *** ruhe has joined #openstack-metering | 18:51 | |
| *** ruhe has quit IRC | 18:51 | |
| openstackgerrit | A change was merged to openstack/python-ceilometerclient: Pass region_name argument to keystone client https://review.openstack.org/47544 | 19:21 |
| openstackgerrit | Doug Hellmann proposed a change to openstack/ceilometer: Fail tests early if mongod is not found https://review.openstack.org/45913 | 19:36 |
| *** maksimov has quit IRC | 19:44 | |
| *** evanjfraser has joined #openstack-metering | 19:49 | |
| *** ruhe has joined #openstack-metering | 19:52 | |
| *** esheffield has joined #openstack-metering | 19:53 | |
| *** julim has joined #openstack-metering | 20:08 | |
| *** ruhe has quit IRC | 20:19 | |
| *** SergeyLukjanov has quit IRC | 20:30 | |
| *** bpokorny has quit IRC | 20:31 | |
| thomasm | eglynn, https://bugs.launchpad.net/ceilometer/+bug/1229395 | 20:32 |
| thomasm | sqlalchemy fix is set up, it looks like mongo was passing because it was skipping when Ming wasn't present. | 20:33 |
| *** sandywalsh has quit IRC | 20:33 | |
| *** evanjfraser has quit IRC | 20:36 | |
| *** bpokorny has joined #openstack-metering | 20:38 | |
| *** bpokorny1 has joined #openstack-metering | 20:39 | |
| *** bpokorny has quit IRC | 20:42 | |
| *** sandywalsh has joined #openstack-metering | 20:45 | |
| *** shakayumi has joined #openstack-metering | 20:52 | |
| *** shakayumi has quit IRC | 20:53 | |
| *** evanjfraser has joined #openstack-metering | 20:56 | |
| *** thomasm has quit IRC | 21:18 | |
| *** bpokorny1 has quit IRC | 21:29 | |
| *** boris-42 has quit IRC | 21:31 | |
| *** bpokorny has joined #openstack-metering | 21:33 | |
| *** julim has quit IRC | 21:37 | |
| *** evanjfraser has quit IRC | 21:37 | |
| *** boris-42 has joined #openstack-metering | 21:38 | |
| *** boris-42 has quit IRC | 21:55 | |
| *** bpokorny has quit IRC | 22:01 | |
| *** evanjfraser has joined #openstack-metering | 22:06 | |
| *** changbl has quit IRC | 22:13 | |
| *** openstack has joined #openstack-metering | 22:32 | |
| *** evanjfraser has quit IRC | 22:38 | |
| *** shakayumi has quit IRC | 22:38 | |
| *** evanjfraser has joined #openstack-metering | 22:49 | |
| *** thomasm has joined #openstack-metering | 23:21 | |
| *** changbl has joined #openstack-metering | 23:22 | |
| *** maksimov has joined #openstack-metering | 23:25 | |
| *** zbitter has quit IRC | 23:38 | |
| *** zbitter has joined #openstack-metering | 23:40 | |
| *** fnaval_ has quit IRC | 23:43 | |
| *** thomasm has quit IRC | 23:48 | |
| *** matsuhashi has joined #openstack-metering | 23:50 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!