8000 Stopping the logfile spam by piping STDERR to /dev/null by madpilot · Pull Request #8081 · home-assistant/core · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Stopping the logfile spam by piping STDERR to /dev/null #8081

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 18, 2017
Merged

Stopping the logfile spam by piping STDERR to /dev/null #8081

merged 1 commit into from
Jun 18, 2017

Conversation

madpilot
Copy link
Contributor

When pinging computers using the Wake On LAN switch, the log files get spammed with

ping: cannot resolve computer.local: Unknown host

which makes sense, if the computer is turned off. However, this isn't really an error.

This patch pipes STDERR to /dev/null, cleaning up the log a little.

@homeassistant
Copy link
Contributor

Hi @madpilot,

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

@mention-bot
Copy link

@madpilot, thanks for your PR! By analyzing the history of the files in this pull request, we identified @fabaff, @titilambert and @Chris-V to be potential reviewers.

Copy link
Member
@fabaff fabaff left a comment

Choose a reason for hiding this comment

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

Looks good to me 🐦

@fabaff fabaff merged commit ef63cfe into home-assistant:dev Jun 18, 2017
@balloob balloob mentioned this pull request Jul 1, 2017
@home-assistant home-assistant locked and limited conversation to collaborators Oct 20, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0