fix(reservation): while expanding, with refquota setting

[abuisine]

Why is this PR required? What issue does it fix?:
Fixes #568

What this PR does?:
Apply reservation while expanding volumes/filesystems

Does this PR require any upgrade changes?:
No

If the changes in this PR are manually verified, list down the scenarios covered::
N/A

Any additional information for your reviewer? :
Continuation of #589

Checklist:

• Fixes ZFS Reservation is not updated when resizing a volume (thinprovisining disabled). #568
• PR Title follows the convention of <type>(<scope>): <subject>
• Has the change log section been updated?
• Commit has unit tests
• Commit has integration tests
• (Optional) Are upgrade changes included in this PR? If not, mention the issue/PR to track:
• (Optional) If documentation changes are required, which issue on https://github.com/openebs/website is used to track

[codecov-commenter]

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.37%. Comparing base (e3d55d7) to head (20fda15).
Report is 4 commits behind head on develop.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@           Coverage Diff            @@
##           develop     #595   +/-   ##
========================================
  Coverage    96.37%   96.37%           
========================================
  Files            1        1           
  Lines          496      496           
========================================
  Hits           478      478           
  Misses          14       14           
  Partials         4        4           

Flag	Coverage Δ
bddtests	96.37% <ø> (ø)

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.

[Abhinandan-Purkait]

Thanks, @abuisine LGTM. Would you mind adding a test for the same? Or we can take that up with a separate issue/pr.

[tiagolobocastro]

Thanks! Should we also update one of the resize tests to check the reservation is updated?

[Abhinandan-Purkait]

@tiagolobocastro Can you please re-approve it. Seems like the two simultaneous operations removed your approval.

[Abhinandan-Purkait]

LGTM. @abuisine Are anymore changes left from your side or can we merge this?

[abuisine]

Except unit tests which could be done in a different MR, nope Le mer. 16 oct. 2024 à 12:19, Abhinandan Purkait ***@***.***> a écrit :

…

***@***.**** approved this pull request. LGTM. @abuisine <https://github.com/abuisine> Are anymore changes left from your side or can we merge this? — Reply to this email directly, view it on GitHub <#595 (review)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAIHLGNXPZ4LHDZSJX2C5WLZ3Y4UDAVCNFSM6AAAAABPWKJVN2VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGNZRHE2DKMJTGM> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[Abhinandan-Purkait]

@abuisine Thanks, merging this now.

[abuisine]

Hi there ! Any chance to see this released soon ?

[Abhinandan-Purkait]

@abuisine Yes, we would be releasing in couple of weeks. cc @avishnu @tiagolobocastro