This is an automated email from the ASF dual-hosted git repository. pcongiusti pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/camel-k.git
commit d08cfdce2d42f9bdca3c6539bb0dffb3e8f369b6 Author: Pranjul Kalsi <[email protected]> AuthorDate: Fri Dec 12 22:22:42 2025 +0530 chore(e2e): update telemetry tests to use properties instead of deprecated trait --- docs/modules/traits/pages/telemetry.adoc | 3 +++ e2e/common/cli/files/JavaDuplicateParams.java | 2 +- e2e/telemetry/telemetry_test.go | 6 +++--- 3 files changed, 7 insertions(+), 4 deletions(-) diff --git a/docs/modules/traits/pages/telemetry.adoc b/docs/modules/traits/pages/telemetry.adoc index 801602cea..c6ce543d9 100644 --- a/docs/modules/traits/pages/telemetry.adoc +++ b/docs/modules/traits/pages/telemetry.adoc @@ -153,11 +153,14 @@ $ kamel run -t telemetry.enabled=true \ [source,console] ---- $ kamel run \ + --dependency camel:opentelemetry \ -p quarkus.otel.exporter.otlp.traces.endpoint=http://jaeger:4317 \ -p quarkus.otel.resource.attributes=service.name=my-integration \ MyRoute.java ---- +NOTE: The `--dependency camel:opentelemetry` is required. Previously, the telemetry trait automatically added this dependency via the `camel-k:telemetry` capability. With direct configuration, you must explicitly include the OpenTelemetry dependency. + === Benefits of Direct Configuration Using properties and dependencies configuration directly provides more flexibility, including: diff --git a/e2e/common/cli/files/JavaDuplicateParams.java b/e2e/common/cli/files/JavaDuplicateParams.java index ef267c31a..c3db38bc8 100644 --- a/e2e/common/cli/files/JavaDuplicateParams.java +++ b/e2e/common/cli/files/JavaDuplicateParams.java @@ -1,4 +1,4 @@ -// camel-k: language=java trait=telemetry.enabled=false trait=affinity.enabled=true property=prop1=false property=foo=bar +// camel-k: language=java trait=container.enabled=true trait=affinity.enabled=true property=prop1=false property=foo=bar /* * Licensed to the Apache Software Foundation (ASF) under one or more diff --git a/e2e/telemetry/telemetry_test.go b/e2e/telemetry/telemetry_test.go index 326afe551..0695b304c 100644 --- a/e2e/telemetry/telemetry_test.go +++ b/e2e/telemetry/telemetry_test.go @@ -39,11 +39,11 @@ func TestTelemetryTrait(t *testing.T) { // Check service is available g.Eventually(ServicesByType(t, ctx, "otlp", corev1.ServiceTypeClusterIP), TestTimeoutLong).ShouldNot(BeEmpty()) - // Create integration and activate traces by telemetry trait + // Create integration and activate traces using OpenTelemetry properties and dependency g.Expect(KamelRun(t, ctx, ns, "files/rest-consumer.yaml", "--name", "rest-consumer", - "-t", "telemetry.enabled=true", - "-t", "telemetry.endpoint=http://opentelemetrycollector.otlp:4317";).Execute()).To(Succeed()) + "--dependency", "camel:opentelemetry", + "-p", "quarkus.otel.exporter.otlp.traces.endpoint=http://opentelemetrycollector.otlp:4317";).Execute()).To(Succeed()) g.Eventually(IntegrationPodPhase(t, ctx, ns, "rest-consumer"), TestTimeoutLong).Should(Equal(corev1.PodRunning)) name := "Bob"
