Controllers with small disks have issues adding new filesystems

Bug #1791170 reported by Frank Miller
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
StarlingX
Fix Released
Medium
Stefan Dinescu

Bug Description

Brief Description
-----------------
The default controller filesystem sizes are based on the underlying root disk size and currently targets a virtual disk and varying disk sizes (<120GB). The current small disk size is a lab specific restriction and is problematic when adding new file system since there is insufficient room to support all file systems. To simplify future changes the checks should be based on a small disk size (not virtual or physical) of only consider small disks that are <240GB or large disks (>240GB). This will simplify the logic to support the new lab restriction of disks that are 240GB and our standard deployment requirement of >500GB.

Severity
--------
Minor

Steps to Reproduce
------------------
n/a

Expected Behavior
------------------
It should be easy to add a new filesystem even on smaller disks

Actual Behavior
----------------
Sometimes the system prevents adding a new filesystem on smaller disks even when there is room.

Reproducibility
---------------
Reproducible on labs with smaller disks or on a virtual environment.

System Configuration
--------------------
Any system config where the controller is using a small disk

Branch/Pull Time/Commit
-----------------------
n/a

Timestamp/Logs
--------------
n/a

Frank Miller (sensfan22)
Changed in starlingx:
assignee: StefanF (stefan) → nobody
assignee: nobody → Stefan Dinescu (stefandinescu)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to stx-config (master)

Fix proposed to branch: master
Review: https://review.openstack.org/600743

Changed in starlingx:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to stx-metal (master)

Fix proposed to branch: master
Review: https://review.openstack.org/600745

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to stx-config (master)

Reviewed: https://review.openstack.org/600743
Committed: https://git.openstack.org/cgit/openstack/stx-config/commit/?id=92cc2240cfbdb99c249c229b94568723d8b86897
Submitter: Zuul
Branch: master

commit 92cc2240cfbdb99c249c229b94568723d8b86897
Author: Stefan Dinescu <email address hidden>
Date: Fri Sep 7 15:44:49 2018 +0300

    Simplify disk space allocation

    Allocating of space on the root disk was done in kickstart, combined
    with config_controller. Various checks for disk and volume group
    space made it difficult to add new filesystem partitions.

    To simplify this, all checks are now merged under disk size checks.

    Kickstart files now create the partitions of the same size for
    rootfs and log partitions, no matter the disk size, with the
    only variable size being the cgts-vg size.

    Thus, any future changes will only need to consider only this
    size for ensuring enough space is present.

    The limit between small and big disks is 240GB, with a minimum
    disk size of 120GB.

    Change-Id: I0ac54d5b2ba72af9505881c5849bc4e76540b3f4
    Closes-bug: 1791170
    Signed-off-by: Stefan Dinescu <email address hidden>

Changed in starlingx:
status: In Progress → Fix Released
Ghada Khalil (gkhalil)
Changed in starlingx:
importance: Undecided → Medium
summary: - Labs with small disks have issues adding new filesystems
+ Controllers with small disks have issues adding new filesystems
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to stx-metal (master)

Reviewed: https://review.openstack.org/600745
Committed: https://git.openstack.org/cgit/openstack/stx-metal/commit/?id=103cccd786ce0a3ec6e5df421b11739469a97db4
Submitter: Zuul
Branch: master

commit 103cccd786ce0a3ec6e5df421b11739469a97db4
Author: Stefan Dinescu <email address hidden>
Date: Fri Sep 7 15:56:12 2018 +0300

    Simplify disk space allocation

    Allocating of space on the root disk was done in kickstart, combined
    with config_controller. Various checks for disk and volume group
    space made it difficult to add new filesystem partitions.

    To simplify this, all checks are now merged under disk size checks.

    Kickstart files now create the partitions of the same size for
    rootfs and log partitions, no matter the disk size, with the
    only variable size being the cgts-vg size.

    Thus, any future changes will only need to consider only this
    size for ensuring enough space is present.

    The limit between small and big disks is 240GB, with a minimum
    disk size of 120GB.

    Depends-On: https://review.openstack.org/#/c/600743/
    Change-Id: I37ecc8eb5468811d1ca3a71f8e2a0629525e8fad
    Closes-bug: 1791170
    Signed-off-by: Stefan Dinescu <email address hidden>

Ghada Khalil (gkhalil)
tags: added: stx.2018.10
Ken Young (kenyis)
tags: added: stx.1.0
removed: stx.2018.10
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.