Bug 903504 - log level of creating networks/bridges events on host should be info
Summary: log level of creating networks/bridges events on host should be info
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Red Hat OpenStack
Classification: Red Hat
Component: openstack-nova
Version: 2.0 (Folsom)
Hardware: Unspecified
OS: Unspecified
unspecified
low
Target Milestone: beta
: 4.0
Assignee: David Ripton
QA Contact: Ofer Blaut
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-01-24 08:25 UTC by Ofer Blaut
Modified: 2019-09-09 13:38 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-09-04 14:11:02 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Launchpad 1214036 0 None None None Never

Description Ofer Blaut 2013-01-24 08:25:48 UTC
Description of problem:

log level of creating networks/bridges events on host should be info 

currently events of creating vlans/bridges ... by nova-compute are in debug level

it should be changes into info level  

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1. change nova.conf to debug=true
2. create new network and launch VM ( to create the bridge )
3. check brctl / ip addr commands are viewed under /var/log/nova/compute.log 
with Info level 
  
Actual results:


Expected results:


Additional info:

Comment 2 David Ripton 2013-02-28 16:54:41 UTC
This is a feature request, and we're in feature freeze upstream, and Perry said not to do this downstream-only and create divergence, so this needs to wait until upstream reopens for feature changes.  So 3.0.

Comment 3 David Ripton 2013-05-16 13:13:31 UTC
4.0 because Grizzly is feature-frozen.  I'll do this upstream for Havana.

Comment 4 David Ripton 2013-08-08 17:24:01 UTC
The oslo patch is https://review.openstack.org/#/c/40872/

Once that gets in, I can drop the nova patch that depends on it.

(I can't just change LOG.debug to LOG.info because the logging is actually inside a library that comes from oslo.)

Comment 5 David Ripton 2013-08-15 14:44:15 UTC
The oslo patch got in.  2548344ec64

I need to merge it into nova, and then I can do the actual logging level patch.

Comment 6 David Ripton 2013-08-19 17:15:27 UTC
The oslo patch was merged to nova in 599af3570a

The patch to use INFO rather than debug for brctl and significant 'ip addr' commands is https://review.openstack.org/#/c/42689/

I also created a launchpad bug for this, which is in the External Tracker field.

Comment 7 David Ripton 2013-08-29 14:10:56 UTC
This got a -1 upstream as creating too much noise in the logs.  Looks unlikely to get in, unless someone makes a strong evidence-based argument for it in Gerrit.

Comment 8 David Ripton 2013-09-04 14:11:02 UTC
This got a -2 upstream.  Not because of a problem with the code, but because two core reviewers think it's too much noise in the logs.  So you have to continue to set the loglevel to DEBUG manually in nova.conf.  Setting to WONTFIX.


Note You need to log in before you can comment on or make changes to this bug.