[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [dm-devel] [Gluster-devel] Puppet-Gluster+ThinP

On 04/09/2014 03:18 PM, Paul Cuzner wrote:

I'm really interested in the thinp best practices too. gluster-deploy has had thinp support for a while now - and I asked the question about best practices a while back - but nothing came back..

Hopefully - you're timing is better than mine!

cc'ing Rajesh since the thinp is all about snapshot enablement.

The amount of space you set aside is very much workload dependent (rate of change, rate of deletion, rate of notifying the storage about the freed space).

Keep in mind with snapshots (and thinly provisioned storage, whether using a software target or thinly provisioned array) we need to issue the "discard" commands down the IO stack in order to let the storage target reclaim space.

That typically means running the fstrim command on the local file system (XFS, ext4, btrfs, etc) every so often. Less typically, you can mount your local file system with "-o discard" to do it inband (but that comes at a performance penalty usually).

There is also a event mechanism to help us get notified when we hit a target configurable watermark ("help, we are running short on real disk, add more or clean up!").

Definitely worth following up with the LVM/device mapper people on how to do this best,



    *From: *"James" <purpleidea gmail com>
    *To: *"Gluster Devel" <gluster-devel nongnu org>
    *Sent: *Thursday, 10 April, 2014 3:13:40 AM
    *Subject: *[Gluster-devel] Puppet-Gluster+ThinP


    Here's a first draft of puppet-gluster w/ thin-p. This patch includes
    documentation updates too! (w00t!)


    FYI: I'll probably rebase this branch.
    FYI: Somewhat untested. Read the commit message.

    Comments welcome :)

    I'm most interested to hear about if everyone is pleased with the way I
    run the thin-p lv command. I think this makes the most sense, but let me
    know if anyone has improvements. Also I'd love to hear about what the
    default values for the parameters should be, but that's a one line
    patch, so no rush for me.


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]