Skip to content

Cluster adapter provides count of manager-eligible nodes - #26836

Merged
moesterheld merged 2 commits into
masterfrom
feature/opensearch-manager-nodes-count-health-reporter
Aug 5, 2026
Merged

Cluster adapter provides count of manager-eligible nodes#26836
moesterheld merged 2 commits into
masterfrom
feature/opensearch-manager-nodes-count-health-reporter

Conversation

@todvora

@todvora todvora commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

/nocl this is just infrastructure for https://github.com/Graylog2/graylog-plugin-enterprise/pull/15041

Description

Implement access to manager-eligible nodes count in all three storage modules.

Motivation and Context

Needed for https://github.com/Graylog2/graylog-plugin-enterprise/pull/15041

How Has This Been Tested?

Added unit test

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactoring (non-breaking change)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have requested a documentation update.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.

@todvora
todvora requested a review from moesterheld August 3, 2026 13:15

@moesterheld moesterheld left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm. Thank you

@moesterheld
moesterheld merged commit d181670 into master Aug 5, 2026
26 checks passed
@moesterheld
moesterheld deleted the feature/opensearch-manager-nodes-count-health-reporter branch August 5, 2026 06:50
patrickmann added a commit that referenced this pull request Aug 5, 2026
Resolves the conflict in ClusterAdapterOS.java: master #26836 added
countOfClusterManagerEligibleNodes() at the same insertion point where this
branch added the bounded clusterHealth(Duration). The two methods are
orthogonal; kept both.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
patrickmann added a commit that referenced this pull request Aug 5, 2026
Resolves the conflict in ClusterAdapterOSTest.java: master #26836 added the
testManagerEligibleNodesCount test at the same slot where this branch added
nodesStatsParsesPerNodeCpuAndHeapPercent. Independent tests; kept both (their
production targets nodesStats()/countOfClusterManagerEligibleNodes() both
survive the auto-merge).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@todvora
todvora restored the feature/opensearch-manager-nodes-count-health-reporter branch August 5, 2026 11:58
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