Closed
Bug 739794
Opened 12 years ago
Closed 12 years ago
change how we specify the index name for elasticsearch
Categories
(support.mozilla.org :: Search, defect, P1)
support.mozilla.org
Search
Tracking
(Not tracked)
RESOLVED
FIXED
2012.7
People
(Reporter: willkg, Assigned: willkg)
References
Details
(Whiteboard: u=dev c=search p=1)
On dev and stage, ES_INDEXES is specified in settings_local.py which is not in our repository and thus we don't have a way of updating it. Because of that, our read/write indexing thing doesn't work. elasticutils get_es() uses ES_INDEXES as the default index. When you create an S, you pass in a model and it looks at ES_INDEXES[model._meta.db_table] to derive the index it's going to fiddle with. Thus this creates the problem where we can't effectively specify the index to use in the settings.py file and we'd need additional access to change things. One possible solution is to change things so that instead of ES_INDEXES and ES_WRITE_INDEXES having to start with ES_INDEX_PREFIX, we instead prepend ES_INDEX_PREFIX to ES_INDEXES and ES_WRITE_INDEXES. Then dev and stage can specify ES_INDEX_PREFIX and we can change the settings as we need to.
Assignee | ||
Comment 1•12 years ago
|
||
I think this is a 1pter. It requires changes to elasticutils, but should otherwise be pretty easy to do. Assigning it to myself. If it doesn't get done in 2012.6, then it should get done in 2012.7. Without this we can't do any reindex-requiring elasticsearch code changes after we ditch Sphinx.
Assignee | ||
Comment 2•12 years ago
|
||
Making this a P1 because it blocks removing Sphinx.
Priority: -- → P1
Updated•12 years ago
|
Target Milestone: 2012.6 → 2012.7
Assignee | ||
Comment 3•12 years ago
|
||
https://github.com/mozilla/kitsune/pull/564
Assignee | ||
Comment 4•12 years ago
|
||
This is all set. Pushing it to production is blocking on two things: 1. Cww is supposed to be doing CTR/metrics for the search results changes. I need to coordinate with him to make sure pushing to production (which will require us to switch to Sphinx for a couple of hours) is ok. 2. We need IT to change ES_INDEX_PREFIX on -dev and -stage. It's fine on prod.
Assignee | ||
Comment 5•12 years ago
|
||
Landed in master in https://github.com/mozilla/kitsune/commit/76638283e619ea7ecc81179aadd3b1f451c24d47 Pushed to production this morning.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•