123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155 |
- package shell
- import (
- "testing"
- "github.com/chrislusf/seaweedfs/weed/pb/master_pb"
- "github.com/chrislusf/seaweedfs/weed/storage/super_block"
- )
- type testMoveCase struct {
- name string
- replication string
- replicas []*VolumeReplica
- sourceLocation location
- targetLocation location
- expected bool
- }
- func TestIsGoodMove(t *testing.T) {
- var tests = []testMoveCase{
- {
- name: "test 100 move to spread into proper data centers",
- replication: "100",
- replicas: []*VolumeReplica{
- {
- location: &location{"dc1", "r1", &master_pb.DataNodeInfo{Id: "dn1"}},
- },
- {
- location: &location{"dc1", "r2", &master_pb.DataNodeInfo{Id: "dn2"}},
- },
- },
- sourceLocation: location{"dc1", "r2", &master_pb.DataNodeInfo{Id: "dn2"}},
- targetLocation: location{"dc2", "r2", &master_pb.DataNodeInfo{Id: "dn3"}},
- expected: true,
- },
- {
- name: "test move to the same node",
- replication: "001",
- replicas: []*VolumeReplica{
- {
- location: &location{"dc1", "r1", &master_pb.DataNodeInfo{Id: "dn1"}},
- },
- {
- location: &location{"dc1", "r1", &master_pb.DataNodeInfo{Id: "dn2"}},
- },
- },
- sourceLocation: location{"dc1", "r1", &master_pb.DataNodeInfo{Id: "dn2"}},
- targetLocation: location{"dc1", "r1", &master_pb.DataNodeInfo{Id: "dn2"}},
- expected: false,
- },
- {
- name: "test move to the same rack, but existing node",
- replication: "001",
- replicas: []*VolumeReplica{
- {
- location: &location{"dc1", "r1", &master_pb.DataNodeInfo{Id: "dn1"}},
- },
- {
- location: &location{"dc1", "r1", &master_pb.DataNodeInfo{Id: "dn2"}},
- },
- },
- sourceLocation: location{"dc1", "r1", &master_pb.DataNodeInfo{Id: "dn2"}},
- targetLocation: location{"dc1", "r1", &master_pb.DataNodeInfo{Id: "dn1"}},
- expected: false,
- },
- {
- name: "test move to the same rack, a new node",
- replication: "001",
- replicas: []*VolumeReplica{
- {
- location: &location{"dc1", "r1", &master_pb.DataNodeInfo{Id: "dn1"}},
- },
- {
- location: &location{"dc1", "r1", &master_pb.DataNodeInfo{Id: "dn2"}},
- },
- },
- sourceLocation: location{"dc1", "r1", &master_pb.DataNodeInfo{Id: "dn2"}},
- targetLocation: location{"dc1", "r1", &master_pb.DataNodeInfo{Id: "dn3"}},
- expected: true,
- },
- {
- name: "test 010 move all to the same rack",
- replication: "010",
- replicas: []*VolumeReplica{
- {
- location: &location{"dc1", "r1", &master_pb.DataNodeInfo{Id: "dn1"}},
- },
- {
- location: &location{"dc1", "r2", &master_pb.DataNodeInfo{Id: "dn2"}},
- },
- },
- sourceLocation: location{"dc1", "r2", &master_pb.DataNodeInfo{Id: "dn2"}},
- targetLocation: location{"dc1", "r1", &master_pb.DataNodeInfo{Id: "dn3"}},
- expected: false,
- },
- {
- name: "test 010 move to spread racks",
- replication: "010",
- replicas: []*VolumeReplica{
- {
- location: &location{"dc1", "r1", &master_pb.DataNodeInfo{Id: "dn1"}},
- },
- {
- location: &location{"dc1", "r2", &master_pb.DataNodeInfo{Id: "dn2"}},
- },
- },
- sourceLocation: location{"dc1", "r2", &master_pb.DataNodeInfo{Id: "dn2"}},
- targetLocation: location{"dc1", "r3", &master_pb.DataNodeInfo{Id: "dn3"}},
- expected: true,
- },
- {
- name: "test 010 move to spread racks",
- replication: "010",
- replicas: []*VolumeReplica{
- {
- location: &location{"dc1", "r1", &master_pb.DataNodeInfo{Id: "dn1"}},
- },
- {
- location: &location{"dc1", "r2", &master_pb.DataNodeInfo{Id: "dn2"}},
- },
- },
- sourceLocation: location{"dc1", "r2", &master_pb.DataNodeInfo{Id: "dn2"}},
- targetLocation: location{"dc1", "r2", &master_pb.DataNodeInfo{Id: "dn3"}},
- expected: true,
- },
- }
- for _, tt := range tests {
- replicaPlacement, _ := super_block.NewReplicaPlacementFromString(tt.replication)
- println("replication:", tt.replication, "expected", tt.expected, "name:", tt.name)
- sourceNode := &Node{
- info: tt.sourceLocation.dataNode,
- dc: tt.sourceLocation.dc,
- rack: tt.sourceLocation.rack,
- }
- targetNode := &Node{
- info: tt.targetLocation.dataNode,
- dc: tt.targetLocation.dc,
- rack: tt.targetLocation.rack,
- }
- if isGoodMove(replicaPlacement, tt.replicas, sourceNode, targetNode) != tt.expected {
- t.Errorf("%s: expect %v move from %v to %s, replication:%v",
- tt.name, tt.expected, tt.sourceLocation, tt.targetLocation, tt.replication)
- }
- }
- }
|