Skip to content

Comments

chore(ci): migrate etcd, memcached, redis, and redis-cluster images to new sources#6435

Merged
Xuanwo merged 11 commits intoapache:mainfrom
kingsword09:chore-ci-migrate
Sep 2, 2025
Merged

chore(ci): migrate etcd, memcached, redis, and redis-cluster images to new sources#6435
Xuanwo merged 11 commits intoapache:mainfrom
kingsword09:chore-ci-migrate

Conversation

@kingsword09
Copy link
Contributor

@kingsword09 kingsword09 commented Jul 18, 2025 •
edited
Loading

Which issue does this PR close?

Closes #6434.

Rationale for this change

Bitnami will restrict free access to most images after August 28th, 2025. This PR migrates all affected images to maintained or legacy sources to ensure future compatibility and CI stability.

What changes are included in this PR?

  • Migrate etcd from bitnami/etcd to gcr.io/etcd-development/etcd
  • Migrate memcached from bitnami/memcached to bitnamisecure/memcached
  • Migrate redis and redis-cluster from bitnami/redis(:7.0) and bitnami/redis-cluster(:7.0) to bitnamilegacy/redis(:7.0) and bitnamilegacy/redis-cluster(:7.0)
  • Update docker-compose and related configuration files accordingly

Are there any user-facing changes?

@kingsword09 kingsword09 requested a review from Xuanwo as a code owner July 18, 2025 23:19
@kingsword09 kingsword09 marked this pull request as draft July 18, 2025 23:19
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. releases-note/ci The PR modifies CI-related content or has a title that begins with "ci" labels Jul 18, 2025
@kingsword09
Copy link
Contributor Author

kingsword09 commented Jul 19, 2025

The new image ‎bitnamisecure/memcached has an issue where enabling authentication results in the error ‎saslpasswd2: command not found, causing the container to exit. This problem has already been reported bitnami/containers Issue #83591. We need to wait for this to be fixed before proceeding.

@kingsword09 kingsword09 force-pushed the chore-ci-migrate branch 3 times, most recently from d113ee7 to 30d0eb2 Compare August 14, 2025 08:09
@kingsword09
Copy link
Contributor Author

kingsword09 commented Aug 14, 2025

Old Image New Image Migration Notes
bitnami/memcached bitnamisecure/memcached bitnami to bitnamisecure
bitnami/etcd:latest gcr.io/etcd-development/etcd:v3.6.4 Official image
docker.io/bitnami/redis:7.0 docker.io/bitnamilegacy/redis:7.4.3-debian-12-r0 bitnami to bitnamilegacy
docker.io/bitnami/redis-cluster:7.0 docker.io/bitnamilegacy/redis-cluster:7.4.3-debian-12-r0 bitnami to bitnamilegacy

@Xuanwo I’m not sure how best to resolve:

  • etcd TLS migration issue:
    The official etcd image does not support the ‎ETCD_ROOT_PASSWORD environment variable, so we are using ‎docker.io/bitnamilegacy/etcd:3.6.4-debian-12-r0 instead.
  • Memcached authentication issue:
    When enabling authentication with the new memcached image, the error ‎saslpasswd2: command not found occurs, so we switched to ‎docker.io/bitnamilegacy/memcached:1.6.39-debian-12-r0.
  • Legacy image maintenance:
    Images from ‎bitnamilegacy will not receive future updates, which may affect long-term maintenance and security, since there seem to be very few actively maintained alternatives available for etcd and memcached..

I’m not sure how to address these problems and would appreciate your suggestions or guidance on possible solutions.

@kingsword09 kingsword09 marked this pull request as ready for review August 22, 2025 07:21
@Xuanwo
Copy link
Member

Xuanwo commented Sep 2, 2025

bitnamilegacy looks good to me so far. Can you create an issue to upstream about the authentication and tls issues? I think this PR itself is good to go.

Xuanwo
Xuanwo approved these changes Sep 2, 2025
View reviewed changes
Copy link
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Thank you for working on this, really appreciated!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Sep 2, 2025
@Xuanwo Xuanwo merged commit 86e7768 into apache:main Sep 2, 2025
39 checks passed
@kingsword09
Copy link
Contributor Author

kingsword09 commented Sep 2, 2025 •
edited
Loading

bitnamilegacy looks good to me so far. Can you create an issue to upstream about the authentication and tls issues? I think this PR itself is good to go.

memcached issue: bitnami/containers#83591
etcd discussions: etcd-io/etcd#20601

Waiting for upstream fixes.

@kingsword09 kingsword09 deleted the chore-ci-migrate branch September 2, 2025 12:15
@kingsword09 kingsword09 mentioned this pull request Sep 4, 2025
Merged
@kingsword09 kingsword09 mentioned this pull request Sep 22, 2025
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Xuanwo Xuanwo Xuanwo approved these changes

Assignees

No one assigned

Labels

lgtm This PR has been approved by a maintainer releases-note/ci The PR modifies CI-related content or has a title that begins with "ci" size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Upcoming changes to Bitnami images and charts

2 participants

@kingsword09 @Xuanwo