From abf8f55857e9e9a249459396fb3f078d423074e9 Mon Sep 17 00:00:00 2001
From: yanavasileva <yanavasileva@users.noreply.github.com>
Date: Fri, 23 Aug 2024 14:17:52 +0200
Subject: [PATCH] remove(tests): remove telemetry configuration property 
 (#4544)

* remove telemetry configuration property from `quarkus-extension` as it no longer exist
* exclude telemetry test in `old-engine` setup as telemetry feature is removed in newer versions; during rolling update users run old engine and data with new schema, so the test is not relevant for the scenario

https://github.com/camunda/camunda-bpm-platform/issues/4483
---
 qa/test-old-engine/pom.xml                                     | 3 +++
 quarkus-extension/README.md                                    | 1 -
 .../quarkus/engine/test/config/mixed-application.properties    | 1 -
 .../test/config/process-engine-config-application.properties   | 1 -
 4 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/qa/test-old-engine/pom.xml b/qa/test-old-engine/pom.xml
index 1406dc9a6a1..dc88d713b4e 100755
--- a/qa/test-old-engine/pom.xml
+++ b/qa/test-old-engine/pom.xml
@@ -424,6 +424,9 @@
 
                 <exclude>**/ManagementServiceTableCountTest.java</exclude>
 
+                <!-- The test is not relevant in `old-engine` setup since the feature has been removed -->
+                <!-- See https://github.com/camunda/camunda-bpm-platform/pull/4544#discussion_r1728748399 -->
+                <exclude>**/ConcurrentTelemetryConfigurationTest.java</exclude>
               </excludes>
               <testFailureIgnore>false</testFailureIgnore>
               <redirectTestOutputToFile>true</redirectTestOutputToFile>
diff --git a/quarkus-extension/README.md b/quarkus-extension/README.md
index 4f3fd07dab5..c5aa8f0101c 100644
--- a/quarkus-extension/README.md
+++ b/quarkus-extension/README.md
@@ -29,7 +29,6 @@ can look like the following:
 quarkus.camunda.cmmn-enabled=false
 quarkus.camunda.dmn-enabled=false
 quarkus.camunda.history=none
-quarkus.camunda.initialize-telemetry=false
 
 # job executor configuration
 quarkus.camunda.job-executor.thread-pool.max-pool-size=12
diff --git a/quarkus-extension/engine/deployment/src/test/resources/org/camunda/bpm/quarkus/engine/test/config/mixed-application.properties b/quarkus-extension/engine/deployment/src/test/resources/org/camunda/bpm/quarkus/engine/test/config/mixed-application.properties
index be92d469ded..37e2ec38e0c 100644
--- a/quarkus-extension/engine/deployment/src/test/resources/org/camunda/bpm/quarkus/engine/test/config/mixed-application.properties
+++ b/quarkus-extension/engine/deployment/src/test/resources/org/camunda/bpm/quarkus/engine/test/config/mixed-application.properties
@@ -2,7 +2,6 @@ quarkus.camunda.cmmn-enabled=false
 quarkus.camunda.dmn-enabled=false
 quarkus.camunda.history=none
 quarkus.camunda.enforce-history-time-to-live=false
-quarkus.camunda.initialize-telemetry=false
 
 quarkus.camunda.job-executor.thread-pool.max-pool-size=12
 quarkus.camunda.job-executor.thread-pool.queue-size=5
diff --git a/quarkus-extension/engine/deployment/src/test/resources/org/camunda/bpm/quarkus/engine/test/config/process-engine-config-application.properties b/quarkus-extension/engine/deployment/src/test/resources/org/camunda/bpm/quarkus/engine/test/config/process-engine-config-application.properties
index 8aa1b707ac4..9222ef1b03b 100644
--- a/quarkus-extension/engine/deployment/src/test/resources/org/camunda/bpm/quarkus/engine/test/config/process-engine-config-application.properties
+++ b/quarkus-extension/engine/deployment/src/test/resources/org/camunda/bpm/quarkus/engine/test/config/process-engine-config-application.properties
@@ -2,6 +2,5 @@ quarkus.camunda.cmmn-enabled=false
 quarkus.camunda.dmn-enabled=false
 quarkus.camunda.history=none
 quarkus.camunda.enforce-history-time-to-live=false
-quarkus.camunda.initialize-telemetry=false
 
 quarkus.datasource.jdbc.url=jdbc:h2:mem:camunda;TRACE_LEVEL_FILE=0;DB_CLOSE_ON_EXIT=FALSE
\ No newline at end of file