Skip to content

[PowerFlex] Add & Remove PowerFlex/ScaleIO MDMs for the storage SDC connections #9903

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
sureshanaparti merged 8 commits into from
May 15, 2025

import fixes after rebase

538d0f1
Select commit
Loading
Failed to load commit list.
Merged

[PowerFlex] Add & Remove PowerFlex/ScaleIO MDMs for the storage SDC connections #9903

import fixes after rebase
538d0f1
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Apr 30, 2025 in 1s

9.29% of diff hit (target 16.42%)

View this Pull Request on Codecov

9.29% of diff hit (target 16.42%)

Annotations

Check warning on line 51 in core/src/main/java/com/cloud/agent/api/ModifyStoragePoolAnswer.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

core/src/main/java/com/cloud/agent/api/ModifyStoragePoolAnswer.java#L50-L51 

Added lines #L50 - L51 were not covered by tests

Check warning on line 34 in core/src/main/java/com/cloud/agent/api/UnprepareStorageClientCommand.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

core/src/main/java/com/cloud/agent/api/UnprepareStorageClientCommand.java#L34 

Added line #L34 was not covered by tests

Check warning on line 37 in core/src/main/java/com/cloud/agent/api/UnprepareStorageClientCommand.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

core/src/main/java/com/cloud/agent/api/UnprepareStorageClientCommand.java#L37 

Added line #L37 was not covered by tests

Check warning on line 55 in core/src/main/java/com/cloud/agent/api/UnprepareStorageClientCommand.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

core/src/main/java/com/cloud/agent/api/UnprepareStorageClientCommand.java#L53-L55 

Added lines #L53 - L55 were not covered by tests

Check warning on line 51 in engine/api/src/main/java/org/apache/cloudstack/engine/subsystem/api/storage/DataStoreDriver.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

engine/api/src/main/java/org/apache/cloudstack/engine/subsystem/api/storage/DataStoreDriver.java#L49-L51 

Added lines #L49 - L51 were not covered by tests

Check warning on line 154 in engine/api/src/main/java/org/apache/cloudstack/engine/subsystem/api/storage/PrimaryDataStoreDriver.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

engine/api/src/main/java/org/apache/cloudstack/engine/subsystem/api/storage/PrimaryDataStoreDriver.java#L152-L154 

Added lines #L152 - L154 were not covered by tests

Check warning on line 83 in engine/schema/src/main/java/com/cloud/storage/dao/StoragePoolHostDaoImpl.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

engine/schema/src/main/java/com/cloud/storage/dao/StoragePoolHostDaoImpl.java#L81-L83 

Added lines #L81 - L83 were not covered by tests

Check warning on line 129 in engine/schema/src/main/java/com/cloud/storage/dao/StoragePoolHostDaoImpl.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

engine/schema/src/main/java/com/cloud/storage/dao/StoragePoolHostDaoImpl.java#L125-L129 

Added lines #L125 - L129 were not covered by tests

Check warning on line 301 in engine/schema/src/main/java/org/apache/cloudstack/storage/datastore/db/PrimaryDataStoreDaoImpl.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

engine/schema/src/main/java/org/apache/cloudstack/storage/datastore/db/PrimaryDataStoreDaoImpl.java#L300-L301 

Added lines #L300 - L301 were not covered by tests

Check warning on line 305 in engine/schema/src/main/java/org/apache/cloudstack/storage/datastore/db/PrimaryDataStoreDaoImpl.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

engine/schema/src/main/java/org/apache/cloudstack/storage/datastore/db/PrimaryDataStoreDaoImpl.java#L305 

Added line #L305 was not covered by tests

Check warning on line 311 in engine/schema/src/main/java/org/apache/cloudstack/storage/datastore/db/PrimaryDataStoreDaoImpl.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

engine/schema/src/main/java/org/apache/cloudstack/storage/datastore/db/PrimaryDataStoreDaoImpl.java#L311 

Added line #L311 was not covered by tests

Check warning on line 581 in engine/schema/src/main/java/org/apache/cloudstack/storage/datastore/db/PrimaryDataStoreDaoImpl.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

engine/schema/src/main/java/org/apache/cloudstack/storage/datastore/db/PrimaryDataStoreDaoImpl.java#L579-L581 

Added lines #L579 - L581 were not covered by tests

Check warning on line 306 in engine/storage/snapshot/src/main/java/org/apache/cloudstack/storage/vmsnapshot/ScaleIOVMSnapshotStrategy.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

engine/storage/snapshot/src/main/java/org/apache/cloudstack/storage/vmsnapshot/ScaleIOVMSnapshotStrategy.java#L305-L306 

Added lines #L305 - L306 were not covered by tests

Check warning on line 308 in engine/storage/snapshot/src/main/java/org/apache/cloudstack/storage/vmsnapshot/ScaleIOVMSnapshotStrategy.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

engine/storage/snapshot/src/main/java/org/apache/cloudstack/storage/vmsnapshot/ScaleIOVMSnapshotStrategy.java#L308 

Added line #L308 was not covered by tests

Check warning on line 389 in engine/storage/snapshot/src/main/java/org/apache/cloudstack/storage/vmsnapshot/ScaleIOVMSnapshotStrategy.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

engine/storage/snapshot/src/main/java/org/apache/cloudstack/storage/vmsnapshot/ScaleIOVMSnapshotStrategy.java#L388-L389 

Added lines #L388 - L389 were not covered by tests

Check warning on line 391 in engine/storage/snapshot/src/main/java/org/apache/cloudstack/storage/vmsnapshot/ScaleIOVMSnapshotStrategy.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

engine/storage/snapshot/src/main/java/org/apache/cloudstack/storage/vmsnapshot/ScaleIOVMSnapshotStrategy.java#L391 

Added line #L391 was not covered by tests

Check warning on line 154 in engine/storage/src/main/java/org/apache/cloudstack/storage/volume/datastore/PrimaryDataStoreHelper.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

engine/storage/src/main/java/org/apache/cloudstack/storage/volume/datastore/PrimaryDataStoreHelper.java#L153-L154 

Added lines #L153 - L154 were not covered by tests

Check warning on line 156 in engine/storage/src/main/java/org/apache/cloudstack/storage/volume/datastore/PrimaryDataStoreHelper.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

engine/storage/src/main/java/org/apache/cloudstack/storage/volume/datastore/PrimaryDataStoreHelper.java#L156 

Added line #L156 was not covered by tests

Check warning on line 158 in engine/storage/src/main/java/org/apache/cloudstack/storage/volume/datastore/PrimaryDataStoreHelper.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

engine/storage/src/main/java/org/apache/cloudstack/storage/volume/datastore/PrimaryDataStoreHelper.java#L158 

Added line #L158 was not covered by tests

Check warning on line 162 in engine/storage/src/main/java/org/apache/cloudstack/storage/volume/datastore/PrimaryDataStoreHelper.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

engine/storage/src/main/java/org/apache/cloudstack/storage/volume/datastore/PrimaryDataStoreHelper.java#L162 

Added line #L162 was not covered by tests

Check warning on line 42 in plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtModifyStoragePoolCommandWrapper.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtModifyStoragePoolCommandWrapper.java#L42 

Added line #L42 was not covered by tests

Check warning on line 45 in plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtModifyStoragePoolCommandWrapper.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtModifyStoragePoolCommandWrapper.java#L44-L45 

Added lines #L44 - L45 were not covered by tests

Check warning on line 49 in plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtModifyStoragePoolCommandWrapper.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtModifyStoragePoolCommandWrapper.java#L48-L49 

Added lines #L48 - L49 were not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/storage/KVMStoragePoolManager.java#L412 

Added line #L412 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/storage/KVMStoragePoolManager.java#L414-L419 

Added lines #L414 - L419 were not covered by tests
View more details on Codecov