Siman-hub commented on PR #787:
URL: https://github.com/apache/skywalking-java/pull/787#issuecomment-3831759920

   > > I've updated the branch. I managed to get the gateway-4.3.x-scenario 
passing locally. The fixes in this push include:
   > > 
   > > 1. **Infrastructure:** Switched the base Docker image to 
eclipse-temurin:17-jdk-jammy (the previous image lacked bash, which was causing 
the runner script to fail silently).
   > 
   > I have a little concern about hard coded to this image only when this 
simple one test scenario. Did not other case require bash?
   The main difference here is the JDK 17 requirement for Spring Cloud Gateway 
4.3.x (via Spring Boot 3), while most other scenarios still run on JDK 8/11.
   It looks like the default JDK 17 images are much more stripped down and 
often don’t include bash, which causes scenario.sh to fail. I pinned 
eclipse-temurin:17-jdk-jammy to ensure a standard Ubuntu user space with the 
tools the runner expects.
   If you’d prefer a different approach like installing bash explicitly or 
aligning this with other scenario, I’m happy to adjust.
   


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