Fix review comments:

- Fix alignment error - Update DEVICE_METADATA|localhost.buffer_model only if it is changed Signed-off-by: Stephen Sun <stephens@nvidia.com>
sonic-net · Dec 8, 2020 · aee20c1 · aee20c1
1 parent a63b850
commit aee20c1
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/config/main.py b/config/main.py
@@ -1630,8 +1630,8 @@ def _update_buffer_calculation_model(config_db, model):
  device_metadata = config_db.get_entry('DEVICE_METADATA', 'localhost')
  if device_metadata.get('buffer_model') != model:
  buffer_model_changed = True
- device_metadata['buffer_model'] = model
- config_db.set_entry('DEVICE_METADATA', 'localhost', device_metadata)
+  device_metadata['buffer_model'] = model
+  config_db.set_entry('DEVICE_METADATA', 'localhost', device_metadata)
  return buffer_model_changed
 
 @qos.command('reload')

diff --git a/tests/mock_tables/config_db.json b/tests/mock_tables/config_db.json
@@ -1365,7 +1365,7 @@
  "BUFFER_PROFILE|alpha_profile": {
  "dynamic_th": "0",
  "pool": "[BUFFER_POOL|ingress_lossless_pool]",
- "headroom_type": "dynamic"
+ "headroom_type": "dynamic"
  },
  "BUFFER_PG|Ethernet0|3-4": {
  "profile": "NULL"

diff --git a/tests/mock_tables/state_db.json b/tests/mock_tables/state_db.json
@@ -349,9 +349,9 @@
  "type": "ingress"
  },
  "BUFFER_PROFILE_TABLE|ingress_lossy_profile": {
- "dynamic_th": "3",
- "pool": "[BUFFER_POOL_TABLE|ingress_lossy_pool]",
- "size": "0"
+ "dynamic_th": "3",
+ "pool": "[BUFFER_POOL_TABLE|ingress_lossy_pool]",
+ "size": "0"
  },
  "BUFFER_PROFILE_TABLE|headroom_profile": {
  "dynamic_th": "0",