Skip to content

Adds sidekiq config to define logger level as WARN#275

Merged
edimossilva merged 1 commit intomainfrom
fix-warnings-redis
Apr 14, 2025
Merged

Adds sidekiq config to define logger level as WARN#275
edimossilva merged 1 commit intomainfrom
fix-warnings-redis

Conversation

@mattfeg
Copy link
Collaborator

@mattfeg mattfeg commented Apr 14, 2025

Fix #258

  • Context
    When running rspec we got the following output:
2025-04-05T11:53:47.468Z pid=90 tid=4ua INFO: Sidekiq 7.2.4 connecting to Redis with options {:size=>10, :pool_name=>"internal", :url=>"redis://redis:6379/0"}
  • Solution
    "We log that line because connecting correctly to Redis is the number one support issue with Sidekiq. The user should see when Sidekiq tries to connect and the parameters it is using to connect."

On specs config, Define Sidekiq logger level as Warning.

sidekiq/sidekiq#6181

@mattfeg mattfeg self-assigned this Apr 14, 2025
@edimossilva edimossilva merged commit 8fbd03c into main Apr 14, 2025
3 checks passed
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.

Check/fix connection with redis during specs

2 participants