Remove macsec skip in 202305 added via PR #10246 (#10820)

What is the motivation for this PR?
Revert the macsec test skip PR, done earlier via PR #10246

How did you do it?
How did you verify/test it?

.azure-pipelines/pr_test_scripts.yaml

@@ -76,12 +76,12 @@ t0-2vlans:
   - dhcp_relay/test_dhcpv6_relay.py
 
 t0-sonic:
-  # - bgp/test_bgp_fact.py
-  # - macsec/test_controlplane.py
-  # - macsec/test_dataplane.py
-  # - macsec/test_deployment.py
-  # - macsec/test_fault_handling.py
-  # - macsec/test_interop_protocol.py
+  - bgp/test_bgp_fact.py
+  - macsec/test_controlplane.py
+  - macsec/test_dataplane.py
+  - macsec/test_deployment.py
+  - macsec/test_fault_handling.py
+  - macsec/test_interop_protocol.py
   - pc/test_retry_count.py
   - platform_tests/test_advanced_reboot.py::test_warm_reboot
 

azure-pipelines.yml

@@ -150,7 +150,6 @@ stages:
           VM_TYPE: vsonic
           KVM_IMAGE_BRANCH: "202305"
           MGMT_BRANCH: "202305"
-          SCRIPTS_EXCLUDE: "bgp/test_bgp_fact.py"
 
 #  - job: wan_elastictest
 #    displayName: "kvmtest-wan by Elastictest"

tests/common/plugins/conditional_mark/tests_mark_conditions.yaml

@@ -645,12 +645,6 @@ ipfwd/test_mtu.py:
 #######################################
 #####           macsec            #####
 #######################################
-macsec:
-  skip:
-    reason: 'Skip for t0-sonic to prevent deadlock with code PR merge'
-    conditions:
-      - "'t0' in topo_name"
-
 macsec/test_dataplane.py::TestDataPlane::test_server_to_neighbor:
   skip:
     reason: 'test_server_to_neighbor needs downstream neighbors, which is not present in this topo'