Tuesday, 2014-03-04

*** raymondr_ has quit IRC00:00
*** lakspace has quit IRC00:10
*** lakspace has joined #openstack-ceilometer00:12
*** julim has quit IRC00:14
*** fnaval has quit IRC00:22
*** prad has joined #openstack-ceilometer00:26
*** rwsu has quit IRC00:27
*** rwsu has joined #openstack-ceilometer00:30
*** ryanpetrello has quit IRC00:31
*** ddutta has joined #openstack-ceilometer00:32
*** ryanpetrello has joined #openstack-ceilometer00:32
*** jdob has quit IRC00:34
*** jdob has joined #openstack-ceilometer00:35
*** jdob has quit IRC00:40
*** dhellmann is now known as dhellmann_00:43
*** openstackgerrit has quit IRC00:44
*** openstackgerrit has joined #openstack-ceilometer00:44
*** ryanpetrello has quit IRC00:44
*** nati_ueno has joined #openstack-ceilometer00:46
*** nati_uen_ has quit IRC00:49
*** nati_ueno has quit IRC00:51
*** ryanpetrello has joined #openstack-ceilometer00:53
*** ryanpetrello has quit IRC00:54
*** flwang has quit IRC00:56
*** nati_ueno has joined #openstack-ceilometer00:57
*** ddutta has quit IRC01:02
*** nati_ueno has quit IRC01:09
*** nati_ueno has joined #openstack-ceilometer01:10
*** nati_ueno has quit IRC01:10
*** nati_ueno has joined #openstack-ceilometer01:11
*** nati_ueno has quit IRC01:12
*** nati_ueno has joined #openstack-ceilometer01:12
*** nati_ueno has quit IRC01:14
*** nati_ueno has joined #openstack-ceilometer01:15
*** nati_ueno has quit IRC01:16
*** nati_ueno has joined #openstack-ceilometer01:16
*** nati_ueno has quit IRC01:17
*** rwsu has quit IRC01:17
*** nati_ueno has joined #openstack-ceilometer01:17
*** nati_ueno has quit IRC01:18
*** nati_ueno has joined #openstack-ceilometer01:19
*** nati_ueno has quit IRC01:22
*** nati_ueno has joined #openstack-ceilometer01:22
*** nati_ueno has quit IRC01:23
*** nati_ueno has joined #openstack-ceilometer01:23
*** nati_ueno has quit IRC01:24
*** nati_ueno has joined #openstack-ceilometer01:25
*** nati_ueno has quit IRC01:26
*** nati_ueno has joined #openstack-ceilometer01:26
*** nati_uen_ has joined #openstack-ceilometer01:27
*** nati_ueno has quit IRC01:27
*** nati_uen_ has quit IRC01:28
*** nosnos has joined #openstack-ceilometer01:29
*** nati_ueno has joined #openstack-ceilometer01:29
*** nati_ueno has quit IRC01:30
*** nati_ueno has joined #openstack-ceilometer01:30
*** nati_ueno has quit IRC01:31
*** nati_ueno has joined #openstack-ceilometer01:31
*** nati_ueno has quit IRC01:32
*** xianghui has joined #openstack-ceilometer02:04
openstackgerritlitong01 proposed a change to openstack/python-ceilometerclient: add notification alarm  https://review.openstack.org/6947402:08
openstackgerritlitong01 proposed a change to openstack/ceilometer: add notification alarm  https://review.openstack.org/6947302:08
*** ryanpetrello has joined #openstack-ceilometer02:14
openstackgerritliusheng proposed a change to openstack/python-ceilometerclient: Correct help string of alarm-craete  https://review.openstack.org/7776102:35
*** flwang has joined #openstack-ceilometer02:47
*** marcoemorais has quit IRC02:49
*** raymondr has joined #openstack-ceilometer02:55
*** ryanpetrello has quit IRC02:55
*** nati_ueno has joined #openstack-ceilometer02:59
*** nati_ueno has quit IRC02:59
*** fnaval has joined #openstack-ceilometer03:38
*** annegentle has quit IRC03:42
*** raymondr has quit IRC04:44
*** nati_ueno has joined #openstack-ceilometer04:45
*** _nadya_ has joined #openstack-ceilometer04:46
*** raymondr has joined #openstack-ceilometer04:46
*** AMike has joined #openstack-ceilometer04:58
*** AMike has joined #openstack-ceilometer04:58
*** _nadya_ has quit IRC05:21
*** shakayumi has joined #openstack-ceilometer05:30
openstackgerritPiyush Masrani proposed a change to openstack/ceilometer: VMware vSphere support: Memory  https://review.openstack.org/7470205:31
*** sayalilunkad has joined #openstack-ceilometer05:56
*** raymondr has quit IRC05:59
*** marcoemorais has joined #openstack-ceilometer06:01
openstackgerritJenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex  https://review.openstack.org/7676206:03
*** _nadya_ has joined #openstack-ceilometer06:16
*** _nadya_ has quit IRC06:23
*** flwang has quit IRC06:58
*** bogdando has joined #openstack-ceilometer07:01
*** saju_m has joined #openstack-ceilometer07:12
*** flwang has joined #openstack-ceilometer07:17
*** sayalilunkad has quit IRC07:20
openstackgerritYuuichi Fujioka proposed a change to openstack/ceilometer: Implements monitoring-network-from-opendaylight  https://review.openstack.org/6389007:21
*** lsmola has joined #openstack-ceilometer07:23
*** nosnos has quit IRC07:31
*** nosnos_ has joined #openstack-ceilometer07:31
openstackgerritEoghan Glynn proposed a change to openstack/ceilometer: Decouple source and sink configuration for pipelines  https://review.openstack.org/7545907:35
openstackgerritEoghan Glynn proposed a change to openstack/ceilometer: Per pipeline pluggable resource discovery  https://review.openstack.org/7597007:35
*** nosnos_ has quit IRC08:04
*** nosnos has joined #openstack-ceilometer08:04
*** taplax has joined #openstack-ceilometer08:06
*** ildikov_ has joined #openstack-ceilometer08:14
*** saju_m has quit IRC08:26
*** julienvey_ has joined #openstack-ceilometer08:28
*** sayalilunkad has joined #openstack-ceilometer08:33
*** flwang has quit IRC08:54
*** xianghui has quit IRC08:54
*** xianghui has joined #openstack-ceilometer08:57
*** flwang has joined #openstack-ceilometer09:00
*** amichon has joined #openstack-ceilometer09:06
amichonhi, I created a bp : https://blueprints.launchpad.net/ceilometer/+spec/handle-keystone-identity-notifications09:07
amichonand I dont't know how this can be reviewed ?09:08
amichonshould I fix a next miletone ? which ?09:08
*** yassine has joined #openstack-ceilometer09:09
*** xianghui has quit IRC09:12
*** xianghui has joined #openstack-ceilometer09:12
*** xianghui has quit IRC09:18
*** marcoemorais has quit IRC09:19
openstackgerritEoghan Glynn proposed a change to openstack/ceilometer: Wider selection of aggregates for mongodb  https://review.openstack.org/7687909:21
openstackgerritEoghan Glynn proposed a change to openstack/ceilometer: Wider selection of aggregates for sqlalchemy  https://review.openstack.org/7627809:21
*** flwang has quit IRC09:24
*** _nadya_ has joined #openstack-ceilometer09:24
*** eglynn has joined #openstack-ceilometer09:27
*** _nadya_ has quit IRC09:30
*** xianghui has joined #openstack-ceilometer09:30
*** saju_m has joined #openstack-ceilometer09:42
eglynnsileht: can you re-approve https://review.openstack.org/76273 ? ... assuming "reverify no bug" is still disabled09:44
*** krtaylor has quit IRC09:44
eglynn(just a spurious-looking tempest neutron failure, but holding up the whole patch series)09:44
silehteglynn, done09:45
eglynnsileht: thank you sir!09:45
silehteglynn, thx for the "reverify no bug" workaround, I was not knew this one :)09:45
eglynnsileht: ... normally to be used sparingly, but today is the day for "bending the rules" me thinks ;)09:47
silehteglynn, sure09:47
eglynnsileht, jd__: so I was harassing the olso-messaging cores in RH to give some love to those patches needed for the ceilo switchover09:52
eglynnI see https://review.openstack.org/61675 & https://review.openstack.org/77136 have landed09:52
eglynnbut not https://review.openstack.org/75365 ;(09:52
silehteglynn, thanks a lot for that !09:52
eglynnsileht: np! I wonder could we possibly live without that last one?09:52
silehteglynn, I have wrote a piece of code without this one, but I'm not sure it works, and it imports private stuffs of oslo.messaging...09:55
silehteglynn, but this one will fix and close before oslo.messaging 1.3.0 will release, I have opened it  as a bug and it was marked high for icehouse09:56
eglynnsileht: is that a new bug that needs to be fixed before 1.3.0 is released?09:58
silehteglynn, yes09:58
eglynnsileht: what's that bug number?10:00
silehthttps://bugs.launchpad.net/oslo.messaging/+bug/128263910:01
eglynnsileht: a-ha, ok, so that's the bug for the re-connection review I mentioned above ... cool, got it10:02
eglynnsileht: so IIUC you've got a potential work-around for 1282639 that "bends the rules" somewhat? (i.e. imports oslo-messaging innards)10:05
eglynnsileht: (... I'm wondering could we get markmc to defer LP 1282639 to the next release and simply rely on your work-around for now)10:06
*** sayalilunkad has quit IRC10:06
*** shakayumi has quit IRC10:07
silehteglynn, I will split the review10:08
eglynnsileht: cool, thanks10:08
eglynnsileht, jd__: ... so given the progress on the oslo-messaging side, what's the thought on possibly begging an FFE for the ceilometer switchover?10:11
*** nati_ueno has quit IRC10:12
*** piyushmasrani has joined #openstack-ceilometer10:12
eglynn... /me is fearing that ceilo will end up as an outlier if stuck on the old RPC layer10:13
jd__eglynn: I don't think it's a good idea – and I'm the one that likes to break thing10:15
jd__but it's a very important part of Ceilometer and I'm afraid to merge that code /that/ late in the process10:15
eglynnjd__: fair enough, your call as PTL10:17
jd__eglynn: yeah :(10:17
jd__I feel obliged to release something that's not going potentially totally borked and not tested10:17
silehteglynn, jd__ switching to oslo.messaging doesn't add new features in ceilometer for now, this is just a requirements to code new features around the rpc10:17
silehteglynn, so having it the first day of Juno is enought10:18
eglynnsileht, jd__: yeah I'm more concerned with the maintainability of icehouse-derived distros than any new features10:18
eglynnsileht, jd__: ... so I suspect that we'll end up with Icehouse-derived distros carrying additional patches10:19
jd__eglynn: well as sileht feature wide everything is the same10:19
jd__+said10:19
eglynnjd__, sileht: so to sum up ...10:24
*** sayalilunkad has joined #openstack-ceilometer10:24
eglynnwe're unwilling to incur the risk at this late stage due to oslo-messaging not being quite ready to meet the ceilo requirements10:24
eglynn... and there being insufficient time for this to be tested fully pre-icehouse10:24
eglynn... but we're pushing ahead to get the switch-over finalized early in the Juno cycle10:25
eglynnjd__, sileht: fair enough summation ^^^?10:25
jd__eglynn: yes!10:25
silehtyep10:26
eglynnfair enough, thanks chaps!10:26
silehtvery early! Day one !10:26
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Use oslo.msg retry API in rpc publisher  https://review.openstack.org/7784510:37
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Consume notif. from multiple message bus  https://review.openstack.org/7761210:37
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Use NotificationPlugin as an oslo.msg endpoint  https://review.openstack.org/7761110:37
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Replace oslo.rpc by oslo.messaging  https://review.openstack.org/5745710:37
*** flwang has joined #openstack-ceilometer10:47
*** xianghui has quit IRC10:59
*** _nadya_ has joined #openstack-ceilometer11:09
*** taplax has quit IRC11:20
*** taplax has joined #openstack-ceilometer11:32
taplax.sms hd OSPR raportointi korjauksessa...11:33
*** julienvey_ has quit IRC11:41
*** _nadya_ has quit IRC11:50
*** yfujioka has quit IRC11:54
*** amichon has quit IRC11:57
nprivalovayesterday nova-notifications worked ok at least on postgress. Today it doesn't work at all :(11:58
*** heyongli has joined #openstack-ceilometer11:59
*** _nadya_ has joined #openstack-ceilometer11:59
*** _nadya_ has quit IRC12:04
nsaje_nprivalova: such are the woes of developers :)12:04
nprivalovansaje_: I'm trying to add tempest tests and have no idea how to move on...Every day create env as on gating and debug nova sounds very bad12:08
*** ekarlso has quit IRC12:09
*** amichon has joined #openstack-ceilometer12:23
*** _nadya_ has joined #openstack-ceilometer12:25
*** _nadya_ has quit IRC12:31
*** julienvey_ has joined #openstack-ceilometer12:35
*** saju_m has quit IRC12:40
openstackgerritNejc Saje proposed a change to openstack/ceilometer: Adds time constraints to alarms  https://review.openstack.org/7539112:41
openstackgerritPiyush Masrani proposed a change to openstack/ceilometer: VMware vSphere support: Memory Usage  https://review.openstack.org/7470212:47
*** _nadya_ has joined #openstack-ceilometer12:50
*** saju_m has joined #openstack-ceilometer12:52
*** Alexei_9871 has joined #openstack-ceilometer12:59
*** _nadya_ has quit IRC13:02
*** saju_m has quit IRC13:04
*** _nadya_ has joined #openstack-ceilometer13:06
*** _nadya_ has quit IRC13:08
openstackgerritPiyush Masrani proposed a change to openstack/ceilometer: VMware vSphere: Changes for cpu_util  https://review.openstack.org/7617713:10
silehtnsaje_, you got a last comment from eglynn13:10
*** dhellmann_ is now known as dhellmann13:11
nsaje_sileht: thanks, on it13:13
*** sayalilunkad has quit IRC13:13
nsaje_eglynn: thanks, that was a too-quick fix since python2.6 lacks total_seconds :(13:13
eglynnnsaje_: yeah, I remember now ... the ceilometer.openstack.common.timeutils.delta_seconds() method handles that aspect13:14
nsaje_I see now, didn't know it was there, great13:15
openstackgerritNejc Saje proposed a change to openstack/ceilometer: Adds time constraints to alarms  https://review.openstack.org/7539113:19
*** lsmola has quit IRC13:23
*** ekarlso has joined #openstack-ceilometer13:23
eglynnjd__: when you get a chance, could you give https://review.openstack.org/75459 & https://review.openstack.org/75970 another look?13:23
*** sayalilunkad has joined #openstack-ceilometer13:24
silehtnsaje_, thx !13:26
nsaje_sileht: np, hope it makes it through the gate on 1st try :)13:31
openstackgerritA change was merged to openstack/ceilometer: Add simple capabilities API  https://review.openstack.org/7627313:34
openstackgerritA change was merged to openstack/ceilometer: Selectable aggregation functions for statistics  https://review.openstack.org/7045413:34
openstackgerritA change was merged to openstack/ceilometer: Selectable aggregate support in mongodb  https://review.openstack.org/7687813:35
*** dhellmann is now known as dhellmann_13:36
*** lsmola has joined #openstack-ceilometer13:36
eglynn\o/ the gate is gating :)13:40
*** krtaylor has joined #openstack-ceilometer13:45
ildikov_eglynn: +1 for the happiness :)13:55
ildikov_nsaje_:ping13:55
nsaje_ildikov_: hey13:56
ildikov_did you try to build the doc?13:56
*** nosnos has quit IRC13:56
ildikov_nsaje_: ^^13:56
*** gordc has joined #openstack-ceilometer13:56
ildikov_nsaje_: I mean the time constraints patch13:56
nsaje_ildikov_: huh, no, by the time I finished fixing things it slipped my mind13:57
ildikov_nsaje_: sphinx does not like the time_constraint in the Alarm's sample13:57
nsaje_ildikov_: why is that?13:57
ildikov_nsaje_: my env is not in the best shape right now, but I assume that if you insert a sample there by hand it should work13:58
ildikov_nsaje_: I'm not sure about the reason it somehow ends up in the __init__ function of the Alarm and then it fails with a mapping issue13:58
ildikov_nsaje_: I had similar problems with the AlarmThresholdRule, but it turned out finally that the AlarmCombinationRule works fine with sphinx, so now it is included in the sample not the threshold rule as only one can be defined at a time...13:59
nsaje_ildikov_: I'll check it out, thanks13:59
ildikov_nsaje_: when I replaced the [AlarmTime....sample()] there with None, the build succeeded, so maybe it will work with a static example14:01
*** heyongli has quit IRC14:01
openstackgerritPiyush Masrani proposed a change to openstack/ceilometer: Changes for networking metrics support for vSphere  https://review.openstack.org/7735514:02
ildikov_nsaje_: I did not have time to try it, I thought it is better to inform you about this problem sooner than later14:03
nsaje_ildikov_: thanks a lot, I'll take care of it14:04
ildikov_nsaje_: np!14:06
*** annegentle has joined #openstack-ceilometer14:09
nsaje_ildikov_: fixed, thanks!14:15
openstackgerritNejc Saje proposed a change to openstack/ceilometer: Adds time constraints to alarms  https://review.openstack.org/7539114:15
nsaje_sileht, eglynn : sorry about this, can you guys check time constraints again?14:15
eglynnnsaje_: sure!14:15
silehtnsaje_, sure14:16
eglynnnsaje_: haha, bitten by the sphinx right? ;)14:16
nsaje_eglynn: yep14:17
*** julim has joined #openstack-ceilometer14:17
ildikov_nsaje_: /me thanks for the quick fix!14:19
ildikov_eglynn: I think you were the only one, who's patch did not fail on the gate with that Error in the sphinx build ;)14:20
eglynnildikov_: phew! ;)14:21
*** lsmola has quit IRC14:24
*** amichon has quit IRC14:26
ildikov_eglynn: sorry about mentioning it, but it was a so unforgettable moment ;)14:27
eglynnildikov_: it made my day so far ;)14:28
*** terriyu has joined #openstack-ceilometer14:31
*** lsmola has joined #openstack-ceilometer14:33
*** lsmola has quit IRC14:33
ildikov_eglynn: I just got a nice rebase task from you not so long ago, just to have a nice day :)14:34
*** lsmola has joined #openstack-ceilometer14:34
openstackgerritEoghan Glynn proposed a change to openstack/ceilometer: Wider selection of aggregates for mongodb  https://review.openstack.org/7687914:34
openstackgerritEoghan Glynn proposed a change to openstack/ceilometer: Wider selection of aggregates for sqlalchemy  https://review.openstack.org/7627814:34
*** changbl has quit IRC14:35
eglynnildikov_: oddly enough I got a rebase-related merge failure on https://review.openstack.org/76278 but yet didn't seem to be any actual conflicts on the rebase14:36
eglynnweird14:36
eglynnperhaps the smart-chaining of builds in the gate can be a little dumb at times ;)14:37
ildikov_eglynn: my code refactor patch was shown that depends on this one you've just copied here14:38
eglynnmeh!14:39
ildikov_eglynn: I had no clue what made the gate believe this, maybe they had a common dependency, or something like, as my patch is in conflict with your previous merged ones14:39
eglynnthe "Gerrit trivial rebase detection script" restores a +2 & +1 with a pair of +2s14:40
eglynn... if the approving reviewer previously gave a +2 on a prior patch version :(14:40
eglynn... nasty!14:40
eglynnsileht: can you morph your +2 to a +1 on https://review.openstack.org/76278 & https://review.openstack.org/76879 ?14:42
eglynnsileht: (... as gerrit has morphed llu's +1 to +2)14:42
ildikov_eglynn: it would have been too easy if we have a deadline without gate and gerrit issues...14:43
silehteglynn, gerrit rebase script just remove the 'Approved +1'14:43
*** sayalilunkad has quit IRC14:43
silehteglynn, both fixed14:43
eglynnildikov_: yeah, it keeps the adrenaline flowing ;)14:43
eglynnsileht: thank you sir!14:43
*** thomasem has joined #openstack-ceilometer14:44
ildikov_eglynn: do I think right that these two patch set of yours should not be in merge conflict with my refactor?14:46
eglynnildikov_: dbs/mongo code de-dupe refactoring? ... checking now14:47
eglynn*db214:47
*** taplax has quit IRC14:49
ildikov_eglynn: yes, I talked about the third patch set, which contains only the record_alarm_change and alarm histort query functions from mongo to the pymongo_base14:49
eglynnildikov_: yeah there's a rejected hunk14:50
eglynnhttp://fpaste.org/82198/94458713/14:50
*** lsmola has quit IRC14:50
eglynnildikov_: (because the line above has changed, not your alarm changes themselves conflicting)14:51
eglynnildikov_: this line: (models.Statistics(**(r['value'])) for r in results['results']),14:51
eglynnildikov_: ... the unrelated line above is used by git to anchor your mods to get_alarm_changes14:52
ildikov_eglynn: which patch sets are we talking about now?14:54
ildikov_eglynn: sorry, I'm a bit lost as we have several ones :)14:54
eglynnildikov_: this puppy https://review.openstack.org/7581114:54
ildikov_eglynn: I had a merge conflict because of the already merged patch sets, as your modifications were around mines14:54
eglynnildikov_: so your dedupe patch seemed to "conflict" with 2 of mine https://review.openstack.org/76278 & https://review.openstack.org/76879 that *also* were rejected by the gate for a supposed conflict14:56
eglynnildikov_: by just manually applying that rejected hunk above, I can rebase your onto mine no prob14:56
eglynnildikov_: (... that would one way of ensuring that "conflict" doesn't recur)14:57
ildikov_eglynn: yes, my little puppy is already rebased, I wanted to check your puppies, if these two cute one will be still in conflict with mine or not as it seemed that they should not be any more14:58
eglynnildikov_: so you've rebased onto master latest OR onto https://review.openstack.org/76278 & https://review.openstack.org/76879?14:59
ildikov_eglynn: ihad to rebase this patch because of your already merged patch sets, the API capabilities and the other mongo related one14:59
eglynnildikov_: a-ha I see14:59
ildikov_eglynn: I rebased to master14:59
ildikov_eglynn: or at least as a first step I rebased to master :)14:59
eglynnildikov_: yep, that should be fine ... as the conflict was in an already-merged patch (for a sec, I thought the conflict was in my as-yet-unmerged patches)15:00
eglynnildikov_: ... /me states the obvious yet again ;)15:00
ildikov_eglynn: is it a solution to rebase my patch set above yours now just to be sure or it would be unnecessary?15:02
eglynnildikov_: no, you're good now having rebased onto master ... your's is completely independent of my unmerged patches (I muddied the waters earlier, apols!)15:03
ildikov_eglynn: your previous ones should have been also completly independent, but true that I have the same feeling regarding to the unmerged ones ;)15:04
ildikov_eglynn: np, it is better to talk about a little bit more, but be prepared for any options :)15:05
openstackgerritIldiko Vancsa proposed a change to openstack/ceilometer: Remove code duplication Part 3  https://review.openstack.org/7581115:06
eglynnildikov_: ... yeah the changes were independent all right15:06
eglynnildikov_: ... but the "anchor line" for one of your hunks was touched by one of my (now merged) patches15:06
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Consume notif. from multiple message bus  https://review.openstack.org/7761215:07
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Use NotificationPlugin as an oslo.msg endpoint  https://review.openstack.org/7761115:07
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Use oslo.msg retry API in rpc publisher  https://review.openstack.org/7784515:07
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Replace oslo.rpc by oslo.messaging  https://review.openstack.org/5745715:07
ildikov_eglynn: yes, I wanted to say something similar like this, I just mentioned too many things at the same time I think15:08
ildikov_eglynn: anyway, if you're sure that the rest of your patch sets will not affect mine, than you have a chance to like my tiny little rebased puppy ;)15:08
eglynnildikov_: yeah I'm pretty sure my two bad boys won't elbow out your's again ;)15:10
*** jergerber has joined #openstack-ceilometer15:12
ildikov_eglynn: then you can help in getting my innocent little girl back to the gate in approved state ;)15:17
gordceglynn: is today feature freeze? having trouble remembering what day it is.15:20
*** fnaval has quit IRC15:21
eglynngordc: AFAIK today is the day that the milestone-proposed branch will be cut15:21
gordceglynn: i see... so is everything sitting in open status right now not in scope of Icehouse? trying to figure out what is in and what isn't.15:23
eglynngordc: IIUC anything that lands today will definitely get into i315:24
eglynngordc: ... whereas anything that lands on master after today, but before some point on thurs, can be backported to milestone-proposed with the aggreement of jd__ & ttx15:25
eglynngordc: ... https://wiki.openstack.org/wiki/GerritJenkinsGithub#Authoring_Changes_for_milestone-proposed15:25
* jd__ nods15:26
gordceglynn: cool cool. these dates are confusing me. i'll let you release guys handle it. :)15:26
*** tongli has joined #openstack-ceilometer15:29
*** lsmola has joined #openstack-ceilometer15:32
*** raymondr has joined #openstack-ceilometer15:32
openstackgerritAkhil Hingane proposed a change to openstack/ceilometer: VMware vSphere support: Disk rates  https://review.openstack.org/7657115:34
*** ryanpetrello has joined #openstack-ceilometer15:37
*** jmckind has joined #openstack-ceilometer15:37
*** Alexei_9871 has quit IRC15:41
eglynngordc: speaking of stuff landing for i3 today15:43
eglynngordc: ... if you've got bandwidth, could you cast your eye over the latest on https://review.openstack.org/75459 & https://review.openstack.org/75970 ?15:43
gordceglynn: sure. i'll take a look at that next.15:44
eglynngordc: thank you sir!15:44
gordcjd__: do you know what time  the branch is cut?15:45
sileht/1/1015:46
*** fnaval has joined #openstack-ceilometer15:48
*** ryanpetrello has quit IRC15:54
*** nati_ueno has joined #openstack-ceilometer15:57
eglynngordc: i3 milestone-proposed will be cut first thing tomorrow morning European time (modulo gate drainage)15:59
gordceglynn: cool cool. good to know.16:00
*** giroro_ has quit IRC16:01
*** Ruetobas has joined #openstack-ceilometer16:03
*** Ruetobas has quit IRC16:07
gordceglynn: i made a comment to https://review.openstack.org/#/c/75459 ... i'm ok with patch so let me know what you think of comment... whether you want to do it in another patch or disregard it.16:09
eglynngordc: ... looking16:09
*** dhellmann_ is now known as dhellmann16:12
*** Ruetobas has joined #openstack-ceilometer16:13
*** _nadya_ has joined #openstack-ceilometer16:22
ildikov_eglynn, gordc: if you have a few minutes and you feel like give this small patch another try (I had to rebase it), I would appreciate it: https://review.openstack.org/7581116:26
eglynnildikov_: looking16:27
gordcildikov_: done.16:28
ildikov_eglynn, gordc: thanks!16:30
*** shakayumi has joined #openstack-ceilometer16:32
*** changbl has joined #openstack-ceilometer16:35
eglynngordc: ... I've responded to your idea about moving support_meter in https://review.openstack.org/#/c/75459/16/ceilometer/pipeline.py16:35
eglynnsileht: can you pull that re-approve trick again on https://review.openstack.org/76278 ?16:37
*** julienvey_ has quit IRC16:38
*** sayalilunkad has joined #openstack-ceilometer16:38
gordceglynn: ok. taking a look.16:39
silehteglynn, I have already saw this bug ... I have asked a reverify16:42
eglynnsileht: thank you sir!16:43
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Consume notif. from multiple message bus  https://review.openstack.org/7761216:45
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Use NotificationPlugin as an oslo.msg endpoint  https://review.openstack.org/7761116:45
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Use oslo.msg retry API in rpc publisher  https://review.openstack.org/7784516:45
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Replace oslo.rpc by oslo.messaging  https://review.openstack.org/5745716:45
*** _cjones_ has joined #openstack-ceilometer16:52
*** _cjones_ has quit IRC16:55
*** _cjones_ has joined #openstack-ceilometer16:55
*** _nadya_ has quit IRC16:58
*** jdob has joined #openstack-ceilometer17:06
gordcsileht: you mentioned kombu requirement is required in oslo.messaging patch... does that mean this patch is valid (adding qpid requirement)? https://review.openstack.org/#/c/73083/17:10
*** mihgen has joined #openstack-ceilometer17:11
*** mihgen has joined #openstack-ceilometer17:11
*** nati_ueno has quit IRC17:13
*** rwsu has joined #openstack-ceilometer17:14
*** _nadya_ has joined #openstack-ceilometer17:20
*** marcoemorais has joined #openstack-ceilometer17:24
*** mihgen has quit IRC17:24
*** _nadya_ has quit IRC17:25
*** yassine has quit IRC17:28
*** terriyu has quit IRC17:42
*** jaypipes has quit IRC17:45
openstackgerritEoghan Glynn proposed a change to openstack/ceilometer: Per pipeline pluggable resource discovery  https://review.openstack.org/7597017:47
*** Yathi has joined #openstack-ceilometer17:47
eglynngordc: thanks for the review of https://review.openstack.org/75970 ... can you see if you like it any better in the light of my responses?17:48
*** terriyu has joined #openstack-ceilometer17:48
gordceglynn: taking a look now.17:49
eglynngordc: thank you sir!17:49
*** Yathi has quit IRC17:50
*** Yathi has joined #openstack-ceilometer17:51
gordceglynn: hmm. this is probably because i don't understand how discovery stuff works but visually, test_per_pipeline_discovery_discovered_only and test_per_agent_discovery_overridden_by_per_pipeline_discovery look to be the same test but for some reason the final assertion is different?  what am i missing between the two? https://review.openstack.org/#/c/75970/8/ceilometer/tests/agentbase.py17:55
*** _nadya_ has joined #openstack-ceilometer17:55
gordceglynn: oh darn. it's because testdiscovery isn't listed in test_per_agent_discovery_overridden_by_per_pipeline_discovery.... nevermind. me being stupid.17:59
*** mihgen has joined #openstack-ceilometer17:59
eglynngordc: ... you had me worried there for a second ;)18:00
*** mihgen has quit IRC18:01
*** Yathi has quit IRC18:01
gordceglynn: keeping you on your toes.18:01
*** Yathi has joined #openstack-ceilometer18:01
eglynnLOL :)18:02
*** Yathi has quit IRC18:03
*** piyushmasrani has quit IRC18:03
*** Yathi has joined #openstack-ceilometer18:03
*** _nadya_ has quit IRC18:03
*** Yathi has quit IRC18:10
*** Yathi has joined #openstack-ceilometer18:10
*** mihgen has joined #openstack-ceilometer18:11
openstackgerritAkhil Hingane proposed a change to openstack/ceilometer: VMware vSphere support: Disk rates  https://review.openstack.org/7657118:13
*** Yathi_ has joined #openstack-ceilometer18:13
*** _nadya_ has joined #openstack-ceilometer18:14
*** nati_ueno has joined #openstack-ceilometer18:22
*** nati_ueno has quit IRC18:22
*** nati_ueno has joined #openstack-ceilometer18:23
*** Yathi has quit IRC18:28
*** Yathi has joined #openstack-ceilometer18:28
*** jaypipes has joined #openstack-ceilometer18:30
*** Yathi has quit IRC18:33
*** jaypipes has quit IRC18:36
openstackgerritA change was merged to openstack/ceilometer: Adds time constraints to alarms  https://review.openstack.org/7539118:37
nsaje_\o/18:39
eglynnnsaje_: nice one! :)18:42
openstackgerritA change was merged to openstack/ceilometer: Wider selection of aggregates for mongodb  https://review.openstack.org/7687918:42
openstackgerritA change was merged to openstack/ceilometer: Wider selection of aggregates for sqlalchemy  https://review.openstack.org/7627818:42
openstackgerritA change was merged to openstack/ceilometer: Decouple source and sink configuration for pipelines  https://review.openstack.org/7545918:43
nsaje_eglynn: well right back at you! :)18:43
openstackgerritA change was merged to openstack/ceilometer: Remove code duplication Part 3  https://review.openstack.org/7581118:43
openstackgerritRob Raymond proposed a change to openstack/ceilometer: Replace hard coded WSGI application creation  https://review.openstack.org/7795718:43
eglynnwow the gate is like waiting at a busstop today18:43
eglynn... tumbleweeds for ages, then 6 buses come all at once ;)18:44
ildikov_\o/18:44
gibi_:)18:44
ildikov_eglynn: LOL :)18:44
nsaje_eglynn: haha :D18:45
ildikov_eglynn: and your puppies did not eat my kitten finally :)18:45
eglynnildikov_: LOL ;)18:45
nsaje_*merge acceptance speech* : I'd like to thank my parents, and also reviewers eglynn , sileht , ildikov_ and gibi_ :)18:47
eglynnnsaje_ wipes away a tear of joy ;)18:48
openstackgerritRob Raymond proposed a change to openstack/ceilometer: Replace hard coded WSGI application creation  https://review.openstack.org/7795718:50
gibi_nsaje_: you are very welcome! :)18:50
*** changbl has quit IRC18:50
ildikov_nsaje_: I know the feeling of this happiness and I'm glad that I could be the part of it :)18:53
*** jaypipes has joined #openstack-ceilometer18:53
*** changbl has joined #openstack-ceilometer18:54
openstackgerritEoghan Glynn proposed a change to openstack/ceilometer: Ensure user metadata mapped for instance notifications  https://review.openstack.org/7795918:56
*** jmckind has quit IRC18:57
eglynn^^^ phew, my last patch proposed for i3 :)18:58
eglynn... /me pushes it down to the wire all always ;)18:58
gibi_just in time :)18:58
*** sayalilunkad has quit IRC19:05
*** eglynn has quit IRC19:10
*** _nadya_ has quit IRC19:15
*** _nadya_ has joined #openstack-ceilometer19:27
*** mihgen has quit IRC19:31
*** lsmola has quit IRC19:35
*** _nadya_ has quit IRC19:36
*** ildikov_ has quit IRC19:39
*** ryanpetrello has joined #openstack-ceilometer19:44
*** eglynn has joined #openstack-ceilometer19:44
*** thomasem_ has joined #openstack-ceilometer19:45
*** thomasem has quit IRC19:46
openstackgerritgordon chung proposed a change to openstack/pycadf: add audit_maps for more OpenStack components  https://review.openstack.org/7511419:53
*** shakayumi has quit IRC20:17
*** shakayumi has joined #openstack-ceilometer20:19
*** ryanpetrello has quit IRC20:24
*** admin0 has joined #openstack-ceilometer20:25
admin0hey guys20:25
admin0i want to know how ceilometer works and collects the data20:26
admin0is it one file that runs on all compute node, or is it one process that runs somewhere and collects metrics from all other components ?20:26
admin0from the architecture diagram, i see that the central agent polls all components20:28
openstackgerritEoghan Glynn proposed a change to openstack/ceilometer: Ensure user metadata mapped for instance notifications  https://review.openstack.org/7795920:29
eglynnadmin0: there's an agent that runs on each compute node to poll the local hypervisor, an other agent that runs centrally to poll public REST APIs20:35
*** eglynn_ has joined #openstack-ceilometer20:36
*** eglynn has quit IRC20:36
*** promulo has joined #openstack-ceilometer20:38
*** jaypipes has quit IRC20:44
*** jmckind has joined #openstack-ceilometer20:58
*** admin0 has quit IRC21:00
*** admin0 has joined #openstack-ceilometer21:00
eglynn_admin0: ... and another service that receives and processes inbound notifications from the other openstack services (nova, glance, cinder ... etc)21:12
eglynn_admin0: ... and another service that collects metering message emitted by the compute & central agents and persists them21:13
eglynn_admin0: ... and another service that evaluates alarm conditions and triggers actions21:13
eglynn_admin0: ... and finally an API service that provides a facade over the metering store21:13
admin0eglynn: what i am trying to do is to locate that file which does the polling, so that instead of sending to the message bus, i can send it to something like graphite21:14
*** promulo_ has joined #openstack-ceilometer21:17
*** promulo has quit IRC21:18
*** admin0 has quit IRC21:19
*** admin0 has joined #openstack-ceilometer21:19
admin0sorry .. got disconnected21:21
admin0eglynn_: so it it possible .. which file/module should i be looking at ?21:22
eglynn_admin0: on the compute node there's a process called the ceilometer-agent-compute21:22
eglynn_admin0: that uses config files called /etc/ceilometer/ceilometer.conf and /etc/ceilometer/pipeline.yaml21:23
eglynn_admin0: the latter config file is used to specify how samples are published21:23
eglynn_admin0: currently we support 3 options for publication ... rpc (i.e. AMQP message bus), upd, file21:24
eglynn_admin0: this was designed to estensible21:24
admin0are there plans to support something like graphite ?21:24
eglynn_*extensible21:25
eglynn_admin0: so a new publisher that send the data elsewhere could be defined21:25
eglynn_admin0: not in the immediate future21:25
admin0eglynn, if I want to extend it, how/where do I start ?21:25
eglynn_admin0: read the code, have a look at the existing publishers21:25
eglynn_admin0: https://github.com/openstack/ceilometer/tree/master/ceilometer/publisher21:26
eglynn_admin0: note that today was the effectively last day for new features to land for the icehouse release21:27
eglynn_admin0: so you've missed that boat21:27
eglynn_admin0: i.e. this would be a Juno feature21:27
admin0eglynn_: i have just began to look into the codes21:27
eglynn_admin0: k, have fun! :)21:27
eglynn_admin0: gotta run now (late here, GMT TZ)21:28
admin0thank you eglynn_ i will catch up with you tomorrow21:28
*** promulo has joined #openstack-ceilometer21:28
eglynn_admin0: cool21:28
*** eglynn_ has quit IRC21:28
admin0thanks21:28
*** promulo_ has quit IRC21:28
*** julim has quit IRC21:31
raymondrQuestion: How does one submit a review that contains changes that affects devstack, when the gate jobs fail21:57
raymondrThe gate jobs fail because the change to devstack has not been committed.21:58
raymondrNot sure if I was clear above21:58
openstackgerritRob Raymond proposed a change to openstack/ceilometer: Replace hard coded WSGI application creation  https://review.openstack.org/7795722:00
*** admin0_ has joined #openstack-ceilometer22:00
*** admin0 has quit IRC22:01
*** admin0_ is now known as admin022:01
*** marcoemorais has quit IRC22:02
*** marcoemorais has joined #openstack-ceilometer22:02
*** thomasem_ has quit IRC22:05
openstackgerritAdrian Turjak proposed a change to openstack/ceilometer: Adds a state metric to ceilometer  https://review.openstack.org/7519322:14
*** marcoemorais has quit IRC22:15
*** admin0 has quit IRC22:15
*** admin0 has joined #openstack-ceilometer22:16
*** dhellmann is now known as dhellmann_22:18
*** Yathi_ has quit IRC22:21
openstackgerritAdrian Turjak proposed a change to openstack/ceilometer: Adds flavor.id metric  https://review.openstack.org/7572822:22
*** Yathi has joined #openstack-ceilometer22:25
*** raymondr has quit IRC22:28
*** raymondr has joined #openstack-ceilometer22:28
*** admin0 has quit IRC22:31
*** raymondr has quit IRC22:33
*** _nadya_ has joined #openstack-ceilometer22:37
*** _nadya_ has quit IRC22:41
*** ondergetekende has joined #openstack-ceilometer22:44
ondergetekendeI'm trying to set up ceilometer, but I really don't know how to test it.22:45
ondergetekendeI've set up glance to send notifications (it sends it to the openstack exchange)22:46
ondergetekendeBut I can't figure out how to let ceilometer to record or at least log these events.22:46
*** parkerr_ has joined #openstack-ceilometer22:49
ondergetekendeIt's ceilometer-collector's task to process those events, right?22:49
*** nati_ueno has quit IRC22:54
*** nati_ueno has joined #openstack-ceilometer22:56
*** jaypipes has joined #openstack-ceilometer23:00
*** marcoemorais has joined #openstack-ceilometer23:04
*** marcoemorais has quit IRC23:09
*** tongli has quit IRC23:10
*** marcoemorais has joined #openstack-ceilometer23:12
*** shakayumi has quit IRC23:14
openstackgerritgordon chung proposed a change to openstack/pycadf: pycadf documentation  https://review.openstack.org/6513923:21
gordcondergetekende: the notification service is what picks up the glance notifications.23:24
*** jergerber has quit IRC23:25
openstackgerritgordon chung proposed a change to openstack/pycadf: pycadf documentation  https://review.openstack.org/6513923:32
*** raymondr has joined #openstack-ceilometer23:34
*** raymondr_ has joined #openstack-ceilometer23:38
*** prad has quit IRC23:38
*** raymondr has quit IRC23:40
openstackgerritgordon chung proposed a change to openstack/pycadf: pycadf documentation  https://review.openstack.org/6513923:56

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