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

FR: Add maxclients in INFO and /metrics #2912

Closed
steache opened this issue Apr 17, 2024 · 3 comments
Closed

FR: Add maxclients in INFO and /metrics #2912

steache opened this issue Apr 17, 2024 · 3 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@steache
Copy link

steache commented Apr 17, 2024

Did you search GitHub Issues and GitHub Discussions First?
y

Is your feature request related to a problem? Please describe.
n

Describe the solution you'd like

Describe alternatives you've considered

  • not doing it :)

Additional context

  • we are pushing dragonfly to production, but in order for us to properly monitor it, we need few more informations :)
  • alternative solution for this is using the default (64000) maxclients / the value set via --maxclients flag per each cluster
  • it's not critical, it just would be 10x times easier for everyone implementing alerting rules :)
  • we've too faced problem, when trying to determine health of cluster (master+slave), because there is no metric-label defining cluster, so we've come up with this funky relabeling rule which fills dragonfly_cluster with first part from (redis-1), (test-2) etc. pod name
  • then you can for example check number of masters per cluster etc. (https://github.com/steache/dragonflydb-alerting-rules)
      relabelings:
        - sourceLabels: [pod]
          separator: "-"
          regex: "(.*)-(.*)"
          targetLabel: dragonfly_cluster
          replacement: "$1"
  • thank you for considering this :)
  • I've tried to implement it myself and send you patch, but I'm absolute c++ noob, so it went sideways (main...steache:dragonfly:main)
@romange romange added enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed labels Apr 17, 2024
@kostasrim
Copy link
Contributor

Hi @steache, thank you for reporting this. We added as an issue for the community but if it remains stale I will take care of it

@racamirko
Copy link
Contributor

@kostasrim I'm volunteering. I'll push the PR and note it here once I run the test suite.

racamirko added a commit to racamirko/dragonfly that referenced this issue Apr 21, 2024
Signed-off-by: Mirko Raca <racamirko@gmail.com>
romange pushed a commit that referenced this issue Apr 21, 2024
…2940)

feat: adding max_clients to metrics and info output (#2912)

Signed-off-by: Mirko Raca <racamirko@gmail.com>
@kostasrim
Copy link
Contributor

@racamirko Thank you for your contribution :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants