BobSong-dev opened a new pull request, #7151:
URL: https://github.com/apache/shenyu/pull/7151

   Fixes #6818
   
   ## Background
   
   The storage Compose scripts ignored failures, left containers running after 
failures, and relied on fixed 30-second and 60-second sleeps. This could let 
storage initialization or service startup fail silently and made the E2E timing 
dependent on machine speed. The Compose files already define service 
healthchecks, but the scripts did not wait on them directly.
   
   ## Changes
   
   - Enable set -euo pipefail in the four CI storage scripts and the storage 
matrix script;
   - Replace fixed sleeps and the invalid local healthcheck helper calls with 
docker compose up --wait;
   - Create the external shenyu network idempotently before startup;
   - Add EXIT cleanup traps and clean up each storage iteration in the matrix 
script.
   
   ## Verification
   
   - Git Bash bash -n: all five storage scripts passed;
   - docker compose config --quiet: all four storage Compose files passed;
   - docker compose up --help: the installed CLI supports --wait;
   - git diff --check: passed;
   - Docker E2E execution was not run because the local Docker daemon was 
unavailable.
   - Compose config emitted the existing warning that the version attribute is 
obsolete.


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