remove shell support

Author: Shichao Qiu

src/containerapp/HISTORY.rst

@@ -11,7 +11,7 @@ upcoming
 * 'az containerapp env premium-ingress': Deprecate `--min-replicas` and `--max-replicas` parameters, use workload profile scale instead.
 * 'az containerapp sessionpool create/update': Support `--probe-yaml`
 * 'az containerapp session stop': Support stop session for custom container sessions
-* 'az containerapp sessionpool create': Add `Shell` and `GpuBase` container types
+* 'az containerapp sessionpool create': Add `GpuBase` container type
 
 1.2.0b3
 ++++++

src/containerapp/azext_containerapp/_help.py

@@ -1967,10 +1967,6 @@
       text: |
           az containerapp sessionpool create -n mysessionpool -g MyResourceGroup \\
               --location eastasia
-    - name: Create or update a Session Pool with container type Shell default settings.
-      text: |
-          az containerapp sessionpool create -n mysessionpool -g MyResourceGroup \\
-              --container-type Shell --location westus3
     - name: Create or update a Session Pool with container type GpuBase default settings.
       text: |
           az containerapp sessionpool create -n mysessionpool -g MyResourceGroup \\

src/containerapp/azext_containerapp/_params.py

@@ -451,7 +451,7 @@ def load_arguments(self, _):
         c.argument('network_status', arg_type=get_enum_type(["EgressEnabled", "EgressDisabled"]), help="Egress is enabled for the Sessions or not.")
 
     with self.argument_context('containerapp sessionpool create', arg_group='Configuration') as c:
-        c.argument('container_type', arg_type=get_enum_type(["CustomContainer", "PythonLTS", "NodeLTS", "Shell", "GpuBase"]), help="The pool type of the Session Pool, default='PythonLTS'")
+        c.argument('container_type', arg_type=get_enum_type(["CustomContainer", "PythonLTS", "NodeLTS", "GpuBase"]), help="The pool type of the Session Pool, default='PythonLTS'")
         c.argument('lifecycle_type', arg_type=get_enum_type(["Timed", "OnContainerExit"]), help="The lifecycle type of the Session Pool", default='Timed')
 
     with self.argument_context('containerapp sessionpool update', arg_group='Configuration') as c:

src/containerapp/azext_containerapp/tests/latest/test_containerapp_sessionpool.py

@@ -47,17 +47,6 @@ def test_containerapp_sessionpool_e2e(self, resource_group, subnet_id, vnet_name
             JMESPathCheck('properties.dynamicPoolConfiguration.lifecycleConfiguration.cooldownPeriodInSeconds', 300),
         ])
 
-        # Create Shell SessionPool
-        sessionpool_name_shell = self.create_random_name(prefix='spshell', length=24)
-        self.cmd('containerapp sessionpool create -g {} -n {} --container-type Shell --cooldown-period {}'.format(
-            resource_group, sessionpool_name_shell, 300), checks=[
-            JMESPathCheck('name', sessionpool_name_shell),
-            JMESPathCheck('properties.containerType', "Shell"),
-            JMESPathCheck('properties.provisioningState', "Succeeded"),
-            JMESPathCheck('properties.dynamicPoolConfiguration.lifecycleConfiguration.lifecycleType', "Timed"),
-            JMESPathCheck('properties.dynamicPoolConfiguration.lifecycleConfiguration.cooldownPeriodInSeconds', 300),
-        ])
-
         # Create GpuBase SessionPool
         sessionpool_name_gpu = self.create_random_name(prefix='spgpu', length=24)
         self.cmd('containerapp sessionpool create -g {} -n {} --container-type GpuBase --cooldown-period {}'.format(
@@ -106,7 +95,7 @@ def test_containerapp_sessionpool_e2e(self, resource_group, subnet_id, vnet_name
 
         # List Session Pools
         sessionpool_list = self.cmd("containerapp sessionpool list -g {}".format(resource_group)).get_output_in_json()
-        self.assertTrue(len(sessionpool_list) == 4)
+        self.assertTrue(len(sessionpool_list) == 3)
 
         # Update Session Pool
         max_concurrent_session = 12
@@ -154,7 +143,6 @@ def test_containerapp_sessionpool_e2e(self, resource_group, subnet_id, vnet_name
 
         self.cmd('containerapp sessionpool delete -g {} -n {} --yes'.format(resource_group, sessionpool_name_python))
         self.cmd('containerapp sessionpool delete -g {} -n {} --yes'.format(resource_group, sessionpool_name_custom))
-        self.cmd('containerapp sessionpool delete -g {} -n {} --yes'.format(resource_group, sessionpool_name_shell))
         self.cmd('containerapp sessionpool delete -g {} -n {} --yes'.format(resource_group, sessionpool_name_gpu))
 
         # List Session Pools