Skip to content

Add cloud logging service feature#1847

Open
Yavor16 wants to merge 4 commits into
masterfrom
export-logs-to-logging-service-squash
Open

Add cloud logging service feature#1847
Yavor16 wants to merge 4 commits into
masterfrom
export-logs-to-logging-service-squash

Add retry

a42c170
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Jun 11, 2026 in 1m 18s

Quality Gate failed

Failed conditions
1 Security Hotspot
77.4% Coverage on New Code (required ≥ 80%)
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Annotations

Check warning on line 57 in multiapps-controller-process/src/main/java/org/cloudfoundry/multiapps/controller/process/steps/UploadAppStep.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this field injection and use constructor injection instead. 

See more on https://sonarcloud.io/project/issues?id=cloudfoundry_multiapps-controller&issues=AZ5tnXudE1924JjNgYTo&open=AZ5tnXudE1924JjNgYTo&pullRequest=1847

Check warning on line 75 in multiapps-controller-process/src/main/java/org/cloudfoundry/multiapps/controller/process/util/OperationInFinalStateHandler.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this field injection and use constructor injection instead. 

See more on https://sonarcloud.io/project/issues?id=cloudfoundry_multiapps-controller&issues=AZ5tnXsME1924JjNgYTb&open=AZ5tnXsME1924JjNgYTb&pullRequest=1847

Check warning on line 30 in multiapps-controller-web/src/main/java/org/cloudfoundry/multiapps/controller/web/configuration/AsyncProcessLoggerConfiguration.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Update this method so that its implementation is not identical to "getAsyncExecutor" on line 20. 

See more on https://sonarcloud.io/project/issues?id=cloudfoundry_multiapps-controller&issues=AZ5tnXrcE1924JjNgYTW&open=AZ5tnXrcE1924JjNgYTW&pullRequest=1847

Check warning on line 395 in multiapps-controller-process/src/test/java/org/cloudfoundry/multiapps/controller/process/util/OperationInFinalStateHandlerTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this and every subsequent useless "eq(...)" invocation; pass the values directly. 

See more on https://sonarcloud.io/project/issues?id=cloudfoundry_multiapps-controller&issues=AZ5tnXwIE1924JjNgYTp&open=AZ5tnXwIE1924JjNgYTp&pullRequest=1847

Check warning on line 44 in multiapps-controller-process/src/main/java/org/cloudfoundry/multiapps/controller/process/steps/CollectCloudLoggingServiceParametersStep.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this field injection and use constructor injection instead. 

See more on https://sonarcloud.io/project/issues?id=cloudfoundry_multiapps-controller&issues=AZ5tnXsUE1924JjNgYTi&open=AZ5tnXsUE1924JjNgYTi&pullRequest=1847

Check failure on line 225 in multiapps-controller-persistence/src/test/java/org/cloudfoundry/multiapps/controller/persistence/services/OperationLogsExporterTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add at least one assertion to this test case. 

See more on https://sonarcloud.io/project/issues?id=cloudfoundry_multiapps-controller&issues=AZ5tnXy7E1924JjNgYT_&open=AZ5tnXy7E1924JjNgYT_&pullRequest=1847

Check warning on line 28 in multiapps-controller-process/src/main/java/org/cloudfoundry/multiapps/controller/process/steps/StageAppStep.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this field injection and use constructor injection instead. 

See more on https://sonarcloud.io/project/issues?id=cloudfoundry_multiapps-controller&issues=AZ5tnXs8E1924JjNgYTk&open=AZ5tnXs8E1924JjNgYTk&pullRequest=1847

Check warning on line 61 in multiapps-controller-persistence/src/main/java/org/cloudfoundry/multiapps/controller/persistence/query/providers/CloudLoggingServiceConfigurationQueryProvider.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Explicitly specify the time zone by passing a ZoneId or a Clock to the .now() method. 

See more on https://sonarcloud.io/project/issues?id=cloudfoundry_multiapps-controller&issues=AZ6wkPFx_DMuGrKc8AKp&open=AZ6wkPFx_DMuGrKc8AKp&pullRequest=1847

Check warning on line 49 in multiapps-controller-web/src/main/java/org/cloudfoundry/multiapps/controller/web/resources/ConfigurationEntriesResource.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this field injection and use constructor injection instead. 

See more on https://sonarcloud.io/project/issues?id=cloudfoundry_multiapps-controller&issues=AZ5tnXruE1924JjNgYTX&open=AZ5tnXruE1924JjNgYTX&pullRequest=1847

Check warning on line 172 in multiapps-controller-persistence/src/main/java/org/cloudfoundry/multiapps/controller/persistence/services/OperationLogsExporter.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Invoke method(s) only conditionally. 

See more on https://sonarcloud.io/project/issues?id=cloudfoundry_multiapps-controller&issues=AZ5tnXyPE1924JjNgYT2&open=AZ5tnXyPE1924JjNgYT2&pullRequest=1847

Check warning on line 35 in multiapps-controller-process/src/main/java/org/cloudfoundry/multiapps/controller/process/steps/CollectCloudLoggingServiceParametersStep.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this field injection and use constructor injection instead. 

See more on https://sonarcloud.io/project/issues?id=cloudfoundry_multiapps-controller&issues=AZ5tnXsUE1924JjNgYTf&open=AZ5tnXsUE1924JjNgYTf&pullRequest=1847

Check warning on line 276 in multiapps-controller-process/src/test/java/org/cloudfoundry/multiapps/controller/process/steps/UploadAppAsyncExecutionTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this and every subsequent useless "eq(...)" invocation; pass the values directly. 

See more on https://sonarcloud.io/project/issues?id=cloudfoundry_multiapps-controller&issues=AZ5tnXw0E1924JjNgYTs&open=AZ5tnXw0E1924JjNgYTs&pullRequest=1847

Check warning on line 51 in multiapps-controller-web/src/main/java/org/cloudfoundry/multiapps/controller/web/resources/ConfigurationEntriesResource.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this field injection and use constructor injection instead. 

See more on https://sonarcloud.io/project/issues?id=cloudfoundry_multiapps-controller&issues=AZ5tnXruE1924JjNgYTY&open=AZ5tnXruE1924JjNgYTY&pullRequest=1847

Check warning on line 60 in multiapps-controller-persistence/src/main/java/org/cloudfoundry/multiapps/controller/persistence/query/providers/CloudLoggingServiceConfigurationQueryProvider.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the unnecessary boolean literal. 

See more on https://sonarcloud.io/project/issues?id=cloudfoundry_multiapps-controller&issues=AZ5tnXyFE1924JjNgYTt&open=AZ5tnXyFE1924JjNgYTt&pullRequest=1847

Check warning on line 32 in multiapps-controller-process/src/main/java/org/cloudfoundry/multiapps/controller/process/steps/CollectCloudLoggingServiceParametersStep.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this field injection and use constructor injection instead. 

See more on https://sonarcloud.io/project/issues?id=cloudfoundry_multiapps-controller&issues=AZ5tnXsUE1924JjNgYTe&open=AZ5tnXsUE1924JjNgYTe&pullRequest=1847

Check failure on line 90 in multiapps-controller-persistence/src/test/java/org/cloudfoundry/multiapps/controller/persistence/services/CloudLoggingServiceConfigurationServiceTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add at least one assertion to this test case. 

See more on https://sonarcloud.io/project/issues?id=cloudfoundry_multiapps-controller&issues=AZ5tnXzEE1924JjNgYUB&open=AZ5tnXzEE1924JjNgYUB&pullRequest=1847

Check warning on line 41 in multiapps-controller-process/src/main/java/org/cloudfoundry/multiapps/controller/process/steps/CollectCloudLoggingServiceParametersStep.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this field injection and use constructor injection instead. 

See more on https://sonarcloud.io/project/issues?id=cloudfoundry_multiapps-controller&issues=AZ5tnXsUE1924JjNgYTh&open=AZ5tnXsUE1924JjNgYTh&pullRequest=1847

Check warning on line 121 in multiapps-controller-persistence/src/main/java/org/cloudfoundry/multiapps/controller/persistence/query/providers/CloudLoggingServiceConfigurationQueryProvider.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Explicitly specify the time zone by passing a ZoneId or a Clock to the .now() method. 

See more on https://sonarcloud.io/project/issues?id=cloudfoundry_multiapps-controller&issues=AZ6wkPFx_DMuGrKc8AKq&open=AZ6wkPFx_DMuGrKc8AKq&pullRequest=1847

Check warning on line 175 in multiapps-controller-process/src/main/java/org/cloudfoundry/multiapps/controller/process/steps/CollectCloudLoggingServiceParametersStep.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Call "Optional#isPresent()" or "!Optional#isEmpty()" before accessing the value. 

See more on https://sonarcloud.io/project/issues?id=cloudfoundry_multiapps-controller&issues=AZ5tnXsUE1924JjNgYTd&open=AZ5tnXsUE1924JjNgYTd&pullRequest=1847

Check warning on line 0 in multiapps-controller-persistence/src/main/java/org/cloudfoundry/multiapps/controller/persistence/query/providers/CloudLoggingServiceConfigurationQueryProvider.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use the "java.time" API for date and time. 

See more on https://sonarcloud.io/project/issues?id=cloudfoundry_multiapps-controller&issues=AZ6wkPFx_DMuGrKc8AKo&open=AZ6wkPFx_DMuGrKc8AKo&pullRequest=1847

Check warning on line 269 in multiapps-controller-persistence/src/main/java/org/cloudfoundry/multiapps/controller/persistence/services/OperationLogsExporter.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Convert this Map to an EnumMap. 

See more on https://sonarcloud.io/project/issues?id=cloudfoundry_multiapps-controller&issues=AZ5tnXyPE1924JjNgYT1&open=AZ5tnXyPE1924JjNgYT1&pullRequest=1847

Check warning on line 63 in multiapps-controller-persistence/src/test/java/org/cloudfoundry/multiapps/controller/persistence/services/ProcessLoggerPersisterTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Swap these 2 arguments so they are in the correct order: expected value, actual value. 

See more on https://sonarcloud.io/project/issues?id=cloudfoundry_multiapps-controller&issues=AZ5tnXyzE1924JjNgYT9&open=AZ5tnXyzE1924JjNgYT9&pullRequest=1847

Check warning on line 32 in multiapps-controller-process/src/main/java/org/cloudfoundry/multiapps/controller/process/steps/ExecuteTaskStep.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this field injection and use constructor injection instead. 

See more on https://sonarcloud.io/project/issues?id=cloudfoundry_multiapps-controller&issues=AZ5tnXtcE1924JjNgYTm&open=AZ5tnXtcE1924JjNgYTm&pullRequest=1847

Check warning on line 258 in multiapps-controller-process/src/test/java/org/cloudfoundry/multiapps/controller/process/steps/UploadAppAsyncExecutionTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this and every subsequent useless "eq(...)" invocation; pass the values directly. 

See more on https://sonarcloud.io/project/issues?id=cloudfoundry_multiapps-controller&issues=AZ5tnXw0E1924JjNgYTr&open=AZ5tnXw0E1924JjNgYTr&pullRequest=1847

Check warning on line 86 in multiapps-controller-persistence/src/test/java/org/cloudfoundry/multiapps/controller/persistence/services/ProcessLoggerPersisterTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Swap these 2 arguments so they are in the correct order: expected value, actual value. 

See more on https://sonarcloud.io/project/issues?id=cloudfoundry_multiapps-controller&issues=AZ5tnXyzE1924JjNgYT-&open=AZ5tnXyzE1924JjNgYT-&pullRequest=1847
View more details on SonarQubeCloud