Closed Bug 764826 Opened 12 years ago Closed 12 years ago

Re-enable elastic search tests

Categories

(Marketplace Graveyard :: Search, defect, P3)

x86
macOS
defect

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.
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
Target Milestone: 2012-08-30 → 2012-09-06
Target Milestone: 2012-09-06 → 2012-09-20
Target Milestone: 2012-09-20 → 2012-09-27
Blocks: 793997
let's turn these tests on as soon as we can. I still see failures: https://ci.mozilla.org/job/marketplace-es/
Priority: -- → P2
Target Milestone: 2012-09-27 → ---
Priority: P2 → P3
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)
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)
Assignee: kumar.mcmillan → nobody
Whiteboard: p=3
Depends on: 784047
Depends on: 803070
Component: Code Quality → Search
Assignee: nobody → hschlichting
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.
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.