Issue #87. Moztrap #5930 #101

Merged
merged 7 commits into from Mar 22, 2013

Conversation

Projects
None yet
4 participants
Contributor

sashakruglov commented Jan 30, 2013

https://moztrap.mozilla.org/manage/case/5930/

this test is not fast. there are 2 reasons for that:

  • test setup (need to create product, test cases and test suites)
  • web client is slow on filtering

hopefully with upcoming API we can speed it up.

Contributor

sashakruglov commented Jan 30, 2013

About the second issue: we could ask devs to clean up the database on dev platform. It might improve performance.

Contributor

AlinT commented Feb 12, 2013

Please resolve the merge conflicts.

Contributor

sashakruglov commented Feb 12, 2013

Resolved merge conflicts

Contributor

AndreiH commented Feb 14, 2013

merge conflicts again :)

Contributor

sashakruglov commented Feb 14, 2013

I guess all others pull requests require updates as well.
I can update this one, and then wait until its merged, then update the next one and till there are none of outstanding PRs.

Contributor

sashakruglov commented Feb 14, 2013

Fixed merge conflict. Again ;)

Contributor

AndreiH commented Feb 15, 2013

I will try to find out if cleaning the database should be a better solution.

Contributor

AndreiH commented Feb 15, 2013

Looks good to me, don't know what we can do to improve performance for now. Maybe somebody else would like something else to add/comment on this pull

Contributor

sashakruglov commented Feb 15, 2013

@AndreiH I heard that @bobsilverberg is working on this problem.
And I'm trying to progress on moztrap restful API, but it requires some time before I can send a pull request for moztrap API client.

Contributor

sashakruglov commented Mar 10, 2013

resolve merge conflicts in pull request.

Contributor

AndreiH commented Mar 22, 2013

again nice pull @sashakruglov, this LGTM :)

Collaborator

stephendonner commented Mar 22, 2013

Merging per the r+

@stephendonner stephendonner added a commit that referenced this pull request Mar 22, 2013

@stephendonner stephendonner Merge pull request #101 from sashakruglov/moztrap_5930
Issue #87. Moztrap #5930
ae4d5fa

@stephendonner stephendonner merged commit ae4d5fa into mozilla:master Mar 22, 2013

sashakruglov deleted the sashakruglov:moztrap_5930 branch Mar 23, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment