-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Flaky specs due to sassc "Segmentation fault" errors #2130
Comments
FYI I tried to repro the failure from this one by running the following locally, but I wasn't able to reproduce the error: rm -r gemfiles/*.lock
export RBENV_VERSION=2.7.5
bin/setup
bundle exec rspec --seed 3813
bundle exec appraisal install
BUNDLE_GEMFILE=gemfiles/rails50.gemfile bundle exec rspec --seed 38463
BUNDLE_GEMFILE=gemfiles/rails51.gemfile bundle exec rspec --seed 18633
BUNDLE_GEMFILE=gemfiles/rails52.gemfile bundle exec rspec --seed 3575
BUNDLE_GEMFILE=gemfiles/rails60.gemfile bundle exec rspec --seed 42679 # This one failed in Circle with a segfault |
If this is something that has occasionally happened in the past, then it could be that some of the extra flakiness is due to the fact that we're now running more tests on Circle, due to #2125 being merged? |
I verified that for the example in my comment above (the "FYI I tried to repro the failure ..." one) that the test that segfaulted was the first I wonder if any of the comments in sass/sassc-ruby#207 are applicable? |
We've been seeing (more regular) test failures due to sassc concurrency issues. This should disable concurrency with Sprockets, as suggested on the `sassc-ruby` project and might mitigate it until we adjust the asset build system. Fixes #2130 sass/sassc-ruby#207
Hmm, interesting. I've seen this in the past and it seems much worse now. I just pushed up this branch with a potential solution. I'm going to run it a few times and see how it behaves on CI: https://github.com/thoughtbot/administrate/compare/nc-sassc-seg-fault-fix?expand=1 |
I have an app that I have been to upgrade to |
That branch did no good and I'm not necessarily convinced that spending much time on it is worthwhile. @gathuku, hmm, good question. I don't see why you wouldn't in principle be able to do so. But you might need to fork the gem and change the dependencies around so that it's never bundled. |
With #2397 merged, we are not using sassc any more. I think this should be fixed...? If not let us know and we'll reopen. |
The Ruby 2.6 & 2.7 tests seem to have suddenly become flaky:
I saw some failures like that in a couple of my PRs, that usually went away if I retried the specs (e.g. by pushing an empty commit).
It might be something from one of my recent commits? It doesn't look like they were flaky before that, unless y'all have already been retrying failures like that on master branch when they happened?
Example failure:
https://app.circleci.com/pipelines/github/thoughtbot/administrate/1662/workflows/238e328c-cdd2-4f65-ae6a-24f9789c20c4/jobs/8803
Some example outputs:
The text was updated successfully, but these errors were encountered: