Skip to content

Windows: Write fatal error to event log if running as service#26725

Merged
vdemeester merged 1 commit intomoby:masterfrom
StefanScherer:log-error-to-event-log
Sep 20, 2016
Merged

Windows: Write fatal error to event log if running as service#26725
vdemeester merged 1 commit intomoby:masterfrom
StefanScherer:log-error-to-event-log

Conversation

@StefanScherer
Copy link
Copy Markdown
Contributor

- What I did

I've updated a TP5 machine to use the latest docker-1.13.0-dev.zip binaries. But I couldn't start the Windows Service docker. I found no useful information in the Event log viewer.

Only after starting the dockerd.exe manually I found a message on stdout that TP5 seems to be too old:

Error starting daemon: The docker daemon requires build 14393 or later of Windows Server 2016 or Windows 10

The message is swallowed if dockerd.exe is started as a service as discussed here. So I looked for a good place to log that error with logrus to the Event log.

- How I did it

- How to verify it

Install the binaries on TP5 and try to start the service and have a look at the Event log.

- Description for the changelog
Write fatal error to the Windows event log when the service could not be started.

- A picture of a cute animal (not mandatory but encouraged)

Not an animal, but the result of this PR:

bildschirmfoto 2016-09-19 um 23 01 02

Signed-off-by: Stefan Scherer [email protected]

@StefanScherer StefanScherer changed the title Write fatal error to event log if running as service Windows: Write fatal error to event log if running as service Sep 19, 2016
@lowenna
Copy link
Copy Markdown
Member

lowenna commented Sep 19, 2016

LGTM

@thaJeztah
Copy link
Copy Markdown
Member

LGTM

@vdemeester
Copy link
Copy Markdown
Member

win2lin failure unrelated…

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants