| *** skelpter has left #openstack-ceilometer | 00:18 | |
| *** matsuhashi has joined #openstack-ceilometer | 00:27 | |
| openstackgerrit | A change was merged to openstack/ceilometer: Refactor tests to remove direct access to test DBManagers https://review.openstack.org/95613 | 00:31 |
|---|---|---|
| *** jaypipes has quit IRC | 00:32 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ceilometer: Updated from global requirements https://review.openstack.org/96819 | 00:34 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/python-ceilometerclient: Updated from global requirements https://review.openstack.org/91238 | 00:39 |
| *** dmsimard_away is now known as dmsimard | 00:51 | |
| openstackgerrit | Lianhao Lu proposed a change to openstack/ceilometer: Skipping central agent pollster when keystone not available https://review.openstack.org/98109 | 01:05 |
| openstackgerrit | Fabio Giannetti proposed a change to openstack/ceilometer-specs: Adding support for health check in Ceilometer API https://review.openstack.org/95817 | 01:08 |
| *** dmsimard is now known as dmsimard_away | 01:18 | |
| *** fnaval has joined #openstack-ceilometer | 01:24 | |
| *** shakayumi has joined #openstack-ceilometer | 01:30 | |
| *** shakayumi has quit IRC | 01:30 | |
| *** nosnos has joined #openstack-ceilometer | 01:33 | |
| *** _cjones__ has quit IRC | 01:35 | |
| *** _cjones_ has joined #openstack-ceilometer | 01:36 | |
| *** nati_uen_ has quit IRC | 01:36 | |
| openstackgerrit | A change was merged to openstack/ceilometer-specs: ceilometer upgrade testing in grenade https://review.openstack.org/98168 | 01:38 |
| *** prad has quit IRC | 01:38 | |
| *** _cjones_ has quit IRC | 01:40 | |
| *** shakayumi has joined #openstack-ceilometer | 01:41 | |
| *** shakayumi has quit IRC | 01:41 | |
| *** thomasem has joined #openstack-ceilometer | 01:59 | |
| *** thomasem has quit IRC | 02:03 | |
| *** _cjones_ has joined #openstack-ceilometer | 02:12 | |
| openstackgerrit | liusheng proposed a change to openstack/ceilometer-specs: Specs for improve-alarm-evaluation https://review.openstack.org/95418 | 02:15 |
| *** _cjones_ has quit IRC | 02:16 | |
| *** nati_ueno has joined #openstack-ceilometer | 02:23 | |
| *** zhikunliu has joined #openstack-ceilometer | 02:26 | |
| *** rwsu has quit IRC | 02:52 | |
| *** _cjones_ has joined #openstack-ceilometer | 03:03 | |
| *** _cjones_ has quit IRC | 03:03 | |
| *** _cjones_ has joined #openstack-ceilometer | 03:04 | |
| *** _cjones_ has quit IRC | 03:08 | |
| *** nati_ueno has quit IRC | 03:10 | |
| openstackgerrit | Pradeep Kilambi proposed a change to openstack/ceilometer: Update Measurement Docs for LBaaS https://review.openstack.org/98513 | 03:11 |
| *** prad has joined #openstack-ceilometer | 03:12 | |
| *** _cjones_ has joined #openstack-ceilometer | 03:36 | |
| *** thomasem has joined #openstack-ceilometer | 03:36 | |
| *** thomasem has quit IRC | 03:38 | |
| *** thomasem has joined #openstack-ceilometer | 03:39 | |
| *** _cjones_ has quit IRC | 03:42 | |
| *** nosnos has quit IRC | 03:43 | |
| *** thomasem_ has joined #openstack-ceilometer | 03:47 | |
| *** thomasem has quit IRC | 03:47 | |
| *** thomasem_ has quit IRC | 03:51 | |
| *** Longgeek has joined #openstack-ceilometer | 03:52 | |
| *** Longgeek has quit IRC | 03:57 | |
| openstackgerrit | A change was merged to openstack/python-ceilometerclient: use mock instead of try...finally https://review.openstack.org/96699 | 04:12 |
| openstackgerrit | A change was merged to openstack/python-ceilometerclient: replace dict.iteritems() with six.iteritems(dict) https://review.openstack.org/95557 | 04:12 |
| *** r3pl4y has joined #openstack-ceilometer | 04:17 | |
| *** fnaval has quit IRC | 04:18 | |
| *** nosnos has joined #openstack-ceilometer | 04:24 | |
| *** r3pl4y has quit IRC | 04:40 | |
| openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: Fix Sphinx directive name in session.py https://review.openstack.org/98834 | 04:45 |
| *** ildikov has quit IRC | 04:51 | |
| *** harlowja is now known as harlowja_away | 04:59 | |
| *** zhikunliu has quit IRC | 05:04 | |
| *** _cjones_ has joined #openstack-ceilometer | 05:08 | |
| *** _cjones_ has quit IRC | 05:12 | |
| *** ildikov has joined #openstack-ceilometer | 05:15 | |
| *** zhikunliu has joined #openstack-ceilometer | 05:17 | |
| openstackgerrit | Lianhao Lu proposed a change to openstack/ceilometer: Fixed unit test TestRealNotification https://review.openstack.org/98953 | 05:27 |
| *** Longgeek has joined #openstack-ceilometer | 05:49 | |
| *** Longgeek has quit IRC | 05:49 | |
| *** Longgeek has joined #openstack-ceilometer | 05:50 | |
| *** Longgeek has quit IRC | 05:51 | |
| *** Longgeek has joined #openstack-ceilometer | 05:51 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/88506 | 06:01 |
| *** Longgeek has quit IRC | 06:04 | |
| *** Longgeek has joined #openstack-ceilometer | 06:04 | |
| *** Longgeek has quit IRC | 06:04 | |
| *** Longgeek has joined #openstack-ceilometer | 06:05 | |
| *** Longgeek has quit IRC | 06:06 | |
| *** Longgeek_ has joined #openstack-ceilometer | 06:06 | |
| *** _cjones_ has joined #openstack-ceilometer | 06:09 | |
| *** _nadya_ has joined #openstack-ceilometer | 06:10 | |
| *** _cjones_ has quit IRC | 06:13 | |
| *** eglynn has joined #openstack-ceilometer | 06:29 | |
| eglynn | jd__, sileht, ildikov, _nadya_: anyone with bandwidth to look at llu's patch https://review.openstack.org/98953 ? | 06:31 |
| eglynn | (relates to the high ceilo unit test failure rate that jogo raised on the ML) | 06:34 |
| sileht | eglynn, done | 06:35 |
| eglynn | sileht: thank you sir! | 06:35 |
| _nadya_ | eglynn: hi! Nobody is looking here https://review.openstack.org/#/c/87249/ :( Is it on review since I-release. is juno-1 code freeze today? | 06:44 |
| eglynn | _nadya_: juno-1 freeze is when we call for the tag | 06:46 |
| eglynn | _nadya_: (the milestone cutting process is little more automated and stream-lined than in previous cycles) | 06:46 |
| eglynn | _nadya_: ... probably tomorrow I suspect given the state of the gate | 06:46 |
| eglynn | _nadya_: ... but I'll have more insight on that after 1:1 with the rel mgr in the early afternoon | 06:47 |
| _nadya_ | eglynn: ok, I see | 06:48 |
| eglynn | _nadya_: ... in the meantime, I'll have a look at your patch this morning (modulo lack of domain expertise on hbase) | 06:48 |
| eglynn | jd__, sileht, ildikov, llu: any bandwidth you guys might have to look at _nadya_'s patch also would be appreciated ... https://review.openstack.org/87249 | 06:49 |
| *** inc0 has joined #openstack-ceilometer | 06:49 | |
| ildikov | _nadya_: hi | 06:49 |
| ildikov | _nadya_: have you seen my questions on the event patch for HBase? | 06:49 |
| _nadya_ | ildikov: hi! yes, I'll try to answer now | 06:50 |
| ildikov | _nadya_: cool, tnx | 06:50 |
| llu | i'll try it again, but be prepared to forgive my illiterate on Hbase | 06:53 |
| *** r0j4z0 has quit IRC | 06:56 | |
| eglynn | llu: thank you sir! | 06:56 |
| *** nosnos has quit IRC | 06:59 | |
| DinaBelova | folks, I caught a cold, so today won't be available all the time :( if you'll need something, please ping me, I always read the backlog... | 07:00 |
| *** matsuhashi has quit IRC | 07:01 | |
| *** matsuhas_ has joined #openstack-ceilometer | 07:01 | |
| _nadya_ | llu, eglynn, ildikov, sileht, about my resources patch. Some part is about HBase-mock implementation. It's the most difficult part of all hbase-related patches. But at the same time it is *only* for tests. So I think that sometimes it doesn't worth to dig this code. But it's up to you, of course | 07:02 |
| *** r3pl4y has joined #openstack-ceilometer | 07:02 | |
| *** nosnos has joined #openstack-ceilometer | 07:05 | |
| openstackgerrit | Chinmaya Bharadwaj proposed a change to openstack/ceilometer: Fixes ceilometer-compute service start failure https://review.openstack.org/97831 | 07:10 |
| *** _cjones_ has joined #openstack-ceilometer | 07:10 | |
| *** chinmay_ has joined #openstack-ceilometer | 07:10 | |
| ildikov | _nadya_: there is still one more question in that patch, that is the only that I'm really concerned about, the rest are just nits :) | 07:11 |
| *** chinmay_ has quit IRC | 07:12 | |
| _nadya_ | ildikov: "This adding 'desc' to trait_types is just a naming issue here, right?" - this one? | 07:13 |
| *** _cjones_ has quit IRC | 07:14 | |
| ildikov | _nadya_: yeap, that is the one | 07:15 |
| ildikov | _nadya_: TBH, I'm not the expert of the event feature and I still need some time for understanding HBase a bit more, so that is why I have these silly questions | 07:18 |
| _nadya_ | ildikov: no, it's absolutely normal question :) I'm not an expert in events too unfortunatelly:( | 07:19 |
| ildikov | _nadya_: BTW, your -1 is for the missing doc strings or for the unanswered question? | 07:22 |
| _nadya_ | ildikov: let me try to explain as I see that. E.g. get_trait_types. Let's consider two events. The first one has trait with name 'trait_1' and date_type = '0'. The second one has 'trait_1' and date_type ='1'. It's not clear what should be returned | 07:24 |
| _nadya_ | ildikov: because of doc strings and 'desc'-'name' | 07:25 |
| *** eglynn is now known as eglynn-commute | 07:25 | |
| *** zqfan_sleep is now known as zqfan | 07:26 | |
| *** liusheng has quit IRC | 07:27 | |
| _nadya_ | ildikov: will be back soon | 07:28 |
| *** _nadya_ has quit IRC | 07:28 | |
| *** eglynn-commute has quit IRC | 07:28 | |
| *** Infitialis has joined #openstack-ceilometer | 07:42 | |
| *** Infitialis has quit IRC | 08:05 | |
| *** vrovachev has joined #openstack-ceilometer | 08:05 | |
| *** Infitialis has joined #openstack-ceilometer | 08:06 | |
| *** _cjones_ has joined #openstack-ceilometer | 08:10 | |
| *** IvanBerezovskiy has joined #openstack-ceilometer | 08:12 | |
| *** hoangdo has joined #openstack-ceilometer | 08:14 | |
| *** _cjones_ has quit IRC | 08:15 | |
| openstackgerrit | Victor Stinner proposed a change to openstack/ceilometer: Respect $TMPDIR environment variable to run tests https://review.openstack.org/98818 | 08:16 |
| *** eglynn has joined #openstack-ceilometer | 08:16 | |
| *** safchain has joined #openstack-ceilometer | 08:20 | |
| openstackgerrit | Lianhao Lu proposed a change to openstack/ceilometer: Skipping central agent pollster when keystone not available https://review.openstack.org/98109 | 08:25 |
| *** _nadya_ has joined #openstack-ceilometer | 08:25 | |
| *** matsuhas_ has quit IRC | 08:38 | |
| *** nosnos has quit IRC | 08:40 | |
| *** nosnos has joined #openstack-ceilometer | 08:43 | |
| *** nosnos has quit IRC | 08:44 | |
| *** matsuhashi has joined #openstack-ceilometer | 08:44 | |
| eglynn | EmilienM: FYI I tried again last night to rustle up some reviewer attention for https://review.openstack.org/94468 | 08:57 |
| EmilienM | eglynn: o\ | 08:57 |
| eglynn | EmilienM: ... sadly to no avail :( | 08:57 |
| EmilienM | eglynn: damn | 08:57 |
| eglynn | EmilienM: ... but it did come out in the wash that there are only 2 cores on the grenade project (... sdague and dtroyer) | 08:57 |
| eglynn | EmilienM: ... which is probably less than the effective minimum | 08:57 |
| EmilienM | yeah | 08:57 |
| eglynn | EmilienM: ... so FYI I was going to ask that sdague & dtroyer consider nominating new core(s) | 08:58 |
| EmilienM | they should allow more core reviewers | 08:58 |
| EmilienM | eglynn: cool, thanks | 08:58 |
| *** nosnos has joined #openstack-ceilometer | 09:03 | |
| *** denis_makogon has joined #openstack-ceilometer | 09:03 | |
| openstackgerrit | Igor Degtiarov proposed a change to openstack/ceilometer: [HBase] Implement events on HBase https://review.openstack.org/91408 | 09:04 |
| *** zhikunliu has quit IRC | 09:09 | |
| *** idegtiarov has joined #openstack-ceilometer | 09:10 | |
| *** denis_makogon has quit IRC | 09:10 | |
| *** _cjones_ has joined #openstack-ceilometer | 09:11 | |
| *** isviridov_ is now known as isviridov | 09:12 | |
| *** denis_makogon has joined #openstack-ceilometer | 09:13 | |
| *** liusheng has joined #openstack-ceilometer | 09:14 | |
| *** _cjones_ has quit IRC | 09:16 | |
| *** llu is now known as llu-away | 09:17 | |
| openstackgerrit | Rikimaru Honjo proposed a change to openstack/ceilometer: Adds caches for image and flavor in compute agent https://review.openstack.org/98301 | 09:17 |
| *** llu-away is now known as llu | 09:17 | |
| ildikov | _nadya_: hey, thank you and the others for the answers and fixes, I did what I could for that patch :) | 09:27 |
| _nadya_ | ildikov: thank you for your time! | 09:30 |
| ildikov | _nadya_: np, sorry for all the silly questions ;) | 09:31 |
| eglynn | DinaBelova, vrovachev, _nadya_: FYI mtreinish floated the idea of using one of the QA/Infra mid-cycle days as a ceilometer testing sub-sprint | 09:34 |
| eglynn | https://wiki.openstack.org/wiki/Qa_Infra_Meetup_2014 (Darmstadt, Germany, prolly July 17th or 18th) | 09:34 |
| *** KurtRao has joined #openstack-ceilometer | 09:34 | |
| eglynn | ... I *may* be able to travel for that one day (depends on whether other Red Hatters will be covering that mid-cycle already) | 09:34 |
| DinaBelova | eglynn, hehe, Darmstadt... | 09:35 |
| DinaBelova | nice :) | 09:35 |
| idegtiarov | ildikov: Thank you for review! :) | 09:35 |
| *** nacim has joined #openstack-ceilometer | 09:35 | |
| eglynn | DinaBelova: exotic, NOT! | 09:35 |
| ildikov | idegtiarov: thanks for the fixes :) | 09:35 |
| DinaBelova | eglynn, :D | 09:35 |
| eglynn | DinaBelova, vrovachev, _nadya_: ... mtrienish made the point that it would be good have ceilo folks with a direct hand in the tempest work also involved | 09:35 |
| eglynn | ... so just throwing the idea out there on the table | 09:36 |
| DinaBelova | ok, thank you so much! | 09:36 |
| _nadya_ | oh...Atlanta, Paris, Darmstadt, Paris again, what else will I miss :) | 09:36 |
| eglynn | LOL :) | 09:36 |
| *** KurtRao has quit IRC | 09:36 | |
| DinaBelova | eglynn, I'll talk with our managers about this :) | 09:36 |
| *** KurtRao has joined #openstack-ceilometer | 09:36 | |
| DinaBelova | _nadya_, I guess first Paris won't be mine too... | 09:36 |
| eglynn | DinaBelova: thank you! ... lemme know the outcome of that and I'll circle back with Matt either way | 09:37 |
| DinaBelova | eglynn, thanks! | 09:37 |
| * eglynn raps self on kunckles for using "circle back" on IRC ... ;) | 09:37 | |
| vrovachev | eglynn: It's nice news :) | 09:39 |
| KurtRao | Hi folks, is the next weekly meeting on June 12th? Information on the wikipage says the next meeting is on Thus May 29th, which seems out of date | 09:40 |
| eglynn | KurtRao: the weekly meeting is on every week on Thursday at 15:00 UTC | 09:43 |
| openstackgerrit | Nadya Privalova proposed a change to openstack/ceilometer: [HBase] get_resource optimization https://review.openstack.org/87249 | 09:43 |
| eglynn | KurtRao: the wiki sometimes lags | 09:43 |
| eglynn | KurtRao: but the calendar never lies! ;) | 09:43 |
| eglynn | KurtRao: ... https://www.google.com/calendar/ical/h102rn64cnl9n2emhc5i3hjjso%40group.calendar.google.com/public/basic.ics | 09:44 |
| KurtRao | eglynn, got it. Thanks a looot~ | 09:44 |
| _nadya_ | eglynn: I'm ok with Igor's patch. May I approve? | 09:45 |
| eglynn | eglynn: absolutely! ... thanks | 09:45 |
| eglynn | KurtRao: ... now updated ... https://wiki.openstack.org/wiki/Meetings/Ceilometer#Next_meeting | 09:46 |
| *** Alexei_987 has quit IRC | 10:02 | |
| *** zqfan is now known as zqfan_bot | 10:10 | |
| *** _cjones_ has joined #openstack-ceilometer | 10:12 | |
| eglynn | hi cores ... just in terms of knocking down all the in-progress reviews for juno-1 target'd patches | 10:12 |
| eglynn | ... can I draw your attention to llu's patch https://review.openstack.org/98109 ? | 10:12 |
| eglynn | ... thanks! :) | 10:13 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Splits hbase storage code base https://review.openstack.org/98721 | 10:14 |
| openstackgerrit | A change was merged to openstack/ceilometer: Remove (c) and remove unnecessary encoding lines https://review.openstack.org/96117 | 10:14 |
| openstackgerrit | A change was merged to openstack/ceilometer: Removes per test testr timeout https://review.openstack.org/98820 | 10:15 |
| openstackgerrit | A change was merged to openstack/ceilometer: Metering LoadBalancer as a Service https://review.openstack.org/95784 | 10:15 |
| DinaBelova | oh my God, I *love* seeing changes merged :) | 10:15 |
| sileht | DinaBelova, it's rare those days | 10:15 |
| DinaBelova | sileht, exactly | 10:15 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Splits hbase storage code base https://review.openstack.org/98721 | 10:16 |
| *** _cjones_ has quit IRC | 10:16 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ceilometer: Updated from global requirements https://review.openstack.org/96819 | 10:17 |
| eglynn | DinaBelova, sileht: yep :) | 10:21 |
| eglynn | ... it's like waiting for a bus in the rain ;) | 10:21 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/python-ceilometerclient: Updated from global requirements https://review.openstack.org/91238 | 10:22 |
| ildikov | eglynn: ... it's not always bad :) | 10:39 |
| eglynn | ildikov: ... yeap, an umbrella helps | 10:40 |
| eglynn | idegtiarov: ok if I break the dependency of https://blueprints.launchpad.net/ceilometer/+spec/hbase-events-feature | 10:41 |
| eglynn | idegtiarov: ... on https://blueprints.launchpad.net/ceilometer/+spec/hbase-events-structure ? | 10:41 |
| eglynn | idegtiarov: ... since the latter is superceeded | 10:42 |
| ildikov | eglynn: ... I threw away my last umbrella years ago :) | 10:42 |
| idegtiarov | eglynn: yes, I think it will be ok. | 10:44 |
| eglynn | idegtiarov: cool, thank you sir! | 10:44 |
| *** denis_makogon has left #openstack-ceilometer | 10:44 | |
| *** ekarlso has quit IRC | 10:57 | |
| *** ekarlso has joined #openstack-ceilometer | 10:58 | |
| openstackgerrit | A change was merged to openstack/ceilometer: Fixed unit test TestRealNotification https://review.openstack.org/98953 | 11:02 |
| DinaBelova | eglynn, btw I started mail discussion about oslo.messaging improvement - as you remember, we had lots of performance bottle necks while the performance testing on larger scale than sql was able to work with because of oslo.messaging.... I'll be really interested in your opinion here (ML thread "[oslo][messaging] Further improvements and refactoring") | 11:12 |
| DinaBelova | dhellmann, I guess it'll be interesting fro you too ^^ | 11:12 |
| DinaBelova | for* | 11:12 |
| eglynn | DinaBelova: ... and also interesting for the newest member of oslo.messaging core :) | 11:14 |
| eglynn | https://review.openstack.org/#/admin/groups/318,members | 11:14 |
| eglynn | ^^^ congrats BTW sileht | 11:14 |
| DinaBelova | sileht, wow! | 11:14 |
| DinaBelova | it's really cool :) | 11:15 |
| DinaBelova | congrats, man! | 11:15 |
| sileht | thanks | 11:15 |
| DinaBelova | sileht, I guess you'll be quite interested in the topic I mentioned too ^^ | 11:15 |
| ildikov | sileht: congrats! :) | 11:16 |
| *** dmsimard_away is now known as dmsimard | 11:17 | |
| *** _nadya_ has quit IRC | 11:26 | |
| *** alexpilotti has joined #openstack-ceilometer | 11:29 | |
| eglynn | _nadya_: according to the whiteboard in LP, https://review.openstack.org/78244 partially addresses hbase-resource-rowkey-enhancement | 11:30 |
| eglynn | ... but I'm not seeing the BP referenced in the commit msg | 11:30 |
| eglynn | ... /me wondering if that BP is really partially implemented by 78244 having already landed? | 11:31 |
| eglynn | whoops _nadya_ just gone ... | 11:31 |
| DinaBelova | eglynn, yes, Nadya's change fixed some of the performance issues | 11:42 |
| DinaBelova | although it was bug fix\ | 11:42 |
| openstackgerrit | Yatin Kumbhare proposed a change to openstack/ceilometer-specs: Specs for email alarm notification https://review.openstack.org/99019 | 11:46 |
| openstackgerrit | A change was merged to openstack/ceilometer: Change pipeline_manager to instance attribute in hooks https://review.openstack.org/98295 | 11:54 |
| EmilienM | eglynn: got +2 +A on https://review.openstack.org/#/c/94468/ | 11:55 |
| EmilienM | which is a good news | 11:56 |
| *** yatin has joined #openstack-ceilometer | 11:56 | |
| *** _nadya_ has joined #openstack-ceilometer | 11:57 | |
| EmilienM | it will be merged in 18min (currently in the gate) | 11:57 |
| eglynn | EmilienM: \o/ ... I was just discussing that with sdague | 11:57 |
| EmilienM | eglynn: next step: https://review.openstack.org/#/c/97430/ | 11:57 |
| EmilienM | eglynn: I'll take care of this one this afternoon | 11:58 |
| eglynn | EmilienM: excellent, thank you sir! | 11:58 |
| EmilienM | eglynn: cool | 11:58 |
| EmilienM | eglynn: I've seen you will be in Paris with Chris, that's cool | 11:58 |
| eglynn | EmilienM: ... yeah, it would be great for you guys to chat about javelin also while Chris is there | 11:59 |
| EmilienM | eglynn: sure, i'll be there also btw | 12:00 |
| *** cmart has joined #openstack-ceilometer | 12:00 | |
| ekarlso | what's javelin ? | 12:01 |
| eglynn | ekarlso: the bit of grenade that makes assertions about resource survivability across upgrades | 12:03 |
| ekarlso | ah | 12:03 |
| eglynn | _nadya_: o/ | 12:07 |
| *** shakayumi has joined #openstack-ceilometer | 12:10 | |
| *** _cjones_ has joined #openstack-ceilometer | 12:13 | |
| *** Alexei_987 has joined #openstack-ceilometer | 12:15 | |
| *** _cjones_ has quit IRC | 12:18 | |
| *** jdob has joined #openstack-ceilometer | 12:22 | |
| *** Longgeek_ has quit IRC | 12:25 | |
| *** Longgeek has joined #openstack-ceilometer | 12:26 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ceilometer: Updated from global requirements https://review.openstack.org/96819 | 12:27 |
| *** dhellmann has quit IRC | 12:28 | |
| *** Ju has quit IRC | 12:35 | |
| *** gordc has joined #openstack-ceilometer | 12:36 | |
| *** heyongli has joined #openstack-ceilometer | 12:42 | |
| *** palar has joined #openstack-ceilometer | 12:43 | |
| *** heyongli has quit IRC | 12:43 | |
| *** heyongli has joined #openstack-ceilometer | 12:43 | |
| *** heyongli has quit IRC | 12:44 | |
| *** yhe_ has joined #openstack-ceilometer | 12:46 | |
| *** palar has quit IRC | 12:47 | |
| *** palar has joined #openstack-ceilometer | 12:48 | |
| jd__ | I'm confused | 12:49 |
| jd__ | does that sound normal we have instances sample where architecture is None? | 12:50 |
| jd__ | looks like it comes in that way in notifications | 12:50 |
| *** yhe_ has quit IRC | 12:50 | |
| *** alexpilotti_ has joined #openstack-ceilometer | 12:53 | |
| sileht | jd__, special case for nsa instance ;p | 12:53 |
| *** alexpilotti has quit IRC | 12:54 | |
| *** alexpilotti_ is now known as alexpilotti | 12:54 | |
| jd__ | haha | 12:56 |
| *** changbl has quit IRC | 12:57 | |
| *** rbowen has joined #openstack-ceilometer | 12:57 | |
| eglynn | jd__: yeah, sounds kinda stealthy ;) | 13:00 |
| jd__ | sounds like it's not enforced in Nova, I wonder if a bad design or something useful | 13:01 |
| DinaBelova | jd__, usual question - bug or feature? | 13:02 |
| DinaBelova | :) | 13:02 |
| jd__ | indeed | 13:02 |
| jd__ | so it looks like the architecture is actually derived from the architecture set on the image | 13:03 |
| jd__ | if it's set | 13:03 |
| jd__ | so it's not really an instance property | 13:03 |
| *** julim has joined #openstack-ceilometer | 13:06 | |
| *** erecio has quit IRC | 13:07 | |
| *** erecio has joined #openstack-ceilometer | 13:08 | |
| *** idegtiarov has quit IRC | 13:09 | |
| *** gordc has quit IRC | 13:12 | |
| *** _cjones_ has joined #openstack-ceilometer | 13:14 | |
| *** promulo has quit IRC | 13:16 | |
| cmart | hello | 13:18 |
| *** _cjones_ has quit IRC | 13:19 | |
| cmart | I have a question regarding alarms and how they are applied to resources. Let's say that I want to create an alarm that applies to ALL of my VM's when the CPU usage reaches a certain percentage. Is this possible to create one alarm "to rule them all"? | 13:21 |
| *** alexpilotti_ has joined #openstack-ceilometer | 13:21 | |
| *** alexpilotti has quit IRC | 13:22 | |
| *** alexpilotti_ is now known as alexpilotti | 13:22 | |
| *** thomasem has joined #openstack-ceilometer | 13:28 | |
| eglynn-office | cmart: just set the threshold query rule to 'user_id=cmarts_UUID' | 13:31 |
| *** yatin has left #openstack-ceilometer | 13:32 | |
| *** yatin has quit IRC | 13:32 | |
| cmart | eglynn-office: and with a meter name called "cpu", of course. And in the case that I want to do the same but, let's say, with images, I would have to use the same query but change the meter name, right? | 13:35 |
| *** inc0 has quit IRC | 13:37 | |
| DinaBelova | eglynn-office, I guess I may need some your help in the qa-channel :) | 13:38 |
| eglynn | DinaBelova: on a call, will hop on after | 13:39 |
| *** matsuhashi has quit IRC | 13:39 | |
| DinaBelova | eglynn, ok | 13:40 |
| *** liusheng has quit IRC | 13:40 | |
| *** Longgeek has quit IRC | 13:42 | |
| *** Longgeek has joined #openstack-ceilometer | 13:42 | |
| DinaBelova | eglynn, http://paste.openstack.org/show/fj9XOMl15XFhtezWBDnK/ is the short variant of the discussion itself | 13:46 |
| DinaBelova | :) | 13:46 |
| openstackgerrit | Eugeniya Kudryashova proposed a change to openstack/python-ceilometerclient: Use HTTPClient from common Oslo code https://review.openstack.org/68939 | 13:48 |
| *** nosnos has quit IRC | 13:50 | |
| *** _nadya_ has quit IRC | 13:50 | |
| *** gordc has joined #openstack-ceilometer | 13:54 | |
| *** Shixiong has joined #openstack-ceilometer | 14:03 | |
| openstackgerrit | A change was merged to openstack/ceilometer: Update Measurement Docs for LBaaS https://review.openstack.org/98513 | 14:03 |
| *** Ju_ has joined #openstack-ceilometer | 14:06 | |
| *** promulo has joined #openstack-ceilometer | 14:09 | |
| openstackgerrit | A change was merged to openstack/ceilometer: [HBase] Implement events on HBase https://review.openstack.org/91408 | 14:09 |
| openstackgerrit | A change was merged to openstack/ceilometer: Skipping central agent pollster when keystone not available https://review.openstack.org/98109 | 14:09 |
| DinaBelova | I see at least 8 merged changes today :) | 14:10 |
| DinaBelova | that looks beautiful :) | 14:10 |
| *** KurtRao has quit IRC | 14:10 | |
| *** KurtRao has joined #openstack-ceilometer | 14:11 | |
| *** jdob has quit IRC | 14:15 | |
| *** _cjones_ has joined #openstack-ceilometer | 14:15 | |
| *** jdob has joined #openstack-ceilometer | 14:15 | |
| *** _nadya_ has joined #openstack-ceilometer | 14:17 | |
| *** _cjones_ has quit IRC | 14:19 | |
| *** r3pl4y has quit IRC | 14:20 | |
| *** dhellmann has joined #openstack-ceilometer | 14:20 | |
| *** isviridov is now known as isviridov|away | 14:24 | |
| *** r3pl4y has joined #openstack-ceilometer | 14:27 | |
| *** jdob_ has joined #openstack-ceilometer | 14:31 | |
| *** jdob has quit IRC | 14:31 | |
| openstackgerrit | Christian Martinez proposed a change to openstack/ceilometer: Adding alarm list filtering by state and meter https://review.openstack.org/96582 | 14:32 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ceilometer: Updated from global requirements https://review.openstack.org/96819 | 14:33 |
| cmart | eglynn: are you around? | 14:36 |
| eglynn | yes | 14:36 |
| cmart | question related to alarms :) | 14:37 |
| *** jaypipes has joined #openstack-ceilometer | 14:37 | |
| eglynn | yes? | 14:37 |
| cmart | This is the "experiment" I ran. Without creating any instances in my dev env, I created an alarm that depends on the amount of instances available. "ceilometer alarm-threshold-create --name alarm1 --description "Alarm-meterinstance" --meter-name instance --threshold 1 --alarm-action 'log://' | 14:38 |
| cmart | this works fine. | 14:39 |
| *** jaypipes has quit IRC | 14:39 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/pycadf: Updated from global requirements https://review.openstack.org/99091 | 14:39 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/python-ceilometerclient: Updated from global requirements https://review.openstack.org/91238 | 14:39 |
| cmart | but when I tried to "retrieve" all the available meters, it didn't returned "instance" as a metric. | 14:40 |
| *** jaypipes has joined #openstack-ceilometer | 14:40 | |
| cmart | I ran ceilometer meter-list | 14:40 |
| *** jdob_ is now known as jdob | 14:40 | |
| cmart | my question is: is the "meter-list" command the best way to retrieve all the available meters? | 14:40 |
| *** jaypipes has quit IRC | 14:41 | |
| cmart | I'm analyzing the new alarm page design and they put a "cool" select box in which you can select the "meter" to be used (and I'm trying to understand what should I used to fill that) | 14:42 |
| *** jaypipes has joined #openstack-ceilometer | 14:42 | |
| eglynn | cmart: the meter-list returns meters that are *per resource* | 14:43 |
| *** jaypipes has quit IRC | 14:44 | |
| cmart | eglynn: so what about the instance meter I used when creating my alarm? Where is that stored? Can it be retrieved with some of the current cli commands? | 14:45 |
| eglynn | cmart: you just want the uniq set of meter names, not the duplicates for say cpu_util for instanceA, cpu_util for instanceB, ... etc | 14:45 |
| cmart | eglynn: yes | 14:45 |
| eglynn | one sec | 14:45 |
| *** jaypipes has joined #openstack-ceilometer | 14:46 | |
| cmart | eglynn: the meter names that could be used to create an alarm appropriately :) | 14:46 |
| *** r3pl4y has quit IRC | 14:47 | |
| eglynn | cmart: stuck in an important discussion on #os-qa, will get back to you as soon as I can | 14:53 |
| cmart | eglynn: no problem. I can wait :) | 14:53 |
| openstackgerrit | Pradeep Kilambi proposed a change to openstack/ceilometer: Fix the meter type for LB Bytes https://review.openstack.org/99108 | 14:54 |
| sileht | someone to take a look to this fix: https://review.openstack.org/#/c/97176/ | 14:56 |
| *** isviridov|away is now known as isviridov | 14:56 | |
| sileht | _nadya_, Does all the hbase reviews you needs for j1 have been merged ? | 14:57 |
| sileht | DinaBelova, ^ ? | 14:57 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Splits hbase storage code base https://review.openstack.org/98721 | 14:58 |
| *** palar_ has joined #openstack-ceilometer | 15:00 | |
| *** Shixiong has quit IRC | 15:00 | |
| *** idegtiarov has joined #openstack-ceilometer | 15:02 | |
| *** palar has quit IRC | 15:03 | |
| DinaBelova | sileht, yes | 15:04 |
| DinaBelova | I see the hbase events merged | 15:04 |
| eglynn | cmart: ... so this is for horizon, right? | 15:04 |
| *** palar_ has quit IRC | 15:05 | |
| eglynn | cmart: ... i.e. for an alarms dashboard in horizon, correct? | 15:05 |
| jaypipes | eglynn: when's the mid-cycle ceilo meetup again? got a link for me? | 15:06 |
| ildikov | sileht: DinaBelova: is this still targeted to j-1: https://review.openstack.org/#/c/87249/6? | 15:06 |
| eglynn | cmart: ... are you familiar with how the metering dashboard gets the list of meters currently? | 15:06 |
| eglynn | cmart: ... e.g. https://github.com/openstack/horizon/blob/master/openstack_dashboard//api/ceilometer.py#L773 | 15:06 |
| eglynn | jaypipes: https://wiki.openstack.org/wiki/Sprints/ParisJuno2014 | 15:06 |
| jaypipes | eglynn: ty sir! | 15:06 |
| eglynn | jaypipes: July 2nd thru' 4th | 15:07 |
| jaypipes | got it. thank yee. | 15:07 |
| DinaBelova | ildikov, yes! | 15:07 |
| DinaBelova | sorry | 15:07 |
| DinaBelova | I missed it | 15:07 |
| *** ildikov_ has joined #openstack-ceilometer | 15:08 | |
| eglynn | cmart: so the point is that this info is already hard-coded into horizon | 15:09 |
| *** ildikov has quit IRC | 15:09 | |
| DinaBelova | _nadya_, about the https://review.openstack.org/#/c/87249/6 - will you be able to provide one more patch set, or I may do it myself | 15:09 |
| DinaBelova | or ask ityaptin, for instance | 15:09 |
| eglynn | cmart: ... still there? | 15:10 |
| *** r3pl4y has joined #openstack-ceilometer | 15:21 | |
| *** Infitialis has quit IRC | 15:21 | |
| openstackgerrit | Pradeep Kilambi proposed a change to openstack/ceilometer: Fix the meter type for LB Bytes https://review.openstack.org/99108 | 15:22 |
| eglynn | DinaBelova: if _nadya_ isn't around and you feel familiar enough with the hbase driver to push a new patchset and/or respond to review comments | 15:27 |
| eglynn | DinaBelova: ... then I'd encourage you to do so :) | 15:27 |
| eglynn | DinaBelova: ... as ttx indicated earlier that anything not landable by EoD should be bumped to j2 | 15:28 |
| DinaBelova | eglynn, well, at least I may fix minor nits :D | 15:28 |
| DinaBelova | I know this | 15:28 |
| eglynn | DinaBelova: cool thanks | 15:28 |
| *** changbl has joined #openstack-ceilometer | 15:29 | |
| *** fnaval has joined #openstack-ceilometer | 15:35 | |
| *** idegtiarov has quit IRC | 15:36 | |
| *** Longgeek has quit IRC | 15:39 | |
| openstackgerrit | Pradeep Kilambi proposed a change to openstack/ceilometer: Fix the meter type for LB Bytes https://review.openstack.org/99108 | 15:39 |
| DinaBelova | eglynn, about one of your comments - "Clearer to dispense with the gen variable?" - sorry, can't get the point.. | 15:41 |
| DinaBelova | in the https://review.openstack.org/#/c/87249/6/ceilometer/storage/impl_hbase.py,cm | 15:41 |
| DinaBelova | may you please explain? | 15:41 |
| *** nacim has quit IRC | 15:43 | |
| eglynn | DinaBelova: not a blocker, just a style nitpick | 15:43 |
| eglynn | DinaBelova: but seemed clearer to just use ... for resource_id, data in resource_table.scan(filter=q): | 15:43 |
| DinaBelova | oh, ok | 15:43 |
| eglynn | DinaBelova: ... feel free to ignore | 15:43 |
| DinaBelova | :) | 15:43 |
| *** changbl has quit IRC | 15:44 | |
| openstackgerrit | Fabio Giannetti proposed a change to openstack/ceilometer-specs: Adding support for health check in Ceilometer API https://review.openstack.org/95817 | 15:48 |
| openstackgerrit | Pradeep Kilambi proposed a change to openstack/ceilometer: Fix the meter type for LB Bytes https://review.openstack.org/99108 | 15:55 |
| *** IvanBerezovskiy has left #openstack-ceilometer | 15:57 | |
| *** julim has quit IRC | 15:59 | |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Separate alarm storage models from other models https://review.openstack.org/99137 | 15:59 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Splits mongo storage code base https://review.openstack.org/99138 | 15:59 |
| *** Ruetobas has quit IRC | 16:01 | |
| *** julim has joined #openstack-ceilometer | 16:02 | |
| *** Ruetobas has joined #openstack-ceilometer | 16:03 | |
| openstackgerrit | Pradeep Kilambi proposed a change to openstack/ceilometer: Fix LBaaS connection meter docs https://review.openstack.org/99140 | 16:04 |
| *** Ruetobas has quit IRC | 16:08 | |
| *** Ruetobas has joined #openstack-ceilometer | 16:08 | |
| *** zqfan_home has joined #openstack-ceilometer | 16:11 | |
| *** zqfan_home is now known as zqfan_ | 16:11 | |
| cmart | eglynn: sorry! I was on a team meeting.. | 16:12 |
| cmart | OK. I'll check that code | 16:12 |
| *** _cjones_ has joined #openstack-ceilometer | 16:13 | |
| *** _cjones_ has quit IRC | 16:13 | |
| *** _cjones_ has joined #openstack-ceilometer | 16:13 | |
| openstackgerrit | Dina Belova proposed a change to openstack/ceilometer: [HBase] get_resource optimization https://review.openstack.org/87249 | 16:14 |
| eglynn | DinaBelova: thanks! | 16:14 |
| DinaBelova | eglynn, np! | 16:14 |
| DinaBelova | sorry I was too slow here :) | 16:14 |
| DinaBelova | was improving my HBase skills as well :) | 16:14 |
| cmart | eglynn: thanks! I'll probably send an email to the ML about this. | 16:15 |
| eglynn | cmart: about what? | 16:15 |
| eglynn | cmart: ... i.e. about what issue in particular? | 16:15 |
| DinaBelova | eglynn, oh, I found the typo in my patch set | 16:16 |
| DinaBelova | one moment | 16:16 |
| eglynn | DinaBelova: k | 16:16 |
| openstackgerrit | Dina Belova proposed a change to openstack/ceilometer: [HBase] get_resource optimization https://review.openstack.org/87249 | 16:16 |
| DinaBelova | here it is :) | 16:17 |
| *** shakayumi is now known as shakayumi_afk | 16:17 | |
| DinaBelova | eglynn, I'll be afk ~15 mins (to grab hot tea and honey - as I really want my voice back), so please don't worry if I don't answer :D | 16:18 |
| eglynn | DinaBelova: np! | 16:18 |
| *** shakayumi_afk has quit IRC | 16:19 | |
| *** jaypipes has quit IRC | 16:20 | |
| *** r3pl4y has quit IRC | 16:20 | |
| openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Remove low-value logging from publication codepath https://review.openstack.org/99147 | 16:24 |
| *** r3pl4y has joined #openstack-ceilometer | 16:27 | |
| *** jaypipes has joined #openstack-ceilometer | 16:33 | |
| openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Remove low-value logging from publication codepath https://review.openstack.org/99147 | 16:35 |
| openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Remove low-value logging from publication codepath https://review.openstack.org/99147 | 16:36 |
| *** isviridov is now known as isviridov|away | 16:40 | |
| DinaBelova | eglynn, how're you feeling about [HBase] get_resource optimization change? | 16:46 |
| DinaBelova | does it look ok now? | 16:46 |
| gordc | ildikov_: comments to https://review.openstack.org/#/c/97261/ would be good. i used Nova guidelines as base but tbh, i don't think we can follow it completely. | 16:48 |
| gordc | if we remove repeating logs in ceilometer we'll essentially have no logs at all which is just as bad (unless we implement some batch/dump logging mechanism) | 16:49 |
| eglynn | DinaBelova: still looking | 16:49 |
| DinaBelova | eglynn, thank you so much! | 16:49 |
| *** changbl has joined #openstack-ceilometer | 16:50 | |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Use oslo.msg retry API in rpc publisher https://review.openstack.org/77845 | 16:53 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Permit usage of notifications for metering https://review.openstack.org/80225 | 16:53 |
| ildikov_ | gordc: it's just not a lucky situation, when the amount of logs kills a system | 16:55 |
| eglynn | DinaBelova: ... thanks for addressing those hbase nits | 16:55 |
| DinaBelova | eglynn, np - it was interesting to discover it for me :) | 16:55 |
| ildikov_ | gordc: somehow we should find a way between the all or nothing situations | 16:56 |
| DinaBelova | ildikov_, my you take a look on the https://review.openstack.org/#/c/87249/ too? As you noticed, it's supposed to land in j1 :) | 16:56 |
| *** _nadya__ has joined #openstack-ceilometer | 16:56 | |
| ildikov_ | gordc: I meant in the review comment that we also have to figure it out that what is not applicable for Ceilometer | 16:56 |
| ildikov_ | DinaBelova: I'm in the middle of it currently, it's just not my area, so it takes a while :) | 16:57 |
| DinaBelova | ildikov_, thank you so much for spending your time :) | 16:57 |
| gordc | ildikov_: that's fine... just pointing out a bp exists just so we can actually capture some logic to our logging choices. | 16:58 |
| ildikov_ | DinaBelova: np, I'm all about learning new things day by day :) | 16:58 |
| *** skelpter has joined #openstack-ceilometer | 16:58 | |
| gordc | ildikov_: i'd be interested in how places like facebook/google handle logs. i can't imagine our tempest tests being anything near a typical small deployment. | 16:59 |
| ildikov_ | gordc: sure, I knew about this, I just asked on the qa channel earlier if it got accepted already or not | 16:59 |
| gordc | ildikov_: i meant our own ceilometer bp... not that nova one | 16:59 |
| *** _nadya_ has quit IRC | 17:00 | |
| ildikov_ | gordc: good point, I was thinking about this too, but it our logs are already too much on the gate, then they are prolly not useful in real life deployments | 17:00 |
| ildikov_ | gordc: ah, ok, it's a bit late for context switching for me now it seems :) | 17:00 |
| *** kun_huang has joined #openstack-ceilometer | 17:01 | |
| eglynn | gordc: yeah it kinda depends on what kind of value can be derived from the logs | 17:01 |
| gordc | ildikov_: i think for me i'm more concerned that logs are useful in real life rather than gate. i don't want gate to dictate real-life. | 17:01 |
| eglynn | gordc: ... i.e. whether the volume of data generated is justified | 17:01 |
| ildikov_ | gordc: I will check that spec, surely not today, but tomorrow seems to be a good day for that | 17:02 |
| eglynn | gordc: ... so I know of one case where extremely verbose logs are generated and retained | 17:02 |
| ildikov_ | gordc: I just meant that when you have too much log, it makes it less useful | 17:02 |
| eglynn | gordc: ... but that was to feed into a lot of metrics-gathering and monitoring being based on log-scraping | 17:03 |
| *** julim has quit IRC | 17:03 | |
| gordc | eglynn: agreed... i guess we'll only get a sense of what's 'too verbose' when ceilometer actually gets deployed in production. | 17:03 |
| ildikov_ | gordc: in my previous project I was working on a load generator tool for traffic test on telco nodes and we had plenty of problems with effective logging | 17:03 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Permit usage of notifications for metering https://review.openstack.org/80225 | 17:03 |
| gordc | ildikov_: take your time. we targeted bp for juno-3 i believe | 17:04 |
| eglynn | gordc: yeah, apparently debug is typical in prod | 17:04 |
| eglynn | gordc: that surprised me TBH | 17:04 |
| gordc | eglynn: yeah i was at the ops session relating to that. i guess everyone just plans for worse case scenarios. | 17:05 |
| ildikov_ | gordc: so I do not want also the gate to dictate our life, but if that setup generates too much log already, than it shows some issues regarding to real life deployments with higher load | 17:05 |
| eglynn | gordc: yeap, INFO won't get you too far when your cloud is on fire | 17:05 |
| gordc | ildikov_: well gate really isn't designed to handle logging. i'd assume if you had a real-life deployment and if you did want the logs, you could build around that. | 17:06 |
| eglynn | right-o folks, I gotta run now ... will be back online later | 17:06 |
| *** eglynn has quit IRC | 17:06 | |
| gordc | ildikov_: just some thoughts... we can adjust the bp as we see fit... just want to make sure gate isn't the deciding factor on everything. :) | 17:07 |
| ildikov_ | gordc: yeap, sure, but it also has to be considered that what is useful information and what is the print it to the logs so that we did everything we could | 17:07 |
| ildikov_ | gordc: you can be sure that I'm more motivated here based on my experience in this area, than because of the gate :) | 17:08 |
| *** harlowja_away is now known as harlowja | 17:08 | |
| gordc | ildikov_: completely agree. the logs we have currently aren't really helpful... othing more than 'you're here' messages. | 17:08 |
| gordc | nothing* | 17:08 |
| gordc | ildikov_: cool cool. we have a few weeks to hash out some ideas and rules. | 17:09 |
| ildikov_ | gordc: in that project we started to work on some intelligent logging, like drop the happy scenario logs, if the user hasn't configured to keep it and stuff like this | 17:09 |
| ildikov_ | gordc: to avoid that issue that you have huge amount of logs, but you cannot find the cause of a failure in it | 17:10 |
| ildikov_ | gordc: anyway, I will check the BP and then we will see how to move on with this :) | 17:10 |
| gordc | ildikov_: sounds good. | 17:11 |
| *** underyx is now known as Underyx|off | 17:11 | |
| *** vrovachev has quit IRC | 17:17 | |
| *** Alexei_987 has quit IRC | 17:21 | |
| *** fnaval has quit IRC | 17:22 | |
| *** fnaval has joined #openstack-ceilometer | 17:22 | |
| *** fnaval has quit IRC | 17:27 | |
| ildikov_ | DinaBelova: still around? | 17:39 |
| DinaBelova | ildikov_, yep | 17:39 |
| ildikov_ | DinaBelova: I added some comments to that patch | 17:39 |
| DinaBelova | ok, I see them | 17:40 |
| * DinaBelova reading | 17:40 | |
| ildikov_ | DinaBelova: the -1 is for the line that should be removed | 17:40 |
| DinaBelova | yes, I see this | 17:40 |
| DinaBelova | sorry | 17:40 |
| DinaBelova | :-S | 17:40 |
| openstackgerrit | Dina Belova proposed a change to openstack/ceilometer: [HBase] get_resource optimization https://review.openstack.org/87249 | 17:42 |
| ildikov_ | DinaBelova: /me sorry to catch that tiny nit | 17:42 |
| DinaBelova | ildikov_, nah, thank you for sure! | 17:42 |
| DinaBelova | dunno how I left it, really | 17:42 |
| ildikov_ | DinaBelova: eglynn said that he will be available later, so I guess he will check this patch again | 17:42 |
| DinaBelova | as for the your question about usage of the QualifierFilter - I'm not sure really.. | 17:43 |
| ildikov_ | DinaBelova: I'm lucky as eclipse informs me about these, it was just strange to me that it is pep8 compatible... :) | 17:43 |
| DinaBelova | _nadya__, if you're here, may you take a look on the https://review.openstack.org/#/c/87249/8/ceilometer/storage/impl_hbase.py,cm - the last question? | 17:43 |
| DinaBelova | ildikov_, I guess I need to reinstall my tox env :) | 17:44 |
| DinaBelova | it looks like old one with old ceilo, probably | 17:44 |
| DinaBelova | that's the only reason, I guess | 17:44 |
| *** _nadya__ has quit IRC | 17:45 | |
| ildikov_ | DinaBelova: no probs, I have trust in the experts, I just did not get the point there, maybe if I would spend a bit more time with it... | 17:46 |
| DinaBelova | ildikov_, I guess I need this too :) | 17:47 |
| *** ildikov_ has quit IRC | 17:53 | |
| *** ildikov_ has joined #openstack-ceilometer | 17:54 | |
| ildikov_ | DinaBelova: ok, the patch is good enough for my knowledge currently | 17:56 |
| ildikov_ | DinaBelova: so I pushed the +2 button, the fun part is that my notebook ran out of battery just before doing that | 17:56 |
| ildikov_ | DinaBelova: but if it was a sign, I let eglynn to find the cause of it ;) | 17:57 |
| DinaBelova | ildikov_, hehe, thank you! | 17:59 |
| ildikov_ | DinaBelova: np :) | 18:00 |
| openstackgerrit | Pradeep Kilambi proposed a change to openstack/ceilometer-specs: Update LBaaS spec to match implemented metrics https://review.openstack.org/98165 | 18:00 |
| ildikov_ | DinaBelova: and now I deserve to leave the office for today | 18:01 |
| DinaBelova | ildikov_, thank one more time :) | 18:01 |
| DinaBelova | see u! | 18:01 |
| ildikov_ | DinaBelova: you really don't need to | 18:01 |
| ildikov_ | DinaBelova: laters :) | 18:01 |
| *** rwsu has joined #openstack-ceilometer | 18:02 | |
| *** jaypipes has quit IRC | 18:03 | |
| *** ildikov_ has quit IRC | 18:06 | |
| *** changbl has quit IRC | 18:06 | |
| *** Alexei_987 has joined #openstack-ceilometer | 18:17 | |
| *** _nadya_ has joined #openstack-ceilometer | 18:19 | |
| *** harlowja is now known as harlowja_away | 18:25 | |
| *** safchain has quit IRC | 18:29 | |
| _nadya_ | eglynn-office, DinaBelova, guys, some comments in get_resources patch are not quite correct, have I enough time to fix them? Or code freeze is too close? | 18:30 |
| *** jogo has left #openstack-ceilometer | 18:31 | |
| DinaBelova | _nadya_, I am not sure.. | 18:31 |
| *** ildikov has joined #openstack-ceilometer | 18:33 | |
| _nadya_ | DinaBelova: ok, will address some issues in separate commit | 18:34 |
| *** nati_ueno has joined #openstack-ceilometer | 18:35 | |
| *** fnaval has joined #openstack-ceilometer | 18:36 | |
| *** zqfan_ has quit IRC | 18:43 | |
| ildikov | _nadya_: eglynn said earlier that the deadline for j-1 is EoD | 18:45 |
| ildikov | _nadya_: don't ask the TZ :) | 18:46 |
| _nadya_ | ildikov: it's ok, the problem is only in comment, I will fix that as separate patch. Besides, I'll add some more comments. Maybe as part of sileht patch about storage/impl code separation | 18:47 |
| ildikov | _nadya_: ah, ok cool, that will be fine in a separate patch | 18:48 |
| *** erecio has quit IRC | 18:48 | |
| ildikov | _nadya_: do you plan to change docstring too or just comments? | 18:49 |
| *** erecio has joined #openstack-ceilometer | 18:49 | |
| _nadya_ | ildikov: I planned to add only comments to make some things clearer. because there are some comments that are not answered and I just don't remember why it was written by me 3 month ago. HBase has some unpredictable behaviour with filters | 18:51 |
| ildikov | _nadya_: well, I noticed that :) | 18:52 |
| ildikov | _nadya_: I just asked because I have some work to do with my doc patch which fixes the autodoc for the source code and I have some fix to do in impl_hbase, so I was just wondering, if I should waiit for your change or not | 18:53 |
| ildikov | _nadya_: but if you plan to change and add only comments, it will not be in conflict with my change, so all good | 18:54 |
| ildikov | _nadya_: and also thanks for taking care of this! | 18:55 |
| *** nati_ueno has quit IRC | 18:57 | |
| _nadya_ | ildikov: I'll start tomorrow. After all changes are merged | 18:59 |
| ildikov | _nadya_: ok, cool, that was my plan too | 19:01 |
| *** harlowja_away is now known as harlowja | 19:03 | |
| EmilienM | eglynn-office: just reposted https://review.openstack.org/#/c/97430/ | 19:03 |
| EmilienM | and rebased | 19:04 |
| EmilienM | eglynn-office: we wait now to have +1 from Jenkins & ping infra to merge it. Then we are good to have Grenade jobs for all ceilometers patchs | 19:04 |
| *** nati_ueno has joined #openstack-ceilometer | 19:09 | |
| *** erecio has quit IRC | 19:10 | |
| *** erecio has joined #openstack-ceilometer | 19:11 | |
| *** _nadya_ has quit IRC | 19:17 | |
| *** nealph has quit IRC | 19:38 | |
| openstackgerrit | A change was merged to openstack/ceilometer-specs: Update LBaaS spec to match implemented metrics https://review.openstack.org/98165 | 19:45 |
| *** Underyx|off is now known as underyx | 19:49 | |
| *** kun_huang has quit IRC | 19:50 | |
| *** eglynn has joined #openstack-ceilometer | 19:53 | |
| *** eglynn is now known as eglynn-afk | 19:53 | |
| *** jaypipes has joined #openstack-ceilometer | 20:00 | |
| *** changbl has joined #openstack-ceilometer | 20:06 | |
| *** fnaval has quit IRC | 20:10 | |
| *** fnaval has joined #openstack-ceilometer | 20:11 | |
| *** nati_ueno has quit IRC | 20:11 | |
| *** fnaval has quit IRC | 20:16 | |
| *** nati_ueno has joined #openstack-ceilometer | 20:19 | |
| *** cmart has quit IRC | 20:31 | |
| *** shadower has quit IRC | 20:33 | |
| *** nijaba has quit IRC | 20:33 | |
| *** robbybb111 has quit IRC | 20:33 | |
| *** ogelbukh has quit IRC | 20:33 | |
| *** shadower has joined #openstack-ceilometer | 20:33 | |
| *** robbybb111 has joined #openstack-ceilometer | 20:33 | |
| *** shadower has quit IRC | 20:33 | |
| *** shadower has joined #openstack-ceilometer | 20:33 | |
| *** nijaba has joined #openstack-ceilometer | 20:34 | |
| *** ogelbukh has joined #openstack-ceilometer | 20:34 | |
| *** dhellman_ has joined #openstack-ceilometer | 20:37 | |
| *** eglynn-afk is now known as eglynn | 20:40 | |
| *** fnaval has joined #openstack-ceilometer | 20:40 | |
| *** dhellmann_ has joined #openstack-ceilometer | 20:41 | |
| *** dhellmann_ has quit IRC | 20:48 | |
| *** dhellmann_ has joined #openstack-ceilometer | 20:48 | |
| *** lsmola__ has joined #openstack-ceilometer | 20:54 | |
| *** dhellmann has quit IRC | 20:54 | |
| *** openstackgerrit has quit IRC | 20:54 | |
| *** lsmola_ has quit IRC | 20:54 | |
| *** dhellmann_ is now known as dhellmann | 20:54 | |
| *** nati_ueno has quit IRC | 20:56 | |
| *** dhellmann has quit IRC | 20:56 | |
| *** dhellman_ is now known as dhellmann | 20:56 | |
| *** rbowen has quit IRC | 21:01 | |
| *** nati_ueno has joined #openstack-ceilometer | 21:05 | |
| *** openstackgerrit has joined #openstack-ceilometer | 21:08 | |
| eglynn | DinaBelova: did you see this gate failure on the hbase patch? | 21:13 |
| eglynn | http://logs.openstack.org/49/87249/9/check/gate-ceilometer-python27/c638b6c/console.html | 21:13 |
| *** nati_ueno has quit IRC | 21:15 | |
| gordc | eglynn: stop working! :) | 21:18 |
| eglynn | gordc: :) | 21:18 |
| gordc | i can reproduce it locally though so it's real. just fyi. | 21:18 |
| eglynn | cool, thanks, good info | 21:18 |
| openstackgerrit | Matt Riedemann proposed a change to openstack/ceilometer: Revert "Fix the floatingip pollster" https://review.openstack.org/99212 | 21:26 |
| *** jaypipes has quit IRC | 21:26 | |
| *** changbl has quit IRC | 21:28 | |
| *** changbl has joined #openstack-ceilometer | 21:29 | |
| *** jaypipes has joined #openstack-ceilometer | 21:29 | |
| *** promulo has quit IRC | 21:31 | |
| *** nati_ueno has joined #openstack-ceilometer | 21:35 | |
| *** jdob has quit IRC | 21:36 | |
| *** jaypipes has quit IRC | 21:37 | |
| *** Ruetobas has quit IRC | 21:41 | |
| *** thomasem has quit IRC | 21:46 | |
| *** eglynn has quit IRC | 21:48 | |
| *** Ruetobas has joined #openstack-ceilometer | 21:49 | |
| *** Ruetobas has quit IRC | 21:53 | |
| *** dhellmann has quit IRC | 21:55 | |
| *** dhellmann has joined #openstack-ceilometer | 21:55 | |
| *** Ruetobas has joined #openstack-ceilometer | 22:00 | |
| *** changbl has quit IRC | 22:00 | |
| *** skelpter has quit IRC | 22:07 | |
| *** gordc has quit IRC | 22:14 | |
| *** nati_ueno has quit IRC | 22:14 | |
| *** thomasem has joined #openstack-ceilometer | 22:31 | |
| *** nati_ueno has joined #openstack-ceilometer | 22:32 | |
| *** dmsimard is now known as dmsimard_away | 22:43 | |
| *** KurtRao has quit IRC | 23:00 | |
| *** KurtRao has joined #openstack-ceilometer | 23:00 | |
| *** fnaval has quit IRC | 23:00 | |
| *** fnaval has joined #openstack-ceilometer | 23:01 | |
| openstackgerrit | A change was merged to openstack/ceilometer: Remove low-value logging from publication codepath https://review.openstack.org/99147 | 23:05 |
| *** fnaval has quit IRC | 23:05 | |
| *** nati_ueno has quit IRC | 23:10 | |
| *** kun_huang has joined #openstack-ceilometer | 23:41 | |
| *** kun_huang has quit IRC | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!