| *** yjiang5 is now known as yjiang5_away | 00:02 | |
| *** nati_ueno has joined #openstack-ceilometer | 00:12 | |
| *** matsuhashi has joined #openstack-ceilometer | 00:23 | |
| *** nati_ueno has quit IRC | 00:29 | |
| *** igor_ has joined #openstack-ceilometer | 00:30 | |
| *** lcostantino has joined #openstack-ceilometer | 00:33 | |
| *** igor_ has quit IRC | 00:34 | |
| *** nati_ueno has joined #openstack-ceilometer | 00:37 | |
| *** fnaval has joined #openstack-ceilometer | 00:38 | |
| *** joesavak has joined #openstack-ceilometer | 00:46 | |
| *** joesavak has quit IRC | 00:52 | |
| *** prad has quit IRC | 00:55 | |
| *** liusheng has quit IRC | 00:56 | |
| *** amalagon has left #openstack-ceilometer | 01:01 | |
| *** nati_ueno has quit IRC | 01:02 | |
| *** _cjones_ has quit IRC | 01:03 | |
| *** nati_ueno has joined #openstack-ceilometer | 01:03 | |
| *** _cjones_ has joined #openstack-ceilometer | 01:04 | |
| *** _cjones_ has quit IRC | 01:08 | |
| *** nati_ueno has quit IRC | 01:14 | |
| *** lcostantino has quit IRC | 01:17 | |
| *** r-daneel has joined #openstack-ceilometer | 01:17 | |
| *** igor_ has joined #openstack-ceilometer | 01:31 | |
| *** igor_ has quit IRC | 01:35 | |
| *** nosnos has joined #openstack-ceilometer | 01:35 | |
| *** nati_ueno has joined #openstack-ceilometer | 01:45 | |
| *** nati_ueno has quit IRC | 01:49 | |
| *** liusheng has joined #openstack-ceilometer | 01:50 | |
| *** pack3t has joined #openstack-ceilometer | 01:51 | |
| *** nati_ueno has joined #openstack-ceilometer | 01:51 | |
| *** scroiset_ has joined #openstack-ceilometer | 01:53 | |
| *** eglynn has quit IRC | 01:54 | |
| *** nati_ueno has quit IRC | 01:54 | |
| *** scroiset has quit IRC | 01:58 | |
| *** packet has quit IRC | 01:58 | |
| *** Shaan7 has quit IRC | 01:58 | |
| *** pack3t is now known as packet | 01:58 | |
| *** r-daneel has quit IRC | 01:59 | |
| *** Shaan7 has joined #openstack-ceilometer | 02:04 | |
| *** Shaan7 has quit IRC | 02:04 | |
| *** Shaan7 has joined #openstack-ceilometer | 02:04 | |
| *** Shaan7 has quit IRC | 02:05 | |
| *** Shaan7 has joined #openstack-ceilometer | 02:05 | |
| *** _cjones_ has joined #openstack-ceilometer | 02:07 | |
| *** Qu310 has joined #openstack-ceilometer | 02:10 | |
| *** packet has quit IRC | 02:12 | |
| *** nati_ueno has joined #openstack-ceilometer | 02:14 | |
| *** _cjones_ has quit IRC | 02:16 | |
| *** alexpilotti has quit IRC | 02:29 | |
| *** nati_ueno has quit IRC | 02:32 | |
| *** igor_ has joined #openstack-ceilometer | 02:32 | |
| *** julim has quit IRC | 02:32 | |
| *** igor_ has quit IRC | 02:36 | |
| *** skelpter has joined #openstack-ceilometer | 02:40 | |
| *** nosnos has quit IRC | 03:03 | |
| openstackgerrit | liusheng proposed a change to openstack/python-ceilometerclient: Fix exception handling of CLI https://review.openstack.org/94554 | 03:04 |
|---|---|---|
| *** fnaval has quit IRC | 03:12 | |
| *** fnaval has joined #openstack-ceilometer | 03:13 | |
| *** zhikunliu has joined #openstack-ceilometer | 03:28 | |
| *** matsuhashi has quit IRC | 03:29 | |
| *** igor_ has joined #openstack-ceilometer | 03:32 | |
| *** matsuhashi has joined #openstack-ceilometer | 03:36 | |
| *** igor_ has quit IRC | 03:37 | |
| *** matsuhashi has quit IRC | 03:43 | |
| *** matsuhashi has joined #openstack-ceilometer | 03:43 | |
| *** matsuhas_ has joined #openstack-ceilometer | 03:45 | |
| *** matsuhashi has quit IRC | 03:45 | |
| *** nosnos has joined #openstack-ceilometer | 03:45 | |
| *** nati_ueno has joined #openstack-ceilometer | 04:28 | |
| *** matsuhas_ has quit IRC | 04:34 | |
| *** fnaval has quit IRC | 04:36 | |
| *** matsuhashi has joined #openstack-ceilometer | 04:43 | |
| *** psharma has joined #openstack-ceilometer | 05:04 | |
| openstackgerrit | liusheng proposed a change to openstack/telemetry-specs: telemetry-specs for "alarm-evaluator-db-access" https://review.openstack.org/95068 | 05:17 |
| openstackgerrit | liusheng proposed a change to openstack/telemetry-specs: telemetry-specs for "alarm-evaluator-db-access" https://review.openstack.org/95068 | 05:20 |
| *** shakayumi has joined #openstack-ceilometer | 05:30 | |
| *** igor_ has joined #openstack-ceilometer | 05:34 | |
| *** igor_ has quit IRC | 05:38 | |
| *** changbl has joined #openstack-ceilometer | 05:52 | |
| *** _nadya_ has joined #openstack-ceilometer | 05:53 | |
| *** _nadya_ has quit IRC | 05:57 | |
| *** alexpilotti has joined #openstack-ceilometer | 06:01 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/88506 | 06:02 |
| *** alexpilotti has quit IRC | 06:06 | |
| *** nati_ueno has quit IRC | 06:08 | |
| *** Ruetobas has quit IRC | 06:14 | |
| *** Ruetobas has joined #openstack-ceilometer | 06:20 | |
| *** Ruetobas has quit IRC | 06:25 | |
| *** Ruetobas has joined #openstack-ceilometer | 06:25 | |
| *** _nadya_ has joined #openstack-ceilometer | 06:28 | |
| *** _nadya_ has quit IRC | 06:38 | |
| *** nati_ueno has joined #openstack-ceilometer | 06:41 | |
| *** alexpilotti has joined #openstack-ceilometer | 06:42 | |
| *** eglynn has joined #openstack-ceilometer | 06:45 | |
| *** AJaeger has left #openstack-ceilometer | 06:45 | |
| *** _nadya_ has joined #openstack-ceilometer | 06:46 | |
| *** alexpilotti has quit IRC | 07:17 | |
| *** idegtiarov has joined #openstack-ceilometer | 07:18 | |
| *** alexpilotti has joined #openstack-ceilometer | 07:25 | |
| *** alexpilotti has quit IRC | 07:28 | |
| openstackgerrit | Mitsuru Kanabuchi proposed a change to openstack/ceilometer: Split bunch of samples into multiple messages https://review.openstack.org/94155 | 07:35 |
| *** nati_ueno has quit IRC | 07:39 | |
| *** igor has joined #openstack-ceilometer | 07:42 | |
| *** rwsu has quit IRC | 07:58 | |
| *** nacim has joined #openstack-ceilometer | 08:00 | |
| eglynn | jd__: good morning sir! | 08:04 |
| eglynn | jd__: we discussed cleaning up https://launchpad.net/~ceilometer-coresec/+members#active at yesterday's meeting | 08:04 |
| jd__ | hi eglynn | 08:05 |
| jd__ | eglynn: who do you want to drop? | 08:05 |
| eglynn | jd__: hey ... turns out I don't hez the powerz | 08:05 |
| jd__ | ha | 08:05 |
| eglynn | ... but you do! | 08:05 |
| eglynn | ... could you prune off the inactive memebers (yjiang, jtran) | 08:05 |
| jd__ | I never managed that coresec group | 08:05 |
| jd__ | I didn't even know it existed | 08:05 |
| eglynn | ... and add in the two newer cores (nprivalova & ildikov) | 08:05 |
| jd__ | what it is used for? | 08:05 |
| eglynn | I thought the interaction with the vulnerabilty mgmt ninjas? | 08:06 |
| jd__ | eglynn: go ahead I set you admin | 08:06 |
| jd__ | eglynn: probably… I wonder if it's documented | 08:06 |
| jd__ | and why it's different than -drivers | 08:06 |
| eglynn | ... if there's a secutiry bug, only the coresec team is copied? | 08:06 |
| jd__ | really no clue | 08:06 |
| eglynn | ... yeah TBH dunno if it's even used for that | 08:06 |
| eglynn | ... best I guess to dot the i's and cross the t's on it anyway | 08:07 |
| jd__ | feel free to remove me as admin once you're set up | 08:08 |
| eglynn | jd__: thanks for that, cleaned up now | 08:11 |
| eglynn | jd__: nah, happy to keep you as admin | 08:11 |
| openstackgerrit | A change was merged to openstack/ceilometer: Implemented metering for Cinder's snapshots https://review.openstack.org/92365 | 08:40 |
| *** matsuhashi has quit IRC | 08:48 | |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/telemetry-specs: Dedicated-alarm-database specs https://review.openstack.org/94857 | 09:00 |
| *** shakayumi has quit IRC | 09:01 | |
| *** kaisero has joined #openstack-ceilometer | 09:08 | |
| *** underyx is now known as Underyx|retard | 09:10 | |
| EmilienM | eglynn: hey, just to let you know I'm working on the review now. | 09:10 |
| eglynn | EmilienM: excellent, thank you sir! | 09:11 |
| EmilienM | eglynn: just to let you know that I'm working on this | 09:11 |
| EmilienM | I would appreciate help but I don't want a patch that override what I'm doing | 09:11 |
| eglynn | EmilienM: cool :) ... note that some of my review comments could just be due to my own ignorance of how grenade works in detail | 09:11 |
| EmilienM | it happened last time with one of your mates | 09:12 |
| eglynn | EmilienM: understood! :) | 09:12 |
| EmilienM | cool. | 09:12 |
| eglynn | EmilienM: (... you're talking about fvollero, right?) | 09:12 |
| EmilienM | nah | 09:12 |
| EmilienM | eglynn: we finally used to work together on the patch, but at the first, we were not synchronized. | 09:12 |
| EmilienM | I don't want to do it again, so it's good to know someone else is going to work in it | 09:13 |
| eglynn | EmilienM: yeap, I hear ya! apologies for that ... | 09:13 |
| eglynn | EmilienM: ... on the grenade stuff | 09:13 |
| EmilienM | so if he wants to help, maybe could he have a look at the current patch | 09:13 |
| EmilienM | eglynn: apologies for what? your mate is awesome, I met him at the last summit | 09:14 |
| eglynn | EmilienM: ... yeah that would be good, I'll suggest that to Chris | 09:14 |
| EmilienM | without him I don't know if we could have Neutron testing in Grenade | 09:14 |
| eglynn | EmilienM: ... well, for the initial lack of synchronization | 09:14 |
| eglynn | EmilienM: (the apols) | 09:14 |
| EmilienM | ok | 09:14 |
| EmilienM | btw, I agree with you at some points for the refacto | 09:15 |
| EmilienM | but maybe they should be addressed in another patch | 09:15 |
| *** shardy_afk is now known as shardy | 09:15 | |
| eglynn | EmilienM: ... also in terms of where cdent could contribute to the grenade effort, I was thinking about improvements to javelin | 09:16 |
| eglynn | EmilienM: ... i.e. the resources intended to survive across the upgrade | 09:16 |
| EmilienM | eglynn: that's a very good idea, javelin is so light today | 09:16 |
| EmilienM | yep | 09:16 |
| *** matsuhashi has joined #openstack-ceilometer | 09:16 | |
| eglynn | EmilienM: ... I was thinking it would be good to be able to make project-specific assertions about those javelin resources | 09:16 |
| EmilienM | alarms, meters, etc | 09:16 |
| eglynn | EmilienM: ... or making a ceilo stats query for cpu_util on a surviving instance, with the query time duration spanning the old and the new | 09:17 |
| EmilienM | eglynn: yeah, it would be awesome to test it. | 09:18 |
| eglynn | EmilienM: ... cool, it might be something that cdent could work on in parallel with your efforts? | 09:18 |
| eglynn | EmilienM: ... i.e. without stepping on your toes | 09:18 |
| EmilienM | eglynn: yeah sure | 09:18 |
| eglynn | EmilienM: cool! | 09:18 |
| EmilienM | eglynn: imho, javelin should be another patchset | 09:19 |
| eglynn | EmilienM: ... totally agree | 09:19 |
| EmilienM | and I'm sure he knows better ceilometer than me, so maybe he would be more easy for him to understand what should been tested | 09:19 |
| EmilienM | s/he/it/ | 09:19 |
| eglynn | EmilienM: ... building on your patchset would be most logical, right? | 09:19 |
| EmilienM | yes, right | 09:19 |
| eglynn | cool | 09:19 |
| EmilienM | like i'm doing with https://review.openstack.org/#/c/94547/ | 09:20 |
| eglynn | EmilienM: cool, I'll talk to cdent about that kind of split when he comes online later on today | 09:20 |
| EmilienM | eglynn: I think the best thing could be a separate patchset on top of mines | 09:20 |
| EmilienM | eglynn: otherwise he won't have ceilometer installed & upgraded | 09:21 |
| eglynn | EmilienM: ... exactly what I had in mind | 09:21 |
| EmilienM | cool, let him now he can reach me anytime on IRC/email | 09:21 |
| eglynn | EmilienM: ... excellent, thank you sir! | 09:21 |
| EmilienM | eglynn: thanks for coordinating efforts, grenade is so important for QA... | 09:21 |
| eglynn | EmilienM: ... and is v. important for ceilo too, coz of our "marching orders" from the TC | 09:23 |
| EmilienM | I see | 09:23 |
| *** _nadya_ has quit IRC | 09:23 | |
| *** _nadya_ has joined #openstack-ceilometer | 09:28 | |
| *** safchain has joined #openstack-ceilometer | 09:31 | |
| *** liusheng has quit IRC | 09:36 | |
| *** nosnos has quit IRC | 09:45 | |
| *** eglynn is now known as eglynn-afk | 09:52 | |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/telemetry-specs: Dedicated-alarm-database specs https://review.openstack.org/94857 | 09:55 |
| sileht | eglynn-afk, I have fixed the nits https://review.openstack.org/94857 | 09:56 |
| sileht | jd__, ^ | 09:56 |
| *** zhikunliu has quit IRC | 10:05 | |
| *** _cjones_ has joined #openstack-ceilometer | 10:08 | |
| *** zhikunliu has joined #openstack-ceilometer | 10:11 | |
| *** matsuhashi has quit IRC | 10:12 | |
| *** _cjones_ has quit IRC | 10:13 | |
| *** matsuhas_ has joined #openstack-ceilometer | 10:15 | |
| *** eglynn-afk is now known as eglynn | 10:17 | |
| eglynn | sileht: thank you sir! ... +2'd | 10:18 |
| *** openstackstatus has quit IRC | 10:20 | |
| *** openstack has joined #openstack-ceilometer | 10:21 | |
| *** openstackstatus has joined #openstack-ceilometer | 10:21 | |
| *** ChanServ sets mode: +v openstackstatus | 10:21 | |
| *** zhikunliu has quit IRC | 10:23 | |
| *** fnaval has joined #openstack-ceilometer | 10:47 | |
| *** matsuhas_ has quit IRC | 10:55 | |
| *** matsuhashi has joined #openstack-ceilometer | 10:57 | |
| *** matsuhashi has quit IRC | 10:58 | |
| *** matsuhashi has joined #openstack-ceilometer | 10:58 | |
| *** matsuhashi has quit IRC | 10:58 | |
| *** matsuhashi has joined #openstack-ceilometer | 10:59 | |
| *** igor_ has joined #openstack-ceilometer | 11:05 | |
| *** igor has quit IRC | 11:08 | |
| *** Underyx|retard is now known as underyx | 11:13 | |
| *** fnaval has quit IRC | 11:19 | |
| *** liusheng has joined #openstack-ceilometer | 11:35 | |
| *** psharma has quit IRC | 11:52 | |
| *** gpocentek has quit IRC | 11:52 | |
| *** eglynn is now known as eglynn-lunch | 12:17 | |
| *** _nadya_ has quit IRC | 12:20 | |
| *** matsuhashi has quit IRC | 12:20 | |
| *** matsuhashi has joined #openstack-ceilometer | 12:21 | |
| *** shakayumi has joined #openstack-ceilometer | 12:23 | |
| *** gordc has joined #openstack-ceilometer | 12:25 | |
| *** matsuhashi has quit IRC | 12:26 | |
| *** underyx is now known as Underyx | 12:26 | |
| *** fnaval has joined #openstack-ceilometer | 12:30 | |
| openstackgerrit | gordon chung proposed a change to openstack/ceilometer: refactor sql backend to improve write speed https://review.openstack.org/94483 | 12:35 |
| *** jdob has joined #openstack-ceilometer | 12:37 | |
| *** matsuhashi has joined #openstack-ceilometer | 12:37 | |
| *** matsuhashi has quit IRC | 12:38 | |
| *** matsuhashi has joined #openstack-ceilometer | 12:38 | |
| *** eglynn-lunch has quit IRC | 12:38 | |
| *** matsuhashi has quit IRC | 12:38 | |
| *** matsuhashi has joined #openstack-ceilometer | 12:39 | |
| *** cdent has joined #openstack-ceilometer | 12:39 | |
| *** AMike has quit IRC | 12:39 | |
| openstackgerrit | A change was merged to openstack/telemetry-specs: Dedicated-alarm-database specs https://review.openstack.org/94857 | 12:40 |
| *** prad_ has joined #openstack-ceilometer | 12:41 | |
| *** matsuhashi has quit IRC | 12:43 | |
| *** _nadya_ has joined #openstack-ceilometer | 12:45 | |
| *** erecio has joined #openstack-ceilometer | 12:51 | |
| openstackgerrit | Ilya Tyaptin proposed a change to openstack/ceilometer: [HBase] get_resource optimization https://review.openstack.org/87249 | 12:53 |
| *** thomasem has joined #openstack-ceilometer | 12:56 | |
| *** Underyx is now known as underyx | 13:02 | |
| *** thomasem_ has joined #openstack-ceilometer | 13:02 | |
| *** thomasem has quit IRC | 13:03 | |
| *** Ju has quit IRC | 13:07 | |
| *** joesavak has joined #openstack-ceilometer | 13:07 | |
| *** julim has joined #openstack-ceilometer | 13:09 | |
| *** cdent has quit IRC | 13:14 | |
| *** rbak has joined #openstack-ceilometer | 13:15 | |
| *** eglynn has joined #openstack-ceilometer | 13:16 | |
| *** rbak has quit IRC | 13:20 | |
| *** fnaval has quit IRC | 13:25 | |
| *** ddieterly has joined #openstack-ceilometer | 13:34 | |
| openstackgerrit | A change was merged to openstack/pycadf: sync oslo https://review.openstack.org/94883 | 13:35 |
| *** r-daneel has joined #openstack-ceilometer | 13:44 | |
| *** _nadya_ has quit IRC | 13:57 | |
| openstackgerrit | Christian Berendt proposed a change to openstack/ceilometer: debug level logs should not be translated https://review.openstack.org/93021 | 13:57 |
| *** fnaval has joined #openstack-ceilometer | 14:02 | |
| openstackgerrit | A change was merged to openstack/ceilometer: Use None instead of mutables in method params defaults https://review.openstack.org/91046 | 14:03 |
| *** cdent has joined #openstack-ceilometer | 14:04 | |
| jd__ | python2.6: >>> gnocchi.openstack.common.jsonutils.loads("\"foo\"") | 14:06 |
| jd__ | 'foo' | 14:06 |
| jd__ | python2.7: >>> gnocchi.openstack.common.jsonutils.loads("\"foo\"") | 14:07 |
| jd__ | u'foo' | 14:07 |
| jd__ | :( | 14:07 |
| *** shakayumi has quit IRC | 14:07 | |
| *** rwsu has joined #openstack-ceilometer | 14:08 | |
| jd__ | ok so that's caused by using simplejson instead of json | 14:08 |
| jd__ | booh booh. | 14:08 |
| *** _cjones_ has joined #openstack-ceilometer | 14:11 | |
| openstackgerrit | gordon chung proposed a change to openstack/ceilometer: refactor sql backend to improve write speed https://review.openstack.org/94483 | 14:15 |
| *** _cjones_ has quit IRC | 14:15 | |
| *** raymondr has joined #openstack-ceilometer | 14:27 | |
| *** _nadya_ has joined #openstack-ceilometer | 14:33 | |
| *** cmart has joined #openstack-ceilometer | 14:34 | |
| *** _nadya_ has quit IRC | 14:38 | |
| cdent | eglynn, EmilienM: I've got a question about the process associated with the grenade changes, if you've a moment. | 14:43 |
| eglynn | cdent: shoot | 14:44 |
| cdent | So my usual dev routine for thing X is: make some changes, run some tests, fix some broken stuff, commit | 14:45 |
| cdent | It seems, at least with grenade and maybe with other things too, that the process here is reason about the right changes, commit them, and then hope | 14:46 |
| cdent | as in: hope that Jenkins etc will let you know where you went wrong | 14:46 |
| cdent | But in the case of these changes because ceilometer doesn't really run in any huge way in grenade we can't really know much of that either. | 14:47 |
| cdent | This confusion is causing a bit of wurble in my head as I try to think about this stuff. | 14:47 |
| eglynn | cdent: do you mean, "shouldn't I be testing this stuff locally before proposing to gerrit and letting CI find my errors?" | 14:48 |
| cdent | effectively, yes | 14:48 |
| eglynn | cdent: in general the answer would be an emphatic yes! | 14:48 |
| eglynn | cdent: ... however | 14:48 |
| cdent | but specifically in this case I'm not sure that CI _can_ find my errors | 14:48 |
| cdent | because, e.g., mongo isn't running | 14:48 |
| eglynn | cdent: ... in the grenade case, there seems to be a significant barrier to entry in terms of local testing | 14:49 |
| cdent | or even: ceilometer is not enabled by default in devstack (or has that changed?) | 14:49 |
| eglynn | cdent: ... one contributor at the grenade session at summit said he resorted to just pattern-matching the changes he thought were needed and allowing Jenkins to find his errors | 14:49 |
| eglynn | cdent: ceilometer isn't enabled by default in devstack | 14:50 |
| eglynn | one sec | 14:50 |
| cdent | brb | 14:50 |
| eglynn | cdent: see here for the localrc required to enable ceilometer ... http://docs.openstack.org/developer/ceilometer/install/development.html | 14:50 |
| eglynn | cdent: also in order to run devstack with mongodb in place of mysql (the default) you'll need s/mysql/mongodb/ in this line: | 14:53 |
| eglynn | https://github.com/openstack-dev/devstack/blob/master/lib/ceilometer#L49 | 14:53 |
| cdent | yeah, I know that stuff, I'm just trying to figure out if there is a bit of paradox of throwing ceilometer changes at CI when they're not really getting engaged | 14:54 |
| cdent | (basically I'm trying to verify my understanding of what's going on) | 14:57 |
| eglynn | cdent: do you mean, "there are not many ceilometer tempest tests, so what does running tempest against the old and the new actually prove? | 14:57 |
| eglynn | " | 14:57 |
| eglynn | cdent: or "I don't think ceilometer is even enabled in the devstack against which grenade runs" | 14:57 |
| cdent | I'm asking about the latter. | 14:58 |
| cdent | But if it is running, then your first statement as well. | 14:58 |
| cdent | one of the review comments mention it | 14:59 |
| eglynn | cdent: a-ha, k ... so tempest is currently running against ceilometer in a limited way, so the devstack used must not be the default out-of-the-box devstack | 14:59 |
| eglynn | cdent: similarly at summit, it was mentioned that the config used by grenade is quite specific and differs from the default | 14:59 |
| eglynn | cdent: I'd have to dig to find exactly where those non-defaults are set | 15:00 |
| * cdent will do the digging | 15:00 | |
| eglynn | cdent: so on to the point ""there are not many ceilometer tempest tests, so what does running tempest against the old and the new actually prove?" | 15:00 |
| cdent | I have to say I'm very surprised by the extent to which the testing setup is imperative rather than declarative. I'm not saying it is bad, just surprised. | 15:01 |
| eglynn | cdent: ... ceilo test count is currently non-zero (just about!) so that's better than nothing | 15:01 |
| eglynn | cdent: ... ceilo test count will increase when we have the sql-a driver in a viable state for the gate | 15:02 |
| eglynn | cdent: ... and also grenade can be extended with assertions on the surviving (javelin) resources, independent of tempest test coverage | 15:02 |
| *** packet has joined #openstack-ceilometer | 15:03 | |
| eglynn | cdent: on your point "very surprised by the extent to which the testing setup is imperative rather than declarative" | 15:03 |
| eglynn | cdent: ... there are probably historical/cultural reasons for the bash-orientation | 15:03 |
| eglynn | cdent: ... but for us, we gotta go to war with the army we've got | 15:04 |
| cdent | :) | 15:04 |
| * eglynn mangles a Rumsfeld-ism ;) | 15:04 | |
| cdent | there are the things we know, and the things we don't know and the things we don't know we don't know | 15:04 |
| eglynn | cdent: touché :) | 15:05 |
| cdent | that's one interesting thing rumsfeld ever said | 15:05 |
| cdent | otherwise he's icky | 15:05 |
| eglynn | LOL :) ... he was one crazy old dude! | 15:06 |
| eglynn | s/was/is/ | 15:06 |
| cdent | is it possible, in gerrit to link to a particular person's entry in the history of a review? | 15:06 |
| *** amalagon_ has joined #openstack-ceilometer | 15:07 | |
| cdent | anyway, it's Dean's first comment on here that has me confused: https://review.openstack.org/#/c/94468/ | 15:07 |
| *** idegtiarov has quit IRC | 15:08 | |
| cdent | nevermind, found it: https://review.openstack.org/#/c/94547/4 | 15:08 |
| *** igor_ has quit IRC | 15:09 | |
| eglynn | cdent: ... did you mean https://review.openstack.org/#/c/94468/1/from-havana/upgrade-ceilometer | 15:09 |
| eglynn | i.e. for Dean's first comment | 15:09 |
| cdent | not the comment itself, his first message on the review | 15:10 |
| cdent | where he says "ceilometer doesn't run" | 15:10 |
| cdent | 94547 is the related patch which does make it run | 15:11 |
| * eglynn reads "For real use it gets enabled in devstack-gate" as "we use a non-default localrc in the devstack gate to enable ceilometer" | 15:12 | |
| eglynn | cdent: the key problem seems to be that https://review.openstack.org/94547 depends on https://review.openstack.org/94468 | 15:12 |
| eglynn | cdent: should be the other way round, no? | 15:13 |
| cdent | I haven't quite grokked the dependency structure yet. | 15:13 |
| cdent | Anyway, if it is enabled in the gate, why have a dependent patch: wouldn't you just have a local hack of some kind? | 15:13 |
| eglynn | cdent: so I think Elilien just needs to do a git rebase -i and transpose the two commits | 15:13 |
| * cdent feels rather bewildered :) | 15:14 | |
| cdent | And at the same time I feel like I'm asking questions like "why is the sky blue?" | 15:14 |
| eglynn | cdent: IIUC it's because grenade doesn't use the same non-default localrc as the non-default localrc used in the tempest/devtsack gate | 15:15 |
| eglynn | cdent: ... i.e. there's no global "gate" config for tempest that applies everywhere | 15:15 |
| * cdent blinks | 15:15 | |
| cdent | I don't think I quite followed that at all. Let me review a bit. | 15:15 |
| eglynn | cdent: ... note that tempest also runs *independently* of grenade in the CI gate | 15:16 |
| * cdent nods | 15:16 | |
| eglynn | cdent: ... tempest came first, grenade is a newer innovation | 15:16 |
| eglynn | cdent: ... both use a crafted localrc with non-default settings | 15:16 |
| eglynn | cdent: ... these non-default localrc files are different for tempest-in-grenade and tempest-standalone-in-the-gate | 15:17 |
| cdent | When Dean says "enabled in devstack-gate" what context is that? | 15:18 |
| *** amalagon_ has quit IRC | 15:18 | |
| cdent | s/what/which/ | 15:19 |
| *** amalagon__ has joined #openstack-ceilometer | 15:19 | |
| eglynn | cdent: ... my reading was that he meant tempest-standalone-in-the-gate | 15:19 |
| eglynn | cdent: ... maybe it would be quicker over the phone? | 15:20 |
| cdent | so the dependent patch is to get it going in automated grenade runs? | 15:20 |
| cdent | I think I mostly get it now. | 15:20 |
| eglynn | cdent: yep | 15:20 |
| cdent | well we're due on the phone in a few minutes anyway | 15:21 |
| eglynn | cdent: ... that's why I said the dependencies should be reversed | 15:21 |
| eglynn | cdent: ... yeah my thought exactly | 15:21 |
| eglynn | cdent: ... so I'll grab a quick coffee, then give you a bell at half past the hour? | 15:21 |
| cdent | yup | 15:22 |
| eglynn | cool | 15:23 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Allow to have different DB for alarm and metering https://review.openstack.org/94829 | 15:43 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Update testrepository configuration https://review.openstack.org/95199 | 15:51 |
| *** Ruetobas has quit IRC | 16:01 | |
| *** Ruetobas has joined #openstack-ceilometer | 16:03 | |
| cmart | Hello.. Anyone with expertise in alarming and db's around? | 16:05 |
| cmart | Mehdi Abaakoouk, I don´t know what's your irc name :S | 16:06 |
| cmart | sileht? | 16:06 |
| *** _cjones_ has joined #openstack-ceilometer | 16:07 | |
| *** Ruetobas has quit IRC | 16:09 | |
| *** Ruetobas has joined #openstack-ceilometer | 16:14 | |
| *** nacim has quit IRC | 16:15 | |
| openstackgerrit | A change was merged to openstack/telemetry-specs: Gnocchi specs https://review.openstack.org/94834 | 16:16 |
| *** safchain has quit IRC | 16:18 | |
| *** raymondr has quit IRC | 16:35 | |
| *** raymondr_ has joined #openstack-ceilometer | 16:39 | |
| *** nati_ueno has joined #openstack-ceilometer | 16:50 | |
| *** yjiang5_away is now known as yjiang5 | 17:02 | |
| *** nati_uen_ has joined #openstack-ceilometer | 17:03 | |
| *** nati_ueno has quit IRC | 17:06 | |
| *** nati_uen_ has quit IRC | 17:07 | |
| *** jaypipes is now known as leakypipes | 17:08 | |
| *** nati_ueno has joined #openstack-ceilometer | 17:08 | |
| *** eglynn has quit IRC | 17:27 | |
| *** _nadya_ has joined #openstack-ceilometer | 17:29 | |
| *** yjiang5 is now known as zjiang5 | 17:31 | |
| *** zjiang5 is now known as zjiang6 | 17:31 | |
| _nadya_ | gordc: hi! are you here? | 17:35 |
| *** nati_ueno has quit IRC | 17:35 | |
| gordc | _nadya_: hey, whatsup? | 17:35 |
| _nadya_ | gordc: is https://review.openstack.org/#/c/94483/ critical for tempest? | 17:36 |
| gordc | _nadya_: it's needed to get multiple workers working for collector... it also improves write speed in general. | 17:37 |
| gordc | i don't think dropping user/project table is enough to improve write speed in sql | 17:38 |
| _nadya_ | gordc: ok, I see. and why don't you want to merge several collectors to devstack? I see red cross from you on your patch | 17:39 |
| gordc | _nadya_: oh, that patch is dependent on the patch above. the devstack multiple worker patch doesn't work with current code. (you can see errors in collector logs) | 17:41 |
| gordc | once the patch above merges, i'll recheck the devstack patch. | 17:42 |
| _nadya_ | gordc: aha. and how are you going to test? do you need any help? we can test these 2 patches on gating-like env, I hope Vadim will be available for this activity on Monday | 17:45 |
| gordc | _nadya_: i gave it a try locally and it worked fine... i also changed collector workers in patchset 5 and the gate tests appear to pass fine as well. | 17:47 |
| gordc | _nadya_: if you have resources then feel free to pull in the patch and test against multiple collector workers. that'd be good. | 17:48 |
| openstackgerrit | ZhiQiang Fan proposed a change to openstack/ceilometer: Fix project authorization check https://review.openstack.org/95229 | 17:50 |
| _nadya_ | gordc: ok, will keep you posted | 17:51 |
| gordc | cool cool. thanks _nadya_ | 17:52 |
| *** _nadya_ has quit IRC | 17:52 | |
| cdent | jd__: you about? I'm trying to understand the term "entity" from the gnocchi spec (looks very good by the way, API is relatively easy to grasp). | 17:54 |
| *** eglynn has joined #openstack-ceilometer | 18:01 | |
| *** eglynn has quit IRC | 18:09 | |
| *** _nadya_ has joined #openstack-ceilometer | 18:20 | |
| *** cjchand_ has joined #openstack-ceilometer | 18:33 | |
| *** packet has quit IRC | 18:38 | |
| *** packet has joined #openstack-ceilometer | 18:39 | |
| *** fnaval has quit IRC | 18:46 | |
| *** fnaval has joined #openstack-ceilometer | 18:47 | |
| *** fnaval has quit IRC | 18:51 | |
| *** cjchand_ has quit IRC | 18:53 | |
| *** _nadya_ has quit IRC | 18:55 | |
| *** prad has joined #openstack-ceilometer | 18:59 | |
| *** prad_ has quit IRC | 19:00 | |
| *** prad_ has joined #openstack-ceilometer | 19:01 | |
| *** fnaval has joined #openstack-ceilometer | 19:01 | |
| *** prad has quit IRC | 19:04 | |
| *** prad_ is now known as prad | 19:04 | |
| *** jdob has quit IRC | 19:08 | |
| *** eglynn has joined #openstack-ceilometer | 19:09 | |
| *** lsmola has quit IRC | 19:20 | |
| *** cdent has quit IRC | 19:24 | |
| *** erecio_1 has joined #openstack-ceilometer | 19:36 | |
| *** erecio has quit IRC | 19:39 | |
| *** eglynn has quit IRC | 19:48 | |
| *** jsavak has joined #openstack-ceilometer | 19:51 | |
| *** joesavak has quit IRC | 19:54 | |
| *** nati_ueno has joined #openstack-ceilometer | 19:54 | |
| *** erecio_2 has joined #openstack-ceilometer | 19:58 | |
| *** erecio_1 has quit IRC | 20:01 | |
| *** erecio_2 has quit IRC | 20:14 | |
| -openstackstatus- NOTICE: Gerrit will be offline for about 20 minutes in order to rename some projects starting at 21:00 UTC. | 20:33 | |
| *** nati_uen_ has joined #openstack-ceilometer | 20:36 | |
| *** nati_ueno has quit IRC | 20:39 | |
| *** joesavak has joined #openstack-ceilometer | 20:41 | |
| *** raymondr_ has quit IRC | 20:41 | |
| *** nati_ueno has joined #openstack-ceilometer | 20:43 | |
| *** jsavak has quit IRC | 20:44 | |
| *** nati_uen_ has quit IRC | 20:45 | |
| *** amalagon__ has left #openstack-ceilometer | 20:46 | |
| *** fnaval has quit IRC | 20:46 | |
| *** nati_uen_ has joined #openstack-ceilometer | 20:46 | |
| *** nati_ueno has quit IRC | 20:48 | |
| *** raymondr_ has joined #openstack-ceilometer | 20:49 | |
| *** fnaval has joined #openstack-ceilometer | 20:52 | |
| *** raymondr_ has quit IRC | 20:52 | |
| *** nati_uen_ has quit IRC | 20:53 | |
| *** nati_uen_ has joined #openstack-ceilometer | 20:55 | |
| *** gordc has left #openstack-ceilometer | 20:56 | |
| *** nati_ueno has joined #openstack-ceilometer | 20:58 | |
| *** nati_uen_ has quit IRC | 21:00 | |
| *** joesavak has quit IRC | 21:03 | |
| *** cmart has quit IRC | 21:07 | |
| *** eglynn has joined #openstack-ceilometer | 21:22 | |
| *** raymondr_ has joined #openstack-ceilometer | 21:23 | |
| *** Ruetobas has quit IRC | 21:26 | |
| -openstackstatus- NOTICE: Gerrit is offline in order to rename some projects. ETA: 22:00. | 21:33 | |
| *** ChanServ changes topic to "Gerrit is offline in order to rename some projects. ETA: 22:00." | 21:33 | |
| *** Ruetobas has joined #openstack-ceilometer | 21:33 | |
| *** raymondr_ has quit IRC | 21:37 | |
| *** nati_ueno has quit IRC | 21:37 | |
| *** raymondr_ has joined #openstack-ceilometer | 21:38 | |
| *** Ruetobas has quit IRC | 21:38 | |
| *** thomasem_ has quit IRC | 21:41 | |
| *** Ruetobas has joined #openstack-ceilometer | 21:44 | |
| *** fnaval has quit IRC | 21:48 | |
| *** packet has quit IRC | 21:49 | |
| *** _nadya_ has joined #openstack-ceilometer | 21:56 | |
| *** _nadya_ has quit IRC | 22:00 | |
| *** Qu310 has quit IRC | 22:08 | |
| *** fnaval has joined #openstack-ceilometer | 22:11 | |
| *** Shaan7 has quit IRC | 22:12 | |
| *** Shaan7 has joined #openstack-ceilometer | 22:13 | |
| *** openstackgerrit has quit IRC | 22:14 | |
| *** openstackgerrit has joined #openstack-ceilometer | 22:14 | |
| *** openstackstatus has quit IRC | 22:18 | |
| *** openstack has joined #openstack-ceilometer | 22:18 | |
| *** liusheng has quit IRC | 22:19 | |
| *** openstackstatus has joined #openstack-ceilometer | 22:19 | |
| *** ChanServ sets mode: +v openstackstatus | 22:19 | |
| *** liusheng has joined #openstack-ceilometer | 22:19 | |
| *** nati_ueno has joined #openstack-ceilometer | 22:23 | |
| *** nati_ueno has quit IRC | 22:26 | |
| *** prad has quit IRC | 22:26 | |
| *** ChanServ changes topic to "OpenStack Telemetry (Ceilometer) | https://wiki.openstack.org/wiki/Ceilometer" | 22:27 | |
| openstackgerrit | Monty Taylor proposed a change to openstack/ceilometer-specs: Updated gitreview file for repo rename https://review.openstack.org/95294 | 22:28 |
| *** fnaval has quit IRC | 22:31 | |
| *** ddieterly has quit IRC | 22:34 | |
| openstackgerrit | Monty Taylor proposed a change to openstack/ceilometer-specs: Updated gitreview file for repo rename https://review.openstack.org/95299 | 22:34 |
| *** eglynn has quit IRC | 22:48 | |
| *** nati_ueno has joined #openstack-ceilometer | 23:09 | |
| *** nati_ueno has quit IRC | 23:13 | |
| *** leakypipes has quit IRC | 23:18 | |
| *** nati_ueno has joined #openstack-ceilometer | 23:26 | |
| *** r-daneel has quit IRC | 23:34 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!