Skip to content

Commit 5641ff4

Browse files
committed
fix merge conflict
2 parents 53073d6 + 68a2442 commit 5641ff4

5 files changed

Lines changed: 34 additions & 37 deletions

File tree

CHANGELOG.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ All notable changes to this project will be documented in this file.
1717
- BREAKING: The `.clusterConfig.metadataStorageDatabase` field has been renamed to `.clusterConfig.metadataDatabase` for consistency ([#814]).
1818
- Document Helm deployed RBAC permissions and remove unnecessary permissions ([#810]).
1919
- Internal operator refactoring: introduce dereference() and validate() steps in the reconciler ([#824]).
20+
- test: Bump vector-aggregator to 0.55.0, replace /graphql call with gRPC calls ([#826]).
2021

2122
### Deleted
2223

@@ -27,6 +28,7 @@ All notable changes to this project will be documented in this file.
2728
[#814]: https://github.com/stackabletech/druid-operator/pull/814
2829
[#818]: https://github.com/stackabletech/druid-operator/pull/818
2930
[#824]: https://github.com/stackabletech/druid-operator/pull/824
31+
[#826]: https://github.com/stackabletech/druid-operator/pull/826
3032

3133
## [26.3.0] - 2026-03-16
3234

tests/templates/kuttl/authorizer/02-install-opa.yaml.j2

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,11 +29,11 @@ metadata:
2929
name: test-opa
3030
spec:
3131
image:
32-
{% if test_scenario['values']['opa'].find(",") > 0 %}
33-
custom: "{{ test_scenario['values']['opa'].split(',')[1] }}"
34-
productVersion: "{{ test_scenario['values']['opa'].split(',')[0] }}"
32+
{% if test_scenario['values']['opa-latest'].find(",") > 0 %}
33+
custom: "{{ test_scenario['values']['opa-latest'].split(',')[1] }}"
34+
productVersion: "{{ test_scenario['values']['opa-latest'].split(',')[0] }}"
3535
{% else %}
36-
productVersion: "{{ test_scenario['values']['opa'] }}"
36+
productVersion: "{{ test_scenario['values']['opa-latest'] }}"
3737
{% endif %}
3838
pullPolicy: IfNotPresent
3939
{% if lookup('env', 'VECTOR_AGGREGATOR') %}

tests/templates/kuttl/logging/01-install-druid-vector-aggregator.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ commands:
55
- script: >-
66
helm install druid-vector-aggregator vector
77
--namespace $NAMESPACE
8-
--version 0.49.0
8+
--version 0.52.0 `# app version 0.55.0`
99
--repo https://helm.vector.dev
1010
--values 01_druid-vector-aggregator-values.yaml
1111
---

tests/templates/kuttl/logging/test_log_aggregation.py

Lines changed: 23 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -1,45 +1,40 @@
1-
#!/usr/bin/env python3
2-
import requests
1+
import json
2+
import subprocess
33

44

55
def check_sent_events():
6-
response = requests.post(
7-
"http://druid-vector-aggregator:8686/graphql",
8-
json={
9-
"query": """
10-
{
11-
transforms(first:100) {
12-
nodes {
13-
componentId
14-
metrics {
15-
sentEventsTotal {
16-
sentEventsTotal
17-
}
18-
}
19-
}
20-
}
21-
}
22-
"""
23-
},
6+
response = subprocess.run(
7+
[
8+
"grpcurl",
9+
"-plaintext",
10+
"-d",
11+
'{"limit": 100}',
12+
"druid-vector-aggregator:8686",
13+
"vector.observability.v1.ObservabilityService/GetComponents",
14+
],
15+
capture_output=True,
16+
text=True,
17+
check=True, # Raise a CalledProcessError if non-zero return
18+
timeout=20, # seconds
2419
)
20+
result = json.loads(response.stdout)
21+
components = result.get("components", [])
22+
transforms = [
23+
c for c in components if c.get("componentType") == "COMPONENT_TYPE_TRANSFORM"
24+
]
2525

26-
assert response.status_code == 200, (
27-
"Cannot access the API of the vector aggregator."
28-
)
29-
30-
result = response.json()
26+
assert len(transforms) > 0, "No transform components found"
3127

32-
transforms = result["data"]["transforms"]["nodes"]
3328
for transform in transforms:
3429
sentEvents = transform["metrics"]["sentEventsTotal"]
3530
componentId = transform["componentId"]
3631

3732
if componentId == "filteredInvalidEvents":
38-
assert sentEvents is None or sentEvents["sentEventsTotal"] == 0, (
33+
assert sentEvents is None or int(sentEvents) == 0, (
3934
"Invalid log events were sent."
4035
)
4136
else:
42-
assert sentEvents is not None and sentEvents["sentEventsTotal"] > 0, (
37+
assert sentEvents is not None and int(sentEvents) > 0, (
4338
f'No events were sent in "{componentId}".'
4439
)
4540

tests/test-definition.yaml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,9 +29,9 @@ dimensions:
2929
- name: zookeeper-latest
3030
values:
3131
- 3.9.4
32-
- name: opa
32+
- name: opa-latest
3333
values:
34-
- 1.12.3
34+
- 1.16.2
3535
- name: hadoop
3636
values:
3737
- 3.4.2
@@ -73,7 +73,7 @@ tests:
7373
dimensions:
7474
- druid
7575
- zookeeper-latest
76-
- opa
76+
- opa-latest
7777
- hadoop
7878
- openshift
7979
- name: ingestion-no-s3-ext
@@ -121,7 +121,7 @@ tests:
121121
dimensions:
122122
- druid
123123
- zookeeper-latest
124-
- opa
124+
- opa-latest
125125
- hadoop-latest
126126
- ldap-use-tls
127127
- ldap-no-bind-credentials

0 commit comments

Comments
 (0)