Skip to content

CLVM: remove inactive libvirt pool on storage pool delete and stop autostartign CLVM pools - #14379

Open
Pearl1594 wants to merge 2 commits into
mainfrom
clvm-delete-storage
Open

CLVM: remove inactive libvirt pool on storage pool delete and stop autostartign CLVM pools#14379
Pearl1594 wants to merge 2 commits into
mainfrom
clvm-delete-storage

Address review comments

4ecc286
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Oct 9, 2026 in 3m 38s

Quality Gate passed

Issues
14 New issues
0 Accepted issues

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

See analysis details on SonarQube Cloud

Annotations

Check warning on line 192 in plugins/hypervisors/kvm/src/test/java/com/cloud/hypervisor/kvm/storage/ClvmStorageAdaptorTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "doThrow". 

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

Check warning on line 125 in plugins/hypervisors/kvm/src/test/java/com/cloud/hypervisor/kvm/storage/ClvmStorageAdaptorTest.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=AaEisSruqGtALy3xVwe8&open=AaEisSruqGtALy3xVwe8&pullRequest=14379

Check failure on line 94 in plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/storage/ClvmStorageAdaptor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to not always return the same value. 

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

Check warning on line 134 in plugins/hypervisors/kvm/src/test/java/com/cloud/hypervisor/kvm/storage/ClvmStorageAdaptorTest.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=AaEisSruqGtALy3xVwe9&open=AaEisSruqGtALy3xVwe9&pullRequest=14379

Check warning on line 142 in plugins/hypervisors/kvm/src/test/java/com/cloud/hypervisor/kvm/storage/ClvmStorageAdaptorTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "verify". 

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

Check warning on line 198 in plugins/hypervisors/kvm/src/test/java/com/cloud/hypervisor/kvm/storage/ClvmStorageAdaptorTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "verify". 

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

Check warning on line 150 in plugins/hypervisors/kvm/src/test/java/com/cloud/hypervisor/kvm/storage/ClvmStorageAdaptorTest.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=AaEisSruqGtALy3xVwfB&open=AaEisSruqGtALy3xVwfB&pullRequest=14379

Check warning on line 187 in plugins/hypervisors/kvm/src/test/java/com/cloud/hypervisor/kvm/storage/ClvmStorageAdaptorTest.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=AaEisSruqGtALy3xVwfC&open=AaEisSruqGtALy3xVwfC&pullRequest=14379

Check warning on line 112 in plugins/hypervisors/kvm/src/test/java/com/cloud/hypervisor/kvm/storage/ClvmStorageAdaptorTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "verify". 

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

Check warning on line 123 in plugins/hypervisors/kvm/src/test/java/com/cloud/hypervisor/kvm/storage/ClvmStorageAdaptorTest.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=AaEisSruqGtALy3xVwe7&open=AaEisSruqGtALy3xVwe7&pullRequest=14379

Check warning on line 143 in plugins/hypervisors/kvm/src/test/java/com/cloud/hypervisor/kvm/storage/ClvmStorageAdaptorTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "verify". 

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

Check warning on line 116 in plugins/hypervisors/kvm/src/test/java/com/cloud/hypervisor/kvm/storage/ClvmStorageAdaptorTest.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=AaEisSruqGtALy3xVwe5&open=AaEisSruqGtALy3xVwe5&pullRequest=14379

Check warning on line 117 in plugins/hypervisors/kvm/src/test/java/com/cloud/hypervisor/kvm/storage/ClvmStorageAdaptorTest.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=AaEisSruqGtALy3xVwe6&open=AaEisSruqGtALy3xVwe6&pullRequest=14379

Check warning on line 141 in plugins/hypervisors/kvm/src/test/java/com/cloud/hypervisor/kvm/storage/ClvmStorageAdaptorTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "verify". 

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