openstackgerrit | A change was merged to openstack/ceilometer: Adds flavor_id in the nova_notifier https://review.openstack.org/69582 | 00:30 |
---|---|---|
openstackgerrit | A change was merged to openstack/python-ceilometerclient: Python 3: fix format_nested_list_of_dict() https://review.openstack.org/72405 | 00:30 |
openstackgerrit | A change was merged to openstack/python-ceilometerclient: Sync with Oslo https://review.openstack.org/72392 | 00:34 |
*** xmltok has quit IRC | 00:55 | |
*** sileht has quit IRC | 01:29 | |
*** sileht has joined #openstack-ceilometer | 01:38 | |
openstackgerrit | liusheng proposed a change to openstack/ceilometer: check domain state before inspecting nics/disks https://review.openstack.org/68882 | 02:02 |
*** flwang has quit IRC | 02:17 | |
*** xianghui has joined #openstack-ceilometer | 02:18 | |
*** gordc has quit IRC | 02:33 | |
*** flwang has joined #openstack-ceilometer | 02:53 | |
*** flwang has quit IRC | 02:59 | |
*** flwang has joined #openstack-ceilometer | 03:00 | |
*** xianghui has quit IRC | 03:01 | |
*** xianghui has joined #openstack-ceilometer | 03:16 | |
*** Yathi has joined #openstack-ceilometer | 03:23 | |
*** Yathi has quit IRC | 03:27 | |
openstackgerrit | liusheng proposed a change to openstack/ceilometer: check domain state before inspecting nics/disks https://review.openstack.org/68882 | 03:54 |
openstackgerrit | Yuuichi Fujioka proposed a change to openstack/python-ceilometerclient: fix help message of deprecated command https://review.openstack.org/72198 | 04:34 |
openstackgerrit | Jenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/62808 | 06:02 |
*** rwsu has quit IRC | 06:07 | |
*** rwsu has joined #openstack-ceilometer | 06:07 | |
*** ildikov_ has quit IRC | 06:16 | |
*** rwsu has quit IRC | 06:25 | |
*** ildikov_ has joined #openstack-ceilometer | 06:31 | |
*** rwsu has joined #openstack-ceilometer | 06:38 | |
*** saju_m has joined #openstack-ceilometer | 06:38 | |
ityaptin | llu: ping | 06:40 |
*** saju_m has quit IRC | 06:43 | |
*** saju_m has joined #openstack-ceilometer | 06:50 | |
*** saju_m has quit IRC | 06:51 | |
*** _nadya_ has joined #openstack-ceilometer | 06:52 | |
*** _nadya_ has quit IRC | 07:01 | |
llu | ityaptin: pong | 07:03 |
*** eglynn has quit IRC | 07:15 | |
openstackgerrit | Dazhao Yu proposed a change to openstack/ceilometer: Make sure use IPv6 sockets for ceilometer in IPv6 environment https://review.openstack.org/67786 | 07:19 |
*** saju_m has joined #openstack-ceilometer | 07:37 | |
*** eglynn has joined #openstack-ceilometer | 07:45 | |
*** saju_m has quit IRC | 07:52 | |
openstackgerrit | liusheng proposed a change to openstack/python-ceilometerclient: Remove unused mock in v2/test_shell.py https://review.openstack.org/72583 | 08:07 |
ityaptin | llu: About https://review.openstack.org/#/c/71595/ and your comments. | 08:20 |
llu | ityaptin: I'm just wondering if this is a coding convention of not? | 08:24 |
ityaptin | llu: In testtools if length of assert mismatch message more than 70 letters it's described with "expected=" and "actual=" | 08:24 |
llu | ityaptin: ok, so this is defined in testtool module? | 08:26 |
ityaptin | llu: Yes. Also in testtools arguments of method assertEqual are placed in order "expected,actual". | 08:31 |
llu | ityaptin: got that, thanks for your info | 08:31 |
ityaptin | llu: thanks for your questions) | 08:33 |
*** eglynn has quit IRC | 08:36 | |
*** saju_m has joined #openstack-ceilometer | 08:51 | |
*** flwang has quit IRC | 08:52 | |
*** yassine has joined #openstack-ceilometer | 09:03 | |
openstackgerrit | Victor Sergeyev proposed a change to openstack/ceilometer: POC: Add test for check sync models and migrations https://review.openstack.org/69674 | 09:04 |
*** eglynn has joined #openstack-ceilometer | 09:11 | |
*** flwang has joined #openstack-ceilometer | 09:30 | |
*** boris-42_ has quit IRC | 09:36 | |
nprivalova | eglynn, sileht, jd__, I need your advice. Yesterday Julien has approved https://review.openstack.org/#/c/68435/ but it wasn't successfully merged. So I've uploaded a new change. Could you please the last comment from me? Are you agree with it? I don't want to hurt Thomas :) | 09:45 |
eglynn | nprivalova: ... looking now | 09:45 |
eglynn | nprivalova: ... ok in theory he's probably right that *if* the upstream project was chronically unresponsive | 09:47 |
eglynn | nprivalova: ... then we would have no options other than (a) to fork, or (b) to find another equivalent dependency | 09:48 |
eglynn | nprivalova: ... however, IMO it's completely unrealistic to hold all developement on the ceilometer hbase driver by requiring that we fork it NOW | 09:48 |
eglynn | nprivalova: ... hold *up all development | 09:49 |
eglynn | nprivalova: ... I mean, the necessary fix may be forthcoming upstream, just slow to land and be released | 09:49 |
eglynn | nprivalova: ... in that case, we should have the option to upper-bound the dependency as a pragmatic workaround | 09:50 |
nprivalova | eglynn: it's responsive, the first fix was created in 2 weeks after we reported a bug. The author responses ok :) I think that we may not to force this fix, wait for a month e.g. | 09:50 |
eglynn | nprivalova: cool, then seems to me most reasonable option is to wait for the fix and work-around in the meantime | 09:51 |
nprivalova | eglynn: yep, ok | 09:51 |
eglynn | nprivalova: ... though if it turns out that months pass and still no fix upstream, we'd have to reconsider | 09:51 |
eglynn | nprivalova: ... I'll put the above thoughts on gerrit also | 09:52 |
nprivalova | eglynn: ok, thank you | 09:52 |
sileht | nprivalova, have you asked to upstream to release a need version ? the workaround looks already in the master of happybase | 09:52 |
*** mihgen has joined #openstack-ceilometer | 09:54 | |
nprivalova | sileht: not yet, will do it ASAP. They have created the fix, yes, but after that they wrote "we have found more problems" | 09:54 |
sileht | nprivalova, ok I have no knowledge about hbase, but if 'the more problems found' was in happybase 0.6 too, this is safe to use the next released version with the workaround and change the requirements with >=0.6,!=0.7 or >0.7 | 09:58 |
nprivalova | sileht: yes, the same as I proposed but looks much safer :) | 09:59 |
* sileht hope upstream released a new version soon | 10:00 | |
nprivalova | sileht: what is the status of notifications in nova, do you know? | 10:00 |
nprivalova | sileht: tempest tests for notifications don't work :( ERROR oslo.messaging.notify._impl_messaging [-] Could not send notification to notifications | 10:01 |
sileht | nprivalova, Oh I haven't dig into this one, I known jd__ have proposed a fix: https://review.openstack.org/#/c/70759/ refused by markmc for now ... | 10:09 |
sileht | nprivalova, jd__ have uploaded a 4° patchset that fix the issue in an other ways, so wait and see | 10:10 |
nprivalova | sileht: just looking into https://review.openstack.org/#/c/64136/10 . it's strange that dsvm-full fails but postgress pass | 10:11 |
* sileht is looking | 10:14 | |
sileht | the oslo.messaging error messages in the postgres console.log scary me | 10:18 |
nprivalova | sileht: yes. will try to install latest devstack and check how it works | 10:22 |
*** saju_m has quit IRC | 10:25 | |
*** saju_m has joined #openstack-ceilometer | 10:26 | |
*** xianghui has quit IRC | 10:29 | |
*** yassine has quit IRC | 10:56 | |
*** Alexei_987 has quit IRC | 11:10 | |
*** boris-42_ has joined #openstack-ceilometer | 11:16 | |
eglynn | jd__: good morning | 11:17 |
eglynn | ... too late for that, good afternoon! :) | 11:17 |
*** mihgen has quit IRC | 11:18 | |
eglynn | jd__: I suspect your preference would be to have *less* on our plates for i3, rather than more | 11:18 |
jd__ | eglynn: likely | 11:18 |
eglynn | jd__: .. but that said, what's your feeling on targeting that source/sink BP to i3? | 11:18 |
eglynn | https://blueprints.launchpad.net/ceilometer/+spec/decoupled-source-sink-discoverable-resources | 11:18 |
eglynn | (perhaps bumping the periodicity BP to Juno to make room?) | 11:19 |
eglynn | i.e. https://blueprints.launchpad.net/ceilometer/+spec/record-sample-periodicity | 11:19 |
jd__ | eglynn: I think it's doable | 11:32 |
eglynn | jd__: cool | 11:32 |
jd__ | if someone/you take it in charge :) | 11:32 |
eglynn | jd__: FWIW my feeling is that something along these lines will be required to allow the SNMP-based hardware polling to be useable in icehouse | 11:32 |
eglynn | jd__: yep, I'd be committing to implementing for i3 | 11:33 |
jd__ | yep | 11:33 |
jd__ | great | 11:33 |
eglynn | jd__: ... so I'll just go ahead and retarget, or is that the PTL's prerogative? | 11:33 |
jd__ | eglynn: no go ahead | 11:37 |
*** mihgen has joined #openstack-ceilometer | 11:40 | |
openstackgerrit | A change was merged to openstack/ceilometer: fix column name and alignment https://review.openstack.org/69524 | 11:44 |
openstackgerrit | A change was merged to openstack/ceilometer: Test case for creating an alarm without auth headers https://review.openstack.org/72406 | 11:46 |
openstackgerrit | A change was merged to openstack/ceilometer: Remove tox locale overrides https://review.openstack.org/72202 | 11:50 |
openstackgerrit | A change was merged to openstack/ceilometer: samples: fix test case status code check https://review.openstack.org/69648 | 11:50 |
openstackgerrit | A change was merged to openstack/ceilometer: Improve help strings https://review.openstack.org/71987 | 11:50 |
*** SergeyLukjanov has quit IRC | 11:51 | |
*** SergeyLukjanov has joined #openstack-ceilometer | 11:55 | |
*** Alexei_987 has joined #openstack-ceilometer | 12:07 | |
*** promulo has quit IRC | 12:10 | |
*** yassine has joined #openstack-ceilometer | 12:11 | |
openstackgerrit | A change was merged to openstack/ceilometer: Implements complex query functionality for samples https://review.openstack.org/62157 | 12:19 |
*** eglynn is now known as eglynn-lunch | 12:29 | |
nprivalova | ildikov_: congrats! :)Very complicated change | 12:33 |
ildikov_ | nprivalova: thanks, I'm so happy, just about to cry :) | 12:38 |
nprivalova | ildikov_: oh, don't cry :) | 12:39 |
ildikov_ | nprivalova: we still have plenty of work to implement the whole bp, but it was the biggest step I think | 12:39 |
ildikov_ | nprivalova: just because I'm happy :) | 12:40 |
nprivalova | ildikov_: yes, sure! The most difficult part is to merge the first part if bp | 12:40 |
ildikov_ | nprivalova: I do not have experience with the second part, but the first is really not easy, I'm sure now :), but I'm so thankful for everyone, who reviewed it and helped | 12:44 |
openstackgerrit | litong01 proposed a change to openstack/ceilometer: add notification alarm https://review.openstack.org/69473 | 12:53 |
openstackgerrit | litong01 proposed a change to openstack/python-ceilometerclient: add notification alarm https://review.openstack.org/69474 | 12:53 |
*** vrovachev has left #openstack-ceilometer | 12:59 | |
*** jdob has joined #openstack-ceilometer | 13:26 | |
*** gordc has joined #openstack-ceilometer | 13:28 | |
*** thomasem has joined #openstack-ceilometer | 13:38 | |
openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: Implements in operator for complex query functionality https://review.openstack.org/66687 | 13:44 |
openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: Implements field validation for complex query functionality https://review.openstack.org/65302 | 13:44 |
openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: Implements complex query functionality for alarms https://review.openstack.org/62158 | 13:44 |
openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: Implements complex query functionality for alarm history https://review.openstack.org/62159 | 13:44 |
*** jmckind has joined #openstack-ceilometer | 14:01 | |
*** chmouel has left #openstack-ceilometer | 14:17 | |
Alexei_987 | jd__: Hi | 14:25 |
*** saju_m has quit IRC | 14:27 | |
*** saju_m has joined #openstack-ceilometer | 14:29 | |
*** promulo has joined #openstack-ceilometer | 14:34 | |
*** prad_ has joined #openstack-ceilometer | 14:53 | |
*** ilyashakhat has quit IRC | 14:54 | |
*** ddutta has joined #openstack-ceilometer | 15:01 | |
*** Yathi has joined #openstack-ceilometer | 15:02 | |
*** eglynn-lunch is now known as eglynn | 15:09 | |
*** mihgen has quit IRC | 15:11 | |
*** ruhe_ has joined #openstack-ceilometer | 15:17 | |
openstackgerrit | Balazs Gibizer proposed a change to openstack/ceilometer: Implements "not" operator for complex query https://review.openstack.org/66892 | 15:19 |
openstackgerrit | Balazs Gibizer proposed a change to openstack/ceilometer: Implements metadata query for complex query feature https://review.openstack.org/66891 | 15:19 |
openstackgerrit | Balazs Gibizer proposed a change to openstack/ceilometer: Implements "not" operator for complex query https://review.openstack.org/66892 | 15:29 |
*** boris-42_ has quit IRC | 15:36 | |
gordc | dhellmann: ping | 15:37 |
*** Yathi has quit IRC | 15:44 | |
*** saju_m has quit IRC | 15:45 | |
*** saju_m has joined #openstack-ceilometer | 15:46 | |
*** ddutta has quit IRC | 16:06 | |
*** mihgen has joined #openstack-ceilometer | 16:08 | |
*** mihgen has quit IRC | 16:19 | |
*** mihgen has joined #openstack-ceilometer | 16:21 | |
*** _nadya_ has joined #openstack-ceilometer | 16:29 | |
*** ruhe_ has quit IRC | 16:43 | |
*** boris-42_ has joined #openstack-ceilometer | 16:52 | |
*** yassine has quit IRC | 16:57 | |
*** DavidPatterson has joined #openstack-ceilometer | 17:01 | |
openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Propogate cacert and insecure flags to glanceclient https://review.openstack.org/72703 | 17:08 |
openstackgerrit | A change was merged to openstack/ceilometer: Refactored run-tests script https://review.openstack.org/71255 | 17:10 |
DavidPatterson | According to the Ceiolometer architecture document, I have been "kindly invited to come and talk to one of the project members" about using the oslo library to send events. :) | 17:13 |
DavidPatterson | I'm working on our (Radware's) LBAAS driver and would like to get load balancer metrics into Ceilometer. | 17:14 |
DavidPatterson | It seems the preferred way (according to the doc) is using oslo. | 17:15 |
DavidPatterson | Any quick pointers? | 17:16 |
DavidPatterson | (or pointers to pointers?) | 17:18 |
*** _nadya_ has quit IRC | 17:28 | |
*** _cjones_ has joined #openstack-ceilometer | 17:30 | |
*** mihgen has quit IRC | 17:32 | |
*** _nadya_ has joined #openstack-ceilometer | 17:45 | |
*** eglynn has quit IRC | 17:47 | |
*** prad_ has quit IRC | 17:54 | |
Alexei_987 | DavidPatterson: right now ceilometer is in the middle of migration to a new messaging library (olso) | 18:16 |
Alexei_987 | DavidPatterson: so the correct way for new plugins would be to use it | 18:16 |
DavidPatterson | I saw that ticket. | 18:16 |
DavidPatterson | ok | 18:16 |
Alexei_987 | DavidPatterson: not sure so that we have a working example of a plugin | 18:17 |
Alexei_987 | DavidPatterson: let me search for 1 sec | 18:17 |
DavidPatterson | ok. thanks. | 18:17 |
Alexei_987 | DavidPatterson: here you go - https://review.openstack.org/#/c/57457/ | 18:19 |
Alexei_987 | you can use this patch as a reference of how new plugins are going to work | 18:20 |
Alexei_987 | DavidPatterson: as you can see it's still not merged so your plugin would have to wait for this patch to land | 18:20 |
DavidPatterson | ok. thanks for your help. | 18:23 |
*** fmarco76 has joined #openstack-ceilometer | 18:40 | |
*** fmarco76 has left #openstack-ceilometer | 18:41 | |
*** ildikov_ has quit IRC | 18:45 | |
*** prad has joined #openstack-ceilometer | 18:45 | |
*** ruhe_ has joined #openstack-ceilometer | 19:41 | |
*** saju_m has quit IRC | 19:49 | |
dhellmann | DavidPatterson: does your driver run inside a neutron service, or is it a separate service? | 19:49 |
*** ruhe_ has quit IRC | 19:50 | |
*** saju_m has joined #openstack-ceilometer | 19:51 | |
*** ruhe- has joined #openstack-ceilometer | 19:55 | |
*** ruhe- has quit IRC | 19:56 | |
*** ruhe has quit IRC | 19:57 | |
*** ruhe has joined #openstack-ceilometer | 19:58 | |
*** saju_m has quit IRC | 20:04 | |
*** saju_m has joined #openstack-ceilometer | 20:05 | |
*** _nadya_ has quit IRC | 20:05 | |
*** _nadya_ has joined #openstack-ceilometer | 20:09 | |
*** Alexei_987 has quit IRC | 20:20 | |
*** _nadya_ has quit IRC | 20:21 | |
*** eglynn has joined #openstack-ceilometer | 20:26 | |
*** saju_m has quit IRC | 20:32 | |
*** eglynn has quit IRC | 20:38 | |
DavidPatterson | dhellmann: it is in a neutron service (/neutron/services/loadbalancer/drivers/radware), but I'm not sure that is the best place for polling the loadbalancers and putting metrics in ceilometer | 20:44 |
dhellmann | DavidPatterson: ah, polling | 20:44 |
dhellmann | well, you can build a new service using the framework in neutron if needed -- that code knows how to send notifications | 20:44 |
dhellmann | you can use the notify stuff in oslo-incubator until the patches land in oslo.messaging, and then port with the rest of neutron | 20:45 |
DavidPatterson | dhellmann: thanks. The best thing for me to do may be to wait for things to settle down in icehouse over the next few weeks | 20:49 |
dhellmann | DavidPatterson: sure, I did want to clear up the advice you'd been given already | 20:50 |
DavidPatterson | dhellmann: I'm new at all this so I appreciate all the help. Thanks. | 20:51 |
dhellmann | DavidPatterson: sure thing! | 20:51 |
*** _nadya_ has joined #openstack-ceilometer | 20:52 | |
*** _nadya_ has quit IRC | 21:00 | |
*** jdob_ has joined #openstack-ceilometer | 21:01 | |
*** ildikov_ has joined #openstack-ceilometer | 21:13 | |
*** jdob_ has quit IRC | 21:15 | |
*** DavidPatterson has left #openstack-ceilometer | 21:18 | |
*** xmltok has joined #openstack-ceilometer | 21:28 | |
*** thomasem has quit IRC | 21:45 | |
*** prad has quit IRC | 22:02 | |
*** prad has joined #openstack-ceilometer | 22:07 | |
*** boris-42_ has quit IRC | 22:13 | |
*** eglynn has joined #openstack-ceilometer | 22:16 | |
*** promulo has quit IRC | 22:19 | |
*** eglynn has quit IRC | 22:19 | |
*** daveed has joined #openstack-ceilometer | 22:25 | |
daveed | Hi guys | 22:25 |
*** flwang has quit IRC | 22:27 | |
daveed | We're trying to switch the ceilometer backend from mysql to mongodb and we're hitting some kind of bug. | 22:27 |
daveed | 2014-02-11 19:58:18.346 6809 ERROR wsme.api [-] Server-side error: "resource_id: Wrong type. Expected '<type 'unicode'>', got '<type 'int'>'". Detail: | 22:27 |
daveed | Traceback (most recent call last): | 22:27 |
daveed | This is the error message we get using ceilometer-api-2013.2.1 when trying to do a "ceilometer meter-list/resource-list" | 22:27 |
*** jmckind has quit IRC | 22:51 | |
*** flwang has joined #openstack-ceilometer | 23:02 | |
*** dhellmann is now known as dhellmann_ | 23:32 | |
*** jdob has quit IRC | 23:40 | |
*** prad has quit IRC | 23:43 | |
*** thomasem has joined #openstack-ceilometer | 23:50 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!