Tuesday, 2014-02-11

openstackgerritA change was merged to openstack/ceilometer: Adds flavor_id in the nova_notifier  https://review.openstack.org/6958200:30
openstackgerritA change was merged to openstack/python-ceilometerclient: Python 3: fix format_nested_list_of_dict()  https://review.openstack.org/7240500:30
openstackgerritA change was merged to openstack/python-ceilometerclient: Sync with Oslo  https://review.openstack.org/7239200:34
*** xmltok has quit IRC00:55
*** sileht has quit IRC01:29
*** sileht has joined #openstack-ceilometer01:38
openstackgerritliusheng proposed a change to openstack/ceilometer: check domain state before inspecting nics/disks  https://review.openstack.org/6888202:02
*** flwang has quit IRC02:17
*** xianghui has joined #openstack-ceilometer02:18
*** gordc has quit IRC02:33
*** flwang has joined #openstack-ceilometer02:53
*** flwang has quit IRC02:59
*** flwang has joined #openstack-ceilometer03:00
*** xianghui has quit IRC03:01
*** xianghui has joined #openstack-ceilometer03:16
*** Yathi has joined #openstack-ceilometer03:23
*** Yathi has quit IRC03:27
openstackgerritliusheng proposed a change to openstack/ceilometer: check domain state before inspecting nics/disks  https://review.openstack.org/6888203:54
openstackgerritYuuichi Fujioka proposed a change to openstack/python-ceilometerclient: fix help message of deprecated command  https://review.openstack.org/7219804:34
openstackgerritJenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex  https://review.openstack.org/6280806:02
*** rwsu has quit IRC06:07
*** rwsu has joined #openstack-ceilometer06:07
*** ildikov_ has quit IRC06:16
*** rwsu has quit IRC06:25
*** ildikov_ has joined #openstack-ceilometer06:31
*** rwsu has joined #openstack-ceilometer06:38
*** saju_m has joined #openstack-ceilometer06:38
ityaptinllu: ping06:40
*** saju_m has quit IRC06:43
*** saju_m has joined #openstack-ceilometer06:50
*** saju_m has quit IRC06:51
*** _nadya_ has joined #openstack-ceilometer06:52
*** _nadya_ has quit IRC07:01
lluityaptin: pong07:03
*** eglynn has quit IRC07:15
openstackgerritDazhao Yu proposed a change to openstack/ceilometer: Make sure use IPv6 sockets for ceilometer in IPv6 environment  https://review.openstack.org/6778607:19
*** saju_m has joined #openstack-ceilometer07:37
*** eglynn has joined #openstack-ceilometer07:45
*** saju_m has quit IRC07:52
openstackgerritliusheng proposed a change to openstack/python-ceilometerclient: Remove unused mock in v2/test_shell.py  https://review.openstack.org/7258308:07
ityaptinllu: About https://review.openstack.org/#/c/71595/ and your comments.08:20
lluityaptin: I'm just wondering if this is a coding convention of not?08:24
ityaptinllu: In testtools if length of  assert mismatch message more than 70 letters it's described with "expected=" and "actual="08:24
lluityaptin: ok, so this is defined in testtool module?08:26
ityaptinllu: Yes. Also in testtools arguments of method assertEqual are  placed in order "expected,actual".08:31
lluityaptin: got that, thanks for your info08:31
ityaptinllu: thanks for your questions)08:33
*** eglynn has quit IRC08:36
*** saju_m has joined #openstack-ceilometer08:51
*** flwang has quit IRC08:52
*** yassine has joined #openstack-ceilometer09:03
openstackgerritVictor Sergeyev proposed a change to openstack/ceilometer: POC: Add test for check sync models and migrations  https://review.openstack.org/6967409:04
*** eglynn has joined #openstack-ceilometer09:11
*** flwang has joined #openstack-ceilometer09:30
*** boris-42_ has quit IRC09:36
nprivalovaeglynn, 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
eglynnnprivalova: ... looking now09:45
eglynnnprivalova: ... ok in theory he's probably right that *if* the upstream project was chronically unresponsive09:47
eglynnnprivalova: ... then we would have no options other than (a) to fork, or (b) to find another equivalent dependency09:48
eglynnnprivalova: ... however, IMO it's completely unrealistic to hold all developement on the ceilometer hbase driver by requiring that we fork it NOW09:48
eglynnnprivalova: ... hold *up all development09:49
eglynnnprivalova: ... I mean, the necessary fix may be forthcoming upstream, just slow to land and be released09:49
eglynnnprivalova: ... in that case, we should have the option to upper-bound the dependency as a pragmatic workaround09:50
nprivalovaeglynn: 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
eglynnnprivalova: cool, then seems to me most reasonable option is to wait for the fix and work-around in the meantime09:51
nprivalovaeglynn: yep, ok09:51
eglynnnprivalova: ... though if it turns out that months pass and still no fix upstream, we'd have to reconsider09:51
eglynnnprivalova: ... I'll put the above thoughts on gerrit also09:52
nprivalovaeglynn: ok, thank you09:52
silehtnprivalova, have you asked to upstream to release a need version ? the workaround looks already in the master of happybase09:52
*** mihgen has joined #openstack-ceilometer09:54
nprivalovasileht: not yet, will do it ASAP. They have created the fix, yes, but after that  they wrote "we have found more problems"09:54
silehtnprivalova, 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.709:58
nprivalovasileht: yes, the same as I proposed but looks much safer :)09:59
* sileht hope upstream released a new version soon10:00
nprivalovasileht: what is the status of notifications in nova, do you know?10:00
nprivalovasileht: tempest tests for notifications don't work :( ERROR oslo.messaging.notify._impl_messaging [-] Could not send notification to notifications10:01
silehtnprivalova, 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
silehtnprivalova, jd__ have uploaded a 4° patchset that fix the issue in an other ways, so wait and see10:10
nprivalovasileht: just looking into https://review.openstack.org/#/c/64136/10 . it's strange that dsvm-full fails but postgress pass10:11
* sileht is looking10:14
silehtthe oslo.messaging error messages in the postgres console.log scary me10:18
nprivalovasileht: yes. will try to install latest devstack and check how it works10:22
*** saju_m has quit IRC10:25
*** saju_m has joined #openstack-ceilometer10:26
*** xianghui has quit IRC10:29
*** yassine has quit IRC10:56
*** Alexei_987 has quit IRC11:10
*** boris-42_ has joined #openstack-ceilometer11:16
eglynnjd__: good morning11:17
eglynn... too late for that, good afternoon! :)11:17
*** mihgen has quit IRC11:18
eglynnjd__: I suspect your preference would be to have *less* on our plates for i3, rather than more11:18
jd__eglynn: likely11:18
eglynnjd__: .. but that said, what's your feeling on targeting that source/sink BP to i3?11:18
eglynnhttps://blueprints.launchpad.net/ceilometer/+spec/decoupled-source-sink-discoverable-resources11:18
eglynn(perhaps bumping the periodicity BP to Juno to make room?)11:19
eglynni.e. https://blueprints.launchpad.net/ceilometer/+spec/record-sample-periodicity11:19
jd__eglynn: I think it's doable11:32
eglynnjd__: cool11:32
jd__if someone/you take it in charge :)11:32
eglynnjd__: FWIW my feeling is that something along these lines will be required to allow the SNMP-based hardware polling to be useable in icehouse11:32
eglynnjd__: yep, I'd be committing to implementing for i311:33
jd__yep11:33
jd__great11:33
eglynnjd__: ... so I'll just go ahead and retarget, or is that the PTL's prerogative?11:33
jd__eglynn: no go ahead11:37
*** mihgen has joined #openstack-ceilometer11:40
openstackgerritA change was merged to openstack/ceilometer: fix column name and alignment  https://review.openstack.org/6952411:44
openstackgerritA change was merged to openstack/ceilometer: Test case for creating an alarm without auth headers  https://review.openstack.org/7240611:46
openstackgerritA change was merged to openstack/ceilometer: Remove tox locale overrides  https://review.openstack.org/7220211:50
openstackgerritA change was merged to openstack/ceilometer: samples: fix test case status code check  https://review.openstack.org/6964811:50
openstackgerritA change was merged to openstack/ceilometer: Improve help strings  https://review.openstack.org/7198711:50
*** SergeyLukjanov has quit IRC11:51
*** SergeyLukjanov has joined #openstack-ceilometer11:55
*** Alexei_987 has joined #openstack-ceilometer12:07
*** promulo has quit IRC12:10
*** yassine has joined #openstack-ceilometer12:11
openstackgerritA change was merged to openstack/ceilometer: Implements complex query functionality for samples  https://review.openstack.org/6215712:19
*** eglynn is now known as eglynn-lunch12:29
nprivalovaildikov_: congrats! :)Very complicated change12:33
ildikov_nprivalova: thanks, I'm so happy, just about to cry :)12:38
nprivalovaildikov_: 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 think12:39
ildikov_nprivalova: just because I'm happy :)12:40
nprivalovaildikov_: yes, sure! The most difficult part is to merge the first part if bp12: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 helped12:44
openstackgerritlitong01 proposed a change to openstack/ceilometer: add notification alarm  https://review.openstack.org/6947312:53
openstackgerritlitong01 proposed a change to openstack/python-ceilometerclient: add notification alarm  https://review.openstack.org/6947412:53
*** vrovachev has left #openstack-ceilometer12:59
*** jdob has joined #openstack-ceilometer13:26
*** gordc has joined #openstack-ceilometer13:28
*** thomasem has joined #openstack-ceilometer13:38
openstackgerritIldiko Vancsa proposed a change to openstack/ceilometer: Implements in operator for complex query functionality  https://review.openstack.org/6668713:44
openstackgerritIldiko Vancsa proposed a change to openstack/ceilometer: Implements field validation for complex query functionality  https://review.openstack.org/6530213:44
openstackgerritIldiko Vancsa proposed a change to openstack/ceilometer: Implements complex query functionality for alarms  https://review.openstack.org/6215813:44
openstackgerritIldiko Vancsa proposed a change to openstack/ceilometer: Implements complex query functionality for alarm history  https://review.openstack.org/6215913:44
*** jmckind has joined #openstack-ceilometer14:01
*** chmouel has left #openstack-ceilometer14:17
Alexei_987jd__: Hi14:25
*** saju_m has quit IRC14:27
*** saju_m has joined #openstack-ceilometer14:29
*** promulo has joined #openstack-ceilometer14:34
*** prad_ has joined #openstack-ceilometer14:53
*** ilyashakhat has quit IRC14:54
*** ddutta has joined #openstack-ceilometer15:01
*** Yathi has joined #openstack-ceilometer15:02
*** eglynn-lunch is now known as eglynn15:09
*** mihgen has quit IRC15:11
*** ruhe_ has joined #openstack-ceilometer15:17
openstackgerritBalazs Gibizer proposed a change to openstack/ceilometer: Implements "not" operator for complex query  https://review.openstack.org/6689215:19
openstackgerritBalazs Gibizer proposed a change to openstack/ceilometer: Implements metadata query for complex query feature  https://review.openstack.org/6689115:19
openstackgerritBalazs Gibizer proposed a change to openstack/ceilometer: Implements "not" operator for complex query  https://review.openstack.org/6689215:29
*** boris-42_ has quit IRC15:36
gordcdhellmann: ping15:37
*** Yathi has quit IRC15:44
*** saju_m has quit IRC15:45
*** saju_m has joined #openstack-ceilometer15:46
*** ddutta has quit IRC16:06
*** mihgen has joined #openstack-ceilometer16:08
*** mihgen has quit IRC16:19
*** mihgen has joined #openstack-ceilometer16:21
*** _nadya_ has joined #openstack-ceilometer16:29
*** ruhe_ has quit IRC16:43
*** boris-42_ has joined #openstack-ceilometer16:52
*** yassine has quit IRC16:57
*** DavidPatterson has joined #openstack-ceilometer17:01
openstackgerritEoghan Glynn proposed a change to openstack/ceilometer: Propogate cacert and insecure flags to glanceclient  https://review.openstack.org/7270317:08
openstackgerritA change was merged to openstack/ceilometer: Refactored run-tests script  https://review.openstack.org/7125517:10
DavidPattersonAccording 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
DavidPattersonI'm working on our (Radware's) LBAAS driver and would like to get load balancer metrics into Ceilometer.17:14
DavidPattersonIt seems the preferred way (according to the doc) is using oslo.17:15
DavidPattersonAny quick pointers?17:16
DavidPatterson(or pointers to pointers?)17:18
*** _nadya_ has quit IRC17:28
*** _cjones_ has joined #openstack-ceilometer17:30
*** mihgen has quit IRC17:32
*** _nadya_ has joined #openstack-ceilometer17:45
*** eglynn has quit IRC17:47
*** prad_ has quit IRC17:54
Alexei_987DavidPatterson: right now ceilometer is in the middle of migration to a new messaging library (olso)18:16
Alexei_987DavidPatterson: so the correct way for new plugins would be to use it18:16
DavidPattersonI saw that ticket.18:16
DavidPattersonok18:16
Alexei_987DavidPatterson: not sure so that we have a working example of a plugin18:17
Alexei_987DavidPatterson: let me search for 1 sec18:17
DavidPattersonok.  thanks.18:17
Alexei_987DavidPatterson: here you go - https://review.openstack.org/#/c/57457/18:19
Alexei_987you can use this patch as a reference of how new plugins are going to work18:20
Alexei_987DavidPatterson: as you can see it's still not merged so your plugin would have to wait for this patch to land18:20
DavidPattersonok.  thanks for your help.18:23
*** fmarco76 has joined #openstack-ceilometer18:40
*** fmarco76 has left #openstack-ceilometer18:41
*** ildikov_ has quit IRC18:45
*** prad has joined #openstack-ceilometer18:45
*** ruhe_ has joined #openstack-ceilometer19:41
*** saju_m has quit IRC19:49
dhellmannDavidPatterson: does your driver run inside a neutron service, or is it a separate service?19:49
*** ruhe_ has quit IRC19:50
*** saju_m has joined #openstack-ceilometer19:51
*** ruhe- has joined #openstack-ceilometer19:55
*** ruhe- has quit IRC19:56
*** ruhe has quit IRC19:57
*** ruhe has joined #openstack-ceilometer19:58
*** saju_m has quit IRC20:04
*** saju_m has joined #openstack-ceilometer20:05
*** _nadya_ has quit IRC20:05
*** _nadya_ has joined #openstack-ceilometer20:09
*** Alexei_987 has quit IRC20:20
*** _nadya_ has quit IRC20:21
*** eglynn has joined #openstack-ceilometer20:26
*** saju_m has quit IRC20:32
*** eglynn has quit IRC20:38
DavidPattersondhellmann: 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 ceilometer20:44
dhellmannDavidPatterson: ah, polling20:44
dhellmannwell, you can build a new service using the framework in neutron if needed -- that code knows how to send notifications20:44
dhellmannyou can use the notify stuff in oslo-incubator until the patches land in oslo.messaging, and then port with the rest of neutron20:45
DavidPattersondhellmann: thanks.  The best thing for me to do may be to wait for things to settle down in icehouse over the next few weeks20:49
dhellmannDavidPatterson: sure, I did want to clear up the advice you'd been given already20:50
DavidPattersondhellmann:  I'm new at all this so I appreciate all the help.  Thanks.20:51
dhellmannDavidPatterson: sure thing!20:51
*** _nadya_ has joined #openstack-ceilometer20:52
*** _nadya_ has quit IRC21:00
*** jdob_ has joined #openstack-ceilometer21:01
*** ildikov_ has joined #openstack-ceilometer21:13
*** jdob_ has quit IRC21:15
*** DavidPatterson has left #openstack-ceilometer21:18
*** xmltok has joined #openstack-ceilometer21:28
*** thomasem has quit IRC21:45
*** prad has quit IRC22:02
*** prad has joined #openstack-ceilometer22:07
*** boris-42_ has quit IRC22:13
*** eglynn has joined #openstack-ceilometer22:16
*** promulo has quit IRC22:19
*** eglynn has quit IRC22:19
*** daveed has joined #openstack-ceilometer22:25
daveedHi guys22:25
*** flwang has quit IRC22:27
daveedWe're trying to switch the ceilometer backend from mysql to mongodb and we're hitting some kind of bug.22:27
daveed2014-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
daveedTraceback (most recent call last):22:27
daveedThis 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 IRC22:51
*** flwang has joined #openstack-ceilometer23:02
*** dhellmann is now known as dhellmann_23:32
*** jdob has quit IRC23:40
*** prad has quit IRC23:43
*** thomasem has joined #openstack-ceilometer23:50

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!