Skip to content

Commit 17efea8

Browse files
swiatekmswiatekm
authored
[chore] Move builder test data into yaml files (#5081)
* Refactor builder tests * Store inputs for builder tests in yaml files * Drop statuses
1 parent 2ab308b commit 17efea8

22 files changed

Lines changed: 2477 additions & 3191 deletions

‎internal/controllers/builder_test.go‎

Lines changed: 116 additions & 3191 deletions
Large diffs are not rendered by default.

‎internal/controllers/targetallocator_reconciler_test.go‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ import (
77
"context"
88
"testing"
99

10+
cmv1 "github.com/cert-manager/cert-manager/pkg/apis/certmanager/v1"
1011
routev1 "github.com/openshift/api/route/v1"
1112
monitoringv1 "github.com/prometheus-operator/prometheus-operator/pkg/apis/monitoring/v1"
1213
"github.com/stretchr/testify/assert"
@@ -38,6 +39,7 @@ func init() {
3839
utilruntime.Must(routev1.AddToScheme(testScheme))
3940
utilruntime.Must(v1alpha1.AddToScheme(testScheme))
4041
utilruntime.Must(v1beta1.AddToScheme(testScheme))
42+
utilruntime.Must(cmv1.AddToScheme(testScheme))
4143
}
4244

4345
func TestTargetAllocatorReconciler_GetCollector(t *testing.T) {

‎internal/controllers/testdata/build_collector_base.input.yaml‎

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
metadata:
2+
name: test
3+
namespace: test
4+
spec:
5+
image: test
6+
replicas: 1
7+
mode: deployment
8+
networkPolicy:
9+
enabled: true
10+
config:
11+
receivers:
12+
examplereceiver:
13+
endpoint: "0.0.0.0:12345"
14+
exporters:
15+
debug:
16+
service:
17+
pipelines:
18+
metrics:
19+
receivers: [examplereceiver]
20+
exporters: [debug]

‎internal/controllers/testdata/build_collector_base.yaml‎

Lines changed: 247 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,247 @@
1+
apiVersion: apps/v1
2+
kind: Deployment
3+
metadata:
4+
labels:
5+
app.kubernetes.io/component: opentelemetry-collector
6+
app.kubernetes.io/instance: test.test
7+
app.kubernetes.io/managed-by: opentelemetry-operator
8+
app.kubernetes.io/name: test-collector
9+
app.kubernetes.io/part-of: opentelemetry
10+
app.kubernetes.io/version: latest
11+
name: test-collector
12+
namespace: test
13+
spec:
14+
replicas: 1
15+
selector:
16+
matchLabels:
17+
app.kubernetes.io/component: opentelemetry-collector
18+
app.kubernetes.io/instance: test.test
19+
app.kubernetes.io/managed-by: opentelemetry-operator
20+
app.kubernetes.io/part-of: opentelemetry
21+
strategy: {}
22+
template:
23+
metadata:
24+
annotations:
25+
opentelemetry-operator-config/sha256: 2d266e55025628659355f1271b689d6fb53648ef6cd5595831f5835d18e59a25
26+
prometheus.io/path: /metrics
27+
prometheus.io/port: "8888"
28+
prometheus.io/scrape: "true"
29+
labels:
30+
app.kubernetes.io/component: opentelemetry-collector
31+
app.kubernetes.io/instance: test.test
32+
app.kubernetes.io/managed-by: opentelemetry-operator
33+
app.kubernetes.io/name: test-collector
34+
app.kubernetes.io/part-of: opentelemetry
35+
app.kubernetes.io/version: latest
36+
spec:
37+
containers:
38+
- args:
39+
- --config=/conf/collector.yaml
40+
env:
41+
- name: POD_NAME
42+
valueFrom:
43+
fieldRef:
44+
fieldPath: metadata.name
45+
- name: GOMEMLIMIT
46+
valueFrom:
47+
resourceFieldRef:
48+
containerName: otc-container
49+
divisor: "0"
50+
resource: limits.memory
51+
- name: GOMAXPROCS
52+
valueFrom:
53+
resourceFieldRef:
54+
containerName: otc-container
55+
divisor: "0"
56+
resource: limits.cpu
57+
image: test
58+
name: otc-container
59+
ports:
60+
- containerPort: 12345
61+
name: examplereceiver
62+
- containerPort: 8888
63+
name: metrics
64+
protocol: TCP
65+
resources: {}
66+
volumeMounts:
67+
- mountPath: /conf
68+
name: otc-internal
69+
dnsConfig: {}
70+
dnsPolicy: ClusterFirst
71+
serviceAccountName: test-collector
72+
shareProcessNamespace: false
73+
volumes:
74+
- configMap:
75+
items:
76+
- key: collector.yaml
77+
path: collector.yaml
78+
name: test-collector-2d266e55
79+
name: otc-internal
80+
---
81+
apiVersion: policy/v1
82+
kind: PodDisruptionBudget
83+
metadata:
84+
labels:
85+
app.kubernetes.io/component: opentelemetry-collector
86+
app.kubernetes.io/instance: test.test
87+
app.kubernetes.io/managed-by: opentelemetry-operator
88+
app.kubernetes.io/name: test-collector
89+
app.kubernetes.io/part-of: opentelemetry
90+
app.kubernetes.io/version: latest
91+
name: test-collector
92+
namespace: test
93+
spec:
94+
maxUnavailable: 1
95+
selector:
96+
matchLabels:
97+
app.kubernetes.io/component: opentelemetry-collector
98+
app.kubernetes.io/instance: test.test
99+
app.kubernetes.io/managed-by: opentelemetry-operator
100+
app.kubernetes.io/part-of: opentelemetry
101+
---
102+
apiVersion: v1
103+
data:
104+
collector.yaml: |
105+
receivers:
106+
examplereceiver:
107+
endpoint: 0.0.0.0:12345
108+
exporters:
109+
debug: null
110+
service:
111+
pipelines:
112+
metrics:
113+
exporters:
114+
- debug
115+
receivers:
116+
- examplereceiver
117+
kind: ConfigMap
118+
metadata:
119+
labels:
120+
app.kubernetes.io/component: opentelemetry-collector
121+
app.kubernetes.io/instance: test.test
122+
app.kubernetes.io/managed-by: opentelemetry-operator
123+
app.kubernetes.io/name: test-collector
124+
app.kubernetes.io/part-of: opentelemetry
125+
app.kubernetes.io/version: latest
126+
name: test-collector-2d266e55
127+
namespace: test
128+
---
129+
apiVersion: v1
130+
kind: ServiceAccount
131+
metadata:
132+
labels:
133+
app.kubernetes.io/component: opentelemetry-collector
134+
app.kubernetes.io/instance: test.test
135+
app.kubernetes.io/managed-by: opentelemetry-operator
136+
app.kubernetes.io/name: test-collector
137+
app.kubernetes.io/part-of: opentelemetry
138+
app.kubernetes.io/version: latest
139+
name: test-collector
140+
namespace: test
141+
---
142+
apiVersion: v1
143+
kind: Service
144+
metadata:
145+
labels:
146+
app.kubernetes.io/component: opentelemetry-collector
147+
app.kubernetes.io/instance: test.test
148+
app.kubernetes.io/managed-by: opentelemetry-operator
149+
app.kubernetes.io/name: test-collector
150+
app.kubernetes.io/part-of: opentelemetry
151+
app.kubernetes.io/version: latest
152+
operator.opentelemetry.io/collector-service-type: base
153+
name: test-collector
154+
namespace: test
155+
spec:
156+
internalTrafficPolicy: Cluster
157+
ports:
158+
- name: examplereceiver
159+
port: 12345
160+
targetPort: 0
161+
selector:
162+
app.kubernetes.io/component: opentelemetry-collector
163+
app.kubernetes.io/instance: test.test
164+
app.kubernetes.io/managed-by: opentelemetry-operator
165+
app.kubernetes.io/part-of: opentelemetry
166+
---
167+
apiVersion: v1
168+
kind: Service
169+
metadata:
170+
annotations:
171+
service.beta.openshift.io/serving-cert-secret-name: test-collector-headless-tls
172+
labels:
173+
app.kubernetes.io/component: opentelemetry-collector
174+
app.kubernetes.io/instance: test.test
175+
app.kubernetes.io/managed-by: opentelemetry-operator
176+
app.kubernetes.io/name: test-collector
177+
app.kubernetes.io/part-of: opentelemetry
178+
app.kubernetes.io/version: latest
179+
operator.opentelemetry.io/collector-headless-service: Exists
180+
operator.opentelemetry.io/collector-service-type: headless
181+
name: test-collector-headless
182+
namespace: test
183+
spec:
184+
clusterIP: None
185+
internalTrafficPolicy: Cluster
186+
ports:
187+
- name: examplereceiver
188+
port: 12345
189+
targetPort: 0
190+
selector:
191+
app.kubernetes.io/component: opentelemetry-collector
192+
app.kubernetes.io/instance: test.test
193+
app.kubernetes.io/managed-by: opentelemetry-operator
194+
app.kubernetes.io/part-of: opentelemetry
195+
---
196+
apiVersion: v1
197+
kind: Service
198+
metadata:
199+
labels:
200+
app.kubernetes.io/component: opentelemetry-collector
201+
app.kubernetes.io/instance: test.test
202+
app.kubernetes.io/managed-by: opentelemetry-operator
203+
app.kubernetes.io/name: test-collector-monitoring
204+
app.kubernetes.io/part-of: opentelemetry
205+
app.kubernetes.io/version: latest
206+
operator.opentelemetry.io/collector-monitoring-service: Exists
207+
operator.opentelemetry.io/collector-service-type: monitoring
208+
name: test-collector-monitoring
209+
namespace: test
210+
spec:
211+
ports:
212+
- name: monitoring
213+
port: 8888
214+
targetPort: 0
215+
selector:
216+
app.kubernetes.io/component: opentelemetry-collector
217+
app.kubernetes.io/instance: test.test
218+
app.kubernetes.io/managed-by: opentelemetry-operator
219+
app.kubernetes.io/part-of: opentelemetry
220+
---
221+
apiVersion: networking.k8s.io/v1
222+
kind: NetworkPolicy
223+
metadata:
224+
labels:
225+
app.kubernetes.io/component: opentelemetry-collector
226+
app.kubernetes.io/instance: test.test
227+
app.kubernetes.io/managed-by: opentelemetry-operator
228+
app.kubernetes.io/name: test-collector-networkpolicy
229+
app.kubernetes.io/part-of: opentelemetry
230+
app.kubernetes.io/version: latest
231+
name: test-collector-networkpolicy
232+
namespace: test
233+
spec:
234+
ingress:
235+
- ports:
236+
- port: 12345
237+
protocol: TCP
238+
- port: 8888
239+
protocol: TCP
240+
podSelector:
241+
matchLabels:
242+
app.kubernetes.io/component: opentelemetry-collector
243+
app.kubernetes.io/instance: test.test
244+
app.kubernetes.io/managed-by: opentelemetry-operator
245+
app.kubernetes.io/part-of: opentelemetry
246+
policyTypes:
247+
- Ingress

‎internal/controllers/testdata/build_collector_ingress.input.yaml‎

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
metadata:
2+
name: test
3+
namespace: test
4+
spec:
5+
image: test
6+
replicas: 1
7+
mode: deployment
8+
ingress:
9+
type: ingress
10+
hostname: example.com
11+
annotations:
12+
something: "true"
13+
config:
14+
receivers:
15+
examplereceiver:
16+
endpoint: "0.0.0.0:12345"
17+
exporters:
18+
debug:
19+
service:
20+
pipelines:
21+
metrics:
22+
receivers: [examplereceiver]
23+
exporters: [debug]

0 commit comments

Comments
 (0)