microsoft / azure-container-apps

Roadmap and issues for Azure Container Apps
MIT License
372 stars 29 forks source link

az cli allows setting of ingress percentage to a deactivated revision #1345

Open DeanPH opened 6 days ago

DeanPH commented 6 days ago

Please provide us with the following information:

This issue is a: (mark with an x)

Issue description

When multiple revisions are enabled, az cli allows you to set 100% traffic to a deactivated revision. Then it also allows you to deactivate the only active revision.

Steps to reproduce

  1. Create a container app and set Revision Mode: Multiple
  2. Create a new revision
  3. Deactivate the new revision created in step 3
  4. Run the following command:
    az containerapp ingress traffic set \
    --name APP_NAME \
    --resource-group RESOURCE_GROUP \
    --revision-weight DEACTIVATED_CA=100 ACTIVE_CA=0
  5. Disable the only active revision:
    az containerapp revision deactivate \
    --name APP_NAME \
    --resource-group RESOURCE_GROUP \
    --revision ACTIVE_CA

Expected behavior [What you expected to happen.] I would expect it to throw errors in 2 places:

Actual behavior [What actually happened.] It allows you to set traffic on a deactivated revision and allows you to deactivate the only active revision.

Screenshots
Screenshots attached...

Additional context

This occurs only when using the az cli. You cannot do this via the Portal.

Image

Image

(Image)

Image

Image

v-vish commented 3 days ago

@DeanPH We are currently investigating this issue. Please allow us some time, and we will get back to you with an update soon.