Savanna manifests do not configure syslogging

Bug #1261479 reported by Vladimir Kuklin
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Medium
Dmitry Ilyin

Bug Description

We need to configure syslogging for Savanna in the same way as we did for all the other services.

Revision history for this message
Vladimir Kuklin (vkuklin) wrote :

164 if $use_syslog and !$debug =~ /(?i)(true|yes)/
165 {
166
167 nova_config
168 {
169 'DEFAULT/log_config': value => "/etc/nova/logging.conf";
170 'DEFAULT/log_file': ensure=> absent;
171 'DEFAULT/logfile': ensure=> absent;
172 'DEFAULT/use_syslog': value => true;
173 'DEFAULT/use_stderr': ensure=> absent;
174 'DEFAULT/syslog_log_facility': value => $syslog_log_facility;
175 }
176
177 file {"nova-logging.conf":
178 content => template('nova/logging.conf.erb'),
179 path => "/etc/nova/logging.conf",
180 require => File[$logdir],
181 }
182 }
183 else {
184 nova_config {
185 'DEFAULT/log_config': ensure=> absent;
186 'DEFAULT/use_syslog': ensure=> absent;
187 'DEFAULT/syslog_log_facility': ensure=> absent;
188 'DEFAULT/use_stderr': ensure=> absent;
189 'DEFAULT/logdir': value=> $logdir;
190 'DEFAULT/logging_context_format_string':
191 value => '%(asctime)s %(levelname)s %(name)s [%(request_id)s %(user_id)s %(project_id)s] %(instance)s %(message)s';
192 'DEFAULT/logging_default_format_string':
193 value => '%(asctime)s %(levelname)s %(name)s [-] %(instance)s %(message)s';
194 }
195 # might be used for stdout logging instead, if configured
196 file {"nova-logging.conf":
197 content => template('nova/logging.conf-nosyslog.erb'),
198 path => "/etc/nova/logging.conf",
199 require => File[$logdir],
200 }
201 }

summary: - Savanna manifests does not configure syslogging
+ Savanna manifests do not configure syslogging
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

Fix proposed to branch: master
Review: https://review.openstack.org/62926

Changed in fuel:
assignee: Ruslan Kamaldinov (ruhe) → Ivan Berezovskiy (iberezovskiy)
status: New → In Progress
Mike Scherbakov (mihgen)
Changed in fuel:
importance: High → Medium
Changed in fuel:
milestone: 4.0 → 4.1
Revision history for this message
Mike Scherbakov (mihgen) wrote :

What's the status, folks? We have today-only before a code freeze call. Please don't keep "in progress" if you don't work on task.

Changed in fuel:
status: In Progress → Triaged
Changed in fuel:
status: Triaged → In Progress
Changed in fuel:
assignee: Ivan Berezovskiy (iberezovskiy) → Dmitry Ilyin (idv1985)
Dmitry Ilyin (idv1985)
Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/62926
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=42686698bc5948c31d27ee4ec08da3c4f251975b
Submitter: Jenkins
Branch: master

commit 42686698bc5948c31d27ee4ec08da3c4f251975b
Author: iberezovskiy <email address hidden>
Date: Wed Dec 18 20:14:37 2013 +0400

    Fix Savanna manifests to configure syslogging

    Closes-Bug #1261479

    Change-Id: Ie41904e7c0642dbfb62698b500735a2db43a3000

tags: added: in progress
Revision history for this message
Anastasia Palkina (apalkina) wrote :

Verified on ISO #235
"build_id": "2014-03-05_07-31-01",
"mirantis": "yes",
"build_number": "235",
"nailgun_sha": "f58aad317829112913f364347b14f1f0518ad371",
"ostf_sha": "dc54d99ddff2f497b131ad1a42362515f2a61afa",
"fuelmain_sha": "16637e2ea0ae6fe9a773aceb9d76c6e3a75f6c3b",
"astute_sha": "f15f5615249c59c826ea05d26707f062c88db32a",
"release": "4.1",
"fuellib_sha": "73313007c0914e602246ea41fa5e8ca2dfead9f8"

tags: removed: in progress
Changed in fuel:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.