Skip to content

Commit

Permalink
Don't auto update summary report time on open
Browse files Browse the repository at this point in the history
  • Loading branch information
jonjlee committed Jul 6, 2011
1 parent 4df2ea2 commit d96fb59
Show file tree
Hide file tree
Showing 4 changed files with 16 additions and 4 deletions.
3 changes: 2 additions & 1 deletion RELEASE-NOTES.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ Compatible with node v0.4.x
Features:

* rps is now a separate stat from result codes
* summary graph time does not auto-update on open

## v0.3.0 (2011/06/16) ##

Expand Down Expand Up @@ -65,4 +66,4 @@ This release adds nodeloadlib and moves to Dygraph for charting.

Initial releases of nodeload. Tags correspond to node compatible versions. To find a version of node that's compatible with a tag release do `git show <tagname>`.

For example: git show v0.1.1
For example: git show v0.1.1
10 changes: 9 additions & 1 deletion TODO
Original file line number Diff line number Diff line change
Expand Up @@ -12,9 +12,17 @@
- Download data as csv
- Console webpage (stats)
- Console webpage (node manager)
- Add mem, disk io read + write + wait monitoring remote.ex.js
- Add mem, disk io read + write + wait monitoring
- Remote testing should also aggregate summary-only stats (e.g. uniques)
- Use stats.StatsGroup in monitoring and remote
- Update READMEs
- Write a DEVELOPERS doc that explains the components
- Add zipf number generator
- Download/copy data in CSV
- Allow output directory to be customized
- Add JMX monitoring support
- Add support for bar graphs
- Methods for graphing histograms
- Timestamp graphs rather than using minutes
- Allow graphs to be overlayed
- Use agents for concurrent connections
5 changes: 4 additions & 1 deletion lib/reporting/summary.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -51,9 +51,11 @@
<script id="source" language="javascript" type="text/javascript">
var raw_reports;
function updateDate(date) {
document.getElementById("timestamp").innerHTML = date || new Date();
}
function update(reports) {
var main = document.getElementById("main"), summaries = document.getElementById("summaries");
document.getElementById("timestamp").innerHTML = new Date();
raw_reports = reports;
reports.forEach(function(report) {
Expand Down Expand Up @@ -101,6 +103,7 @@
http.open("GET", "/reports");
http.onreadystatechange=function() {
if (http.readyState == 4 && http.status == 200) {
updateDate();
update(JSON.parse(http.responseText));
}
}
Expand Down
2 changes: 1 addition & 1 deletion nodeload.js

Large diffs are not rendered by default.

0 comments on commit d96fb59

Please sign in to comment.