Skip to content

Commit 7474b14

Browse files
committed
[GCU] Fix missing backend in dry run (#2347)
What I did Fix sonic-net/sonic-buildimage#11912 How I did it Add the missing 'remove_backend_tables_from_config' in dry run. How to verify it Run unit test and manual test locally.
1 parent e40139f commit 7474b14

File tree

2 files changed

+5
-0
lines changed

2 files changed

+5
-0
lines changed

generic_config_updater/change_applier.py

+4
Original file line numberDiff line numberDiff line change
@@ -65,6 +65,10 @@ def apply(self, change):
6565
self.config_wrapper.apply_change_to_config_db(change)
6666

6767

68+
def remove_backend_tables_from_config(self, data):
69+
return data
70+
71+
6872
class ChangeApplier:
6973

7074
updater_conf = None

tests/generic_config_updater/change_applier_test.py

+1
Original file line numberDiff line numberDiff line change
@@ -281,6 +281,7 @@ def test_apply__calls_apply_change_to_config_db(self):
281281

282282
# Act
283283
applier.apply(change)
284+
applier.remove_backend_tables_from_config(change)
284285

285286
# Assert
286287
applier.config_wrapper.apply_change_to_config_db.assert_has_calls([call(change)])

0 commit comments

Comments
 (0)