diff --git a/web/api/api.go b/web/api/api.go index 572eb515d..da69775e7 100644 --- a/web/api/api.go +++ b/web/api/api.go @@ -24,7 +24,7 @@ import ( type MetricsService struct { gorest.RestService `root:"/api/" consumes:"application/json" produces:"application/json"` - query gorest.EndPoint `method:"GET" path:"/query?{expr:string}&{json:string}" output:"string"` + query gorest.EndPoint `method:"GET" path:"/query?{expr:string}&{as_text:string}" output:"string"` queryRange gorest.EndPoint `method:"GET" path:"/query_range?{expr:string}&{end:int64}&{range:int64}&{step:int64}" output:"string"` metrics gorest.EndPoint `method:"GET" path:"/metrics" output:"string"` diff --git a/web/api/query.go b/web/api/query.go index 85c0866fc..3ce26049b 100644 --- a/web/api/query.go +++ b/web/api/query.go @@ -37,7 +37,7 @@ func (serv MetricsService) setAccessControlHeaders(rb *gorest.ResponseBuilder) { rb.AddHeader("Access-Control-Expose-Headers", "Date") } -func (serv MetricsService) Query(expr string, formatJson string) string { +func (serv MetricsService) Query(expr string, asText string) string { exprNode, err := rules.LoadExprFromString(expr) if err != nil { return ast.ErrorToJSON(err) @@ -48,7 +48,8 @@ func (serv MetricsService) Query(expr string, formatJson string) string { rb := serv.ResponseBuilder() serv.setAccessControlHeaders(rb) var format ast.OutputFormat - if formatJson != "" { + // BUG(julius): Use Content-Type negotiation. + if asText == "" { format = ast.JSON rb.SetContentType(gorest.Application_Json) } else { diff --git a/web/static/css/graph.css b/web/static/css/graph.css index 096784414..b8c190789 100644 --- a/web/static/css/graph.css +++ b/web/static/css/graph.css @@ -107,9 +107,8 @@ input[title=*]:hover:after { display: inline; } -.console { - white-space: pre; - overflow: scroll; +.console_table { + margin-top: 5px; } .ui-widget { diff --git a/web/static/js/graph.js b/web/static/js/graph.js index 4eb78304d..e91295d22 100644 --- a/web/static/js/graph.js +++ b/web/static/js/graph.js @@ -55,10 +55,10 @@ Prometheus.Graph.prototype.initialize = function() { self.id = Prometheus.Graph.numGraphs++; // Set default options. - self.options['id'] = self.id; - self.options['range_input'] = self.options['range_input'] || "1h"; - self.options['stacked_checked'] = self.options['stacked'] ? "checked" : ""; - self.options['tab'] = self.options['tab'] || 0; + self.options["id"] = self.id; + self.options["range_input"] = self.options["range_input"] || "1h"; + self.options["stacked_checked"] = self.options["stacked"] ? "checked" : ""; + self.options["tab"] = self.options["tab"] || 0; // Draw graph controls and container from Handlebars template. @@ -77,22 +77,22 @@ Prometheus.Graph.prototype.initialize = function() { self.consoleTab = graphWrapper.find(".console"); self.graphTab = graphWrapper.find(".graph_container"); - self.tabs = graphWrapper.find(".tabs");; - self.tab = $(self.tabs.find("> div")[self.options['tab']]); // active tab + self.tabs = graphWrapper.find(".tabs"); + self.tab = $(self.tabs.find("> div")[self.options["tab"]]); // active tab self.tabs.tabs({ - active: self.options['tab'], + active: self.options["tab"], activate: function(e, ui) { storeGraphOptionsInUrl(); self.tab = ui.newPanel - if (self.tab.hasClass('reload')) { // reload if flagged with class 'reload' + if (self.tab.hasClass("reload")) { // reload if flagged with class "reload" self.submitQuery(); } } }); // Return moves focus back to expr instead of submitting. - self.insertMetric.bind('keydown', 'return', function(e) { + self.insertMetric.bind("keydown", "return", function(e) { self.expr.focus(); self.expr.val(self.expr.val()); @@ -113,12 +113,12 @@ Prometheus.Graph.prototype.initialize = function() { self.endDate.val(""); } self.endDate.change(function() { self.submitQuery() }); - self.refreshInterval.change(function() { self.updateRefresh() }) + self.refreshInterval.change(function() { self.updateRefresh() }); self.stacked.change(function() { self.updateGraph(); }); self.queryForm.submit(function() { - self.consoleTab.addClass('reload'); - self.graphTab.addClass('reload'); + self.consoleTab.addClass("reload"); + self.graphTab.addClass("reload"); self.submitQuery(); return false; }); @@ -131,10 +131,10 @@ Prometheus.Graph.prototype.initialize = function() { self.queryForm.find("button[name=dec_end]").click(function() { self.decreaseEnd(); }); self.insertMetric.change(function() { - self.expr.selection('replace', {text: self.insertMetric.val(), mode: 'before'}) + self.expr.selection("replace", {text: self.insertMetric.val(), mode: "before"}); self.insertMetric.focus(); // refocusing }); - self.insertMetric.click(function() { self.expr.focus() }) + self.insertMetric.click(function() { self.expr.focus() }); self.expr.focus(); // TODO: move to external Graph method. @@ -152,12 +152,12 @@ Prometheus.Graph.prototype.populateInsertableMetrics = function() { url: "/api/metrics", dataType: "json", success: function(json, textStatus) { - var availableMetrics = [] + var availableMetrics = []; for (var i = 0; i < json.length; i++) { self.insertMetric[0].options.add(new Option(json[i], json[i])); - availableMetrics.push(json[i]) + availableMetrics.push(json[i]); } - self.expr.autocomplete({source: availableMetrics}) + self.expr.autocomplete({source: availableMetrics}); }, error: function() { alert("Error loading available metrics!"); @@ -191,7 +191,7 @@ Prometheus.Graph.prototype.getOptions = function() { } } }); - options['tab'] = self.tabs.tabs("option", "active"); + options["tab"] = self.tabs.tabs("option", "active"); return options; }; @@ -256,8 +256,8 @@ Prometheus.Graph.prototype.getOrSetEndDate = function() { Prometheus.Graph.prototype.setEndDate = function(date) { var self = this; - dateString = date.getFullYear() + '-' + (date.getMonth()+1) + '-' + date.getDate() + ' ' + - date.getHours() + ':' + date.getMinutes(); + dateString = date.getFullYear() + "-" + (date.getMonth()+1) + "-" + date.getDate() + " " + + date.getHours() + ":" + date.getMinutes(); self.endDate.val(""); self.endDate.appendDtpicker({"current": dateString}); }; @@ -277,7 +277,7 @@ Prometheus.Graph.prototype.decreaseEnd = function() { Prometheus.Graph.prototype.submitQuery = function() { var self = this; if (!self.expr.val()) { - return + return; } self.spinner.show(); @@ -293,26 +293,22 @@ Prometheus.Graph.prototype.submitQuery = function() { self.queryForm.find("input[name=end]").val(endDate); if (self.queryXhr) { - self.queryXhr.abort() + self.queryXhr.abort(); } - var url - var data + var url; + var success; if (self.tab[0] == self.graphTab[0]) { - url = self.queryForm.attr("action") - data = self.queryForm.serialize() - dataType = "json" - success = function(json, textStatus) { self.handleGraphResponse(json, textStatus) } + url = self.queryForm.attr("action"); + success = function(json, textStatus) { self.handleGraphResponse(json, textStatus); }; } else { - url = '/api/query' - data = self.expr.serialize() - dataType = "text" - success = function(text, textStatus) { self.handleConsoleResponse(text, textStatus) } + url = "/api/query"; + success = function(text, textStatus) { self.handleConsoleResponse(text, textStatus); }; } self.queryXhr = $.ajax({ method: self.queryForm.attr("method"), url: url, - dataType: dataType, + dataType: "json", data: self.queryForm.serialize(), success: success, error: function(xhr, resp) { @@ -330,16 +326,16 @@ Prometheus.Graph.prototype.updateRefresh = function() { var self = this; if (self.timeoutID) { - window.clearTimeout(self.timeoutID) + window.clearTimeout(self.timeoutID); } interval = self.parseDuration(self.refreshInterval.val()); if (!interval) { return }; self.timeoutID = window.setTimeout(function() { - self.submitQuery() - self.updateRefresh() - }, interval * SECOND) + self.submitQuery(); + self.updateRefresh(); + }, interval * SECOND); } Prometheus.Graph.prototype.renderLabels = function(labels) { @@ -349,7 +345,7 @@ Prometheus.Graph.prototype.renderLabels = function(labels) { labelStrings.push("" + label + ": " + labels[label]); } } - return labels = "
Element | +Value | + + + +
---|