Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Possible race conditions in travis test matrix #1979

Closed
yakky opened this issue Jun 30, 2013 · 5 comments
Closed

Possible race conditions in travis test matrix #1979

yakky opened this issue Jun 30, 2013 · 5 comments

Comments

@yakky
Copy link
Member

yakky commented Jun 30, 2013

Some test jobs fails pretty consistently when the whole build matrix is run at once.
Travis support pointed at some db-name related race condition. Same environments running running one-by-one succed.
@digi604
Copy link
Contributor

digi604 commented Jul 1, 2013

it seams to occur during the frontend tests... i tried to fix it... and tried to some approaches... but failed :(

@digi604
Copy link
Contributor

digi604 commented Jul 9, 2013

This line is strange as fuck as well: https://github.com/divio/django-cms/blob/develop/cms/tests/frontend.py#L44

@yakky
Copy link
Member Author

yakky commented Jul 12, 2013

I've been able to reproduce this locally. Investigating

@yakky
Copy link
Member Author

yakky commented Jul 14, 2013

https://github.com/divio/django-cms/blob/develop/cms/tests/frontend.py#L44 must be removed and proper menu cache clearing must be added instead.

Clearing via menu_pool.clear(all=True) triggers some issue in the LiveServerTestCase, while a plain cache.clear looks rather rude

yakky added a commit to yakky/django-cms that referenced this issue Jul 14, 2013
@yakky
Remove frontend tests from travis until django-cms#1979 is solved
a82445c
@yakky yakky mentioned this issue Jul 14, 2013
Merged
digi604 added a commit that referenced this issue Jul 15, 2013
@digi604
Merge pull request #2052 from yakky/feature/no_selenium_on_travis
326fd99 
Remove frontend tests from travis until #1979 is solved
@yakky
Copy link
Member Author

yakky commented Dec 22, 2013

No longer relevant

@yakky yakky closed this as completed Dec 22, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@digi604 @yakky