Monday, 2015-07-06

*** Kennan2 is now known as Kennan00:33
*** promulo__ has joined #openstack-ceilometer00:44
*** promulo has quit IRC00:48
*** liusheng has quit IRC01:18
*** Longgeek_ has joined #openstack-ceilometer01:46
*** Longgeek has quit IRC01:49
*** Longgeek_ has quit IRC02:47
*** Longgeek has joined #openstack-ceilometer02:47
*** Longgeek_ has joined #openstack-ceilometer03:16
*** Longgeek has quit IRC03:18
*** ddieterly has quit IRC03:57
*** ildikov has quit IRC04:06
*** deepthi has joined #openstack-ceilometer04:08
*** liusheng has joined #openstack-ceilometer04:41
*** ddieterly has joined #openstack-ceilometer04:58
*** ddieterly has quit IRC05:03
*** dikonoor has joined #openstack-ceilometer05:06
*** _nadya_ has joined #openstack-ceilometer05:28
*** _nadya_ has quit IRC05:45
*** ig0r__ has joined #openstack-ceilometer05:52
*** ig0r_ has quit IRC05:55
*** _nadya_ has joined #openstack-ceilometer06:14
*** ddieterly has joined #openstack-ceilometer06:28
*** ddieterly has quit IRC06:32
*** jwcroppe has joined #openstack-ceilometer06:43
*** belmoreira has joined #openstack-ceilometer06:53
*** exploreshaifali has joined #openstack-ceilometer06:53
*** ankita_wagh has joined #openstack-ceilometer07:23
*** _nadya_ has quit IRC07:24
*** mikedillion has joined #openstack-ceilometer07:26
*** exploreshaifali has quit IRC07:26
*** mikedillion has left #openstack-ceilometer07:29
*** ddieterly has joined #openstack-ceilometer07:29
*** _nadya_ has joined #openstack-ceilometer07:32
*** ddieterly has quit IRC07:33
*** ankita_wagh has quit IRC07:34
*** Ala has joined #openstack-ceilometer07:36
*** ildikov has joined #openstack-ceilometer07:42
*** _nadya_ has quit IRC07:46
*** safchain has joined #openstack-ceilometer07:54
*** exploreshaifali has joined #openstack-ceilometer08:12
*** _nadya_ has joined #openstack-ceilometer08:18
*** _nadya_ has quit IRC08:28
*** ddieterly has joined #openstack-ceilometer08:30
*** Ala has quit IRC08:33
*** ddieterly has quit IRC08:34
*** tshtilma has joined #openstack-ceilometer08:36
*** arunman has joined #openstack-ceilometer08:37
*** yassine_ has joined #openstack-ceilometer08:37
*** Ala has joined #openstack-ceilometer08:42
*** _nadya_ has joined #openstack-ceilometer08:46
*** tshtilma has quit IRC08:58
jd__the sileht is back!!09:24
silehtjd__, :)09:30
*** ddieterly has joined #openstack-ceilometer09:31
*** ddieterly has quit IRC09:35
*** yassine_ has quit IRC09:55
*** ildikov has quit IRC09:59
*** eglynn has joined #openstack-ceilometer10:02
*** Kennan has quit IRC10:05
*** lan_ has quit IRC10:07
*** Ala has quit IRC10:08
*** ildikov has joined #openstack-ceilometer10:14
*** lan_ has joined #openstack-ceilometer10:19
*** Kennan has joined #openstack-ceilometer10:21
*** ddieterly has joined #openstack-ceilometer10:31
*** ddieterly has quit IRC10:36
*** lan__ has joined #openstack-ceilometer10:37
openstackgerritMerged openstack/gnocchi: sqlalchemy: fix lazy-loading of attributes  https://review.openstack.org/19830610:38
*** Kennan2 has joined #openstack-ceilometer10:39
*** arunman has quit IRC10:40
*** lan_ has quit IRC10:40
*** Kennan has quit IRC10:41
*** Longgeek_ has quit IRC10:41
*** Longgeek has joined #openstack-ceilometer10:42
*** cdent has joined #openstack-ceilometer10:44
*** yatin has joined #openstack-ceilometer10:48
*** Kennan2 is now known as Kennan10:48
*** yatin has quit IRC10:57
*** yassine_ has joined #openstack-ceilometer11:11
*** yassine_ has quit IRC11:11
*** yatin has joined #openstack-ceilometer11:14
*** deepthi has quit IRC11:29
*** ildikov has quit IRC11:31
*** ddieterly has joined #openstack-ceilometer11:32
*** jaosorior has joined #openstack-ceilometer11:34
*** ddieterly has quit IRC11:37
jd__don't forget to review aodh patches11:41
*** thorst has joined #openstack-ceilometer11:42
*** ildikov has joined #openstack-ceilometer11:44
_nadya_jd__: starting now :)12:02
_nadya_jd__: looks like we don't have enough tests on gate, right?12:06
_nadya_jd__: sorry, I see  802. That's not bad12:07
*** tshtilma has joined #openstack-ceilometer12:07
*** ildikov has quit IRC12:22
*** edmondsw has joined #openstack-ceilometer12:24
jd___nadya_: we have everything except devstack-gate/tempest12:26
*** gordc has joined #openstack-ceilometer12:27
cdentlooks like prad and _nadya_ (with a little bit of llu) are the dependent variables on the midcycle schedule. This is what we end up with to match their schedules: https://www.google.com/calendar/embed?src=cXZqcTY2ZXJjcnYxbG9kaGtoMHZwNmxrNDRAZ3JvdXAuY2FsZW5kYXIuZ29vZ2xlLmNvbQ12:27
cdentThat's not fixed yet, need prad to check my work.12:28
_nadya_cdent: Let me check12:30
_nadya_cdent: google tells me that testing is in 7am12:30
cdentyes, which is within one of the zones you blocked out: 7-9, 10-1, 3-612:32
*** ddieterly has joined #openstack-ceilometer12:33
openstackgerritMerged openstack/ceilometer: Switch to using pbr's autodoc capability  https://review.openstack.org/12044112:37
*** ddieterly has quit IRC12:38
_nadya_cdent: looks like time converter lied to me. ok, I will be available. But if my schedule is a problem, I can fix it. And actually I want if it doesn't break the schedule12:38
*** ildikov has joined #openstack-ceilometer12:39
cdentWe can still make adjustments, that schedule is not set in stone, that's just what I was able to fit in with the requirements. If you've got a different _actual_ schedule than if you could add a nadya2 line to the doodle poll I'll see what i can make fit in a more civilised fashion: http://doodle.com/gzhumwaakrqds38h12:40
_nadya_cdent: I'll check now12:41
_nadya_cdent: aha, the final calendar is in UTC too. I thought Google has used my TZ12:43
_nadya_cdent: everything is ok then12:43
cdentcool, when prad shows up I'll make sure he agrees with my assessment and then we can make any remaining adjustments if necessary12:45
gordcthat's in UTC?12:47
*** exploreshaifali has quit IRC12:49
*** yatin has quit IRC12:49
* gordc is going to coin flip whether or not i wake up at 3am...12:49
gordcalso will coinflip whether i wake up for coinflip12:49
_nadya_gordc: yep, UTC :)12:49
_nadya_gordc: I believe 1 hour for tests is enough so the second day you may sleep one hour more12:50
*** exploreshaifali has joined #openstack-ceilometer12:51
gordclol thanks12:51
cdentthe controlling factor on the tests being then is _nadya_, gordc so perhaps you can do some smooth talking to get a reschedule?12:55
gordccdent: i'm ok. either way it's 3am something.12:57
gordcfunctional testing for me seems pretty straight forward so i'm fine with timing.12:57
*** Ala has joined #openstack-ceilometer12:58
jasonamyersgood morning12:59
_nadya_gordc, cdent actually I have two questions: 1. how are we going to test all pieces together and 2. Rally tests12:59
*** exploreshaifali has quit IRC13:00
cdent_nadya_ sounds like you got the beginnings of an etherpad to add here: https://wiki.openstack.org/wiki/Meetings/Ceilometer/Liberty_Virtual_Mid-Cycle#Topics13:00
gordcall pieces as in all ceilometer pieces?13:01
_nadya_gordc, cdent so maybe we move it to the end of second day?13:01
_nadya_gordc: yep13:01
jasonamyerswoot for 2AM starts!13:01
_nadya_jasonamyers: it's utc13:01
cdentjasonamyers: your flexibility has made the scheduling _much_ easier13:02
jasonamyershaha I'm nobody you should schedule like you like13:02
*** julim has joined #openstack-ceilometer13:03
jasonamyersgonna be fun, really looking forward to this13:03
gordcjasonamyers: it's typical euro-centric views. :)13:03
jasonamyersI won't snore too much13:03
cdentgordc: I'll have you know that I paid strict attention to the voting on both doodles and topics13:04
cdentDEMOCRACY FOR THE WIN13:04
* gordc forgot about camera... will not be sigining in13:04
jasonamyersdat camera at 2am ... gonna be awesome13:06
*** yassine_ has joined #openstack-ceilometer13:08
openstackgerritMerged openstack/aodh: Fix options registeration in tests  https://review.openstack.org/19707113:19
*** annasort has joined #openstack-ceilometer13:24
*** rbrooker has joined #openstack-ceilometer13:29
openstackgerritMerged openstack/aodh: Replaces methods deprecated in pymongo3.0  https://review.openstack.org/19661113:29
_nadya_jd__: ping13:29
openstackgerritMerged openstack/aodh: Move aodh.alarm.storage to aodh.storage  https://review.openstack.org/19672213:29
_nadya_cdent:13:30
openstackgerritMerged openstack/aodh: Change the default api server port  https://review.openstack.org/19821013:30
cdenthi _nadya_13:30
_nadya_cdent: hi :)13:30
_nadya_cdent: I hae some errors after applying "Move aodh.alarm.storage to aodh.storage "13:30
cdentthere's a fair few dependent patches in that stack, yeah?13:31
cdentbrb13:31
_nadya_cdent: there are no dependency as I see...13:32
* cdent will check when he's b from the brb13:32
_nadya_cdent: anyway,  https://gist.github.com/anonymous/48992116a8764bf2120a13:32
_nadya_cdent: I'll run it again in new venv13:33
_nadya_cdent: to be sure13:33
*** claudiub has joined #openstack-ceilometer13:34
*** liusheng_ has joined #openstack-ceilometer13:36
*** exploreshaifali has joined #openstack-ceilometer13:37
*** claudiub has quit IRC13:38
*** gordc_internal has joined #openstack-ceilometer13:40
*** promulo__ has quit IRC13:40
*** Longgeek has quit IRC13:43
*** [1]tshtilma has joined #openstack-ceilometer13:53
*** prad has joined #openstack-ceilometer13:53
cdent_nadya_: I think that error is expected when there is not coordination backend set13:53
*** tshtilma has quit IRC13:54
*** [1]tshtilma is now known as tshtilma13:54
*** r-daneel has joined #openstack-ceilometer13:56
*** rbak has quit IRC13:56
cdentprad: hola, I've made a first stab at a schedule. If you could check my math that would be dandy: https://www.google.com/calendar/embed?src=cXZqcTY2ZXJjcnYxbG9kaGtoMHZwNmxrNDRAZ3JvdXAuY2FsZW5kYXIuZ29vZ2xlLmNvbQ13:56
cdentthe thing I'm seeing _nadya_, jd__ is that the postgresql tests aren't working at all "too many clients already" during the table creation process13:58
*** tongli has joined #openstack-ceilometer13:59
jd__cdent: in aodh? yeah we should fix it and gate on that13:59
cdentso known issue then?13:59
pradcdent, morning14:00
pradcdent, sure i'll take a look now14:00
pradildikov, thx for the updates to the abstract, looks good to me14:01
pradcdent, time is UTC i presume?14:01
cdentprad, on that link, yes14:03
ildikovprad: hi. great, thanks14:04
pradcdent, hmm the morning schedule on that link is a bit rough on US folks.. ?14:05
cdenta) no us folks said they were required to attend those sessions14:06
cdentb) the sessions which you said you needed to attend are in the afternoon14:06
cdentc) basically the entire schedule is based on what you and _nadya_ said in terms of availability and "required to attend"14:06
cdentSo using the voting that people did that's the schedule that maps14:07
*** r-daneel has quit IRC14:07
cdentc.5) everyone else had long windows14:09
pradcdent, yea understand, its fine with me14:09
pradcdent, wasnt sure if people who are interested but havnt added themselves to the doodle want to dial in14:10
*** ddieterly has joined #openstack-ceilometer14:10
pradbut we can only satisfy only so many14:10
*** rbak has joined #openstack-ceilometer14:11
cdentyou don't vote, you don't git14:11
*** Ephur has joined #openstack-ceilometer14:11
pradyea14:11
pradcdent, looks good to me then14:12
cdentgordc: can you make a final declaration of finally declared, finally?14:12
*** liusheng_ has quit IRC14:17
*** logan2 has quit IRC14:17
*** r-daneel has joined #openstack-ceilometer14:20
*** logan2 has joined #openstack-ceilometer14:41
*** Longgeek has joined #openstack-ceilometer14:43
*** jruano has joined #openstack-ceilometer14:47
*** Longgeek has quit IRC14:49
*** ddieterly has quit IRC14:54
*** ddieterly has joined #openstack-ceilometer14:57
thorstcdent: have a min to discuss the powervm-compute-inspector blueprint?14:57
*** ddieterly has quit IRC14:58
*** Ephur has quit IRC15:01
*** ddieterly has joined #openstack-ceilometer15:02
*** serverascode has quit IRC15:05
*** serverascode has joined #openstack-ceilometer15:05
*** logan2 has quit IRC15:11
*** logan2 has joined #openstack-ceilometer15:19
*** belmoreira has quit IRC15:27
*** obedmr has joined #openstack-ceilometer15:33
*** gordc_internal has quit IRC15:34
*** gordc_internal has joined #openstack-ceilometer15:34
gordccdent: are the topics ordered by popularity?15:36
*** gordc_internal has quit IRC15:36
*** gordc_internal has joined #openstack-ceilometer15:36
cdentgordc: there are ordered by popularity here: https://docs.google.com/spreadsheets/d/1mKLlRtTUWEq62ZrIi4Xr7EoV7SwGUNPORa7yauXYCbk/edit#gid=015:37
*** gordc_internal has quit IRC15:37
cdenton the calendar they are ordered by where they fit for the people who said they need to be there on here: https://etherpad.openstack.org/p/ceilometer-liberty-midcycle15:37
cdentthorst: still around15:37
cdent?15:37
thorstyep15:38
cdentwhat's up?15:38
thorstwanted to talk some process stuff real quick15:38
*** exploreshaifali has quit IRC15:38
cdentgordc might be better for that, but we're both here, so go for it.15:38
thorstso I was initially targetting to put the powervm driver in ceilometer core.  Had a great discussion with dhellman.  Turns out I just didn't understand the plugin model 100%15:39
gordccdent: kk. was going to ask for schematisation to be flipped with delete/deprecate15:40
thorsthe recommended that we do it initially in a stackforge project (ceilometer-powervm) since its tied to pypowervm which is pretty mature, but not 1.0 yet15:40
thorstwanted to make sure that was OK from a process side?  And if so, I could propose an update to the blueprint to reflect that15:41
cdentgordc: the reason for d or do being in the afternoon is to make sure that it includes prad15:42
cdentthorst if you're happy to have it as an external plugin as far as I'm concerned that is a much better way to make it go15:42
cdentand I think everyone else would agree too15:43
thorstcdent: I don't mind it while it matures.  I would love to propose it into core perhaps the next release?15:43
gordcthorst: is the blocker that pypowervm isn't going to get into requirements?15:43
cdentthorst: If you like I'm sure that would be fine but I think over the long term we'll be encouraging more and more people to externally plug-in15:44
thorstgordc: yeah.  Which had a great discussion15:44
cdent(or at least we should)15:44
gordccdent: oh... i don't see his name on that topic. maybe it got deleted.15:44
thorsthonestly don't mind the external plugin approach at the moment15:44
cdentgordc: It's not but that's because I made an executive decision: he needs to be there whether he realizes it or not because of TheEmployer™15:45
thorstjust want to make sure I can still doc it properly, get reviews (second set of eyes to make sure we're doing things nicely), etc...15:45
cdentalso, I made the afternoon sessions long on purpose15:45
gordcthorst: ok with plugin approach15:45
cdentthorst: if you make it externally you'll need to make sure that you explicitly invited ceilo people to the review as people won't see it on their radar15:46
gordccdent: what's the debate on delete/deprecate? or just in anticipation of one?15:46
thorstyeah, I don't mind doing that.  Just don't want to frustrate anyone with excess reviews15:46
thorstbut I would find a lot of value in having people review.15:46
cdentgordc: it's not so much a debate as listing off and acknowledging all the things that will need to happen: devstack plugins, tempest plugins, grenade plugins, notification of downstream people (if we delete)15:47
cdentbut if you're cool with that stuff already being acknowledge then no problem moving it15:47
*** zhiyan has quit IRC15:47
*** yassine_ has quit IRC15:47
*** zhiyan has joined #openstack-ceilometer15:47
cdentI just had the impression that we hadn't really been thinking that hard about the details of what it really means to be making all these changes15:47
cdentand wanted to make sure we set aside some proper time15:47
cdentand the schematisation stuff doesn't need anytime because nothing will actually happen ;)15:48
gordccdent: ... i ...  can't argue with that.15:48
*** eglynn is now known as eglynn-afk15:49
cdentyou can do it gordc, I know you can!15:49
gordccdent: me 2 years ago thought that... i will be fooled no more.15:50
* sileht will write a bot to count how many time he see that : 'schematisation stuff doesn't need anytime because nothing will actually happen'15:50
cdentI'm only parroting what I've already seen15:50
* cdent would love for the people who want it to get it15:50
* sileht too15:50
cdentbut I no longer believe it is necessary. In fact I think it is an anti-pattern15:51
gordcyou both lie.15:51
* cdent likes to say "anti-pattern"15:51
cdentsileht: now that you're back you okay with that schedule, have we left out something precious to you?15:52
cdentgordc: you can keep on thinking about it until tomorrow? and then we make a final decision? If you want to make some changes check with prad, _nadya_ and llu. Everyone else is quite flexible.15:53
silehtcdent, I'm not sure I have totally understand what happen, I guess yes :p15:54
cdent:)15:55
gordccdent: i'm ok with schedule. it's going to be less than ideal for americas regardless.15:55
*** Ala has quit IRC15:55
gordcuntil we get more participants from here, i can't really change anything.15:56
cdentwe had a vote to decide a) what topics we would mid-cycle about d) what hours during the day people are available. Then we decided how many votes were necessary for a topic to be relevant.15:56
cdent^ sileht15:56
cdentgordc: get on that gordc, we need more participants15:56
gordcfly me to south america. i'll do some recruiting down there.15:57
gordci hear rio is a hotspot for devs15:57
*** tshtilma has quit IRC15:57
gordcand the mountains in chile.15:57
cdentI'll gladly go recruiting in patagonia if you like15:58
gordcno! i already offered.15:58
silehtcdent, the planning looks for me15:58
* sileht is adjusting its alarm clock15:59
sileht*looks good*16:00
*** Longgeek has joined #openstack-ceilometer16:00
*** ddieterly has quit IRC16:01
*** thorst is now known as thorst_afk16:02
*** ddieterly has joined #openstack-ceilometer16:02
*** rwsu has joined #openstack-ceilometer16:04
*** Longgeek has quit IRC16:05
*** safchain has quit IRC16:08
*** ankita_wagh has joined #openstack-ceilometer16:17
*** exploreshaifali has joined #openstack-ceilometer16:24
*** ankita_wagh has quit IRC16:25
*** jwcroppe_ has joined #openstack-ceilometer16:27
*** jwcroppe has quit IRC16:30
*** jwcroppe_ has quit IRC16:32
*** pauldix has joined #openstack-ceilometer16:32
*** jaosorior has quit IRC16:36
*** packet has joined #openstack-ceilometer16:39
*** tshtilma has joined #openstack-ceilometer16:40
*** tongli has quit IRC16:42
*** tongli has joined #openstack-ceilometer16:42
*** ankita_wagh has joined #openstack-ceilometer16:44
*** rbrooker has quit IRC16:46
*** ankita_wagh has quit IRC16:46
*** ankita_wagh has joined #openstack-ceilometer16:46
*** tongli has quit IRC16:47
*** ildikov has quit IRC16:48
*** ondergetekende_ has quit IRC16:50
*** ondergetekende has joined #openstack-ceilometer16:50
pradgordc, hey16:51
pradgordc, thx for the review, regarding couple of your comments16:51
gordcprad: sure16:51
pradgordc, when you say make each definition an individual end point16:52
pradgordc, do you see an issue with current way of initializing each definition ? or just suggesting an alternative16:53
openstackgerritMatt Riedemann proposed openstack/ceilometer: delete python bytecode before every test run  https://review.openstack.org/19881516:54
gordcan alternative... it's not a big deal i think. so currently, all our 'code definitions' are endpoints and they each deal with one meter.16:54
gordcyours is a single endpoint which deals with multiple meters.16:54
* cdent sighs16:54
pradright16:55
cdent(at deleting *pyc)16:55
openstackgerritMatt Riedemann proposed openstack/ceilometer: delete python bytecode before every test run  https://review.openstack.org/19881516:55
gordcit's more inline with how event definitions work so that's fine... i'm not even sure if it's a performance boost anyways so you can ignore16:55
pradok cool16:55
pradgordc, secondly,16:55
pradthe vol=1 comment16:56
pradthe reason i have it the way it is is cos jsonpath_rw hates numbers .. so i future we might need some plugin based approach16:56
pradfor now this seemed like a simple way to default to 1 until we deprecate these meters16:57
*** dscottsf has joined #openstack-ceilometer16:57
openstackgerritMatt Riedemann proposed openstack/ceilometer: delete python bytecode before every test run  https://review.openstack.org/19881516:57
*** dscottsf has quit IRC16:58
gordcprad: you mean my comment on the get_volume method?16:58
pradgordc, yea volume not optional comment16:58
prad"that said, i feel like vol should be a required attribute"16:59
gordcyou can't add a definition that says volume: 1?17:00
*** _nadya_ has quit IRC17:00
*** _cjones_ has joined #openstack-ceilometer17:00
*** Longgeek has joined #openstack-ceilometer17:01
gordcprad: can't you change parse_fields to detect if it's an int?17:01
pradgordc, not with the way it is now.. jsonpath_rw doesnt like it.. I could have a separate nonmeters class and override it perhaps17:02
pradgordc, perhaps , but then we'll run into cases where vol is an arithmetic expression.. for just this case i can do an int check, but thats not solving the whole problem. Thought i save this for a plugin approach later17:03
pradbut i'm cool with doing it just for vol =1 meters for now and worry about others later17:03
gordcprad: yeah, we should have better support in parse_fields (if you want to continue supporting volume=1 stuff)17:04
gordcit's pretty weird to say: 'don't put a volume if it's a number'17:05
*** Longgeek has quit IRC17:06
pradgordc, ok np, i'll look into something more elegant17:06
*** thorst_afk is now known as thorst17:07
openstackgerritgordon chung proposed openstack/ceilometer: add mandatory limit value to meter list  https://review.openstack.org/19881817:10
*** tongli has joined #openstack-ceilometer17:10
*** promulo has joined #openstack-ceilometer17:17
*** logan2 has quit IRC17:21
*** akgunjal has joined #openstack-ceilometer17:25
*** akgunjal has quit IRC17:37
*** logan2 has joined #openstack-ceilometer17:50
*** jwcroppe has joined #openstack-ceilometer17:53
*** jaosorior has joined #openstack-ceilometer17:59
*** ankita_w_ has joined #openstack-ceilometer18:00
*** ankita_wagh has quit IRC18:03
*** Kennan2 has joined #openstack-ceilometer18:03
*** Kennan has quit IRC18:04
pradgordc, so regarding having the whole metadata as part of the sample, i kept it for backward compatibility.. i know there are cases we look at some info in the metadata.18:05
pradgordc, you want to trim this somehow or skip it entirely?18:06
*** pauldix has quit IRC18:11
*** pauldix has joined #openstack-ceilometer18:12
*** tshtilma has quit IRC18:22
gordcprad: i was thinking more that we could have a defineable metadata section. so similar to 'traits' section in events18:23
gordcif you don't define a metadata, you get nothing.18:23
gordcso in case of compute meters, we define a static set of metadata18:23
*** dikonoor has quit IRC18:24
gordcprad: or like how we define resources in gnocchi (with specific attributes)18:25
pradgordc, hmm that mean the user needs full understanding of a specific event structure? is it ok to put that burden on the user?18:25
pradi'm not sure18:26
prads/event/metadata18:27
gordcfair. the problem with dumping entire message is that it's not resource metadata18:28
gordcit's just a raw dump.18:28
gordcby associating that dump with resource and sample is going to cause db to explode.18:28
gordcthere needs to be separation i think.18:29
gordci should add this is probably outside scope of your patch... but i think being able to define your metadata is useful.18:31
pradgordc, yea i understand your view.. with my patch we'll definitely make it easier to do this kind of a change in future18:32
pradi'm happy to look into trimming the metadata as a separate issue18:32
pradregarding the tests, i agree with you that its not very efficient to test every possible sample generated for each meter18:33
gordcprad: yeah, that point i was hoping it's possible to continue forward with.18:36
pradyep18:42
*** ddieterly has quit IRC18:44
ankita_w_Hi gordc, So meters are getting built on the compute nodes and then sent to the message bus right ?18:47
gordcankita_w_: right. so it's two different paths.18:47
ankita_w_and notification agent listens to notifications which are bascially events18:47
gordcfrom polling agent, they poll data -> build meter -> publish to queue -> collector stores it18:47
openstackgerritPradeep Kilambi proposed openstack/ceilometer: Declarative meters support  https://review.openstack.org/19763318:47
gordcfrom notification agent, services(nova/cinder/etc) publish data -> notification agent picks it up -> build meter/event -> publish to queue -> collector stores it18:48
ankita_w_I have a plugin for notifications which modifies the data when published on end points18:48
ankita_w_I want to use that plugin for meters built by the compute agent as well18:49
ankita_w_Is there a way to do that18:49
gordcthe data from polling agents (currently) do not have any connection to the notification you get in notification agent18:49
ankita_w_if I add the metering topic to notification_topics, in ceilometer.conf , why is it not working ?18:50
gordcankita_w_: cdent had a proposal to do exactly that18:50
gordchttps://review.openstack.org/#/c/192286/18:50
ankita_w_notification_topics = notifications, metering18:50
gordcso that has nothing to do with polling agents (compute agent)18:50
gordccompute agents publish to queue to be consumed by collector18:51
ankita_w_so who listens to the topics we specify in the conf ?18:51
ankita_w_why cant we make the notification agent listen to the topic on which polling agent is publishing?18:53
gordcthat is for internal queue. which the collector consumes18:53
gordchttp://docs.openstack.org/developer/ceilometer/architecture.html#high-level-architecture18:54
gordcthat is the basic architecture as it currently stands... cdents proposal is required to do what you want to do.18:55
cdentankita_w_: the changes I have in progress will let you do what you want18:55
gordcthe polling agent does not communicate with notification agent... it communicates with collector.18:55
cdentbasically the pollsters will send notifications rather than doing rpc to the collector18:55
cdentbut it will be a while yet18:55
ankita_w_yeah got it....thanku gordc and cdent18:56
cdentpresumably there would be some way to publish to something that made a notification18:56
cdentrather than dispatch to the collector?18:56
cdentBTW gordc: I have my prototype working™ but testing it effectively in the context of the existing ceilo tests is proving a real bear18:56
gordccdent: delete all the tests18:57
ankita_w_just to clarify- notification_agent listens to the notification_topics which are specified in the conf ? or is it the collector ?18:58
cdentwhich conf do you mean ankita_w_ ?19:00
ankita_w_ceilometer.conf on ceilometer server19:00
cdentokay, just wanted to be sure you weren't talking about nova's conf or something like that...19:01
*** tongli has quit IRC19:01
ankita_w_ok...who listens to those topics ?19:01
*** tongli has joined #openstack-ceilometer19:02
*** Longgeek has joined #openstack-ceilometer19:02
cdentankita_w_: I'm just looking into, to make sure I give you the actually correct answer rather than my guess19:02
gordcankita_w_: sorry. metering_topic or notification+topic?19:04
ankita_w_notification_topic19:04
cdentankita_w_: so yeah, the notification agent lists for notification_topics. In fact, each plugin sets up a Target for each topic19:04
ankita_w_the default metering_topic  is set to metering, where all the meters are published right ?19:05
gordcyes19:05
gordcthat is what all the agents publish to19:06
ankita_w_if I set notification_topics = notifications, metering, wont the notification agent also receive meters published on metering topic ?19:06
*** tongli has quit IRC19:06
gordcno... we publish to metering.sample.19:06
gordcnotification agent only listens to xyz.info and xyz.debug19:06
ankita_w_got it :)19:07
ankita_w_thanks a bunch19:07
cdentthis is all far too opaque19:07
gordcankita_w_: you can add sample. but you'll have notification agent and collector both grabbing from the queue19:07
*** tongli has joined #openstack-ceilometer19:07
gordccdent: job security19:08
ankita_w_the only reason am trying to do is ...my plugin is already working to modify the notifications in the format our system understands. I want to meters to go thru the same path instead of building a new plugin19:09
ankita_w_the  meters*19:09
gordcankita_w_: you can specify a custom topic when publishing.19:10
gordcankita_w_: have the notification agent listen to that topic... you'll still need to listen to .sample level though19:11
ankita_w_so can I specify notification_topics = notifications, metering.sample or does it require additional changes19:12
*** tongli has quit IRC19:12
gordchttps://github.com/openstack/ceilometer/blob/master/ceilometer/event/endpoint.py19:13
gordcin the same way we define, info and error level, you need a sample level19:13
ankita_w_got it...thanks a lot..this solved my problem19:15
gordcankita_w_: cool cool19:15
*** Longgeek has quit IRC19:27
openstackgerritChris Dent proposed openstack/ceilometer: [WIP] Spike at making the pollsters send notifications  https://review.openstack.org/19886519:28
cdentgordc I could with some commentary on that ^19:28
cdentat your leisure, I'm done for tonight19:28
* cdent waves19:29
*** cdent has quit IRC19:29
*** Ephur has joined #openstack-ceilometer19:30
*** tongli has joined #openstack-ceilometer19:38
*** tongli has quit IRC19:44
*** pauldix has quit IRC19:45
*** dencaval has quit IRC19:54
*** pauldix has joined #openstack-ceilometer19:56
*** pauldix has quit IRC19:58
*** exploreshaifali has quit IRC20:10
*** promulo__ has joined #openstack-ceilometer20:15
*** promulo has quit IRC20:16
openstackgerritPradeep Kilambi proposed openstack/ceilometer: Declarative meters support  https://review.openstack.org/19763320:19
ankita_w_gordc : I tried what we had just discussed about adding sample level in endpoint. I am still not getting the meters. I am running ceilometer juno20:20
*** ddieterly has joined #openstack-ceilometer20:21
*** Longgeek has joined #openstack-ceilometer20:24
*** packet has quit IRC20:25
*** pauldix has joined #openstack-ceilometer20:30
gordcankita_w_: do you have collector running?/20:39
ankita_w_yes20:39
gordcnotification agent and collector would be listening to same queue20:40
gordcand fighting over messages.20:40
gordcif you run rabbitmqctl list_queues do you see anthing on metering.sample queue?20:40
ankita_w_lemme check20:40
ankita_w_metering.info020:42
ankita_w_metering.sample020:42
ankita_w_metering.sample.info020:42
gordcdid you put 'metering.sample' as your notification_topic?20:42
*** tongli has joined #openstack-ceilometer20:44
ankita_w_I changed it to metering20:44
ankita_w_afterwards20:44
ankita_w_and restarted20:44
*** ildikov has joined #openstack-ceilometer20:45
*** pauldix has quit IRC20:46
gordcankita_w_: ah i see... if you have both collector/notification agent running they'll be fighting on same queue.20:47
gordcankita_w_: i need to step out for a bit. i can probably look at it later tonight if you want to send email... or i can look at it tomorrow20:48
ankita_w_ok thanks20:48
*** tongli has quit IRC20:48
*** tongli has joined #openstack-ceilometer20:49
*** gordc has quit IRC20:49
*** Longgeek has quit IRC20:50
*** thorst has quit IRC21:07
*** jaosorior has quit IRC21:16
*** fawadkhaliq has joined #openstack-ceilometer21:31
*** julim has quit IRC21:34
*** jruano has quit IRC21:35
*** pauldix has joined #openstack-ceilometer21:36
*** promulo has joined #openstack-ceilometer21:41
*** promulo__ has quit IRC21:41
*** eglynn-afk has quit IRC21:42
*** dscottsf has joined #openstack-ceilometer21:45
*** rjaiswal has joined #openstack-ceilometer21:46
*** Longgeek has joined #openstack-ceilometer21:47
*** dscottsf has quit IRC21:47
*** fawadkhaliq has quit IRC21:52
*** eglynn-afk has joined #openstack-ceilometer21:55
*** Kennan has joined #openstack-ceilometer22:02
*** Kennan2 has quit IRC22:03
*** Longgeek has quit IRC22:06
*** ildikov has quit IRC22:06
*** ddieterly has quit IRC22:08
*** ddieterly has joined #openstack-ceilometer22:09
*** ddieterly has quit IRC22:12
*** ddieterl_ has joined #openstack-ceilometer22:13
*** ddieterl_ has quit IRC22:13
*** prad has quit IRC22:15
*** ddieterly has joined #openstack-ceilometer22:16
*** ddieterl_ has joined #openstack-ceilometer22:17
*** ddieter__ has joined #openstack-ceilometer22:18
*** ddieterly has quit IRC22:19
*** ddieter__ has quit IRC22:20
*** ddieterly has joined #openstack-ceilometer22:21
*** ddieterl_ has quit IRC22:21
*** jruano has joined #openstack-ceilometer22:27
*** tongli has quit IRC22:33
*** tongli has joined #openstack-ceilometer22:34
*** eglynn-afk has quit IRC22:35
*** tongli has quit IRC22:37
*** tongli has joined #openstack-ceilometer22:37
*** tongli has quit IRC22:42
*** pauldix has quit IRC22:45
*** edmondsw has quit IRC22:47
*** tongli has joined #openstack-ceilometer22:48
*** ddieterly has quit IRC22:52
*** tongli has quit IRC22:53
openstackgerritRohit Jaiswal proposed openstack/ceilometer: Implementation of dynamically reloadable pipeline  https://review.openstack.org/19084222:56
*** jruano has quit IRC22:57
*** tongli has joined #openstack-ceilometer22:57
*** tongli has quit IRC23:02
*** rbak has quit IRC23:02
*** ankita_wagh has joined #openstack-ceilometer23:04
*** ankita_w_ has quit IRC23:07
*** Ephur has quit IRC23:07
*** ankita_wagh has quit IRC23:12
*** ankita_wagh has joined #openstack-ceilometer23:12
*** tongli has joined #openstack-ceilometer23:18
*** tongli has quit IRC23:23
*** thorst has joined #openstack-ceilometer23:24
*** thorst has quit IRC23:30
*** Longgeek has joined #openstack-ceilometer23:32
*** Longgeek has quit IRC23:50

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