Skip to content

improvement(CLDSRV-995): remove obsolete memory backend Dockerfile - #6296

Open
francoisferrand wants to merge 1 commit into
development/9.5from
cldsrv-995-node-24
Open

francoisferrand wants to merge 1 commit into
development/9.5from
cldsrv-995-node-24

Conversation

@francoisferrand

Copy link
Copy Markdown
Contributor

Description

Remove the unused DockerfileMem, which still used the unsupported Node.js 6 image and is not referenced by current build, CI, or compose configuration.

Validation

git diff --check

Issue: CLDSRV-995

@francoisferrand
francoisferrand changed the base branch from development/9.0 to development/9.5 September 20, 2026 17:27
@codecov

codecov Bot commented Sep 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.51%. Comparing base (e552011) to head (0444881).
✅ All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph
see 1 file with indirect coverage changes

@@                 Coverage Diff                 @@
##           development/9.5    #6296      +/-   ##
===================================================
- Coverage            86.56%   86.51%   -0.05%     
===================================================
  Files                  213      213              
  Lines                14621    14621              
===================================================
- Hits                 12657    12650       -7     
- Misses                1964     1971       +7     
Flag Coverage Δ
checksums-disabled-tests 35.37% <ø> (ø)
file-ft-tests 70.02% <ø> (-0.05%) ⬇️
file-ft-tests-null-compat 70.50% <ø> (ø)
kmip-ft-tests 28.14% <ø> (ø)
mongo-v0-ft-tests 71.11% <ø> (ø)
mongo-v1-ft-tests 71.11% <ø> (ø)
multiple-backend 36.12% <ø> (ø)
s3c-ft-tests-v0 65.04% <ø> (ø)
s3c-ft-tests-v0-null-compat 65.09% <ø> (ø)
s3c-ft-tests-v1 65.03% <ø> (+0.01%) ⬆️
sur-tests 36.63% <ø> (-0.03%) ⬇️
sur-tests-inflights 39.46% <ø> (-0.03%) ⬇️
unit 74.31% <ø> (ø)
utapi-v2-tests 35.35% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Comment thread DockerfileMem
@@ -1,22 +0,0 @@
FROM node:6-slim

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues — this removes an obsolete Dockerfile referencing unsupported Node.js 6, which is correctly not referenced by any current build, CI, or compose config.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant