Merge lp:~james-page/charms/trusty/nova-cloud-controller/tox into lp:~openstack-charmers-archive/charms/trusty/nova-cloud-controller/next

Proposed by James Page
Status: Merged
Merged at revision: 202
Proposed branch: lp:~james-page/charms/trusty/nova-cloud-controller/tox
Merge into: lp:~openstack-charmers-archive/charms/trusty/nova-cloud-controller/next
Diff against target: 85 lines (+58/-0)
5 files modified
.bzrignore (+2/-0)
.testr.conf (+8/-0)
requirements.txt (+11/-0)
test-requirements.txt (+8/-0)
tox.ini (+29/-0)
To merge this branch: bzr merge lp:~james-page/charms/trusty/nova-cloud-controller/tox
Reviewer Review Type Date Requested Status
Corey Bryant (community) Approve
Review via email: mp+276219@code.launchpad.net
To post a comment you must log in.
Revision history for this message
uosci-testing-bot (uosci-testing-bot) wrote :

charm_lint_check #12920 nova-cloud-controller-next for james-page mp276219
    LINT OK: passed

Build: http://10.245.162.77:8080/job/charm_lint_check/12920/

Revision history for this message
uosci-testing-bot (uosci-testing-bot) wrote :

charm_unit_test #11996 nova-cloud-controller-next for james-page mp276219
    UNIT OK: passed

Build: http://10.245.162.77:8080/job/charm_unit_test/11996/

Revision history for this message
uosci-testing-bot (uosci-testing-bot) wrote :

charm_amulet_test #7661 nova-cloud-controller-next for james-page mp276219
    AMULET OK: passed

Build: http://10.245.162.77:8080/job/charm_amulet_test/7661/

Revision history for this message
Corey Bryant (corey.bryant) wrote :

Looks good and tox tests run successfully.

review: Approve
Revision history for this message
Corey Bryant (corey.bryant) wrote :

Note: I added actions dir to lint tests

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file '.bzrignore'
2--- .bzrignore 2015-04-22 21:48:17 +0000
3+++ .bzrignore 2015-10-30 06:30:39 +0000
4@@ -1,3 +1,5 @@
5 bin
6 .coverage
7+.testrepository
8+.tox
9 tags
10
11=== added file '.testr.conf'
12--- .testr.conf 1970-01-01 00:00:00 +0000
13+++ .testr.conf 2015-10-30 06:30:39 +0000
14@@ -0,0 +1,8 @@
15+[DEFAULT]
16+test_command=OS_STDOUT_CAPTURE=${OS_STDOUT_CAPTURE:-1} \
17+ OS_STDERR_CAPTURE=${OS_STDERR_CAPTURE:-1} \
18+ OS_TEST_TIMEOUT=${OS_TEST_TIMEOUT:-60} \
19+ ${PYTHON:-python} -m subunit.run discover -t ./ ./unit_tests $LISTOPT $IDOPTION
20+
21+test_id_option=--load-list $IDFILE
22+test_list_option=--list
23
24=== added file 'requirements.txt'
25--- requirements.txt 1970-01-01 00:00:00 +0000
26+++ requirements.txt 2015-10-30 06:30:39 +0000
27@@ -0,0 +1,11 @@
28+# The order of packages is significant, because pip processes them in the order
29+# of appearance. Changing the order has an impact on the overall integration
30+# process, which may cause wedges in the gate later.
31+PyYAML>=3.1.0
32+simplejson>=2.2.0
33+netifaces>=0.10.4
34+netaddr>=0.7.12,!=0.7.16
35+Jinja2>=2.6 # BSD License (3 clause)
36+six>=1.9.0
37+dnspython>=1.12.0
38+psutil>=1.1.1,<2.0.0
39
40=== added file 'test-requirements.txt'
41--- test-requirements.txt 1970-01-01 00:00:00 +0000
42+++ test-requirements.txt 2015-10-30 06:30:39 +0000
43@@ -0,0 +1,8 @@
44+# The order of packages is significant, because pip processes them in the order
45+# of appearance. Changing the order has an impact on the overall integration
46+# process, which may cause wedges in the gate later.
47+coverage>=3.6
48+mock>=1.2
49+flake8>=2.2.4,<=2.4.1
50+os-testr>=0.4.1
51+charm-tools
52
53=== added file 'tox.ini'
54--- tox.ini 1970-01-01 00:00:00 +0000
55+++ tox.ini 2015-10-30 06:30:39 +0000
56@@ -0,0 +1,29 @@
57+[tox]
58+envlist = lint,py27
59+skipsdist = True
60+
61+[testenv]
62+setenv = VIRTUAL_ENV={envdir}
63+ PYTHONHASHSEED=0
64+install_command =
65+ pip install --allow-unverified python-apt {opts} {packages}
66+commands = ostestr {posargs}
67+
68+[testenv:py27]
69+basepython = python2.7
70+deps = -r{toxinidir}/requirements.txt
71+ -r{toxinidir}/test-requirements.txt
72+
73+[testenv:lint]
74+basepython = python2.7
75+deps = -r{toxinidir}/requirements.txt
76+ -r{toxinidir}/test-requirements.txt
77+commands = flake8 {posargs} hooks unit_tests tests
78+ charm proof
79+
80+[testenv:venv]
81+commands = {posargs}
82+
83+[flake8]
84+ignore = E402,E226
85+exclude = hooks/charmhelpers

Subscribers

People subscribed via source and target branches