Skip to content

config cleanup - #13884

Open
DaanHoogland wants to merge 49 commits into
mainfrom
ghi10752-configCleanup
Open

config cleanup#13884
DaanHoogland wants to merge 49 commits into
mainfrom
ghi10752-configCleanup

more literals removed

7cfb751
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Sep 3, 2026 in 7m 12s

Quality Gate failed

Failed conditions
26.3% Coverage on New Code (required ≥ 40%)
C Reliability Rating on New Code (required ≥ B)

See analysis details on SonarQube Cloud

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

Annotations

Check failure on line 25 in engine/api/src/main/java/org/apache/cloudstack/engine/subsystem/api/storage/StorageCacheManager.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "Storage" 3 times. 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAFdVpfqJXQQ7HDE9R1&open=AaAFdVpfqJXQQ7HDE9R1&pullRequest=13884

Check failure on line 97 in api/src/main/java/com/cloud/network/NetworkService.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "Hidden" 5 times. 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAFdVvZqJXQQ7HDE9R6&open=AaAFdVvZqJXQQ7HDE9R6&pullRequest=13884

Check failure on line 26 in server/src/main/java/com/cloud/projects/ProjectManager.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "false" 3 times. 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAFdWJnqJXQQ7HDE9SK&open=AaAFdWJnqJXQQ7HDE9SK&pullRequest=13884

Check failure on line 59 in server/src/main/java/com/cloud/server/ManagementServer.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "Specify the public bridge on host for public network" 3 times. 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAFdWBdqJXQQ7HDE9SB&open=AaAFdWBdqJXQQ7HDE9SB&pullRequest=13884

Check warning on line 120 in server/src/main/java/com/cloud/network/ExternalFirewallDeviceManagerImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Split this “Monster Class” into smaller and more specialized ones to reduce its dependencies on other classes from 31 to the maximum authorized 20 or less. 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAHquAoRdE24u2TNGQL&open=AaAHquAoRdE24u2TNGQL&pullRequest=13884

Check failure on line 31 in plugins/hypervisors/baremetal/src/main/java/com/cloud/baremetal/manager/BaremetalManager.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "Advanced" 6 times. 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAHqtBpRdE24u2TNGQH&open=AaAHqtBpRdE24u2TNGQH&pullRequest=13884

Check failure on line 35 in plugins/network-elements/elastic-loadbalancer/src/main/java/com/cloud/network/lb/ElasticLoadBalancerManager.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "Advanced" 4 times. 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAHqtW-RdE24u2TNGQJ&open=AaAHqtW-RdE24u2TNGQJ&pullRequest=13884

Check warning on line 150 in server/src/main/java/com/cloud/alert/AlertManagerImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the "_privateIPCapacityThreshold" field and declare it as a local variable in the relevant methods. 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAFdWG4qJXQQ7HDE9SG&open=AaAFdWG4qJXQQ7HDE9SG&pullRequest=13884

Check warning on line 142 in server/src/main/java/com/cloud/network/ExternalLoadBalancerDeviceManagerImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Split this “Monster Class” into smaller and more specialized ones to reduce its dependencies on other classes from 37 to the maximum authorized 20 or less. 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAHqt8KRdE24u2TNGQK&open=AaAHqt8KRdE24u2TNGQK&pullRequest=13884

Check warning on line 1235 in server/src/main/java/com/cloud/resource/ResourceManagerImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a primitive boolean expression here. 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAKNgPi3ej0ChNocDo0&open=AaAKNgPi3ej0ChNocDo0&pullRequest=13884

Check failure on line 181 in server/src/main/java/com/cloud/configuration/Config.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename this constant name to match the regular expression '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAHMVMSH6sOtJrG0c_h&open=AaAHMVMSH6sOtJrG0c_h&pullRequest=13884

Check warning on line 152 in server/src/main/java/com/cloud/alert/AlertManagerImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the "_vlanCapacityThreshold" field and declare it as a local variable in the relevant methods. 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAFdWG4qJXQQ7HDE9SJ&open=AaAFdWG4qJXQQ7HDE9SJ&pullRequest=13884

Check warning on line 3510 in server/src/main/java/com/cloud/user/AccountManagerImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a primitive boolean expression here. 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAKNg3S3ej0ChNocDo2&open=AaAKNg3S3ej0ChNocDo2&pullRequest=13884

Check warning on line 46 in server/src/test/java/com/cloud/resource/DiscovererBaseTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "when". 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAPZ2ubGw8sSOR9q8QV&open=AaAPZ2ubGw8sSOR9q8QV&pullRequest=13884

Check warning on line 151 in server/src/main/java/com/cloud/alert/AlertManagerImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the "_secondaryStorageCapacityThreshold" field and declare it as a local variable in the relevant methods. 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAFdWG4qJXQQ7HDE9SI&open=AaAFdWG4qJXQQ7HDE9SI&pullRequest=13884

Check failure on line 87 in api/src/main/java/com/cloud/network/NetworkService.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "Network" 3 times. 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAHMUAhH6sOtJrG0c_Z&open=AaAHMUAhH6sOtJrG0c_Z&pullRequest=13884

Check failure on line 57 in server/src/main/java/com/cloud/server/ManagementServer.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "Hidden" 10 times. 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAFdWBdqJXQQ7HDE9R_&open=AaAFdWBdqJXQQ7HDE9R_&pullRequest=13884

Check warning on line 1555 in server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Merge this if statement with the enclosing one. 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAGbHtbTV2yRQvmzXM0&open=AaAGbHtbTV2yRQvmzXM0&pullRequest=13884

Check failure on line 52 in api/src/main/java/com/cloud/deploy/DeploymentClusterPlanner.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "Advanced" 6 times. 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAHMT-LH6sOtJrG0c_X&open=AaAHMT-LH6sOtJrG0c_X&pullRequest=13884

Check failure on line 380 in server/src/main/java/com/cloud/server/StatsCollector.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename field "vmStatsInterval" to prevent any misunderstanding/clash with field "VmStatsInterval". 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAHMU2cH6sOtJrG0c_d&open=AaAHMU2cH6sOtJrG0c_d&pullRequest=13884

Check failure on line 31 in engine/components-api/src/main/java/com/cloud/vm/snapshot/VMSnapshotManager.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "Advanced" 3 times. 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAHMT2AH6sOtJrG0c_V&open=AaAHMT2AH6sOtJrG0c_V&pullRequest=13884

Check warning on line 880 in server/src/test/java/com/cloud/configuration/ConfigurationManagerImplTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "mock". 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAGbHxQTV2yRQvmzXM1&open=AaAGbHxQTV2yRQvmzXM1&pullRequest=13884

Check warning on line 2078 in server/src/main/java/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a primitive boolean expression here. 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAKNgTE3ej0ChNocDo1&open=AaAKNgTE3ej0ChNocDo1&pullRequest=13884

Check warning on line 49 in server/src/test/java/com/cloud/resource/DiscovererBaseTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "when". 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAPZ2ubGw8sSOR9q8QY&open=AaAPZ2ubGw8sSOR9q8QY&pullRequest=13884

Check failure on line 130 in server/src/main/java/com/cloud/deploy/FirstFitPlanner.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename field "implicitHostTags" to prevent any misunderstanding/clash with field "ImplicitHostTags" defined in interface "com.cloud.deploy.DeploymentClusterPlanner". 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAHMU_JH6sOtJrG0c_g&open=AaAHMU_JH6sOtJrG0c_g&pullRequest=13884
View more details on SonarQubeCloud