gridcentric / canary

OpenStack Nova performance stats collection framework (based on collectd).
http://docs.gridcentric.com/canary
12 stars 5 forks source link

Allow possiblity to change graphs freshness by changing frequency of polling #2

Closed sbauza closed 11 years ago

sbauza commented 11 years ago

Hi Canary team, By using your cool GUI, I had trouble with graphs freshness because the polling frequency was too high and was raising Django limits "OverLimit: This request was rate-limited. (HTTP 413)"

Consequently, I propose a quick patch adding possibility to dynamically change the frequency of the refreshs on the GUI. The idea is to store the Object value of the setInterval, plus the graph settings, and stopping/starting a new setInterval with the desired value.

Please excuse the visual I did, as I had no time to specifically look at the DOM for integrating properly.

timbot commented 11 years ago

Awesome, and awesome idea!

We might fix up the patch a tiny bit before merging if that's okay with you.

Keep em coming :-D

sbauza commented 11 years ago

Hi,

Sure, you can adapt the patch as your needs, in particular regarding the GUI integration. I had no clear idea where to put the textarea with the content ;-)

amscanne commented 11 years ago

@sbauza, I accidentally closed this pull request. I pulled the commit into our internal CI system though, so the code (adapted a tiny bit) will make it's way up to the canary repo in the near future.