openstackgerrit | Cyril Roelandt proposed a change to openstack/python-ceilometerclient: Python 3: fix test_sample_list https://review.openstack.org/65836 | 01:11 |
---|---|---|
*** s2r2 has quit IRC | 01:21 | |
*** s2r2 has joined #openstack-ceilometer | 01:29 | |
openstackgerrit | A change was merged to openstack/ceilometer: Empty files should no longer contain copyright https://review.openstack.org/63912 | 01:38 |
*** flwang1 has joined #openstack-ceilometer | 02:02 | |
*** sandywalsh has quit IRC | 02:03 | |
*** flwang has quit IRC | 02:03 | |
*** flwang has joined #openstack-ceilometer | 02:06 | |
*** flwang1 has quit IRC | 02:07 | |
*** coolsvap has quit IRC | 02:14 | |
*** sandywalsh has joined #openstack-ceilometer | 02:19 | |
*** flwang1 has joined #openstack-ceilometer | 02:30 | |
*** flwang has quit IRC | 02:30 | |
*** flwang has joined #openstack-ceilometer | 03:00 | |
*** flwang1 has quit IRC | 03:04 | |
openstackgerrit | Fengqian.gao proposed a change to openstack/ceilometer: Add pagination support for sqlalchemy database https://review.openstack.org/35454 | 03:32 |
openstackgerrit | Fengqian.gao proposed a change to openstack/ceilometer: Change pagination related methods of mongodb and db2 https://review.openstack.org/41869 | 03:32 |
*** flwang has quit IRC | 04:10 | |
openstackgerrit | A change was merged to openstack/python-ceilometerclient: Fix the ceilometerlient log curl request incorrectly https://review.openstack.org/63279 | 04:18 |
*** flwang has joined #openstack-ceilometer | 04:24 | |
openstackgerrit | A change was merged to openstack/ceilometer: Fix work of udp publisher https://review.openstack.org/62656 | 04:25 |
*** flwang has quit IRC | 04:44 | |
openstackgerrit | Alexei Kornienko proposed a change to openstack/python-ceilometerclient: Added argcomplete lib to enable cli completition https://review.openstack.org/65864 | 05:00 |
*** coolsvap has joined #openstack-ceilometer | 05:02 | |
*** nadya_ has joined #openstack-ceilometer | 05:19 | |
*** nadya_ has quit IRC | 05:28 | |
*** SergeyLukjanov has joined #openstack-ceilometer | 05:34 | |
*** SergeyLukjanov has quit IRC | 05:41 | |
*** flwang has joined #openstack-ceilometer | 05:52 | |
openstackgerrit | Jenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/62808 | 06:06 |
openstackgerrit | A change was merged to openstack/python-ceilometerclient: Use Resource() class from common Oslo code https://review.openstack.org/62883 | 06:12 |
*** Alexei_987 has joined #openstack-ceilometer | 06:23 | |
*** ildikov_ has quit IRC | 06:43 | |
openstackgerrit | A change was merged to openstack/ceilometer: Allow customized reseller_prefix in Ceilometer middleware for Swift https://review.openstack.org/62155 | 06:47 |
*** Alexei_987 has quit IRC | 06:59 | |
*** ildikov has joined #openstack-ceilometer | 07:33 | |
*** Alexei_987 has joined #openstack-ceilometer | 07:57 | |
*** shengjiemin has joined #openstack-ceilometer | 08:27 | |
shengjiemin | Hi guys, quick question - | 08:27 |
shengjiemin | can'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-ceilometer | 09:13 | |
*** eglynn has joined #openstack-ceilometer | 09:14 | |
*** yassine has joined #openstack-ceilometer | 09:22 | |
*** yassine has quit IRC | 09:29 | |
*** yassine has joined #openstack-ceilometer | 09:29 | |
*** yassine has quit IRC | 09:29 | |
*** yassine has joined #openstack-ceilometer | 09:29 | |
*** yassine has quit IRC | 09:30 | |
*** yassine has joined #openstack-ceilometer | 09:30 | |
*** yassine has quit IRC | 09:31 | |
*** yassine has joined #openstack-ceilometer | 09:31 | |
*** yassine has quit IRC | 09:32 | |
*** yassine has joined #openstack-ceilometer | 09:32 | |
*** _ruhe is now known as ruhe | 09:33 | |
*** nadya_ has joined #openstack-ceilometer | 09:41 | |
*** nadya_ has quit IRC | 09:52 | |
ityaptin | jd__: Hi) I'm about https://blueprints.launchpad.net/ceilometer/+spec/run-all-pollsters-on-demand | 09:53 |
ityaptin | jd__, can you discuss this now? | 09:54 |
*** sayali has joined #openstack-ceilometer | 09:57 | |
*** sayali_ has joined #openstack-ceilometer | 09:57 | |
*** nadya_ has joined #openstack-ceilometer | 10:03 | |
*** shengjiemin has quit IRC | 10:06 | |
jd__ | ityaptin: listening | 10:08 |
*** SergeyLukjanov has joined #openstack-ceilometer | 10:10 | |
ityaptin | jd__: 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 |
ityaptin | I run one method of agents by rpc the same way as agents make rpc call of collector's "record_metering_data" | 10:22 |
ityaptin | jd__: 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 _ruhe | 10:26 | |
ityaptin | jd__: put to rabbit queue message with rpc call of agent's method | 10:46 |
*** _ruhe is now known as ruhe | 11:00 | |
ityaptin | jd__: 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_away | 11:04 | |
*** ruhe_away is now known as ruhe | 11:04 | |
*** ruhe is now known as _ruhe | 11:33 | |
*** sayali has quit IRC | 11:41 | |
*** _ruhe is now known as ruhe | 11:45 | |
*** kwhitney2 has quit IRC | 12:23 | |
*** kwhitney1 has joined #openstack-ceilometer | 12:24 | |
nprivalova | eglynn, 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 |
nprivalova | jd__, ping | 12:42 |
jd__ | ityaptin: that looks good, I think you just need to get the API server to send that call no? | 12:42 |
openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: PoC: use Colander instead of WSME in the API https://review.openstack.org/65935 | 12:46 |
ityaptin | jd__: yes, and I want add api controller for this | 12:46 |
*** coolsvap has quit IRC | 12: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 enough | 12:46 |
nprivalova | jd__, I have two questions for you :) | 12:48 |
nprivalova | Actually 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 blessing | 12:55 |
nprivalova | jd__, thanks! :) | 12:56 |
openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: PoC: use Colander instead of WSME in the API https://review.openstack.org/65935 | 12:59 |
*** nadya_ has joined #openstack-ceilometer | 13:04 | |
*** SergeyLukjanov is now known as _SergeyLukjanov | 13:05 | |
*** SergeyLukjanov has joined #openstack-ceilometer | 13:08 | |
*** nadya_ has quit IRC | 13:09 | |
openstackgerrit | A change was merged to openstack/ceilometer: Remove redundant meter (name,type,unit) tuples from Resource model https://review.openstack.org/65671 | 13:15 |
*** sandywalsh has quit IRC | 13:18 | |
*** sandywalsh has joined #openstack-ceilometer | 13:32 | |
*** thomasem has joined #openstack-ceilometer | 13:42 | |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Convert alarm timestamp to PrecisionTimestamp https://review.openstack.org/63131 | 13:46 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: [don't review] oslo.db tests https://review.openstack.org/65677 | 13:46 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Set the SQL Float precision https://review.openstack.org/63132 | 13:46 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Run unit tests against MySQL https://review.openstack.org/59489 | 13:46 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Run tests against PostgreSQL https://review.openstack.org/63049 | 13:46 |
*** jdob has joined #openstack-ceilometer | 13:47 | |
sileht | jd__, can take a look to https://review.openstack.org/#/c/61814/ | 13:49 |
jd__ | ure | 13:51 |
jd__ | sure | 13:51 |
*** sayali has joined #openstack-ceilometer | 13:52 | |
*** sayali_ has joined #openstack-ceilometer | 13:52 | |
*** gordc has joined #openstack-ceilometer | 14:01 | |
jd__ | sileht: did you steal https://bugs.launchpad.net/oslo/+bug/1263908 btw? | 14:06 |
sileht | jd__, yes | 14:06 |
sileht | jd__, really stolen :) | 14:06 |
jd__ | cool thanks :) | 14:07 |
jd__ | sileht: what's the status of oslo.messaging migration? | 14:10 |
sileht | jd__, I'm waiting for markmc to continue | 14:11 |
jd__ | sileht: ok | 14:11 |
sileht | jd__, I have (re)ping him yesterday for reviewing | 14:11 |
jd__ | I changed the status to 'slow progress' | 14:11 |
*** jdob has quit IRC | 14:15 | |
openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Replace mongo aggregation with plain ol' map-reduce https://review.openstack.org/65962 | 14:16 |
*** jdob has joined #openstack-ceilometer | 14:16 | |
*** jdob has quit IRC | 14:19 | |
*** jdob has joined #openstack-ceilometer | 14:19 | |
*** eglynn is now known as eglynn-is-hypogl | 14:22 | |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: [don't review] oslo.db tests https://review.openstack.org/65677 | 14:23 |
*** eglynn-is-hypogl is now known as eglynn-lunch | 14:28 | |
thomasem | Hey everyone! | 14:34 |
nprivalova | jd__, Ilya has updated https://blueprints.launchpad.net/ceilometer/+spec/run-all-pollsters-on-demand | 14:34 |
*** prad has joined #openstack-ceilometer | 14: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 eglynn | 14:43 | |
eglynn | jd__: lolz :) | 14:43 |
nprivalova | dhellman and gordc, are you here? | 14:48 |
Alexei_987 | jd__: ping | 14:56 |
jd__ | Alexei_987: ? | 14:56 |
Alexei_987 | jd__: question about this mapreduce stuff | 14:56 |
Alexei_987 | jd__: it seems that we cannot store results if we provide a timestamp | 14:57 |
jd__ | a time range yeah | 14:57 |
Alexei_987 | jd__: but I think that we can store them if we call this method without a timestamp | 14:57 |
jd__ | which is likely 1% of cases? | 14:57 |
Alexei_987 | jd__: and it's called that way in controller get_one | 14:58 |
Alexei_987 | well I don't know actuall percentage | 14:58 |
Alexei_987 | but it would be 100 times faster than do a realtime mapreduce | 14:58 |
Alexei_987 | so I propose to implement 2 execution paths | 14:58 |
Alexei_987 | 1) without timestamp (fast) | 14:59 |
Alexei_987 | 2) with timestamp (mapreduce in realtime) | 14:59 |
jd__ | 100 times faster on a 1 % code path, I'm not that interested | 14:59 |
jd__ | but I admit that it makes sense for get_one | 14:59 |
jd__ | so if you want to optimize get_one, go for it | 14:59 |
jd__ | you may build a get_resource() method in the drivers (just an idea) | 14:59 |
Alexei_987 | yeah :) I'll see what I can do | 15: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_987 | well I think that this timestamp handling is quite crappy | 15:01 |
Alexei_987 | it has even mark on it # FIXME(dhellmann): This may not perform very well, | 15:01 |
nprivalova | eglynn, 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 investigations | 15:01 |
eglynn | Alexei_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 |
eglynn | nprivalova: thanks, will review this afternoon | 15:03 |
nprivalova | eglynn, thanks! | 15:07 |
*** coolsvap has joined #openstack-ceilometer | 15:09 | |
jd__ | nprivalova: I'll read and answer on the ml :) | 15:10 |
nprivalova | jd__, great! :) | 15:11 |
*** gordc has quit IRC | 15:15 | |
*** jmckind has joined #openstack-ceilometer | 15:41 | |
*** jmckind has quit IRC | 15:56 | |
*** herndon_ has joined #openstack-ceilometer | 15:56 | |
*** herndon_ has quit IRC | 16:01 | |
*** gordc has joined #openstack-ceilometer | 16:03 | |
gordc | nprivalova: sorry for delayed response. did you still need anything from me? | 16:17 |
*** SergeyLukjanov has quit IRC | 16:30 | |
*** ozialien has joined #openstack-ceilometer | 16:53 | |
*** ruhe is now known as _ruhe | 16:59 | |
*** _ruhe is now known as ruhe | 17:07 | |
*** Alexei_987 has quit IRC | 17:17 | |
*** SergeyLukjanov has joined #openstack-ceilometer | 17:19 | |
*** eglynn has quit IRC | 17:39 | |
*** ruhe is now known as _ruhe | 17:43 | |
*** nadya_ has joined #openstack-ceilometer | 18:32 | |
*** SergeyLukjanov has quit IRC | 18:35 | |
*** SergeyLukjanov has joined #openstack-ceilometer | 18:38 | |
*** nadya_ has quit IRC | 18:45 | |
openstackgerrit | John Herndon proposed a change to openstack/python-ceilometerclient: Support the Event API https://review.openstack.org/54582 | 19:00 |
*** herndon_ has joined #openstack-ceilometer | 19:01 | |
*** herndon_ has quit IRC | 19:02 | |
*** herndon_ has joined #openstack-ceilometer | 19:11 | |
*** _ruhe is now known as ruhe | 19:18 | |
*** sayali has quit IRC | 19:25 | |
*** herndon_ has quit IRC | 19:28 | |
thomasem | gordc, ping | 19:29 |
gordc | thomasem: whatup | 19:31 |
thomasem | gordc, 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.py | 19:32 |
gordc | ok, taking a look at it now. | 19:33 |
thomasem | gordc, 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 |
thomasem | So, 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 |
thomasem | But, it sounds like pep8 doesn't like doing that either, so I'm not sure of the best approach there. Any ideas? | 19:35 |
gordc | hmm... 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 |
gordc | i 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 |
thomasem | gordc, Yeah, I'd like to understand it better, might bug some resident python guru | 19:37 |
gordc | it just looked strange to me so i thought i'd ask in case it was left in unintentionally. | 19:37 |
thomasem | gordc, It just might get confusing, is all. | 19:37 |
thomasem | having inconsistent references | 19:38 |
thomasem | Nah, I'm glad you did. | 19:38 |
thomasem | :P | 19:38 |
gordc | thomasem: yeah, the confusing part was why i brought it up... no idea what right solutions is. | 19:38 |
thomasem | coolio, well I'll try to get more info anyway, but it does work for now. | 19:38 |
thomasem | We could also just move all of the references over to models.* instead of storage.models.* :P | 19:39 |
thomasem | for consistency | 19:40 |
gordc | yeah... 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.lol | 19:40 |
*** ruhe is now known as _ruhe | 19:51 | |
*** herndon_ has joined #openstack-ceilometer | 20:15 | |
*** prad has quit IRC | 20:32 | |
*** flwang has quit IRC | 20:45 | |
*** jmckind has joined #openstack-ceilometer | 20:45 | |
*** sandywalsh has quit IRC | 20:55 | |
*** yassine has quit IRC | 20:57 | |
*** yassine has joined #openstack-ceilometer | 20:57 | |
*** yassine has quit IRC | 20:58 | |
*** yassine has joined #openstack-ceilometer | 20:58 | |
*** yassine has quit IRC | 20:59 | |
*** flwang has joined #openstack-ceilometer | 21:00 | |
*** prad has joined #openstack-ceilometer | 21:03 | |
*** sandywalsh has joined #openstack-ceilometer | 21:06 | |
*** SergeyLukjanov has quit IRC | 21:29 | |
*** jdob has quit IRC | 22:00 | |
*** jmckind has quit IRC | 22:23 | |
*** ozialien has quit IRC | 22:34 | |
*** thomasem has quit IRC | 22:43 | |
*** prad has quit IRC | 22:49 | |
*** prad has joined #openstack-ceilometer | 22:53 | |
*** herndon_ has quit IRC | 23:22 | |
*** ozialien has joined #openstack-ceilometer | 23:31 | |
*** kwhitney1 has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!