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.

Sign up for GitHub

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

Jump to bottom

Add NDMP metrics #2498

Closed
schumijo opened this issue Nov 21, 2023 · 4 comments · Fixed by #2531 or #2667
Closed

Add NDMP metrics #2498

schumijo opened this issue Nov 21, 2023 · 4 comments · Fixed by #2531 or #2667
Assignees
@rahulguptajss
Labels
24.02 customer feature New feature or request status/done

Comments

@schumijo
Copy link

schumijo commented Nov 21, 2023 •
edited

Describe the solution you'd like
Can you add NDMP metrics from REST API endpoint /api/protocols/ndmp/sessions ?

Additional context
Could be interesting to monitor number of concurrent sessions, statistics about performances.....
@schumijo schumijo added the feature New feature or request label Nov 21, 2023
@cgrinds cgrinds added the 24.02 label Nov 28, 2023
@Hardikl Hardikl linked a pull request Dec 5, 2023 that will close this issue
feat: adding ndmp session rest template #2531
Merged
@Hardikl
Copy link
Contributor

Hardikl commented Dec 7, 2023

@schumijo

This is list of fields returned from Rest call /api/protocols/ndmp/sessions, and I have added one PR in draft stage which have few of these counters.
image

Could you please share which are the counters would be helpful for you for performance statistics.

@schumijo
Copy link
Author

schumijo commented Dec 7, 2023

Hi @Hardikl

Thank you for you answer
I am mainly interested by the number of sessions and with SVM, UUID and data.operation

@Hardikl
Copy link
Contributor

Hardikl commented Feb 6, 2024

@schumijo NDMP session rest template is now available via nightly builds. By default this template is disabled. You can uncomment here and let us know your feedback.

@rahulguptajss rahulguptajss linked a pull request Feb 14, 2024 that will close this issue
fix: align template name with object name for ndmp #2667
Merged
@rahulguptajss
Copy link
Contributor

verified in main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rahulguptajss rahulguptajss

Labels
24.02 customer feature New feature or request status/done
Projects
None yet
Milestone
No milestone
4 participants
@cgrinds @rahulguptajss @schumijo @Hardikl