Unable to prevent heat from logging to stdout without all heat proccesses logging to the same file

Bug #1226287 reported by Cody A.W. Somerville
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Heat
Fix Released
Medium
Cody A.W. Somerville
oslo-incubator
Fix Released
Medium
Davanum Srinivas (DIMS)

Bug Description

If log_file is not set, oslo will output logs to stdout even if log_dir is set. If log_file is set, all heat proccesses will log to the same file.

Changed in heat:
status: New → Invalid
summary: - Unable to configure heat processes to log to different files
+ Unable to prevent heat from logging to stdout without all heat
+ proccesses logging to the same file
description: updated
Changed in heat:
milestone: none → havana-rc1
Revision history for this message
Steven Hardy (shardy) wrote :

Hmm, is this invalid as indicated in the status? Or valid as indicated by the rc1 assignment?

Revision history for this message
Cody A.W. Somerville (cody-somerville) wrote :

Oops. Yes, the bug is valid.

Updated bug status.

Changed in heat:
status: Invalid → New
Steven Hardy (shardy)
Changed in heat:
milestone: havana-rc1 → icehouse-1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to oslo-incubator (master)

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

Changed in oslo:
assignee: nobody → Davanum Srinivas (DIMS) (dims-v)
status: New → In Progress
Mark McLoughlin (markmc)
Changed in oslo:
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to oslo-incubator (master)

Reviewed: https://review.openstack.org/54730
Committed: http://github.com/openstack/oslo-incubator/commit/a6f40aca82632313ceec6732e51110ef536adc33
Submitter: Jenkins
Branch: master

commit a6f40aca82632313ceec6732e51110ef536adc33
Author: Davanum Srinivas <email address hidden>
Date: Thu Oct 31 08:57:36 2013 -0400

    Don't log to stdout when log_dir is set

    In addition to checking if log_file is set, we should check
    for log_dir as well. Previously, If log_file is not set, oslo
    will output logs to stdout even if log_dir is set. The only way
    for example the Heat folks avoided logging to stdout was to
    set log_file, but then all the heat processes ended up logging
    to the same file. So a check for both log_dir and log_file
    should avoid any output to stdout. Please note that use_stderr
    should be set to false as well to avoid any output to console

    Change-Id: Ie2886da679daedea0197ee3b3963ebedb8f11a0b
    Closes-Bug: #1226287

Changed in oslo:
status: In Progress → Fix Committed
Changed in heat:
assignee: nobody → Cody A.W. Somerville (cody-somerville)
status: New → Triaged
importance: Undecided → Medium
Changed in heat:
milestone: icehouse-1 → icehouse-2
Thierry Carrez (ttx)
Changed in oslo:
milestone: none → icehouse-1
status: Fix Committed → Fix Released
Revision history for this message
Steve Baker (steve-stevebaker) wrote :

Is there still a change required to heat for this?

Changed in heat:
milestone: icehouse-2 → icehouse-3
Thierry Carrez (ttx)
Changed in heat:
milestone: icehouse-3 → icehouse-rc1
Revision history for this message
Steve Baker (steve-stevebaker) wrote :

Cody, is this still an issue? Are their heat changes required to use the oslo changes?

Changed in heat:
milestone: icehouse-rc1 → next
Thierry Carrez (ttx)
Changed in oslo:
milestone: icehouse-1 → 2014.1
Revision history for this message
Angus Salkeld (asalkeld) wrote :
Changed in heat:
status: Triaged → Fix Committed
milestone: next → juno-rc1
Thierry Carrez (ttx)
Changed in heat:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in heat:
milestone: juno-rc1 → 2014.2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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