{"payload":{"feedbackUrl":"https://github.com/orgs/community/discussions/53140","repo":{"id":52558133,"defaultBranch":"master","name":"sonic-mgmt","ownerLogin":"sonic-net","currentUserCanPush":false,"isFork":false,"isEmpty":false,"createdAt":"2016-02-25T21:28:59.000Z","ownerAvatar":"https://avatars.githubusercontent.com/u/102750714?v=4","public":true,"private":false,"isOrgOwned":true},"refInfo":{"name":"","listCacheKey":"v0:1727165814.0","currentOid":""},"activityList":{"items":[{"before":"327b253d062f67dd0ea0493658d4f9a42f4b8496","after":"2aa1d5fc63eef13467bb7b0e518610edc338034b","ref":"refs/heads/master","pushedAt":"2024-09-30T07:04:55.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"yejianquan","name":"Jianquan Ye","path":"/yejianquan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/4735332?s=80&v=4"},"commit":{"message":"add ptfhost check statement to prevent failure on test_posttest.py (#14783)\n\nPTFHost can be None in some cases (eg. ixia) and this was not handled correctly in test_collect_ptf_logs. Add ptfhost check to prevent error caused by None value.\r\n\r\nPTFHost can be None in some cases (eg. ixia) and this was not handled correctly in test_collect_ptf_logs. Add ptfhost check to prevent error caused by None value.\r\n\r\nDescription of PR\r\nSummary:\r\nPTFHost can be None in some cases (eg. ixia) and this was not handled correctly in test_collect_ptf_logs. Add ptfhost check to prevent error caused by None value.\r\n\r\nApproach\r\nWhat is the motivation for this PR?\r\nResolve unexpected error during log collection\r\n\r\nHow did you do it?\r\ncheck for None value\r\n\r\nco-authorized by: jianquanye@microsoft.com","shortMessageHtmlLink":"add ptfhost check statement to prevent failure on test_posttest.py (#…"}},{"before":"5f34b48e51f7b3b164e79e2d55430932a5164703","after":"327b253d062f67dd0ea0493658d4f9a42f4b8496","ref":"refs/heads/master","pushedAt":"2024-09-30T07:03:00.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"yejianquan","name":"Jianquan Ye","path":"/yejianquan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/4735332?s=80&v=4"},"commit":{"message":"refactor: move skip condition to tests mark (#14795)\n\nDescription of PR\r\nMove the multi-asic skip condition in override_config_table/test_override_config_table.py to test_mark_conditions.yaml to speed up the skip process.\r\n\r\nApproach\r\nWhat is the motivation for this PR?\r\nCurrently, it takes 1 ~ 2 hours to skip the override_config_table/test_override_config_table.py test on T2 chassis because there's some unnecessary setup steps before it reaches multi-asic pytest.skip() condition. We can improve this by moving the skip condition to test_mark_conditions.yaml.\r\n\r\nHow did you do it?\r\nHow did you verify/test it?\r\nI ran the updated code in T2 chassis and it only took ~1 min to skip the test. Meanwhile, I can confirm that this test is running as normal in single-asic Testbeds.\r\n\r\nco-authorized by: jianquanye@microsoft.com","shortMessageHtmlLink":"refactor: move skip condition to tests mark (#14795)"}},{"before":"fbfef99ad46607c48eef7a8651a211a8d53aa18e","after":"3412a8c5b27f62828879e7a7ab4cb5d4699a9445","ref":"refs/heads/202405","pushedAt":"2024-09-30T06:48:11.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"mssonicbld","name":null,"path":"/mssonicbld","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/79238446?s=80&v=4"},"commit":{"message":"Fix customMsg from unexpected fanity failure (#14792)\n\nNew feature delivered recently, add customMsg for sanity check failures, was causing unexpected sanity_check results. This was due to premature cache key reset of pre_sanity_check_failed and post_sanity_check_failed. Removed those two keys from getting reset before sessionfinish to resolve the issue\r\n\r\nImportant cache keys, used to determine session exit status:\r\npre_sanity_check_failed & post_sanity_check_failed => These combination of keys will return exit status 10, 11, 12\r\n10 = PRE_SANITY_CHECK_FAILED_RC\r\n11 = POST_SANITY_CHECK_FAILED_RC\r\n12 = SANITY_CHECK_FAILED_RC\r\n\r\nduthosts_fixture_failed => If this is true, we return exit status code 15\r\n15 = DUTHOSTS_FIXTURE_FAILED_RC\r\n\r\nDescription of PR\r\nSummary:\r\nNew feature delivered recently, add customMsg for sanity check failures, was causing unexpected sanity_check results. This was due to premature cache key reset of pre_sanity_check_failed and post_sanity_check_failed. Removed those two keys from getting reset before sessionfinish to resolve the issue\r\n\r\nType of change\r\n Bug fix\r\n Testbed and Framework(new/improvement)\r\n Test case(new/improvement)\r\nBack port request\r\n 202012\r\n 202205\r\n 202305\r\n 202311\r\n 202405\r\nApproach\r\nWhat is the motivation for this PR?\r\nHow did you do it?\r\nRemove premature cache reset of pre_sanity_check_failed and post_sanity_check_failed, before it reaches sessionfinish\r\n\r\nHow did you verify/test it?\r\nManually tested, simulating pre_sanity_checked_failed and post_sanity_check_failed, to observe expected exit satus codes from 10,11, and 12.\r\nExit status code can be retrieved and displayed, using 'echo $?'\r\neg:\r\nimage\r\n\r\nco-authorized by: jianquanye@microsoft.com","shortMessageHtmlLink":"Fix customMsg from unexpected fanity failure (#14792)"}},{"before":"486abb8bc57b080f435cb2d7e6fb86570dde4256","after":"5f34b48e51f7b3b164e79e2d55430932a5164703","ref":"refs/heads/master","pushedAt":"2024-09-30T03:38:30.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"yejianquan","name":"Jianquan Ye","path":"/yejianquan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/4735332?s=80&v=4"},"commit":{"message":"Fix customMsg from unexpected fanity failure (#14792)\n\nNew feature delivered recently, add customMsg for sanity check failures, was causing unexpected sanity_check results. This was due to premature cache key reset of pre_sanity_check_failed and post_sanity_check_failed. Removed those two keys from getting reset before sessionfinish to resolve the issue\r\n\r\nImportant cache keys, used to determine session exit status:\r\npre_sanity_check_failed & post_sanity_check_failed => These combination of keys will return exit status 10, 11, 12\r\n10 = PRE_SANITY_CHECK_FAILED_RC\r\n11 = POST_SANITY_CHECK_FAILED_RC\r\n12 = SANITY_CHECK_FAILED_RC\r\n\r\nduthosts_fixture_failed => If this is true, we return exit status code 15\r\n15 = DUTHOSTS_FIXTURE_FAILED_RC\r\n\r\nDescription of PR\r\nSummary:\r\nNew feature delivered recently, add customMsg for sanity check failures, was causing unexpected sanity_check results. This was due to premature cache key reset of pre_sanity_check_failed and post_sanity_check_failed. Removed those two keys from getting reset before sessionfinish to resolve the issue\r\n\r\nType of change\r\n Bug fix\r\n Testbed and Framework(new/improvement)\r\n Test case(new/improvement)\r\nBack port request\r\n 202012\r\n 202205\r\n 202305\r\n 202311\r\n 202405\r\nApproach\r\nWhat is the motivation for this PR?\r\nHow did you do it?\r\nRemove premature cache reset of pre_sanity_check_failed and post_sanity_check_failed, before it reaches sessionfinish\r\n\r\nHow did you verify/test it?\r\nManually tested, simulating pre_sanity_checked_failed and post_sanity_check_failed, to observe expected exit satus codes from 10,11, and 12.\r\nExit status code can be retrieved and displayed, using 'echo $?'\r\neg:\r\nimage\r\n\r\nco-authorized by: jianquanye@microsoft.com","shortMessageHtmlLink":"Fix customMsg from unexpected fanity failure (#14792)"}},{"before":"53a2e9c47ecdb83034e57472bec89775869be91b","after":"fbfef99ad46607c48eef7a8651a211a8d53aa18e","ref":"refs/heads/202405","pushedAt":"2024-09-29T09:00:04.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kevinskwang","name":"Kevin Wang","path":"/kevinskwang","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/65380078?s=80&v=4"},"commit":{"message":"Revert \"Full Mesh Traffic Sanity verification (#11968)\" (#14754)\n\nThis reverts commit e15dacafa96b7beca988d0bdfba003688e5d3ae2.","shortMessageHtmlLink":"Revert \"Full Mesh Traffic Sanity verification (#11968)\" (#14754)"}},{"before":"0a909acd8e6c8f697074af79f2636635af6a86b0","after":"53a2e9c47ecdb83034e57472bec89775869be91b","ref":"refs/heads/202405","pushedAt":"2024-09-29T02:00:31.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kevinskwang","name":"Kevin Wang","path":"/kevinskwang","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/65380078?s=80&v=4"},"commit":{"message":"TC-specific ignore SAI_SWITCH_ATTR, SAI_OBJECT_TYPE logs (#14786)","shortMessageHtmlLink":"TC-specific ignore SAI_SWITCH_ATTR, SAI_OBJECT_TYPE logs (#14786)"}},{"before":"2eec9106cdf04eeed073464537f56956d784c18c","after":"486abb8bc57b080f435cb2d7e6fb86570dde4256","ref":"refs/heads/master","pushedAt":"2024-09-27T22:22:25.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"arlakshm","name":"Arvindsrinivasan Lakshmi Narasimhan","path":"/arlakshm","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/55814491?s=80&v=4"},"commit":{"message":"[chassis] Add skip for test_vs_chassis_setup.py on non-vs testbeds in tests_mark_conditions.yaml (#14750)\n\nWhat is the motivation for this PR?\r\nStop tests in test_vs_chassis_setup from running and failing on non vs_chassis.\r\n\r\nHow did you do it?\r\nAdd skip condition for test if asic type is not 'vs' in tests_mark_conditions.yaml\r\n\r\nHow did you verify/test it?\r\nRan test on physical T2 testbed to verify it skips.","shortMessageHtmlLink":"[chassis] Add skip for test_vs_chassis_setup.py on non-vs testbeds in…"}},{"before":"6cfca34cabde76ca505a00591a5df16c595a94e7","after":"0a909acd8e6c8f697074af79f2636635af6a86b0","ref":"refs/heads/202405","pushedAt":"2024-09-27T15:48:02.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"bingwang-ms","name":null,"path":"/bingwang-ms","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/66248323?s=80&v=4"},"commit":{"message":"Enable fdb and qos tests for t0-standalone-* topology (#14737)\n\n* solve conflicts in the PR #14041\r\n\r\nSigned-off-by: Janetxxx \r\n\r\n* Enable fdb and qos tests for t0-standalone-* topology (#13019)\r\n\r\n* Whitelist t0-standalone-* topology for enabling FDB test cases.\r\n\r\n* Whitelist t0-standalone-* topology to enable QoS tests.\r\n\r\n* Update test marks.\r\n\r\n---------\r\n\r\nSigned-off-by: Janetxxx \r\nCo-authored-by: Riff ","shortMessageHtmlLink":"Enable fdb and qos tests for t0-standalone-* topology (#14737)"}},{"before":"8dc356426596018be42eb625599a302e9d199284","after":"6cfca34cabde76ca505a00591a5df16c595a94e7","ref":"refs/heads/202405","pushedAt":"2024-09-27T09:13:30.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"mssonicbld","name":null,"path":"/mssonicbld","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/79238446?s=80&v=4"},"commit":{"message":"[platform_tests/counterpoll]: Don't run counterpoll_watermark test on supervisor (#14724)\n\nSelect only frontend hwskus for counterpoll_queue_watermark_pg_drop test\r\nAssert if waiting for COUNTER_DB to be populated fails\r\nBugLink: #14721\r\n\r\nSummary:\r\nFixes #14721\r\n\r\nApproach\r\nWhat is the motivation for this PR?\r\nplatform test test_counterpoll_watermark_pg_drop runs and fails on the sup cards, where it should only be running on linecards. Test case fixture needs to be modified to only select frontend cards. Additionally, the test should fail if it fails to wait on the COUNTER_DB on becoming populated\r\n\r\nHow did you do it?\r\nHow did you verify/test it?\r\nRun on testbed - successfully skips supervisor, and asserts properly if the counter_db is not populated\r\n\r\nSigned-off-by: Liam Kearney ","shortMessageHtmlLink":"[platform_tests/counterpoll]: Don't run counterpoll_watermark test on…"}},{"before":"21c0df8f86c77c276f92b5b56f782f3aba602ff8","after":"2eec9106cdf04eeed073464537f56956d784c18c","ref":"refs/heads/master","pushedAt":"2024-09-27T08:01:15.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"wangxin","name":"Xin Wang","path":"/wangxin","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/913946?s=80&v=4"},"commit":{"message":"Move test_cont_warm_reboot.py to PR test (#14778)\n\nWhat is the motivation for this PR?\r\nIn this PR, we move test script test_cont_warm_reboot.py to PR test.\r\n\r\nHow did you do it?\r\nHow did you verify/test it?\r\nTestbedName\tFilePath\tRunDate\tSuccessCount\tFailureCount\tTotalTests\tSuccessRate\r\nvms-kvm-t0\tplatform_tests/test_cont_warm_reboot.py\t2024-09-27 00:00:00.0000000\t5\t0\t5\t1\r\nvms-kvm-t0\tplatform_tests/test_cont_warm_reboot.py\t2024-09-26 00:00:00.0000000\t34\t0\t34\t1","shortMessageHtmlLink":"Move test_cont_warm_reboot.py to PR test (#14778)"}},{"before":"976f78be540ca0ac1e7b128ec7c9c5698acd9ccb","after":"8dc356426596018be42eb625599a302e9d199284","ref":"refs/heads/202405","pushedAt":"2024-09-27T07:29:48.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"mssonicbld","name":null,"path":"/mssonicbld","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/79238446?s=80&v=4"},"commit":{"message":"Adding a new testcase to check if mac learning happens correctly (#13912)","shortMessageHtmlLink":"Adding a new testcase to check if mac learning happens correctly (#13912"}},{"before":"77166badd016640db4bc2708ddfcbde117a0e7d5","after":"976f78be540ca0ac1e7b128ec7c9c5698acd9ccb","ref":"refs/heads/202405","pushedAt":"2024-09-27T07:29:45.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"mssonicbld","name":null,"path":"/mssonicbld","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/79238446?s=80&v=4"},"commit":{"message":"tacacs/test_ro_disk.py: Increase wait time for modular chassis after reboot (#14718)\n\nDescription of PR\r\nThe testcase only waits for 5 mins after Supervisor reboot, this is not enough in case of modular chassis. This causes all testcases to fail after this test stating that Host is unavailable.\r\n\r\nSummary:\r\nFixes # (issue)\r\n\r\nApproach\r\nWhat is the motivation for this PR?\r\nWait time of 5 minutes is not enough for supervisor node. Especially in this testcase where ssh check is disabled.\r\n\r\nHow did you do it?\r\nIncreased the time for supervisor node.\r\n\r\nHow did you verify/test it?\r\nValidated on modular chassis with T2 profile\r\n\r\nco-authorized by: jianquanye@microsoft.com","shortMessageHtmlLink":"tacacs/test_ro_disk.py: Increase wait time for modular chassis after …"}},{"before":"e2229ebd55f385e42a6876dc658a0274e3ee4d50","after":"77166badd016640db4bc2708ddfcbde117a0e7d5","ref":"refs/heads/202405","pushedAt":"2024-09-27T07:29:41.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"mssonicbld","name":null,"path":"/mssonicbld","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/79238446?s=80&v=4"},"commit":{"message":"Add custom message for pre/post sanity check results (#14755)\n\nExpand custom message feature to find potentially risky tests and modify existing custom message to be expandable for future use.\r\n\r\nDescription of PR\r\nExpand custom message feature to find potentially risky tests and modify existing custom message to be expandable for future use.\r\n\r\nApproach\r\nWhat is the motivation for this PR?\r\nTo enhance debugging by adding extra information into customMsg we already have, and also make it expandable so others can add debugging information to be utlised in ADE\r\n\r\nco-authorized by: jianquanye@microsoft.com","shortMessageHtmlLink":"Add custom message for pre/post sanity check results (#14755)"}},{"before":"468294cb67c14e26f86d529c8baa87ab63c292c5","after":"21c0df8f86c77c276f92b5b56f782f3aba602ff8","ref":"refs/heads/master","pushedAt":"2024-09-27T04:52:15.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"yejianquan","name":"Jianquan Ye","path":"/yejianquan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/4735332?s=80&v=4"},"commit":{"message":"[platform_tests/counterpoll]: Don't run counterpoll_watermark test on supervisor (#14724)\n\nSelect only frontend hwskus for counterpoll_queue_watermark_pg_drop test\r\nAssert if waiting for COUNTER_DB to be populated fails\r\nBugLink: #14721\r\n\r\nSummary:\r\nFixes #14721\r\n\r\nApproach\r\nWhat is the motivation for this PR?\r\nplatform test test_counterpoll_watermark_pg_drop runs and fails on the sup cards, where it should only be running on linecards. Test case fixture needs to be modified to only select frontend cards. Additionally, the test should fail if it fails to wait on the COUNTER_DB on becoming populated\r\n\r\nHow did you do it?\r\nHow did you verify/test it?\r\nRun on testbed - successfully skips supervisor, and asserts properly if the counter_db is not populated\r\n\r\nSigned-off-by: Liam Kearney ","shortMessageHtmlLink":"[platform_tests/counterpoll]: Don't run counterpoll_watermark test on…"}},{"before":"64ea98491308e98a9e5831319fa43eb3377f3200","after":"e2229ebd55f385e42a6876dc658a0274e3ee4d50","ref":"refs/heads/202405","pushedAt":"2024-09-27T04:27:42.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"yejianquan","name":"Jianquan Ye","path":"/yejianquan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/4735332?s=80&v=4"},"commit":{"message":"Revert PR:12186 crm_stats_ipv6_route_available counter was not decremented, as we have a fix in platform side of code (#14775)\n\nDescription of PR\r\n#14104\r\nReverting this PR as we have fix on the platform side to take care of the CRM route resource usage related issue.\r\n\r\nThis change is merged in master #14602\r\nThis is a conflict merge to 202405 branch.\r\n\r\nSummary:\r\nFixes # (issue)\r\n\r\nCRM route resource fix is committed in the platform side of code. the additional delay and checks added part of the PR/14104 is no longer needed.\r\n\r\nApproach\r\nWhat is the motivation for this PR?\r\nRevert the cisco specific change added in sonic-mgmt code.\r\n\r\nHow did you do it?\r\nHow did you verify/test it?\r\nVerified running sonic mgmt tests/test_crm.py::test_crm_route\r\n\r\nco-authorized by: jianquanye@microsoft.com","shortMessageHtmlLink":"Revert PR:12186 crm_stats_ipv6_route_available counter was not decrem…"}},{"before":"44905dbeca1c60946d0ded075ec3dd1410191a9d","after":"468294cb67c14e26f86d529c8baa87ab63c292c5","ref":"refs/heads/master","pushedAt":"2024-09-27T03:57:50.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"yejianquan","name":"Jianquan Ye","path":"/yejianquan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/4735332?s=80&v=4"},"commit":{"message":"Add custom message for pre/post sanity check results (#14755)\n\nExpand custom message feature to find potentially risky tests and modify existing custom message to be expandable for future use.\r\n\r\nDescription of PR\r\nExpand custom message feature to find potentially risky tests and modify existing custom message to be expandable for future use.\r\n\r\nApproach\r\nWhat is the motivation for this PR?\r\nTo enhance debugging by adding extra information into customMsg we already have, and also make it expandable so others can add debugging information to be utlised in ADE\r\n\r\nco-authorized by: jianquanye@microsoft.com","shortMessageHtmlLink":"Add custom message for pre/post sanity check results (#14755)"}},{"before":"2d4203733423c80f9bc4b4b33505e181c8ab173c","after":"44905dbeca1c60946d0ded075ec3dd1410191a9d","ref":"refs/heads/master","pushedAt":"2024-09-27T03:22:37.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"yejianquan","name":"Jianquan Ye","path":"/yejianquan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/4735332?s=80&v=4"},"commit":{"message":"tacacs/test_ro_disk.py: Increase wait time for modular chassis after reboot (#14718)\n\nDescription of PR\r\nThe testcase only waits for 5 mins after Supervisor reboot, this is not enough in case of modular chassis. This causes all testcases to fail after this test stating that Host is unavailable.\r\n\r\nSummary:\r\nFixes # (issue)\r\n\r\nApproach\r\nWhat is the motivation for this PR?\r\nWait time of 5 minutes is not enough for supervisor node. Especially in this testcase where ssh check is disabled.\r\n\r\nHow did you do it?\r\nIncreased the time for supervisor node.\r\n\r\nHow did you verify/test it?\r\nValidated on modular chassis with T2 profile\r\n\r\nco-authorized by: jianquanye@microsoft.com","shortMessageHtmlLink":"tacacs/test_ro_disk.py: Increase wait time for modular chassis after …"}},{"before":"8a6ffc3d490176d62f1ccf6093aa7d8f612dc176","after":"2d4203733423c80f9bc4b4b33505e181c8ab173c","ref":"refs/heads/master","pushedAt":"2024-09-27T02:32:44.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kevinskwang","name":"Kevin Wang","path":"/kevinskwang","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/65380078?s=80&v=4"},"commit":{"message":"Adding a new testcase to check if mac learning happens correctly (#13912)","shortMessageHtmlLink":"Adding a new testcase to check if mac learning happens correctly (#13912"}},{"before":"414c9a7b55c351b89fa75373bc6ca24686b394f2","after":"64ea98491308e98a9e5831319fa43eb3377f3200","ref":"refs/heads/202405","pushedAt":"2024-09-27T02:22:44.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kevinskwang","name":"Kevin Wang","path":"/kevinskwang","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/65380078?s=80&v=4"},"commit":{"message":"TC-specific ignore for errors when using copper cables (#14771)","shortMessageHtmlLink":"TC-specific ignore for errors when using copper cables (#14771)"}},{"before":"2f1cac5865ed4c8087442f96427a1c8872b9bf9c","after":"6137af95516814a9f3568afd5b5976fae20196ec","ref":"refs/heads/202311","pushedAt":"2024-09-27T02:22:41.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kevinskwang","name":"Kevin Wang","path":"/kevinskwang","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/65380078?s=80&v=4"},"commit":{"message":"TC-specific ignore for errors when using copper cables (#14770)","shortMessageHtmlLink":"TC-specific ignore for errors when using copper cables (#14770)"}},{"before":"64c4d4148e0927102058bbb6f884db5afe9c49a9","after":"2f1cac5865ed4c8087442f96427a1c8872b9bf9c","ref":"refs/heads/202311","pushedAt":"2024-09-27T02:04:47.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"mssonicbld","name":null,"path":"/mssonicbld","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/79238446?s=80&v=4"},"commit":{"message":"Update test_qos_sai for ComputeAI config (#14232)\n\n* support qos params for ComputeAI\r\n\r\n* fix format\r\n\r\n* support ComputeAI config for qos_sai script\r\n\r\nSigned-off-by: Zhixin Zhu \r\n\r\n* Adjust indentation\r\n\r\nSigned-off-by: Zhixin Zhu \r\n\r\n* fix pre-commit check failure\r\n\r\n* get dscp value from config\r\n\r\nSigned-off-by: Zhixin Zhu \r\n\r\n* Support DscpQueueMapping and Dwrr cases in computeAI config\r\n\r\nSigned-off-by: Zhixin Zhu \r\n\r\n* recover dual_tor code in DscpMappingPB\r\n\r\nSigned-off-by: Zhixin Zhu \r\n\r\n* fix wrr extra packet in queue\r\n\r\nSigned-off-by: Zhixin Zhu \r\n\r\n---------\r\n\r\nSigned-off-by: Zhixin Zhu ","shortMessageHtmlLink":"Update test_qos_sai for ComputeAI config (#14232)"}},{"before":"1893815a9c8ebeda1dbabfce12ca783e7a46660c","after":"64c4d4148e0927102058bbb6f884db5afe9c49a9","ref":"refs/heads/202311","pushedAt":"2024-09-27T02:04:44.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"mssonicbld","name":null,"path":"/mssonicbld","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/79238446?s=80&v=4"},"commit":{"message":"adjust packet size for testQosSaiLossyQueueVoqMultiSrc (#13384)","shortMessageHtmlLink":"adjust packet size for testQosSaiLossyQueueVoqMultiSrc (#13384)"}},{"before":"c1b4913d23bdd067c1d87e9a59aa53d40a2be16e","after":"414c9a7b55c351b89fa75373bc6ca24686b394f2","ref":"refs/heads/202405","pushedAt":"2024-09-27T02:04:41.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"mssonicbld","name":null,"path":"/mssonicbld","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/79238446?s=80&v=4"},"commit":{"message":"add pause_garp_service fixture for test_pfcwd_actions (#14301)\n\nWhat is the motivation for this PR?\r\ntest_pfcwd_actions case failed on dualtor sometimes due to ping CMD failure\r\n\r\nHow did you do it?\r\nimport pause_garp_service fixture to stop garp_service during test\r\n\r\nHow did you verify/test it?\r\nLocal run\r\npfcwd/test_pfcwd_function.py::TestPfcwdFunc::test_pfcwd_actions[str2-7260cx3-acs-12] PASSED [ 20%]\r\npfcwd/test_pfcwd_function.py::TestPfcwdFunc::test_pfcwd_multi_port[str2-7260cx3-acs-12] PASSED [ 40%]\r\npfcwd/test_pfcwd_function.py::TestPfcwdFunc::test_pfcwd_mmu_change[str2-7260cx3-acs-12] PASSED [ 60%]\r\npfcwd/test_pfcwd_function.py::TestPfcwdFunc::test_pfcwd_port_toggle[str2-7260cx3-acs-12] PASSED [ 80%]\r\npfcwd/test_pfcwd_function.py::TestPfcwdFunc::test_pfcwd_no_traffic[str2-7260cx3-acs-12] SKIPPED (This test is applicable only for cisco-8000) [100%]\r\n\r\nElastic run\r\nhttps://elastictest.org/scheduler/testplan/66d33dda9ae803ee15b3bc52\r\n\r\nAny platform specific information?\r\ndualtor","shortMessageHtmlLink":"add pause_garp_service fixture for test_pfcwd_actions (#14301)"}},{"before":"edc4d2d69ceb88cf24c6d079124ef71ec64eeefe","after":"1893815a9c8ebeda1dbabfce12ca783e7a46660c","ref":"refs/heads/202311","pushedAt":"2024-09-27T02:04:38.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"mssonicbld","name":null,"path":"/mssonicbld","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/79238446?s=80&v=4"},"commit":{"message":"add pause_garp_service fixture for test_pfcwd_actions (#14301)\n\nWhat is the motivation for this PR?\r\ntest_pfcwd_actions case failed on dualtor sometimes due to ping CMD failure\r\n\r\nHow did you do it?\r\nimport pause_garp_service fixture to stop garp_service during test\r\n\r\nHow did you verify/test it?\r\nLocal run\r\npfcwd/test_pfcwd_function.py::TestPfcwdFunc::test_pfcwd_actions[str2-7260cx3-acs-12] PASSED [ 20%]\r\npfcwd/test_pfcwd_function.py::TestPfcwdFunc::test_pfcwd_multi_port[str2-7260cx3-acs-12] PASSED [ 40%]\r\npfcwd/test_pfcwd_function.py::TestPfcwdFunc::test_pfcwd_mmu_change[str2-7260cx3-acs-12] PASSED [ 60%]\r\npfcwd/test_pfcwd_function.py::TestPfcwdFunc::test_pfcwd_port_toggle[str2-7260cx3-acs-12] PASSED [ 80%]\r\npfcwd/test_pfcwd_function.py::TestPfcwdFunc::test_pfcwd_no_traffic[str2-7260cx3-acs-12] SKIPPED (This test is applicable only for cisco-8000) [100%]\r\n\r\nElastic run\r\nhttps://elastictest.org/scheduler/testplan/66d33dda9ae803ee15b3bc52\r\n\r\nAny platform specific information?\r\ndualtor","shortMessageHtmlLink":"add pause_garp_service fixture for test_pfcwd_actions (#14301)"}},{"before":"e77e31ac00d40048c2798c4addbb7d03ebb11cb7","after":"8a6ffc3d490176d62f1ccf6093aa7d8f612dc176","ref":"refs/heads/master","pushedAt":"2024-09-27T01:44:42.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"StormLiangMS","name":null,"path":"/StormLiangMS","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/89824293?s=80&v=4"},"commit":{"message":"[Test Gap] Add a test script to cover show ip bgp nettwork (#14656)\n\nWhat is the motivation for this PR?\r\nTo address test gap #5069\r\n\r\nHow did you do it?\r\nCompare the output of show ip bgp network and docker exec -i bgp vtysh -c \"show bgp ipv4 all\"\r\nCompare the routes and total path value is equal or not for both command\r\nBoth cover ipv4 and ipv6 command.\r\nHow did you verify/test it?\r\nRun bgp/test_bgp_command.py\r\n\r\nAny platform specific information?\r\nAll platforms\r\n\r\nSupported testbed topology if it's a new test case?\r\nAll topologies","shortMessageHtmlLink":"[Test Gap] Add a test script to cover show ip bgp nettwork (#14656)"}},{"before":"ca973f36304bf1f35dc40db947376cdfde26d6cb","after":"e77e31ac00d40048c2798c4addbb7d03ebb11cb7","ref":"refs/heads/master","pushedAt":"2024-09-27T01:36:40.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"wangxin","name":"Xin Wang","path":"/wangxin","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/913946?s=80&v=4"},"commit":{"message":"Skip pfcwd_all_port_storm test on Arista-7060X6-64PE-256x200G (#14756)\n\nSigned-off-by: Janetxxx ","shortMessageHtmlLink":"Skip pfcwd_all_port_storm test on Arista-7060X6-64PE-256x200G (#14756)"}},{"before":"30e80479e72220fbaa00bc66e78ded6c2d4f1f5f","after":"ca973f36304bf1f35dc40db947376cdfde26d6cb","ref":"refs/heads/master","pushedAt":"2024-09-27T01:21:23.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"wangxin","name":"Xin Wang","path":"/wangxin","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/913946?s=80&v=4"},"commit":{"message":"Generate console connection information for devices (#14523)\n\nWhat is the motivation for this PR?\r\nFor devices, generate console connection entries if this information is available. This allows people to connect to those devices via console if regular SSH access is not working.\r\n\r\nAlso, place arguments to the script into multiple groups, for readability purposes, and make the SSH config syntax checker be validated at argument validation time instead of later.\r\n\r\nSigned-off-by: Saikrishna Arcot ","shortMessageHtmlLink":"Generate console connection information for devices (#14523)"}},{"before":"3e8708ddeda727fff0af993f7dfeacaea948a610","after":"30e80479e72220fbaa00bc66e78ded6c2d4f1f5f","ref":"refs/heads/master","pushedAt":"2024-09-27T01:21:02.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"wangxin","name":"Xin Wang","path":"/wangxin","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/913946?s=80&v=4"},"commit":{"message":"Add sonic-vpp image download instructions and `t1` vtestbed topology (#14608)\n\nWhat is the motivation for this PR?\r\nAdd a no-lag t1 kvm testbed\r\nUpdate sonic vm template to allow nested virtualization\r\nAdd link to sonic-vpp build instructions\r\n\r\nHow did you do it?\r\n\r\nHow did you verify/test it?\r\nTried to deploy a t1 kvm testbed on Azure VM with sonic-vpp image. Was able to get bgp sessions up.","shortMessageHtmlLink":"Add sonic-vpp image download instructions and t1 vtestbed topology (#…"}},{"before":"efc89c6c0c7903805e2d6230da0dc275eea6b34d","after":"edc4d2d69ceb88cf24c6d079124ef71ec64eeefe","ref":"refs/heads/202311","pushedAt":"2024-09-26T21:40:19.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"yxieca","name":"Ying Xie","path":"/yxieca","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/18753401?s=80&v=4"},"commit":{"message":"[202311][dhcp_relay] Cherry-pick dhcp_relay changes (#14741)\n\n* [dhcp_relay] Refactor dhcp_relay for dualtor (#14012)\r\n\r\nWhat is the motivation for this PR?\r\nThere is useless parameter testbed_mode in test_dhcp_relay. Also, skip test in test case is insufficient\r\n\r\nHow did you do it?\r\nRemove useless parameter testbed_mode\r\nUse condition mark to skip test\r\n\r\nHow did you verify/test it?\r\nRun test in T0 / Dualtor testbeds, test passed\r\n\r\n* [dhcp_relay] Add stress test for restarting dhcp_relay (#14641)\r\n\r\nWhat is the motivation for this PR?\r\nAddress test gap for this change: sonic-net/sonic-buildimage#20021\r\nPrevously, dhcrelay would hit the issue that it wouldn't relay any packets if there are packets come when dhcrelay startup. This issue has been fixed from image side by sonic-net/sonic-buildimage#20021. This PR is to add test for it.\r\n\r\nHow did you do it?\r\nAdd stress test with dhcp_relay restart:\r\nKeep sending DHCP packets\r\nRestart dhcp_relay\r\nCheck socket buffer\r\nRun general dhcp relay test.\r\n\r\nHow did you verify/test it?\r\nRun test on m0/t0/dualtor topos, all passed","shortMessageHtmlLink":"[202311][dhcp_relay] Cherry-pick dhcp_relay changes (#14741)"}},{"before":"6fe4aece20622906057598c2f66096811a329b07","after":"3e8708ddeda727fff0af993f7dfeacaea948a610","ref":"refs/heads/master","pushedAt":"2024-09-26T18:13:14.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"arlakshm","name":"Arvindsrinivasan Lakshmi Narasimhan","path":"/arlakshm","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/55814491?s=80&v=4"},"commit":{"message":"Fix Python3 str decode issues (#14332)\n\nWhat is the motivation for this PR?\r\nThis fixes an issue due to Python 3 migration, where decode() is no longer a method of str.\r\n\r\nHow did you do it?\r\nFor cases where we actually need the unicode-escape conversion, I convert to bytes first, then re-decode to str to decode the escape sequences. In other cases where we are just taking the raw output of a command, I just removed the decode().\r\n\r\nHow did you verify/test it?\r\nI ran the relevant sonic-mgmt tests and verified that the tests now pass and no longer raise exceptions due to decode() not being a method of str or AnsibleUnsafeText.","shortMessageHtmlLink":"Fix Python3 str decode issues (#14332)"}}],"hasNextPage":true,"hasPreviousPage":false,"activityType":"all","actor":null,"timePeriod":"all","sort":"DESC","perPage":30,"startCursor":"Y3Vyc29yOnYyOpK7MjAyNC0wOS0zMFQwNzowNDo1NS4wMDAwMDBazwAAAATEW6u-","endCursor":"Y3Vyc29yOnYyOpK7MjAyNC0wOS0yNlQxODoxMzoxNC4wMDAwMDBazwAAAATB0hsP"}},"title":"Activity · sonic-net/sonic-mgmt"}