Merge lp:~ivoks/charms/trusty/neutron-api-contrail/remove-running-services into lp:~sdn-charmers/charms/trusty/neutron-api-contrail/trunk

Proposed by Ante Karamatić
Status: Merged
Merged at revision: 13
Proposed branch: lp:~ivoks/charms/trusty/neutron-api-contrail/remove-running-services
Merge into: lp:~sdn-charmers/charms/trusty/neutron-api-contrail/trunk
Diff against target: 25 lines (+2/-2)
2 files modified
hooks/neutron_api_contrail_hooks.py (+1/-1)
hooks/neutron_api_contrail_utils.py (+1/-1)
To merge this branch: bzr merge lp:~ivoks/charms/trusty/neutron-api-contrail/remove-running-services
Reviewer Review Type Date Requested Status
Robert Ayres (community) Approve
Review via email: mp+304936@code.launchpad.net

Description of the change

No need on installing a package that pulls in almost all Contrail services. We can easily use python library to compare Contrail versions.

To post a comment you must log in.
Revision history for this message
Robert Ayres (robert-ayres) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'hooks/neutron_api_contrail_hooks.py'
2--- hooks/neutron_api_contrail_hooks.py 2016-05-19 03:29:07 +0000
3+++ hooks/neutron_api_contrail_hooks.py 2016-09-05 15:41:40 +0000
4@@ -31,7 +31,7 @@
5 write_plugin_config
6 )
7
8-PACKAGES = [ "neutron-plugin-contrail", "contrail-config-openstack" ]
9+PACKAGES = [ "neutron-plugin-contrail" ]
10
11 hooks = Hooks()
12 config = config()
13
14=== modified file 'hooks/neutron_api_contrail_utils.py'
15--- hooks/neutron_api_contrail_utils.py 2016-05-07 02:33:48 +0000
16+++ hooks/neutron_api_contrail_utils.py 2016-09-05 15:41:40 +0000
17@@ -20,7 +20,7 @@
18 except CalledProcessError:
19 return None
20
21-CONTRAIL_VERSION = dpkg_version("contrail-config-openstack")
22+CONTRAIL_VERSION = dpkg_version("python-contrail")
23 OPENSTACK_VERSION = dpkg_version("neutron-server")
24
25 def contrail_api_ctx():

Subscribers

People subscribed via source and target branches