Skip to content

Commit

Permalink
Fix a typo in mellanox_buffer_migrator (sonic-net#1090)
Browse files Browse the repository at this point in the history
This type causes the db_migrator to update the buffer profiles even if they don't match the default

Signed-off-by: Stephen Sun <stephens@nvidia.com>
  • Loading branch information
stephenxs authored Sep 3, 2020
1 parent 8152cec commit aeaada0
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion scripts/mellanox_buffer_migrator.py
Original file line number Diff line number Diff line change
Expand Up @@ -354,7 +354,7 @@ def mlnx_migrate_buffer_profile(self, old_version, new_version):
return True

for name, profile in buffer_profile_old_configure.iteritems():
if name in buffer_profile_conf.keys() and profile == buffer_profile_old_configure[name]:
if name in buffer_profile_conf.keys() and profile == buffer_profile_conf[name]:
continue
# return if any default profile isn't in cofiguration
log.log_notice("Default profile {} isn't in database or doesn't match default value".format(name))
Expand All @@ -363,3 +363,5 @@ def mlnx_migrate_buffer_profile(self, old_version, new_version):
for name, profile in buffer_profile_new_configure.iteritems():
log.log_info("Successfully migrate profile {}".format(name))
self.configDB.set_entry('BUFFER_PROFILE', name, profile)

return True

0 comments on commit aeaada0

Please sign in to comment.