*** GB21 has joined #openstack-searchlight | 05:01 | |
*** GB21 has quit IRC | 06:30 | |
-openstackstatus- NOTICE: Gerrit is going to be restarted due to bad performance | 07:15 | |
*** GB21 has joined #openstack-searchlight | 07:32 | |
*** GB21 has quit IRC | 09:01 | |
*** GB21 has joined #openstack-searchlight | 09:20 | |
*** exploreshaifali has joined #openstack-searchlight | 09:28 | |
*** openstackgerrit has quit IRC | 10:48 | |
*** openstackgerrit has joined #openstack-searchlight | 10:49 | |
*** exploreshaifali has quit IRC | 10:55 | |
*** krotscheck_vaca is now known as krotscheck | 11:13 | |
*** exploreshaifali has joined #openstack-searchlight | 11:24 | |
*** GB21 has quit IRC | 11:46 | |
*** yingjun has joined #openstack-searchlight | 12:00 | |
*** yingjun has quit IRC | 12:13 | |
*** yingjun has joined #openstack-searchlight | 12:13 | |
*** GB21 has joined #openstack-searchlight | 12:15 | |
*** yingjun has quit IRC | 12:18 | |
*** GB21 has quit IRC | 12:28 | |
openstackgerrit | Rick Aulino proposed openstack/searchlight: Zero Downtime Re-indexing Unit Tests. https://review.openstack.org/300203 | 14:03 |
---|---|---|
*** sigmavirus24_awa is now known as sigmavirus24 | 14:09 | |
*** exploreshaifali has quit IRC | 14:30 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 15:04 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 15:04 | |
openstackgerrit | Rick Aulino proposed openstack/searchlight: Zero Downtime Re-indexing Unit Tests. https://review.openstack.org/300203 | 15:08 |
*** RickA-HP has joined #openstack-searchlight | 15:12 | |
*** TravT_ has quit IRC | 15:33 | |
openstackgerrit | Rick Aulino proposed openstack/searchlight: Zero Downtime Re-indexing Unit Tests. https://review.openstack.org/300203 | 15:45 |
*** bpokorny has joined #openstack-searchlight | 15:46 | |
*** david-lyle_ has quit IRC | 15:48 | |
*** david-lyle has joined #openstack-searchlight | 15:48 | |
*** TravT has joined #openstack-searchlight | 16:07 | |
*** lakshmiS has joined #openstack-searchlight | 16:11 | |
*** TravT has quit IRC | 16:11 | |
*** TravT has joined #openstack-searchlight | 16:12 | |
openstackgerrit | Steve McLellan proposed openstack/searchlight-specs: WIP Cross-region search spec https://review.openstack.org/301227 | 16:37 |
*** exploreshaifali has joined #openstack-searchlight | 16:58 | |
*** TravT has quit IRC | 17:04 | |
sjmc7 | RickA-HP: taking a final look at https://review.openstack.org/#/c/300203, had a question about the exception handler in https://review.openstack.org/#/c/300203/7/searchlight/elasticsearch/plugins/utils.py | 17:53 |
sjmc7 | since we’re ignoring 404s, are there other exceptions we would expect that we can silently recover from? | 17:53 |
*** bpokorny has quit IRC | 17:59 | |
*** TravT has joined #openstack-searchlight | 18:22 | |
*** TravT has quit IRC | 18:22 | |
*** TravT has joined #openstack-searchlight | 18:22 | |
*** bpokorny has joined #openstack-searchlight | 18:30 | |
openstackgerrit | Travis Tripp proposed openstack/searchlight-ui: Remove unused hz-array-field-directive https://review.openstack.org/301287 | 18:50 |
*** sigmavirus24 is now known as sigmavirus24_awa | 18:58 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 18:58 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 19:21 | |
RickA-HP | sjmc7: Looking at the exception handler. | 19:28 |
RickA-HP | sjmc7: Since we are not specifying a specific key, I don't think we'll get a KeyError. | 19:30 |
sjmc7 | err.. IndexError | 19:30 |
sjmc7 | ? | 19:30 |
RickA-HP | We could get an index ut of range if current_aliases is empty, which may be possible since we are ignoring 404. | 19:31 |
sjmc7 | right. are there other errors we would reasonably expect to happen and that we’d want to ignore? | 19:31 |
RickA-HP | The only other errors I would expect are ES connection errors. | 19:32 |
RickA-HP | We could remove the ignore from get_alias and just keep the exception handler as is. | 19:33 |
RickA-HP | Otherwise we are trapping the same exception twice! | 19:33 |
sjmc7 | why not catch the IndexError? | 19:34 |
sjmc7 | catching all Exceptions and ignoring them is not recommended | 19:34 |
RickA-HP | Instead of ignoring the 404 and then catching the side effect, I would prefer to just catch the 404. | 19:35 |
sjmc7 | ok, that’s fine | 19:37 |
RickA-HP | I think this raises the larger issue of how we handle ES connection failures, but that is another blueprint! | 19:37 |
*** nikhil_k is now known as nikhil | 19:41 | |
*** exploreshaifali has quit IRC | 19:44 | |
*** TravT has quit IRC | 20:13 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 20:17 | |
*** TravT has joined #openstack-searchlight | 20:47 | |
openstackgerrit | Rick Aulino proposed openstack/searchlight: Zero Downtime Re-indexing Unit Tests. https://review.openstack.org/300203 | 21:02 |
*** TravT has quit IRC | 21:11 | |
*** bpokorny has quit IRC | 21:35 | |
*** david-lyle_ has joined #openstack-searchlight | 22:02 | |
*** david-lyle has quit IRC | 22:03 | |
*** david-lyle_ is now known as david-lyle | 22:03 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 22:20 | |
*** TravT has joined #openstack-searchlight | 22:21 | |
*** TravT has quit IRC | 22:22 | |
*** TravT has joined #openstack-searchlight | 22:24 | |
*** david-lyle has quit IRC | 22:44 | |
*** david-lyle has joined #openstack-searchlight | 22:47 | |
*** TravT has quit IRC | 23:29 | |
*** TravT has joined #openstack-searchlight | 23:32 | |
*** TravT has quit IRC | 23:40 | |
openstackgerrit | Tin Lam proposed openstack/searchlight: Breaking up utils.py https://review.openstack.org/299117 | 23:49 |
*** TravT has joined #openstack-searchlight | 23:50 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!