| *** yingjun has joined #openstack-searchlight | 00:33 | |
| *** yingjun has quit IRC | 00:33 | |
| *** yingjun has joined #openstack-searchlight | 00:33 | |
| *** yingjun has quit IRC | 00:33 | |
| *** yingjun has joined #openstack-searchlight | 00:34 | |
| *** david-lyle_ is now known as david-lyle | 00:35 | |
| openstackgerrit | Li Yingjun proposed openstack/searchlight: Add option to reindex notification-less plugins https://review.openstack.org/394489 | 00:38 |
|---|---|---|
| *** shu-mutou-AWAY is now known as shu-mutou | 01:53 | |
| openstackgerrit | Cao Xuan Hoang proposed openstack/searchlight-ui: Clean imports in code https://review.openstack.org/398721 | 03:14 |
| *** GB21 has joined #openstack-searchlight | 04:37 | |
| *** pcaruana has joined #openstack-searchlight | 07:18 | |
| *** yingjun has quit IRC | 09:38 | |
| *** openstackgerrit has quit IRC | 09:48 | |
| *** openstackgerrit has joined #openstack-searchlight | 09:49 | |
| *** GB21 has quit IRC | 10:05 | |
| *** GB21 has joined #openstack-searchlight | 10:49 | |
| *** catintheroof has joined #openstack-searchlight | 12:46 | |
| *** GB21 has quit IRC | 12:56 | |
| *** matt-borland has joined #openstack-searchlight | 14:15 | |
| *** yingjun has joined #openstack-searchlight | 15:00 | |
| sjmc7 | Courtesy Searchlight meeting reminder at 15:00 UTC in #openstack-meeting-4: nikhil_k, rosmaita, TravT, david-lyle, sjmc7, GB21, lei-zh, yingjun, RickA-HP, robcresswell, matt-borland, tyr, Kevin_Zheng | 15:00 |
| *** lei-zh has joined #openstack-searchlight | 15:38 | |
| openstackgerrit | Merged openstack/searchlight: Allow notification handler cls to be None https://review.openstack.org/394225 | 15:53 |
| *** pcaruana has quit IRC | 16:10 | |
| *** lei-zh has quit IRC | 16:41 | |
| *** yingjun has quit IRC | 17:22 | |
| *** yingjun has joined #openstack-searchlight | 17:23 | |
| *** yingjun has quit IRC | 17:27 | |
| *** AJaeger has joined #openstack-searchlight | 17:48 | |
| AJaeger | sjmc7, are you around? | 17:49 |
| sjmc7 | yeah, hi | 17:50 |
| * AJaeger likes to discsus https://review.openstack.org/#/c/356757/ | 17:50 | |
| AJaeger | Is the es1 job now working - or is it time to remove the job? | 17:50 |
| sjmc7 | no, we need to remove it | 17:50 |
| sjmc7 | i can put up a patch but fighting some fires at the moment | 17:50 |
| AJaeger | sjmc7: Let me handle that - as part of a reorganization of your test setup. I'll propose something to serachlight and project-config and ping you on that one. Ok? | 17:52 |
| sjmc7 | that’d be awesome, thankyou | 17:52 |
| AJaeger | until later... | 17:52 |
| openstackgerrit | Andreas Jaeger proposed openstack/python-searchlightclient: Add tools/test-setup.sh script to setup test environment https://review.openstack.org/399158 | 18:02 |
| openstackgerrit | Andreas Jaeger proposed openstack/searchlight: Add tools/test-setup.sh script to setup test environment https://review.openstack.org/399159 | 18:02 |
| AJaeger | sjmc7: there will be a corresponding infra change for the changes above - so that searchlight will use the default python setup and we can remove the special treatment. See https://review.openstack.org/399105 for general framework. | 18:03 |
| * AJaeger marks the two changes as WIP until 399105 is in. | 18:04 | |
| sjmc7 | ok, thanks. this is to get rid of our special snowflake test job? | 18:05 |
| sjmc7 | i don’t know if the client requires the server for its tests | 18:06 |
| sjmc7 | sorry, if the client requires elasticsearch | 18:06 |
| AJaeger | sjmc7: right now it is setup this way... | 18:06 |
| sjmc7 | ok. we might be able to remove that for the client tests | 18:06 |
| sjmc7 | i’ll take a look when i get a moment, i’m a bit slammed right now. appreciate you doing this for us | 18:07 |
| AJaeger | sjmc7: would be nice if somebody could confirm this for unittests like tox -e py27 | 18:07 |
| sjmc7 | for the unit tests, elasticsearch will not be required | 18:07 |
| sjmc7 | only for the functional ones | 18:07 |
| AJaeger | how do you call the functional ones? | 18:07 |
| sjmc7 | ah, sorry, they’re run as part of -e py27, excuse me | 18:07 |
| sjmc7 | i am almost 100% certain the client tests don’t require e-s tho | 18:08 |
| sjmc7 | they rely on mocked data | 18:08 |
| AJaeger | that would be cool, then one of my changes i not needed ;) | 18:08 |
| sjmc7 | yeah :) i’ll confirm and -2 it if that’s the case | 18:08 |
| AJaeger | sjmc7: kill those fires and then double check. Thanks | 18:08 |
| AJaeger | sjmc7: the project-config change is https://review.openstack.org/#/c/399169/ . Just for your backlog, no need to check this today... | 18:23 |
| sjmc7 | ok, will do. thanks for doing that for us | 18:23 |
| AJaeger | sjmc7: https://review.openstack.org/#/q/status:open++branch:master+topic:test-setup shows the whole story on what I'm add - nice cleanup and greater flexibility. searchlight was a nice proof of concept that my change is worth it ;) | 18:25 |
| sjmc7 | glad we were a good guinea pig :) | 18:25 |
| AJaeger | ;) | 18:29 |
| AJaeger | searchlight team, your stable/mitaka and liberty branches are broken - https://review.openstack.org/#/q/I56d8c5f21e5dc62e1bace93f79d8d017d6df07e7,n,z shouldn't fail on those. Could somebody check these, please? | 19:41 |
| *** openstackgerrit has quit IRC | 19:48 | |
| *** openstackgerrit has joined #openstack-searchlight | 19:49 | |
| sjmc7 | AJaeger: i’ll take a look | 19:49 |
| sjmc7 | wow, you got a full house on liberty | 19:51 |
| sjmc7 | liberty is a PBR related unpleasantness; looks like setup.cfg needs updating | 19:51 |
| sjmc7 | mitaka.. looks like elasticsearch isn’t starting properly | 19:52 |
| *** catintheroof has quit IRC | 23:29 | |
| *** catintheroof has joined #openstack-searchlight | 23:30 | |
| *** catintheroof has quit IRC | 23:35 | |
| *** shu-mutou is now known as shu-mutou-AWAY | 23:42 | |
| *** matt-borland has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!