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.

Sign up for GitHub

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

Jump to bottom

feat: Add Longhorn V2 Data Engine #653

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

feat: Add Longhorn V2 Data Engine #653

wants to merge 4 commits into from

Commits on Nov 1, 2024

  1. feat: Add Longhorn V2 Data Engine 

    Related issue: harvester/harvester#5274

Signed-off-by: Tim Serong <tserong@suse.com>
Co-authored-by: Jillian <67180770+jillian-maroket@users.noreply.github.com>
    @tserong @jillian-maroket
    tserong and jillian-maroket committed Nov 1, 2024
    Configuration menu
    Copy the full SHA
    de40ea2 View commit details
    Browse the repository at this point in the history

  2. feat: LHv2: Add important info boxes 

    Signed-off-by: Tim Serong <tserong@suse.com>
    @tserong
    tserong committed Nov 1, 2024
    Configuration menu
    Copy the full SHA
    a322bcd View commit details
    Browse the repository at this point in the history

  3. feat: LHv2: add note about non-NVMe disk size requirements 

    This one is a little obscure.  When testing with non-NVMe disks,
Longhorn uses the aio bdev driver, which assumes the block size
is 4096 bytes, and thus requires the disk to be an even multiple
of that figure.  We hit this in testing as described in
harvester/harvester#6861.

Ordinarily one should be using NVMe disks, which end up using the
nvme bdev driver, which picks up the actual block size of the
underlying device and everything just works.  This subtlety
(nvme bdev driver working regardless of disk size, but aio bdev
driver requiring 4096 byte blocks) is apparently not documented
anywhere else (e.g. it's not mentioned in the Longhorn docs)
which is why I'm adding it here, just in case anyone else hits
it and gets confused.

Signed-off-by: Tim Serong <tserong@suse.com>
    @tserong
    tserong committed Nov 1, 2024
    Configuration menu
    Copy the full SHA
    815fa5e View commit details
    Browse the repository at this point in the history

  4. Update docs/advanced/longhorn-v2.md 

    Co-authored-by: Jillian <67180770+jillian-maroket@users.noreply.github.com>
    @tserong @jillian-maroket
    tserong and jillian-maroket authored Nov 1, 2024
    Configuration menu
    Copy the full SHA
    fc2355e View commit details
    Browse the repository at this point in the history