Skip to content

/ saltfs

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

Call intermittent checker #453

Merged
merged 1 commit into from Aug 18, 2016
Merged

Call intermittent checker #453

merged 1 commit into from Aug 18, 2016

Conversation

@notriddle
Copy link
Contributor

notriddle commented Aug 4, 2016 •
edited by larsbergstrom

Part of servo/servo#12629

This change is Reviewable
@aneeshusa
Copy link
Member

aneeshusa commented Aug 4, 2016

You'll also need to update buildbot/master/files/config/master.cfg to add new Buildbot Builders so Buildbot knows to schedule + run the jobs.
buildbot/master/files/config/steps.yml Outdated
@@ -27,6 +27,14 @@ mac-nightly:
- ./mach package --release
- ./etc/ci/upload_nightly.sh mac

linux-rel-intermittent:
- ./mach build --release
- ./etc/ci/check_intermittents.sh

This comment has been minimized.

Sign in to view
Copy link
@aneeshusa

aneeshusa Aug 4, 2016

Member

You can add --log-raw intermittents.log so that Buildbot will pick up the log file automatically.
@aneeshusa
Copy link
Member

aneeshusa commented Aug 9, 2016

@asajeffrey I think you wanted the intermittent checker to run nightly instead of on each build, right?
@aneeshusa aneeshusa self-assigned this Aug 9, 2016
@asajeffrey
Copy link
Member

asajeffrey commented Aug 10, 2016

Indeed, we're running the intermittents 300 times each!
@aneeshusa
Copy link
Member

aneeshusa commented Aug 10, 2016

r=me after squash and the servo/servo PR lands.
@notriddle
Call intermittent checker
Loading status checks…
21d29e9 
Part of servo/servo#12629
@notriddle notriddle force-pushed the notriddle:intermittents branch from 94828be to 21d29e9 Aug 10, 2016
@notriddle
Copy link
Contributor Author

notriddle commented Aug 10, 2016

Squished.
@aneeshusa
Copy link
Member

aneeshusa commented Aug 10, 2016

servo/servo#12629 landed.

@bors-servo r+
@bors-servo
Copy link
Contributor

bors-servo commented Aug 10, 2016

📌 Commit 21d29e9 has been approved by aneeshusa
@aneeshusa aneeshusa mentioned this pull request Aug 10, 2016
Closed
@bors-servo
Copy link
Contributor

bors-servo commented Aug 10, 2016

⌛ Testing commit 21d29e9 with merge 57e57c6...
bors-servo added a commit that referenced this pull request Aug 10, 2016
@bors-servo
Auto merge of #453 - notriddle:intermittents, r=aneeshusa
Loading status checks…
57e57c6 
Call intermittent checker

Part of servo/servo#12629

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

bors-servo commented Aug 10, 2016

💥 Test timed out
@aneeshusa
Copy link
Member

aneeshusa commented Aug 17, 2016

@bors-servo r+
@bors-servo
Copy link
Contributor

bors-servo commented Aug 17, 2016

💡 This pull request was already approved, no need to approve it again.

  • This pull request previously failed. You should add more commits to fix the bug, or use retry to trigger a build again.
@bors-servo
Copy link
Contributor

bors-servo commented Aug 17, 2016

📌 Commit 21d29e9 has been approved by aneeshusa
@aneeshusa
Copy link
Member

aneeshusa commented Aug 17, 2016

@bors-servo retry
@bors-servo
Copy link
Contributor

bors-servo commented Aug 17, 2016

⌛ Testing commit 21d29e9 with merge 5813ae5...
bors-servo added a commit that referenced this pull request Aug 17, 2016
@bors-servo
Auto merge of #453 - notriddle:intermittents, r=aneeshusa
Loading status checks…
5813ae5 
Call intermittent checker

Part of servo/servo#12629

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

bors-servo commented Aug 17, 2016

💔 Test failed - status-travis
@aneeshusa
Copy link
Member

aneeshusa commented Aug 18, 2016

@bors-servo retry
@bors-servo
Copy link
Contributor

bors-servo commented Aug 18, 2016

⌛ Testing commit 21d29e9 with merge 6bcf41e...
bors-servo added a commit that referenced this pull request Aug 18, 2016
@bors-servo
Auto merge of #453 - notriddle:intermittents, r=aneeshusa
Loading status checks…
6bcf41e 
Call intermittent checker

Part of servo/servo#12629

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

bors-servo commented Aug 18, 2016

☀️ Test successful - status-travis
@bors-servo bors-servo merged commit 21d29e9 into servo:master Aug 18, 2016
1 of 2 checks passed
1 of 2 checks passed
continuous-integration/travis-ci/pr The Travis CI build failed
Details
homu Test successful
Details
@notriddle notriddle deleted the notriddle:intermittents branch Aug 18, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@aneeshusa aneeshusa

Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
@notriddle @aneeshusa @asajeffrey @bors-servo
You can’t perform that action at this time.