[Gluster-devel] Puppet-Gluster+ThinP

[Ric Wheeler]

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,

Ric

>
> --------------------------------------------------------------------------------
>
>     *From: *"James" <purpleidea at gmail.com>
>     *To: *"Gluster Devel" <gluster-devel at nongnu.org>
>     *Sent: *Thursday, 10 April, 2014 3:13:40 AM
>     *Subject: *[Gluster-devel] Puppet-Gluster+ThinP
>
>     Okay,
>
>     Here's a first draft of puppet-gluster w/ thin-p. This patch includes
>     documentation updates too! (w00t!)
>
>     https://github.com/purpleidea/puppet-gluster/tree/feat/thinp
>
>     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.
>
>     Cheers,
>     James
>