Skip to content

Commit 7bcbb0e

Browse files
authored
fix: add prometheus scrape annotations to DM pods (#6982)
1 parent 993bafc commit 7bcbb0e

4 files changed

Lines changed: 12 additions & 0 deletions

File tree

pkg/controllers/dm/tasks/pod.go

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,8 @@ import (
3535
"github.com/pingcap/tidb-operator/v2/pkg/utils/task/v3"
3636
)
3737

38+
const metricsPath = "/metrics"
39+
3840
func TaskPod(state *ReconcileContext, c client.Client) task.Task {
3941
return task.NameTaskFunc("Pod", func(ctx context.Context) task.Result {
4042
logger := logr.FromContextOrDiscard(ctx)
@@ -166,6 +168,7 @@ func newPod(cluster *v1alpha1.Cluster, dm *v1alpha1.DM) *corev1.Pod {
166168
k8s.LabelKeyK8sAppName: k8s.LabelValK8sAppNameDMCluster,
167169
},
168170
),
171+
Annotations: maputil.Merge(k8s.AnnoProm(coreutil.DMPort(dm), metricsPath)),
169172
OwnerReferences: []metav1.OwnerReference{
170173
*metav1.NewControllerRef(dm, v1alpha1.SchemeGroupVersion.WithKind("DM")),
171174
},

pkg/controllers/dm/tasks/pod_test.go

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -123,6 +123,9 @@ func TestTaskPod(t *testing.T) {
123123
assert.Len(t, pod.Spec.Containers[0].Ports, 2)
124124
assert.Equal(t, int32(8261), pod.Spec.Containers[0].Ports[0].ContainerPort)
125125
assert.Equal(t, int32(8291), pod.Spec.Containers[0].Ports[1].ContainerPort)
126+
assert.Equal(t, "true", pod.Annotations["prometheus.io/scrape"])
127+
assert.Equal(t, "8261", pod.Annotations["prometheus.io/port"])
128+
assert.Equal(t, "/metrics", pod.Annotations["prometheus.io/path"])
126129
})
127130
}
128131
}

pkg/controllers/dmworker/tasks/pod.go

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,8 @@ import (
3535
"github.com/pingcap/tidb-operator/v2/pkg/utils/task/v3"
3636
)
3737

38+
const metricsPath = "/metrics"
39+
3840
func TaskPod(state *ReconcileContext, c client.Client) task.Task {
3941
return task.NameTaskFunc("Pod", func(ctx context.Context) task.Result {
4042
logger := logr.FromContextOrDiscard(ctx)
@@ -166,6 +168,7 @@ func newPod(cluster *v1alpha1.Cluster, dw *v1alpha1.DMWorker) *corev1.Pod {
166168
k8s.LabelKeyK8sAppName: k8s.LabelValK8sAppNameDMCluster,
167169
},
168170
),
171+
Annotations: maputil.Merge(k8s.AnnoProm(coreutil.DMWorkerPort(dw), metricsPath)),
169172
OwnerReferences: []metav1.OwnerReference{
170173
*metav1.NewControllerRef(dw, v1alpha1.SchemeGroupVersion.WithKind("DMWorker")),
171174
},

pkg/controllers/dmworker/tasks/pod_test.go

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -97,6 +97,9 @@ func TestTaskPod(t *testing.T) {
9797
assert.Equal(t, k8s.LabelValK8sAppNameDMCluster, pod.Labels[k8s.LabelKeyK8sAppName])
9898
require.Len(t, pod.Spec.Containers[0].Ports, 1)
9999
assert.Equal(t, int32(8262), pod.Spec.Containers[0].Ports[0].ContainerPort)
100+
assert.Equal(t, "true", pod.Annotations["prometheus.io/scrape"])
101+
assert.Equal(t, "8262", pod.Annotations["prometheus.io/port"])
102+
assert.Equal(t, "/metrics", pod.Annotations["prometheus.io/path"])
100103
})
101104
}
102105
}

0 commit comments

Comments
 (0)