Skip to content

Bugfix/CSTACKEX-254: Change the minimum storagepool size to 20MB and throw an exception in case of invalid capacity bytes instead of adjusting to minimum requirement - #94

Merged
sandeeplocharla merged 1 commit into
mainfrom
bugfix/CSTACKEX-254
Aug 20, 2026

Conversation

@sandeeplocharla

@sandeeplocharla sandeeplocharla commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

Description

This PR corrects the minimum allowed storage pool size from 1.56GB to 20MB as per ONTAP guidelines for FlexVol. Also, this removes the behaviour to silently correct the invalid capacity bytes to min. requirement, instead it'll throw an exception to let the user decide on the corrective action.

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?

Case-1: Storage Pool Capacity Bytes = 214B, less than the min. req of 20MB
Screenshot 2026-08-20 at 7 08 03 AM

Case-2: Storage Pool Capacity Bytes = 20971520B or 20MB
Screenshot 2026-08-20 at 7 15 58 AM
Screenshot 2026-08-20 at 7 16 27 AM
Screenshot 2026-08-20 at 7 17 16 AM

… case of invalid capacity bytes instead of adjusting to minimum requirement
@github-actions

Copy link
Copy Markdown

🔴 Test Coverage Grade: D — Marginal

Metric Value
Line coverage 24.59%
Branch coverage 18.76%

Grade Scale

Grade Line Coverage Meaning
🟢 A ≥ 80% Excellent - this code sleeps well at night 😴
🟡 B 60-79% Good - almost there, don't stop now 😉
🟠 C 40-59% Acceptable - your code is wearing a seatbelt, but no airbags 😬
🔴 D 20-39% Marginal - boldly shipping where no test has gone before 🖖
⛔ F < 20% Failing - tests? what tests? 🔥

Branch coverage is shown as a secondary signal. Grade is determined by line coverage.
View full Actions run

@sandeeplocharla sandeeplocharla self-assigned this Aug 20, 2026
@sandeeplocharla
sandeeplocharla merged commit c7c8e09 into main Aug 20, 2026
12 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants