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

CI: Disable linux 32 bits travis build #7263

Merged
merged 1 commit into from
Jan 24, 2019

Conversation

bcardiff
Copy link
Member

@bcardiff bcardiff commented Jan 3, 2019

linux 32 bits build in travis are failing a lot due to OOM (Out of memory) but on CircleCI they are running fine/better.

This disabled that portion in travis, keeping linux64 builds and notifications to IRC/Slack

@bcardiff bcardiff added this to the 0.27.1 milestone Jan 3, 2019
@bcardiff bcardiff requested a review from RX14 January 3, 2019 19:53
@straight-shoota
Copy link
Member

straight-shoota commented Jan 3, 2019

I fear this is only treating the symptoms.

We know from #6961 that even 64-bit linux specs tend to run OOM on travis, even if there should be plenty memory available.
Has someone actually looked into that?

@RX14
Copy link
Contributor

RX14 commented Jan 3, 2019

travis and circle's 32bit builds should fail as often as each other's. The only difference should be on 64bit builds.

@bcardiff
Copy link
Member Author

bcardiff commented Jan 5, 2019

@RX14 I don't follow why the difference should be on 64bit builds. I've usually seen OOM in 32 bits in travis mostly.

@RX14
Copy link
Contributor

RX14 commented Jan 5, 2019

@bcardiff the difference is because the travis and circleci build VMs have different amounts of memory. In 32bit that shouldn't matter because you can only address 4gb of it. But it seems to OOM more on 32bit travis somehow. I don't understand it.

@straight-shoota
Copy link
Member

That's actually similar on 64-bits, only far less frequent. Travis on 64 bit occassionally runs OOM, on circle we only know one instance where this happend.

@bcardiff
Copy link
Member Author

What do we want to do regarding this PR? should we keep 32 bits only in circle or leave things as they are right now for the time being?

@straight-shoota
Copy link
Member

Yes, let's merge this for now. We can always reconsider 32-bit on travis after #6961 is solved.

@straight-shoota
Copy link
Member

Now 32-bit on circleci fails pretty regularly as well (about every third build).

@asterite
Copy link
Member

Alternatively I was thinking of writing a bash script or any script that, in 32-bit ci, would run specs split by directory. So one run for all spec/std/*, one run for all spec/std/json/**, one run for all spec/compiler/semantic/**, something like that. The compiler specs leak memory and on 32 bits it's more noticeable. I think doing some hack/fix on CI 32 bits is a better way to go rather than disabling it (by I'm a newb writing bash scripts).

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

Successfully merging this pull request may close these issues.

5 participants