Skip to content
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.

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

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

Open
1 task done
DeanPH opened this issue Nov 15, 2024 · 1 comment
Open
1 task done
Labels
investigating currently looking into the issue

Comments

@DeanPH
Copy link

DeanPH commented Nov 15, 2024

Please provide us with the following information:

This issue is a: (mark with an x)

  • bug report -> please search issues before submitting

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
  1. 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:

  • "Cannot set traffic on an deactivated revision"
  • "Cannot deactivate the only action revision"

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

@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs: triage 🔍 Pending a first pass to read, tag, and assign label Nov 15, 2024
@v-vish v-vish added investigating currently looking into the issue and removed Needs: triage 🔍 Pending a first pass to read, tag, and assign labels Nov 19, 2024
@v-vish
Copy link
Collaborator

v-vish commented Nov 19, 2024

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
investigating currently looking into the issue
Projects
None yet
Development

No branches or pull requests

2 participants