Skip to content

Commit a8d8ab2

Browse files
committed
chore(integration): match the resolved endpoint before its version prefix
The previous pattern looked for endpoint_url, which is the request URL and carries a version prefix, so it did not match and would have needed an update on every version bump. Match configured_endpoint instead, which is the endpoint resolved from configuration. Log fields are rendered with surrounding quotes, so include them.
1 parent c2d5199 commit a8d8ab2

1 file changed

Lines changed: 6 additions & 4 deletions

File tree

test/integration/cases/adp-config-stream/config.yaml

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,11 +29,13 @@ procedure:
2929
pattern: "Waiting for initial configuration from Datadog Agent"
3030
timeout: 60s
3131
# The forwarder starts one endpoint I/O task per resolved endpoint before any payload is sent, and
32-
# each one reports the URL it will send to. Matching that URL confirms the streamed site reached
33-
# endpoint resolution, and that the defaulted dd_url did not silently take its place. This depends
34-
# on the endpoint_url field of the "Starting endpoint I/O task." record.
32+
# each one reports the endpoint it resolved from configuration. Matching that endpoint confirms the
33+
# streamed site reached endpoint resolution, and that the defaulted dd_url did not silently take
34+
# its place. This depends on the configured_endpoint field of the "Starting endpoint I/O task."
35+
# record, which is the endpoint before the version prefix is applied. The sibling endpoint_url
36+
# field carries that prefix, so it embeds the ADP version and is not matched here.
3537
- assertion: log_contains
36-
pattern: "endpoint_url:https://app\\.datadoghq\\.eu"
38+
pattern: 'configured_endpoint:"https://app\.datadoghq\.eu"'
3739
regex: true
3840
timeout: 60s
3941
# The same site, read back from the translated runtime configuration that components are built

0 commit comments

Comments
 (0)