Skip to content

Add Host HA support for CLVM/CLVM_NG pools and retry transient lock refusals on activation - #14450

Draft
Pearl1594 wants to merge 1 commit into
mainfrom
clvm-ha-support
Draft

Add Host HA support for CLVM/CLVM_NG pools and retry transient lock refusals on activation#14450
Pearl1594 wants to merge 1 commit into
mainfrom
clvm-ha-support

add host HA support and retry transient lock refusals on activation

fdcace1
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Oct 20, 2026 in 4m 40s

Quality Gate passed

Issues
6 New issues
0 Accepted issues

Measures
0 Security Hotspots
46.7% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check warning on line 34 in plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/storage/ClvmSanlockHaChecker.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add a private constructor to hide the implicit public one. 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaFa_-KZdA74Gae3fbB1&open=AaFa_-KZdA74Gae3fbB1&pullRequest=14450

Check warning on line 104 in plugins/hypervisors/kvm/src/test/java/org/apache/cloudstack/kvm/ha/KVMHostActivityCheckerTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of thrown exception 'java.lang.Exception', as it cannot be thrown from method's body. 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaFa_9-DdA74Gae3fbBz&open=AaFa_9-DdA74Gae3fbBz&pullRequest=14450

Check warning on line 1221 in server/src/test/java/com/cloud/storage/ClvmPoolManagerTest.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=AaFa_-RXdA74Gae3fbB4&open=AaFa_-RXdA74Gae3fbB4&pullRequest=14450

Check failure on line 71 in server/src/main/java/com/cloud/storage/clvm/ClvmPoolManager.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=AaFa_-PpdA74Gae3fbB3&open=AaFa_-PpdA74Gae3fbB3&pullRequest=14450

Check warning on line 75 in plugins/hypervisors/kvm/src/main/java/org/apache/cloudstack/kvm/ha/KVMHostActivityChecker.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=apache_cloudstack&issues=AaFa_-N7dA74Gae3fbB2&open=AaFa_-N7dA74Gae3fbB2&pullRequest=14450

Check warning on line 116 in plugins/hypervisors/kvm/src/test/java/org/apache/cloudstack/kvm/ha/KVMHostActivityCheckerTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of thrown exception 'java.lang.Exception', as it cannot be thrown from method's body. 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaFa_9-DdA74Gae3fbB0&open=AaFa_9-DdA74Gae3fbB0&pullRequest=14450
View more details on SonarQubeCloud