| *** ddieterly has quit IRC | 00:04 | |
| *** jaypipes has quit IRC | 00:11 | |
| *** fnaval has joined #openstack-ceilometer | 00:17 | |
| *** matsuhashi has joined #openstack-ceilometer | 00:30 | |
| *** nati_ueno has quit IRC | 00:31 | |
| *** ddieterly has joined #openstack-ceilometer | 00:33 | |
| *** nati_ueno has joined #openstack-ceilometer | 00:35 | |
| *** thomasem has quit IRC | 00:53 | |
| *** _cjones_ has quit IRC | 01:04 | |
| *** _cjones_ has joined #openstack-ceilometer | 01:04 | |
| *** liusheng has joined #openstack-ceilometer | 01:07 | |
| *** _cjones_ has quit IRC | 01:08 | |
| *** nati_ueno has quit IRC | 01:31 | |
| *** ddieterly has quit IRC | 01:43 | |
| *** ddieterly has joined #openstack-ceilometer | 01:44 | |
| *** nosnos has joined #openstack-ceilometer | 01:47 | |
| *** ddieterly has quit IRC | 01:48 | |
| *** zhikunliu has quit IRC | 01:49 | |
| *** isviridov is now known as isvirido | 01:57 | |
| *** isvirido is now known as isviridov | 01:57 | |
| *** isviridov is now known as isviridov|away | 01:57 | |
| *** thomasem has joined #openstack-ceilometer | 02:20 | |
| *** thomasem has quit IRC | 02:22 | |
| *** raymondr_ has quit IRC | 02:35 | |
| *** matsuhashi has quit IRC | 02:45 | |
| *** zhikunliu has joined #openstack-ceilometer | 02:49 | |
| *** shakayumi has quit IRC | 02:58 | |
| *** raymondr has joined #openstack-ceilometer | 03:29 | |
| *** nosnos has quit IRC | 03:40 | |
| *** matsuhashi has joined #openstack-ceilometer | 03:47 | |
| *** matsuhashi has quit IRC | 03:54 | |
| *** matsuhashi has joined #openstack-ceilometer | 03:54 | |
| *** matsuhashi has quit IRC | 03:57 | |
| *** matsuhashi has joined #openstack-ceilometer | 03:58 | |
| *** matsuhashi has quit IRC | 04:03 | |
| *** matsuhashi has joined #openstack-ceilometer | 04:03 | |
| *** psharma has joined #openstack-ceilometer | 04:22 | |
| *** nosnos has joined #openstack-ceilometer | 04:42 | |
| *** raymondr has quit IRC | 04:49 | |
| *** inc0_ has joined #openstack-ceilometer | 05:54 | |
| *** inc0_ has quit IRC | 05:54 | |
| *** inc0_ has joined #openstack-ceilometer | 05:55 | |
| *** _nadya_ has joined #openstack-ceilometer | 05:57 | |
| *** ildikov has quit IRC | 05:57 | |
| *** _nadya_ has quit IRC | 06:07 | |
| *** ildikov has joined #openstack-ceilometer | 06:18 | |
| *** matsuhashi has quit IRC | 06:30 | |
| *** matsuhashi has joined #openstack-ceilometer | 06:31 | |
| *** fnaval has quit IRC | 06:33 | |
| *** _nadya_ has joined #openstack-ceilometer | 06:34 | |
| *** _nadya_ has quit IRC | 06:49 | |
| *** _nadya_ has joined #openstack-ceilometer | 06:53 | |
| *** matsuhashi has quit IRC | 07:11 | |
| *** matsuhas_ has joined #openstack-ceilometer | 07:16 | |
| *** matsuhas_ has quit IRC | 07:16 | |
| *** matsuhashi has joined #openstack-ceilometer | 07:17 | |
| *** IvanBerezovskiy has joined #openstack-ceilometer | 07:27 | |
| *** idegtiarov has joined #openstack-ceilometer | 07:39 | |
| *** eglynn has quit IRC | 07:54 | |
| *** nacim has joined #openstack-ceilometer | 07:55 | |
| jd__ | ildikov: what's wrong with https://review.openstack.org/#/c/96258/ that you -2'ed it? | 07:58 |
|---|---|---|
| ildikov | jd__: it's just marked as WIP | 08:00 |
| *** r0j4z0 has quit IRC | 08:00 | |
| *** lsmola has quit IRC | 08:07 | |
| liusheng | Hi | 08:09 |
| liusheng | I have reported a bug, https://bugs.launchpad.net/python-ceilometerclient/+bug/1324759 about "--resource-metadata" is unavailable in sample-create | 08:10 |
| liusheng | but, when check the code and trying to upload a patch, I found I have made a mistake, | 08:10 |
| liusheng | the right way to use sample-create is: | 08:10 |
| liusheng | ceilometer --debug sample-create -r resource1 -m meter1 --meter-type delta --meter-unit unit1 --sample-volume 1 --resource-metadata '{"abc":"abc"}' | 08:10 |
| liusheng | unlike other metadata related commands of openstack, is the usage of --resource-metadata a little weird ? | 08:12 |
| *** lsmola has joined #openstack-ceilometer | 08:13 | |
| *** admin0 has joined #openstack-ceilometer | 08:20 | |
| *** shardy_afk is now known as shardy | 08:28 | |
| ildikov | jd__: sorry, I had a meeting | 08:30 |
| jd__ | ildikov: np | 08:30 |
| *** eglynn has joined #openstack-ceilometer | 08:30 | |
| ildikov | jd__: so I would like to ask a few things from dhellmann before mark it as "ready to fly" | 08:30 |
| jd__ | fair enough | 08:30 |
| jd__ | I was just curious | 08:30 |
| ildikov | jd__: if you have a few sec to check it and see something too odd, then just scream, not all my questions are sphinx related :) | 08:31 |
| ildikov | jd__: BTW do you know any details about venv in tox.ini? | 08:31 |
| jd__ | ildikov: probably, shoot | 08:31 |
| ildikov | jd__: so it is used for the docs job and for packaging | 08:32 |
| ildikov | jd__: but now it starts a Mongo instance, which is not required for these two I mentioned | 08:32 |
| ildikov | jd__: so I started to check that where else it is used | 08:32 |
| *** safchain has joined #openstack-ceilometer | 08:33 | |
| jd__ | ildikov: mongo is started only in the unit test run, the docs target in tox.ini does not use it | 08:34 |
| ildikov | jd__: I did not find too many other jobs, but if you know anything, which needs it, then please share :) | 08:34 |
| ildikov | jd__: the docs gate job use the venv for running build_sphinx | 08:34 |
| jd__ | yeah it's jused used for the py* targets | 08:34 |
| jd__ | ildikov: ah I didn't know that | 08:34 |
| ildikov | jd__: and that gate job is generic for all projects | 08:34 |
| jd__ | we could try to remove that setup-env call I guess | 08:35 |
| jd__ | not sure why it's there | 08:35 |
| ildikov | is it really used for the py* targets? | 08:35 |
| jd__ | not the one in venv, since they have their own defiition | 08:36 |
| ildikov | so I just checked that venv for Nova for instance and there was only a {posargs} string and nothing else | 08:36 |
| jd__ | so it's safe to remove I'd say | 08:36 |
| jd__ | just try and send a patch,you'll see :) | 08:36 |
| ildikov | jd__: ok, cool, I will upload another WIP patch for this :) | 08:36 |
| ildikov | jd__: that was the plan, but if I saw right in git you created that venv, so I thought that it worth a question :) | 08:37 |
| jd__ | sure :) | 08:37 |
| ildikov | jd__: another thing that I volunteered yesterday for the pipeline part of the Gnocchi integration | 08:38 |
| ildikov | jd__: I do not know when it will be actual, I will try to catch up as soon as I can | 08:38 |
| jd__ | ok | 08:38 |
| ildikov | jd__: so if you need more hands, just let me know | 08:38 |
| jd__ | thanks ildikov! | 08:40 |
| ildikov | jd__: it's true in general of course, so if it reached the state of having multiple tasks that can be done in parallel, then I will try to participate | 08:40 |
| ildikov | jd__: :) | 08:40 |
| ildikov | jd__: thanks for the tox help, I need to run now, it is a meeting day :S | 08:41 |
| jd__ | you're welcome, have fun ildikov :) | 08:41 |
| ildikov | jd__: tnx, the internal project meetings are always fun ;) | 08:41 |
| *** ildikov has quit IRC | 08:46 | |
| *** ildikov has joined #openstack-ceilometer | 08:47 | |
| *** _nadya_ has quit IRC | 08:50 | |
| *** raymondr has joined #openstack-ceilometer | 08:50 | |
| *** raymondr has quit IRC | 08:55 | |
| eglynn | jd__: hola! | 08:57 |
| jd__ | eglynn: hola amigo como esta in tu casa? | 08:57 |
| eglynn | jd__: ... you have surpassed the limits of my language skills ;) | 08:58 |
| eglynn | jd__: we've a couple of backports needing a +1 to get them over the line ... https://review.openstack.org/95988 & https://review.openstack.org/95489 | 08:58 |
| jd__ | vamos a la playa senor zorro! | 08:58 |
| eglynn | jd__: (apevec wants to close off the first stable/icehouse release today if poss) | 08:58 |
| jd__ | k | 08:59 |
| jd__ | you actually approved https://review.openstack.org/#/c/95988/ if I read correcly no? | 09:00 |
| eglynn | jd__ sorry wrong link, I meant https://review.openstack.org/92048 | 09:00 |
| jd__ | done | 09:01 |
| eglynn | jd__: thank you sir! | 09:03 |
| *** cdent has joined #openstack-ceilometer | 09:18 | |
| zqfan | <eglynn><jd__> thanks | 09:27 |
| *** zqfan is now known as zqfan_leave | 09:27 | |
| eglynn | zqfan_leave: unfortunately the gate failing that patch https://jenkins07.openstack.org/job/gate-ceilometer-python26/491/console | 09:29 |
| eglynn | zqfan_leave: all patches on stable/icehouse failing with "I/O operation on closed file" when flushing the log file | 09:31 |
| eglynn | zqfan_leave: (.... so not specific to your patch) | 09:31 |
| *** ityaptin has joined #openstack-ceilometer | 09:31 | |
| DinaBelova | folks, btw, who's going to implement central-agents HA in the Juno? | 09:40 |
| DinaBelova | yeah, o/ hello! | 09:41 |
| eglynn | DinaBelova: cyril, possibly (TBC) | 09:42 |
| DinaBelova | eglynn, oh, ok :) | 09:42 |
| DinaBelova | and btw - some time ago I registered irc channel #tooz for the tooz folks))) | 09:43 |
| DinaBelova | nobody still there :D | 09:43 |
| DinaBelova | I said about that __ld some time ago)))) | 09:43 |
| DinaBelova | __jd ^ :) | 09:43 |
| DinaBelova | heh | 09:43 |
| DinaBelova | jd__ | 09:43 |
| jd__ | we're hanging on #openstack-state-management actually, DinaBelova | 09:43 |
| DinaBelova | :D | 09:44 |
| DinaBelova | why don't you like #tooz? :) | 09:44 |
| jd__ | it's not that I don't like, it's just that if we start doing a channel for every library… that's a lot of fragmentation :) | 09:44 |
| DinaBelova | :D | 09:45 |
| jd__ | so we're hanging out with the cool kids of taskflow and oslo | 09:45 |
| DinaBelova | okay, got it) | 09:45 |
| jd__ | I just got your dbsync patch merged btw DinaBelova | 09:45 |
| DinaBelova | hehe, I see it) | 09:45 |
| *** cdent_ has joined #openstack-ceilometer | 09:48 | |
| cdent_ | eglynn: what if anything do I need to do when this happens: "check-grenade-dsvm FAILURE in 20m 55s" | 09:49 |
| *** cdent has quit IRC | 09:49 | |
| *** cdent_ is now known as cdent | 09:49 | |
| jd__ | run | 09:49 |
| eglynn | cdent: ... grab your flak jacket and helmet ;) | 09:49 |
| cdent | that's how it feels | 09:50 |
| eglynn | cdent: ... is that reported by Jenkins? | 09:50 |
| eglynn | cdent: ... or running locally? | 09:50 |
| cdent | yeah, it's on https://review.openstack.org/#/c/96514/7 | 09:50 |
| cdent | two failures, one is a recognized error, the other not | 09:50 |
| cdent | both grenade | 09:50 |
| eglynn | cdent: ... so there are often transient failure in the tempest/grenade runs | 09:51 |
| DinaBelova | eglynn, when you'll have some time, please take a look on the HBase events spec - https://review.openstack.org/#/c/96417/ | 09:51 |
| DinaBelova | idegtiarov has fixed the comments | 09:51 |
| eglynn | cdent: ... find the related bug and then leave a comment on gerrit of form: | 09:51 |
| DinaBelova | eglynn, as j1 is close) | 09:51 |
| eglynn | cdent: recheck bug NNN | 09:51 |
| eglynn | DinaBelova: sure | 09:51 |
| DinaBelova | eglynn: thank you sir! | 09:52 |
| cdent | what about for the failure that is not recognized? | 09:52 |
| eglynn | cdent: *all* test jobs will be retried in the recheck (including check-grenade-dsvm-neutron) | 09:53 |
| cdent | ah, so by tweaking at the recognized one we can force the other one to retry and with any luck it will clear? | 09:53 |
| eglynn | cdent: ... i.e. not just the jobs for which the elastic recheck found a match (check-grenade-dsvm in your case) | 09:53 |
| cdent | that seems....flakey? | 09:53 |
| * cdent takes some anti-test-ocd meds | 09:54 | |
| eglynn | cdent: ... well the good citizen thing to do would be to dig into the logs and "classify the failure" | 09:54 |
| eglynn | cdent: ... but for now, just see how the recheck goes | 09:55 |
| cdent | thanks | 09:56 |
| eglynn | cdent: ... see "Adding Bug Signatures" in https://github.com/openstack-infra/elastic-recheck for details of adding a classification | 09:57 |
| cdent | eglynn: one more related question: | 10:02 |
| eglynn | yep? | 10:02 |
| *** vrovachev has joined #openstack-ceilometer | 10:02 | |
| cdent | this comment in the related patch: https://review.openstack.org/#/c/92074/2/ceilometerclient/v2/options.py,cm | 10:03 |
| cdent | my changes now clobber than entire section | 10:03 |
| cdent | and somehow the rebase reinjected some badness | 10:03 |
| cdent | what's the right thing to do now? | 10:03 |
| eglynn | cdent: "reinjected some badness"? | 10:04 |
| eglynn | cdent: ... as in added a conflict? | 10:04 |
| cdent | no, an already merged pep8 fix is now gone | 10:05 |
| eglynn | cdent: one sec, looking again | 10:06 |
| *** _nadya_ has joined #openstack-ceilometer | 10:07 | |
| eglynn | cdent: how exactly did you do the rebase? | 10:08 |
| * cdent locates the stuff | 10:08 | |
| cdent | https://wiki.openstack.org/wiki/Gerrit_Workflow#Add_dependency | 10:08 |
| eglynn | cdent: in the first patchset on https://review.openstack.org/92074 that line ("#single char split (<,=) ") is unchanged | 10:09 |
| eglynn | cdent: so a simple rebase of that original patch on master, would not have effected that individual like | 10:09 |
| eglynn | *lin | 10:10 |
| eglynn | *line, arggh! | 10:10 |
| eglynn | cdent: did you follow the instructions with the step "# Edit files, add files to git" ? | 10:11 |
| cdent | yes | 10:11 |
| cdent | I had been doing something much more complicated with rebase, but then found that, and since I had my changes in the stash, I just made a new branch off the parent change and then applied my changes | 10:12 |
| *** underyx is now known as Underyx|off | 10:12 | |
| eglynn | cdent: and did that "edit files" step involve manually copying over the file content from your original change, or more like "git stash pop" ? | 10:13 |
| cdent | I did a git stash apply, but what was in my stash didn't have that line. | 10:14 |
| cdent | When 'git review' ran it neeed a rebase, and there were conflicts and resolving those may have been where the problem was injected. | 10:14 |
| cdent | What I'm mostly curous about is what's the proper way to fix it, especially if there's some chance that my changes (which remove that line) will follow on. | 10:15 |
| eglynn | yeah, possibly ... which is partly why /me prefers to use git in more old-skool "obvious" way | 10:16 |
| *** matsuhashi has quit IRC | 10:16 | |
| eglynn | cdent: ... so how would you approach fixing it up? | 10:17 |
| cdent | It depends on what the preferences are for how history is handled. | 10:18 |
| cdent | If the idea is that everything on master happens in order and clean and squashed then the 'special characters" patchset commit could be amended | 10:20 |
| cdent | and then rebase my changes on that new patch | 10:20 |
| eglynn | cdent: ... preference is to make least change to the original patch, not sure what you mean by clean and squashed | 10:20 |
| eglynn | cdent: ... why not just take a simple, if round-about, approach | 10:21 |
| * cdent listens | 10:21 | |
| *** matsuhashi has joined #openstack-ceilometer | 10:21 | |
| eglynn | $ git format-patch -1 -o cdent_patch.patch ; git reset --hard HEAD^ | 10:23 |
| eglynn | $ sed -i 's/#single char split/# single char split/' ceilometerclient/v2/options.py ; git commit -a -m "add back space" | 10:23 |
| eglynn | $ sed -i 's/# single char split/#single char split/' ceilometerclient/v2/options.py ; git commit -a -m "remove space again" | 10:23 |
| eglynn | $ git am cdent_patch.patch | 10:24 |
| eglynn | $ git rebase -i origin/master | 10:24 |
| *** ildikov has quit IRC | 10:24 | |
| *** ildikov has joined #openstack-ceilometer | 10:25 | |
| eglynn | ... then squash "add back space" into original patch | 10:25 |
| eglynn | ... and squash "Refactor split_by_op and split_by_datatype" into "remove space again" | 10:25 |
| eglynn | remove the noise from the commit messages and you're done | 10:25 |
| * cdent interprets | 10:26 | |
| * eglynn 's git usage is a tad round-about ;) | 10:26 | |
| *** matsuhashi has quit IRC | 10:27 | |
| cdent | I think I'm confused because I don't understand why you are making the two commis? | 10:27 |
| cdent | commits | 10:27 |
| *** matsuhashi has joined #openstack-ceilometer | 10:28 | |
| eglynn | "add back space" and "remove space again" are inverse changes, so inserting that pair into the sequence is fine becase they cancel each other out | 10:28 |
| eglynn | the "remove space again" then just neatly avoids a conflict on the re-apply of cdent_patch.patch | 10:29 |
| eglynn | cdent: does that idea make sense? | 10:30 |
| cdent | so how is this simpler and or better than my suggestion? (what you say makes technical sense but I don't understand the goal sense, but I have to admit I'm not very awake today (yet)) | 10:31 |
| cdent | I suspect I'm missing a piece of data that hasn't clicked. | 10:32 |
| eglynn | cdent: exact same idea: fix up the old, re-apply the new | 10:34 |
| eglynn | cdent: ... the point was simply to avoid any rejects when re-applying the new | 10:34 |
| eglynn | cdent: ... but just do whatever works, it's just a single space after all | 10:35 |
| cdent | :) | 10:35 |
| cdent | Let's put "clean and squashed" on the agenda for the pub. | 10:35 |
| *** Underyx|off is now known as underyx | 10:47 | |
| vrovachev | eglynn: hello! | 10:56 |
| eglynn | vrovachev: hey | 10:56 |
| vrovachev | yesterday ~2am Sean caught me :( | 10:56 |
| vrovachev | https://docs.google.com/document/d/1KxL4gu68C0s9n6ekLEfWWD3CeYm-Q3dod2hCVAv7ZsE/edit?usp=sharing | 10:56 |
| vrovachev | and really I did not get his idea | 10:57 |
| vrovachev | it looks like he's against feature flag one more time :-\ | 10:57 |
| * eglynn looks | 10:57 | |
| vrovachev | and well - he suggests that ceilometer should manage this brunch stuff itself somehow | 10:58 |
| vrovachev | we were thinking about feature flag telemetry-quick-sql in tempest | 10:58 |
| vrovachev | that should be get from branch version in the devstack | 10:58 |
| vrovachev | from the env variable | 10:58 |
| vrovachev | and we need to get how to set this variable - I wanted to do that from devstack gate env vagiable and that's it | 10:59 |
| vrovachev | and Sean asks to set it somehow in the ceilo :-\ | 11:00 |
| *** matsuhashi has quit IRC | 11:00 | |
| * eglynn needs to google "bogomips" | 11:00 | |
| *** matsuhashi has joined #openstack-ceilometer | 11:00 | |
| eglynn | ... a-ha "bogus MIPs" | 11:00 |
| vrovachev | probably I just don't get some things... | 11:01 |
| eglynn | vrovachev: "... let's take this offline" <=== was that in the middle of the qa weekly meeting? | 11:02 |
| vrovachev | yep | 11:02 |
| eglynn | vrovachev: well, here's a naughty thought ... | 11:03 |
| eglynn | vrovachev: what if we were to "abuse" the capabilities API to make this discoverable? | 11:03 |
| eglynn | vrovachev: ... e.g. add a top-level capability called production_ready or some-such | 11:04 |
| eglynn | vrovachev: ... e.g. set it true in Juno | 11:04 |
| eglynn | vrovachev: ... but leave it absent == False in Icehouse | 11:05 |
| *** matsuhashi has quit IRC | 11:05 | |
| * eglynn no likee the tone in this comment ... | 11:05 | |
| eglynn | "just to be a bit harsh we could always rip ceilo out of the icehouse gate to get around this too..." | 11:05 |
| DinaBelova | eglynn, so do you mean we can make some ceilo request to get this info? | 11:06 |
| vrovachev | eglynn: It would solve a problem :) | 11:06 |
| eglynn | DinaBelova: yep, GET /v2/capabilities | 11:07 |
| eglynn | ... currently we just have pure API capabilities | 11:07 |
| DinaBelova | eglynn, ok, I'll provide this change | 11:07 |
| DinaBelova | and vrovachev will have the opportunity to use it | 11:07 |
| eglynn | ... this would need to be something orthogonal to API capability, more relating to an "implementation capability" | 11:08 |
| *** matsuhashi has joined #openstack-ceilometer | 11:09 | |
| eglynn | ... so maybe we have a another top-level field in that capabilities dict | 11:09 |
| DinaBelova | eglynn, I just wonder how we can make this capability branch-dependent | 11:10 |
| cdent | 'am i slow': true | 11:10 |
| eglynn | ... e.g. {"api": {"alarms:history:query:complex": true, ...}, "storage": {"production_ready": true, "highy_available": false, ...}} | 11:10 |
| eglynn | DinaBelova: well, absent capabilities are intended to be interpreted as False | 11:11 |
| eglynn | DinaBelova: ... so we just add the new capability on Juno | 11:11 |
| eglynn | DinaBelova: ... leave Icehouse untouched | 11:11 |
| DinaBelova | eglynn, a-ha! | 11:11 |
| ildikov | eglynn: s/highy/highly/ ;) | 11:12 |
| DinaBelova | so just new code in juno) | 11:12 |
| cdent | eglynn++ | 11:12 |
| * eglynn is highy as a kitey ;) | 11:12 | |
| ildikov | eglynn: :) | 11:12 |
| *** _nadya_ has quit IRC | 11:14 | |
| eglynn | ... for now, probably an idea to keep the "surface area" of that new storage capability small | 11:14 |
| eglynn | ... lest there be objections about airing the backend-deployment's "dirty laundry" in public | 11:16 |
| eglynn | (e.g. whethter the metering DB is deployed in HA mode or SPoF) | 11:16 |
| DinaBelova | eglynn, ildikov - please take a look on https://review.openstack.org/96728 | 11:17 |
| DinaBelova | it should fix our gate | 11:17 |
| eglynn | DinaBelova: is that needed on stable also? | 11:17 |
| eglynn | (stable/icehouse) | 11:18 |
| DinaBelova | eglynn - I guess if it's using icehouse keystoneclient - then no.. | 11:18 |
| DinaBelova | lemme check it | 11:18 |
| eglynn | IIRC the dependency isn't version-capped | 11:18 |
| *** openstackgerrit has joined #openstack-ceilometer | 11:18 | |
| *** _nadya_ has joined #openstack-ceilometer | 11:18 | |
| *** openstackgerrit has quit IRC | 11:19 | |
| DinaBelova | eglynn, yep, it should be backported | 11:20 |
| eglynn | vrovachev, DinaBelova: ... just thinking about this some more | 11:20 |
| eglynn | ... so maybe there's still a problem here as ceilometer will still be *running* in the icehouse case | 11:20 |
| eglynn | ... the tempest tests will be disabled == goodness | 11:20 |
| DinaBelova | eglynn, mmm, sorry, can't get your point | 11:21 |
| eglynn | ... the ceilo collector will still be chewing thru CPU == badness | 11:21 |
| DinaBelova | a-ha | 11:21 |
| DinaBelova | you want no running icehouse job at all, I guess)) :D | 11:21 |
| eglynn | well, maybe just not running the tests would suffice | 11:22 |
| eglynn | ... but wasn't the CPU load imposed by the collector an aspect of the original issue? | 11:22 |
| DinaBelova | eglynn - I guess it was | 11:22 |
| DinaBelova | and lond running message processing too | 11:23 |
| DinaBelova | and lots of other things gone after the v1 was removed, for instance | 11:23 |
| DinaBelova | the question is: do we want tests no running at all on the icehouse? | 11:23 |
| eglynn | DinaBelova: ... actually what I am saying, we do have a test already running in icehouse, right? | 11:24 |
| DinaBelova | as some of them are ok now - some alarm api I guess | 11:24 |
| eglynn | DinaBelova: ... yeah the alarm test | 11:24 |
| DinaBelova | yes, exactly | 11:24 |
| eglynn | ... so in that sense the CPU load of ceilo/icehouse must be tolerable | 11:24 |
| eglynn | (tolerable for the gate that is ...) | 11:25 |
| *** openstackgerrit has joined #openstack-ceilometer | 11:25 | |
| * eglynn was worrying unnecessarily ... | 11:25 | |
| DinaBelova | well - probably we may still have this capability "quick enough sql" and all tests that cannot pass gate without it will be marked as "we need this capability" | 11:25 |
| DinaBelova | vrovachev will call ceilo from the tests | 11:26 |
| DinaBelova | and check it | 11:26 |
| DinaBelova | is that a good plan? | 11:26 |
| vrovachev | If in icehouse runned only alam tests, it's ok | 11:26 |
| vrovachev | and i can implemented it | 11:27 |
| vrovachev | in tempest | 11:27 |
| DinaBelova | eglynn? | 11:27 |
| DinaBelova | ^^ | 11:27 |
| eglynn | DinaBelova, vrovachev: ... so would it be elegant to have a new tempest test decorator along the lines of: | 11:27 |
| eglynn | @telemetry.required_capablity({"storage": {"production_ready": True}}) | 11:27 |
| eglynn | ... decorator calls out to GET /v2/capability and bails on the test if the required capability is absent or False | 11:28 |
| eglynn | or maybe more direct and readable as: @telemetry.required_capablity("storage:production_ready") | 11:29 |
| eglynn | *required_capability | 11:29 |
| eglynn | DinaBelova: yep, absolutely, sounds like a plan :) | 11:30 |
| eglynn | DinaBelova: so you're gonna add the capability on the ceilo side, and vrovachev will add the check in the test? | 11:30 |
| vrovachev | i can do it using decorator skipif(variable=true) and define variable in setupclass | 11:30 |
| DinaBelova | eglynn, yes, exactly | 11:31 |
| DinaBelova | vrovachev's idea is ok for me too) | 11:31 |
| eglynn | DinaBelova, vrovachev: excellent! ... thank you both :) | 11:31 |
| vrovachev | eglynn: thank you :) | 11:32 |
| DinaBelova | eglynn, ildikov - https://review.openstack.org/#/c/96730/ | 11:33 |
| DinaBelova | cherry-pick for the stable/icehouse | 11:33 |
| eglynn | DinaBelova: that was quick :) | 11:33 |
| DinaBelova | do we have gerrit-bot here? | 11:33 |
| DinaBelova | I do not see its messages | 11:33 |
| eglynn | DinaBelova: well we did have ... maybe the bot just works West Coast hours ;) | 11:34 |
| DinaBelova | ok, so you don't see it too) | 11:35 |
| DinaBelova | ok) | 11:35 |
| eglynn | DinaBelova: https://github.com/openstack-infra/config/blob/master/modules/gerritbot/files/gerritbot_channel_config.yaml#L86 | 11:35 |
| eglynn | DinaBelova: ... if I'm reading that right, the bot will only log for master? | 11:36 |
| ildikov | DinaBelova: I reported a bug about this, just to track this issue: https://bugs.launchpad.net/ceilometer/+bug/1324885 | 11:37 |
| DinaBelova | heh - even if so, my previous change was to master | 11:37 |
| eglynn | ... whereas some of the other projects seem to be logging for various combinations of stable also | 11:37 |
| *** isviridov|away is now known as isviridov | 11:37 | |
| DinaBelova | eglynn, well, anyway bot is away 0 last change was to master, I see no message too | 11:37 |
| DinaBelova | ildikov, ok, thanks! | 11:37 |
| ildikov | DinaBelova: and for Icehouse, I'm not your person as for the approval :( :) | 11:38 |
| DinaBelova | well, it was - just take a look please (sometimes I do naughty git magic that ends up not really nice :D) | 11:38 |
| eglynn | ildikov: ... the stable-maint team warmly welcomes new volunteers :) | 11:38 |
| DinaBelova | ildikov I trust you to see if I was ok there :D:D:D | 11:38 |
| DinaBelova | ildikov - look, I just triggered your new access rights there ;) | 11:39 |
| ildikov | DinaBelova: sure, I checked it anyway | 11:39 |
| ildikov | eglynn: wow, good to know :) | 11:39 |
| DinaBelova | jd__, you here? we're trying to fix the gate ;) | 11:41 |
| jd__ | which gate DinaBelova ? | 11:41 |
| DinaBelova | jd__ - ours :D | 11:41 |
| DinaBelova | https://review.openstack.org/#/c/96730/ | 11:41 |
| ildikov | eglynn: I thought that it is the group of special people, who are close to God or something like, to be able to decide what can land on a stable branch and what not, so not available for a girl like me ;) | 11:41 |
| DinaBelova | we have failing py26 and py27 due to new keystoneclient | 11:41 |
| DinaBelova | for icehouse/stable too | 11:42 |
| DinaBelova | as you have second +2.. | 11:42 |
| DinaBelova | ;) | 11:42 |
| jd__ | DinaBelova: approved | 11:42 |
| DinaBelova | jd__, ok, thanks | 11:42 |
| eglynn | ildikov: IIRC apevec did some spring-cleaning on the membership recently, but it's still a fairly big group | 11:43 |
| eglynn | ildikov: ... https://review.openstack.org/#/admin/groups/120,members | 11:43 |
| *** eglynn is now known as eglynn-lunch | 11:51 | |
| ildikov | eglynn-lunch: I see, I never checked, I just saw those few, who approve the patches on the ceilo stable branches | 11:51 |
| underyx | hey, um, so, I hope this is not exclusively a development channel, cause I have this silly question... | 11:55 |
| underyx | http://docs.openstack.org/developer/ceilometer/events.html#events-from-notifications says "the Event format is meant for efficient processing and querying, there are other means available for archiving notifications (i.e. for audit purposes, etc)" | 11:55 |
| underyx | how bad would it be to still use events for auditing, or what exactly do this 'other means' refer to? | 11:56 |
| underyx | *does this | 11:57 |
| *** matsuhashi has quit IRC | 12:00 | |
| *** eglynn-lunch has quit IRC | 12:00 | |
| *** matsuhashi has joined #openstack-ceilometer | 12:00 | |
| *** zhikunliu has quit IRC | 12:01 | |
| *** matsuhashi has quit IRC | 12:02 | |
| *** matsuhashi has joined #openstack-ceilometer | 12:02 | |
| *** matsuhashi has quit IRC | 12:12 | |
| *** matsuhashi has joined #openstack-ceilometer | 12:13 | |
| *** Alexei_9871 has joined #openstack-ceilometer | 12:13 | |
| *** matsuhashi has quit IRC | 12:13 | |
| *** jdob has joined #openstack-ceilometer | 12:20 | |
| *** openstackgerrit has quit IRC | 12:22 | |
| *** thomasem has joined #openstack-ceilometer | 12:22 | |
| *** thomasem has quit IRC | 12:22 | |
| *** openstackgerrit has joined #openstack-ceilometer | 12:22 | |
| DinaBelova | eglynn, ildikov - hehe, it works :) https://review.openstack.org/#/c/96728/ | 12:22 |
| DinaBelova | just gate and that's it) | 12:22 |
| *** thomasem has joined #openstack-ceilometer | 12:23 | |
| *** eglynn-lunch has joined #openstack-ceilometer | 12:28 | |
| *** nosnos has quit IRC | 12:40 | |
| openstackgerrit | Longgeek proposed a change to openstack/ceilometer: Testing https://review.openstack.org/96748 | 12:41 |
| ildikov | DinaBelova: :) | 12:42 |
| *** cmart has joined #openstack-ceilometer | 12:47 | |
| *** prad has joined #openstack-ceilometer | 12:48 | |
| *** fnaval has joined #openstack-ceilometer | 12:50 | |
| openstackgerrit | Dina Belova proposed a change to openstack/ceilometer: Add new 'storage': {'production_ready': True} capability https://review.openstack.org/96752 | 12:53 |
| DinaBelova | eglynn-lunch ^^ - did you mean smth like this change? | 12:53 |
| DinaBelova | eglynn-lunch, ildikov, btw - what's the THSTACK CI job running on the ceilometer? I just realized I see it first time reallly :-\ | 12:56 |
| openstackgerrit | Longgeek proposed a change to openstack/ceilometer: Add new 'storage': {'production_ready': True} capability https://review.openstack.org/96752 | 12:57 |
| cdent | oh dear | 12:59 |
| DinaBelova | cdent == Longgeek ? | 13:00 |
| cdent | no | 13:00 |
| DinaBelova | it's cool | 13:00 |
| DinaBelova | :D | 13:00 |
| openstackgerrit | Cristian A Sanchez proposed a change to openstack/python-ceilometerclient: Make HTTP client aware of no_proxy variable https://review.openstack.org/95565 | 13:00 |
| DinaBelova | I'm just asking to get the idea what's the THSTACK CI and who's Longgeek | 13:00 |
| DinaBelova | :D | 13:00 |
| cdent | I'm just oh dearing at what appears to be some broken automation that looks like it could stumble over lots of things.... | 13:00 |
| DinaBelova | hehe :( | 13:01 |
| ildikov | DinaBelova: where did you see THSTACK? | 13:02 |
| DinaBelova | in the https://review.openstack.org/#/c/96752/2 | 13:02 |
| ildikov | DinaBelova: sorry, I'm distracted a bit with some internal stuff :S | 13:02 |
| DinaBelova | moreover - there is some strange stuff with Longgeek - this bot/man came to my change and reverted my gate fix | 13:03 |
| DinaBelova | wtf??? | 13:03 |
| ildikov | DinaBelova: a-ha, ok, so it added some review comments | 13:03 |
| DinaBelova | Longgeek stuff is not so bad - he/it just made this commit not related to the gate fix (my changes were chained) | 13:04 |
| DinaBelova | but as for the THSTACK comments... | 13:04 |
| DinaBelova | :-\ | 13:04 |
| ildikov | DinaBelova: I checked it on zuul, that where is this job and I couldn't see it | 13:04 |
| DinaBelova | http://zuul.thstack.com ? | 13:05 |
| DinaBelova | some new 3rd party zuul? | 13:05 |
| ildikov | DinaBelova: BTW as for the Capabilities API, I think you need to add this storage option to here too: https://github.com/openstack/ceilometer/blob/master/ceilometer/storage/base.py#L111 | 13:05 |
| DinaBelova | ildikov, ah, yes, missed it | 13:05 |
| DinaBelova | will do | 13:05 |
| ildikov | DinaBelova: as for the new zuul, maybe it is better to ask it on infra, just to be sure :) | 13:07 |
| DinaBelova | okay, I just wanted to be sure you see this also first time) | 13:07 |
| ildikov | DinaBelova: yes, I do :) | 13:08 |
| *** psharma has quit IRC | 13:08 | |
| DinaBelova | due to THSTACK: [17:10:27] <anteaya> it is a new third party ci account | 13:11 |
| ildikov | DinaBelova: ok, cool, tnx for the info | 13:12 |
| *** julim has joined #openstack-ceilometer | 13:13 | |
| *** safchain has quit IRC | 13:27 | |
| openstackgerrit | Artur Svechnikov proposed a change to openstack/python-ceilometerclient: Add methods to resource classes https://review.openstack.org/91554 | 13:34 |
| *** admin0 has quit IRC | 13:35 | |
| *** safchain has joined #openstack-ceilometer | 13:39 | |
| DinaBelova | eglynn-lunch, ildikov - I'd like to add new classmethod get_storage_capabilities in the storage impls and new dict storing this info - to separate these two groups | 13:42 |
| DinaBelova | in other case I'll need to write some separation code in the get_capabilities method, that looks ugly | 13:42 |
| DinaBelova | are you ok with my variant? | 13:42 |
| *** Ruetobas has quit IRC | 13:42 | |
| eglynn-lunch | DinaBelova: yep, that sounds reasonable | 13:42 |
| DinaBelova | ok! | 13:42 |
| *** fnaval has quit IRC | 13:42 | |
| ildikov | DinaBelova: the separation is approved ;) | 13:43 |
| ildikov | DinaBelova: BTW is capabilities a good name for this | 13:44 |
| ildikov | DinaBelova: ? | 13:44 |
| DinaBelova | well, why not :-\ It's the storage driver's capability to work) | 13:44 |
| ildikov | eglynn-lunch: ^^, I know you like the naming meetings ;) | 13:44 |
| eglynn-lunch | DinaBelova: so CapabilitiesController.get() becomes something like ... | 13:44 |
| eglynn-lunch | return Capabilities(api=_flatten_capabilities(conn.get_api_capabilities()), | 13:45 |
| eglynn-lunch | storage=_flatten_capabilities(conn.get_storage_capabilities())) | 13:45 |
| DinaBelova | yep | 13:45 |
| eglynn-lunch | DinaBelova: cool, sounds great! :) | 13:45 |
| ildikov | DinaBelova: well, it should be a deafault behavior and not capability, :) but well, I just wanted to ask it before you implement it | 13:45 |
| *** eglynn-lunch is now known as eglynn | 13:45 | |
| eglynn | ildikov: yeah we're slightly "abusing" the capabilities terminology here | 13:46 |
| ildikov | DinaBelova: it just sounds like a list of supported features and not that it performs well or not | 13:47 |
| *** _nadya_ has quit IRC | 13:47 | |
| DinaBelova | ildikov, eglynn - so what do you suggest here? | 13:47 |
| DinaBelova | Should I continue what were we speaking before? | 13:47 |
| DinaBelova | capabilities stuff? | 13:48 |
| *** _nadya_ has joined #openstack-ceilometer | 13:48 | |
| eglynn | DinaBelova: yes, I think so (as opposed to inventing a totally new mechanism to allow this info to become discoverable) | 13:48 |
| DinaBelova | eglynn, ok | 13:48 |
| *** _nadya_ has quit IRC | 13:49 | |
| *** Ruetobas has joined #openstack-ceilometer | 13:50 | |
| DinaBelova | btw, when I'm writing strings like "OK! Thanks))" - these two last '))' symbols are Russian variant of smilie :-\ And well - any combination of ")"*n, n>=1 My folks just said me that it's used only in Russia))) | 13:50 |
| DinaBelova | so my '))' == ':-)' :D | 13:51 |
| ildikov | DinaBelova: well, I may would call it performance or something, but I let you decide what you prefer :) | 13:51 |
| DinaBelova | okay, cool)) | 13:52 |
| DinaBelova | argh - '))' ^^ | 13:52 |
| DinaBelova | >_< | 13:52 |
| ildikov | DinaBelova: hmm, I was wondering once for a minute, when I had that time that what are these ')', '))' and so on parts at the end of your lines, but I hoped that these are for smilies ;) | 13:53 |
| DinaBelova | :D they are! | 13:53 |
| ildikov | DinaBelova: we can call them lazy smilies ;) | 13:54 |
| DinaBelova | yes, definitely! :) | 13:54 |
| *** Ruetobas has quit IRC | 13:54 | |
| ildikov | DinaBelova: cool :) | 13:54 |
| *** jergerber has joined #openstack-ceilometer | 13:56 | |
| *** gordc has joined #openstack-ceilometer | 13:57 | |
| *** jdob has quit IRC | 14:00 | |
| *** eglynn is now known as eglynn-on-call | 14:00 | |
| *** Ruetobas has joined #openstack-ceilometer | 14:00 | |
| *** amalagon_ has joined #openstack-ceilometer | 14:03 | |
| *** inc0_ has quit IRC | 14:06 | |
| *** amalagon has quit IRC | 14:07 | |
| *** fnaval has joined #openstack-ceilometer | 14:08 | |
| openstackgerrit | Igor Degtiarov proposed a change to openstack/ceilometer-specs: Hbase-events-feature specs https://review.openstack.org/96417 | 14:11 |
| *** rwsu has joined #openstack-ceilometer | 14:12 | |
| openstackgerrit | Dina Belova proposed a change to openstack/ceilometer: Add new 'storage': {'production_ready': True} capability https://review.openstack.org/96752 | 14:13 |
| jd__ | folks | 14:14 |
| DinaBelova | ildikov, eglynn-on-call - new variant :D | 14:14 |
| DinaBelova | ^^ | 14:14 |
| jd__ | https://wiki.openstack.org/wiki/Sprints/ParisJuno2014 if you didn't notice on openstack-dev yet | 14:14 |
| jd__ | I'd be happy to have you there :) | 14:14 |
| eglynn-on-call | jd__: cool :) | 14:14 |
| eglynn-on-call | jd__: ... sounds like this has grown into a mini-summit ;) | 14:15 |
| DinaBelova | jd__ - hehe! nice ;) | 14:15 |
| eglynn-on-call | DinaBelova: will look after call | 14:15 |
| eglynn-on-call | DinaBelova: thanks! | 14:15 |
| DinaBelova | ok, thanks! | 14:15 |
| jd__ | eglynn-on-call: yeah but I hope we'll write more code than we talk ;) | 14:15 |
| eglynn-on-call | jd__: impose a strict no-talking rule! | 14:16 |
| eglynn-on-call | jd__: ... or require that all talking is through the medium of perfectly-accented French ;) | 14:16 |
| idegtiarov | eglynn: Hello! After discussion of a draft version of my patch for event feature in Hbase, I have changed the structure for row_key from rts_mess_id --> ts_mess_id. And because of it I have to make some changes in spec for hbase-event-feature. Could you please look at it when you will have some time. | 14:16 |
| jd__ | eglynn-on-call: haha | 14:17 |
| ildikov | jd__: cool, looks great :) | 14:17 |
| *** ddieterly has joined #openstack-ceilometer | 14:18 | |
| DinaBelova | well, small offtopic here too: https://plus.google.com/u/1/photos/115127308412852581322/albums/6014935177627183745 | 14:18 |
| DinaBelova | my ATL photos | 14:18 |
| DinaBelova | :D | 14:19 |
| DinaBelova | not really about the summit, but still :) | 14:19 |
| *** ddieterl_ has joined #openstack-ceilometer | 14:20 | |
| ildikov | DinaBelova: I would freeze to death again by even watching Summit photos... ;) | 14:21 |
| DinaBelova | hehe - well, there are no Conference Centre photos there) | 14:21 |
| DinaBelova | only other things) | 14:21 |
| DinaBelova | Altanta was really nice place speaking about sightseeing and smth new) | 14:22 |
| *** ddieterly has quit IRC | 14:22 | |
| ildikov | DinaBelova: nice photos! | 14:22 |
| *** shakamunyi has joined #openstack-ceilometer | 14:22 | |
| DinaBelova | And orchids-maniac, so their botanical garden made strong impression on me | 14:22 |
| DinaBelova | I was shocked really) | 14:23 |
| ildikov | we didn't go there finally, but those pics looks really good, I love orchids | 14:23 |
| DinaBelova | :) | 14:24 |
| cdent | the pics from the zoo are fantastic | 14:25 |
| DinaBelova | we found cool dependency - in HK were pandas, in ATL were pandas... | 14:26 |
| DinaBelova | hm :-\ | 14:26 |
| DinaBelova | that might mean smth) | 14:26 |
| cdent | :) | 14:26 |
| *** raymondr_ has joined #openstack-ceilometer | 14:28 | |
| openstackgerrit | A change was merged to openstack/ceilometer: Fix testing gate due to new keystoneclient release https://review.openstack.org/96728 | 14:31 |
| *** jmckind has joined #openstack-ceilometer | 14:32 | |
| DinaBelova | yeah!!! | 14:33 |
| ildikov | DinaBelova: \o/ :) | 14:34 |
| *** aviau has quit IRC | 14:35 | |
| *** openstackgerrit has quit IRC | 14:36 | |
| *** openstackgerrit has joined #openstack-ceilometer | 14:36 | |
| *** jmckind has quit IRC | 14:36 | |
| gordc | very nice pictures DinaBelova! you guys visited so many places. | 14:37 |
| DinaBelova | gordc - well, yes)) we were really busy, but that's funny) | 14:37 |
| jd__ | indeed | 14:39 |
| gordc | i visited the aquarium as well (since it was next to hotel)... pretty neat... a lot better than the one we have in Toronto. | 14:40 |
| DinaBelova | gordc - it's the biggest in the World! | 14:40 |
| DinaBelova | and I was really impressed) | 14:40 |
| *** dhellmann_ is now known as dhellmann | 14:40 | |
| gordc | DinaBelova: oh really? cool. i didn't do any research on it. i was bored on Saturday and saw it next door so i visited. :) | 14:41 |
| *** promulo has quit IRC | 14:43 | |
| DinaBelova | gordc, :D well, I looked for this because I realized that whale sharks (~3 of them) are swimming around me, that was, well, quit impressive :D | 14:44 |
| gordc | yeah, that was cool. we have no whale sharks where i am. | 14:44 |
| gordc | can i use kickstarter to fund my trip to OpenStack Paris party? that's what kickstarter is for right? :) | 14:45 |
| jd__ | so many fishes but so no sushis, unbelievable | 14:45 |
| jd__ | gordc: I think kickstarter had worse thing :-) | 14:46 |
| jd__ | gordc: no way your current employer would? | 14:46 |
| openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: Clean tox.ini https://review.openstack.org/96781 | 14:47 |
| gordc | jd__: you should ask them for sushi. i think it's cheaper there :) | 14:48 |
| gordc | jd__: i'll need to ask. i just need to come up with a good excuse on what i'll be 'working' on. | 14:49 |
| jd__ | gordc: sounds achievable :) | 14:49 |
| *** aviau has joined #openstack-ceilometer | 14:49 | |
| gordc | jd__: 'i need money to go to Paris so i can build the greatest OpenStack product ever' i don't think they'll ask any questions if i say that. | 14:50 |
| jd__ | I wouldn't. | 14:51 |
| cmart | gordc: LOL!! | 14:51 |
| gordc | cool. problem solved. see you all in Paris. | 14:51 |
| ildikov | gordc: LOL :) cool :) | 14:52 |
| gordc | ildikov: :) | 14:53 |
| *** aviau has quit IRC | 14:54 | |
| jd__ | ildikov: you're not that far away I would guess? | 14:54 |
| ildikov | jd__: do you mean from project budget PoV? ;) | 14:55 |
| jd__ | ildikov: I meant distance, though it's proportional :) | 14:55 |
| ildikov | jd__: but phisically you're right, IIRC it's a two hours long flight | 14:55 |
| *** aviau has joined #openstack-ceilometer | 14:56 | |
| ildikov | jd__: project budget is never proportional, if I think of my manager's face, when I mention that I may would like to travel to no matter where :S :) | 14:57 |
| jd__ | haha | 14:57 |
| openstackgerrit | Cristian A Sanchez proposed a change to openstack/python-ceilometerclient: Make HTTP client aware of no_proxy variable https://review.openstack.org/95565 | 14:58 |
| ildikov | jd__: but as it is soo close compared to e.g: the US, I plan to go anyway, they should let me go for a few days fun coding vacation, so as usual, I have a plan B | 14:58 |
| openstackgerrit | Christian Martinez proposed a change to openstack/ceilometer: Adding alarm list filtering by state and meter https://review.openstack.org/96582 | 15:00 |
| prad | looks like someone really wants to implement a wardrobe for ceilometer project .. seeing bugs on dresses. Did we upset someone at the summit with the way we dressed ;) | 15:00 |
| jd__ | yeah it's getting boring | 15:00 |
| prad | yea | 15:00 |
| *** zqfan has joined #openstack-ceilometer | 15:01 | |
| DinaBelova | ildikov, about your first comment here: https://review.openstack.org/#/c/96752/3/ceilometer/storage/impl_hbase.py,cm | 15:08 |
| DinaBelova | sorry, can't get your point | 15:08 |
| DinaBelova | can you explain this? | 15:08 |
| ildikov | DinaBelova: I just meant that if we call it storage capabilities everywhere, then we can call it maybe available_storage_capability or something like | 15:12 |
| ildikov | DinaBelova: just to make the code more readable | 15:12 |
| DinaBelova | oh, ok - but due to your second comment - probably we don't need this at all? | 15:13 |
| ildikov | DinaBelova: yes, but it depends on what is decided about the default value in base | 15:13 |
| DinaBelova | well, I gues true is ok here | 15:13 |
| DinaBelova | unless we'll have some other issue :-\ | 15:14 |
| ildikov | DinaBelova: I just thought that maybe we could change this value based on the regular perf tests in the future | 15:14 |
| DinaBelova | are you ok with it? | 15:14 |
| ildikov | DinaBelova: also if we add a new driver maybe, than we can say that first we would test the performance with it | 15:14 |
| DinaBelova | well, ok - in this case I'll make it False by default | 15:15 |
| amalagon_ | hi jd__ do you know if the draft of the api spec (http://docs-draft.openstack.org/34/94834/9/gate/gate-telemetry-specs-docs/4654c39/doc/build/html/specs/gnocchi.html) will go on docs.openstack.org soon? | 15:15 |
| jd__ | amalagon_: unlikely | 15:15 |
| DinaBelova | and will keep this update_capabilities code | 15:15 |
| DinaBelova | with some more readable name :) | 15:15 |
| amalagon_ | jd__: ok, thanks | 15:16 |
| ildikov | DinaBelova: I would suggest go for False with the first version | 15:16 |
| DinaBelova | ildikov, ok | 15:16 |
| ildikov | DinaBelova: later if we have good experience with performance overall, then we can modify it | 15:16 |
| ildikov | DinaBelova: sorry, I'm on the pessimistic side :) | 15:17 |
| DinaBelova | ok)) | 15:17 |
| ildikov | DinaBelova: however, I call it realism ;) | 15:17 |
| DinaBelova | Well, I quite agree here too)) | 15:17 |
| *** safchain has quit IRC | 15:20 | |
| openstackgerrit | Dina Belova proposed a change to openstack/ceilometer: Add new 'storage': {'production_ready': True} capability https://review.openstack.org/96752 | 15:21 |
| ildikov | DinaBelova: cool :) | 15:21 |
| DinaBelova | ildikov ^^ new patch set) | 15:22 |
| ildikov | jd__: is it a good enough sign for the tox clean up, if the gate jobs (pep8, py26, py27, docs) look green? | 15:22 |
| ildikov | jd__: or is there anything else to check? | 15:22 |
| ildikov | DinaBelova: yeap, I saw it | 15:22 |
| DinaBelova | ildikov cool, thabks! | 15:23 |
| jd__ | ildikov: it's enough AFAIK | 15:27 |
| ildikov | jd__: cool, thanks | 15:31 |
| *** nealph has quit IRC | 15:35 | |
| *** zqfan has quit IRC | 15:41 | |
| *** lsmola has quit IRC | 15:41 | |
| *** idegtiarov has quit IRC | 15:41 | |
| *** promulo has joined #openstack-ceilometer | 15:41 | |
| jd__ | hey Gnocchi got his first gate broken by external dep | 15:44 |
| * jd__ makes a wish | 15:44 | |
| openstackgerrit | Martin Geisler proposed a change to openstack/ceilometer: Remove (c) and remove unnecessary encoding lines https://review.openstack.org/96117 | 15:44 |
| *** Guest37432 has quit IRC | 15:47 | |
| openstackgerrit | Martin Geisler proposed a change to openstack/python-ceilometerclient: Remove © and remove unnecessary encoding lines https://review.openstack.org/95880 | 15:50 |
| openstackgerrit | Martin Geisler proposed a change to openstack/ceilometer: Correct Emacs file variable specification https://review.openstack.org/95863 | 15:53 |
| *** changbl has quit IRC | 15:56 | |
| *** Ruetobas has quit IRC | 16:01 | |
| *** IvanBerezovskiy has left #openstack-ceilometer | 16:02 | |
| *** Ruetobas has joined #openstack-ceilometer | 16:03 | |
| *** Ruetobas has quit IRC | 16:09 | |
| *** Ruetobas has joined #openstack-ceilometer | 16:14 | |
| *** cdent_ has joined #openstack-ceilometer | 16:16 | |
| *** cdent has quit IRC | 16:16 | |
| *** cdent_ has quit IRC | 16:16 | |
| *** cdent has joined #openstack-ceilometer | 16:16 | |
| *** vrovachev has quit IRC | 16:17 | |
| *** shakamunyi has quit IRC | 16:20 | |
| *** julim has quit IRC | 16:21 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ceilometer: Updated from global requirements https://review.openstack.org/96819 | 16:24 |
| *** yjiang5_away is now known as yjiang5 | 16:27 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/python-ceilometerclient: Updated from global requirements https://review.openstack.org/91238 | 16:32 |
| eglynn-on-call | cdent: further thoughts on https://review.openstack.org/#/c/96514/7/ceilometerclient/v2/options.py | 16:41 |
| eglynn-on-call | cdent: ... methinks we're overthinking the regex | 16:41 |
| cdent | yeah, just reading that | 16:41 |
| eglynn-on-call | cdent: ... but we should be able to at least match the simplest, cases such as a single character value | 16:41 |
| cdent | I agree, but on the other hand even your simplified solution doesn't nail the criteria | 16:41 |
| eglynn-on-call | sure, as I said in the comment | 16:42 |
| cdent | which leaves us where? | 16:42 |
| cdent | it seems like someone or something needs to introduce some constraint(s) | 16:42 |
| cdent | it's relatively easy to get 1 and 2 and operators in values | 16:43 |
| cdent | but not 1 and 2 and operators in both keys and values | 16:43 |
| cdent | or rather: we can have operators in keys, but not (easily) adjacent to the proper operator | 16:45 |
| eglynn-on-call | ... maybe just split the string on the first instance of a greedily matched operator? | 16:45 |
| cdent | that's effectively saying the same thing as my "or rather..." statement | 16:46 |
| eglynn-on-call | "or rather..." statement? | 16:46 |
| cdent | "we can have operators in keys, but not (easily) adjacent to the proper operator" | 16:46 |
| cdent | i'll experiment with a more explicit split and see what I can get to percolate out | 16:47 |
| eglynn-on-call | my though expressed in the review was that an operator character adjacent to another operator character is always assumed to be part of a potential operator match | 16:49 |
| eglynn-on-call | (before a potential key match) | 16:49 |
| eglynn-on-call | as is "we can have operators in keys, but not (easily) adjacent to the proper operator" is an acceptable restriction | 16:50 |
| *** ildikov has quit IRC | 16:50 | |
| eglynn-on-call | cdent: ... right-o, I gotta run | 16:51 |
| cdent | which do we want: split on the first thing that looks like a operator | 16:51 |
| cdent | nm | 16:51 |
| cdent | I'll have something in a moment | 16:51 |
| cdent | we can look at it more next week | 16:51 |
| eglynn-on-call | yep I'm thinking greedily match operator first | 16:52 |
| *** Ju has quit IRC | 16:52 | |
| eglynn-on-call | ... I'll be back online later, if you just wanna leave a comment on gerrit | 16:52 |
| eglynn-on-call | ... gotta run now | 16:52 |
| cdent | cool, have a good eve | 16:52 |
| eglynn-on-call | cdent: yep you too! ... (back in the office Tuesday, Monday is a public holiday here) | 16:53 |
| cdent | right on | 16:53 |
| *** jdob has joined #openstack-ceilometer | 17:03 | |
| *** nacim has quit IRC | 17:06 | |
| openstackgerrit | Dina Belova proposed a change to openstack/ceilometer: Add new 'storage': {'production_ready': True} capability https://review.openstack.org/96752 | 17:09 |
| *** julim has joined #openstack-ceilometer | 17:26 | |
| *** shakayumi has joined #openstack-ceilometer | 17:29 | |
| *** shakayumi has quit IRC | 17:29 | |
| eglynn-on-call | cdent: a thought while driving ... | 17:30 |
| eglynn-on-call | cdent: ... would the simple non-greedy version '(.+?)(=|!=|>|<|<=|>=)(.+?)' do the needful? | 17:30 |
| cdent | I'm just committing something that may do the needful, using re.split() | 17:30 |
| cdent | assuming we have similar agreement on needful-nes | 17:30 |
| cdent | s-ness-ness-ness | 17:30 |
| cdent | Also: you're an addict, get off the internetz! | 17:31 |
| cmart | cdent: LOL!! | 17:32 |
| DinaBelova | eglynn-on-call - heh, still on the call? | 17:32 |
| DinaBelova | quite long one) | 17:32 |
| eglynn-on-call | cdent: cool enough, I think the non-greedy modifier would fulfill those 3 criteria ... but maybe split clearer, I dunno | 17:32 |
| eglynn-on-call | cdent: LOL :) ... I'm outta here | 17:33 |
| * cdent waves | 17:33 | |
| eglynn-on-call | DinaBelova: ... a world record marathon meeting ;) | 17:33 |
| DinaBelova | indeed! | 17:33 |
| * eglynn-on-call was just lazily about reverting his nick | 17:33 | |
| DinaBelova | :) | 17:33 |
| *** eglynn-on-call is now known as eglynn | 17:33 | |
| DinaBelova | okay, I just was worried you're going to die on this meeting - such long calls usually make people crazy) | 17:33 |
| eglynn | DinaBelova: BTW ... thanks for the storage capabilities patch! I'll take another pass later on | 17:34 |
| DinaBelova | eglynn, ok, thank you for looking on it) | 17:34 |
| eglynn | right-o have a great weekend folks! | 17:34 |
| cdent | is heading to real rock tommorrow, thankfully | 17:35 |
| eglynn | cdent: free climbing? | 17:35 |
| cdent | that's a bit of a weird term meaning different things to different people | 17:35 |
| cdent | I'll be doing trad climbing tomorrow (placing the gear myself) | 17:36 |
| eglynn | cdent: ... or whatever it is one calls the high-adrenalin version, without the safety net of being roped in | 17:36 |
| cdent | I'll have a rope. | 17:36 |
| eglynn | phew! :) | 17:36 |
| cdent | free climbing means with a rope, but not weighting the rope unless you have to | 17:36 |
| cdent | aid climbing is weighting all kinds of things | 17:36 |
| cdent | free solo is no rope where you ought to have one | 17:36 |
| eglynn | cdent: a-ha, got it | 17:36 |
| cdent | bouldering is no rope, but close to the ground | 17:36 |
| eglynn | cdent: ... well, except the weighting bit | 17:37 |
| eglynn | cdent: a-ha, bouldering is what I was prolly thinking of | 17:37 |
| cdent | that's the one that always has the videos of bros with their shirts off with dub step backgrounds and the like, and they're holding on to super tiny holds | 17:38 |
| * eglynn did a single indoor climb in Brno, and thinks he can bluff his way thru a conversation with a real climber ;) | 17:38 | |
| eglynn | cdent: "bros with their shirts off" ... LOL, love it :) | 17:38 |
| cdent | and they say things like "sick bro" and "psyche!" and if they are feeling european "allez" or "venga" | 17:39 |
| eglynn | cdent: ... well "venga" all the way so! | 17:40 |
| openstackgerrit | Chris Dent proposed a change to openstack/python-ceilometerclient: Refactor split_by_op and split_by_datatype https://review.openstack.org/96514 | 17:42 |
| * cdent is firmly a member of the regex now you've got two problems camp | 17:46 | |
| *** gordc-mobile has joined #openstack-ceilometer | 17:49 | |
| *** changbl has joined #openstack-ceilometer | 17:58 | |
| openstackgerrit | A change was merged to openstack/ceilometer: reconnect to mongodb on connection failure https://review.openstack.org/86609 | 18:01 |
| *** changbl has quit IRC | 18:04 | |
| openstackgerrit | Chris Dent proposed a change to openstack/python-ceilometerclient: Refactor split_by_op and split_by_datatype https://review.openstack.org/96514 | 18:10 |
| *** erecio has joined #openstack-ceilometer | 18:11 | |
| *** gordc-mobile has quit IRC | 18:11 | |
| *** erecio has quit IRC | 18:19 | |
| *** fnaval has quit IRC | 18:19 | |
| *** erecio has joined #openstack-ceilometer | 18:19 | |
| *** fnaval has joined #openstack-ceilometer | 18:19 | |
| openstackgerrit | Cristian A Sanchez proposed a change to openstack/python-ceilometerclient: Make HTTP client aware of no_proxy variable https://review.openstack.org/95565 | 18:23 |
| *** fnaval has quit IRC | 18:24 | |
| *** cmart has left #openstack-ceilometer | 18:39 | |
| *** _cjones_ has joined #openstack-ceilometer | 18:40 | |
| *** _cjones_ has quit IRC | 18:41 | |
| *** robbybb has quit IRC | 18:47 | |
| *** robbybb has joined #openstack-ceilometer | 18:47 | |
| *** ildikov has joined #openstack-ceilometer | 18:54 | |
| *** prad has quit IRC | 18:59 | |
| *** Alexei_9871 has quit IRC | 19:10 | |
| *** admin0 has joined #openstack-ceilometer | 19:15 | |
| *** _nadya_ has joined #openstack-ceilometer | 19:30 | |
| *** _nadya_ has quit IRC | 19:32 | |
| *** cdent has quit IRC | 19:34 | |
| *** _nadya_ has joined #openstack-ceilometer | 19:36 | |
| *** changbl has joined #openstack-ceilometer | 19:40 | |
| *** gordc has quit IRC | 19:45 | |
| *** fnaval has joined #openstack-ceilometer | 20:02 | |
| *** _nadya_ has quit IRC | 20:09 | |
| *** terriyu has joined #openstack-ceilometer | 20:11 | |
| *** terriyu has quit IRC | 20:11 | |
| *** thomasem has quit IRC | 20:15 | |
| *** erecio has quit IRC | 20:24 | |
| *** _nadya_ has joined #openstack-ceilometer | 20:35 | |
| *** admin0 has quit IRC | 20:38 | |
| *** admin0 has joined #openstack-ceilometer | 20:39 | |
| *** _nadya_ has quit IRC | 20:41 | |
| *** jdob has quit IRC | 20:43 | |
| *** admin0 has quit IRC | 20:43 | |
| *** admin0 has joined #openstack-ceilometer | 20:44 | |
| *** DrBacchus has joined #openstack-ceilometer | 20:47 | |
| *** rbowen has quit IRC | 20:49 | |
| ildikov | dhellmann: hi | 20:49 |
| *** admin0 has quit IRC | 20:51 | |
| ildikov | dhellmann: will you have a few minutes for me today or maybe on Monday for one or two sphinx/doc related questions? | 20:51 |
| *** fnaval has quit IRC | 20:55 | |
| *** fnaval has joined #openstack-ceilometer | 20:56 | |
| *** pnascimento has joined #openstack-ceilometer | 20:59 | |
| *** fnaval has quit IRC | 21:00 | |
| *** openstackgerrit has quit IRC | 21:01 | |
| *** raymondr_ has quit IRC | 21:05 | |
| *** DrBacchus has quit IRC | 21:07 | |
| *** rbowen has joined #openstack-ceilometer | 21:07 | |
| *** rbowen has quit IRC | 21:08 | |
| *** rbowen has joined #openstack-ceilometer | 21:08 | |
| *** pnascimento has quit IRC | 21:12 | |
| *** _cjones_ has joined #openstack-ceilometer | 21:15 | |
| *** promulo has quit IRC | 21:36 | |
| *** rbowen has quit IRC | 21:37 | |
| *** pnascimento has joined #openstack-ceilometer | 21:49 | |
| *** underyx is now known as Underyx | 21:52 | |
| *** fnaval has joined #openstack-ceilometer | 22:06 | |
| *** eglynn has quit IRC | 22:14 | |
| *** openstackgerrit has joined #openstack-ceilometer | 22:31 | |
| *** openstackgerrit has quit IRC | 22:35 | |
| *** openstackgerrit has joined #openstack-ceilometer | 22:35 | |
| *** ddieterl_ has quit IRC | 22:45 | |
| *** pnascimento has quit IRC | 22:49 | |
| *** admin0 has joined #openstack-ceilometer | 22:51 | |
| *** admin0 has quit IRC | 22:51 | |
| *** jergerber has quit IRC | 22:52 | |
| *** raymondr has joined #openstack-ceilometer | 22:52 | |
| *** shardy has quit IRC | 22:52 | |
| *** raymondr has quit IRC | 22:59 | |
| dhellmann | ildikov: let's talk on monday | 23:22 |
| *** ildikov has quit IRC | 23:31 | |
| *** ildikov has joined #openstack-ceilometer | 23:31 | |
| *** zhikunliu has joined #openstack-ceilometer | 23:37 | |
| *** terriyu has joined #openstack-ceilometer | 23:49 | |
| *** terriyu has quit IRC | 23:49 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!