Skip to content

Allow storage.overprovisioning.factor to be <1#6218

Merged
nvazquez merged 1 commit into
apache:mainfrom
CLDIN:storage-underprovision-issue-6205
Apr 7, 2022
Merged

Allow storage.overprovisioning.factor to be <1#6218
nvazquez merged 1 commit into
apache:mainfrom
CLDIN:storage-underprovision-issue-6205

Conversation

@wido
Copy link
Copy Markdown
Contributor

@wido wido commented Apr 6, 2022

Description

Allow storage.overprovisioning.factor to be <1 which allows underprovisiong of pools

By setting the value to 0.5 (example) we can underprovision a storage pool. This is useful in
cases with Ceph where it reports the raw size of the cluster where the replication factor is
not taken into account.

Fixes: ##6205

Types of changes

  • Enhancement (improves an existing feature and functionality)

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

  • Minor

Bug Severity

  • Minor

…isioning of storage pools

By setting the value to 0.5 (example) we can underprovision a storage pool. This is useful in
cases with Ceph where it reports the raw size of the cluster where the replication factor is
not taken into account.

Fixes: #apache#6205
@nvazquez
Copy link
Copy Markdown
Contributor

nvazquez commented Apr 6, 2022

Thanks @wido - @blueorangutan package

@blueorangutan
Copy link
Copy Markdown

Packaging result: ✔️ el7 ✔️ el8 ✔️ debian ✔️ suse15. SL-JID 3089

@nvazquez
Copy link
Copy Markdown
Contributor

nvazquez commented Apr 6, 2022

@blueorangutan test

@blueorangutan
Copy link
Copy Markdown

@nvazquez a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

@blueorangutan
Copy link
Copy Markdown

Trillian Build Failed (tid-3833)

@nvazquez
Copy link
Copy Markdown
Contributor

nvazquez commented Apr 6, 2022

@blueorangutan test

@blueorangutan
Copy link
Copy Markdown

@nvazquez a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

@blueorangutan
Copy link
Copy Markdown

Trillian test result (tid-3834)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 38432 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6218-t3834-kvm-centos7.zip
Smoke tests completed. 92 look OK, 0 have errors
Only failed tests results shown below:

Test Result Time (s) Test File

Copy link
Copy Markdown
Contributor

@DaanHoogland DaanHoogland left a comment

Choose a reason for hiding this comment

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

cltgm

Copy link
Copy Markdown
Contributor

@Pearl1594 Pearl1594 left a comment

Choose a reason for hiding this comment

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

Code LGTM

Copy link
Copy Markdown
Member

@GabrielBrascher GabrielBrascher left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

6 participants