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

Diagnostics page reports 0 weight mapping as 100 #5811

Open
buckner opened this issue Dec 13, 2024 · 0 comments
Open

Diagnostics page reports 0 weight mapping as 100 #5811

buckner opened this issue Dec 13, 2024 · 0 comments
Labels
t:bug Something isn't working

Comments

@buckner
Copy link

buckner commented Dec 13, 2024

Describe the bug
If I define a mapping to have a weight of 0 it is displayed in the route table of the diagnostics page ([host]/ambassador/v0/diag/) as having a weight of 100%:
image
image
I've tested it and the weight is actually 0 despite what it says in the GUI

To Reproduce

Define two mappings:

---
apiVersion: getambassador.io/v2
kind: Mapping
metadata:
  name: nginx-primary
  namespace: buckner
spec:
  prefix: /
  service: nginx-primary.buckner
---
apiVersion: getambassador.io/v2
kind: Mapping
metadata:
  name: nginx-canary
  namespace: buckner
spec:
  prefix: /
  service: nginx-canary.buckner
  weight: 0

Now check the route table in the diagnostic page and it will show both at 100% instead of the 0-weight mapping as 0%
Expected behavior
I expect it to show the correct percentage. If I modify the weight to anything over 0 it works correctly:
image

Versions (please complete the following information):

  • Ambassador: 3.9.1
  • Kubernetes environment: Google Kubernetes Engine
  • Version: 1.30.5

Additional context
Add any other context about the problem here.

@dosubot dosubot bot added the t:bug Something isn't working label Dec 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
t:bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant