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

Enable a non-gated gczeal builder #373

Merged
merged 2 commits into from May 14, 2016
Merged

Enable a non-gated gczeal builder #373

merged 2 commits into from May 14, 2016

Conversation

@larsbergstrom
Copy link
Contributor

larsbergstrom commented May 13, 2016

This adds a new linux-rel-gczeal builder, but does not currently gate builds on it. It builds release servo with debug mozjs and then runs the WPT tests with four different debug-mode GC settings to stress our bindings.

It is not currently gated because I have no idea how reliable these tests will be.

I've also added another EC2 instance (servo-linux3) for use by this. Note that we're out of static IPs for our account, so you have to go to the EC2 console to get the temp IP and we can't really assign it to a static .servo.org address.

r? @edunham @aneeshusa

cc @nox


This change is Reviewable

@metajack
Copy link
Contributor

metajack commented May 13, 2016

We should be able to request more static IPs.

- ./mach test-wpt --release --processes 24 --pref js.mem.gc.zeal.level=2 --pref js.mem.gc.zeal.frequency=1 --log-raw test-wpt-every-alloc.log --log-errorsummary wpt-errorsummary-every-alloc.log
- ./mach test-wpt --release --processes 24 --pref js.mem.gc.zeal.level=2 --pref js.mem.gc.zeal.frequency=100 --log-raw test-wpt-every-hundred.log --log-errorsummary wpt-errorsummary-every-hundred.log
- ./mach test-wpt --release --processes 24 --pref js.mem.gc.zeal.level=4 --pref js.mem.gc.zeal.frequency=1 --log-raw test-wpt-pre-barriers.log --log-errorsummary wpt-errorsummary-pre-barriers.log
- ./mach test-wpt --release --processes 24 --pref js.mem.gc.zeal.level=11 --pref js.mem.gc.zeal.frequency=1 --log-raw test-wpt-post-barriers.log --log-errorsummary wpt-errorsummary-post-barriers.log

This comment has been minimized.

Copy link
@aneeshusa

aneeshusa May 13, 2016

Member

I'm glad this goes to eleven. 🎸

@aneeshusa
Copy link
Member

aneeshusa commented May 13, 2016

@bors-servo r+

You can also use Salt to find the IP addresses of any minion with salt 'servo-linux3' grains.get ipv4 and salt 'servo-linux3' grains.get ipv6.

@bors-servo
Copy link
Contributor

bors-servo commented May 13, 2016

📌 Commit ae204bf has been approved by aneeshusa

@bors-servo
Copy link
Contributor

bors-servo commented May 13, 2016

Testing commit ae204bf with merge 49a297c...

bors-servo added a commit that referenced this pull request May 13, 2016
Enable a non-gated gczeal builder

This adds a new `linux-rel-gczeal` builder, but does *not* currently gate builds on it. It builds release servo with debug mozjs and then runs the WPT tests with four different debug-mode GC settings to stress our bindings.

It is not currently gated because I have no idea how reliable these tests will be.

I've also added another EC2 instance (servo-linux3) for use by this. Note that we're out of static IPs for our account, so you have to go to the EC2 console to get the temp IP and we can't really assign it to a static .servo.org address.

r? @edunham @aneeshusa

cc @nox

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/373)
<!-- Reviewable:end -->
@aneeshusa
Copy link
Member

aneeshusa commented May 13, 2016

Oh, you'll want to cherry-pick f6f9ee4 before this commit to get around the line length lint failure.

@bors-servo
Copy link
Contributor

bors-servo commented May 13, 2016

💔 Test failed - travis

edunham and others added 2 commits May 11, 2016
servo/servo#8641 made headless go away, so we don't
need a separate environment for it any more.
@larsbergstrom larsbergstrom force-pushed the larsbergstrom:gczeal branch from ae204bf to e595b17 May 14, 2016
@larsbergstrom
Copy link
Contributor Author

larsbergstrom commented May 14, 2016

@bors-servo r=aneeshusa

@bors-servo
Copy link
Contributor

bors-servo commented May 14, 2016

📌 Commit e595b17 has been approved by aneeshusa

@bors-servo
Copy link
Contributor

bors-servo commented May 14, 2016

Testing commit e595b17 with merge f10a2ca...

bors-servo added a commit that referenced this pull request May 14, 2016
Enable a non-gated gczeal builder

This adds a new `linux-rel-gczeal` builder, but does *not* currently gate builds on it. It builds release servo with debug mozjs and then runs the WPT tests with four different debug-mode GC settings to stress our bindings.

It is not currently gated because I have no idea how reliable these tests will be.

I've also added another EC2 instance (servo-linux3) for use by this. Note that we're out of static IPs for our account, so you have to go to the EC2 console to get the temp IP and we can't really assign it to a static .servo.org address.

r? @edunham @aneeshusa

cc @nox

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/373)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented May 14, 2016

☀️ Test successful - travis

@bors-servo bors-servo merged commit e595b17 into servo:master May 14, 2016
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
You can’t perform that action at this time.