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

number_of_replicas #483

Merged
merged 3 commits into from
Nov 1, 2022

Conversation

mac2000
Copy link
Contributor

@mac2000 mac2000 commented Oct 18, 2021

motivation: hunting for indices with number_of_replicas being set to zero, which might cause data loss in certain scenarios, need this metric to configure alert

output will be:

# HELP elasticsearch_indices_settings_number_of_replicas index setting number_of_replicas
# TYPE elasticsearch_indices_settings_number_of_replicas gauge
elasticsearch_indices_settings_number_of_replicas{index="foo"} 3
elasticsearch_indices_settings_number_of_replicas{index="bar"} 0

closes #482

Signed-off-by: Alexandr Marchenko <[email protected]>
@mac2000 mac2000 force-pushed the number_of_replicas branch from 785a319 to 2a320bf Compare October 18, 2021 12:27
SuperQ
SuperQ previously requested changes Jan 19, 2022
Copy link
Contributor

@SuperQ SuperQ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor nit, otherwise LGTM.

collector/indices_settings.go Outdated Show resolved Hide resolved
@mac2000 mac2000 requested a review from SuperQ January 26, 2022 17:05
@mac2000
Copy link
Contributor Author

mac2000 commented Jan 26, 2022

sorry for a noob question but seems like I have stuck with this DCO whatever, is there a chance you can press the button or should I figure out how to fix this check?

@rccrdpccl
Copy link

Any chance we can get this merged @SuperQ ? I would love to have this metric exported

@SuperQ
Copy link
Contributor

SuperQ commented Aug 4, 2022

This needs a DCO sign-off. You can use git commit -s --amend to add it.

Co-authored-by: Ben Kochie <[email protected]>
Signed-off-by: Alexandr Marchenko <[email protected]>
@mac2000 mac2000 force-pushed the number_of_replicas branch from 7bcb364 to ed48174 Compare August 5, 2022 05:52
@mac2000
Copy link
Contributor Author

mac2000 commented Aug 5, 2022

@SuperQ thank you, seems like it did the trick

Copy link
Contributor

@sysadmind sysadmind left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The README missed the update of the metric name.

README.md Outdated Show resolved Hide resolved
Thanks indeed

Co-authored-by: Joe Adams <[email protected]>
Signed-off-by: Alexandr Marchenko <[email protected]>
@mac2000 mac2000 force-pushed the number_of_replicas branch from f5fc035 to 9a69bc2 Compare August 9, 2022 08:22
@mac2000 mac2000 requested a review from sysadmind August 9, 2022 08:26
@mac2000
Copy link
Contributor Author

mac2000 commented Oct 24, 2022

@SuperQ @sysadmind is there any chance it will be merged somewhere in the future or should something else be done?

@sysadmind sysadmind dismissed SuperQ’s stale review November 1, 2022 00:47

Review is now stale

@sysadmind sysadmind merged commit 416fa22 into prometheus-community:master Nov 1, 2022
@sysadmind
Copy link
Contributor

LGTM. Thanks!

SuperQ added a commit that referenced this pull request Jun 29, 2023
BREAKING CHANGES:

The flag `--es.cluster_settings` has been renamed to `--collector.clustersettings`.

* [CHANGE] Rename --es.cluster_settings to --collector.clustersettings
* [FEATURE] Add ILM metrics #513
* [ENHANCEMENT] Add ElasticCloud node roles to role label #652
* [ENHANCEMENT] Add ability to use AWS IAM role for authentication #653
* [ENHANCEMENT] Add metric for index replica count #483
* [BUGFIX] Set elasticsearch_clusterinfo_version_info guage to 1 #728
* [BUGFIX] Fix index field counts with nested fields #675

---------

Signed-off-by: Joe Adams <[email protected]>
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.

number_of_replicas
4 participants