Skip to content

HDDS-10047. Add number of datanodes, total capacity/used space to SCMNodeMetrics#5905

Merged
kerneltime merged 1 commit intoapache:masterfrom
dombizita:HDDS-10047
Jan 2, 2024
Merged

HDDS-10047. Add number of datanodes, total capacity/used space to SCMNodeMetrics#5905
kerneltime merged 1 commit intoapache:masterfrom
dombizita:HDDS-10047

Conversation

@dombizita
Copy link
Contributor

What changes were proposed in this pull request?

I added three new metrics to the SCMNodeMetrics:

  • Number of datanodes
  • Total cluster installed capacity
  • Total current Ozone utilization

With this change these metrics will be available in the SCM's JMX endpoint. This is an addition to the already existing SCMNodeMetrics, which will be useful when we want to gather information about the overall state of the cluster.

https://issues.apache.org/jira/browse/HDDS-10047

How was this patch tested?

I added the new metrics to the already existing tests, green CI: https://github.com/dombizita/ozone/actions/runs/7387348419

Change-Id: I7d66b19b812c9ca9edbeb7c4ff8d5abc8242cbfd
@dombizita dombizita requested a review from sodonnel January 2, 2024 16:24
Copy link
Contributor

@sodonnel sodonnel left a comment

Choose a reason for hiding this comment

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

LGTM

@kerneltime kerneltime merged commit f782b64 into apache:master Jan 2, 2024
kerneltime added a commit that referenced this pull request Jan 2, 2024
@adoroszlai
Copy link
Contributor

Please pay attention when merging PRs. If it has a single commit, GitHub uses the commit's message by default, not the PR title.

jojochuang pushed a commit to jojochuang/ozone that referenced this pull request Feb 1, 2024
…space to SCMNodeMetrics (apache#5905)

(cherry picked from commit dfd5cee)

Change-Id: If69e4d8cec80d9c74b6ad3a239de97befc15a35b
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.

4 participants