Aias00 opened a new pull request, #6374:
URL: https://github.com/apache/shenyu/pull/6374

   ## Summary
   - stabilizes `SentinelPluginTest` by triggering the rate-limit path with 
concurrent requests
   - asserts over observed result sets instead of relying on two sequential 
requests landing in the same Sentinel QPS window
   - targets the `build (shenyu-integrated-test-http)` failure seen after #6361 
on master
   
   ## Root Cause
   The master push for #6361 failed in `build (shenyu-integrated-test-http)` at 
`Check test result`. The modified Sentinel integration coverage used sequential 
requests to expect one pass and one limited/fallback response. On a loaded CI 
runner, those requests can cross Sentinel's one-second QPS window, making the 
assertion timing-dependent.
   
   ## Verification
   - `./mvnw -f shenyu-integrated-test/pom.xml -pl shenyu-integrated-test-http 
-am -DskipTests test-compile`
   - `git diff --check`
   
   ## Not Tested
   - Full local Docker `integrated-test-http` run; the CI-built Docker images 
and dependent services were not present locally.


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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to