You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@apisix.apache.org by GitBox <gi...@apache.org> on 2020/12/10 01:40:51 UTC

[GitHub] [apisix-dashboard] nic-chen opened a new pull request #1000: fix: backend e2e test failed

nic-chen opened a new pull request #1000:
URL: https://github.com/apache/apisix-dashboard/pull/1000


   Please answer these questions before submitting a pull request
   
   - Why submit this pull request?
   - [x] Bugfix
   - [ ] New feature provided
   - [ ] Improve performance
   
   - Related issues
   
   close #998 
   
   ___
   ### Bugfix
   - Description
   
   In `route_service_upstream_test.go`, r2 is used as the route ID. As a result, r1 and r2 may coexist, and the same uri is used, which may produce different results
   
   
   - How to fix?
   
   Route always use the same ID, and confirm that the uri has not been created before use
   
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [apisix-dashboard] moonming merged pull request #1000: fix: fix router name conflict in route_service_upstream_test.go

Posted by GitBox <gi...@apache.org>.
moonming merged pull request #1000:
URL: https://github.com/apache/apisix-dashboard/pull/1000


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org