search

bosun-monitor / bosun

Time Series Alerting Framework
http://bosun.org
MIT License
3.4k stars 494 forks source link

Do we need the extra ()s? #64

Closed kylebrandt closed 10 years ago

kylebrandt commented 10 years ago

sum(q("sum:rate{counter,,1}:linux.proc.cpu{host=ny-git01,name=scollector}", "2d", "") > .2)

Needs to be:

sum((q("sum:rate{counter,,1}:linux.proc.cpu{host=ny-git01,name=scollector}", "2d", "") > .2))

maddyblue commented 10 years ago

I'm not sure what you're talking about. Does the first expression not work or parse?

On Thu, Apr 24, 2014 at 9:47 AM, Kyle Brandt notifications@github.comwrote:

sum(q("sum:rate{counter,,1}:linux.proc.cpu{host=ny-git01,name=scollector}", "2d", "") > .2)

Needs to be:

sum((q("sum:rate{counter,,1}:linux.proc.cpu{host=ny-git01,name=scollector}", "2d", "") > .2))

— Reply to this email directly or view it on GitHubhttps://github.com/StackExchange/tsaf/issues/64 .

kylebrandt commented 10 years ago

Correct:

image