Snehapar/add source to containerapp update

.github/CODEOWNERS

@@ -128,6 +128,8 @@
 
 /src/timeseriesinsights/ @jiasli
 
+/src/palo-alto-networks/ @kairu-ms @ZengTaoxu
+
 /src/portal/ @YalinLi0312
 
 /src/import-export/ @xfz11
@@ -240,8 +242,6 @@
 
 /src/containerapp/ @ruslany @sanchitmehta @ebencarek @JennyLawrance @howang-ms @vinisoto @chinadragon0515 @vturecek @torosent @pagariyaalok @Juliehzl @jijohn14
 
-/src/containerapp-preview/ @ruslany @sanchitmehta @ebencarek @JennyLawrance @howang-ms @vinisoto @chinadragon0515 @vturecek @torosent @pagariyaalok @Juliehzl @jijohn14
-
 /src/scvmm/ @nascarsayan
 
 /src/spring/ @yuwzho
@@ -307,3 +307,5 @@
 /src/command-change/ @allyw
 
 /src/graphservices/ @amatukmolina
+
+/src/site-recovery/ @calvinhzy

.github/policies/resourceManagement.yml

@@ -207,7 +207,7 @@ configuration:
       - or:
         - payloadType: Issues
       - isAction:
-          action: Opened
+          action: Labeled
       then:
       - if:
         - hasLabel:
@@ -2525,6 +2525,9 @@ configuration:
         - titleContains:
             pattern: '\b(aks|Aks|AKS|acs|Acs|ACS|openshift|Openshift|OPENSHIFT)\b'
             isRegex: True
+        - bodyContains:
+            pattern: az \b(aks|acs|openshift)\b
+            isRegex: True
       then:
       - requestReview:
           reviewer: zhoxing-ms
@@ -2533,6 +2536,7 @@ configuration:
       - assignTo:
           users:
           - zhoxing-ms
+          - yanzhudd
       - addLabel:
           label: Auto-Assign
       - requestReview:
@@ -4234,7 +4238,7 @@ configuration:
       - addLabel:
           label: Auto-Assign
       - requestReview:
-          reviewer: wangzelin007
+          reviewer: yanzhudd
       - assignTo:
           users:
           - zhoxing-ms
@@ -4591,6 +4595,8 @@ configuration:
           reviewer: Juliehzl
       - requestReview:
           reviewer: jijohn14
+      - requestReview:
+          reviewer: Greedygre
       description: '[container app] Auto assign labels and reviewers based on PR title/description.'
     - if:
       - payloadType: Issues
@@ -5068,12 +5074,36 @@ configuration:
           label: Auto-Assign
       - requestReview:
           reviewer: zhoxing-ms
+      - requestReview:
+          reviewer: bavneetsingh16
       - assignTo:
           users:
           - zhoxing-ms
-      - requestReview:
-          reviewer: wangzelin007
       description: '[k8s-extension] Auto assign labels and reviewers based on PR title/description.'
+    - if:
+      - payloadType: Pull_Request
+      - isAction:
+          action: Opened
+      - or:
+        - titleContains:
+            pattern: k8s-configuration
+            isRegex: True
+        - bodyContains:
+            pattern: k8s-configuration
+            isRegex: True
+      then:
+      - requestReview:
+          reviewer: yonzhan
+      - addLabel:
+          label: Auto-Assign
+      - requestReview:
+          reviewer: zhoxing-ms
+      - requestReview:
+          reviewer: bavneetsingh16
+      - assignTo:
+          users:
+          - zhoxing-ms
+      description: '[k8s-configuration] Auto assign labels and reviewers based on PR title/description.'
     - if:
       - payloadType: Issues
       - or:
@@ -5348,6 +5378,9 @@ configuration:
       - isAction:
           action: Opened
       - or:
+        - titleContains:
+            pattern: '[Spring]'
+            isRegex: False
         - titleContains:
             pattern: az \b(spring)\b
             isRegex: True
@@ -5535,5 +5568,49 @@ configuration:
           users:
           - kairu-ms
       description: '[Arc Appliance] Auto assign labels and reviewers based on PR title/description.'
+    - if:
+      - payloadType: Pull_Request
+      - isAction:
+          action: Opened
+      - or:
+        - titleContains:
+            pattern: '\b([Ss]ervice ?[Cc]onnector)\b'
+            isRegex: True
+        - bodyContains:
+            pattern: '\b([Ss]ervice ?[Cc]onnector)\b'
+            isRegex: True
+      then:
+      - requestReview:
+          reviewer: kairu-ms
+      - requestReview:
+          reviewer: jsntcy
+      - addLabel:
+          label: Auto-Assign
+      - assignTo:
+          users:
+          - kairu-ms
+      description: '[Service Connector] Auto assign labels and reviewers based on PR title/description.'
+    - if:
+      - payloadType: Pull_Request
+      - isAction:
+          action: Opened
+      - or:
+        - titleContains:
+            pattern: '\b([Cc]onnected ?[Vv]mware)\b'
+            isRegex: True
+        - bodyContains:
+            pattern: '\b([Cc]onnected ?[Vv]mware)\b'
+            isRegex: True
+      then:
+      - requestReview:
+          reviewer: kairu-ms
+      - requestReview:
+          reviewer: jsntcy
+      - addLabel:
+          label: Auto-Assign
+      - assignTo:
+          users:
+          - kairu-ms
+      description: '[Connected Vmware] Auto assign labels and reviewers based on PR title/description.'
 onFailure: 
 onSuccess: 

.gitignore

@@ -120,4 +120,7 @@ _build/
 # Mac desktop service store files
 .DS_Store
 
-_az_debug/
+_az_debug/
+
+# Ignore temporary test output files.
+/temp/

azure-pipelines.yml

@@ -155,6 +155,7 @@ jobs:
 
 - job: IndexRefDocVerify
   displayName: "Verify Ref Docs"
+  continueOnError: true
   pool:
     name: 'pool-ubuntu-2004'
   steps:

linter_exclusions.yml

@@ -2170,23 +2170,68 @@ networkcloud kubernetescluster agentpool wait:
     kubernetes_cluster_name:
       rule_exclusions:
       - option_length_too_long
+networkfabric acl create:
+  parameters:
+    dynamic_match_configurations:
+      rule_exclusions:
+      - option_length_too_long
 networkfabric controller create:
   parameters:
     workload_er_connections:
       rule_exclusions:
       - option_length_too_long
+    is_workload_management_network_enabled:
+      rule_exclusions:
+      - option_length_too_long
+networkfabric controller update:
+  parameters:
+    workload_er_connections:
+      rule_exclusions:
+      - option_length_too_long
+  rule_exclusions:
+  - missing_command_test_coverage
+networkfabric fabric update:
+  rule_exclusions:
+  - missing_command_test_coverage
 networkfabric rack create:
   rule_exclusions:
   - missing_command_test_coverage
 networkfabric rack delete:
   rule_exclusions:
   - missing_command_test_coverage
+networkfabric device create:
+  rule_exclusions:
+  - missing_command_test_coverage
 networkfabric device delete:
   rule_exclusions:
   - missing_command_test_coverage
+networkfabric interface create:
+  rule_exclusions:
+  - missing_command_test_coverage
 networkfabric interface delete:
   rule_exclusions:
   - missing_command_test_coverage
+networkfabric nni create:
+  parameters:
+    npb_static_route_configuration:
+      rule_exclusions:
+      - option_length_too_long
+    option_b_layer3_configuration:
+      rule_exclusions:
+      - option_length_too_long
+networkfabric nni update:
+  parameters:
+    npb_static_route_configuration:
+      rule_exclusions:
+      - option_length_too_long
+    option_b_layer3_configuration:
+      rule_exclusions:
+      - option_length_too_long
+  rule_exclusions:
+  - missing_command_test_coverage
+networkfabric l2domain update:
+  rule_exclusions:
+  - missing_command_test_coverage
 networkfabric l3domain create:
   parameters:
     aggregate_route_configuration:
@@ -2201,11 +2246,66 @@ networkfabric l3domain create:
     redistribute_static_routes:
       rule_exclusions:
       - option_length_too_long
+networkfabric l3domain update:
+  parameters:
+    aggregate_route_configuration:
+      rule_exclusions:
+      - option_length_too_long
+    connected_subnet_route_policy:
+      rule_exclusions:
+      - option_length_too_long
+    redistribute_connected_subnets:
+      rule_exclusions:
+      - option_length_too_long
+    redistribute_static_routes:
+      rule_exclusions:
+      - option_length_too_long
+  rule_exclusions:
+  - missing_command_test_coverage
 networkfabric internalnetwork create:
   parameters:
     static_route_configuration:
       rule_exclusions:
       - option_length_too_long
+networkfabric internalnetwork update:
+  parameters:
+    static_route_configuration:
+      rule_exclusions:
+      - option_length_too_long
+  rule_exclusions:
+  - missing_command_test_coverage
+networkfabric internetgateway update:
+  parameters:
+    internet_gateway_rule_id:
+      rule_exclusions:
+      - option_length_too_long
+networkfabric internetgatewayrule update:
+  rule_exclusions:
+  - missing_command_test_coverage
+networkfabric externalnetwork update:
+  rule_exclusions:
+  - missing_command_test_coverage
+networkfabric ipextendedcommunity create:
+  parameters:
+    ip_extended_community_rules:
+      rule_exclusions:
+      - option_length_too_long
+networkfabric tap update:
+  rule_exclusions:
+  - missing_command_test_coverage
+networkfabric taprule create:
+  parameters:
+    dynamic_match_configurations:
+      rule_exclusions:
+      - option_length_too_long
+    polling_interval_in_seconds:
+      rule_exclusions:
+      - option_length_too_long
+networkfabric taprule update:
+  parameters:
+    dynamic_match_configurations:
+      rule_exclusions:
+      - option_length_too_long
 notification-hub authorization-rule create:
   parameters:
     notification_hub_name:

scripts/ci/index_ref_doc.py

@@ -50,35 +50,24 @@
 class TestIndexRefDocsMeta(type):
     def __new__(mcs, name, bases, _dict):
 
-        def gen_test(ext_name, ext_url, filename, dep_url):
+        def gen_test(ext_name, ext_url, filename):
             def test(self):
-                if dep_url.get(ext_name):
-                    dep_file = get_whl_from_url(dep_url[ext_name][0], dep_url[ext_name][1], self.whl_dir)
-                else:
-                    dep_file = None
                 ext_file = get_whl_from_url(ext_url, filename, self.whl_dir)
                 ref_doc_out_dir = os.path.join(REF_DOC_OUT_DIR, ext_name)
                 if not os.path.isdir(ref_doc_out_dir):
                     os.mkdir(ref_doc_out_dir)
                 script_args = [sys.executable, REF_GEN_SCRIPT, '--extension-file', ext_file, '--output-dir',
                                ref_doc_out_dir]
-                if dep_file:
-                    script_args.extend(['--dependent-file', dep_file])
                 try:
                     check_call(script_args)
                 except CalledProcessError as e:
                     traceback.print_exc()
                     raise e
             return test
 
-        dep_url = {}
         for ext_name, ext_url, filename in ALL_TESTS:
             test_name = "test_ref_doc_%s" % ext_name
-            # The containerapp-preview extension is a special case,
-            # it must depend on the continerapp extension and cannot run independently.
-            if ext_name == 'containerapp':
-                dep_url['containerapp-preview'] = [ext_url, filename]
-            _dict[test_name] = gen_test(ext_name, ext_url, filename, dep_url)
+            _dict[test_name] = gen_test(ext_name, ext_url, filename)
         return type.__new__(mcs, name, bases, _dict)
 
 

scripts/ci/verify_linter.py

@@ -70,17 +70,9 @@ def _cmd(cmd):
         check_call(cmd, shell=True)
 
     def add_from_code(self):
-        # The containerapp-preview extension is a special case,
-        # it must depend on the continerapp extension and cannot run independently.
-        if self.extension_name == 'containerapp-preview':
-            self._cmd('azdev extension add containerapp')
         self._cmd('azdev extension add {}'.format(self.extension_name))
 
     def remove(self):
-        # The containerapp-preview extension is a special case,
-        # it must depend on the continerapp extension and cannot run independently.
-        if self.extension_name == 'containerapp-preview':
-            self._cmd('azdev extension remove containerapp')
         self._cmd('azdev extension remove {}'.format(self.extension_name))
 
     def linter(self):