Skip to content
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

feat: Add Redis Sentinel connection support #191

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

GSokol
Copy link

@GSokol GSokol commented Nov 16, 2020

Description

Add support for Redis Sentinel via the following options:

  • --url sentinel://:mypass@sentinel.01:26379
  • --sentinelExtraEndpoints sentinel.02:26379 sentinel.03:26379 --, --sentinel-extra-endpoints sentinel.02:26379 sentinel.03:26379 --sentinel.03:26379 --`
  • --sentinelMasterName myredis, --sentinel-master-name myredis

Add support of the new configuration options via the environment variables as well:

  • EXPORTER_SENTINEL_EXTRA_ENDPOINTS
  • EXPORTER_SENTINEL_MASTER_NAME

This is a

  • Bug Fix
  • Feature
  • Documentation
  • Other

Checklists

Commit style

  • Changes are on a branch with a descriptive name eg. fix/missing-queue, docs/setup-guide

  • Commits start with one of feat: fix: docs: chore: or similar

  • No excessive commits, eg: there should be no fix: commits for bugs that existed only on the PR branch (see guide-to-interactive-rebasing)

Protected files

The following files should not change unless they are directly a part of your change.

  • yarn.lock (unless package.json is also modified, then only the new/updated package should be changed here)

  • package.json (renovate bot should handle all routine updates)

  • package-lock.json (Should not exist as this project uses yarn)

  • tsconfig.json (only make it stricter, making it more lenient requires more discussion)

  • tslint.json (only make it stricter, making it more lenient requires more discussion)

Add support for Redis Sentinel via the following options:
- `--url sentinel://:mypass@sentinel.01:26379`
- `--sentinelExtraEndpoints sentinel.02:26379 sentinel.03:26379 --`
- `--sentinelMasterName myredis`
Add support for `--sentinel-extra-endpoints` and
`--sentinel-master-name` via the environment variables:
- `EXPORTER_SENTINEL_EXTRA_ENDPOINTS`
- `EXPORTER_SENTINEL_MASTER_NAME`
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.

1 participant