datalust / seq-forwarder

Local collection and reliable forwarding of log data to Seq
Apache License 2.0
57 stars 16 forks source link

Changing install drive causes installation failure #23

Closed danjohnso closed 7 years ago

danjohnso commented 7 years ago

When changing the installation path to D: drive (in my case), creating files runs ok, but the last step fails and aborts the installation. Looks like the last step does not respect the install path specified and is falling back to the default path on C:

The transacted install has completed. Service installed successfully. [10:44:25 FTL] Could not update config System.IO.DirectoryNotFoundException: Could not find a part of the path 'C:\ProgramData\Seq\Forwarder\SeqForwarder.json'. at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath) at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, Int32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions options, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost) at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost) at System.IO.StreamReader..ctor(String path, Encoding encoding, Boolean detectEncodingFromByteOrderMarks, Int32 bufferSize, Boolean checkHost) at System.IO.File.InternalReadAllText(String path, Encoding encoding, Boolean checkHost) at Seq.Forwarder.Config.SeqForwarderConfig.Read(String filename) at Seq.Forwarder.Cli.Commands.ConfigCommand.Run(TextWriter cout)

nblumhardt commented 7 years ago

Thanks for the report, Dan. I think I've tracked this down - putting together a fix now.

danjohnso commented 7 years ago

Thanks for the quick fix! Finishing up a POC to replace our internal logging tools with serilog/seq and this piece is huge for us being able to use do it