kemko/nomad
1
0
Fork 0
mirror of https://github.com/kemko/nomad.git synced 2026-01-01 16:05:42 +03:00
Code Issues Packages Projects Releases Wiki Activity

agent: fix panic when logging about protocol version config use. (#12962)

Browse Source 
The log line comes before the agent logger has been setup,
therefore we need to use the UI logging to avoid panic.
This commit is contained in:
James Rasell
2022-05-13 09:28:43 +02:00
committed by GitHub
parent e611b099d3
commit bab219a8ba
2 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
.changelog/12962.txt Normal file
View File

@@ -0,0 +1,3 @@
```release-note:bug
agent: fixed a panic on startup when the `server.protocol_version` config parameter was set
```

2
command/agent/command.go
View File

@@ -436,7 +436,7 @@ func (c *Command) IsValidConfig(config, cmdConfig *Config) bool {
// ProtocolVersion has never been used. Warn if it is set as someone // ProtocolVersion has never been used. Warn if it is set as someone
// has probably made a mistake. // has probably made a mistake.
if config.Server.ProtocolVersion != 0 { if config.Server.ProtocolVersion != 0 {
c.agent.logger.Warn("Please remove deprecated protocol_version field from config.") c.Ui.Warn("Please remove deprecated protocol_version field from config.")
} }
return true return true