Closed
Bug 764826
Opened 12 years ago
Closed 12 years ago
Re-enable elastic search tests
Categories
(Marketplace Graveyard :: Search, defect, P3)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: kumar, Assigned: hschlichting)
References
Details
(Whiteboard: p=3)
AMO/Marketplace relies on ES for our pages that will be consumed the heaviest: public browsing and searching. We must run these tests in our CI suite to maintain code quality and prevent regressions.
Currently these tests are disabled because of jenkins performance issues. This is a bug to work with IT to re-enable the tests (without email spam) so we can work to stabilize jenkins enough to run the tests all the time in our main builder.
Comment 1•12 years ago
|
||
I think this is a dupe of the already existing ES bugs in this milestone, but throwing it in the mix.
Assignee: nobody → kumar.mcmillan
Target Milestone: --- → 2012-08-30
Reporter | ||
Updated•12 years ago
|
Target Milestone: 2012-08-30 → 2012-09-06
Reporter | ||
Updated•12 years ago
|
Target Milestone: 2012-09-06 → 2012-09-20
Reporter | ||
Updated•12 years ago
|
Target Milestone: 2012-09-20 → 2012-09-27
Reporter | ||
Comment 2•12 years ago
|
||
let's turn these tests on as soon as we can. I still see failures: https://ci.mozilla.org/job/marketplace-es/
Reporter | ||
Updated•12 years ago
|
Priority: -- → P2
Target Milestone: 2012-09-27 → ---
Reporter | ||
Updated•12 years ago
|
Priority: P2 → P3
Comment 3•12 years ago
|
||
We also want to run lib.es.tests now in the test suite.
I see other tests there I am not sure we launch:
$ find lib -name 'tests'
lib/es/tests
lib/geoip/tests
lib/misc/tests
I guess we can tweak scripts/run_mkt_tests.sh to run mkt *and* lib
What do you think Kumar ?
Flags: needinfo?(kumar.mcmillan)
Reporter | ||
Comment 4•12 years ago
|
||
Oh yeah, true. We can add that in settings https://github.com/mozilla/zamboni/blob/master/mkt/settings.py#L153 ; it just needs an additional --where arg for the lib path (it can be specified multiple times).
Flags: needinfo?(kumar.mcmillan)
Updated•12 years ago
|
Assignee: kumar.mcmillan → nobody
Updated•12 years ago
|
Whiteboard: p=3
Updated•12 years ago
|
Component: Code Quality → Search
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → hschlichting
Assignee | ||
Comment 5•12 years ago
|
||
We have enabled the Jenkins ES job and all the tests should pass again. We skipped a couple, but there's extra open bugs for those.
Assignee | ||
Updated•12 years ago
|
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
•