Closed
Bug 1150540
Opened 10 years ago
Closed 10 years ago
Fix Travis-CI build failing for related tiles
Categories
(Content Services Graveyard :: Tiles: Administration Front-End, defect)
Content Services Graveyard
Tiles: Administration Front-End
Tracking
(Not tracked)
RESOLVED
FIXED
Iteration:
40.2 - 27 Apr
People
(Reporter: oyiptong, Assigned: tspurway)
References
()
Details
(Whiteboard: .004)
The build is failing, because of two unit tests not passing.
The unit tests work fine on local environments, but somehow failing on Travis-CI machines. The tests need to pass before the code is merged.
Reporter | ||
Comment 1•10 years ago
|
||
My strategy is to create a travis-like environment and running CI there
Reporter | ||
Comment 2•10 years ago
|
||
Running tests on an Ubuntu 14.04 linux machine succeeds, with Python 2.7.3.
It seems there is something that is different in the Travis-CI environment.
Will re-create that environment with the travis chef cookbooks https://github.com/travis-ci/travis-cookbooks
Reporter | ||
Comment 3•10 years ago
|
||
Tried another CI solution: https://circleci.com/gh/oyiptong/splice/tree/bug-1126191-related-tiles
It failed as well
Reporter | ||
Comment 4•10 years ago
|
||
Actually, I didn't run tests in the correct branch on an Ubuntu 14.04 machine when I ran it before.
Running fab build on the correct branch yields the same error as on Travis and CircleCI.
Reporter | ||
Comment 5•10 years ago
|
||
My investigation led me to believe it is a platform difference issue.
We develop on Mac OS X and deploy to a linux distribution. This seems to be an issue of platform.
The issue occurs on: TravisCI, CircleCI and in an Ubuntu VM on my local machine.
The changes probably need to be investigated and changed to work on both OSX and Linux
Assignee: oyiptong → tspurway
Reporter | ||
Comment 6•10 years ago
|
||
It looks like sqlite3 is the culprit. Both CircleCI and Travis seem to be using an older version of sqlite that doesn't allow for the insertion of multiple rows in one INSERT query.
The solution may be to use postgresql in CI.
I'll investigate
Assignee: tspurway → oyiptong
Reporter | ||
Comment 7•10 years ago
|
||
Created bug 1153249 for the switch to postgresql which blocks this bug
Reporter | ||
Updated•10 years ago
|
Assignee: oyiptong → tspurway
Updated•10 years ago
|
Iteration: 40.1 - 13 Apr → 40.2 - 27 Apr
Assignee | ||
Updated•10 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•