Skip to content

CP-15745: Use xenserver-build-env for travis build #2599

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Mar 2, 2016
Merged

CP-15745: Use xenserver-build-env for travis build #2599

merged 1 commit into from
Mar 2, 2016

Conversation

@johnelse
Copy link
Contributor

No description provided.

@johnelse johnelse force-pushed the cp-15745 branch 5 times, most recently from c236e4c to a29dc67 Compare February 29, 2016 00:11
@johnelse
Copy link
Contributor Author

Travis is failing because the external repo is returning 403. Once that is fixed, this should work.

@simonjbeaumont
Copy link
Contributor

Cool. This looks sane. Although it's blocked behind the yum repo 403s.

@djs55
Copy link
Contributor

djs55 commented Mar 1, 2016

How are you finding docker on travis? I've not gotten around to trying it myself.

@johnelse
Copy link
Contributor Author

johnelse commented Mar 1, 2016

@djs55 it works well actually - the xenserver-build-env CI has been using it for a while now. Just add services: docker to your .travis.yml and it all seems to Just Work. There were a few early issues with stopping and rm'ing containers, but those seem to be fixed now.

@johnelse johnelse force-pushed the cp-15745 branch 4 times, most recently from 19d46fa to a0b7035 Compare March 1, 2016 15:23
@johnelse
Copy link
Contributor Author

johnelse commented Mar 1, 2016

Depends on xenserver/xenserver-build-env#55

@johnelse johnelse force-pushed the cp-15745 branch 2 times, most recently from 1c490d8 to 263ef00 Compare March 1, 2016 18:15
@johnelse
Copy link
Contributor Author

johnelse commented Mar 1, 2016

Updated - config is now all via .travis.yml.

@johnelse
CP-15745: Use xenserver-build-env for travis build
382f74d 
Signed-off-by: John Else <john.else@citrix.com>
@simonjbeaumont
Copy link
Contributor

This looks good now! Expecting a ✅ from travis now that xenserver/xenserver-build-env#55 has been merged.

@simonjbeaumont
Copy link
Contributor

Not waiting for @xen-git since this change is only to the travis.yml.

simonjbeaumont added a commit that referenced this pull request Mar 2, 2016
@simonjbeaumont
Merge pull request #2599 from johnelse/cp-15745
7a3f389 
CP-15745: Use xenserver-build-env for travis build
@simonjbeaumont simonjbeaumont merged commit 7a3f389 into xapi-project:master Mar 2, 2016
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

reviewed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@johnelse @simonjbeaumont @djs55