Skip to content

Commit df9c299

Browse files
mpfz0rbernd
authored andcommitted
Add metrics for sidecar resources (Graylog2#5632)
These endpoints are used periodically by every sidecar. It makes sense to have these metrics in case we want to diagnose a performance bottle neck.
1 parent 52a8a1c commit df9c299

File tree

2 files changed

+4
-0
lines changed

2 files changed

+4
-0
lines changed

graylog2-server/src/main/java/org/graylog/plugins/sidecar/rest/resources/CollectorResource.java

+2
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616
*/
1717
package org.graylog.plugins.sidecar.rest.resources;
1818

19+
import com.codahale.metrics.annotation.Timed;
1920
import com.google.common.collect.ImmutableMap;
2021
import com.google.common.hash.Hashing;
2122
import io.swagger.annotations.Api;
@@ -123,6 +124,7 @@ public Collector getCollector(@ApiParam(name = "id", required = true)
123124
}
124125

125126
@GET
127+
@Timed
126128
@RequiresPermissions(SidecarRestPermissions.COLLECTORS_READ)
127129
@Produces(MediaType.APPLICATION_JSON)
128130
@ApiOperation(value = "List all collectors")

graylog2-server/src/main/java/org/graylog/plugins/sidecar/rest/resources/ConfigurationResource.java

+2
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616
*/
1717
package org.graylog.plugins.sidecar.rest.resources;
1818

19+
import com.codahale.metrics.annotation.Timed;
1920
import com.google.common.collect.ImmutableMap;
2021
import com.google.common.hash.Hashing;
2122
import io.swagger.annotations.Api;
@@ -196,6 +197,7 @@ public ValidationResult validateConfiguration(@Valid @ApiParam("configuration")
196197
}
197198

198199
@GET
200+
@Timed
199201
@Path("/render/{sidecarId}/{configurationId}")
200202
@Produces(MediaType.APPLICATION_JSON)
201203
@RequiresPermissions(SidecarRestPermissions.CONFIGURATIONS_READ)

0 commit comments

Comments
 (0)