Skip to content

[Microsoft.App] Update the error shown on ConsumptionOnly env creation #9527

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
jepetty wants to merge 3 commits into from
Closed

[Microsoft.App] Update the error shown on ConsumptionOnly env creation #9527

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
82435bf
Update the error shown if you try to make a ConsumptionOnly environme…
jepetty Jan 15, 2026
f5b84f2
Merge remote-tracking branch 'upstream/main' into updateWPError
jepetty Jan 21, 2026
97e0a5a
Update HISTORY.rst with changes
jepetty Jan 21, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion src/containerapp/HISTORY.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,8 @@ Release History
upcoming
++++++
* 'az containerapp function invocations': Update application insights query
* 'az containerapp function keys': Update minimum replica check
* 'az containerapp function keys': Update minimum replica check
* 'az containerapp env create': Improved error message for environment kind incompatibility

1.3.0b1
++++++
Expand Down
18 changes: 11 additions & 7 deletions src/containerapp/azext_containerapp/containerapp_env_decorator.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -110,14 +110,14 @@ def set_up_managed_identity(self):
self.managed_env_def["identity"] = identity_def

def set_up_workload_profiles(self):
if self.get_argument_enable_workload_profiles():
# If the environment exists, infer the environment type
existing_environment = None
try:
existing_environment = self.client.show(cmd=self.cmd, resource_group_name=self.get_argument_resource_group_name(), name=self.get_argument_name())
except Exception as e:
handle_non_404_status_code_exception(e)
# If the environment exists, infer the environment type
existing_environment = None
try:
existing_environment = self.client.show(cmd=self.cmd, resource_group_name=self.get_argument_resource_group_name(), name=self.get_argument_name())
except Exception as e:
handle_non_404_status_code_exception(e)

if self.get_argument_enable_workload_profiles():
if existing_environment and safe_get(existing_environment, "properties", "workloadProfiles") is None:
# check if input params include -w/--enable-workload-profiles
if self.cmd.cli_ctx.data.get('safe_params') and ('-w' in self.cmd.cli_ctx.data.get('safe_params') or '--enable-workload-profiles' in self.cmd.cli_ctx.data.get('safe_params')):
Expand Down Expand Up @@ -148,6 +148,10 @@ def set_up_workload_profiles(self):
}
workload_profiles.append(serverless_gpu_profile)
self.managed_env_def["properties"]["workloadProfiles"] = workload_profiles
else:
if existing_environment and safe_get(existing_environment, "properties", "workloadProfiles") is not None:
raise ValidationError(
f"Existing environment {self.get_argument_name()} uses workload profiles. If you want to use Consumption-Only environment, please create a new one.")

def set_up_custom_domain_configuration(self):
if self.get_argument_hostname():
Expand Down
Loading