Skip to content

Prevent redis pool to initialize in unit test #43774

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Prevent redis pool to initialize in unit test #43774

wants to merge 1 commit into from

Conversation

hezean
Copy link
Contributor

@hezean hezean commented Jan 10, 2025

As 52d419a bumps spring-data bom to 2024.1.2-SNAPSHOT, an unconditional preparePool invocation is added to the start lifecycle of JedisConnectionFactory (see: spring-projects/spring-data-redis@2bcdd36). That is, with min-idle set to 1, the pool will attempt to create a socket over the dummy domain foo, failing the bean creation.

Related to:

spring-projects/spring-data-redis@b7381ac introduces an unconditionally preparePool invocation when starting JedisConnectionFactory/LettuceConnectionFactory, causing it attempts to create a socket to the dummy domain name `foo`.

Signed-off-by: He Zean <[email protected]>
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jan 10, 2025
@snicoll snicoll changed the title Fix Redis configuration with pool test failure Prevent redis pool to initalizer in unit test Jan 11, 2025
@snicoll snicoll added type: task A general task and removed status: waiting-for-triage An issue we've not yet triaged labels Jan 11, 2025
@snicoll snicoll self-assigned this Jan 11, 2025
@snicoll snicoll added this to the 3.3.8 milestone Jan 11, 2025
@snicoll snicoll changed the title Prevent redis pool to initalizer in unit test Prevent redis pool to initialize in unit test Jan 11, 2025
snicoll pushed a commit that referenced this pull request Jan 11, 2025
@snicoll snicoll closed this in 9a10c1a Jan 11, 2025
@snicoll
Copy link
Member

snicoll commented Jan 11, 2025

@hezean thanks very much.

@hezean hezean deleted the redis-pool branch January 11, 2025 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: task A general task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants