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.

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

CP-8629: Adding travis. #155

Closed
wants to merge 3 commits into from
Closed

CP-8629: Adding travis. #155

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
c035ac3
CP-8629: Adding travis requirements.
siddharthv Jun 3, 2014
441f2df
CP-8629: Don't use travis for xs64bit yet
Jun 3, 2014
ea66f3d
CP-8629: Add travis and coveralls badges for master.
siddharthv Jun 5, 2014
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions .noserc
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
[nosetests]
verbosity=3
16 changes: 16 additions & 0 deletions .travis.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
language: python
python:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please also make sure, that we only enable master at this point?

http://docs.travis-ci.com/user/build-configuration/#White--or-blacklisting-branches

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, I have a pull request to exclude xs64bit: #160

- "2.6"
# command to install dependencies
install:
- "pip install -r requirements.txt --use-mirrors"
# command to run tests
script:
- "pylint --version"
- "PYTHONPATH=./snapwatchd:./drivers pylint --rcfile=tests/pylintrc ./drivers/*.py"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We no longer need snapwatchd. I also have a commit to remove it from the scripts:

https://github.com/matelakat/sm/commit/ef54cefd5375e288bb82ba43d9e4ead55dcd4f85

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, maybe pylint needs it...

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think its best that we include snapwatchd too

- "cd drivers; nosetests --with-coverage -c ../.noserc ../tests/test_*.py"
after_success:
coveralls
branches:
except:
- xs64bit
2 changes: 2 additions & 0 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
Storage Manager for XenServer
=============================
[![Build Status](https://travis-ci.org/xapi-project/sm.svg?branch=master)](https://travis-ci.org/xapi-project/sm)
[![Coverage Status](https://coveralls.io/repos/xapi-project/sm/badge.png?branch=master)](https://coveralls.io/r/xapi-project/sm?branch=master)

This repository contains the code which forms the Storage Management layer for XenSever. It consists of a series of "plug-ins" to xapi (the Xen management layer) which are written primarily in python.

Expand Down
4 changes: 4 additions & 0 deletions requirements.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
mock
xenapi
coveralls
pylint