Skip to content
This repository has been archived by the owner on Nov 9, 2020. It is now read-only.

Support older versions of e2fsprogs (or custom filesystem creation flags)? #629

Closed
brunotm opened this issue Oct 17, 2016 · 3 comments
Closed

Comments

@brunotm
Copy link
Contributor

brunotm commented Oct 17, 2016

Older versions of e2fsprogs (mkfs.ext2/3/4) asks for confirmation when creating a filesystem on a entire device and not a partition (eg. /dev/sdb) and make the filesystem creation to fail.

This was due to a broken check which was removed in 06/2015.

Should support for these versions be added?
If so, add them through logic in fs.Mkfs() or through a fs-config.json configuration file, allowing users to customize filesystem creation for any filesystem?

Eg.

{
    "ext4": {
        "flags": ["-F", "-m", "1"]
    },
    "btrfs": {
        "flags": ["-d", "single"]
    },
    "xfs": {
        "flags": ["-l", "lazy-count=1"]
    }
}
@govint
Copy link
Contributor

govint commented Oct 17, 2016

IOW if the docker volume create should include an option (along with the
rest that's supported today) of what FS to create and/or the command to run and
the client simply runs that command. Allowing any FS to be used to format
the disk and for those apps that intend to use a raw disk a "no-format"
option as well.

On Mon, Oct 17, 2016 at 11:49 PM, Bruno Moura notifications@github.com
wrote:

Older versions of e2fsprogs (mkfs.ext2/3/4) asks for confirmation when
creating a filesystem on a entire device and not a partition (eg. /dev/sdb)
and make the filesystem creation to fail.

This was due to a broken check which was removed in 06/2015
http://git.kernel.org/cgit/fs/ext2/e2fsprogs.git/commit/?id=97e9b970769b1afe462f82390cb5876cf037b903
.

Should support for these versions be added?
If so, add them through logic in fs.Mkfs() or through a fs-config.json
configuration file, allowing users to customize filesystem creation for any
filesystem?

Eg.

{
"ext4": {
"flags": ["-F", "-m", "1"]
},
"btrfs": {
"flags": ["-d", "single"]
},
"xfs": {
"flags": ["-l", "lazy-count=1"]
}
}


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#629, or mute the
thread
https://github.com/notifications/unsubscribe-auth/APHseFfOnMYjkOxdgyArs8sBJFcPOVESks5q07wfgaJpZM4KY8AD
.

@brunotm
Copy link
Contributor Author

brunotm commented Oct 18, 2016

@govint @msterin

This is also needed for #612, unless there is no need to support linux distributions with older version of e2fsprogs. - we know that the VM2 used in the CI (ubuntu 14.04) has an older version.

@msterin
Copy link
Contributor

msterin commented Oct 20, 2016

Resolved by #631

@msterin msterin closed this as completed Oct 20, 2016
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants