Merge lp:~ivoks/charms/precise/nova-cloud-controller/syslog into lp:~charmers/charms/precise/nova-cloud-controller/trunk

Proposed by Ante Karamatić
Status: Superseded
Proposed branch: lp:~ivoks/charms/precise/nova-cloud-controller/syslog
Merge into: lp:~charmers/charms/precise/nova-cloud-controller/trunk
Diff against target: 117 lines (+19/-1)
10 files modified
config.yaml (+6/-0)
hooks/nova_cc_context.py (+1/-0)
revision (+1/-1)
templates/essex/nova.conf (+1/-0)
templates/folsom/nova.conf (+1/-0)
templates/folsom/ovs_quantum_plugin.ini (+3/-0)
templates/folsom/quantum.conf (+1/-0)
templates/havana/neutron.conf (+1/-0)
templates/havana/nvp.ini (+1/-0)
templates/havana/ovs_neutron_plugin.ini (+3/-0)
To merge this branch: bzr merge lp:~ivoks/charms/precise/nova-cloud-controller/syslog
Reviewer Review Type Date Requested Status
James Page Needs Fixing
Marco Ceppi (community) Abstain
Review via email: mp+203347@code.launchpad.net

Description of the change

Logging to syslog is one of common requirements in enterprise setup.

To post a comment you must log in.
Revision history for this message
Marco Ceppi (marcoceppi) wrote :

Deferring to openstack-charmers

review: Abstain
Revision history for this message
James Page (james-page) wrote :

I discuss with Yolanda I we agreed a generic SyslogConfig context should be used across all charms; makes it easier to expand support/update later on.

review: Needs Fixing

Unmerged revisions

59. By Ante Karamatić

Add option of logging to syslog

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'config.yaml'
2--- config.yaml 2013-11-08 05:41:39 +0000
3+++ config.yaml 2014-01-27 15:14:06 +0000
4@@ -126,6 +126,12 @@
5 ssl_key:
6 type: string
7 description: SSL key to use with certificate specified as ssl_cert.
8+ use-syslog:
9+ type: boolean
10+ default: False
11+ description: |
12+ By default, all services will log into their corresponding log files.
13+ Setting this to True will force all services to log to the syslog.
14 # Neutron NVP Plugin configuration
15 nvp-controllers:
16 type: string
17
18=== modified file 'hooks/nova_cc_context.py'
19--- hooks/nova_cc_context.py 2013-10-16 10:47:19 +0000
20+++ hooks/nova_cc_context.py 2014-01-27 15:14:06 +0000
21@@ -139,6 +139,7 @@
22 def __call__(self):
23 ctxt = super(NeutronCCContext, self).__call__()
24 ctxt['external_network'] = config('neutron-external-network')
25+ ctxt['use_syslog'] = config('use-syslog')
26 if 'nvp' in [config('quantum-plugin'), config('neutron-plugin')]:
27 _config = config()
28 for k, v in _config.iteritems():
29
30=== modified file 'revision'
31--- revision 2013-11-08 05:41:39 +0000
32+++ revision 2014-01-27 15:14:06 +0000
33@@ -1,1 +1,1 @@
34-311
35+312
36
37=== modified file 'templates/essex/nova.conf'
38--- templates/essex/nova.conf 2013-08-19 21:13:45 +0000
39+++ templates/essex/nova.conf 2014-01-27 15:14:06 +0000
40@@ -14,6 +14,7 @@
41 --connection_type=libvirt
42 --root_helper=sudo nova-rootwrap
43 --verbose
44+--use_syslog={{ use_syslog }}
45 --ec2_private_dns_show_ip
46 {% if database_host -%}
47 --sql_connection=mysql://{{ database_user }}:{{ database_password }}@{{ database_host }}/{{ database }}
48
49=== modified file 'templates/folsom/nova.conf'
50--- templates/folsom/nova.conf 2013-10-28 19:03:14 +0000
51+++ templates/folsom/nova.conf 2014-01-27 15:14:06 +0000
52@@ -14,6 +14,7 @@
53 connection_type=libvirt
54 root_helper=sudo nova-rootwrap /etc/nova/rootwrap.conf
55 verbose=True
56+use_syslog={{ use_syslog }}
57 ec2_private_dns_show_ip=True
58 api_paste_config=/etc/nova/api-paste.ini
59 volumes_path=/var/lib/nova/volumes
60
61=== modified file 'templates/folsom/ovs_quantum_plugin.ini'
62--- templates/folsom/ovs_quantum_plugin.ini 2013-08-19 21:13:45 +0000
63+++ templates/folsom/ovs_quantum_plugin.ini 2014-01-27 15:14:06 +0000
64@@ -3,6 +3,9 @@
65 # [ WARNING ]
66 # Configuration file maintained by Juju. Local changes may be overwritten.
67 ###############################################################################
68+[DEFAULT]
69+use_syslog = {{ use_syslog }}
70+
71 [OVS]
72 tunnel_id_ranges = 1:1000
73 tenant_network_type = gre
74
75=== modified file 'templates/folsom/quantum.conf'
76--- templates/folsom/quantum.conf 2013-08-19 21:13:45 +0000
77+++ templates/folsom/quantum.conf 2014-01-27 15:14:06 +0000
78@@ -15,6 +15,7 @@
79 {% if core_plugin -%}
80 core_plugin = {{ core_plugin }}
81 {% endif -%}
82+use_syslog = {{ use_syslog }}
83 api_paste_config = /etc/quantum/api-paste.ini
84 auth_strategy = keystone
85 control_exchange = quantum
86
87=== modified file 'templates/havana/neutron.conf'
88--- templates/havana/neutron.conf 2013-12-12 20:10:37 +0000
89+++ templates/havana/neutron.conf 2014-01-27 15:14:06 +0000
90@@ -8,6 +8,7 @@
91 bind_host = 0.0.0.0
92 auth_strategy = keystone
93 notification_driver = neutron.openstack.common.notifier.rpc_notifier
94+use_syslog = {{ use_syslog }}
95 {% if neutron_bind_port -%}
96 bind_port = {{ neutron_bind_port }}
97 {% else -%}
98
99=== modified file 'templates/havana/nvp.ini'
100--- templates/havana/nvp.ini 2013-10-16 11:14:35 +0000
101+++ templates/havana/nvp.ini 2014-01-27 15:14:06 +0000
102@@ -9,3 +9,4 @@
103 nvp_controllers = {{ nvp_controllers }}
104 default_tz_uuid = {{ nvp_tz_uuid }}
105 default_l3_gw_service_uuid = {{ nvp_l3_uuid }}
106+use_syslog = {{ use_syslog }}
107
108=== modified file 'templates/havana/ovs_neutron_plugin.ini'
109--- templates/havana/ovs_neutron_plugin.ini 2013-09-25 10:49:12 +0000
110+++ templates/havana/ovs_neutron_plugin.ini 2014-01-27 15:14:06 +0000
111@@ -1,3 +1,6 @@
112+[DEFAULT]
113+use_syslog = {{ use_syslog }}
114+
115 [OVS]
116 tunnel_id_ranges = 1:1000
117 tenant_network_type = gre

Subscribers

People subscribed via source and target branches