Skip to content

Commit 8cdd63d

Browse files
authored
fix(k8s): phase tag (#64)
* fix(k8s): phase tag * chore: add pod_name * chore: add pod_name
1 parent e92d5f5 commit 8cdd63d

File tree

4 files changed

+9
-9
lines changed

4 files changed

+9
-9
lines changed

caas/kubernetes/pod/README.md

+4-4
Original file line numberDiff line numberDiff line change
@@ -17,10 +17,10 @@ module "datadog-monitors-caas-kubernetes-pod" {
1717

1818
Creates DataDog monitors with the following checks:
1919

20-
- Kubernetes Pod phase status failed
2120
- Kubernetes Pod {{pod_name}} container {{kube_container_name}} killed by OOM on {{kube_cluster_name}}
22-
- Kubernetes Pod terminated abnormally
23-
- Kubernetes Pod waiting errors
21+
- Kubernetes Pod {{pod_name}} phase status failed
22+
- Kubernetes Pod {{pod_name}} terminated abnormally
23+
- Kubernetes Pod {{pod_name}} waiting errors
2424
- Kubernetes Pods in {{kube_replica_set}} frequently restarted on {{kube_cluster_name}}
2525

2626
<!-- BEGIN_TF_DOCS -->
@@ -96,7 +96,7 @@ Creates DataDog monitors with the following checks:
9696
| <a name="input_pod_phase_status_message"></a> [pod\_phase\_status\_message](#input\_pod\_phase\_status\_message) | Custom message for Pod phase status monitor | `string` | `""` | no |
9797
| <a name="input_pod_phase_status_time_aggregator"></a> [pod\_phase\_status\_time\_aggregator](#input\_pod\_phase\_status\_time\_aggregator) | Monitor aggregator for Pod phase status [available values: min, max or avg] | `string` | `"max"` | no |
9898
| <a name="input_pod_phase_status_timeframe"></a> [pod\_phase\_status\_timeframe](#input\_pod\_phase\_status\_timeframe) | Monitor timeframe for Pod phase status [available values: `last_#m` (1, 5, 10, 15, or 30), `last_#h` (1, 2, or 4), or `last_1d`] | `string` | `"last_5m"` | no |
99-
| <a name="input_pod_status_group_by"></a> [pod\_status\_group\_by](#input\_pod\_status\_group\_by) | Select group by element on monitors (phase status) | `list` | <pre>[<br> "kube_namespace",<br> "kube_cluster_name"<br>]</pre> | no |
99+
| <a name="input_pod_status_group_by"></a> [pod\_status\_group\_by](#input\_pod\_status\_group\_by) | Select group by element on monitors (phase status) | `list` | <pre>[<br> "pod_name",<br> "kube_namespace",<br> "kube_cluster_name"<br>]</pre> | no |
100100
| <a name="input_prefix_slug"></a> [prefix\_slug](#input\_prefix\_slug) | Prefix string to prepend between brackets on every monitors names | `string` | `""` | no |
101101
| <a name="input_tags"></a> [tags](#input\_tags) | Global variables | `list(string)` | <pre>[<br> "type:caas",<br> "provider:kubernetes",<br> "resource:kubernetes-pod"<br>]</pre> | no |
102102
| <a name="input_team"></a> [team](#input\_team) | n/a | `string` | `"claranet"` | no |

caas/kubernetes/pod/inputs.tf

+1-1
Original file line numberDiff line numberDiff line change
@@ -264,6 +264,6 @@ variable "pod_group_by" {
264264
}
265265

266266
variable "pod_status_group_by" {
267-
default = ["kube_namespace", "kube_cluster_name"]
267+
default = ["pod_name", "kube_namespace", "kube_cluster_name"]
268268
description = "Select group by element on monitors (phase status)"
269269
}

caas/kubernetes/pod/modules.tf

+1-1
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ module "filter-tags-phase" {
1818
filter_tags_custom = var.filter_tags_custom
1919
filter_tags_custom_excluded = var.filter_tags_custom_excluded
2020
filter_tags_separator = var.filter_tags_separator
21-
extra_tags_excluded = ["phase:pending", "phase:running", "phase:succeeded", "phase:unknown"]
21+
extra_tags_excluded = ["pod_phase:pending", "pod_phase:running", "pod_phase:succeeded", "pod_phase:unknown"]
2222
}
2323

2424
module "filter-tags-nocontainercreating" {

caas/kubernetes/pod/monitors-k8s-pod.tf

+3-3
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
resource "datadog_monitor" "pod_phase_status" {
22
count = var.pod_phase_status_enabled == "true" ? 1 : 0
3-
name = "${var.prefix_slug == "" ? "" : "[${var.prefix_slug}]"}[${var.environment}] Kubernetes Pod phase status failed"
3+
name = "${var.prefix_slug == "" ? "" : "[${var.prefix_slug}]"}[${var.environment}] Kubernetes Pod {{pod_name}} phase status failed"
44
message = coalesce(var.pod_phase_status_message, var.message)
55
type = "metric alert"
66

@@ -29,7 +29,7 @@ EOQ
2929

3030
resource "datadog_monitor" "error" {
3131
count = var.error_enabled == "true" ? 1 : 0
32-
name = "${var.prefix_slug == "" ? "" : "[${var.prefix_slug}]"}[${var.environment}] Kubernetes Pod waiting errors"
32+
name = "${var.prefix_slug == "" ? "" : "[${var.prefix_slug}]"}[${var.environment}] Kubernetes Pod {{pod_name}} waiting errors"
3333
message = coalesce(var.error_message, var.message)
3434
type = "query alert"
3535

@@ -59,7 +59,7 @@ EOQ
5959

6060
resource "datadog_monitor" "terminated" {
6161
count = var.terminated_enabled == "true" ? 1 : 0
62-
name = "${var.prefix_slug == "" ? "" : "[${var.prefix_slug}]"}[${var.environment}] Kubernetes Pod terminated abnormally"
62+
name = "${var.prefix_slug == "" ? "" : "[${var.prefix_slug}]"}[${var.environment}] Kubernetes Pod {{pod_name}} terminated abnormally"
6363
message = coalesce(var.terminated_message, var.message)
6464
type = "query alert"
6565

0 commit comments

Comments
 (0)