*** raymondr_ has quit IRC | 00:00 | |
*** lakspace has quit IRC | 00:10 | |
*** lakspace has joined #openstack-ceilometer | 00:12 | |
*** julim has quit IRC | 00:14 | |
*** fnaval has quit IRC | 00:22 | |
*** prad has joined #openstack-ceilometer | 00:26 | |
*** rwsu has quit IRC | 00:27 | |
*** rwsu has joined #openstack-ceilometer | 00:30 | |
*** ryanpetrello has quit IRC | 00:31 | |
*** ddutta has joined #openstack-ceilometer | 00:32 | |
*** ryanpetrello has joined #openstack-ceilometer | 00:32 | |
*** jdob has quit IRC | 00:34 | |
*** jdob has joined #openstack-ceilometer | 00:35 | |
*** jdob has quit IRC | 00:40 | |
*** dhellmann is now known as dhellmann_ | 00:43 | |
*** openstackgerrit has quit IRC | 00:44 | |
*** openstackgerrit has joined #openstack-ceilometer | 00:44 | |
*** ryanpetrello has quit IRC | 00:44 | |
*** nati_ueno has joined #openstack-ceilometer | 00:46 | |
*** nati_uen_ has quit IRC | 00:49 | |
*** nati_ueno has quit IRC | 00:51 | |
*** ryanpetrello has joined #openstack-ceilometer | 00:53 | |
*** ryanpetrello has quit IRC | 00:54 | |
*** flwang has quit IRC | 00:56 | |
*** nati_ueno has joined #openstack-ceilometer | 00:57 | |
*** ddutta has quit IRC | 01:02 | |
*** nati_ueno has quit IRC | 01:09 | |
*** nati_ueno has joined #openstack-ceilometer | 01:10 | |
*** nati_ueno has quit IRC | 01:10 | |
*** nati_ueno has joined #openstack-ceilometer | 01:11 | |
*** nati_ueno has quit IRC | 01:12 | |
*** nati_ueno has joined #openstack-ceilometer | 01:12 | |
*** nati_ueno has quit IRC | 01:14 | |
*** nati_ueno has joined #openstack-ceilometer | 01:15 | |
*** nati_ueno has quit IRC | 01:16 | |
*** nati_ueno has joined #openstack-ceilometer | 01:16 | |
*** nati_ueno has quit IRC | 01:17 | |
*** rwsu has quit IRC | 01:17 | |
*** nati_ueno has joined #openstack-ceilometer | 01:17 | |
*** nati_ueno has quit IRC | 01:18 | |
*** nati_ueno has joined #openstack-ceilometer | 01:19 | |
*** nati_ueno has quit IRC | 01:22 | |
*** nati_ueno has joined #openstack-ceilometer | 01:22 | |
*** nati_ueno has quit IRC | 01:23 | |
*** nati_ueno has joined #openstack-ceilometer | 01:23 | |
*** nati_ueno has quit IRC | 01:24 | |
*** nati_ueno has joined #openstack-ceilometer | 01:25 | |
*** nati_ueno has quit IRC | 01:26 | |
*** nati_ueno has joined #openstack-ceilometer | 01:26 | |
*** nati_uen_ has joined #openstack-ceilometer | 01:27 | |
*** nati_ueno has quit IRC | 01:27 | |
*** nati_uen_ has quit IRC | 01:28 | |
*** nosnos has joined #openstack-ceilometer | 01:29 | |
*** nati_ueno has joined #openstack-ceilometer | 01:29 | |
*** nati_ueno has quit IRC | 01:30 | |
*** nati_ueno has joined #openstack-ceilometer | 01:30 | |
*** nati_ueno has quit IRC | 01:31 | |
*** nati_ueno has joined #openstack-ceilometer | 01:31 | |
*** nati_ueno has quit IRC | 01:32 | |
*** xianghui has joined #openstack-ceilometer | 02:04 | |
openstackgerrit | litong01 proposed a change to openstack/python-ceilometerclient: add notification alarm https://review.openstack.org/69474 | 02:08 |
---|---|---|
openstackgerrit | litong01 proposed a change to openstack/ceilometer: add notification alarm https://review.openstack.org/69473 | 02:08 |
*** ryanpetrello has joined #openstack-ceilometer | 02:14 | |
openstackgerrit | liusheng proposed a change to openstack/python-ceilometerclient: Correct help string of alarm-craete https://review.openstack.org/77761 | 02:35 |
*** flwang has joined #openstack-ceilometer | 02:47 | |
*** marcoemorais has quit IRC | 02:49 | |
*** raymondr has joined #openstack-ceilometer | 02:55 | |
*** ryanpetrello has quit IRC | 02:55 | |
*** nati_ueno has joined #openstack-ceilometer | 02:59 | |
*** nati_ueno has quit IRC | 02:59 | |
*** fnaval has joined #openstack-ceilometer | 03:38 | |
*** annegentle has quit IRC | 03:42 | |
*** raymondr has quit IRC | 04:44 | |
*** nati_ueno has joined #openstack-ceilometer | 04:45 | |
*** _nadya_ has joined #openstack-ceilometer | 04:46 | |
*** raymondr has joined #openstack-ceilometer | 04:46 | |
*** AMike has joined #openstack-ceilometer | 04:58 | |
*** AMike has joined #openstack-ceilometer | 04:58 | |
*** _nadya_ has quit IRC | 05:21 | |
*** shakayumi has joined #openstack-ceilometer | 05:30 | |
openstackgerrit | Piyush Masrani proposed a change to openstack/ceilometer: VMware vSphere support: Memory https://review.openstack.org/74702 | 05:31 |
*** sayalilunkad has joined #openstack-ceilometer | 05:56 | |
*** raymondr has quit IRC | 05:59 | |
*** marcoemorais has joined #openstack-ceilometer | 06:01 | |
openstackgerrit | Jenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/76762 | 06:03 |
*** _nadya_ has joined #openstack-ceilometer | 06:16 | |
*** _nadya_ has quit IRC | 06:23 | |
*** flwang has quit IRC | 06:58 | |
*** bogdando has joined #openstack-ceilometer | 07:01 | |
*** saju_m has joined #openstack-ceilometer | 07:12 | |
*** flwang has joined #openstack-ceilometer | 07:17 | |
*** sayalilunkad has quit IRC | 07:20 | |
openstackgerrit | Yuuichi Fujioka proposed a change to openstack/ceilometer: Implements monitoring-network-from-opendaylight https://review.openstack.org/63890 | 07:21 |
*** lsmola has joined #openstack-ceilometer | 07:23 | |
*** nosnos has quit IRC | 07:31 | |
*** nosnos_ has joined #openstack-ceilometer | 07:31 | |
openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Decouple source and sink configuration for pipelines https://review.openstack.org/75459 | 07:35 |
openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Per pipeline pluggable resource discovery https://review.openstack.org/75970 | 07:35 |
*** nosnos_ has quit IRC | 08:04 | |
*** nosnos has joined #openstack-ceilometer | 08:04 | |
*** taplax has joined #openstack-ceilometer | 08:06 | |
*** ildikov_ has joined #openstack-ceilometer | 08:14 | |
*** saju_m has quit IRC | 08:26 | |
*** julienvey_ has joined #openstack-ceilometer | 08:28 | |
*** sayalilunkad has joined #openstack-ceilometer | 08:33 | |
*** flwang has quit IRC | 08:54 | |
*** xianghui has quit IRC | 08:54 | |
*** xianghui has joined #openstack-ceilometer | 08:57 | |
*** flwang has joined #openstack-ceilometer | 09:00 | |
*** amichon has joined #openstack-ceilometer | 09:06 | |
amichon | hi, I created a bp : https://blueprints.launchpad.net/ceilometer/+spec/handle-keystone-identity-notifications | 09:07 |
amichon | and I dont't know how this can be reviewed ? | 09:08 |
amichon | should I fix a next miletone ? which ? | 09:08 |
*** yassine has joined #openstack-ceilometer | 09:09 | |
*** xianghui has quit IRC | 09:12 | |
*** xianghui has joined #openstack-ceilometer | 09:12 | |
*** xianghui has quit IRC | 09:18 | |
*** marcoemorais has quit IRC | 09:19 | |
openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Wider selection of aggregates for mongodb https://review.openstack.org/76879 | 09:21 |
openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Wider selection of aggregates for sqlalchemy https://review.openstack.org/76278 | 09:21 |
*** flwang has quit IRC | 09:24 | |
*** _nadya_ has joined #openstack-ceilometer | 09:24 | |
*** eglynn has joined #openstack-ceilometer | 09:27 | |
*** _nadya_ has quit IRC | 09:30 | |
*** xianghui has joined #openstack-ceilometer | 09:30 | |
*** saju_m has joined #openstack-ceilometer | 09:42 | |
eglynn | sileht: can you re-approve https://review.openstack.org/76273 ? ... assuming "reverify no bug" is still disabled | 09:44 |
*** krtaylor has quit IRC | 09:44 | |
eglynn | (just a spurious-looking tempest neutron failure, but holding up the whole patch series) | 09:44 |
sileht | eglynn, done | 09:45 |
eglynn | sileht: thank you sir! | 09:45 |
sileht | eglynn, thx for the "reverify no bug" workaround, I was not knew this one :) | 09:45 |
eglynn | sileht: ... normally to be used sparingly, but today is the day for "bending the rules" me thinks ;) | 09:47 |
sileht | eglynn, sure | 09:47 |
eglynn | sileht, jd__: so I was harassing the olso-messaging cores in RH to give some love to those patches needed for the ceilo switchover | 09:52 |
eglynn | I see https://review.openstack.org/61675 & https://review.openstack.org/77136 have landed | 09:52 |
eglynn | but not https://review.openstack.org/75365 ;( | 09:52 |
sileht | eglynn, thanks a lot for that ! | 09:52 |
eglynn | sileht: np! I wonder could we possibly live without that last one? | 09:52 |
sileht | eglynn, 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 |
sileht | eglynn, 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 icehouse | 09:56 |
eglynn | sileht: is that a new bug that needs to be fixed before 1.3.0 is released? | 09:58 |
sileht | eglynn, yes | 09:58 |
eglynn | sileht: what's that bug number? | 10:00 |
sileht | https://bugs.launchpad.net/oslo.messaging/+bug/1282639 | 10:01 |
eglynn | sileht: a-ha, ok, so that's the bug for the re-connection review I mentioned above ... cool, got it | 10:02 |
eglynn | sileht: so IIUC you've got a potential work-around for 1282639 that "bends the rules" somewhat? (i.e. imports oslo-messaging innards) | 10:05 |
eglynn | sileht: (... 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 IRC | 10:06 | |
*** shakayumi has quit IRC | 10:07 | |
sileht | eglynn, I will split the review | 10:08 |
eglynn | sileht: cool, thanks | 10:08 |
eglynn | sileht, 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 IRC | 10:12 | |
*** piyushmasrani has joined #openstack-ceilometer | 10:12 | |
eglynn | ... /me is fearing that ceilo will end up as an outlier if stuck on the old RPC layer | 10:13 |
jd__ | eglynn: I don't think it's a good idea – and I'm the one that likes to break thing | 10:15 |
jd__ | but it's a very important part of Ceilometer and I'm afraid to merge that code /that/ late in the process | 10:15 |
eglynn | jd__: fair enough, your call as PTL | 10:17 |
jd__ | eglynn: yeah :( | 10:17 |
jd__ | I feel obliged to release something that's not going potentially totally borked and not tested | 10:17 |
sileht | eglynn, 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 rpc | 10:17 |
sileht | eglynn, so having it the first day of Juno is enought | 10:18 |
eglynn | sileht, jd__: yeah I'm more concerned with the maintainability of icehouse-derived distros than any new features | 10:18 |
eglynn | sileht, jd__: ... so I suspect that we'll end up with Icehouse-derived distros carrying additional patches | 10:19 |
jd__ | eglynn: well as sileht feature wide everything is the same | 10:19 |
jd__ | +said | 10:19 |
eglynn | jd__, sileht: so to sum up ... | 10:24 |
*** sayalilunkad has joined #openstack-ceilometer | 10:24 | |
eglynn | we're unwilling to incur the risk at this late stage due to oslo-messaging not being quite ready to meet the ceilo requirements | 10:24 |
eglynn | ... and there being insufficient time for this to be tested fully pre-icehouse | 10:24 |
eglynn | ... but we're pushing ahead to get the switch-over finalized early in the Juno cycle | 10:25 |
eglynn | jd__, sileht: fair enough summation ^^^? | 10:25 |
jd__ | eglynn: yes! | 10:25 |
sileht | yep | 10:26 |
eglynn | fair enough, thanks chaps! | 10:26 |
sileht | very early! Day one ! | 10:26 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Use oslo.msg retry API in rpc publisher https://review.openstack.org/77845 | 10:37 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Consume notif. from multiple message bus https://review.openstack.org/77612 | 10:37 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Use NotificationPlugin as an oslo.msg endpoint https://review.openstack.org/77611 | 10:37 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Replace oslo.rpc by oslo.messaging https://review.openstack.org/57457 | 10:37 |
*** flwang has joined #openstack-ceilometer | 10:47 | |
*** xianghui has quit IRC | 10:59 | |
*** _nadya_ has joined #openstack-ceilometer | 11:09 | |
*** taplax has quit IRC | 11:20 | |
*** taplax has joined #openstack-ceilometer | 11:32 | |
taplax | .sms hd OSPR raportointi korjauksessa... | 11:33 |
*** julienvey_ has quit IRC | 11:41 | |
*** _nadya_ has quit IRC | 11:50 | |
*** yfujioka has quit IRC | 11:54 | |
*** amichon has quit IRC | 11:57 | |
nprivalova | yesterday nova-notifications worked ok at least on postgress. Today it doesn't work at all :( | 11:58 |
*** heyongli has joined #openstack-ceilometer | 11:59 | |
*** _nadya_ has joined #openstack-ceilometer | 11:59 | |
*** _nadya_ has quit IRC | 12:04 | |
nsaje_ | nprivalova: such are the woes of developers :) | 12:04 |
nprivalova | nsaje_: 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 bad | 12:08 |
*** ekarlso has quit IRC | 12:09 | |
*** amichon has joined #openstack-ceilometer | 12:23 | |
*** _nadya_ has joined #openstack-ceilometer | 12:25 | |
*** _nadya_ has quit IRC | 12:31 | |
*** julienvey_ has joined #openstack-ceilometer | 12:35 | |
*** saju_m has quit IRC | 12:40 | |
openstackgerrit | Nejc Saje proposed a change to openstack/ceilometer: Adds time constraints to alarms https://review.openstack.org/75391 | 12:41 |
openstackgerrit | Piyush Masrani proposed a change to openstack/ceilometer: VMware vSphere support: Memory Usage https://review.openstack.org/74702 | 12:47 |
*** _nadya_ has joined #openstack-ceilometer | 12:50 | |
*** saju_m has joined #openstack-ceilometer | 12:52 | |
*** Alexei_9871 has joined #openstack-ceilometer | 12:59 | |
*** _nadya_ has quit IRC | 13:02 | |
*** saju_m has quit IRC | 13:04 | |
*** _nadya_ has joined #openstack-ceilometer | 13:06 | |
*** _nadya_ has quit IRC | 13:08 | |
openstackgerrit | Piyush Masrani proposed a change to openstack/ceilometer: VMware vSphere: Changes for cpu_util https://review.openstack.org/76177 | 13:10 |
sileht | nsaje_, you got a last comment from eglynn | 13:10 |
*** dhellmann_ is now known as dhellmann | 13:11 | |
nsaje_ | sileht: thanks, on it | 13:13 |
*** sayalilunkad has quit IRC | 13:13 | |
nsaje_ | eglynn: thanks, that was a too-quick fix since python2.6 lacks total_seconds :( | 13:13 |
eglynn | nsaje_: yeah, I remember now ... the ceilometer.openstack.common.timeutils.delta_seconds() method handles that aspect | 13:14 |
nsaje_ | I see now, didn't know it was there, great | 13:15 |
openstackgerrit | Nejc Saje proposed a change to openstack/ceilometer: Adds time constraints to alarms https://review.openstack.org/75391 | 13:19 |
*** lsmola has quit IRC | 13:23 | |
*** ekarlso has joined #openstack-ceilometer | 13:23 | |
eglynn | jd__: 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-ceilometer | 13:24 | |
sileht | nsaje_, thx ! | 13:26 |
nsaje_ | sileht: np, hope it makes it through the gate on 1st try :) | 13:31 |
openstackgerrit | A change was merged to openstack/ceilometer: Add simple capabilities API https://review.openstack.org/76273 | 13:34 |
openstackgerrit | A change was merged to openstack/ceilometer: Selectable aggregation functions for statistics https://review.openstack.org/70454 | 13:34 |
openstackgerrit | A change was merged to openstack/ceilometer: Selectable aggregate support in mongodb https://review.openstack.org/76878 | 13:35 |
*** dhellmann is now known as dhellmann_ | 13:36 | |
*** lsmola has joined #openstack-ceilometer | 13:36 | |
eglynn | \o/ the gate is gating :) | 13:40 |
*** krtaylor has joined #openstack-ceilometer | 13:45 | |
ildikov_ | eglynn: +1 for the happiness :) | 13:55 |
ildikov_ | nsaje_:ping | 13:55 |
nsaje_ | ildikov_: hey | 13:56 |
ildikov_ | did you try to build the doc? | 13:56 |
*** nosnos has quit IRC | 13:56 | |
ildikov_ | nsaje_: ^^ | 13:56 |
*** gordc has joined #openstack-ceilometer | 13:56 | |
ildikov_ | nsaje_: I mean the time constraints patch | 13:56 |
nsaje_ | ildikov_: huh, no, by the time I finished fixing things it slipped my mind | 13:57 |
ildikov_ | nsaje_: sphinx does not like the time_constraint in the Alarm's sample | 13: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 work | 13: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 issue | 13: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, thanks | 13:59 |
ildikov_ | nsaje_: when I replaced the [AlarmTime....sample()] there with None, the build succeeded, so maybe it will work with a static example | 14:01 |
*** heyongli has quit IRC | 14:01 | |
openstackgerrit | Piyush Masrani proposed a change to openstack/ceilometer: Changes for networking metrics support for vSphere https://review.openstack.org/77355 | 14:02 |
ildikov_ | nsaje_: I did not have time to try it, I thought it is better to inform you about this problem sooner than later | 14:03 |
nsaje_ | ildikov_: thanks a lot, I'll take care of it | 14:04 |
ildikov_ | nsaje_: np! | 14:06 |
*** annegentle has joined #openstack-ceilometer | 14:09 | |
nsaje_ | ildikov_: fixed, thanks! | 14:15 |
openstackgerrit | Nejc Saje proposed a change to openstack/ceilometer: Adds time constraints to alarms https://review.openstack.org/75391 | 14:15 |
nsaje_ | sileht, eglynn : sorry about this, can you guys check time constraints again? | 14:15 |
eglynn | nsaje_: sure! | 14:15 |
sileht | nsaje_, sure | 14:16 |
eglynn | nsaje_: haha, bitten by the sphinx right? ;) | 14:16 |
nsaje_ | eglynn: yep | 14:17 |
*** julim has joined #openstack-ceilometer | 14: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 |
eglynn | ildikov_: phew! ;) | 14:21 |
*** lsmola has quit IRC | 14:24 | |
*** amichon has quit IRC | 14:26 | |
ildikov_ | eglynn: sorry about mentioning it, but it was a so unforgettable moment ;) | 14:27 |
eglynn | ildikov_: it made my day so far ;) | 14:28 |
*** terriyu has joined #openstack-ceilometer | 14:31 | |
*** lsmola has joined #openstack-ceilometer | 14:33 | |
*** lsmola has quit IRC | 14: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-ceilometer | 14:34 | |
openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Wider selection of aggregates for mongodb https://review.openstack.org/76879 | 14:34 |
openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Wider selection of aggregates for sqlalchemy https://review.openstack.org/76278 | 14:34 |
*** changbl has quit IRC | 14:35 | |
eglynn | ildikov_: 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 rebase | 14:36 |
eglynn | weird | 14:36 |
eglynn | perhaps 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 here | 14:38 |
eglynn | meh! | 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 ones | 14:39 |
eglynn | the "Gerrit trivial rebase detection script" restores a +2 & +1 with a pair of +2s | 14:40 |
eglynn | ... if the approving reviewer previously gave a +2 on a prior patch version :( | 14:40 |
eglynn | ... nasty! | 14:40 |
eglynn | sileht: can you morph your +2 to a +1 on https://review.openstack.org/76278 & https://review.openstack.org/76879 ? | 14:42 |
eglynn | sileht: (... 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 |
sileht | eglynn, gerrit rebase script just remove the 'Approved +1' | 14:43 |
*** sayalilunkad has quit IRC | 14:43 | |
sileht | eglynn, both fixed | 14:43 |
eglynn | ildikov_: yeah, it keeps the adrenaline flowing ;) | 14:43 |
eglynn | sileht: thank you sir! | 14:43 |
*** thomasem has joined #openstack-ceilometer | 14: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 |
eglynn | ildikov_: dbs/mongo code de-dupe refactoring? ... checking now | 14:47 |
eglynn | *db2 | 14:47 |
*** taplax has quit IRC | 14: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_base | 14:49 |
eglynn | ildikov_: yeah there's a rejected hunk | 14:50 |
eglynn | http://fpaste.org/82198/94458713/ | 14:50 |
*** lsmola has quit IRC | 14:50 | |
eglynn | ildikov_: (because the line above has changed, not your alarm changes themselves conflicting) | 14:51 |
eglynn | ildikov_: this line: (models.Statistics(**(r['value'])) for r in results['results']), | 14:51 |
eglynn | ildikov_: ... the unrelated line above is used by git to anchor your mods to get_alarm_changes | 14: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 |
eglynn | ildikov_: this puppy https://review.openstack.org/75811 | 14:54 |
ildikov_ | eglynn: I had a merge conflict because of the already merged patch sets, as your modifications were around mines | 14:54 |
eglynn | ildikov_: 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 conflict | 14:56 |
eglynn | ildikov_: by just manually applying that rejected hunk above, I can rebase your onto mine no prob | 14:56 |
eglynn | ildikov_: (... 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 more | 14:58 |
eglynn | ildikov_: 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 one | 14:59 |
eglynn | ildikov_: a-ha I see | 14:59 |
ildikov_ | eglynn: I rebased to master | 14:59 |
ildikov_ | eglynn: or at least as a first step I rebased to master :) | 14:59 |
eglynn | ildikov_: 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 |
eglynn | ildikov_: ... /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 |
eglynn | ildikov_: 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 |
openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: Remove code duplication Part 3 https://review.openstack.org/75811 | 15:06 |
eglynn | ildikov_: ... yeah the changes were independent all right | 15:06 |
eglynn | ildikov_: ... but the "anchor line" for one of your hunks was touched by one of my (now merged) patches | 15:06 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Consume notif. from multiple message bus https://review.openstack.org/77612 | 15:07 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Use NotificationPlugin as an oslo.msg endpoint https://review.openstack.org/77611 | 15:07 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Use oslo.msg retry API in rpc publisher https://review.openstack.org/77845 | 15:07 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Replace oslo.rpc by oslo.messaging https://review.openstack.org/57457 | 15:07 |
ildikov_ | eglynn: yes, I wanted to say something similar like this, I just mentioned too many things at the same time I think | 15: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 |
eglynn | ildikov_: yeah I'm pretty sure my two bad boys won't elbow out your's again ;) | 15:10 |
*** jergerber has joined #openstack-ceilometer | 15:12 | |
ildikov_ | eglynn: then you can help in getting my innocent little girl back to the gate in approved state ;) | 15:17 |
gordc | eglynn: is today feature freeze? having trouble remembering what day it is. | 15:20 |
*** fnaval has quit IRC | 15:21 | |
eglynn | gordc: AFAIK today is the day that the milestone-proposed branch will be cut | 15:21 |
gordc | eglynn: 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 |
eglynn | gordc: IIUC anything that lands today will definitely get into i3 | 15:24 |
eglynn | gordc: ... 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__ & ttx | 15:25 |
eglynn | gordc: ... https://wiki.openstack.org/wiki/GerritJenkinsGithub#Authoring_Changes_for_milestone-proposed | 15:25 |
* jd__ nods | 15:26 | |
gordc | eglynn: cool cool. these dates are confusing me. i'll let you release guys handle it. :) | 15:26 |
*** tongli has joined #openstack-ceilometer | 15:29 | |
*** lsmola has joined #openstack-ceilometer | 15:32 | |
*** raymondr has joined #openstack-ceilometer | 15:32 | |
openstackgerrit | Akhil Hingane proposed a change to openstack/ceilometer: VMware vSphere support: Disk rates https://review.openstack.org/76571 | 15:34 |
*** ryanpetrello has joined #openstack-ceilometer | 15:37 | |
*** jmckind has joined #openstack-ceilometer | 15:37 | |
*** Alexei_9871 has quit IRC | 15:41 | |
eglynn | gordc: speaking of stuff landing for i3 today | 15:43 |
eglynn | gordc: ... 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 |
gordc | eglynn: sure. i'll take a look at that next. | 15:44 |
eglynn | gordc: thank you sir! | 15:44 |
gordc | jd__: do you know what time the branch is cut? | 15:45 |
sileht | /1/10 | 15:46 |
*** fnaval has joined #openstack-ceilometer | 15:48 | |
*** ryanpetrello has quit IRC | 15:54 | |
*** nati_ueno has joined #openstack-ceilometer | 15:57 | |
eglynn | gordc: i3 milestone-proposed will be cut first thing tomorrow morning European time (modulo gate drainage) | 15:59 |
gordc | eglynn: cool cool. good to know. | 16:00 |
*** giroro_ has quit IRC | 16:01 | |
*** Ruetobas has joined #openstack-ceilometer | 16:03 | |
*** Ruetobas has quit IRC | 16:07 | |
gordc | eglynn: 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 |
eglynn | gordc: ... looking | 16:09 |
*** dhellmann_ is now known as dhellmann | 16:12 | |
*** Ruetobas has joined #openstack-ceilometer | 16:13 | |
*** _nadya_ has joined #openstack-ceilometer | 16: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/75811 | 16:26 |
eglynn | ildikov_: looking | 16:27 |
gordc | ildikov_: done. | 16:28 |
ildikov_ | eglynn, gordc: thanks! | 16:30 |
*** shakayumi has joined #openstack-ceilometer | 16:32 | |
*** changbl has joined #openstack-ceilometer | 16:35 | |
eglynn | gordc: ... I've responded to your idea about moving support_meter in https://review.openstack.org/#/c/75459/16/ceilometer/pipeline.py | 16:35 |
eglynn | sileht: can you pull that re-approve trick again on https://review.openstack.org/76278 ? | 16:37 |
*** julienvey_ has quit IRC | 16:38 | |
*** sayalilunkad has joined #openstack-ceilometer | 16:38 | |
gordc | eglynn: ok. taking a look. | 16:39 |
sileht | eglynn, I have already saw this bug ... I have asked a reverify | 16:42 |
eglynn | sileht: thank you sir! | 16:43 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Consume notif. from multiple message bus https://review.openstack.org/77612 | 16:45 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Use NotificationPlugin as an oslo.msg endpoint https://review.openstack.org/77611 | 16:45 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Use oslo.msg retry API in rpc publisher https://review.openstack.org/77845 | 16:45 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Replace oslo.rpc by oslo.messaging https://review.openstack.org/57457 | 16:45 |
*** _cjones_ has joined #openstack-ceilometer | 16:52 | |
*** _cjones_ has quit IRC | 16:55 | |
*** _cjones_ has joined #openstack-ceilometer | 16:55 | |
*** _nadya_ has quit IRC | 16:58 | |
*** jdob has joined #openstack-ceilometer | 17:06 | |
gordc | sileht: 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-ceilometer | 17:11 | |
*** mihgen has joined #openstack-ceilometer | 17:11 | |
*** nati_ueno has quit IRC | 17:13 | |
*** rwsu has joined #openstack-ceilometer | 17:14 | |
*** _nadya_ has joined #openstack-ceilometer | 17:20 | |
*** marcoemorais has joined #openstack-ceilometer | 17:24 | |
*** mihgen has quit IRC | 17:24 | |
*** _nadya_ has quit IRC | 17:25 | |
*** yassine has quit IRC | 17:28 | |
*** terriyu has quit IRC | 17:42 | |
*** jaypipes has quit IRC | 17:45 | |
openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Per pipeline pluggable resource discovery https://review.openstack.org/75970 | 17:47 |
*** Yathi has joined #openstack-ceilometer | 17:47 | |
eglynn | gordc: 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-ceilometer | 17:48 | |
gordc | eglynn: taking a look now. | 17:49 |
eglynn | gordc: thank you sir! | 17:49 |
*** Yathi has quit IRC | 17:50 | |
*** Yathi has joined #openstack-ceilometer | 17:51 | |
gordc | eglynn: 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.py | 17:55 |
*** _nadya_ has joined #openstack-ceilometer | 17:55 | |
gordc | eglynn: 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-ceilometer | 17:59 | |
eglynn | gordc: ... you had me worried there for a second ;) | 18:00 |
*** mihgen has quit IRC | 18:01 | |
*** Yathi has quit IRC | 18:01 | |
gordc | eglynn: keeping you on your toes. | 18:01 |
*** Yathi has joined #openstack-ceilometer | 18:01 | |
eglynn | LOL :) | 18:02 |
*** Yathi has quit IRC | 18:03 | |
*** piyushmasrani has quit IRC | 18:03 | |
*** Yathi has joined #openstack-ceilometer | 18:03 | |
*** _nadya_ has quit IRC | 18:03 | |
*** Yathi has quit IRC | 18:10 | |
*** Yathi has joined #openstack-ceilometer | 18:10 | |
*** mihgen has joined #openstack-ceilometer | 18:11 | |
openstackgerrit | Akhil Hingane proposed a change to openstack/ceilometer: VMware vSphere support: Disk rates https://review.openstack.org/76571 | 18:13 |
*** Yathi_ has joined #openstack-ceilometer | 18:13 | |
*** _nadya_ has joined #openstack-ceilometer | 18:14 | |
*** nati_ueno has joined #openstack-ceilometer | 18:22 | |
*** nati_ueno has quit IRC | 18:22 | |
*** nati_ueno has joined #openstack-ceilometer | 18:23 | |
*** Yathi has quit IRC | 18:28 | |
*** Yathi has joined #openstack-ceilometer | 18:28 | |
*** jaypipes has joined #openstack-ceilometer | 18:30 | |
*** Yathi has quit IRC | 18:33 | |
*** jaypipes has quit IRC | 18:36 | |
openstackgerrit | A change was merged to openstack/ceilometer: Adds time constraints to alarms https://review.openstack.org/75391 | 18:37 |
nsaje_ | \o/ | 18:39 |
eglynn | nsaje_: nice one! :) | 18:42 |
openstackgerrit | A change was merged to openstack/ceilometer: Wider selection of aggregates for mongodb https://review.openstack.org/76879 | 18:42 |
openstackgerrit | A change was merged to openstack/ceilometer: Wider selection of aggregates for sqlalchemy https://review.openstack.org/76278 | 18:42 |
openstackgerrit | A change was merged to openstack/ceilometer: Decouple source and sink configuration for pipelines https://review.openstack.org/75459 | 18:43 |
nsaje_ | eglynn: well right back at you! :) | 18:43 |
openstackgerrit | A change was merged to openstack/ceilometer: Remove code duplication Part 3 https://review.openstack.org/75811 | 18:43 |
openstackgerrit | Rob Raymond proposed a change to openstack/ceilometer: Replace hard coded WSGI application creation https://review.openstack.org/77957 | 18:43 |
eglynn | wow the gate is like waiting at a busstop today | 18: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 :D | 18:45 |
ildikov_ | eglynn: and your puppies did not eat my kitten finally :) | 18:45 |
eglynn | ildikov_: LOL ;) | 18:45 |
nsaje_ | *merge acceptance speech* : I'd like to thank my parents, and also reviewers eglynn , sileht , ildikov_ and gibi_ :) | 18:47 |
eglynn | nsaje_ wipes away a tear of joy ;) | 18:48 |
openstackgerrit | Rob Raymond proposed a change to openstack/ceilometer: Replace hard coded WSGI application creation https://review.openstack.org/77957 | 18:50 |
gibi_ | nsaje_: you are very welcome! :) | 18:50 |
*** changbl has quit IRC | 18: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-ceilometer | 18:53 | |
*** changbl has joined #openstack-ceilometer | 18:54 | |
openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Ensure user metadata mapped for instance notifications https://review.openstack.org/77959 | 18:56 |
*** jmckind has quit IRC | 18: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 IRC | 19:05 | |
*** eglynn has quit IRC | 19:10 | |
*** _nadya_ has quit IRC | 19:15 | |
*** _nadya_ has joined #openstack-ceilometer | 19:27 | |
*** mihgen has quit IRC | 19:31 | |
*** lsmola has quit IRC | 19:35 | |
*** _nadya_ has quit IRC | 19:36 | |
*** ildikov_ has quit IRC | 19:39 | |
*** ryanpetrello has joined #openstack-ceilometer | 19:44 | |
*** eglynn has joined #openstack-ceilometer | 19:44 | |
*** thomasem_ has joined #openstack-ceilometer | 19:45 | |
*** thomasem has quit IRC | 19:46 | |
openstackgerrit | gordon chung proposed a change to openstack/pycadf: add audit_maps for more OpenStack components https://review.openstack.org/75114 | 19:53 |
*** shakayumi has quit IRC | 20:17 | |
*** shakayumi has joined #openstack-ceilometer | 20:19 | |
*** ryanpetrello has quit IRC | 20:24 | |
*** admin0 has joined #openstack-ceilometer | 20:25 | |
admin0 | hey guys | 20:25 |
admin0 | i want to know how ceilometer works and collects the data | 20:26 |
admin0 | is 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 |
admin0 | from the architecture diagram, i see that the central agent polls all components | 20:28 |
openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Ensure user metadata mapped for instance notifications https://review.openstack.org/77959 | 20:29 |
eglynn | admin0: 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 APIs | 20:35 |
*** eglynn_ has joined #openstack-ceilometer | 20:36 | |
*** eglynn has quit IRC | 20:36 | |
*** promulo has joined #openstack-ceilometer | 20:38 | |
*** jaypipes has quit IRC | 20:44 | |
*** jmckind has joined #openstack-ceilometer | 20:58 | |
*** admin0 has quit IRC | 21:00 | |
*** admin0 has joined #openstack-ceilometer | 21: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 them | 21:13 |
eglynn_ | admin0: ... and another service that evaluates alarm conditions and triggers actions | 21:13 |
eglynn_ | admin0: ... and finally an API service that provides a facade over the metering store | 21:13 |
admin0 | eglynn: 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 graphite | 21:14 |
*** promulo_ has joined #openstack-ceilometer | 21:17 | |
*** promulo has quit IRC | 21:18 | |
*** admin0 has quit IRC | 21:19 | |
*** admin0 has joined #openstack-ceilometer | 21:19 | |
admin0 | sorry .. got disconnected | 21:21 |
admin0 | eglynn_: 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-compute | 21:22 |
eglynn_ | admin0: that uses config files called /etc/ceilometer/ceilometer.conf and /etc/ceilometer/pipeline.yaml | 21:23 |
eglynn_ | admin0: the latter config file is used to specify how samples are published | 21:23 |
eglynn_ | admin0: currently we support 3 options for publication ... rpc (i.e. AMQP message bus), upd, file | 21:24 |
eglynn_ | admin0: this was designed to estensible | 21:24 |
admin0 | are there plans to support something like graphite ? | 21:24 |
eglynn_ | *extensible | 21:25 |
eglynn_ | admin0: so a new publisher that send the data elsewhere could be defined | 21:25 |
eglynn_ | admin0: not in the immediate future | 21:25 |
admin0 | eglynn, if I want to extend it, how/where do I start ? | 21:25 |
eglynn_ | admin0: read the code, have a look at the existing publishers | 21:25 |
eglynn_ | admin0: https://github.com/openstack/ceilometer/tree/master/ceilometer/publisher | 21:26 |
eglynn_ | admin0: note that today was the effectively last day for new features to land for the icehouse release | 21:27 |
eglynn_ | admin0: so you've missed that boat | 21:27 |
eglynn_ | admin0: i.e. this would be a Juno feature | 21:27 |
admin0 | eglynn_: i have just began to look into the codes | 21:27 |
eglynn_ | admin0: k, have fun! :) | 21:27 |
eglynn_ | admin0: gotta run now (late here, GMT TZ) | 21:28 |
admin0 | thank you eglynn_ i will catch up with you tomorrow | 21:28 |
*** promulo has joined #openstack-ceilometer | 21:28 | |
eglynn_ | admin0: cool | 21:28 |
*** eglynn_ has quit IRC | 21:28 | |
admin0 | thanks | 21:28 |
*** promulo_ has quit IRC | 21:28 | |
*** julim has quit IRC | 21:31 | |
raymondr | Question: How does one submit a review that contains changes that affects devstack, when the gate jobs fail | 21:57 |
raymondr | The gate jobs fail because the change to devstack has not been committed. | 21:58 |
raymondr | Not sure if I was clear above | 21:58 |
openstackgerrit | Rob Raymond proposed a change to openstack/ceilometer: Replace hard coded WSGI application creation https://review.openstack.org/77957 | 22:00 |
*** admin0_ has joined #openstack-ceilometer | 22:00 | |
*** admin0 has quit IRC | 22:01 | |
*** admin0_ is now known as admin0 | 22:01 | |
*** marcoemorais has quit IRC | 22:02 | |
*** marcoemorais has joined #openstack-ceilometer | 22:02 | |
*** thomasem_ has quit IRC | 22:05 | |
openstackgerrit | Adrian Turjak proposed a change to openstack/ceilometer: Adds a state metric to ceilometer https://review.openstack.org/75193 | 22:14 |
*** marcoemorais has quit IRC | 22:15 | |
*** admin0 has quit IRC | 22:15 | |
*** admin0 has joined #openstack-ceilometer | 22:16 | |
*** dhellmann is now known as dhellmann_ | 22:18 | |
*** Yathi_ has quit IRC | 22:21 | |
openstackgerrit | Adrian Turjak proposed a change to openstack/ceilometer: Adds flavor.id metric https://review.openstack.org/75728 | 22:22 |
*** Yathi has joined #openstack-ceilometer | 22:25 | |
*** raymondr has quit IRC | 22:28 | |
*** raymondr has joined #openstack-ceilometer | 22:28 | |
*** admin0 has quit IRC | 22:31 | |
*** raymondr has quit IRC | 22:33 | |
*** _nadya_ has joined #openstack-ceilometer | 22:37 | |
*** _nadya_ has quit IRC | 22:41 | |
*** ondergetekende has joined #openstack-ceilometer | 22:44 | |
ondergetekende | I'm trying to set up ceilometer, but I really don't know how to test it. | 22:45 |
ondergetekende | I've set up glance to send notifications (it sends it to the openstack exchange) | 22:46 |
ondergetekende | But I can't figure out how to let ceilometer to record or at least log these events. | 22:46 |
*** parkerr_ has joined #openstack-ceilometer | 22:49 | |
ondergetekende | It's ceilometer-collector's task to process those events, right? | 22:49 |
*** nati_ueno has quit IRC | 22:54 | |
*** nati_ueno has joined #openstack-ceilometer | 22:56 | |
*** jaypipes has joined #openstack-ceilometer | 23:00 | |
*** marcoemorais has joined #openstack-ceilometer | 23:04 | |
*** marcoemorais has quit IRC | 23:09 | |
*** tongli has quit IRC | 23:10 | |
*** marcoemorais has joined #openstack-ceilometer | 23:12 | |
*** shakayumi has quit IRC | 23:14 | |
openstackgerrit | gordon chung proposed a change to openstack/pycadf: pycadf documentation https://review.openstack.org/65139 | 23:21 |
gordc | ondergetekende: the notification service is what picks up the glance notifications. | 23:24 |
*** jergerber has quit IRC | 23:25 | |
openstackgerrit | gordon chung proposed a change to openstack/pycadf: pycadf documentation https://review.openstack.org/65139 | 23:32 |
*** raymondr has joined #openstack-ceilometer | 23:34 | |
*** raymondr_ has joined #openstack-ceilometer | 23:38 | |
*** prad has quit IRC | 23:38 | |
*** raymondr has quit IRC | 23:40 | |
openstackgerrit | gordon chung proposed a change to openstack/pycadf: pycadf documentation https://review.openstack.org/65139 | 23:56 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!