On Fri, Mar 22, 2013 at 3:58 PM, Sage Weil <[email protected] (mailto:[email protected])> wrote: > On Fri, 22 Mar 2013, Gregory Farnum wrote: > > I suspect users are going to easily get in trouble without a more > > rigid separation between multi-linked and single-linked buckets. It's > > probably best if anybody who's gone to the trouble of setting up a DAG > > can't wipe it out without being very explicit ? so for instance "move" > > should only work against a bucket with a single parent. > > > > > Good idea; I'll add that. > > > Rather than > > defaulting to all ancestors, removals should (for multiply-linked > > buckets) require users to either specify a set of ancestors or to pass > > in a "--all" flag. > > > > > 'rm' only works on an empty bucket, so I'm not sure there is much danger > is removing all links (and the bucket) in that case? >
Well, if "unlink" takes it out of the tree without requiring it to be empty then that would be one case. > > > Also, I suspect that "rm" actually deletes the bucket while "unlink" > > simply removes it from all parents (but leaves it in the tree); that > > distinction might need to be a little stronger (or is possibly not > > appropriate to leave in the CLI?). > > > > > That's right. The "remove" versus "unlink" verbs make that pretty clear > to me, at least... Are you suggesting this be clarified in the docs, or > that the command set change? I think once we settle on the CLI, John can > make a pass through the crush docs and make sure these commands are > explained. > That makes sense if you know the internal representation of the CRUSH map; less so if you're thinking of it in terms from related fields like, say, a filesystem. ;) I admit I don't have any better suggestions and strong docs might be just fine, but the distinction is one that's very visible to users *afterwards* but not beforehand. I'm imagining a lot of crush maps with 5 unlinked parallel trees in our future. I guess mostly I'm just not sure what the use case is for keeping around unlinked buckets so I'd rather keep the set of CLI options simpler. It's not that hard to link a bucket into the tree where you want it and then remove the other links. -Greg -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
