Skip to content

crowbar: Do not use @logger in forks where it's not available anymore#369

Open
vuntz wants to merge 1 commit intocrowbar:masterfrom
vuntz:no-logger-no-error
Open

crowbar: Do not use @logger in forks where it's not available anymore#369
vuntz wants to merge 1 commit intocrowbar:masterfrom
vuntz:no-logger-no-error

Conversation

@vuntz
Copy link
Copy Markdown
Member

@vuntz vuntz commented Feb 22, 2016

This is what is causing these lines in the logs:
log writing failed. closed stream

This is what is causing these lines in the logs:
  log writing failed. closed stream
end
@logger.debug("wait_for_chef_clients: Waiting for already running chef-clients on #{node_name}.")
if use_logger
@logger.debug("wait_for_chef_clients: Waiting for already running chef-clients on #{node_name}.")
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Metrics/LineLength: Line is too long. 103/100

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

7 participants