Skip to content
This repository has been archived by the owner on Feb 7, 2024. It is now read-only.

Add overhead to requested size when creating DVs #469

Merged
merged 1 commit into from Feb 1, 2021

Conversation

mansam
Copy link
Contributor

@mansam mansam commented Jan 28, 2021

This attempts to get the storage class filesystem overhead map from the CDIConfig. If it can't be found, it uses the default overhead defined in CDI. The size of PVCs requested by the mapper is increased by the percentage defined in the map for that PVC's storage class, or by the default if no value was specified for that storage class.

https://bugzilla.redhat.com/show_bug.cgi?id=1864577

Signed-off-by: Sam Lucidi slucidi@redhat.com

Signed-off-by: Sam Lucidi <slucidi@redhat.com>
@kubevirt-bot
Copy link

@mansam: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@kubevirt-bot kubevirt-bot added dco-signoff: yes Indicates the PR's author has DCO signed all their commits. do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. size/L labels Jan 28, 2021
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

LGTM

@kubevirt-bot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: fdupont-redhat, mansam
To complete the pull request process, please assign
You can assign the PR to them by writing /assign in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ghost ghost assigned mansam Feb 1, 2021
@pkliczewski pkliczewski merged commit 86eddce into kubevirt:master Feb 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dco-signoff: yes Indicates the PR's author has DCO signed all their commits. do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants