Skip to content

Conversation

kingsword09
Copy link
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
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
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