diff --git a/.changelog/14615.txt b/.changelog/14615.txt new file mode 100644 index 000000000000..6d8bb481f8b8 --- /dev/null +++ b/.changelog/14615.txt @@ -0,0 +1,3 @@ +```release-note:bug +client: Fixed a bug where network fingerprinters were not reloaded when the client configuration was reloaded with SIGHUP +``` diff --git a/client/client.go b/client/client.go index cfe8fcd6c007..0493c2c2b885 100644 --- a/client/client.go +++ b/client/client.go @@ -297,6 +297,9 @@ type Client struct { endpoints rpcEndpoints streamingRpcs *structs.StreamingRpcRegistry + // fingerprintManager is the FingerprintManager registered by the client + fingerprintManager *FingerprintManager + // pluginManagers is the set of PluginManagers registered by the client pluginManagers *pluginmanager.PluginGroup @@ -440,14 +443,14 @@ func NewClient(cfg *config.Config, consulCatalog consul.CatalogAPI, consulProxie return nil, fmt.Errorf("node setup failed: %v", err) } - fingerprintManager := NewFingerprintManager( + c.fingerprintManager = NewFingerprintManager( cfg.PluginSingletonLoader, c.GetConfig, cfg.Node, c.shutdownCh, c.updateNodeFromFingerprint, c.logger) c.pluginManagers = pluginmanager.New(c.logger) // Fingerprint the node and scan for drivers - if err := fingerprintManager.Run(); err != nil { + if err := c.fingerprintManager.Run(); err != nil { return nil, fmt.Errorf("fingerprinting failed: %v", err) } @@ -742,6 +745,8 @@ func (c *Client) Reload(newConfig *config.Config) error { return c.reloadTLSConnections(newConfig.TLSConfig) } + c.fingerprintManager.Reload() + return nil }