21:01:04 #startmeeting Ceilometer 21:01:04 Meeting started Wed Jan 15 21:01:04 2014 UTC and is due to finish in 60 minutes. The chair is jd__. Information about MeetBot at http://wiki.debian.org/MeetBot. 21:01:05 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 21:01:07 The meeting name has been set to 'ceilometer' 21:01:12 #link https://wiki.openstack.org/wiki/Meetings/Ceilometer 21:01:14 o/ 21:01:17 hello 21:01:24 o/ 21:01:27 \o 21:01:31 o/ 21:01:32 o/ 21:01:33 o/ 21:01:42 o/ 21:02:00 hi everyone 21:02:03 o/ 21:02:05 o/ 21:02:14 o/ 21:02:17 #topic Milestone status icehouse-2 / icehouse-3 21:02:37 so icehouse-2 is going to be tagged next week, and bad news, I had to postpone a large portion of our blueprints 21:02:55 #link https://launchpad.net/ceilometer/+milestone/icehouse-2 21:03:02 good progress on https://blueprints.launchpad.net/ceilometer/+spec/rates-derived-from-cumulative 21:03:06 should land in time 21:03:12 we definitely need all the code proposed by the end of the week now 21:03:21 and we should take seriously some time to review 21:03:29 Yup. 21:03:56 I'm hopefull I'll have https://blueprints.launchpad.net/ceilometer/+spec/exclude-low-sample-count in a position to bump back from i-3 to i-2 by EoD tmrw 21:04:06 we have a larger codebase for complex query: https://review.openstack.org/#/q/status:open+project:openstack/ceilometer+branch:master+topic:bp/complex-filter-expressions-in-api-queries,n,z 21:04:14 as for icehouse-3, I'll have probably to cut some blueprint out as it seems unrealistic we implement everything in time at this stage 21:04:20 #link https://launchpad.net/ceilometer/+milestone/icehouse-3 21:04:39 I'll be in review-mode for a few days once I get that done 21:04:54 I will finish the doc by the end of this week, that will be the last patch, all the other part is ready, except one tiny python 2.6 issue in tests for the last patch 21:04:57 o/ 21:04:58 https://blueprints.launchpad.net/ceilometer/+spec/notification-pipelines should be early i-3 21:05:19 jd__: which blueprints are we looking to drop? 21:05:26 gordc: I don't know yet 21:05:31 gordc: though suggestions are welcome 21:06:00 jd__: is https://blueprints.launchpad.net/oslo/+spec/service-sync likely to be done by early/mid i-3 cycle? 21:06:06 jd__: cool cool, i can't really speak for any of them since i'm not assigned to any. guess we can follow up with assignees. 21:06:27 * eglynn asks as https://blueprints.launchpad.net/ceilometer/+spec/rebase-alarm-partition-coordination depends on it 21:06:29 eglynn: part of it, but while it progresses, it still progresses slowly 21:06:42 jd__: cool enough 21:06:57 harlowja_away is helping yassine on tooz, but it definitely takes time 21:07:10 I suggest focusing reviews on changes associated with blueprints 21:07:19 and leaving anything else for after the deadline 21:07:22 dhellmann: +1 21:07:38 I think this BP is done, fyi: https://blueprints.launchpad.net/ceilometer/+spec/specify-event-api 21:07:43 dhellmann: +1 21:08:09 dhellmann: ... and i-2 bugs too, there are still 4 outstanding 21:08:11 dhellmann: +1 21:08:28 eglynn: yep 21:08:33 eglynn: +1 21:09:05 are all 4 of https://bugs.launchpad.net/bugs/1252988 https://bugs.launchpad.net/bugs/1244182 https://bugs.launchpad.net/bugs/1240474 https://bugs.launchpad.net/bugs/1260176 likely to land in time? 21:09:07 herndon: great 21:09:08 Launchpad bug 1252988 in ceilometer "counter_volume did not represent duration for instance,network,port, subnet, router, " [Low,Confirmed] 21:09:30 (... otherwise we should prolly think about bumping to i-3 also) 21:09:46 ... i-3 is gonna be a chunky one ;) 21:10:00 eglynn: I don't think so, I should remove myself from those 21:10:19 jd__: k 21:10:31 I hope 1240474 will be approved 21:10:41 patch for bug 1260176 was already there for review 21:10:42 Launchpad bug 1260176 in ceilometer "different behavior of alarm creation on different DB backend" [Undecided,In progress] https://launchpad.net/bugs/1260176 21:10:46 llu-laptop, will this land in I2 https://blueprints.launchpad.net/ceilometer/+spec/support-resources-pipeline-item ? 21:11:01 unfortunately wanting to fix a bug and assigning it to me doesn't make me more time to do it :( 21:11:27 if anyone else has bandwidth to grab one of those first two bugs mentioned (1252988 & 1244182), please knock yourself out! 21:11:33 lsmola we need to review I think 21:11:40 lsmola: patches are already there. Just got some new reviewes, will address them today 21:11:52 llu-laptop: it's on my list to review 21:11:55 llu-laptop, excellent 21:11:59 llu-laptop: (will do so tmrw) 21:11:59 clearly if you have spare time don't hesitate to ask for anything to do from bugs and blueprints :) 21:12:23 dkranz also wants us to check https://bugs.launchpad.net/ceilometer/+bug/1268730 21:12:24 Launchpad bug 1268730 in ceilometer "ERRORs in ceilometer-acentral log after succesful tempest run" [High,Triaged] 21:12:39 jd__, thanks, would be nice to have it, so we can implement it to tuskar in i3 21:12:41 gordc started to do so, but we need to fix it somewhere 21:13:01 jd__: i think i might have a solution for that one. not sure if it's the right solution... but it's a solution. 21:13:03 I may spend time tomorrow on it if nobody does as it seems important 21:13:14 gordc: please comment on the bug :) 21:13:28 jd__: will do. 21:14:44 #topic Tempest integration 21:14:52 any news on that nadya_ ? 21:15:06 We still have not merged agent https://review.openstack.org/#/c/55276/ 21:15:16 the main update :) 21:15:23 any blocker, or just missing reviews? 21:15:37 no blockers, need review 21:15:41 great :) 21:15:56 anything to add on that anyone? 21:15:59 dkranz has +2'd it ... which is a good start 21:16:00 and we started to implement https://review.openstack.org/#/c/66551/ 21:16:13 who else is a good tempest core to hit up fro reviews? 21:16:18 to create tests for pollsters 21:16:19 s/fro/for/ 21:16:50 we had even +2 from Sean Dague 21:17:22 and one more thing 21:17:31 nadya_: you may have to bust out your thumb-screws and pressurize the tempest cores into doing reviews a bit quicker 21:17:42 I would ask sdague but know that he is in Montreal at the tempest/neutron thing 21:18:00 now we have updated client and several reviews are needed to be updated 21:18:21 but all of them became abandoned 21:18:48 I'm afraid that people who started implementing tests decided to refuse from this ides 21:18:52 nadya_: There was a lot of confusion about what was ready for review. 21:19:18 nadya_: I think it would help to send a message to the qa list identifying the reviews that are needed now, and ask 21:19:19 dkranz, actually nothing else is ready as I know( 21:19:35 nadya_: I assumed from the complaint about speedy reviews... 21:20:06 The tempest team agreed two weeks ago to give high priority to heat and ceilometer tests 21:20:41 dkranz: the point about tempest review turnaround speed may just have been /me jumping to conclusions 21:20:59 (... so apols for casting aspertions!) 21:21:05 eglynn: ok :) 21:21:08 ok, so if I have no responses from change-request owners we will add our tests that were deleted from gerrit-reviews 21:21:32 eglynn: After the attempts to review things that were not ready, we were waiting for some kind of go-ahead as to which were ready 21:21:52 I think there will be fast response when that happens 21:21:57 dkranz: understood, thanks! 21:22:23 I think we need at least a small set of tests for i2, dkranz, is it possible yet? or too late? 21:23:10 nadya_: I don't know what the i2 date is but we should be able to review in a day once we are pointed to something. 21:23:30 THe gerrit ui makes it really hard to "find the ceilo patches" 21:23:42 we had several ones on review along with client but deleted because of duplication 21:23:54 yep, that's the truth :( 21:23:58 The tempest core team is a follow-the-sun group :) 21:24:36 dkranz, ok, so we will keep in touch :) I think we may move on 21:24:49 dkranz: https://review.openstack.org/#/q/status:open+project:openstack/ceilometer,n,z ? 21:25:22 (but, yah, the gerrit ui... could use work :P) 21:25:28 dragondm: I meant in tempest... 21:25:38 dkranz: Ah, IC. 21:26:06 dragondm: We need the latest version of gerrit which supports filename search 21:26:14 project:openstack/tempest branch:master topic:bp/add-basic-ceilometer-tests 21:26:14 But no timetable for that yet. 21:26:35 dkranz: yah. Some ability for extra tagging might be nice too :P 21:27:02 https://review.openstack.org/#/q/project:openstack/tempest+branch:master+topic:bp/add-basic-ceilometer-tests,n,z to be more hyperlinked 21:27:16 jd__, ping :) 21:27:28 nadya_: ? 21:27:39 ah you're good sorry :) 21:27:55 #topic Release python-ceilometerclient? 21:28:01 herndon: anything else needed other than https://github.com/openstack/python-ceilometerclient/commit/c74231e7 ? 21:28:10 (... I thought thomasem might have something in the works around boxing off traits in the CLI output) 21:28:19 (... if not, I'll just go ahead and cut ceiloclient 1.0.9) 21:28:39 eglynn, Nope, nothing new. Only what was already in John's patch. :) 21:28:50 great then :) 21:29:00 k, 1.0.9 1st thing tmrw then! 21:29:09 err, one sec 21:29:18 herndon: yep? 21:29:35 current implementation is fine, but there is a bug in the event api that I am working on fixing, will require a new cut of the client. 21:29:53 I think it's ok to release it now, but there is one more patch coming :( 21:29:55 a-ha, k, wanna wait for that so? 21:30:09 https://review.openstack.org/#/c/66936/ 21:30:26 I don't really care, up to you 21:30:42 I was in the middle of writing the patch when I this meeting started 21:30:44 herndon: k, I'll review the patch and if it looks like we can get it landed quickly then I'll wait 21:31:03 that patch is to fix the server, still working on the client 21:31:25 ok 21:31:34 #topic Open discussion 21:31:36 herndon: fair enough, what's the approx ETA on the client patch? 21:31:45 (... if the next day or 2, lets wait for it) 21:31:45 tomorrow 21:31:57 herndon: k, let's wait on that so 21:32:02 herndon: thanks! 21:32:04 if you got anything else to bring, now is the moment folks 21:32:31 apols, /me gotta run soon-ish ... 21:32:51 @jd__, if you can assign this blueprint to me, that will be great. 21:32:53 https://blueprints.launchpad.net/ceilometer/+spec/alarm-on-notification 21:34:07 tongli: done! 21:34:14 @jd__, thanks. 21:34:32 I'm working on aggregation blueprint and will create a first version of MySQL stuff. Want to invite everybode to mailing list [Ceilometer] Aggregation discussion 21:35:53 *will create on this week. because HBase implementation is hard to be tried 21:37:17 and we had a brainstorm about flattening metadata. but Im not ready to tell you everything in 2 a.m. :) 21:38:14 hehe 21:38:19 we can't blame you :) 21:38:34 the main question for me: should we resolve https://bugs.launchpad.net/ceilometer/+bug/1268628 or not 21:38:35 Launchpad bug 1268628 in ceilometer "recursive_keypairs doesn't throw 'separator' param to next iteration" [Undecided,New] 21:39:48 nadya_: will try to add some comments later today 21:40:00 thanks gordc! 21:40:59 time to wrap up? 21:41:14 may I try to ask? 21:42:19 I've changed my mind, will ask in our channel, too technical question :) 21:42:27 your call! :) 21:42:31 #endmeeting