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

Typo in $graphite->send() #82

Closed
vmdude opened this issue Jun 23, 2016 · 0 comments
Closed

Typo in $graphite->send() #82

vmdude opened this issue Jun 23, 2016 · 0 comments
Assignees
Labels

Comments

@vmdude
Copy link
Member

vmdude commented Jun 23, 2016

The $graphite->send() path parameter should not have trailing dot as it's added by default

For instance instead of

$graphite->send(path => "vi.", data => $vcenter_exec_duration_h)

We should use

$graphite->send(path => "vi", data => $vcenter_exec_duration_h)

If not, when enabling graphite trace, we see

vmw..xxx.quickstats.cpu.demand 102206 1466681776
vmw..xxx.quickstats.cpu.distributedCpuEntitlement 88651 1466681776
vmw..xxx.quickstats.cpu.effective 457488 1466681776
vmw..xxx.quickstats.cpu.staticCpuEntitlement 453552 1466681776
@vmdude vmdude added the bug label Jun 23, 2016
@vmdude vmdude added this to the 0.99d - Nova Prospekt milestone Jun 23, 2016
@vmdude vmdude self-assigned this Jun 23, 2016
rschitz added a commit that referenced this issue Aug 29, 2016
+ vm.CpuUtilization
#82
@vmdude vmdude closed this as completed Jun 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant