Skip to content

Conversation

bzp2010
Copy link
Collaborator

@bzp2010 bzp2010 commented Aug 24, 2025

Description

I implemented inline upstream splitting in #319, but there seem to be some issues with the update and delete logic, specifically:

  1. The split upstream may not be updated correctly.
  2. When only the inline upstream is updated in the service, the service conf version is incorrectly increased.

This PR thoroughly reorganized this section and optimized the code.

Checklist

  • 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

@bzp2010 bzp2010 added the test/apisix-standalone Trigger the APISIX standalone test on the PR label Aug 24, 2025
@bzp2010 bzp2010 merged commit fa15268 into main Aug 25, 2025
37 of 39 checks passed
@bzp2010 bzp2010 deleted the bzp/fix-bas-inline-upstream-update-delete branch August 25, 2025 01:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test/apisix-standalone Trigger the APISIX standalone test on the PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants