Skip to content

Add sentinel and cluster support - #101

Open
TinOo512 wants to merge 11 commits into
logstash-plugins:mainfrom
OwletCare:add-sentinel-cluster-support
Open

TinOo512 wants to merge 11 commits into
logstash-plugins:mainfrom
OwletCare:add-sentinel-cluster-support

Conversation

@TinOo512

@TinOo512 TinOo512 commented Jun 15, 2026 •

Copy link
Copy Markdown

Add sentinel and cluster support to the plugin. Needed to update redis client to v5 in order to have a decent support of the cluster feature specially around evalsha required for the lua script.

Tackle issues: #3 , #10, #54 and #99

@cla-checker-service

cla-checker-service Bot commented Jun 15, 2026 •

Copy link
Copy Markdown

💚 CLA has been signed

@github-actions

Copy link
Copy Markdown

Changelog and Version Management

No changelog or version changes detected. You can either update them manually or use a comment command to update them automatically on merge:

/pr_prepare release:<patch|minor|major> — bump version + update changelog
/pr_prepare changelog:add — changelog entry only (no version bump)

/pr_prepare release:patch
- Add the changelog entry here
  - (Additional details can be provided in multiple lines, respecting markdown formatting)

Omit the entry line to use the PR title. If multiple commands are posted, the last one wins.

@TinOo512
TinOo512 force-pushed the add-sentinel-cluster-support branch 22 times, most recently from 894ea41 to 1c36f9c Compare June 16, 2026 21:38
@TinOo512 TinOo512 changed the title Add sentinel cluster support Add sentinel and cluster support Jun 16, 2026
@TinOo512
TinOo512 force-pushed the add-sentinel-cluster-support branch from 1c36f9c to d749ef4 Compare June 16, 2026 21:53
@robbavey

Copy link
Copy Markdown
Member

@TinOo512 Would you mind signing the CLA please

@TinOo512

TinOo512 commented Jul 7, 2026

Copy link
Copy Markdown
Author

@robbavey signed.

@TinOo512

Copy link
Copy Markdown
Author

Any update?

@TinOo512
TinOo512 force-pushed the add-sentinel-cluster-support branch from b549236 to 5d5615b Compare July 27, 2026 12:23
@TinOo512

Copy link
Copy Markdown
Author

I rebased on top of master and I made my previous .travis.yml changes to .ci/docker-compose.override.yml

@TinOo512
TinOo512 force-pushed the add-sentinel-cluster-support branch from bec48f3 to b92ddfb Compare September 24, 2026 08:53
@TinOo512

Copy link
Copy Markdown
Author

Just rebased on top of master. Any update on the review?

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.

2 participants