{"action":"created","issue":{"url":"https://api.github.com/repos/EventStore/EventStore/issues/521","labels_url":"https://api.github.com/repos/EventStore/EventStore/issues/521/labels{/name}","comments_url":"https://api.github.com/repos/EventStore/EventStore/issues/521/comments","events_url":"https://api.github.com/repos/EventStore/EventStore/issues/521/events","html_url":"https://github.com/EventStore/EventStore/issues/521","id":88645464,"number":521,"title":"Clusternode stat \"proc-cpu\" value","user":{"login":"GojiSoft","id":151849,"avatar_url":"https://avatars.githubusercontent.com/u/151849?v=3","gravatar_id":"","url":"https://api.github.com/users/GojiSoft","html_url":"https://github.com/GojiSoft","followers_url":"https://api.github.com/users/GojiSoft/followers","following_url":"https://api.github.com/users/GojiSoft/following{/other_user}","gists_url":"https://api.github.com/users/GojiSoft/gists{/gist_id}","starred_url":"https://api.github.com/users/GojiSoft/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/GojiSoft/subscriptions","organizations_url":"https://api.github.com/users/GojiSoft/orgs","repos_url":"https://api.github.com/users/GojiSoft/repos","events_url":"https://api.github.com/users/GojiSoft/events{/privacy}","received_events_url":"https://api.github.com/users/GojiSoft/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-06-16T07:32:05Z","updated_at":"2015-06-16T08:19:00Z","closed_at":null,"body":"The value for this stat is read from the performance counter \"Process -> % Processor Time\" and added to the cluster node stats as is. The value is notionally a percentage, but is not scaled to 100%, it is actually scaled to the number of logical processors. For example, if there are 8 logical processors the maximum value is 800% (ish). For monitoring purposes, it would be great if this value was scaled to 100%, or perhaps the number of logical processors could be included in the stats so that it could be scaled after elsewhere.\r\n\r\nI'm happy to make either change and make a pull request, or any other suggestions. Scaling the value to 100% seems to make most sense upstream to monitoring tools??? \r\n\r\nThanks."},"comment":{"url":"https://api.github.com/repos/EventStore/EventStore/issues/comments/112328641","html_url":"https://github.com/EventStore/EventStore/issues/521#issuecomment-112328641","issue_url":"https://api.github.com/repos/EventStore/EventStore/issues/521","id":112328641,"user":{"login":"GojiSoft","id":151849,"avatar_url":"https://avatars.githubusercontent.com/u/151849?v=3","gravatar_id":"","url":"https://api.github.com/users/GojiSoft","html_url":"https://github.com/GojiSoft","followers_url":"https://api.github.com/users/GojiSoft/followers","following_url":"https://api.github.com/users/GojiSoft/following{/other_user}","gists_url":"https://api.github.com/users/GojiSoft/gists{/gist_id}","starred_url":"https://api.github.com/users/GojiSoft/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/GojiSoft/subscriptions","organizations_url":"https://api.github.com/users/GojiSoft/orgs","repos_url":"https://api.github.com/users/GojiSoft/repos","events_url":"https://api.github.com/users/GojiSoft/events{/privacy}","received_events_url":"https://api.github.com/users/GojiSoft/received_events","type":"User","site_admin":false},"created_at":"2015-06-16T08:19:00Z","updated_at":"2015-06-16T08:19:00Z","body":"Good idea - I've added \"proc-cpuScaled\" and pushed it to \"enahancedesstats\"\nwhich has a pull request on it.\n\nWill take a look at snmp support. We do have access to perf counters, etc,\njust looking at a couple of small enhancements to the cluster stats so that\nthere is a single source to deal with.\n\nThanks.\n\nOn 16 June 2015 at 08:58, Greg Young <notifications@github.com> wrote:\n\n> Heh well you can undo. Why not name it cpuScaled etc then and I think it's\n> ok to accept.\n>\n> Btw in commercial there is snmp up port as well which is often better\n> depending on networking setup\n>\n> On Tuesday, June 16, 2015, Tim Lloyd <notifications@github.com> wrote:\n>\n> > I was a little unsure of this option i.e. publishing an unscaled value\n> and\n> > number of logical processors, as it then puts the burden of knowing what\n> to\n> > do with the process stat on downstream users of the value. I'm thinking\n> > mainly from the perspective of monitoring tools though where a 100%\n> scaled\n> > value makes sense - easy to configure an alert against a 0-100 bounded\n> > value.\n> >\n> > I have been playing around with the scaled value, and unfortunately\n> pushed\n> > to a branch I've got a pull request on - haven't quite got my pull\n> request\n> > workflow nailed :(\n> >\n> >\n> > On 16 June 2015 at 08:50, Greg Young <notifications@github.com\n> > <javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:\n> >\n> > > Maybe including num CPUs instead of scaling?\n> > >\n> > > On Tuesday, June 16, 2015, Tim Lloyd <notifications@github.com\n> > <javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:\n> > >\n> > > > The value for this stat is read from the performance counter \"Process\n> > ->\n> > > %\n> > > > Processor Time\" and added to the cluster node stats as is. The value\n> is\n> > > > notionally a percentage, but is not scaled to 100%, it is actually\n> > scaled\n> > > > to the number of logical processors. For example, if there are 8\n> > logical\n> > > > processors the maximum value is 800% (ish). For monitoring purposes,\n> it\n> > > > would be great if this value was scaled to 100%, or perhaps the\n> number\n> > of\n> > > > logical processors could be included in the stats so that it could be\n> > > > scaled after elsewhere.\n> > > >\n> > > > I'm happy to make either change and make a pull request, or any other\n> > > > suggestions. Scaling the value to 100% seems to make most sense\n> > upstream\n> > > to\n> > > > monitoring tools???\n> > > >\n> > > > Thanks.\n> > > >\n> > > > —\n> > > > Reply to this email directly or view it on GitHub\n> > > > <https://github.com/EventStore/EventStore/issues/521>.\n> > > >\n> > >\n> > >\n> > > --\n> > > Studying for the Turing test\n> > >\n> > > —\n> > > Reply to this email directly or view it on GitHub\n> > > <\n> >\n> https://github.com/EventStore/EventStore/issues/521#issuecomment-112324082\n> > >\n> > > .\n> > >\n> >\n> > —\n> > Reply to this email directly or view it on GitHub\n> > <\n> https://github.com/EventStore/EventStore/issues/521#issuecomment-112325074\n> >\n>\n> > .\n> >\n>\n>\n> --\n> Studying for the Turing test\n>\n> —\n> Reply to this email directly or view it on GitHub\n> <https://github.com/EventStore/EventStore/issues/521#issuecomment-112325270>\n> .\n>\n"}}