Friday, 2014-01-10

openstackgerritCyril Roelandt proposed a change to openstack/python-ceilometerclient: Python 3: fix test_sample_list  https://review.openstack.org/6583601:11
*** s2r2 has quit IRC01:21
*** s2r2 has joined #openstack-ceilometer01:29
openstackgerritA change was merged to openstack/ceilometer: Empty files should no longer contain copyright  https://review.openstack.org/6391201:38
*** flwang1 has joined #openstack-ceilometer02:02
*** sandywalsh has quit IRC02:03
*** flwang has quit IRC02:03
*** flwang has joined #openstack-ceilometer02:06
*** flwang1 has quit IRC02:07
*** coolsvap has quit IRC02:14
*** sandywalsh has joined #openstack-ceilometer02:19
*** flwang1 has joined #openstack-ceilometer02:30
*** flwang has quit IRC02:30
*** flwang has joined #openstack-ceilometer03:00
*** flwang1 has quit IRC03:04
openstackgerritFengqian.gao proposed a change to openstack/ceilometer: Add pagination support for sqlalchemy database  https://review.openstack.org/3545403:32
openstackgerritFengqian.gao proposed a change to openstack/ceilometer: Change pagination related methods of mongodb and db2  https://review.openstack.org/4186903:32
*** flwang has quit IRC04:10
openstackgerritA change was merged to openstack/python-ceilometerclient: Fix the ceilometerlient log curl request incorrectly  https://review.openstack.org/6327904:18
*** flwang has joined #openstack-ceilometer04:24
openstackgerritA change was merged to openstack/ceilometer: Fix work of udp publisher  https://review.openstack.org/6265604:25
*** flwang has quit IRC04:44
openstackgerritAlexei Kornienko proposed a change to openstack/python-ceilometerclient: Added argcomplete lib to enable cli completition  https://review.openstack.org/6586405:00
*** coolsvap has joined #openstack-ceilometer05:02
*** nadya_ has joined #openstack-ceilometer05:19
*** nadya_ has quit IRC05:28
*** SergeyLukjanov has joined #openstack-ceilometer05:34
*** SergeyLukjanov has quit IRC05:41
*** flwang has joined #openstack-ceilometer05:52
openstackgerritJenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex  https://review.openstack.org/6280806:06
openstackgerritA change was merged to openstack/python-ceilometerclient: Use Resource() class from common Oslo code  https://review.openstack.org/6288306:12
*** Alexei_987 has joined #openstack-ceilometer06:23
*** ildikov_ has quit IRC06:43
openstackgerritA change was merged to openstack/ceilometer: Allow customized reseller_prefix in Ceilometer middleware for Swift  https://review.openstack.org/6215506:47
*** Alexei_987 has quit IRC06:59
*** ildikov has joined #openstack-ceilometer07:33
*** Alexei_987 has joined #openstack-ceilometer07:57
*** shengjiemin has joined #openstack-ceilometer08:27
shengjieminHi guys, quick question -08:27
shengjiemincan't find it anywhere from the documentation, is there a meter in Ceilometer allows me to charge users based on the uptime of the instances they have.08:31
*** julienvey has joined #openstack-ceilometer09:13
*** eglynn has joined #openstack-ceilometer09:14
*** yassine has joined #openstack-ceilometer09:22
*** yassine has quit IRC09:29
*** yassine has joined #openstack-ceilometer09:29
*** yassine has quit IRC09:29
*** yassine has joined #openstack-ceilometer09:29
*** yassine has quit IRC09:30
*** yassine has joined #openstack-ceilometer09:30
*** yassine has quit IRC09:31
*** yassine has joined #openstack-ceilometer09:31
*** yassine has quit IRC09:32
*** yassine has joined #openstack-ceilometer09:32
*** _ruhe is now known as ruhe09:33
*** nadya_ has joined #openstack-ceilometer09:41
*** nadya_ has quit IRC09:52
ityaptinjd__: Hi) I'm about https://blueprints.launchpad.net/ceilometer/+spec/run-all-pollsters-on-demand09:53
ityaptinjd__, can you discuss this now?09:54
*** sayali has joined #openstack-ceilometer09:57
*** sayali_ has joined #openstack-ceilometer09:57
*** nadya_ has joined #openstack-ceilometer10:03
*** shengjiemin has quit IRC10:06
jd__ityaptin: listening10:08
*** SergeyLukjanov has joined #openstack-ceilometer10:10
ityaptinjd__: I implemented rpc calls of run_pollsters at agents, but I don't know where we have to run this rpc call.10:16
jd__what do you mean run this rpc call?10:18
ityaptinI run one method of agents by rpc the same way as agents make rpc call of collector's "record_metering_data"10:22
ityaptinjd__: I run one method of agents by rpc the same way as agents make rpc call of collector's "record_metering_data"10:23
*** ruhe is now known as _ruhe10:26
ityaptinjd__: put to rabbit queue message with rpc call of agent's method10:46
*** _ruhe is now known as ruhe11:00
ityaptinjd__: sorry, early I misunderstood question. More about run this rpc call: I added method which publish message with target 'run_pollsters' to rpc publisher and planned call it from api.11:01
*** ruhe is now known as ruhe_away11:04
*** ruhe_away is now known as ruhe11:04
*** ruhe is now known as _ruhe11:33
*** sayali has quit IRC11:41
*** _ruhe is now known as ruhe11:45
*** kwhitney2 has quit IRC12:23
*** kwhitney1 has joined #openstack-ceilometer12:24
nprivalovaeglynn, hi! I've updated an etherpad https://etherpad.openstack.org/p/ceilometer-aggregation . All of this should be in blueprint description but I'd like to show it to you first :)12:37
nprivalovajd__, ping12:42
jd__ityaptin: that looks good, I think you just need to get the API server to send that call no?12:42
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: PoC: use Colander instead of WSME in the API  https://review.openstack.org/6593512:46
ityaptinjd__: yes, and I want add api controller for this12:46
*** coolsvap has quit IRC12:46
jd__ityaptin: ok I think it would be good to describe more of this in the blueprint FWIW, otherwise I guess it looks good enough12:46
nprivalovajd__, I have two questions for you :)12:48
nprivalovaActually for all... the idea to have HBase installed in devstack and gating is ok? I think it will be useful. what do you think, guys?12:51
jd__nprivalova: if you want to build integration tests you have my blessing12:55
nprivalovajd__, thanks! :)12:56
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: PoC: use Colander instead of WSME in the API  https://review.openstack.org/6593512:59
*** nadya_ has joined #openstack-ceilometer13:04
*** SergeyLukjanov is now known as _SergeyLukjanov13:05
*** SergeyLukjanov has joined #openstack-ceilometer13:08
*** nadya_ has quit IRC13:09
openstackgerritA change was merged to openstack/ceilometer: Remove redundant meter (name,type,unit) tuples from Resource model  https://review.openstack.org/6567113:15
*** sandywalsh has quit IRC13:18
*** sandywalsh has joined #openstack-ceilometer13:32
*** thomasem has joined #openstack-ceilometer13:42
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Convert alarm timestamp to PrecisionTimestamp  https://review.openstack.org/6313113:46
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: [don't review] oslo.db tests  https://review.openstack.org/6567713:46
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Set the SQL Float precision  https://review.openstack.org/6313213:46
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Run unit tests against MySQL  https://review.openstack.org/5948913:46
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Run tests against PostgreSQL  https://review.openstack.org/6304913:46
*** jdob has joined #openstack-ceilometer13:47
silehtjd__, can take a look to https://review.openstack.org/#/c/61814/13:49
jd__ure13:51
jd__sure13:51
*** sayali has joined #openstack-ceilometer13:52
*** sayali_ has joined #openstack-ceilometer13:52
*** gordc has joined #openstack-ceilometer14:01
jd__sileht: did you steal https://bugs.launchpad.net/oslo/+bug/1263908 btw?14:06
silehtjd__, yes14:06
silehtjd__, really stolen :)14:06
jd__cool thanks :)14:07
jd__sileht: what's the status of oslo.messaging migration?14:10
silehtjd__, I'm waiting for markmc to continue14:11
jd__sileht: ok14:11
silehtjd__, I have (re)ping him yesterday for reviewing14:11
jd__I changed the status to 'slow progress'14:11
*** jdob has quit IRC14:15
openstackgerritEoghan Glynn proposed a change to openstack/ceilometer: Replace mongo aggregation with plain ol' map-reduce  https://review.openstack.org/6596214:16
*** jdob has joined #openstack-ceilometer14:16
*** jdob has quit IRC14:19
*** jdob has joined #openstack-ceilometer14:19
*** eglynn is now known as eglynn-is-hypogl14:22
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: [don't review] oslo.db tests  https://review.openstack.org/6567714:23
*** eglynn-is-hypogl is now known as eglynn-lunch14:28
thomasemHey everyone!14:34
nprivalovajd__, Ilya has updated https://blueprints.launchpad.net/ceilometer/+spec/run-all-pollsters-on-demand14:34
*** prad has joined #openstack-ceilometer14:35
jd__eglynn-lunch: your nicknames make me think about http://en.wikipedia.org/wiki/Martine_(character), I don't know if it's known outside of France though :)14:37
*** eglynn-lunch is now known as eglynn14:43
eglynnjd__: lolz :)14:43
nprivalovadhellman and gordc, are you here?14:48
Alexei_987jd__: ping14:56
jd__Alexei_987: ?14:56
Alexei_987jd__: question about this mapreduce stuff14:56
Alexei_987jd__: it seems that we cannot store results if we provide a timestamp14:57
jd__a time range yeah14:57
Alexei_987jd__: but I think that we can store them if we call this method without a timestamp14:57
jd__which is likely 1% of cases?14:57
Alexei_987jd__: and it's called that way in controller get_one14:58
Alexei_987well I don't know actuall percentage14:58
Alexei_987but it would be 100 times faster than do a realtime mapreduce14:58
Alexei_987so I propose to implement 2 execution paths14:58
Alexei_9871) without timestamp (fast)14:59
Alexei_9872) with timestamp (mapreduce in realtime)14:59
jd__100 times faster on a 1 % code path, I'm not that interested14:59
jd__but I admit that it makes sense for get_one14:59
jd__so if you want to optimize get_one, go for it14:59
jd__you may build a get_resource() method in the drivers (just an idea)14:59
Alexei_987yeah :) I'll see what I can do15:00
jd__Alexei_987: but yeah that's why I think your idea as described originally in the review was not going to work, but it can be a good complement of eglynn patch if you care about it :)15:00
Alexei_987well I think that this timestamp handling is quite crappy15:01
Alexei_987it has even mark on it # FIXME(dhellmann): This may not perform very well,15:01
nprivalovaeglynn, hi! I've updated etherpad https://etherpad.openstack.org/p/ceilometer-aggregation . Please take a look when you have time please. I just need any feedback. Smth like:  'it may work' or 'we don't need it. forget this idea' :) jd__, I have the same plea for you. Maybe I can provide more examples for you or some performance investigations15:01
eglynnAlexei_987: such an optimization for the get_one resource case would be useful I think ... so you've got the bandwidth go for it!15:03
eglynnnprivalova: thanks, will review this afternoon15:03
nprivalovaeglynn, thanks!15:07
*** coolsvap has joined #openstack-ceilometer15:09
jd__nprivalova: I'll read and answer on the ml :)15:10
nprivalovajd__, great! :)15:11
*** gordc has quit IRC15:15
*** jmckind has joined #openstack-ceilometer15:41
*** jmckind has quit IRC15:56
*** herndon_ has joined #openstack-ceilometer15:56
*** herndon_ has quit IRC16:01
*** gordc has joined #openstack-ceilometer16:03
gordcnprivalova: sorry for delayed response. did you still need anything from me?16:17
*** SergeyLukjanov has quit IRC16:30
*** ozialien has joined #openstack-ceilometer16:53
*** ruhe is now known as _ruhe16:59
*** _ruhe is now known as ruhe17:07
*** Alexei_987 has quit IRC17:17
*** SergeyLukjanov has joined #openstack-ceilometer17:19
*** eglynn has quit IRC17:39
*** ruhe is now known as _ruhe17:43
*** nadya_ has joined #openstack-ceilometer18:32
*** SergeyLukjanov has quit IRC18:35
*** SergeyLukjanov has joined #openstack-ceilometer18:38
*** nadya_ has quit IRC18:45
openstackgerritJohn Herndon proposed a change to openstack/python-ceilometerclient: Support the Event API  https://review.openstack.org/5458219:00
*** herndon_ has joined #openstack-ceilometer19:01
*** herndon_ has quit IRC19:02
*** herndon_ has joined #openstack-ceilometer19:11
*** _ruhe is now known as ruhe19:18
*** sayali has quit IRC19:25
*** herndon_ has quit IRC19:28
thomasemgordc, ping19:29
gordcthomasem: whatup19:31
thomasemgordc, hey hey, wanted to ping you regarding the comment about importing the models file directly in v2.py on this review: https://review.openstack.org/#/c/57353/5/ceilometer/api/controllers/v2.py19:32
gordcok, taking a look at it now.19:33
thomasemgordc, So, IIRC, I did that because of the set_traits function which I think is executed by WSME/Pecan. It wasn't able to find storage.models.Trait, I'm a bit fuzzy on the import subtleties there, but I noticed models isn't imported in storage/__init__.py.19:33
thomasemSo, it could be storage.models.Trait and that seemingly duplicate import could be removed if we imported models in the storage module itself.19:34
thomasemBut, it sounds like pep8 doesn't like doing that either, so I'm not sure of the best approach there. Any ideas?19:35
gordchmm... yeah, i'm not really familiar with how imports link between submodules. not sure if i have a recommendation on best way to proceed with it.19:36
gordci don't know if it causes any issues having some references to storage.models.xyz and other directly at models.xyz but if it doesn't really make a difference i'm ok with keeping it (and letting someone else who knows better fix it if needed).19:37
thomasemgordc, Yeah, I'd like to understand it better, might bug some resident python guru19:37
gordcit just looked strange to me so i thought i'd ask in case it was left in unintentionally.19:37
thomasemgordc, It just might get confusing, is all.19:37
thomasemhaving inconsistent references19:38
thomasemNah, I'm glad you did.19:38
thomasem:P19:38
gordcthomasem: yeah, the confusing part was why i brought it up... no idea what right solutions is.19:38
thomasemcoolio, well I'll try to get more info anyway, but it does work for now.19:38
thomasemWe could also just move all of the references over to models.* instead of storage.models.* :P19:39
thomasemfor consistency19:40
gordcyeah... that's the alternative... there's a boatload of references to it storage.models so i didn't want to directly suggest you fix all of them.lol19:40
*** ruhe is now known as _ruhe19:51
*** herndon_ has joined #openstack-ceilometer20:15
*** prad has quit IRC20:32
*** flwang has quit IRC20:45
*** jmckind has joined #openstack-ceilometer20:45
*** sandywalsh has quit IRC20:55
*** yassine has quit IRC20:57
*** yassine has joined #openstack-ceilometer20:57
*** yassine has quit IRC20:58
*** yassine has joined #openstack-ceilometer20:58
*** yassine has quit IRC20:59
*** flwang has joined #openstack-ceilometer21:00
*** prad has joined #openstack-ceilometer21:03
*** sandywalsh has joined #openstack-ceilometer21:06
*** SergeyLukjanov has quit IRC21:29
*** jdob has quit IRC22:00
*** jmckind has quit IRC22:23
*** ozialien has quit IRC22:34
*** thomasem has quit IRC22:43
*** prad has quit IRC22:49
*** prad has joined #openstack-ceilometer22:53
*** herndon_ has quit IRC23:22
*** ozialien has joined #openstack-ceilometer23:31
*** kwhitney1 has quit IRC23:59

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