Skip to content

Base unit of metric values is unspecified when using the metrics endpoint #13813

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

Closed
wants to merge 1 commit into from

Conversation

joshiste
Copy link
Contributor

closes #13791

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jul 18, 2018
@wilkinsona wilkinsona self-requested a review July 19, 2018 05:41
Copy link
Member

@wilkinsona wilkinsona left a comment

Choose a reason for hiding this comment

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

Thanks for the PR. Can you please update the documentation to describe the new fields?

@snicoll snicoll added the status: waiting-for-feedback We need additional information before we can continue label Jul 19, 2018
@joshiste
Copy link
Contributor Author

@wilkinsona I did the requested changes.

@wilkinsona wilkinsona changed the title Add metric description and base unit to metrics endpoint Base unit of metric values is unspecified when using the metrics endpoint Jul 20, 2018
@wilkinsona wilkinsona added type: bug A general bug and removed status: waiting-for-feedback We need additional information before we can continue status: waiting-for-triage An issue we've not yet triaged labels Jul 20, 2018
@wilkinsona wilkinsona added this to the 2.0.x milestone Jul 20, 2018
@snicoll snicoll self-assigned this Jul 23, 2018
snicoll added a commit that referenced this pull request Jul 24, 2018
* pr/13813:
  Polish "Add metric description and base unit to metrics endpoint"
  Add metric description and base unit to metrics endpoint
@snicoll snicoll closed this in 7176c54 Jul 24, 2018
@snicoll snicoll modified the milestones: 2.0.x, 2.0.4 Jul 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Include a meter id's base unit and description in the metrics endpoint's output
4 participants