Skip to content
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.

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

test: thinpool auto-extend test #308

Merged
merged 1 commit into from
Jun 14, 2024
Merged

test: thinpool auto-extend test #308

merged 1 commit into from
Jun 14, 2024

Conversation

dsharma-dc
Copy link
Contributor

This change adds a test that verifies a thinpool auto-extends in size as configured in the lvm.conf file.

Pull Request template

Please, go through these steps before you submit a PR.

Why is this PR required? What issue does it fix?:
This adds a test for verifying thinpool auto-extend.

What this PR does?:
Adds a new test which fills up the LVM logical volume to a certain capacity so that
auto-extend of underlying thinpool kicks in.

Does this PR require any upgrade changes?:
No

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

Any additional information for your reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs

Checklist:

  • Fixes #
  • 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/openebs-docs is used to track them:

tests/utils.go Outdated Show resolved Hide resolved
@dsharma-dc dsharma-dc force-pushed the thinpool_config branch 3 times, most recently from 8812ab0 to 1f13848 Compare June 13, 2024 10:26
@dsharma-dc dsharma-dc mentioned this pull request Jun 13, 2024
This change adds a test that verifies a thinpool auto-extends in size
as configured in the lvm.conf file.

Signed-off-by: Diwakar Sharma <[email protected]>
Copy link
Contributor

@abhilashshetty04 abhilashshetty04 left a comment

Choose a reason for hiding this comment

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

Thanks, Lgtm

@dsharma-dc dsharma-dc merged commit 4a57750 into develop Jun 14, 2024
5 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