Skip to content

[Mellanox] Preserve explicit buffer_model=dynamic in buffer migrator #4609

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
william8545 wants to merge 1 commit into
base: master
Choose a base branch
from
+5 −2
Open

[Mellanox] Preserve explicit buffer_model=dynamic in buffer migrator #4609

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 5 additions & 2 deletions scripts/mellanox_buffer_migrator.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -821,9 +821,12 @@ def mlnx_flush_new_buffer_configuration(self):
if not self.ready:
return True

if not self.is_buffer_config_default and not self.is_buffer_config_empty or self.is_default_traditional_model:
metadata = self.configDB.get_entry('DEVICE_METADATA', 'localhost')
already_dynamic = metadata.get('buffer_model') == 'dynamic' and not self.is_default_traditional_model

if (not self.is_buffer_config_default and not self.is_buffer_config_empty
or self.is_default_traditional_model) and not already_dynamic:
log.log_notice("No item pending to be updated")
metadata = self.configDB.get_entry('DEVICE_METADATA', 'localhost')
metadata['buffer_model'] = 'traditional'
self.configDB.set_entry('DEVICE_METADATA', 'localhost', metadata)
log.log_notice("Set buffer_model as traditional")
Expand Down
Loading