FrankChen021 opened a new pull request, #20368:
URL: https://github.com/apache/druid/pull/20368

   ### Description
   
   The Docker image build downloads `bash-static` from a GitHub release with a 
single `wget` attempt. A temporary connection reset therefore fails the entire 
Docker test job.
   
   For example, [this Docker test 
job](https://github.com/apache/druid/actions/runs/35176848274/job/105060323072) 
failed while following the GitHub release redirect:
   
   ```text
   wget: error getting response: Connection reset by peer
   ```
   
   This change installs `curl` in the intermediate Alpine stage and downloads 
`bash-static` with redirect handling, HTTP error detection, and three retries 
separated by five seconds. Persistent failures still fail the image build. The 
final Druid image is unchanged because `curl` remains confined to the 
intermediate stage.
   
   #### Release note
   
   None. This change only improves Docker build reliability.
   
   <hr>
   
   ##### Key changed/added classes in this PR
   
   * `distribution/docker/Dockerfile`
   
   <hr>
   
   This PR has:
   
   - [x] been self-reviewed.
   - [x] added comments explaining the "why" and the intent of the code 
wherever would not be obvious for an unfamiliar reader.
   
   Validation:
   
   - `git diff --check`
   - Verified the exact `curl` command downloads the referenced GitHub release 
asset and follows its redirect.
   - Full Docker-stage validation was not run locally because Docker is 
unavailable in the local environment.
   


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to