| *** yingjun has joined #openstack-searchlight | 00:04 | |
| *** bpokorny_ has joined #openstack-searchlight | 00:14 | |
| *** bpokorny_ has quit IRC | 00:15 | |
| *** bpokorny_ has joined #openstack-searchlight | 00:15 | |
| *** bpokorn__ has joined #openstack-searchlight | 00:17 | |
| *** bpokorny_ has quit IRC | 00:17 | |
| *** bpokorny has quit IRC | 00:18 | |
| *** bpokorn__ has quit IRC | 00:18 | |
| *** bpokorny has joined #openstack-searchlight | 00:19 | |
| *** sigmavirus24 is now known as sigmavirus24_awa | 00:45 | |
| openstackgerrit | Lakshmi N Sampath proposed openstack/searchlight: Image type should be a facet https://review.openstack.org/317741 | 00:54 |
|---|---|---|
| *** bpokorny has quit IRC | 00:57 | |
| openstackgerrit | Lakshmi N Sampath proposed openstack/searchlight: Resource types for reference and parent id https://review.openstack.org/317167 | 01:03 |
| *** Kevin_Zheng has joined #openstack-searchlight | 01:03 | |
| *** yingjun has quit IRC | 01:06 | |
| *** shu-mutou-AFK is now known as shu-mutou | 01:17 | |
| *** yingjun has joined #openstack-searchlight | 01:31 | |
| openstackgerrit | Lakshmi N Sampath proposed openstack/searchlight: Image type should be a facet https://review.openstack.org/317741 | 01:43 |
| openstackgerrit | Lakshmi N Sampath proposed openstack/searchlight: Resource types for reference and parent id https://review.openstack.org/317167 | 02:19 |
| openstackgerrit | Lakshmi N Sampath proposed openstack/searchlight: Image type should be a facet https://review.openstack.org/317741 | 02:39 |
| *** lakshmiS has quit IRC | 02:52 | |
| *** tyr_ has joined #openstack-searchlight | 02:59 | |
| *** tyr_ has quit IRC | 03:04 | |
| openstackgerrit | Travis Tripp proposed openstack/searchlight: Resource types for reference and parent id https://review.openstack.org/317167 | 03:31 |
| openstackgerrit | Travis Tripp proposed openstack/searchlight: Resource types for reference and parent id https://review.openstack.org/317167 | 03:33 |
| openstackgerrit | Li Yingjun proposed openstack/searchlight-ui: Change None to Search for breadcrumb nav https://review.openstack.org/318432 | 03:54 |
| *** tyr_ has joined #openstack-searchlight | 04:08 | |
| *** tyr_ has quit IRC | 04:13 | |
| *** GB21 has joined #openstack-searchlight | 04:46 | |
| *** GB21 has quit IRC | 04:51 | |
| *** yingjun has quit IRC | 04:56 | |
| *** GB21 has joined #openstack-searchlight | 05:06 | |
| *** GB21 has quit IRC | 05:37 | |
| *** GB21 has joined #openstack-searchlight | 06:14 | |
| *** itisha has joined #openstack-searchlight | 06:22 | |
| *** exploreshaifali has joined #openstack-searchlight | 06:54 | |
| *** yingjun has joined #openstack-searchlight | 07:26 | |
| *** exploreshaifali has quit IRC | 07:27 | |
| *** yingjun has quit IRC | 07:46 | |
| *** GB21 has quit IRC | 07:55 | |
| *** exploreshaifali has joined #openstack-searchlight | 07:56 | |
| *** yingjun has joined #openstack-searchlight | 07:59 | |
| *** exploreshaifali has quit IRC | 08:02 | |
| *** yingjun has quit IRC | 08:06 | |
| *** GB21 has joined #openstack-searchlight | 08:21 | |
| *** exploreshaifali has joined #openstack-searchlight | 08:42 | |
| *** yingjun has joined #openstack-searchlight | 08:43 | |
| *** lei-zh has joined #openstack-searchlight | 09:25 | |
| *** GB21 has quit IRC | 09:30 | |
| *** yingjun has quit IRC | 09:34 | |
| *** GB21 has joined #openstack-searchlight | 09:59 | |
| *** shu-mutou is now known as shu-mutou-AFK | 10:05 | |
| *** lei-zh has quit IRC | 10:19 | |
| *** exploreshaifali has quit IRC | 10:29 | |
| *** exploreshaifali has joined #openstack-searchlight | 11:04 | |
| *** exploreshaifali has quit IRC | 11:39 | |
| *** exploreshaifali has joined #openstack-searchlight | 12:06 | |
| *** yingjun has joined #openstack-searchlight | 12:20 | |
| *** GB21 has quit IRC | 12:24 | |
| *** exploreshaifali has quit IRC | 12:49 | |
| *** tyr_ has joined #openstack-searchlight | 14:36 | |
| *** tyr_ has quit IRC | 14:41 | |
| *** sigmavirus24_awa is now known as sigmavirus24 | 14:48 | |
| openstackgerrit | Travis Tripp proposed openstack/searchlight: Add Release notes for fixes backported to 0.2.1 https://review.openstack.org/318365 | 14:51 |
| TravT | Courtesy Searchlight meeting reminder 1 minute in #openstack-meeting-4: lakshmiS, nikhil_k, rosmaita, TravT, david-lyle, sjmc7, itisha, GB21, briancline, lei-zh, yingjun, RickA-HP | 14:59 |
| *** lei-zh has joined #openstack-searchlight | 15:01 | |
| *** tyr_ has joined #openstack-searchlight | 15:10 | |
| *** itisha has quit IRC | 15:19 | |
| *** lcastell has joined #openstack-searchlight | 15:25 | |
| *** bpokorny has joined #openstack-searchlight | 15:57 | |
| *** lakshmiS has joined #openstack-searchlight | 16:01 | |
| *** bpokorny has quit IRC | 16:05 | |
| *** yingjun has quit IRC | 16:10 | |
| *** yingjun has joined #openstack-searchlight | 16:10 | |
| *** bpokorny has joined #openstack-searchlight | 16:14 | |
| *** yingjun has quit IRC | 16:15 | |
| *** lei-zh has left #openstack-searchlight | 16:21 | |
| openstackgerrit | Tyr Johanson proposed openstack/searchlight-ui: WIP Integrating sl-magic-search and searchlight-ui https://review.openstack.org/317760 | 16:47 |
| openstackgerrit | Tyr Johanson proposed openstack/searchlight-ui: Searchlight version of Magic Search https://review.openstack.org/317133 | 16:52 |
| *** bpokorny has quit IRC | 16:55 | |
| openstackgerrit | Merged openstack/searchlight-ui: Search Syntax Directive https://review.openstack.org/317073 | 16:56 |
| openstackgerrit | Travis Tripp proposed openstack/searchlight: Add Release notes for fixes backported to 0.2.1 https://review.openstack.org/318365 | 17:01 |
| openstackgerrit | Tyr Johanson proposed openstack/searchlight-ui: Searchlight version of Magic Search https://review.openstack.org/317133 | 17:03 |
| *** bpokorny has joined #openstack-searchlight | 17:06 | |
| openstackgerrit | Steve McLellan proposed openstack/searchlight: Correct cinder 'exchange' value https://review.openstack.org/318822 | 17:21 |
| openstackgerrit | Tyr Johanson proposed openstack/searchlight-ui: WIP Integrating sl-magic-search and searchlight-ui https://review.openstack.org/317760 | 17:25 |
| sjmc7 | TravT: lakshmiS: just looking over the plugin info patch again | 17:30 |
| sjmc7 | for nova, security groups and AZ isn’t an id, just a name | 17:30 |
| sjmc7 | do we still want to have that linkage? | 17:30 |
| lakshmiS | yes TravT wanted to lookup based on name | 17:30 |
| TravT | well, i suggested it to lakshmi in last one | 17:30 |
| sjmc7 | ok, just checking | 17:31 |
| TravT | because you can still look it up based on name | 17:31 |
| TravT | i believe | 17:31 |
| TravT | if that is wrong, then we can change | 17:31 |
| sjmc7 | sure, as long as you know that it’s a name | 17:31 |
| TravT | well, there isn't an id for it, i think. | 17:32 |
| sjmc7 | no | 17:32 |
| TravT | so name is basically the id | 17:32 |
| sjmc7 | they’re not identical as far as a consumer is concerned though | 17:33 |
| TravT | well, at least on UI side, we are registering a load function | 17:33 |
| sjmc7 | i.e. if it’s a name i need to do an additional lookup if i’m going to the nova api, or need to use a name: term to look it up with us | 17:33 |
| TravT | each resource type has a javascript item load function which takes an input to get the thing | 17:34 |
| TravT | i think it is okay for now... | 17:35 |
| sjmc7 | ok | 17:35 |
| sjmc7 | i guess we can add something if it becomes a problem | 17:35 |
| TravT | getting these into facets is nice... and i really still think we need to get this info without querying facets | 17:35 |
| sjmc7 | names aren’t unique, being the most obvious one | 17:36 |
| sjmc7 | yeah, i kind of wish we’d though about where to put mapping-type stuff a bit more | 17:36 |
| TravT | i think we either need to get this info available via plugin endpoint | 17:38 |
| TravT | and / or support a /mapping endpoint | 17:38 |
| sjmc7 | yeah. i’m not super keen on having it twice | 17:38 |
| sjmc7 | i think in retrospect the mappings we should’ve done in something like /plugin_info/mapping | 17:39 |
| TravT | a /mapping endpoint would increase the likelihood that we can re-use other ES libraries | 17:39 |
| *** TravT_ has joined #openstack-searchlight | 17:41 | |
| TravT_ | internet hiccup | 17:41 |
| TravT_ | not sure if this went through | 17:41 |
| TravT_ | 11:39 TravT: a /mapping endpoint would increase the likelihood that we can re-use other ES libraries | 17:41 |
| TravT_ | 11:40 TravT: e.g. GET searchlight-search/_mapping | 17:41 |
| sjmc7 | hmmm | 17:41 |
| lakshmiS | good from a ES tools perspective | 17:43 |
| *** TravT has quit IRC | 17:44 | |
| lakshmiS | its a tree facet without a count ;) | 17:45 |
| lakshmiS | one way could be replace facets api with mapping where it follows the ES mapping structure and have the count/otherinfo in _meta | 17:46 |
| openstackgerrit | Merged openstack/searchlight: Resource types for reference and parent id https://review.openstack.org/317167 | 17:47 |
| sjmc7 | it’s more that the facet payload contains a lot of information that probably won’t change in the lifetime of whatever’s consuming it | 17:50 |
| sjmc7 | so maybe would’ve made more sense ot have the /plugins info represent the static information and /facets just the counts | 17:51 |
| sjmc7 | and values | 17:51 |
| sjmc7 | hindsight is a wonderful thing | 17:52 |
| TravT_ | sjmc7 i'm not sure the two aren't okay to co-exist | 17:57 |
| TravT_ | it is nice from consumer of facet to just have resource type in it | 17:57 |
| sjmc7 | yeah, maybe not. just not keen on two ways to get the same information | 17:58 |
| openstackgerrit | Tyr Johanson proposed openstack/searchlight-ui: WIP Integrating sl-magic-search and searchlight-ui https://review.openstack.org/317760 | 17:59 |
| sjmc7 | TravT_: think we should consider https://review.openstack.org/#/c/318822/ for backport | 18:01 |
| TravT_ | i was just looking at that, but got distracted | 18:01 |
| sjmc7 | it was a lot of fun discovering it | 18:01 |
| sjmc7 | i’m going to eat cheese, biab | 18:02 |
| TravT_ | i don't quite understand how things worked before | 18:02 |
| *** TravT_ is now known as TravT | 18:17 | |
| *** jaimguer has quit IRC | 18:26 | |
| *** jaimguer has joined #openstack-searchlight | 18:28 | |
| openstackgerrit | Tyr Johanson proposed openstack/searchlight-ui: WIP Integrating sl-magic-search and searchlight-ui https://review.openstack.org/317760 | 18:34 |
| *** tyr_ has quit IRC | 18:38 | |
| sjmc7 | TravT: rabbit works in mysterious ways | 18:44 |
| sjmc7 | it works because ultimately the queue is what gets listened to | 18:45 |
| sjmc7 | and since cinder binds the queue to the ‘openstack’ exchange it doesn’t matter that SL was also binding it to a previously non-existent ‘cinder’ one | 18:45 |
| sjmc7 | when i turned pools on, the new queue created was only bound to ‘cinder’ and received nada | 18:45 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/searchlight: Updated from global requirements https://review.openstack.org/318870 | 18:48 |
| TravT | sjmc7: got it. | 18:51 |
| TravT | i'll test out that patch this afternoon | 18:51 |
| TravT | i have to go in to the office for a couple meetings right now | 18:51 |
| TravT | so will be offline for a bit | 18:51 |
| *** TravT has quit IRC | 18:58 | |
| openstackgerrit | Steve McLellan proposed openstack/searchlight: Add totals to facets https://review.openstack.org/308582 | 19:52 |
| *** tyr_ has joined #openstack-searchlight | 19:56 | |
| *** TravT has joined #openstack-searchlight | 20:02 | |
| *** TravT has quit IRC | 20:16 | |
| *** TravT has joined #openstack-searchlight | 20:16 | |
| *** lakshmiS has quit IRC | 21:45 | |
| sjmc7 | TravT: should we change the documentation to recommend notification pools, given that it won’t have been tested anywhere near as much as a separate topic? | 22:11 |
| sjmc7 | or add it as an experimental option and update the docs later in newton? | 22:11 |
| sjmc7 | that sounds a bit safer | 22:12 |
| TravT | ummm | 22:12 |
| TravT | just sec. | 22:13 |
| TravT | so, we still have configurable topics | 22:17 |
| sjmc7 | yep, either is an option | 22:17 |
| TravT | and make pool usage be default? | 22:17 |
| sjmc7 | just a question of which one we recommend | 22:17 |
| TravT | i like recommending the option that doesn't require changing the configuration of other services. | 22:17 |
| sjmc7 | for newton, i’d def say pools, but given we intend to backport it there’s some risk | 22:17 |
| TravT | but from a technical reason, i'm not sure other advantages | 22:18 |
| sjmc7 | i will not be 100% confident in it until we’ve tested it a lot more | 22:18 |
| TravT | so, let's make it the default and document the ability to not use it | 22:18 |
| TravT | newton 1 is the time to do that | 22:19 |
| sjmc7 | i mean specifically for this patch that we intend to backport | 22:19 |
| TravT | hmmm... | 22:19 |
| TravT | i'm not confident in backporting that as the default | 22:19 |
| sjmc7 | me neither | 22:19 |
| sjmc7 | ok, i’ll add it as an experimetnal alternative in the docs, and we can update it for newton | 22:20 |
| TravT | that makes sense to me | 22:20 |
| *** sigmavirus24 is now known as sigmavirus24_awa | 22:20 | |
| openstackgerrit | Steve McLellan proposed openstack/searchlight: Enable notification messaging pools https://review.openstack.org/318967 | 22:30 |
| *** TravT has quit IRC | 22:34 | |
| *** tyr_ has quit IRC | 23:19 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!