Skip to content

Fixed src datastore on copy check for PowerFlex/ScaleIO storage driver - #9310

Merged
sureshanaparti merged 1 commit into
apache:4.18from
shapeblue:powerflex_scaleio_copy_check
Jun 28, 2024
Merged

Fixed src datastore on copy check for PowerFlex/ScaleIO storage driver#9310
sureshanaparti merged 1 commit into
apache:4.18from
shapeblue:powerflex_scaleio_copy_check

Conversation

@sureshanaparti

@sureshanapartisureshanaparti commented Jun 28, 2024

Copy link
Copy Markdown
Contributor

Description

This PR fixes src datastore on copy check for PowerFlex/ScaleIO storage driver.

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (improves an existing feature and functionality)
  • Cleanup (Code refactoring and cleanup, that may add test cases)
  • build/CI
  • test (unit or integration test code)

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

  • Major
  • Minor

Bug Severity

  • BLOCKER
  • Critical
  • Major
  • Minor
  • Trivial

Screenshots (if appropriate):

How Has This Been Tested?

Tested volume migration from pool NFS -> PowerFlex and PowerFlex -> PowerFlex. NFS -> PowerFlex fails with not suitable error.

ROOTVolume_NFSToPowerFlexDATAVolume_NFStoPowerFlex

How did you try to break this feature and the system with this change?

@sureshanaparti

Copy link
Copy Markdown
ContributorAuthor

@blueorangutan package

@blueorangutan

Copy link
Copy Markdown

@sureshanaparti a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@slavkapslavkap left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

code LGTM

@codecov

codecovBot commented Jun 28, 2024

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 12.24%. Comparing base (6b25ed7) to head (36356cd).

FilesPatch %Lines
...atastore/driver/ScaleIOPrimaryDataStoreDriver.java0.00%1 Missing ⚠️
Additional details and impacted files
@@ Coverage Diff @@## 4.18 #9310 +/- ##
============================================
- Coverage 12.24% 12.24% -0.01% + Complexity 9296 9294 -2 
============================================
Files 4699 4699 Lines 414339 414339 Branches 53023 53311 +288 ============================================
- Hits 50731 50727 -4 - Misses 357303 357308 +5 + Partials 6305 6304 -1 
FlagCoverage Δ
unittests12.24% <0.00%> (-0.01%)⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@blueorangutan

Copy link
Copy Markdown

Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 10187

@vishesh92vishesh92 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

clgtm

@sureshanaparti
sureshanaparti merged commit 983f164 into apache:4.18Jun 28, 2024
@DaanHoogland
DaanHoogland deleted the powerflex_scaleio_copy_check branch July 1, 2024 06:51
dhslove pushed a commit to ablecloud-team/ablestack-cloud that referenced this pull request Jul 2, 2024
shwstppr pushed a commit to shapeblue/cloudstack that referenced this pull request Sep 5, 2024
* Fixed src datastore on copy check for PowerFlex/ScaleIO storage driver (apache#9310)
* Ignore non-managed pools for storage pool access preparation (apache#9376)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants

@sureshanaparti@blueorangutan@vishesh92@slavkap