matthew/sonic-buildimage
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

[hostcfgd] If feature state entry not in the cache, add a default state (#5777)

Browse Source 
Our use case is to register new features in runtime. The previous change which introduced the cache broke this capability and caused hostcfgd crash.

Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
This commit is contained in:
Stepan Blyshchak 2020-11-06 20:24:31 +02:00 committed by GitHub
parent ad555d9ffd
commit 9bc693ce6e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
files/image_config/hostcfgd/hostcfgd
View File

@ -368,6 +368,8 @@ class HostConfigDaemon:
syslog.syslog(syslog.LOG_WARNING, "Enable state of feature '{}' is None".format(feature_name)) syslog.syslog(syslog.LOG_WARNING, "Enable state of feature '{}' is None".format(feature_name))
return return
self.cached_feature_states.setdefault(feature_name, 'disabled')
# Enable/disable the container service if the feature state was changed from its previous state. # Enable/disable the container service if the feature state was changed from its previous state.
if self.cached_feature_states[feature_name] != state: if self.cached_feature_states[feature_name] != state:
self.cached_feature_states[feature_name] = state self.cached_feature_states[feature_name] = state