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

fix(thin): use volume group size to create thin pool #49

Merged
merged 5 commits into from
May 5, 2021

Conversation

prateekpandey14
Copy link
Member

Signed-off-by: prateekpandey14 [email protected]

Pull Request template

Why is this PR required? What issue does it fix?:
Handle the thin pool creation which can not be greater or equal to volumegroup size.

What this PR does?:

Earlier PV size has been used while thin pool creation, which throws insufficient space error in case if thin volume
size greater or equal then the actual volumegroup size.

As part of this change we are using the half of volumegroup size to make it lesser while creating thin pool.

Does this PR require any upgrade changes?:
NO

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

Checklist:

Earlier PV size has been used while thin pool creation,
which throws insufficient space error in case if thin volume
size greater or equal then the actual volumegroup size.

As part of this change we are using the half of volumegroup size
to make it lesser while creating thin pool.

Signed-off-by: prateekpandey14 <[email protected]>
pkg/lvm/lvm_util.go Outdated Show resolved Hide resolved
pkg/lvm/lvm_util.go Outdated Show resolved Hide resolved
pkg/lvm/lvm_util.go Outdated Show resolved Hide resolved
Signed-off-by: prateekpandey14 <[email protected]>
Copy link
Contributor

@pawanpraka1 pawanpraka1 left a comment

Choose a reason for hiding this comment

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

looks good.

@pawanpraka1 pawanpraka1 merged commit 1052092 into openebs:master May 5, 2021
@prateekpandey14 prateekpandey14 deleted the handle-vg-size branch May 5, 2021 07:02
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.

pvc stucks in pending state in case of thin provisioning when pvc size is more than volume group size
3 participants