Skip to content

Commit affaf4a

Browse files
authored
Merge pull request scribd#61 from scribd/jimp/grammar_fix
Grammar edits
2 parents d391cfe + 4db0b95 commit affaf4a

File tree

1 file changed

+12
-11
lines changed

1 file changed

+12
-11
lines changed

_posts/2020-04-28-elasticache-slowlog-metrics-for-datadog.md

Lines changed: 12 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -11,15 +11,16 @@ tags:
1111
team: Core Infrastructure
1212
---
1313

14-
All managed services will have trade-offs, when we adopted AWS ElastiCache we
15-
could no longer use Datadog's excellent excellent [Redis
14+
All managed services will have trade-offs. When Scribd adopted AWS ElastiCache we
15+
could no longer use Datadog's excellent [Redis
1616
integration](https://docs.datadoghq.com/integrations/redisdb/)
17-
and some killer metrics we couldn't live without.
17+
and lost some killer metrics we couldn't live without.
1818
We deployed the [AWS ElastiCache
19-
integration](https://docs.datadoghq.com/integrations/amazon_elasticache/#overview).
20-
for Datadog which returned some of the desired metrics back to our dashbards
21-
with one notable exception: "slowlog" metrics. The Redis
22-
[`SLOWLOG`](https://redis.io/commands/slowlog) is used to help identify queries
19+
integration](https://docs.datadoghq.com/integrations/amazon_elasticache/#overview)
20+
for Datadog which returned the desired metrics back to our dashbards
21+
with one notable exception: "slowlog" metrics.
22+
23+
The Redis [`SLOWLOG`](https://redis.io/commands/slowlog) is used to help identify queries
2324
which are taking too long to execute. We use the slowlog metrics provided by the
2425
Datadog Redis integration alert us when a Redis server's behavior starts to go
2526
south, a key indicator of looming user-impactful production issues.
@@ -46,9 +47,9 @@ page](https://github.com/scribd/elasticache-slowlog-to-datadog/releases). To
4647
deploy directly to AWS from the console, upload the “Full zip distribution” and
4748
supply the [required
4849
parameters](https://github.com/scribd/elasticache-slowlog-to-datadog#parameters).
49-
I’d recommend using our Terraform wrapper, however.
50+
I’d recommend using our Terraform module, however.
5051

51-
## The Terraform wrapper
52+
## The Terraform Module
5253

5354
The second part of the equation is the Terraform module:
5455
[terraform-elasticache-slowlog-to-datadog](https://github.com/scribd/terraform-elasticache-slowlog-to-datadog)
@@ -62,8 +63,8 @@ S3](https://www.terraform.io/docs/providers/aws/r/lambda_function.html#specifyin
6263
However, I like the approach of maintaining a separate repository and build
6364
pipeline, as this works around Terraform’s [intentionally limited build
6465
functionality](https://github.com/hashicorp/terraform/issues/8344#issuecomment-361014199).
65-
In essence, the terraform wrapper merely [consumes the
66-
elasticache-slowlog-to-datadog
66+
The terraform module consumes the
67+
[elasticache-slowlog-to-datadog
6768
artifact](https://github.com/scribd/terraform-elasticache-slowlog-to-datadog/blob/master/main.tf#L97).
6869

6970
## Usage

0 commit comments

Comments
 (0)