Skip to content

ci: switches memcached image to bitnamisecure image#6583

Draft
kingsword09 wants to merge 5 commits intomainfrom
migrate-memcached
Draft

ci: switches memcached image to bitnamisecure image#6583
kingsword09 wants to merge 5 commits intomainfrom
migrate-memcached

Conversation

@kingsword09
Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Related #6435.

Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

@kingsword09 kingsword09 requested a review from Xuanwo as a code owner September 22, 2025 07:24
@dosubot dosubot Bot added size:S This PR changes 10-29 lines, ignoring generated files. releases-note/ci The PR modifies CI-related content or has a title that begins with "ci" labels Sep 22, 2025
@kingsword09 kingsword09 marked this pull request as draft September 22, 2025 07:37
Comment on lines +29 to +40
- name: Print memcached logs on failure
if: failure()
shell: bash
working-directory: fixtures/memcached
run: |
echo "==== docker ps -a ===="
docker ps -a
echo "==== docker logs memcached-memcached-1 ===="
docker logs memcached-memcached-1 || true
echo "==== docker compose logs ===="
docker compose -f docker-compose-memcached-with-auth.yml logs || true

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Do we need to to remove this?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

There still seems to be a problem with the bitnamisecure/memcached image, but I haven't figured out exactly what's going on, the authentication doesn't seem to be reporting errors anymore, but the tests are timing out a ton!

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

Labels

releases-note/ci The PR modifies CI-related content or has a title that begins with "ci" size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants