accre / lstore

LStore - A fault-tolerant, performant distributed data storage framework.
http://www.lstore.org
Apache License 2.0
4 stars 5 forks source link

Base config pkg for use of Vanderbilt L-Store sys #23

Closed hellermf closed 8 years ago

hellermf commented 8 years ago

Adds a package that provides a basic starting configuration for using the primary Vanderbilt L-Store system and a template for other configurations including running a local L-Server.

PerilousApricot commented 8 years ago

I'm not a super fan of this. We should certainly have tools and base configs for people to use, but baking in the current configuration as a package seems off to me. Are we going to make new packages every time we need to change the configuration, then deploy the packages to all of the machines? That sounds like what configuration management is supposed to do.

tacketar commented 8 years ago

I'm definitely in favor of it but I wouldn't put it in lstore-release and I'd separate out the RID stuff. I'd make a new lstore-sample-config package or some such and slap all the config files in there EXCEPT all the rid-*.cfg files. Instead I'd just make a blank rid.cfg containing the file format.

You'd also need a helper script to sed the lserver IP addresses and the INI include paths. This would provide the base for a fresh L-Store install with intelligent defaults which we sorely need. I routinely setup new L-Servers and it's a PITA. I don't care for it to be VU specific. I'd like a generic package that would setup all the directories, install a base os_file structure, intelligent config, and /lfs_roots. This package would assume lstore-release is installed already which is just binaries. All that would be missing is the RID config. We could add a few of the existing helper scripts to make that go quick.

This would greatly simplify setting up a completely fresh L-Store system.

Alan

On 11/6/2015 3:06 PM, Andrew Melo wrote:

I'm not a super fan of this. We should certainly have tools and base configs for people to use, but baking in the current configuration as a package seems off to me. Are we going to make new packages every time we need to change the configuration, then deploy the packages to all of the machines? That sounds like what configuration management is supposed to do.

Instead, I think we need:

*

— Reply to this email directly or view it on GitHub https://github.com/accre/lstore-release/pull/23#issuecomment-154540187.

PerilousApricot commented 8 years ago

I think we're shooting towards two separate things simultaneously and it's a bit muddled:

Taking what you'd like to do, Alan, I suggest the following:

It's dark in this basement.

PerilousApricot commented 8 years ago

Or, some sort of "lfs_setup_config.sh" script could be provided, which would automate the drudgery. Making directories, modifying a configuration, and all that jazz wouldn't be hard to script up, if there was a clear description of each step

It's dark in this basement.

hellermf commented 8 years ago

I meant to do a little write up explaining my thoughts after creating the PR but got sidetracked. This is some stuff that I had held off on pushing upstream because I wasn't really satisfied with the state that it was in. However I finally decided that was worth pushing because it is pretty useful even in its current form (setting up clients and L-servers is no longer a PITA). Additionally I just wanted to get the conversation started by providing a straw-man implementation that can be attacked and morphed into a good general sample configuration.

Right now what I have muddles client and server configuration. It defaults to providing a working configuration for a client using the Vanderbilt L-Store system. The reconfig_lserver.sh script can be used to automatically generate a configuration for running a local LServer (basically just sed the IP address).

Things I would like to see:

hellermf commented 8 years ago

I didn't see some of the comments until after I posted my most recent one, clearly we are all thinking many of the same things.

tacketar commented 8 years ago

Agreed. -Alan

On 11/6/2015 4:19 PM, Matthew Heller wrote:

I didn't see some of the comments until after I posted my most recent one, clearly we are all thinking many of the same things.

— Reply to this email directly or view it on GitHub https://github.com/accre/lstore-release/pull/23#issuecomment-154559831.

PerilousApricot commented 8 years ago

I'm gonna take a stab at doing what we discussed.

Matt - do you want to take a stab at making a "general" cpacklist for RPM and DEB?

It's dark in this basement.

PerilousApricot commented 8 years ago

Since this branch splits off a branch I rebased, I had to rebase this one as well. I didn't touch this one, I just made a new one (feature/toplevel-package) via cherry-picking commits.

PerilousApricot commented 8 years ago

superseded by #24