CLDSRV-717: Update sproxyd container#5897
Conversation
Use latest sproxyd with latest configs changes from federation And add sproxyd logs into artifacts The latest sproxyd has: - adminapi for monitoring (but ignored here) - disable FUSE S3C-4129 scality/Federation#5653 - nginx openresty replaces tengine S3C-8479 scality/Federation#5495
Hello bourgoismickael,My role is to assist you with the merge of this Available options
Available commands
Status report is not available. |
Request integration branchesWaiting for integration branch creation to be requested by the user. To request integration branches, please comment on this pull request with the following command: Alternatively, the |
|
/create_integration_branches |
Integration data createdI have created the integration data for the additional destination branches.
The following branches will NOT be impacted:
You can set option The following options are set: create_integration_branches |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
The following options are set: create_integration_branches |
There was a problem hiding this comment.
Pull Request Overview
This PR updates the sproxyd container from version 7.10.6.8 to 9.5.1.0 and aligns configurations with the latest federation changes, specifically replacing nginx/tengine with openresty and disabling FUSE functionality.
Key changes:
- Update sproxyd container version to 9.5.1.0
- Replace nginx with openresty in supervisord configuration
- Add adminapi support and disable FUSE mounting in sproxyd command
- Remove FUSE setup steps from CI and add sproxyd logs to artifacts
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| .github/docker/sproxyd/Dockerfile | Updates base image version and simplifies configuration copying |
| .github/docker/sproxyd/conf/supervisord.conf | Switches from nginx to openresty and adds new sproxyd flags |
| .github/docker/sproxyd/conf/nginx.conf | Restructures configuration for openresty compatibility |
| .github/docker/docker-compose.yaml | Adds volume mapping for sproxyd logs collection |
| .github/actions/setup-ci/action.yaml | Removes FUSE-related setup steps |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## development/9.0 #5897 +/- ##
================================================
Coverage 83.19% 83.19%
================================================
Files 188 188
Lines 12091 12091
================================================
Hits 10059 10059
Misses 2032 2032
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
|
/approve |
|
I have successfully merged the changeset of this pull request
The following branches have NOT changed:
Please check the status of the associated issue CLDSRV-717. Goodbye bourgoismickael. The following options are set: approve, create_integration_branches |
Use latest sproxyd with latest configs changes from federation
And add sproxyd logs into artifacts
The latest sproxyd has:
S3C-4129 https://github.com/scality/Federation/pull/5653
S3C-8479 https://github.com/scality/Federation/pull/5495