Merge lp:~taoliu-7/charms/trusty/contrail-analytics/node-manager into lp:~sdn-charmers/charms/trusty/contrail-analytics/trunk

Proposed by Tony Liu
Status: Merged
Merged at revision: 11
Proposed branch: lp:~taoliu-7/charms/trusty/contrail-analytics/node-manager
Merge into: lp:~sdn-charmers/charms/trusty/contrail-analytics/trunk
Diff against target: 146 lines (+70/-8)
5 files modified
hooks/contrail_analytics_hooks.py (+12/-7)
hooks/contrail_analytics_utils.py (+27/-1)
templates/contrail-analytics-nodemgr (+7/-0)
templates/contrail-analytics-nodemgr.conf (+12/-0)
templates/contrail-nodemgr-analytics.ini (+12/-0)
To merge this branch: bzr merge lp:~taoliu-7/charms/trusty/contrail-analytics/node-manager
Reviewer Review Type Date Requested Status
Robert Ayres (community) Approve
Review via email: mp+272161@code.launchpad.net
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/contrail_analytics_hooks.py'
2--- hooks/contrail_analytics_hooks.py 2015-07-21 19:09:14 +0000
3+++ hooks/contrail_analytics_hooks.py 2015-09-23 18:39:18 +0000
4@@ -34,10 +34,13 @@
5 fix_services,
6 write_analytics_api_config,
7 write_collector_config,
8- write_query_engine_config
9+ write_query_engine_config,
10+ write_nodemgr_config,
11+ install_nodemgr_config
12 )
13
14-PACKAGES = [ "contrail-analytics", "contrail-utils", "python-jinja2" ]
15+PACKAGES = [ "contrail-analytics", "contrail-utils", "contrail-nodemgr",
16+ "python-jinja2" ]
17
18 hooks = Hooks()
19 config = config()
20@@ -74,11 +77,13 @@
21 @hooks.hook("contrail-discovery-relation-broken")
22 @restart_on_change({"/etc/contrail/contrail-collector.conf": ["contrail-collector"],
23 "/etc/contrail/contrail-query-engine.conf": ["contrail-query-engine"],
24- "/etc/contrail/contrail-analytics-api.conf": ["contrail-analytics-api"]})
25+ "/etc/contrail/contrail-analytics-api.conf": ["contrail-analytics-api"],
26+ "/etc/contrail/contrail-analytics-nodemgr.conf": ["contrail-analytics-nodemgr"]})
27 def contrail_discovery_relation():
28 write_collector_config()
29 write_query_engine_config()
30 write_analytics_api_config()
31+ write_nodemgr_config()
32
33 @hooks.hook("http-services-relation-joined")
34 def http_services_joined():
35@@ -94,9 +99,10 @@
36 @hooks.hook()
37 def install():
38 configure_installation_source(config["openstack-origin"])
39- configure_sources(True, "install-sources")
40+ configure_sources(True, "install-sources", "install-keys")
41 apt_upgrade(fatal=True, dist=True)
42 apt_install(PACKAGES, fatal=True)
43+ install_nodemgr_config()
44 fix_permissions()
45 fix_services()
46
47@@ -111,9 +117,8 @@
48 write_collector_config()
49 write_query_engine_config()
50 write_analytics_api_config()
51- for service in [ "contrail-collector", "contrail-query-engine",
52- "contrail-analytics-api" ]:
53- service_restart(service)
54+ write_nodemgr_config()
55+ service_restart("supervisor-analytics")
56
57 if __name__ == "__main__":
58 main()
59
60=== modified file 'hooks/contrail_analytics_utils.py'
61--- hooks/contrail_analytics_utils.py 2015-07-23 20:44:36 +0000
62+++ hooks/contrail_analytics_utils.py 2015-09-23 18:39:18 +0000
63@@ -1,6 +1,9 @@
64 import os
65 from socket import gethostbyname
66-from subprocess import check_output
67+from subprocess import (
68+ check_output,
69+ check_call
70+)
71
72 from charmhelpers.core.hookenv import (
73 related_units,
74@@ -66,3 +69,26 @@
75 ctx.update(discovery_ctx())
76 render("contrail-query-engine.conf",
77 "/etc/contrail/contrail-query-engine.conf", ctx)
78+
79+def install_nodemgr_config():
80+ ctx = {}
81+ render("contrail-nodemgr-analytics.ini",
82+ "/etc/contrail/supervisord_analytics_files/contrail-nodemgr-analytics.ini",
83+ ctx, "contrail", "contrail", 0644)
84+ render("contrail-analytics-nodemgr",
85+ "/etc/init.d/contrail-analytics-nodemgr",
86+ ctx, "root", "root", 0755)
87+ check_call(["supervisorctl",
88+ "-s", "unix:///tmp/supervisord_analytics.sock",
89+ "reread"])
90+ check_call(["supervisorctl",
91+ "-s", "unix:///tmp/supervisord_analytics.sock",
92+ "add", "contrail-analytics-nodemgr"])
93+
94+def write_nodemgr_config():
95+ ctx = {}
96+ ctx.update(discovery_ctx())
97+ render("contrail-analytics-nodemgr.conf",
98+ "/etc/contrail/contrail-analytics-nodemgr.conf",
99+ ctx, "contrail", "contrail", 0644)
100+
101
102=== added file 'templates/contrail-analytics-nodemgr'
103--- templates/contrail-analytics-nodemgr 1970-01-01 00:00:00 +0000
104+++ templates/contrail-analytics-nodemgr 2015-09-23 18:39:18 +0000
105@@ -0,0 +1,7 @@
106+#!/usr/bin/env bash
107+
108+# chkconfig: 2345 99 01
109+# description: Juniper Network Analytics Node Manager
110+
111+supervisorctl -s unix:///tmp/supervisord_analytics.sock ${1} `basename ${0}`
112+
113
114=== added file 'templates/contrail-analytics-nodemgr.conf'
115--- templates/contrail-analytics-nodemgr.conf 1970-01-01 00:00:00 +0000
116+++ templates/contrail-analytics-nodemgr.conf 2015-09-23 18:39:18 +0000
117@@ -0,0 +1,12 @@
118+###############################################################################
119+# [ WARNING ]
120+# Configuration file maintained by Juju. Local changes may be overwritten.
121+###############################################################################
122+
123+[DISCOVERY]
124+server = {{ disc_server_ip }}
125+port = 5998
126+
127+[COLLECTOR]
128+#server_list = ip1:port1 ip2:port2
129+
130
131=== added file 'templates/contrail-nodemgr-analytics.ini'
132--- templates/contrail-nodemgr-analytics.ini 1970-01-01 00:00:00 +0000
133+++ templates/contrail-nodemgr-analytics.ini 2015-09-23 18:39:18 +0000
134@@ -0,0 +1,12 @@
135+###############################################################################
136+# [ WARNING ]
137+# Configuration file maintained by Juju. Local changes may be overwritten.
138+###############################################################################
139+
140+[eventlistener:contrail-analytics-nodemgr]
141+command=/bin/bash -c "exec /usr/bin/contrail-nodemgr"
142+events=PROCESS_COMMUNICATION,PROCESS_STATE,TICK_60
143+buffer_size=10000
144+stdout_logfile=/var/log/contrail/contrail-analytics-nodemgr-stdout.log
145+stderr_logfile=/var/log/contrail/contrail-analytics-nodemgr-stderr.log
146+

Subscribers

People subscribed via source and target branches

to all changes: