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

Add the android-x86 build job #852

Merged
merged 2 commits into from Jul 12, 2018
Merged

Add the android-x86 build job #852

merged 2 commits into from Jul 12, 2018

Conversation

@SimonSapin
Copy link
Member

SimonSapin commented Jul 11, 2018 •
edited by larsbergstrom

CC servo/servo#21124

Should not be deployed before servo/servo#21162 lands, as that would likely break all builds.

This change is Reviewable
@SimonSapin
Fix /dev/kvm permissions on linux-kvm builders
56b8c95
@SimonSapin
Add the android-x86 build job
Loading status checks…
c5fe5fa 
CC servo/servo#21124
@jdm
jdm approved these changes Jul 11, 2018
View changes
@SimonSapin
Copy link
Member Author

SimonSapin commented Jul 12, 2018

@bors-servo r=jdm
@bors-servo
Copy link
Contributor

bors-servo commented Jul 12, 2018

📌 Commit c5fe5fa has been approved by jdm
@bors-servo
Copy link
Contributor

bors-servo commented Jul 12, 2018

⌛ Testing commit c5fe5fa with merge 20c0a8f...
bors-servo added a commit that referenced this pull request Jul 12, 2018
@bors-servo
Auto merge of #852 - servo:kvm, r=jdm
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
20c0a8f 
Add the android-x86 build job

CC servo/servo#21124

Should not be deployed before servo/servo#21162 lands, as that would likely break all builds.

<!-- 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/852)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Jul 12, 2018

☀️ Test successful - status-travis
Approved by: jdm
Pushing 20c0a8f to master...
@bors-servo bors-servo merged commit c5fe5fa into master Jul 12, 2018
3 checks passed
3 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
homu Test successful
Details
This was referenced Jul 12, 2018
Closed
Closed
@SimonSapin SimonSapin deleted the kvm branch Jul 12, 2018
@SimonSapin
Copy link
Member Author

SimonSapin commented Jul 12, 2018

Compared to #850 (comment), running salt-run fileserver.clear_cache helped ID: /home/servo/buildbot/slave Function: file.recurse succeed, but the other task still failed on the new server

          ID: buildbot-slave
    Function: service.running
      Result: False
     Comment: Failed to start the service
     Started: 07:02:11.274890
    Duration: 57.718 ms
     Changes:   
              ----------
              buildbot-slave:
                  False
@SimonSapin
Copy link
Member Author

SimonSapin commented Jul 12, 2018

/etc/init/buildbot-slave.conf exists on servo-linux-kvm1, but service buildbot-slave start says: Failed to start buildbot-slave.service: Unit buildbot-slave.service not found. Maybe this is because Ubuntu switched to systemd between 14.04 and 16.04?
@SimonSapin SimonSapin mentioned this pull request Jul 12, 2018
Merged
bors-servo added a commit that referenced this pull request Jul 12, 2018
@bors-servo
Auto merge of #853 - servo:kvm, r=nox
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
d37e2c6 
Install a systemd service file, not upstart, on Ubuntu 16.04

Follow up to #852 (comment)

<!-- 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/853)
<!-- Reviewable:end -->
@jdm jdm removed the S-needs-deploy label Nov 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jdm jdm

Assignees

@jdm jdm

Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
@SimonSapin @bors-servo @jdm @highfive
You can’t perform that action at this time.