| *** promulo__ has joined #openstack-ceilometer | 00:21 | |
| *** promulo has quit IRC | 00:25 | |
| *** llu has joined #openstack-ceilometer | 00:52 | |
| *** mstyne has quit IRC | 01:07 | |
| *** ddieterly has quit IRC | 01:28 | |
| *** tobasco_ has joined #openstack-ceilometer | 01:36 | |
| *** tobasco has quit IRC | 01:37 | |
| *** idegtiarov_ has joined #openstack-ceilometer | 01:37 | |
| *** idegtiarov has quit IRC | 01:38 | |
| flwang1 | sileht: ping | 01:41 |
|---|---|---|
| *** liusheng has joined #openstack-ceilometer | 01:41 | |
| *** ankita_wagh has joined #openstack-ceilometer | 01:45 | |
| *** flwang1 has quit IRC | 02:18 | |
| *** flwang1 has joined #openstack-ceilometer | 02:19 | |
| *** mstyne has joined #openstack-ceilometer | 02:21 | |
| *** ddieterly has joined #openstack-ceilometer | 02:29 | |
| *** ankita_wagh has quit IRC | 02:33 | |
| *** ddieterly has quit IRC | 02:33 | |
| openstackgerrit | gordon chung proposed openstack/ceilometer: WIP restrict admin event access https://review.openstack.org/218706 | 02:38 |
| *** ddieterly has joined #openstack-ceilometer | 02:56 | |
| openstackgerrit | Lianhao Lu proposed openstack/ceilometer: Added snmp declarative hardware pollster https://review.openstack.org/217667 | 03:09 |
| *** jroll has quit IRC | 03:26 | |
| *** liusheng has quit IRC | 03:27 | |
| *** liusheng has joined #openstack-ceilometer | 03:28 | |
| *** jroll has joined #openstack-ceilometer | 03:31 | |
| *** ankita_wagh has joined #openstack-ceilometer | 03:34 | |
| *** I has joined #openstack-ceilometer | 03:35 | |
| *** I is now known as Guest25059 | 03:36 | |
| *** rbak has quit IRC | 03:36 | |
| *** links has joined #openstack-ceilometer | 03:40 | |
| *** jwcroppe_ has joined #openstack-ceilometer | 03:45 | |
| *** jwcroppe has quit IRC | 03:47 | |
| *** Guest25059 has quit IRC | 03:49 | |
| *** haigang has joined #openstack-ceilometer | 03:49 | |
| *** deepthi has joined #openstack-ceilometer | 03:56 | |
| *** ddieterly has quit IRC | 04:06 | |
| *** sonuk has joined #openstack-ceilometer | 04:40 | |
| *** yprokule has joined #openstack-ceilometer | 04:57 | |
| *** ig0r__ has joined #openstack-ceilometer | 04:59 | |
| *** ig0r_ has quit IRC | 04:59 | |
| *** ddieterly has joined #openstack-ceilometer | 05:06 | |
| *** ddieterly has quit IRC | 05:11 | |
| *** ankita_wagh has quit IRC | 05:23 | |
| *** haigang has quit IRC | 05:23 | |
| *** ankita_wagh has joined #openstack-ceilometer | 05:36 | |
| *** gzhai has joined #openstack-ceilometer | 05:37 | |
| gzhai | help | 05:41 |
| *** eglynn__ has joined #openstack-ceilometer | 05:41 | |
| *** haigang has joined #openstack-ceilometer | 05:47 | |
| *** ankita_wagh has quit IRC | 05:50 | |
| *** ddieterly has joined #openstack-ceilometer | 06:07 | |
| *** ddieterly has quit IRC | 06:12 | |
| *** eglynn__ has quit IRC | 06:18 | |
| *** afazekas_ has joined #openstack-ceilometer | 06:19 | |
| *** eglynn__ has joined #openstack-ceilometer | 06:20 | |
| *** eglynn__ has quit IRC | 06:25 | |
| *** haigang has quit IRC | 06:27 | |
| *** haigang has joined #openstack-ceilometer | 06:28 | |
| *** haigang has quit IRC | 06:29 | |
| openstackgerrit | Zhai, Edwin proposed openstack/aodh: Enable event alarm by default https://review.openstack.org/218746 | 06:35 |
| *** haigang has joined #openstack-ceilometer | 06:36 | |
| *** nadya has joined #openstack-ceilometer | 06:37 | |
| *** eglynn__ has joined #openstack-ceilometer | 07:02 | |
| *** nadya has quit IRC | 07:05 | |
| *** ddieterly has joined #openstack-ceilometer | 07:08 | |
| *** eglynn__ has quit IRC | 07:09 | |
| *** ddieterly has quit IRC | 07:13 | |
| *** Ala has joined #openstack-ceilometer | 07:33 | |
| *** yassine__ has joined #openstack-ceilometer | 07:40 | |
| *** safchain has joined #openstack-ceilometer | 07:48 | |
| *** ddieterly has joined #openstack-ceilometer | 08:09 | |
| *** ddieterly has quit IRC | 08:14 | |
| *** eglynn__ has joined #openstack-ceilometer | 08:33 | |
| *** maestro_ has joined #openstack-ceilometer | 08:41 | |
| maestro_ | /HELP : I would like to integrate ceilometer alarm with heat for autoscaling, can any one please help me out | 08:42 |
| *** haigang has quit IRC | 08:44 | |
| *** haigang has joined #openstack-ceilometer | 08:44 | |
| *** haigang has quit IRC | 08:50 | |
| *** haigang has joined #openstack-ceilometer | 08:52 | |
| *** ildikov has joined #openstack-ceilometer | 09:01 | |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: devstack: remove ceilometer from service list https://review.openstack.org/218299 | 09:01 |
| *** ddieterly has joined #openstack-ceilometer | 09:10 | |
| *** ddieterly has quit IRC | 09:14 | |
| *** haigang has quit IRC | 09:20 | |
| *** r-mibu has joined #openstack-ceilometer | 09:26 | |
| r-mibu | gzhai, sorry i'm in | 09:27 |
| gzhai | r-mibu: hi | 09:28 |
| gzhai | r-mibu: how do you create event alarm. I tried REST API, but the project_id is always None. | 09:28 |
| r-mibu | gzhai, I'm still working on cli. | 09:29 |
| gzhai | r-mibu: it will only find the alarms with same project_id as in the notification, so find nothing:( | 09:30 |
| r-mibu | gzhai, you mean that event-alarm-evaluator cannot find project id from event? | 09:30 |
| r-mibu | or talking about client facing API to create alarm entry? | 09:31 |
| gzhai | it can find the right project_id, but alarm created by REST has no project_id | 09:31 |
| gzhai | so it find no alarm. | 09:32 |
| r-mibu | ok. so it should be bug on api side | 09:32 |
| r-mibu | did you tried demo user? | 09:33 |
| gzhai | cli is okay to create threshold alarm with project_id, but I saw the same REST payload. So just curious. | 09:33 |
| *** haigang has joined #openstack-ceilometer | 09:33 | |
| gzhai | Not yet for demo. I'm using admin. Will try it. | 09:34 |
| r-mibu | thanks for testing! | 09:34 |
| gzhai | It's not event alarm code bug:) | 09:36 |
| gzhai | But if the project_id of event is valid, and project_id of event-alarm is None. Do you think we need fire it? | 09:36 |
| *** ig0r_ has joined #openstack-ceilometer | 09:37 | |
| r-mibu | Maybe yes, admin may want to know events from his infra to capture failure. | 09:38 |
| *** ig0r__ has quit IRC | 09:39 | |
| r-mibu | But, it's hard to support right now... | 09:39 |
| r-mibu | Currently, admin can create event-alarm on be half of user, but those alarms is visible to user. | 09:41 |
| gzhai | ok. BTW, demo user has None project_id too via REST API. | 09:42 |
| r-mibu | ok, will check it | 09:43 |
| gzhai | I even see nothing with "ceilometer alarm-list" although the REST API return right response. | 09:47 |
| gzhai | how did you create event-alarm? also REST api? | 09:48 |
| r-mibu | you mean there is no type=event alarms? | 09:48 |
| r-mibu | I created them without cli | 09:48 |
| r-mibu | sorry, without cli shell, but I used cli python module with some hacking | 09:49 |
| gzhai | curl -g -i -X 'POST' 'http://10.239.159.54:8042/v2/alarms' -H 'User-Agent: ceilometerclient.openstack.common.apiclient' -H 'Content-Type: application/json' -H "X-Auth-Token: $1" -d '{"event_rule": {"event_type": "compute.instance.update" }, "alarm_actions": ["log://"], "type": "event", "name": "edwin_event_alarm", "repeat_actions": false}' | 09:50 |
| gzhai | My REST script. | 09:50 |
| gzhai | anyway, it's bug out of event-alarm code. | 09:52 |
| r-mibu | your input seems ok | 09:53 |
| gzhai | end user want to see cli first. | 09:53 |
| r-mibu | yep | 09:53 |
| r-mibu | will do shortly | 09:53 |
| gzhai | r-mibu: Thank you! | 09:53 |
| r-mibu | back to project_id=None events, I may create new option like 'evaluate_all_admin_alrams' to evaluate all admin alarms on each event | 09:53 |
| r-mibu | gzhai, thank you, too :) | 09:54 |
| *** yassine__ has quit IRC | 10:00 | |
| *** ddieterly has joined #openstack-ceilometer | 10:10 | |
| *** haigang has quit IRC | 10:14 | |
| *** haigang has joined #openstack-ceilometer | 10:15 | |
| *** ddieterly has quit IRC | 10:15 | |
| *** gzhai has quit IRC | 10:33 | |
| *** idegtiarov has joined #openstack-ceilometer | 10:48 | |
| *** yassine__ has joined #openstack-ceilometer | 11:04 | |
| *** yassine__ has quit IRC | 11:04 | |
| *** ddieterly has joined #openstack-ceilometer | 11:11 | |
| *** ddieterly has quit IRC | 11:16 | |
| *** nadya has joined #openstack-ceilometer | 11:17 | |
| *** yassine__ has joined #openstack-ceilometer | 11:20 | |
| *** deepthi has quit IRC | 11:22 | |
| *** yprokule has quit IRC | 11:24 | |
| *** yprokule has joined #openstack-ceilometer | 11:24 | |
| openstackgerrit | Merged openstack/ceilometer: Removing unused dependency: discover https://review.openstack.org/218294 | 11:25 |
| *** gordc has joined #openstack-ceilometer | 11:43 | |
| *** openstackgerrit has quit IRC | 11:46 | |
| *** openstackgerrit has joined #openstack-ceilometer | 11:47 | |
| openstackgerrit | Julien Danjou proposed openstack/ceilometer: Adding liusheng to MAINTAINERS https://review.openstack.org/218819 | 11:57 |
| nadya | maestro_: please contact idegtiarov | 11:59 |
| openstackgerrit | Julien Danjou proposed openstack/ceilometer: Adding pradk to MAINTAINERS https://review.openstack.org/218822 | 12:00 |
| openstackgerrit | Julien Danjou proposed openstack/aodh: Adding liusheng to MAINTAINERS https://review.openstack.org/218823 | 12:00 |
| *** nadya has quit IRC | 12:01 | |
| *** ddieterly has joined #openstack-ceilometer | 12:12 | |
| sileht | jd__, gordc I have finished to prepare https://github.com/sileht/python-gnocchiclient | 12:16 |
| sileht | it have crud and search of resources | 12:16 |
| sileht | tests and docs | 12:16 |
| jd__ | coolio | 12:16 |
| sileht | Do I import it in openstackci, now ? | 12:16 |
| *** ddieterly has quit IRC | 12:16 | |
| sileht | I also prepared launchpad/project-config/gouvernance changes | 12:17 |
| sileht | overview of the doc if needed: https://python-gnocchiclient.readthedocs.org/en/latest/ | 12:17 |
| jd__ | sileht: I think so | 12:18 |
| jd__ | sileht: typo in https://python-gnocchiclient.readthedocs.org/en/latest/shell.html "gnocch" instead of "gnocchi" | 12:18 |
| sileht | fixed | 12:19 |
| gordc | sileht: nice! | 12:20 |
| gordc | how much of this overlaps with dispatcher code? should we try to remove it for Liberty? | 12:20 |
| sileht | gordc, the gnocchi_client.py in ceilometer should be replaced by this | 12:21 |
| sileht | gordc, but we need to finish the lib first :p | 12:21 |
| gordc | :) | 12:21 |
| gordc | we can target it as a bug... so we don't need to rush. | 12:21 |
| sileht | I open it | 12:22 |
| gordc | thanks | 12:22 |
| sileht | same thing for aodh evaluator | 12:22 |
| jd__ | cooooliioooo | 12:22 |
| sileht | both bugs opened | 12:24 |
| gordc | cool cool. | 12:26 |
| *** edmondsw has joined #openstack-ceilometer | 12:32 | |
| *** dikonoor has joined #openstack-ceilometer | 12:34 | |
| *** dikonoo has joined #openstack-ceilometer | 12:35 | |
| *** llu has quit IRC | 12:41 | |
| *** llu has joined #openstack-ceilometer | 12:41 | |
| *** ddieterly has joined #openstack-ceilometer | 12:46 | |
| *** dikonoo has quit IRC | 12:50 | |
| dikonoor | hi gordc: | 12:53 |
| sileht | jd__, if you have an idea of how to fix that cleanly: https://bugs.launchpad.net/gnocchi/+bug/1490553 | 12:54 |
| openstack | Launchpad bug 1490553 in Gnocchi "search with invalid uuid raise 500" [Undecided,New] | 12:54 |
| gordc | dikonoor: hi | 12:55 |
| dikonoor | gordc:GM..4 things 1) Do you know of some core who you think should be taking a look at https://review.openstack.org/#/c/211509/ 2)Did you get a chance to take a look at the support for passing (A or B) into the events filter query ? | 12:55 |
| dikonoor | 3)could you take a look at https://review.openstack.org/#/c/217844/ when you get a chance . If this sounds ok, I can proceed with writing UTs. | 12:55 |
| dikonoor | gordc: 4) When is the code freeze date? | 12:56 |
| *** cmyster has left #openstack-ceilometer | 12:56 | |
| *** afazekas__ has joined #openstack-ceilometer | 12:56 | |
| gordc | dikonoor: 1) i'll ping someone. | 12:56 |
| maestro_ | how to see ceilometer alarm triggering ? | 12:56 |
| gordc | dikonoor: 2) i've started looking at it... hbase is going to be the hardest to patch | 12:57 |
| jd__ | sileht: voluptuous should convert the string to a valid uuid object | 12:57 |
| jd__ | sileht: if you have a query to reproduce it as a test in gabbi it should be easy to fix | 12:57 |
| gordc | dikonoor: 3) i'm ok with this implementation | 12:57 |
| gordc | dikonoor: 4) code freeze is this week. it starts tomorrow but we can push a few days. | 12:58 |
| *** afazekas_ has quit IRC | 12:58 | |
| *** yprokule has quit IRC | 12:59 | |
| sileht | jd__, lets me use --debug on the new client ;p | 12:59 |
| dikonoor | gordc: ok..thanks for your quick answers.. I will try and get 3) in today and then keep the last patch for the rbac BP ready so that when your patch is in, I can submit it | 12:59 |
| sileht | jd__, REQ: curl -g -i -X POST http://192.168.3.51:8041/v1/search/resource/generic? -H "User-Agent: python-keystoneclient" -H "Content-Type: application/json" -H "X-Auth-Token: {SHA1}3621ff659ededa91ad8ecf2b6926a2aab35d2558" -d '{"=": {"id": "cd9eeff"}}' | 12:59 |
| gordc | dikonoor: what's the last patch? | 13:00 |
| jd__ | sileht: ok I'll fix that | 13:00 |
| sileht | jd__, cool thx | 13:00 |
| *** yprokule has joined #openstack-ceilometer | 13:00 | |
| dikonoor | gordc: last set of changes are for the admin users..admin users should be able to see all events for their project + events with no/empty/null project_id . This is where it requires your changes | 13:01 |
| dikonoor | gordc: currently, admin users can see all events , including the events from other projects | 13:01 |
| gordc | dikonoor: is this a change to policy file? | 13:01 |
| gordc | jd__: shall i write an email to point to core nominations? | 13:02 |
| dikonoor | gordc: no ..this is a change in the event.py code..and will come in the else condition on top of these changes >> https://review.openstack.org/#/c/217844/1/ceilometer/api/controllers/v2/events.py | 13:02 |
| jd__ | gordc: "point"? | 13:02 |
| jd__ | gordc: you can add ceilometer-core to the review and aodh-core to the review | 13:02 |
| jd__ | gordc: that ought to be enough, we don't want to spam openstack-dev and then get yelled at | 13:03 |
| gordc | jd__: just to say the nominations exists. we did it last time (the message was don't reply to this, but vote on gerrit) | 13:03 |
| gordc | jd__: we always get yelled at anyways | 13:04 |
| *** edmondsw has quit IRC | 13:04 | |
| jd__ | gordc: ok, well go ahead then :) | 13:05 |
| *** ddieterly has quit IRC | 13:09 | |
| sileht | jd__, keystonemiddleware is in test-requirements file but import it in a couple of place | 13:10 |
| sileht | jd__, forget when next version will be released we can drop everything | 13:11 |
| dikonoor | gordc: does my prev msg answer your query? | 13:12 |
| openstackgerrit | Ryota MIBU proposed openstack/python-ceilometerclient: Support alarm-event-{create,update} to cli https://review.openstack.org/218846 | 13:15 |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/gnocchi: move keystonemiddleware to requirements.txt https://review.openstack.org/218847 | 13:15 |
| jd__ | sileht: for gnocchiclient? | 13:16 |
| jd__ | ah got it | 13:16 |
| jd__ | yep makes sense | 13:16 |
| sileht | jd__, api just don't start if keystonemiddleware is not installed | 13:17 |
| sileht | jd__, python-gnocchiclient bootstrap a gnocchi-api without keystone into the virtualenv for test the client against a reall API and fall into that bug | 13:18 |
| sileht | for test/to test | 13:18 |
| jd__ | ahah awesome | 13:19 |
| *** kbyrne has quit IRC | 13:19 | |
| jd__ | I like how you test gnocchiclient | 13:19 |
| sileht | I hate writting mock | 13:20 |
| gordc | dikonoor: sort of? we can merge that patch without my admin patch | 13:20 |
| gordc | it would just allow admin to see everything (which is existing behaviuor) | 13:21 |
| dikonoor | gordc: https://review.openstack.org/#/c/217844/ patch is not dependent on your changes and we can merge | 13:22 |
| gordc | right. | 13:22 |
| dikonoor | gordc: After your changes, I will need to submit another patch that uses your changes to filter events for admins such that admins can see only events from their project + events that have no project_id | 13:23 |
| gordc | dikonoor: my patch would include that | 13:24 |
| dikonoor | ah..ok..then i wouldn't need to make any changes | 13:24 |
| gordc | dikonoor: hopefully not. | 13:25 |
| dikonoor | gordc: ok.. | 13:25 |
| gordc | dikonoor: i think if you add test cases for your non-admin patch we should be good to go | 13:25 |
| dikonoor | gordc: and I guess these will go into test_events_scenarios.py :).. some testcases so that the events returned are specific to that user | 13:26 |
| gordc | dikonoor: that works for me. | 13:27 |
| dikonoor | gordc: ok will get that in today | 13:27 |
| *** liusheng has quit IRC | 13:28 | |
| *** links has quit IRC | 13:30 | |
| *** maestro_ has quit IRC | 13:30 | |
| *** kbyrne has joined #openstack-ceilometer | 13:32 | |
| *** thorst1 has joined #openstack-ceilometer | 13:32 | |
| *** edmondsw has joined #openstack-ceilometer | 13:33 | |
| *** safchain has quit IRC | 13:35 | |
| *** liusheng_ has joined #openstack-ceilometer | 13:36 | |
| *** yprokule has quit IRC | 13:36 | |
| *** yprokule has joined #openstack-ceilometer | 13:38 | |
| *** annasort has joined #openstack-ceilometer | 13:39 | |
| *** ddieterly has joined #openstack-ceilometer | 13:43 | |
| openstackgerrit | Merged openstack/aodh: Use new location of subunit2html https://review.openstack.org/218161 | 13:47 |
| *** pradk has joined #openstack-ceilometer | 13:54 | |
| *** Kennan has quit IRC | 14:03 | |
| *** Kennan2 has joined #openstack-ceilometer | 14:03 | |
| *** safchain has joined #openstack-ceilometer | 14:05 | |
| *** afazekas__ has quit IRC | 14:08 | |
| *** rbak has joined #openstack-ceilometer | 14:09 | |
| *** jroll has quit IRC | 14:09 | |
| *** jroll has joined #openstack-ceilometer | 14:09 | |
| *** jasonamyers has joined #openstack-ceilometer | 14:09 | |
| *** Ephur has joined #openstack-ceilometer | 14:09 | |
| *** boris-42 has joined #openstack-ceilometer | 14:11 | |
| jasonamyers | woot pradk | 14:11 |
| pradk | thx :) | 14:13 |
| *** ildikov has quit IRC | 14:20 | |
| *** r-daneel has joined #openstack-ceilometer | 14:22 | |
| openstackgerrit | gordon chung proposed openstack/ceilometer: deprecate db2 nosql driver https://review.openstack.org/218880 | 14:32 |
| *** ildikov has joined #openstack-ceilometer | 14:36 | |
| openstackgerrit | gordon chung proposed openstack/ceilometer: deprecate db2 nosql driver https://review.openstack.org/218880 | 14:38 |
| *** jaosorior has joined #openstack-ceilometer | 14:44 | |
| openstackgerrit | gordon chung proposed openstack/aodh: remove db2 nosql driver https://review.openstack.org/218886 | 14:45 |
| *** liusheng_ has quit IRC | 14:50 | |
| *** Ala has quit IRC | 14:53 | |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: convert UUID correctly before doing a search https://review.openstack.org/218891 | 14:56 |
| *** jruano has joined #openstack-ceilometer | 15:02 | |
| openstackgerrit | gordon chung proposed openstack/ceilometer: deprecate db2 nosql driver https://review.openstack.org/218880 | 15:07 |
| openstackgerrit | gordon chung proposed openstack/ceilometer: deprecate db2 nosql driver https://review.openstack.org/218880 | 15:15 |
| openstackgerrit | Ryota MIBU proposed openstack/python-ceilometerclient: Support alarm-event-{create,update} https://review.openstack.org/218846 | 15:16 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ceilometer: Updated from global requirements https://review.openstack.org/216964 | 15:30 |
| *** david-ly_ has joined #openstack-ceilometer | 15:31 | |
| *** david-ly_ is now known as david-lyle_ | 15:33 | |
| *** david-lyle has quit IRC | 15:33 | |
| *** david-lyle_ is now known as david-lyle | 15:35 | |
| openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: catch invalid datetime conversion in search https://review.openstack.org/218926 | 15:49 |
| *** jasonamyers has quit IRC | 15:51 | |
| *** maubry has joined #openstack-ceilometer | 15:53 | |
| *** yassine__ has quit IRC | 15:54 | |
| *** r-mibu has left #openstack-ceilometer | 15:57 | |
| openstackgerrit | Julien Danjou proposed openstack/ceilometer: polling: remove deprecated agents https://review.openstack.org/212498 | 16:13 |
| *** vishwanathj has joined #openstack-ceilometer | 16:18 | |
| *** safchain has quit IRC | 16:19 | |
| openstackgerrit | Julien Danjou proposed openstack/ceilometer: polling: remove deprecated agents https://review.openstack.org/212498 | 16:20 |
| *** ankita_wagh has joined #openstack-ceilometer | 16:26 | |
| *** maubry has quit IRC | 16:39 | |
| *** rjaiswal has joined #openstack-ceilometer | 16:41 | |
| *** maubry has joined #openstack-ceilometer | 16:44 | |
| *** ankita_wagh has quit IRC | 16:57 | |
| *** vishwanathj has quit IRC | 17:00 | |
| *** exploreshaifali has joined #openstack-ceilometer | 17:04 | |
| *** julim has joined #openstack-ceilometer | 17:11 | |
| ildikov | pradk: sorry for all those comments on the docs | 17:15 |
| ildikov | pradk: if it does not get merged in the early phase then it can be much more painful as a code patch... :( | 17:16 |
| pradk | ildikov, np, thx for the comments, i'll revisit the docs this afternoon | 17:16 |
| ildikov | pradk: so if that helps, feel free to blame me for the latest ones, even if I wasn't the one who added them :) | 17:17 |
| pradk | ildikov, understand | 17:17 |
| pradk | hehe | 17:17 |
| ildikov | pradk: I wrote the original content in the Admin Guide, it felt like suffering for all my sins in all my lives :) | 17:18 |
| ildikov | pradk: ok, it wasn't that bad, but I know it can be difficult to get it done | 17:18 |
| pradk | ildikov, ah i think it wasnt too bad.. gave us a good place to detail the functionality | 17:19 |
| ildikov | pradk: having a chapter in that guide is really good | 17:20 |
| pradk | yea agreed | 17:20 |
| ildikov | and I'malso glad that more and more people go and add docco about their changes there also and not just to the in-tree docs | 17:21 |
| pradk | ildikov, yea we all should. It makes consumption much easier | 17:22 |
| ildikov | pradk: yeap, agreed | 17:23 |
| ildikov | pradk: so thanks for showing a good example :) | 17:24 |
| pradk | sure, thank you for the reviews as well | 17:24 |
| *** yprokule has quit IRC | 17:25 | |
| ildikov | pradk: np :) | 17:25 |
| ildikov | pradk: ok, it's time for me to leave the office :) | 17:25 |
| ildikov | pradk: laters :) | 17:25 |
| pradk | sure, laters! | 17:25 |
| *** ildikov has quit IRC | 17:31 | |
| *** ankita_wagh has joined #openstack-ceilometer | 17:33 | |
| *** jaosorior has quit IRC | 17:35 | |
| *** ig0r__ has joined #openstack-ceilometer | 17:43 | |
| *** ig0r_ has quit IRC | 17:46 | |
| *** llu has quit IRC | 17:55 | |
| *** llu has joined #openstack-ceilometer | 17:55 | |
| *** julim has quit IRC | 18:18 | |
| *** ildikov has joined #openstack-ceilometer | 18:20 | |
| *** julim has joined #openstack-ceilometer | 18:21 | |
| *** exploreshaifali has quit IRC | 18:28 | |
| *** ankita_w_ has joined #openstack-ceilometer | 18:33 | |
| *** ankita_wagh has quit IRC | 18:37 | |
| *** ig0r__ has quit IRC | 18:48 | |
| openstackgerrit | Divya K Konoor proposed openstack/ceilometer: Event filtering for non-admin users https://review.openstack.org/217844 | 18:54 |
| *** ankita_w_ has quit IRC | 18:54 | |
| *** ankita_wagh has joined #openstack-ceilometer | 18:54 | |
| openstackgerrit | Rohit Jaiswal proposed openstack/ceilometer: Requeuing event with workload_partitioning on publish failure https://review.openstack.org/217980 | 18:54 |
| *** edmondsw has quit IRC | 18:56 | |
| openstackgerrit | Merged openstack/ceilometer: Incorrect Links are updated https://review.openstack.org/217335 | 18:57 |
| *** edmondsw has joined #openstack-ceilometer | 18:57 | |
| dikonoor | gordc: hi gordc. I am done with https://review.openstack.org/#/c/217844/. Please take a look when you get a chance | 18:58 |
| dikonoor | gordc:Thanks | 18:58 |
| *** dikonoor has quit IRC | 19:00 | |
| *** maubry_ has joined #openstack-ceilometer | 19:00 | |
| *** maubry has quit IRC | 19:02 | |
| *** maubry_ is now known as maubry | 19:02 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ceilometer: Updated from global requirements https://review.openstack.org/216964 | 19:02 |
| *** jwcroppe has joined #openstack-ceilometer | 19:13 | |
| *** jwcroppe_ has quit IRC | 19:15 | |
| *** ankita_wagh has quit IRC | 19:24 | |
| *** promulo__ has quit IRC | 19:32 | |
| *** exploreshaifali has joined #openstack-ceilometer | 19:33 | |
| *** thorst1 has quit IRC | 19:47 | |
| openstackgerrit | Julien Danjou proposed openstack/ceilometer: polling: remove deprecated agents https://review.openstack.org/212498 | 19:53 |
| *** ankita_wagh has joined #openstack-ceilometer | 19:55 | |
| *** david-lyle has quit IRC | 20:01 | |
| *** david-lyle has joined #openstack-ceilometer | 20:06 | |
| *** thorst1 has joined #openstack-ceilometer | 20:14 | |
| *** rjaiswal has quit IRC | 20:20 | |
| *** idegtiarov has quit IRC | 20:24 | |
| *** _cjones_ has joined #openstack-ceilometer | 20:41 | |
| *** _cjones_ has quit IRC | 20:42 | |
| *** Samolo has joined #openstack-ceilometer | 20:49 | |
| Samolo | Hi all, I noticed that on Kilo, when doing a ceilometer sample-list for a particular project ID end for the meter instance, I also get some event mixed with sample | 20:50 |
| Samolo | like instance.create, instance,exist etc.... | 20:50 |
| Samolo | Do you know, please, how to remove the event from the sample list please ? | 20:51 |
| gordc | Samolo: do you have an example output? | 20:51 |
| Samolo | gordc: Hi, of course, give me a second please | 20:51 |
| *** ildikov has quit IRC | 20:54 | |
| Samolo | gordc: When doing a sample-list with --debug, I see the returner json, here is two entry : http://pastebin.com/R1gatwYa | 20:55 |
| Samolo | the first one is a sample, the second one is an event ( as you can see in the resource_metadata.event_type | 20:55 |
| *** jwcroppe_ has joined #openstack-ceilometer | 20:55 | |
| Samolo | and both are returned by sample-list | 20:56 |
| gordc | Samolo: so they're actually both samples, it's just that the first sample is derived from polling, and the second is derived from a notification | 20:56 |
| *** jwcroppe has quit IRC | 20:57 | |
| Samolo | gordc: oh ok, there is a way to know from witch service the notification come from ? to disable it | 20:57 |
| gordc | an Event has a different schema | 20:57 |
| Samolo | it must be nova | 20:57 |
| gordc | Samolo: if it's an instance notification, it's coming from Nova | 20:57 |
| gordc | you can either disable nova from emitting events OR the better way is to modify the pipeline.yaml your notification agent is using to stop generating instance meters | 20:58 |
| gordc | you'll need to make sure the pipeline aren't shared between polling/notification agents... (they typically are) | 20:59 |
| Samolo | gordc: the only flag I see in nova is instance_usage_audit = False | 21:00 |
| Samolo | gordc: this mean that the notifier is disable or am I wrong please ? | 21:00 |
| Samolo | notifiaction* | 21:00 |
| flwang1 | dhellmann: ping | 21:00 |
| *** jwcroppe has joined #openstack-ceilometer | 21:02 | |
| gordc | Samolo: you may need to disable notify_on_state_change as well | 21:04 |
| *** jwcroppe_ has quit IRC | 21:04 | |
| Samolo | gordc: omg you seem rigth, we have notify_on_state_change = vm_and_task_state | 21:04 |
| Samolo | on each compute | 21:04 |
| gordc | Samolo: there are a lot of options... | 21:05 |
| Samolo | gordc: oh yeah, by the way, thanks for last time (https://bugs.launchpad.net/ceilometer/+bug/1489132) | 21:05 |
| openstack | Launchpad bug 1489132 in Ceilometer kilo "instance_type metadata is different between notifications and polling" [Undecided,In progress] - Assigned to Liusheng (liusheng) | 21:05 |
| gordc | Samolo: not a problem. we're working on backporting that as well to kilo | 21:06 |
| Samolo | gordc: thanks, I will be able to use ceilometer for billing maybe one day ! | 21:06 |
| gordc | here's hoping :) but yeah, feel free to keep opening bugs as you find them. | 21:07 |
| *** jwcroppe_ has joined #openstack-ceilometer | 21:07 | |
| Samolo | gordc: thanks, little shy for now, but I will make an effort | 21:07 |
| Samolo | ;) | 21:08 |
| gordc | cool cool | 21:08 |
| *** jwcroppe has quit IRC | 21:10 | |
| *** julim has quit IRC | 21:14 | |
| Samolo | gordc: one last thing about "you'll need to make sure the pipeline aren't shared between polling/notification agents..." | 21:22 |
| Samolo | gordc: how can I check that please ? | 21:22 |
| gordc | Samolo: if you're passing the same ceilometer.conf file into your polling services and notification services, then they are pointing to same pipeline.yaml file | 21:23 |
| Samolo | gordc: I haven't setup everything there, I want to check if the polling and notification service share the same pipeline.yaml, have you any hint ? | 21:26 |
| gordc | ^^ that's it. when you start your service (polling/notification) you have to provide a ceilometer.conf file | 21:27 |
| Samolo | gordc: oh thanks, wil lcheck that | 21:28 |
| gordc | the actual option is https://github.com/openstack/ceilometer/blob/master/ceilometer/pipeline.py#L41 | 21:28 |
| *** jwcroppe has joined #openstack-ceilometer | 21:28 | |
| *** jwcroppe_ has quit IRC | 21:31 | |
| Samolo | gordc: thanks again | 21:31 |
| gordc | np | 21:31 |
| openstackgerrit | ZhiQiang Fan proposed openstack/ceilometer: Prevent ceilometer expirer from causing deadlocks https://review.openstack.org/165566 | 21:32 |
| *** ddieterly has quit IRC | 21:36 | |
| *** maubry has quit IRC | 21:56 | |
| flwang1 | gordc: ping | 22:04 |
| gordc | flwang1: whatsup? | 22:05 |
| flwang1 | gordc: we're running into a migration problem, not sure if it's known issue | 22:06 |
| flwang1 | we're using icehouse and going to upgrade to juno | 22:06 |
| flwang1 | and we're using mod_wsgi for api | 22:06 |
| flwang1 | the mod_wsgi is from the ceilometer doc | 22:06 |
| flwang1 | however, after migrate to juno | 22:06 |
| flwang1 | we found the sample-create command always return http 500 error | 22:07 |
| flwang1 | and there is no log in ceilometer api log | 22:07 |
| flwang1 | any ideas? | 22:07 |
| gordc | flwang1: did your service even start? | 22:08 |
| flwang1 | yes | 22:08 |
| flwang1 | and the weird thing is, if we stop apache2 and start the ceilometer api by command line, like ceilometer-api, then it works | 22:08 |
| gordc | hmm.. and ceilometer-api worked fine with mod_wsgi in icehouse? | 22:09 |
| flwang1 | gordc: yes | 22:10 |
| gordc | pradk: ^ you have any ida | 22:10 |
| gordc | idea* | 22:10 |
| gordc | flwang1: i'm not really sure myself. | 22:10 |
| flwang1 | so it's like a migration issue, but given the sample create is not a used widel | 22:10 |
| flwang1 | widely, so i'm not sure if it has been raised by anybody | 22:11 |
| flwang1 | gordc: yep, it's really weird | 22:11 |
| gordc | i think people actually use that quite a bit. | 22:11 |
| flwang1 | gordc: ok | 22:11 |
| gordc | i don't really undestand why there's no log | 22:12 |
| flwang1 | me too :( | 22:12 |
| gordc | flwang1: you can create a bug and mention how you've started/configured api... | 22:13 |
| gordc | i don't really have time to debug it currently though. sorry. | 22:13 |
| flwang1 | gordc: ok, got it, will do. thanks a lot | 22:13 |
| flwang1 | btw, a quick question | 22:13 |
| flwang1 | IIRC, we have already split the notification agent from central agent since icehouse, right? | 22:14 |
| flwang1 | so if i don't start notification agent, can ceilometer still create samples(by sample-create) successfully? | 22:14 |
| *** jruano has quit IRC | 22:14 | |
| gordc | flwang1: the notification agent split from collector service in juno | 22:15 |
| gordc | using sample-create, there should be a direct connection to db from api service (pre-Liberty) | 22:16 |
| *** maubry has joined #openstack-ceilometer | 22:18 | |
| flwang1 | gordc: the 'direct' param is supported since Liberty, right? | 22:18 |
| flwang1 | or Kilo? | 22:18 |
| gordc | flwang1: Liberty | 22:18 |
| flwang1 | ok | 22:19 |
| gordc | without it, it will go from api -> notification -> collector | 22:19 |
| gordc | becasue we do all processing at notification agent now. | 22:19 |
| flwang1 | gordc: yep, i saw the code change, seems now we're publishing all the metrics (even from pollster) to notification bus by notification agent, and then collected by the central agent, right? | 22:21 |
| gordc | s/central agent/collector/ | 22:21 |
| gordc | central agent does polling | 22:21 |
| gordc | collector does storage | 22:21 |
| gordc | notification does pipeline processing | 22:22 |
| flwang1 | yes, the ceilometer-collector | 22:22 |
| flwang1 | typo | 22:22 |
| gordc | :) | 22:22 |
| flwang1 | thanks for the clarification, man | 22:23 |
| gordc | np | 22:23 |
| *** Samolo has quit IRC | 22:27 | |
| *** edmondsw has quit IRC | 22:28 | |
| flwang1 | gordc: re the notification agent stuff, i saw this https://review.openstack.org/#/c/56577/ | 22:28 |
| flwang1 | gordc: https://blueprints.launchpad.net/ceilometer/+spec/split-collector | 22:29 |
| flwang1 | i saw this feature is in icehouse, but you said it's in juno, am i missing something? | 22:30 |
| gordc | flwang1: it's possible i just misremember cycle... it's very old ;) | 22:31 |
| flwang1 | :) | 22:32 |
| *** pradk has quit IRC | 22:33 | |
| *** thorst1 has quit IRC | 22:35 | |
| *** maubry has quit IRC | 22:44 | |
| *** gzhai has joined #openstack-ceilometer | 22:44 | |
| *** maubry has joined #openstack-ceilometer | 22:48 | |
| *** maubry has quit IRC | 22:49 | |
| *** jruano has joined #openstack-ceilometer | 22:58 | |
| *** rbak has quit IRC | 23:01 | |
| openstackgerrit | gordon chung proposed openstack/ceilometer: WIP restrict admin event access https://review.openstack.org/218706 | 23:02 |
| *** gordc has quit IRC | 23:02 | |
| *** exploreshaifali has quit IRC | 23:04 | |
| *** ddieterly has joined #openstack-ceilometer | 23:20 | |
| *** maubry has joined #openstack-ceilometer | 23:24 | |
| *** jruano has quit IRC | 23:29 | |
| *** ankita_wagh has quit IRC | 23:35 | |
| *** maubry has quit IRC | 23:38 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!