Skip to content

Commit

Permalink
fix: TestMonitoringStackController/Verify_multi-namespace_support
Browse files Browse the repository at this point in the history
  • Loading branch information
lihongyan1 committed Jul 14, 2023
1 parent 9b663b7 commit a3862f7
Showing 1 changed file with 21 additions and 9 deletions.
30 changes: 21 additions & 9 deletions test/e2e/monitoring_stack_controller_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -861,7 +861,7 @@ func namespaceSelectorTest(t *testing.T) {

promClient := framework.NewPrometheusClient("http://localhost:9090")
if pollErr := wait.Poll(5*time.Second, 5*time.Minute, func() (bool, error) {
query := `prometheus_build_info{namespace=~"test-ns-.*"}`
query := `version{namespace=~"test-ns-.*"}`
result, err := promClient.Query(query)
if err != nil {
return false, nil
Expand Down Expand Up @@ -889,7 +889,7 @@ func deployDemoApp(t *testing.T, nsName string, nsLabels, resourceLabels map[str
}

// deploy a pod, service, service-monitor into that namespace
prom := newPrometheusPod(t, "prometheus", ns.Name)
prom := newPrometheusPod(t, "prometheus-example-app", ns.Name)
if err := f.K8sClient.Create(context.Background(), prom); err != nil {
return fmt.Errorf("failed to create demo app %s/%s: %w", nsName, prom.Name, err)
}
Expand All @@ -902,15 +902,15 @@ func deployDemoApp(t *testing.T, nsName string, nsLabels, resourceLabels map[str
// these are prometheus ports
svc.Spec.Ports = []corev1.ServicePort{{
Name: "metrics",
Port: 9090,
TargetPort: intstr.FromInt(9090),
Port: 8080,
TargetPort: intstr.FromInt(8080),
}}

if err := f.K8sClient.Create(context.Background(), svc); err != nil {
return fmt.Errorf("failed to create service for demo app %s/%s: %w", nsName, svc.Name, err)
}

svcMon := newServiceMonitor(t, ns.Name, "prometheus", resourceLabels, svcLabels, "metrics")
svcMon := newServiceMonitor(t, ns.Name, "prometheus-example-app", resourceLabels, svcLabels, "metrics")
if err := f.K8sClient.Create(context.Background(), svcMon); err != nil {
return fmt.Errorf("failed to create servicemonitor for demo service %s/%s: %w", nsName, svcMon.Name, err)
}
Expand Down Expand Up @@ -973,6 +973,7 @@ func newService(t *testing.T, name, namespace string, labels, selector map[strin
}

func newPrometheusPod(t *testing.T, name, ns string) *corev1.Pod {
var bF = false
pod := &corev1.Pod{
TypeMeta: metav1.TypeMeta{
APIVersion: corev1.SchemeGroupVersion.String(),
Expand All @@ -983,16 +984,27 @@ func newPrometheusPod(t *testing.T, name, ns string) *corev1.Pod {
Namespace: ns,
Labels: map[string]string{
"app.kubernetes.io/name": "prometheus",
"app.kubernetes.io/version": "2.39.1",
"app.kubernetes.io/version": "multiarch-v0.1.0",
},
},
Spec: corev1.PodSpec{
Containers: []corev1.Container{{
Name: "prometheus",
Image: "quay.io/prometheus/prometheus:v2.39.1",
Name: "prometheus-example-app",
Image: "quay.io/openshifttest/prometheus-example-app@sha256:382dc349f82d730b834515e402b48a9c7e2965d0efbc42388bd254f424f6193e",
SecurityContext: &corev1.SecurityContext{
AllowPrivilegeEscalation: &bF,
SeccompProfile: &corev1.SeccompProfile{
Type: "RuntimeDefault",
},
Capabilities: &corev1.Capabilities{
Drop: []corev1.Capability{
"ALL",
},
},
},
Ports: []corev1.ContainerPort{{
Name: "metrics",
ContainerPort: 9090,
ContainerPort: 8080,
}},
}},
},
Expand Down

0 comments on commit a3862f7

Please sign in to comment.