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

Shuffle jobs between builders to hopefully reduce turnaround time. #84

Merged
merged 1 commit into from Jul 29, 2015
Merged

Shuffle jobs between builders to hopefully reduce turnaround time. #84

merged 1 commit into from Jul 29, 2015

Conversation

@Ms2ger
Copy link
Contributor

Ms2ger commented Jul 29, 2015

Review on Reviewable
@Ms2ger
Shuffle jobs between builders to hopefully reduce turnaround time.
Loading status checks…
f9fe4c4
@larsbergstrom
Copy link
Contributor

larsbergstrom commented Jul 29, 2015

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

bors-servo commented Jul 29, 2015

📌 Commit f9fe4c4 has been approved by larsbergstrom
bors-servo pushed a commit that referenced this pull request Jul 29, 2015
bors-servo
Auto merge of #84 - servo:shuffle, r=larsbergstrom
Loading status checks…
d363d21 
Shuffle jobs between builders to hopefully reduce turnaround time.



<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/84)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Jul 29, 2015

⌛ Testing commit f9fe4c4 with merge d363d21...
@bors-servo
Copy link
Contributor

bors-servo commented Jul 29, 2015

💔 Test failed - travis
larsbergstrom added a commit that referenced this pull request Jul 29, 2015
@larsbergstrom
Merge pull request #84 from servo/shuffle
Loading status checks…
bc8f1fe 
Shuffle jobs between builders to hopefully reduce turnaround time.
@larsbergstrom larsbergstrom merged commit bc8f1fe into master Jul 29, 2015
0 of 3 checks passed
0 of 3 checks passed
continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
Details
continuous-integration/travis-ci/push The Travis CI build could not complete due to an error
Details
homu Test failed
Details
@larsbergstrom
Copy link
Contributor

larsbergstrom commented Jul 29, 2015

travis is still broken for saltfs
@Ms2ger Ms2ger deleted the shuffle branch Jul 29, 2015
@metajack
Copy link
Contributor

metajack commented Jul 29, 2015

@edunham Any luck with a fix?
@edunham edunham mentioned this pull request Jul 29, 2015
Merged
bors-servo pushed a commit that referenced this pull request Jul 29, 2015
bors-servo
Auto merge of #86 - edunham:travis-fix, r=metajack
Loading status checks…
55d68d6 
Install Salt from ppa rather than curl|sh

Old way was fragile, and broke, as noted most recently in #84 (comment) 

It'd be even better to use Travis's containers, but that's blocked on travis-ci/apt-source-safelist#82 . Once Saltstack is an approved PPA, it'll be trivial to refactor the `.travis.yml` to stop using sudo as described in http://docs.travis-ci.com/user/migrating-from-legacy/ .

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/86)
<!-- Reviewable:end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
@Ms2ger @larsbergstrom @bors-servo @metajack
You can’t perform that action at this time.