The Puppet Labs Issue Tracker has Moved: https://tickets.puppetlabs.com

Bug #13584

in daemon mode, master swallows errors during startup

Added by Chris Price over 2 years ago. Updated about 2 years ago.

Status:ClosedStart date:04/02/2012
Priority:UrgentDue date:
Assignee:-% Done:

0%

Category:logging
Target version:3.0.0
Affected Puppet version: Branch:https://github.com/puppetlabs/puppet/pull/621
Keywords:

We've Moved!

Ticket tracking is now hosted in JIRA: https://tickets.puppetlabs.com

This issue is currently not available for export. If you are experiencing the issue described below, please file a new ticket in JIRA. Once a new ticket has been created, please add a link to it that points back to this Redmine ticket.


Description

Because of the way that logging is handled / initialized during master startup, failures that occur before the logging is completely initialized seem to just disappear into the void. This makes it very tricky to diagnose problems that happen during startup, e.g. during acceptance testing on the CI nodes.

Would be extremely useful to find a way to get the master to keep STDOUT/STDERR alive until logging has been successfully initialized, or register a “bootstrapping” log destination for use during startup so that these errors don’t just disappear.


Related issues

Blocks Puppet - Bug #13588: log dir is not permissioned properly Closed 04/02/2012

History

#1 Updated by Chris Price over 2 years ago

After further investigation it looks like Puppet::Util#exit_on_fail is probably the main culprit of this type of issue. It catches exceptions and calls “log_exception”, with no way of knowing whether or not our logging has been initialized yet.

#2 Updated by Chris Price over 2 years ago

  • Status changed from Accepted to In Topic Branch Pending Review
  • Branch set to https://github.com/puppetlabs/puppet/pull/619

#3 Updated by Chris Price over 2 years ago

  • Status changed from In Topic Branch Pending Review to Code Insufficient

#4 Updated by Chris Price over 2 years ago

  • Status changed from Code Insufficient to In Topic Branch Pending Review
  • Branch changed from https://github.com/puppetlabs/puppet/pull/619 to https://github.com/puppetlabs/puppet/pull/621

#5 Updated by Chris Price over 2 years ago

  • Assignee deleted (Chris Price)

#6 Updated by Andrew Parker about 2 years ago

  • Target version changed from 3.x to 3.0.0

Merged in commit b19931a

#7 Updated by Andrew Parker about 2 years ago

  • Status changed from In Topic Branch Pending Review to Merged - Pending Release

#8 Updated by Moses Mendoza about 2 years ago

  • Status changed from Merged - Pending Release to Closed

released in 3.0.0rc1

Also available in: Atom PDF