From 6f86f92f2a9a78518c8fc78ed2cd63feb0cdc202 Mon Sep 17 00:00:00 2001 From: Sonic Build Admin Date: Thu, 19 Feb 2026 09:13:21 +0000 Subject: [PATCH] configlet/test_add_rack.py Add comparison ignore for extra entries added by generic patcher ### Description of PR Summary: When patching the DUT to remove t0 peers, the test fails the DB comparison due to extra entries being added to APP_DB. These entries only get added when generic patcher in SONiC (`config apply-patch`) is used to remove t0 from the DUT. Ideally this should be fixed in `config apply-patch` code. This PR is a workaround for the issue since the extra entries are harmless and does not get used. Fixes # (issue) ### Type of change - [ ] Bug fix - [ ] Testbed and Framework(new/improvement) - [ ] New Test case - [ ] Skipped for non-supported platforms - [ ] Test case improvement ### Back port request - [ ] 202205 - [ ] 202305 - [ ] 202311 - [ ] 202405 - [ ] 202411 - [x] msft_202412 - [x] 202505 ### Approach #### What is the motivation for this PR? `configlet/test_add_rack.py` is failing. #### How did you do it? Adding extra entries to ignore. These entries seems to be harmless and not used. However it is not understood why it is being added by `config apply-patch`. Again, ideally this should be fixed in `config apply-patch` code. This PR is a workaround for the issue since the extra entries are harmless and does not get used. #### How did you verify/test it? Test no longer fails #### Any platform specific information? Broadcom platform is used for testing. #### Supported testbed topology if it's a new test case? ### Documentation --- tests/common/configlet/utils.py | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/tests/common/configlet/utils.py b/tests/common/configlet/utils.py index 5c8e31d209..3dcd08f5c7 100644 --- a/tests/common/configlet/utils.py +++ b/tests/common/configlet/utils.py @@ -119,7 +119,14 @@ def do_pause(secs, msg): "keys_skip_val_comp": { "last_up_time", "last_down_time", - "flap_count" + "flap_count", + "pre1", + "pre2", + "pre3", + "post1", + "post2", + "post3", + "main" } }, "state-db": {