Merge lp:~tvansteenburgh/charms/trusty/nova-compute/fix-tests into lp:~openstack-charmers-archive/charms/trusty/nova-compute/trunk

Proposed by Tim Van Steenburgh
Status: Merged
Merged at revision: 75
Proposed branch: lp:~tvansteenburgh/charms/trusty/nova-compute/fix-tests
Merge into: lp:~openstack-charmers-archive/charms/trusty/nova-compute/trunk
Diff against target: 59 lines (+23/-2)
3 files modified
Makefile (+10/-2)
README.txt (+11/-0)
config.yaml (+2/-0)
To merge this branch: bzr merge lp:~tvansteenburgh/charms/trusty/nova-compute/fix-tests
Reviewer Review Type Date Requested Status
Charles Butler (community) Approve
Review via email: mp+234035@code.launchpad.net

Description of the change

Fix tests, proof errors.

To post a comment you must log in.
Revision history for this message
Charles Butler (lazypower) wrote :

+1 LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'Makefile'
2--- Makefile 2014-08-13 15:55:07 +0000
3+++ Makefile 2014-09-10 00:05:20 +0000
4@@ -5,9 +5,9 @@
5 @flake8 --exclude hooks/charmhelpers hooks unit_tests
6 @charm proof
7
8-test:
9+test: .venv
10 @echo Starting tests...
11- @$(PYTHON) /usr/bin/nosetests --nologcapture --with-coverage unit_tests
12+ .venv/bin/nosetests --nologcapture --with-coverage unit_tests
13
14 bin/charm_helpers_sync.py:
15 @mkdir -p bin
16@@ -20,3 +20,11 @@
17 publish: lint test
18 bzr push lp:charms/nova-compute
19 bzr push lp:charms/trusty/nova-compute
20+
21+.venv:
22+ sudo apt-get install python-virtualenv python-apt
23+ virtualenv .venv --system-site-packages
24+ .venv/bin/pip install -I nose pyyaml mock
25+
26+clean:
27+ rm -rf .venv
28
29=== added file 'README.txt'
30--- README.txt 1970-01-01 00:00:00 +0000
31+++ README.txt 2014-09-10 00:05:20 +0000
32@@ -0,0 +1,11 @@
33+============
34+nova-compute
35+============
36+
37+Compute node for Openstack nova.
38+
39+OpenStack Compute, codenamed Nova, is a cloud computing fabric controller. In
40+addition to its "native" API (the OpenStack API), it also supports the Amazon
41+EC2 API.
42+
43+Maintainer: Adam Gandelman <adamg@canonical.com>
44
45=== modified file 'config.yaml'
46--- config.yaml 2014-09-02 15:25:14 +0000
47+++ config.yaml 2014-09-10 00:05:20 +0000
48@@ -91,9 +91,11 @@
49 description: Network interface on which to build bridge
50 instances-path:
51 type: string
52+ default:
53 description: Instance path to use - empty means default of /var/lib/nova/instances
54 config-flags:
55 type: string
56+ default:
57 description: Comma separated list of key=value config flags to be set in nova.conf.
58 nagios_context:
59 default: "juju"

Subscribers

People subscribed via source and target branches