Skip to content

fix(ai-proxy-multi): dead loop when retrying#12012

Merged
shreemaan-abhishek merged 3 commits into
apache:masterfrom
shreemaan-abhishek:ai-proxy-bal-count
Mar 3, 2025
Merged

fix(ai-proxy-multi): dead loop when retrying#12012
shreemaan-abhishek merged 3 commits into
apache:masterfrom
shreemaan-abhishek:ai-proxy-bal-count

Conversation

@shreemaan-abhishek

@shreemaan-abhishek shreemaan-abhishek commented Mar 1, 2025

Copy link
Copy Markdown
Contributor

Description

The number of retries is counted on ctx.ai_balancer_try_count, the variable name mismatch will lead to infinite retry loop. This has been fixed.

  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible (If not, please discuss on the APISIX mailing list first)

@shreemaan-abhishek
shreemaan-abhishek marked this pull request as ready for review March 1, 2025 09:43
@dosubot dosubot Bot added size:M This PR changes 30-99 lines, ignoring generated files. bug Something isn't working labels Mar 1, 2025
Comment thread apisix/plugins/ai-proxy-multi.lua
@shreemaan-abhishek
shreemaan-abhishek merged commit 752d939 into apache:master Mar 3, 2025
@shreemaan-abhishek
shreemaan-abhishek deleted the ai-proxy-bal-count branch March 3, 2025 03:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants