{App Service} az appservice plan create: Add default OS as Linux (#33395)

Author: Shi1810

src/azure-cli/azure/cli/command_modules/appservice/_params.py

@@ -126,7 +126,9 @@ def load_arguments(self, _):
 subscription than the app service environment, please use the resource ID for --app-service-environment parameter. ",
                    local_context_attribute=LocalContextAttribute(name='ase_name', actions=[LocalContextAction.GET]))
         c.argument('sku', arg_type=sku_arg_type)
-        c.argument('is_linux', action='store_true', required=False, help='host web app on Linux worker')
+        c.argument('is_linux', arg_type=get_three_state_flag(), default=None, required=False,
+                   help='Host web app on Linux worker. Defaults to true unless --hyper-v is specified. '
+                        'Use "--is-linux false" to create a Windows plan.')
         c.argument('hyper_v', action='store_true', required=False, help='Host Windows Container Web App on Hyper-V worker.')
         c.argument('per_site_scaling', action='store_true', required=False, help='Enable per-app scaling at the '
                                                                                  'App Service plan level to allow for '

src/azure-cli/azure/cli/command_modules/appservice/_validators.py

@@ -111,6 +111,12 @@ def _validate_asp_sku(sku, app_service_environment, zone_redundant):
 
 def validate_asp_create(namespace):
     validate_tags(namespace)
+    # is_linux is None when not explicitly provided by the user (default).
+    # Resolve the default: Linux unless --hyper-v is specified.
+    if namespace.is_linux is None:
+        namespace.is_linux = not namespace.hyper_v
+    elif namespace.is_linux and namespace.hyper_v:
+        raise MutuallyExclusiveArgumentError('Usage error: --is-linux true and --hyper-v cannot be used together.')
     if namespace.sku is None:
         if namespace.is_linux:
             namespace.sku = 'P0V3'
@@ -122,8 +128,6 @@ def validate_asp_create(namespace):
             namespace.sku = 'B1'
     sku = _normalize_sku(namespace.sku)
     _validate_asp_sku(sku, namespace.app_service_environment, namespace.zone_redundant)
-    if namespace.is_linux and namespace.hyper_v:
-        raise MutuallyExclusiveArgumentError('Usage error: --is-linux and --hyper-v cannot be used together.')
 
 
 def validate_functionapp_asp_create(namespace):

src/azure-cli/azure/cli/command_modules/appservice/custom.py

@@ -4979,12 +4979,18 @@ def is_async_response(poller, timeout_seconds=30):
     return status_code == 202
 
 
-def create_app_service_plan(cmd, resource_group_name, name, is_linux, hyper_v, per_site_scaling=False,
+def create_app_service_plan(cmd, resource_group_name, name, is_linux, hyper_v, per_site_scaling=False,  # pylint: disable=too-many-branches
                             app_service_environment=None, sku=None, number_of_workers=None, location=None,
                             tags=None, no_wait=False, zone_redundant=False, async_scaling_enabled=None,
                             is_managed_instance=None, mi_system_assigned=None, mi_user_assigned=None,
                             default_identity=None, rdp_enabled=None, vnet=None, subnet=None,
                             registry_adapters=None, install_scripts=None, storage_mounts=None):
+    if is_linux is None:
+        is_linux = not hyper_v
+    elif is_linux and hyper_v:
+        raise MutuallyExclusiveArgumentError('--hyper-v creates a Windows container plan and cannot be combined '
+                                             'with --is-linux true. Omit --is-linux or use "--is-linux false".')
+
     if sku is None:
         sku = 'P0V3' if is_linux else 'B1'
 
@@ -5073,6 +5079,9 @@ def pre_operations(self):
             args = self.ctx.args
             args.no_wait = no_wait
 
+    os_type = 'Linux' if is_linux else ('Hyper-V' if hyper_v else 'Windows')
+    logger.warning("Creating App Service Plan '%s' (%s, SKU: %s).", name, os_type, sku)
+
     poller = AppServicePlanCreateWithNoWait(cli_ctx=cmd.cli_ctx)(command_args={
         "name": name,
         "resource_group": resource_group_name,
@@ -5098,9 +5107,6 @@ def pre_operations(self):
         "storage_mounts": storage_mounts,
     })
 
-    os_type = 'Linux' if is_linux else ('Hyper-V' if hyper_v else 'Windows')
-    logger.warning("Creating App Service Plan '%s' (%s).", name, os_type)
-
     if no_wait:
         return poller.result()