Skip to content

Conversation

@maobaolong
Copy link
Member

@maobaolong maobaolong commented Jun 28, 2020

What changes were proposed in this pull request?

We need a Datanode home page. After this, we can do more effort to see the starttime version, build info and more information about hddsdatanode service.

What is the link to the Apache JIRA

HDDS-3885

How was this patch tested?

start datanode and see the root page.

http://localhost:9882/

image

@maobaolong maobaolong force-pushed the dnui branch 2 times, most recently from 7d0b209 to 9bb9583 Compare June 28, 2020 03:52
@maobaolong
Copy link
Member Author

@adoroszlai Please take a look at this PR, thanks.

Copy link
Contributor

@adoroszlai adoroszlai left a comment

Choose a reason for hiding this comment

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

Thanks @maobaolong for working on this.

Do you know why Rpc Metrics page is empty for datanode?

@adoroszlai adoroszlai requested review from elek and vivekratnavel June 30, 2020 08:05
@maobaolong
Copy link
Member Author

@adoroszlai Thanks for your review. AFAIK, the reason why Rpc Metrics page is empty for datanode is that datanode has no rpc service.PTAL.

@maobaolong maobaolong requested a review from adoroszlai June 30, 2020 11:48
@adoroszlai
Copy link
Contributor

Thanks @maobaolong for updating the patch. Let's wait for others to take a look.

@maobaolong
Copy link
Member Author

@adoroszlai Thank you for your review.

@maobaolong
Copy link
Member Author

@xiaoyuyao Please take a look at this PR, if we take this PR into 0.6.0, it would be better, thanks.

@maobaolong
Copy link
Member Author

@ChenSammi Thank you for correct me, PTAL.

@ChenSammi
Copy link
Contributor

Tested it locally. +1. Thanks @maobaolong for the contribution.

@ChenSammi ChenSammi merged commit cbd75cb into apache:master Jul 15, 2020
ChenSammi pushed a commit that referenced this pull request Jul 22, 2020
rakeshadr pushed a commit to rakeshadr/hadoop-ozone that referenced this pull request Sep 3, 2020
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.

3 participants