Skip to content

StorageManager tech debt: trivial warnings - #14562

Open
DaanHoogland wants to merge 1 commit into
ghi12316-storage-unused-paramsfrom
ghi12316-storage-trivial-warnings
Open

DaanHoogland wants to merge 1 commit into
ghi12316-storage-unused-paramsfrom
ghi12316-storage-trivial-warnings

storage: clean up trivial warnings

5a9c055
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Nov 6, 2026 in 3m 3s

Quality Gate failed

Annotations

Check failure on line 1930 in server/src/main/java/com/cloud/storage/StorageManagerImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 125 to the 15 allowed.

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

Check warning on line 3514 in server/src/main/java/com/cloud/storage/StorageManagerImpl.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=apache_cloudstack&issues=AaGxozZJhCsovMxdfRYB&open=AaGxozZJhCsovMxdfRYB&pullRequest=14562

Check warning on line 2078 in server/src/main/java/com/cloud/storage/StorageManagerImpl.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=AaGxozZJhCsovMxdfRX-&open=AaGxozZJhCsovMxdfRX-&pullRequest=14562

Check warning on line 2038 in server/src/main/java/com/cloud/storage/StorageManagerImpl.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=AaGxozZJhCsovMxdfRX9&open=AaGxozZJhCsovMxdfRX9&pullRequest=14562

Check warning on line 433 in server/src/main/java/com/cloud/storage/StorageManagerImpl.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=apache_cloudstack&issues=AaGxozZJhCsovMxdfRX_&open=AaGxozZJhCsovMxdfRX_&pullRequest=14562

Check warning on line 717 in server/src/main/java/com/cloud/storage/StorageManagerImpl.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=AaGxozZJhCsovMxdfRX8&open=AaGxozZJhCsovMxdfRX8&pullRequest=14562

Check warning on line 3507 in server/src/main/java/com/cloud/storage/StorageManagerImpl.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=apache_cloudstack&issues=AaGxozZJhCsovMxdfRYA&open=AaGxozZJhCsovMxdfRYA&pullRequest=14562