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

Add top level metrics document to summarize monitoring endpoints #5923

Merged
merged 1 commit into from May 15, 2018

Conversation

nitisht
Copy link
Contributor

@nitisht nitisht commented May 11, 2018

Description

Minio server supports healthcheck and prometheus related
unauthenticated endpoints. This document summarizes this
information in a single place and add links for more detailed
documentation if needed.

Motivation and Context

Detailed Prometheus document is available in Cookbook by
minio/docs#215.

See #5914

How Has This Been Tested?

NA

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • 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 updated the documentation accordingly.
  • I have added unit tests to cover my changes.
  • I have added/updated functional tests in mint. (If yes, add mint PR # here: )
  • All new and existing tests passed.

@codecov
Copy link

codecov bot commented May 11, 2018

Codecov Report

Merging #5923 into master will increase coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5923      +/-   ##
==========================================
+ Coverage   59.57%   59.61%   +0.03%     
==========================================
  Files         214      214              
  Lines       30786    30786              
==========================================
+ Hits        18342    18354      +12     
+ Misses      10899    10886      -13     
- Partials     1545     1546       +1
Impacted Files Coverage Δ
cmd/update-notifier.go 81.66% <0%> (+19.99%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5c21e89...c5b4642. Read the comment docs.

@deekoder
Copy link
Contributor

we have to approve cookbook pr first. Change the url and then delete the doc from here

Minio server supports healthcheck and prometheus related
unauthenticated endpoints. This document summarizes this
information in a single place and add links for more detailed
documentation if needed.
@nitisht nitisht changed the title Remove Prometheus document from docs Add top level metrics document to summarize monitoring endpoints May 15, 2018
@deekoder deekoder merged commit 9cab0f2 into minio:master May 15, 2018
@nitisht nitisht deleted the prometheus branch June 26, 2018 04:12
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.

None yet

3 participants