| *** fnaval has joined #openstack-ceilometer | 00:21 | |
| *** changbl has joined #openstack-ceilometer | 00:25 | |
| *** _nadya_ has quit IRC | 00:26 | |
| *** nati_ueno has joined #openstack-ceilometer | 00:32 | |
| *** nsaje has quit IRC | 00:38 | |
| *** nsaje has joined #openstack-ceilometer | 00:41 | |
| *** Shaan7 has quit IRC | 00:44 | |
| *** fnaval has quit IRC | 00:48 | |
| *** fnaval has joined #openstack-ceilometer | 00:48 | |
| *** _nadya_ has joined #openstack-ceilometer | 00:52 | |
| *** fnaval has quit IRC | 00:53 | |
| openstackgerrit | liusheng proposed a change to openstack/ceilometer: Fix the return of statistic with getting no sample https://review.openstack.org/80467 | 01:07 |
|---|---|---|
| *** fnaval has joined #openstack-ceilometer | 01:11 | |
| *** _cjones_ has quit IRC | 01:12 | |
| *** _cjones_ has joined #openstack-ceilometer | 01:13 | |
| *** yjiang5 is now known as yjiang5_away | 01:13 | |
| *** _cjones_ has quit IRC | 01:17 | |
| *** Shaan7 has joined #openstack-ceilometer | 01:18 | |
| *** alexpilotti has quit IRC | 01:28 | |
| *** nosnos has joined #openstack-ceilometer | 01:33 | |
| *** nealph has quit IRC | 01:41 | |
| *** _nadya_ has quit IRC | 01:50 | |
| *** mengxd has joined #openstack-ceilometer | 01:57 | |
| *** yfujioka has joined #openstack-ceilometer | 02:06 | |
| *** _nadya_ has joined #openstack-ceilometer | 02:17 | |
| *** yfujioka has quit IRC | 02:29 | |
| *** yfujioka has joined #openstack-ceilometer | 02:29 | |
| *** renlt has joined #openstack-ceilometer | 02:29 | |
| *** yfujioka has quit IRC | 02:34 | |
| *** promulo has joined #openstack-ceilometer | 02:35 | |
| *** nati_ueno has quit IRC | 02:46 | |
| *** nijaba has quit IRC | 02:46 | |
| *** nijaba has joined #openstack-ceilometer | 02:59 | |
| *** nijaba has quit IRC | 02:59 | |
| *** nijaba has joined #openstack-ceilometer | 02:59 | |
| openstackgerrit | liusheng proposed a change to openstack/ceilometer: Fix the return of statistic with getting no sample https://review.openstack.org/80467 | 03:11 |
| *** shakamunyi has joined #openstack-ceilometer | 03:27 | |
| *** sdake_ has quit IRC | 03:38 | |
| *** nosnos has quit IRC | 03:45 | |
| *** Alexei_987 has quit IRC | 03:57 | |
| *** mengxd has quit IRC | 04:29 | |
| *** nosnos has joined #openstack-ceilometer | 04:53 | |
| *** matsuhashi has joined #openstack-ceilometer | 04:54 | |
| *** matsuhashi has quit IRC | 05:24 | |
| *** ildikov has quit IRC | 05:30 | |
| *** matsuhashi has joined #openstack-ceilometer | 05:40 | |
| openstackgerrit | Thomas Herve proposed a change to openstack/ceilometer: Add a alarm notification using trusts https://review.openstack.org/87110 | 05:41 |
| *** _nadya_ has quit IRC | 05:51 | |
| *** fnaval has quit IRC | 05:59 | |
| *** fnaval has joined #openstack-ceilometer | 06:00 | |
| *** sdake_ has joined #openstack-ceilometer | 06:02 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/88506 | 06:03 |
| *** fnaval has quit IRC | 06:04 | |
| *** nati_ueno has joined #openstack-ceilometer | 06:09 | |
| *** ildikov has joined #openstack-ceilometer | 06:10 | |
| *** eglynn has joined #openstack-ceilometer | 06:12 | |
| *** matsuhashi has quit IRC | 06:14 | |
| *** _nadya_ has joined #openstack-ceilometer | 06:17 | |
| *** matsuhashi has joined #openstack-ceilometer | 06:24 | |
| *** nati_ueno has quit IRC | 06:25 | |
| *** nati_ueno has joined #openstack-ceilometer | 06:33 | |
| *** shakamunyi has quit IRC | 06:36 | |
| *** _nadya_ has quit IRC | 06:41 | |
| *** _nadya_ has joined #openstack-ceilometer | 06:41 | |
| *** nati_ueno has quit IRC | 06:48 | |
| *** rwsu has quit IRC | 06:57 | |
| *** nati_ueno has joined #openstack-ceilometer | 06:58 | |
| *** nosnos has quit IRC | 07:02 | |
| *** matsuhashi has quit IRC | 07:15 | |
| *** _nadya_ has quit IRC | 07:15 | |
| *** matsuhashi has joined #openstack-ceilometer | 07:15 | |
| *** _nadya_ has joined #openstack-ceilometer | 07:16 | |
| openstackgerrit | Thomas Herve proposed a change to openstack/ceilometer: Add a alarm notification using trusts https://review.openstack.org/87110 | 07:17 |
| *** matsuhashi has quit IRC | 07:28 | |
| *** matsuhashi has joined #openstack-ceilometer | 07:29 | |
| *** urulama has joined #openstack-ceilometer | 07:29 | |
| *** idegtiarov has joined #openstack-ceilometer | 07:29 | |
| *** _nadya_ has quit IRC | 07:30 | |
| *** shakamunyi has joined #openstack-ceilometer | 07:33 | |
| *** _nadya_ has joined #openstack-ceilometer | 07:34 | |
| *** shakamunyi has quit IRC | 07:38 | |
| *** nati_ueno has quit IRC | 07:48 | |
| *** nacim has joined #openstack-ceilometer | 07:53 | |
| *** ildikov_ has joined #openstack-ceilometer | 08:02 | |
| *** ildikov has quit IRC | 08:05 | |
| *** matsuhashi has quit IRC | 08:06 | |
| *** matsuhashi has joined #openstack-ceilometer | 08:07 | |
| *** nosnos has joined #openstack-ceilometer | 08:11 | |
| therve | eglynn, Hi, do you want to talk about the trust thing here? | 08:13 |
| eglynn | sure | 08:13 |
| eglynn | therve: so the question was simply the s/v2.0/v3/ in the auth URL | 08:13 |
| therve | Right | 08:13 |
| therve | eglynn, So trusts are a v3 only extension, so I need v3 | 08:14 |
| eglynn | therve: ... just wondering whether it would better to fail in that case and log the version mismatch? | 08:14 |
| therve | I don't know if the rest of ceilometer works with v3, does it? | 08:14 |
| eglynn | therve: ... i.e. to respect the preference for v2 expressed in the config | 08:14 |
| eglynn | therve: ... good question, I don't think we've explicitly test that | 08:14 |
| therve | Yeah I understand your concern. | 08:14 |
| eglynn | therve: ... is the basic token issueance/verification the same under v3? | 08:15 |
| therve | Personally I was more thinking it was easier as a migration standpoint | 08:15 |
| therve | eglynn, *mostly* :) | 08:15 |
| therve | But the devil is in the details | 08:15 |
| eglynn | therve: ... i.e. is there any reason to suspect the v3 all-around usage would be a potential problem for ceilo | 08:15 |
| eglynn | therve: ... I guess we could just test it and see | 08:16 |
| therve | Yeah | 08:16 |
| eglynn | therve: ... how does heat work in that respect, does it now require v3 in the auth_url or do the substitution under the covers? | 08:16 |
| therve | eglynn, It used to do the later | 08:17 |
| eglynn | ... but then changed? | 08:17 |
| therve | No it still does, I wanted to verify | 08:17 |
| eglynn | a-ha k | 08:18 |
| eglynn | either way we prolly should have a common approach | 08:18 |
| eglynn | i.e. either both ceilo & heat respects the v2.0 in the configured URL | 08:18 |
| eglynn | or both reject it when trusts are used, simply log a warning | 08:19 |
| therve | Well in the case of Heat the default configuration only supports v3 nowadays, but we can raise an error? | 08:19 |
| therve | I guess not, we need to support upgrade nicely :/ | 08:19 |
| therve | This is different in my patch because it's an optional new notifier | 08:20 |
| *** guest1212321 has joined #openstack-ceilometer | 08:20 | |
| eglynn | a-ha so heat *only* supports v3 now, I thought you meant above that it does s/v2.0/v3/ under the covers | 08:20 |
| *** guest1212321 has quit IRC | 08:21 | |
| *** ildikov has joined #openstack-ceilometer | 08:21 | |
| therve | eglynn, I mean it always uses keystone v3, *and* it does the s/v2.0/v3/ | 08:21 |
| therve | Mostly so that old config files work | 08:21 |
| eglynn | therve: hmmm, just checking the auth_url for v2.0 when the alarm is created would work either as config could change before the alarm actually fires | 08:22 |
| eglynn | therve: ... I suppose it all depends on our interpretation of "v2.0" in the auth_url | 08:23 |
| *** ildikov_ has quit IRC | 08:23 | |
| eglynn | therve: ... does it mean ... | 08:23 |
| eglynn | (a) "restrict to v2 KS API only", or | 08:24 |
| eglynn | (b) "require at least v2 API or greater", or | 08:24 |
| eglynn | (c) "I'm just putting in the version specifer coz it seems like the traditional thing to do" ;) | 08:24 |
| eglynn | therve: .... if (b) or (c) then obviously your code is fine | 08:25 |
| therve | I'd tend to say c | 08:26 |
| eglynn | therve: ... you know what, you could well be correct there | 08:26 |
| therve | eglynn, I think if you don't want v3 to be used you would limit it at keystone level | 08:26 |
| *** shardy_afk is now known as shardy | 08:26 | |
| eglynn | therve: ... good point | 08:26 |
| eglynn | therve: ... k, can you capture that thought on gerrit for posterity and I'll withdraw my -1 | 08:27 |
| therve | eglynn, I will, thanks! | 08:27 |
| *** eglynn is now known as eglynn-afk | 08:28 | |
| *** lsmola has joined #openstack-ceilometer | 08:29 | |
| *** eglynn-afk is now known as eglynn | 08:32 | |
| eglynn | ityaptin: good morning! | 08:33 |
| eglynn | ityaptin: ... WRT that discussion yesterday about pushing your loadgen scripts up to github | 08:33 |
| eglynn | ityaptin: ... I'd like to try to preproducing some of the mongodb issues you saw | 08:33 |
| eglynn | ityaptin: ... esp. the mongo sort overflow ("Overflow sort stage buffered data usage ...") | 08:34 |
| eglynn | ityaptin: ... so can you shoot me off the link when/if that stuff makes it up to github? | 08:35 |
| eglynn | ityaptin: ... ta much! | 08:35 |
| *** matsuhashi has quit IRC | 08:36 | |
| eglynn | ityaptin, DinaBelova, _nadya_: also one other question for you folks related to ^^^ | 08:37 |
| eglynn | ... are you planning to present these performance data at summit? | 08:37 |
| *** idegtiarov has quit IRC | 08:38 | |
| *** idegtiarov has joined #openstack-ceilometer | 08:39 | |
| _nadya_ | eglynn: hi! | 08:41 |
| eglynn | _nadya_: good morning! :) | 08:42 |
| eglynn | _nadya_: ... so I was just wondering about (a) getting access to the loadgen script to reproduce the apparent mongo bugs, and (b) summit exposure of these data | 08:43 |
| _nadya_ | eglynn: I was planning DinaBelova will present results on summit. Maybe on model-refactoring-session | 08:43 |
| DinaBelova | eglynnб щ. | 08:43 |
| DinaBelova | heh, sorry | 08:43 |
| DinaBelova | eglynn, o/ | 08:43 |
| _nadya_ | eglynn: yep, we will publish scripts, sure | 08:43 |
| eglynn | _nadya_: cool, Alexei is that the main presenter of that session, correct? | 08:43 |
| eglynn | _nadya_: thanks! | 08:44 |
| eglynn | DinaBelova: ... my Cyrillic is a bit rusty ;) | 08:44 |
| _nadya_ | eglynn: yes, we are working with him about performance too | 08:44 |
| DinaBelova | eglynn, I'll be on summit with the results of the performance testing and (possibly) found bottle necks | 08:44 |
| DinaBelova | and Alexey will be there too)) | 08:44 |
| eglynn | excellent! :) | 08:45 |
| DinaBelova | and he will know everything too) | 08:45 |
| DinaBelova | so we'll try to give you full picture | 08:45 |
| DinaBelova | about what's happening there | 08:45 |
| eglynn | I'd like to have a chance to play with the script before summit | 08:45 |
| eglynn | (not that I doubt your results, but I'm concerned about the mongo issues) | 08:46 |
| DinaBelova | eglynn, as soon as possible ityaptin will publish it | 08:46 |
| *** nosnos has quit IRC | 08:46 | |
| DinaBelova | I guess before enf og this day :-| | 08:46 |
| DinaBelova | heh, only typos)) | 08:46 |
| eglynn | DinaBelova: excellent, thanks! :) | 08:47 |
| DinaBelova | I had 2 yoga hours just now, so my fingers don't love the idea of typing :D | 08:47 |
| eglynn | (... I'd like to get at least the root cause analysis done on the mongo buffer overflow issue before summit, as that looks like a potential stinker) | 08:47 |
| DinaBelova | eglynn, yes, it looks so | 08:47 |
| eglynn | DinaBelova: LOL :) | 08:48 |
| eglynn | ... I thought yoga was supposed to promote ultimate flexibility | 08:48 |
| DinaBelova | and full relax to the end of practice) | 08:48 |
| DinaBelova | if it's long enough)) | 08:48 |
| DinaBelova | so I'm too relaxed now :D | 08:48 |
| eglynn | ... I could use some of that :) | 08:49 |
| DinaBelova | ;) | 08:49 |
| *** eglynn is now known as eglynn-afk | 08:51 | |
| _nadya_ | eglynn-afk: After the tests we have questions about all backends :) I'm wondering about HBase too. Because simple "scan" operation takes 120 seconds. Cluster definitely needs to be tuned | 08:52 |
| ildikov | _nadya_: hi! | 08:54 |
| ildikov | _nadya_: I assume you examine the DB models too WRT the test results, right? | 08:54 |
| _nadya_ | ildikov: hi! | 08:54 |
| DinaBelova | _nadya_, I guess that hbase stuff may have reasons in the happyBase lib - as everything there seems to be first loaded to the memory, then doing smth else there :-| I feel quite suspicious about this fact | 08:55 |
| *** Alexei_987 has joined #openstack-ceilometer | 08:56 | |
| _nadya_ | DinaBelova: hmm, don't think so... there is a 'batch' field in scan so I hope it is used and not all the data goes to memory | 08:56 |
| DinaBelova | _nadya_, I'm operating the words ityaptin said me :? | 08:56 |
| _nadya_ | ildikov: yep, but mostly during working on hbase | 08:56 |
| DinaBelova | I may be wrong here for sure | 08:57 |
| _nadya_ | ildikov: if you have specific questions please ask | 08:57 |
| ildikov | _nadya_: a-ha, I see, Alexei's session will focus to HBase too? | 08:58 |
| _nadya_ | ildikov: no, don't think so | 08:58 |
| ildikov | _nadya_: currently I do not have any specific question, I was just wondering that the results can be interesting from the data model PoV too | 08:59 |
| ildikov | _nadya_: ok, cool | 08:59 |
| DinaBelova | ildikov, o/ hello) | 09:00 |
| ildikov | DinaBelova: hi :) | 09:00 |
| _nadya_ | ildikov: ah, yes. that's why I assume that performance results may be shown on model-session | 09:00 |
| ildikov | DinaBelova: I do not know your TZ, so I'm sure in the good morning/afternoon/evening question :) | 09:00 |
| DinaBelova | good afternoon I guess)) | 09:01 |
| DinaBelova | MSK TZ | 09:01 |
| ildikov | _nadya_: cool, sounds great | 09:01 |
| DinaBelova | UTC +4 | 09:01 |
| ildikov | DinaBelova: a-ha, good afternoon then :) | 09:02 |
| DinaBelova | ;) | 09:02 |
| ildikov | _nadya_, DinaBelova: BTW I will have a testing session hopefully on the summit with many topics | 09:03 |
| DinaBelova | ildikov, sounds great! | 09:03 |
| ildikov | _nadya_, DinaBelova: eglynn-afk has some tempest related stuff too, so if you have any additions, then w would appriciate any inputs | 09:04 |
| ildikov | _nadya_, DinaBelova: or if DinaBelova, you could join to that session, it would be also good, if you're involved in the tempest work too maybe | 09:05 |
| DinaBelova | ildikov, I'll be there)) | 09:05 |
| DinaBelova | just say me day and time) | 09:05 |
| _nadya_ | ildikov: yep, do you have an etherpad for this session? | 09:06 |
| ildikov | DinaBelova: ok, cool :) I will inform you once the schedule is ready for the design summit | 09:06 |
| ildikov | _nadya_: currently just in my head :), but I planned to prepare one soon, end of this week or at the beginning of the next | 09:07 |
| _nadya_ | ildikov: ping me when the doc is created. I will add something | 09:08 |
| ildikov | _nadya_, DinaBelova: it seems that it will be an intensive and busy 40 mins so I need to organize it well | 09:09 |
| ildikov | _nadya_: sure, I will do, thanks | 09:09 |
| *** renlt has quit IRC | 09:10 | |
| DinaBelova | ildikov, cool, waiting for your ping) | 09:11 |
| ildikov | DinaBelova: cool, thanks | 09:12 |
| *** vrovachev has quit IRC | 09:39 | |
| DinaBelova | ildikov, eglynn-afk, _nadya_ - btw, while playing with the ceilo+devstack I founf rejoin issue with compute-agent - I've proposed the smaaaal change to the devstack - asre you ok with it? https://review.openstack.org/#/c/89845/ | 09:41 |
| *** matsuhashi has joined #openstack-ceilometer | 09:47 | |
| *** nosnos_ has joined #openstack-ceilometer | 09:52 | |
| *** eglynn-afk is now known as eglynn | 10:06 | |
| *** piyushmasrani has joined #openstack-ceilometer | 10:06 | |
| eglynn | DinaBelova: good on the devstack rejoin | 10:06 |
| eglynn | DinaBelova: ... BTW Dean Troyer is usually good to nominate as a reviewer for devstack changes | 10:07 |
| *** matsuhashi has quit IRC | 10:11 | |
| *** matsuhas_ has joined #openstack-ceilometer | 10:14 | |
| *** ildikov has quit IRC | 10:20 | |
| *** ildikov has joined #openstack-ceilometer | 10:21 | |
| *** matsuhas_ has quit IRC | 10:30 | |
| *** matsuhashi has joined #openstack-ceilometer | 10:42 | |
| *** matsuhashi has quit IRC | 10:43 | |
| *** waterkinfe has joined #openstack-ceilometer | 10:44 | |
| *** eglynn is now known as eglynn-afk | 10:51 | |
| *** renlt has joined #openstack-ceilometer | 10:52 | |
| *** ildikov has quit IRC | 10:52 | |
| *** ildikov has joined #openstack-ceilometer | 10:52 | |
| *** _nadya_ has quit IRC | 11:00 | |
| *** nosnos_ has quit IRC | 11:03 | |
| DinaBelova | eglynn, oh, ok! Good to know) | 11:03 |
| *** alexpilotti has joined #openstack-ceilometer | 11:17 | |
| *** nacim has quit IRC | 11:19 | |
| *** ildikov_ has joined #openstack-ceilometer | 11:20 | |
| *** julim has joined #openstack-ceilometer | 11:21 | |
| *** julim has quit IRC | 11:22 | |
| *** ildikov has quit IRC | 11:22 | |
| *** _nadya_ has joined #openstack-ceilometer | 11:31 | |
| openstackgerrit | A change was merged to openstack/ceilometer: Add a alarm notification using trusts https://review.openstack.org/87110 | 11:33 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ceilometer: Updated from global requirements https://review.openstack.org/88714 | 11:33 |
| *** alexpilotti has quit IRC | 11:35 | |
| *** julim has joined #openstack-ceilometer | 11:36 | |
| *** alexpilotti has joined #openstack-ceilometer | 11:36 | |
| *** promulo has quit IRC | 11:45 | |
| *** gordc has joined #openstack-ceilometer | 11:49 | |
| *** alexpilotti has quit IRC | 11:56 | |
| *** alexpilotti has joined #openstack-ceilometer | 12:00 | |
| *** erecio has joined #openstack-ceilometer | 12:03 | |
| *** renlt has quit IRC | 12:04 | |
| *** prad has joined #openstack-ceilometer | 12:12 | |
| *** fnaval has joined #openstack-ceilometer | 12:18 | |
| *** promulo has joined #openstack-ceilometer | 12:26 | |
| *** jdob has joined #openstack-ceilometer | 12:30 | |
| *** eglynn-afk is now known as eglynn | 12:35 | |
| *** ityaptin has quit IRC | 12:44 | |
| *** alexpilotti has quit IRC | 12:45 | |
| *** _nadya_ has quit IRC | 12:51 | |
| *** _nadya_ has joined #openstack-ceilometer | 12:51 | |
| *** erecio_1 has joined #openstack-ceilometer | 12:53 | |
| *** dperaza has quit IRC | 12:54 | |
| *** dperaza1 has joined #openstack-ceilometer | 12:54 | |
| *** erecio has quit IRC | 12:56 | |
| *** eglynn has quit IRC | 12:59 | |
| *** nacim has joined #openstack-ceilometer | 13:00 | |
| *** alexpilotti has joined #openstack-ceilometer | 13:03 | |
| *** _nadya_ has quit IRC | 13:04 | |
| *** joesavak has joined #openstack-ceilometer | 13:10 | |
| *** prad has quit IRC | 13:11 | |
| *** alexpilotti has quit IRC | 13:16 | |
| *** alexpilotti has joined #openstack-ceilometer | 13:20 | |
| *** fnaval has quit IRC | 13:23 | |
| *** waterkinfe has quit IRC | 13:28 | |
| *** thomasem has joined #openstack-ceilometer | 13:29 | |
| *** _nadya_ has joined #openstack-ceilometer | 13:34 | |
| *** parkerr_ has joined #openstack-ceilometer | 13:39 | |
| *** nealph has joined #openstack-ceilometer | 13:40 | |
| *** prad has joined #openstack-ceilometer | 13:47 | |
| openstackgerrit | Dina Belova proposed a change to openstack/ceilometer: Remove "# noqa" leftovers for gettextutils._ https://review.openstack.org/90375 | 13:48 |
| *** _nadya_ has quit IRC | 13:51 | |
| *** nealph has quit IRC | 13:53 | |
| *** eglynn has joined #openstack-ceilometer | 13:54 | |
| *** _nadya_ has joined #openstack-ceilometer | 14:04 | |
| *** nealph has joined #openstack-ceilometer | 14:25 | |
| *** erecio_1 has quit IRC | 14:30 | |
| *** ildikov_ has quit IRC | 14:30 | |
| *** ityaptin has joined #openstack-ceilometer | 14:38 | |
| *** gordc1 has joined #openstack-ceilometer | 14:40 | |
| *** gordc has quit IRC | 14:42 | |
| *** fnaval has joined #openstack-ceilometer | 14:46 | |
| openstackgerrit | Dina Belova proposed a change to openstack/ceilometer: Fix SAWarning about 'IN' predicate with an empty sequence https://review.openstack.org/90392 | 14:53 |
| DinaBelova | gordc1, eglynn, ildikov_, _nadya_ ^^ - not sure this should be fixed this way - possibly that needs to be caught for all backends + not giving the opportunity to create these filters at all | 15:02 |
| *** idegtiarov has quit IRC | 15:03 | |
| *** gordc1 is now known as gordc | 15:04 | |
| DinaBelova | I guess it'll be better not to allow such filters at all | 15:04 |
| DinaBelova | and fix the test, that checks that {"in": {"resource_id": []}} filter returns [] - as this filter makes no sense at all | 15:05 |
| eglynn | DinaBelova: thanks for the heads-up, will look shortly | 15:05 |
| sileht | DinaBelova, you means ensure at the API level we have not empty set ? | 15:06 |
| DinaBelova | sileht, yes - as this filter makes no sense | 15:06 |
| DinaBelova | it'll be always [] | 15:06 |
| DinaBelova | the result I mean | 15:06 |
| DinaBelova | that's matter of validation imho | 15:06 |
| DinaBelova | sileht, was trying to find your IRC nick for a lo-o-ong time))) | 15:07 |
| sileht | :) | 15:07 |
| sileht | DinaBelova, I agree so, I have think it was just a sqla things, but if is for all backends, API is better place to filter this. | 15:08 |
| DinaBelova | eglynn, do you agree here with me and sileht? | 15:08 |
| * eglynn is a slooow reader ;) | 15:09 | |
| eglynn | ... still digesting the patch | 15:09 |
| *** erecio_1 has joined #openstack-ceilometer | 15:11 | |
| DinaBelova | eglynn, well I got your idea from the review) | 15:11 |
| DinaBelova | ok, will provide next patch set | 15:11 |
| sileht | DinaBelova, I think minItems is missing here: http://git.openstack.org/cgit/openstack/ceilometer/tree/ceilometer/api/controllers/v2.py#n1168 | 15:13 |
| DinaBelova | sileht, ok, will check it | 15:14 |
| sileht | DinaBelova, this is just a though, I didn't know well jsonschema (and I think jsonschema is not very readable) | 15:16 |
| *** ashes274 has joined #openstack-ceilometer | 15:19 | |
| *** ashes274 has quit IRC | 15:20 | |
| *** ashestakov__ has quit IRC | 15:20 | |
| *** ashestakov_ has quit IRC | 15:20 | |
| *** ashestakov has quit IRC | 15:20 | |
| *** erecio_1 has quit IRC | 15:32 | |
| *** Shaan7 has quit IRC | 15:36 | |
| *** fabio has joined #openstack-ceilometer | 15:37 | |
| *** fabio has quit IRC | 15:41 | |
| *** fabio has joined #openstack-ceilometer | 15:41 | |
| *** jaypipes has joined #openstack-ceilometer | 15:41 | |
| *** shakamunyi has joined #openstack-ceilometer | 15:42 | |
| *** Shaan7 has joined #openstack-ceilometer | 15:46 | |
| *** erecio_1 has joined #openstack-ceilometer | 15:59 | |
| *** Ruetobas has quit IRC | 16:01 | |
| *** reazem has quit IRC | 16:02 | |
| *** Ruetobas has joined #openstack-ceilometer | 16:03 | |
| *** _nadya_ has quit IRC | 16:03 | |
| *** _cjones_ has joined #openstack-ceilometer | 16:04 | |
| *** _nadya_ has joined #openstack-ceilometer | 16:06 | |
| *** Ruetobas has quit IRC | 16:08 | |
| openstackgerrit | Dina Belova proposed a change to openstack/ceilometer: Don't allow queries with 'IN' predicate with an empty sequence https://review.openstack.org/90392 | 16:08 |
| DinaBelova | folks, it seems to me that py27 gate is failing :-| | 16:09 |
| DinaBelova | for some reason :-{ | 16:09 |
| *** urulama has quit IRC | 16:10 | |
| eglynn | DinaBelova: lots of these: TypeError: argument of type \'NoneType\' is not iterable | 16:12 |
| eglynn | DinaBelova: relating to the ... if 'in' in filter_expr: | 16:13 |
| eglynn | DinaBelova: if filter_expr and 'in' in filter_expr: | 16:13 |
| DinaBelova | eglynn, I've provided new patch set | 16:13 |
| DinaBelova | with validation schema | 16:13 |
| DinaBelova | not that first version | 16:13 |
| *** Ruetobas has joined #openstack-ceilometer | 16:14 | |
| eglynn | DinaBelova: ... a-ha, I didn't see a more recent vote from Jenkins | 16:14 |
| eglynn | DinaBelova: ... I gues you're peeking at zuul | 16:14 |
| DinaBelova | http://logs.openstack.org/75/90375/1/check/gate-ceilometer-python27/7d5b460/console.html - that's other my change | 16:15 |
| DinaBelova | logs | 16:15 |
| DinaBelova | pretty the the same | 16:15 |
| *** reazem has joined #openstack-ceilometer | 16:15 | |
| DinaBelova | and by the way, for it that nice #noqa removal did not pass pep8 testing - quite strange - as locally and for climate (other project) it works cool now | 16:15 |
| eglynn | DinaBelova: hmmm, that failure could be a spurious, try a recheck | 16:16 |
| DinaBelova | eglynn, I'll try to find the bug in the gate bugs - possibly that's known issue for someone already | 16:17 |
| eglynn | DinaBelova: cool | 16:17 |
| *** nacim has quit IRC | 16:21 | |
| DinaBelova | eglynn, well, no, it's smth new) | 16:21 |
| DinaBelova | will recheck no bug) | 16:21 |
| eglynn | DinaBelova: cool | 16:22 |
| sileht | eglynn, DinaBelova this is due to the eventlet.sleep(0) to force task switching: http://git.openstack.org/cgit/openstack/ceilometer/tree/ceilometer/tests/test_collector.py#n173 | 16:24 |
| DinaBelova | sileht, well, it was always there :-\ | 16:25 |
| sileht | I will try to rewrite the test | 16:25 |
| DinaBelova | why it's failing now? | 16:26 |
| DinaBelova | or is that the common stuff for the tests? | 16:26 |
| sileht | the eventlet magic | 16:26 |
| DinaBelova | heh | 16:26 |
| DinaBelova | ok, so I won | 16:26 |
| DinaBelova | won't be so worried) | 16:26 |
| *** yjiang5_away is now known as yjiang5 | 16:32 | |
| *** sdake_ has quit IRC | 16:32 | |
| *** _nadya_ has quit IRC | 16:35 | |
| openstackgerrit | A change was merged to openstack/ceilometer: Made get_capabilities a classmethod instead of object method https://review.openstack.org/88027 | 16:37 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Remove eventlet.sleep(0) in collector tests https://review.openstack.org/90420 | 16:38 |
| sileht | DinaBelova, ^ | 16:39 |
| *** terriyu has joined #openstack-ceilometer | 16:48 | |
| *** terriyu has quit IRC | 16:50 | |
| *** _nadya_ has joined #openstack-ceilometer | 16:51 | |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Remove eventlet.sleep(0) in collector tests https://review.openstack.org/90420 | 16:54 |
| *** _nadya_ has quit IRC | 16:56 | |
| *** igor has joined #openstack-ceilometer | 16:59 | |
| *** igor__ has quit IRC | 17:00 | |
| *** bada has quit IRC | 17:02 | |
| *** openstackstatus has quit IRC | 17:07 | |
| *** openstackstatus has joined #openstack-ceilometer | 17:08 | |
| *** erecio_1 has quit IRC | 17:15 | |
| *** erecio has joined #openstack-ceilometer | 17:17 | |
| *** fnaval has quit IRC | 17:27 | |
| *** fnaval has joined #openstack-ceilometer | 17:27 | |
| *** nati_ueno has joined #openstack-ceilometer | 17:29 | |
| *** nati_uen_ has joined #openstack-ceilometer | 17:31 | |
| *** fnaval has quit IRC | 17:32 | |
| *** nati_ueno has quit IRC | 17:34 | |
| *** eglynn has quit IRC | 18:01 | |
| *** ildikov has joined #openstack-ceilometer | 18:06 | |
| *** fnaval has joined #openstack-ceilometer | 18:19 | |
| *** shardy is now known as shardy_afk | 18:21 | |
| *** nati_uen_ has quit IRC | 18:30 | |
| *** nati_uen_ has joined #openstack-ceilometer | 18:31 | |
| *** nati_ue__ has joined #openstack-ceilometer | 18:35 | |
| *** nati_uen_ has quit IRC | 18:38 | |
| *** eglynn has joined #openstack-ceilometer | 18:39 | |
| *** jaypipes is now known as leakyoldpipes | 18:39 | |
| *** _nadya_ has joined #openstack-ceilometer | 18:50 | |
| *** thomasem has quit IRC | 19:15 | |
| *** thomasem has joined #openstack-ceilometer | 19:33 | |
| *** joesavak has quit IRC | 19:35 | |
| *** igor has quit IRC | 19:36 | |
| *** thomasem has quit IRC | 19:37 | |
| *** _nadya_ has quit IRC | 19:45 | |
| *** eglynn has quit IRC | 19:47 | |
| *** shakamunyi has quit IRC | 19:50 | |
| *** piyushmasrani has quit IRC | 20:12 | |
| *** jdob has quit IRC | 20:15 | |
| *** jdob has joined #openstack-ceilometer | 20:15 | |
| *** eglynn has joined #openstack-ceilometer | 20:17 | |
| *** jdob has quit IRC | 20:24 | |
| *** jdob has joined #openstack-ceilometer | 20:24 | |
| *** igor has joined #openstack-ceilometer | 20:30 | |
| *** eglynn has quit IRC | 20:32 | |
| *** fabio has quit IRC | 20:32 | |
| *** igor has quit IRC | 20:34 | |
| *** joesavak has joined #openstack-ceilometer | 20:42 | |
| *** openstackstatus has quit IRC | 20:45 | |
| *** nealph has quit IRC | 20:46 | |
| *** openstackstatus has joined #openstack-ceilometer | 20:46 | |
| *** nealph has joined #openstack-ceilometer | 20:47 | |
| *** nealph has quit IRC | 20:48 | |
| *** nealph has joined #openstack-ceilometer | 20:49 | |
| *** nealph is now known as nealph_afk | 20:49 | |
| *** nealph_afk has quit IRC | 20:49 | |
| *** openstackstatus has quit IRC | 20:50 | |
| *** openstackstatus has joined #openstack-ceilometer | 20:51 | |
| *** jsavak has joined #openstack-ceilometer | 20:56 | |
| *** ChanServ sets mode: +v openstackstatus | 20:57 | |
| -openstackstatus- NOTICE: Gerrit will be unavailable for a few hours starting at 1600 UTC on Monday April 28th for an upgrade. See https://wiki.openstack.org/wiki/GerritUpgrade | 20:58 | |
| *** joesavak has quit IRC | 20:59 | |
| *** erecio has quit IRC | 21:15 | |
| *** promulo has quit IRC | 21:19 | |
| *** eglynn has joined #openstack-ceilometer | 21:22 | |
| *** thomasem has joined #openstack-ceilometer | 21:26 | |
| *** jdob has quit IRC | 21:29 | |
| *** igor has joined #openstack-ceilometer | 21:30 | |
| *** thomasem has quit IRC | 21:32 | |
| *** igor has quit IRC | 21:35 | |
| *** nati_ue__ has quit IRC | 21:35 | |
| *** nati_uen_ has joined #openstack-ceilometer | 21:36 | |
| *** gordc has quit IRC | 21:39 | |
| *** eglynn has quit IRC | 21:40 | |
| *** nati_ueno has joined #openstack-ceilometer | 21:47 | |
| *** nati_uen_ has quit IRC | 21:51 | |
| *** eglynn has joined #openstack-ceilometer | 21:54 | |
| *** thomasem has joined #openstack-ceilometer | 21:55 | |
| *** eglynn has quit IRC | 21:59 | |
| *** thomasem has quit IRC | 22:00 | |
| *** jsavak has quit IRC | 22:03 | |
| *** prad has quit IRC | 22:09 | |
| *** fnaval has quit IRC | 22:23 | |
| *** igor_ has joined #openstack-ceilometer | 22:30 | |
| *** leakyoldpipes has quit IRC | 22:31 | |
| *** igor_ has quit IRC | 22:34 | |
| *** nati_ueno has quit IRC | 22:35 | |
| *** nati_ueno has joined #openstack-ceilometer | 22:36 | |
| openstackgerrit | Rui Chen proposed a change to openstack/ceilometer: Add memory stats meter to libvirt inspector. https://review.openstack.org/90498 | 22:39 |
| *** promulo has joined #openstack-ceilometer | 22:55 | |
| *** fnaval has joined #openstack-ceilometer | 23:03 | |
| openstackgerrit | Rui Chen proposed a change to openstack/ceilometer: Add memory stats meter to libvirt inspector https://review.openstack.org/90498 | 23:14 |
| openstackgerrit | Rui Chen proposed a change to openstack/ceilometer: Add memory stats meter to libvirt inspector https://review.openstack.org/90498 | 23:29 |
| *** igor has joined #openstack-ceilometer | 23:30 | |
| *** igor has quit IRC | 23:35 | |
| openstackgerrit | A change was merged to openstack/ceilometer: Remove eventlet.sleep(0) in collector tests https://review.openstack.org/90420 | 23:40 |
| *** fnaval has quit IRC | 23:41 | |
| *** fnaval has joined #openstack-ceilometer | 23:42 | |
| *** fnaval has quit IRC | 23:47 | |
| *** shakamunyi has joined #openstack-ceilometer | 23:55 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ceilometer: Updated from global requirements https://review.openstack.org/88714 | 23:56 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!