Thomas Vachuska | 4f1a60c | 2014-10-28 13:39:07 -0700 | [diff] [blame] | 1 | /* |
Brian O'Connor | a09fe5b | 2017-08-03 21:12:30 -0700 | [diff] [blame] | 2 | * Copyright 2014-present Open Networking Foundation |
Thomas Vachuska | 4f1a60c | 2014-10-28 13:39:07 -0700 | [diff] [blame] | 3 | * |
| 4 | * Licensed under the Apache License, Version 2.0 (the "License"); |
| 5 | * you may not use this file except in compliance with the License. |
| 6 | * You may obtain a copy of the License at |
| 7 | * |
| 8 | * http://www.apache.org/licenses/LICENSE-2.0 |
| 9 | * |
| 10 | * Unless required by applicable law or agreed to in writing, software |
| 11 | * distributed under the License is distributed on an "AS IS" BASIS, |
| 12 | * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 13 | * See the License for the specific language governing permissions and |
| 14 | * limitations under the License. |
| 15 | */ |
Brian O'Connor | abafb50 | 2014-12-02 22:26:20 -0800 | [diff] [blame] | 16 | package org.onosproject.cluster.impl; |
Ayaka Koshibe | 406d010 | 2014-09-24 16:08:12 -0700 | [diff] [blame] | 17 | |
Claudine Chiu | dce0815 | 2016-03-09 18:19:28 +0000 | [diff] [blame] | 18 | import java.util.List; |
Ayaka Koshibe | 406d010 | 2014-09-24 16:08:12 -0700 | [diff] [blame] | 19 | import java.util.Set; |
Claudine Chiu | dce0815 | 2016-03-09 18:19:28 +0000 | [diff] [blame] | 20 | import java.util.function.Consumer; |
Ayaka Koshibe | 406d010 | 2014-09-24 16:08:12 -0700 | [diff] [blame] | 21 | |
Claudine Chiu | dce0815 | 2016-03-09 18:19:28 +0000 | [diff] [blame] | 22 | import com.google.common.collect.ImmutableList; |
| 23 | import com.google.common.collect.ImmutableSet; |
Victor Silva | f2b9d03 | 2016-09-19 19:43:20 -0300 | [diff] [blame] | 24 | import org.easymock.EasyMock; |
Ayaka Koshibe | 406d010 | 2014-09-24 16:08:12 -0700 | [diff] [blame] | 25 | import org.junit.After; |
| 26 | import org.junit.Before; |
| 27 | import org.junit.Test; |
Claudine Chiu | dce0815 | 2016-03-09 18:19:28 +0000 | [diff] [blame] | 28 | import org.onlab.junit.TestUtils; |
Ray Milkey | cc53abd | 2015-02-19 12:31:33 -0800 | [diff] [blame] | 29 | import org.onlab.packet.IpAddress; |
Victor Silva | f2b9d03 | 2016-09-19 19:43:20 -0300 | [diff] [blame] | 30 | import org.onosproject.cfg.ComponentConfigService; |
Brian O'Connor | abafb50 | 2014-12-02 22:26:20 -0800 | [diff] [blame] | 31 | import org.onosproject.cluster.ClusterService; |
| 32 | import org.onosproject.cluster.ControllerNode; |
Brian O'Connor | abafb50 | 2014-12-02 22:26:20 -0800 | [diff] [blame] | 33 | import org.onosproject.cluster.DefaultControllerNode; |
| 34 | import org.onosproject.cluster.NodeId; |
Thomas Vachuska | 36002e6 | 2015-05-19 16:12:29 -0700 | [diff] [blame] | 35 | import org.onosproject.common.event.impl.TestEventDispatcher; |
Brian O'Connor | abafb50 | 2014-12-02 22:26:20 -0800 | [diff] [blame] | 36 | import org.onosproject.mastership.MastershipService; |
| 37 | import org.onosproject.mastership.MastershipStore; |
| 38 | import org.onosproject.mastership.MastershipTermService; |
| 39 | import org.onosproject.net.DeviceId; |
Simon Hunt | 5361221 | 2016-12-04 17:19:52 -0800 | [diff] [blame] | 40 | import org.onosproject.net.config.NetworkConfigServiceAdapter; |
Claudine Chiu | dce0815 | 2016-03-09 18:19:28 +0000 | [diff] [blame] | 41 | import org.onosproject.net.region.Region; |
| 42 | import org.onosproject.net.region.RegionId; |
| 43 | import org.onosproject.net.region.RegionStore; |
| 44 | import org.onosproject.net.region.impl.RegionManager; |
| 45 | import org.onosproject.store.cluster.StaticClusterService; |
| 46 | import org.onosproject.store.region.impl.DistributedRegionStore; |
| 47 | import org.onosproject.store.service.TestStorageService; |
Thomas Vachuska | c97aa61 | 2015-06-23 16:00:18 -0700 | [diff] [blame] | 48 | import org.onosproject.store.trivial.SimpleMastershipStore; |
Ayaka Koshibe | 406d010 | 2014-09-24 16:08:12 -0700 | [diff] [blame] | 49 | |
Ayaka Koshibe | ea5b4ce | 2014-10-11 14:17:17 -0700 | [diff] [blame] | 50 | import com.google.common.collect.Sets; |
Madan Jampani | de003d9 | 2015-05-11 17:14:20 -0700 | [diff] [blame] | 51 | import com.google.common.util.concurrent.Futures; |
Jordan Halterman | 61aeb35 | 2017-10-18 16:22:17 -0700 | [diff] [blame] | 52 | import org.onosproject.upgrade.impl.UpgradeServiceAdapter; |
Ayaka Koshibe | ea5b4ce | 2014-10-11 14:17:17 -0700 | [diff] [blame] | 53 | |
Victor Silva | f2b9d03 | 2016-09-19 19:43:20 -0300 | [diff] [blame] | 54 | import static org.easymock.EasyMock.anyObject; |
| 55 | import static org.easymock.EasyMock.expect; |
| 56 | import static org.easymock.EasyMock.expectLastCall; |
| 57 | import static org.easymock.EasyMock.replay; |
Claudine Chiu | dce0815 | 2016-03-09 18:19:28 +0000 | [diff] [blame] | 58 | import static org.junit.Assert.*; |
Ray Milkey | cc53abd | 2015-02-19 12:31:33 -0800 | [diff] [blame] | 59 | import static org.onosproject.net.MastershipRole.MASTER; |
| 60 | import static org.onosproject.net.MastershipRole.NONE; |
| 61 | import static org.onosproject.net.MastershipRole.STANDBY; |
Thomas Vachuska | 42e8cce | 2015-07-29 19:25:18 -0700 | [diff] [blame] | 62 | import static org.onosproject.net.NetTestTools.injectEventDispatcher; |
Claudine Chiu | dce0815 | 2016-03-09 18:19:28 +0000 | [diff] [blame] | 63 | import static org.onosproject.net.region.Region.Type.METRO; |
Ayaka Koshibe | 406d010 | 2014-09-24 16:08:12 -0700 | [diff] [blame] | 64 | |
| 65 | /** |
| 66 | * Test codifying the mastership service contracts. |
| 67 | */ |
| 68 | public class MastershipManagerTest { |
| 69 | |
| 70 | private static final NodeId NID_LOCAL = new NodeId("local"); |
| 71 | private static final NodeId NID_OTHER = new NodeId("foo"); |
Pavlin Radoslavov | 444b519 | 2014-10-28 10:45:19 -0700 | [diff] [blame] | 72 | private static final IpAddress LOCALHOST = IpAddress.valueOf("127.0.0.1"); |
Ayaka Koshibe | 406d010 | 2014-09-24 16:08:12 -0700 | [diff] [blame] | 73 | private static final DeviceId DEV_MASTER = DeviceId.deviceId("of:1"); |
| 74 | private static final DeviceId DEV_OTHER = DeviceId.deviceId("of:2"); |
| 75 | |
Claudine Chiu | dce0815 | 2016-03-09 18:19:28 +0000 | [diff] [blame] | 76 | private static final RegionId RID1 = RegionId.regionId("r1"); |
| 77 | private static final RegionId RID2 = RegionId.regionId("r2"); |
| 78 | private static final DeviceId DID1 = DeviceId.deviceId("foo:d1"); |
| 79 | private static final DeviceId DID2 = DeviceId.deviceId("foo:d2"); |
| 80 | private static final DeviceId DID3 = DeviceId.deviceId("foo:d3"); |
| 81 | private static final NodeId NID1 = NodeId.nodeId("n1"); |
| 82 | private static final NodeId NID2 = NodeId.nodeId("n2"); |
| 83 | private static final NodeId NID3 = NodeId.nodeId("n3"); |
| 84 | private static final NodeId NID4 = NodeId.nodeId("n4"); |
| 85 | private static final ControllerNode CNODE1 = |
| 86 | new DefaultControllerNode(NID1, IpAddress.valueOf("127.0.1.1")); |
| 87 | private static final ControllerNode CNODE2 = |
| 88 | new DefaultControllerNode(NID2, IpAddress.valueOf("127.0.1.2")); |
| 89 | private static final ControllerNode CNODE3 = |
| 90 | new DefaultControllerNode(NID3, IpAddress.valueOf("127.0.1.3")); |
| 91 | private static final ControllerNode CNODE4 = |
| 92 | new DefaultControllerNode(NID4, IpAddress.valueOf("127.0.1.4")); |
| 93 | |
| 94 | |
Ayaka Koshibe | 406d010 | 2014-09-24 16:08:12 -0700 | [diff] [blame] | 95 | private MastershipManager mgr; |
| 96 | protected MastershipService service; |
Claudine Chiu | dce0815 | 2016-03-09 18:19:28 +0000 | [diff] [blame] | 97 | private TestRegionManager regionManager; |
| 98 | private RegionStore regionStore; |
| 99 | private TestClusterService testClusterService; |
Ayaka Koshibe | 406d010 | 2014-09-24 16:08:12 -0700 | [diff] [blame] | 100 | |
| 101 | @Before |
Claudine Chiu | dce0815 | 2016-03-09 18:19:28 +0000 | [diff] [blame] | 102 | public void setUp() throws Exception { |
Ayaka Koshibe | 406d010 | 2014-09-24 16:08:12 -0700 | [diff] [blame] | 103 | mgr = new MastershipManager(); |
| 104 | service = mgr; |
Thomas Vachuska | 42e8cce | 2015-07-29 19:25:18 -0700 | [diff] [blame] | 105 | injectEventDispatcher(mgr, new TestEventDispatcher()); |
Claudine Chiu | dce0815 | 2016-03-09 18:19:28 +0000 | [diff] [blame] | 106 | testClusterService = new TestClusterService(); |
| 107 | mgr.clusterService = testClusterService; |
Jordan Halterman | 61aeb35 | 2017-10-18 16:22:17 -0700 | [diff] [blame] | 108 | mgr.upgradeService = new UpgradeServiceAdapter(); |
Yuta HIGUCHI | 0c6e184 | 2014-11-05 22:34:23 -0800 | [diff] [blame] | 109 | mgr.store = new TestSimpleMastershipStore(mgr.clusterService); |
Claudine Chiu | dce0815 | 2016-03-09 18:19:28 +0000 | [diff] [blame] | 110 | regionStore = new DistributedRegionStore(); |
| 111 | TestUtils.setField(regionStore, "storageService", new TestStorageService()); |
| 112 | TestUtils.callMethod(regionStore, "activate", |
| 113 | new Class<?>[] {}); |
| 114 | regionManager = new TestRegionManager(); |
| 115 | TestUtils.setField(regionManager, "store", regionStore); |
| 116 | regionManager.activate(); |
| 117 | mgr.regionService = regionManager; |
Victor Silva | f2b9d03 | 2016-09-19 19:43:20 -0300 | [diff] [blame] | 118 | |
| 119 | ComponentConfigService mockConfigService = |
| 120 | EasyMock.createMock(ComponentConfigService.class); |
| 121 | expect(mockConfigService.getProperties(anyObject())).andReturn(ImmutableSet.of()); |
| 122 | mockConfigService.registerProperties(mgr.getClass()); |
| 123 | expectLastCall(); |
| 124 | mockConfigService.unregisterProperties(mgr.getClass(), false); |
| 125 | expectLastCall(); |
| 126 | expect(mockConfigService.getProperties(anyObject())).andReturn(ImmutableSet.of()); |
| 127 | mgr.cfgService = mockConfigService; |
| 128 | replay(mockConfigService); |
| 129 | |
Ayaka Koshibe | 406d010 | 2014-09-24 16:08:12 -0700 | [diff] [blame] | 130 | mgr.activate(); |
| 131 | } |
| 132 | |
| 133 | @After |
| 134 | public void tearDown() { |
| 135 | mgr.deactivate(); |
| 136 | mgr.clusterService = null; |
Thomas Vachuska | 42e8cce | 2015-07-29 19:25:18 -0700 | [diff] [blame] | 137 | injectEventDispatcher(mgr, null); |
Claudine Chiu | dce0815 | 2016-03-09 18:19:28 +0000 | [diff] [blame] | 138 | regionManager.deactivate(); |
| 139 | mgr.regionService = null; |
Ayaka Koshibe | 406d010 | 2014-09-24 16:08:12 -0700 | [diff] [blame] | 140 | mgr.store = null; |
| 141 | } |
| 142 | |
| 143 | @Test |
| 144 | public void setRole() { |
| 145 | mgr.setRole(NID_OTHER, DEV_MASTER, MASTER); |
Yuta HIGUCHI | 0c6e184 | 2014-11-05 22:34:23 -0800 | [diff] [blame] | 146 | assertEquals("wrong local role:", NONE, mgr.getLocalRole(DEV_MASTER)); |
Madan Jampani | de003d9 | 2015-05-11 17:14:20 -0700 | [diff] [blame] | 147 | assertEquals("wrong obtained role:", STANDBY, Futures.getUnchecked(mgr.requestRoleFor(DEV_MASTER))); |
Ayaka Koshibe | 406d010 | 2014-09-24 16:08:12 -0700 | [diff] [blame] | 148 | |
| 149 | //set to master |
| 150 | mgr.setRole(NID_LOCAL, DEV_MASTER, MASTER); |
| 151 | assertEquals("wrong local role:", MASTER, mgr.getLocalRole(DEV_MASTER)); |
| 152 | } |
| 153 | |
| 154 | @Test |
| 155 | public void relinquishMastership() { |
Ayaka Koshibe | b62aab5 | 2014-10-24 13:15:25 -0700 | [diff] [blame] | 156 | //no backups - should just turn to NONE for device. |
Ayaka Koshibe | d9f693e | 2014-09-29 18:04:54 -0700 | [diff] [blame] | 157 | mgr.setRole(NID_LOCAL, DEV_MASTER, MASTER); |
| 158 | assertEquals("wrong role:", MASTER, mgr.getLocalRole(DEV_MASTER)); |
| 159 | mgr.relinquishMastership(DEV_MASTER); |
| 160 | assertNull("wrong master:", mgr.getMasterFor(DEV_OTHER)); |
Ayaka Koshibe | b62aab5 | 2014-10-24 13:15:25 -0700 | [diff] [blame] | 161 | assertEquals("wrong role:", NONE, mgr.getLocalRole(DEV_MASTER)); |
Ayaka Koshibe | d9f693e | 2014-09-29 18:04:54 -0700 | [diff] [blame] | 162 | |
| 163 | //not master, nothing should happen |
Ayaka Koshibe | b62aab5 | 2014-10-24 13:15:25 -0700 | [diff] [blame] | 164 | mgr.setRole(NID_LOCAL, DEV_OTHER, NONE); |
Ayaka Koshibe | d9f693e | 2014-09-29 18:04:54 -0700 | [diff] [blame] | 165 | mgr.relinquishMastership(DEV_OTHER); |
| 166 | assertNull("wrong role:", mgr.getMasterFor(DEV_OTHER)); |
| 167 | |
| 168 | //provide NID_OTHER as backup and relinquish |
| 169 | mgr.setRole(NID_LOCAL, DEV_MASTER, MASTER); |
| 170 | assertEquals("wrong master:", NID_LOCAL, mgr.getMasterFor(DEV_MASTER)); |
| 171 | mgr.setRole(NID_OTHER, DEV_MASTER, STANDBY); |
| 172 | mgr.relinquishMastership(DEV_MASTER); |
| 173 | assertEquals("wrong master:", NID_OTHER, mgr.getMasterFor(DEV_MASTER)); |
Ayaka Koshibe | 406d010 | 2014-09-24 16:08:12 -0700 | [diff] [blame] | 174 | } |
| 175 | |
| 176 | @Test |
| 177 | public void requestRoleFor() { |
| 178 | mgr.setRole(NID_LOCAL, DEV_MASTER, MASTER); |
| 179 | mgr.setRole(NID_OTHER, DEV_OTHER, MASTER); |
| 180 | |
| 181 | //local should be master for one but standby for other |
Madan Jampani | de003d9 | 2015-05-11 17:14:20 -0700 | [diff] [blame] | 182 | assertEquals("wrong role:", MASTER, Futures.getUnchecked(mgr.requestRoleFor(DEV_MASTER))); |
| 183 | assertEquals("wrong role:", STANDBY, Futures.getUnchecked(mgr.requestRoleFor(DEV_OTHER))); |
Ayaka Koshibe | 406d010 | 2014-09-24 16:08:12 -0700 | [diff] [blame] | 184 | } |
| 185 | |
| 186 | @Test |
| 187 | public void getMasterFor() { |
| 188 | mgr.setRole(NID_LOCAL, DEV_MASTER, MASTER); |
| 189 | mgr.setRole(NID_OTHER, DEV_OTHER, MASTER); |
| 190 | assertEquals("wrong master:", NID_LOCAL, mgr.getMasterFor(DEV_MASTER)); |
| 191 | assertEquals("wrong master:", NID_OTHER, mgr.getMasterFor(DEV_OTHER)); |
| 192 | |
| 193 | //have NID_OTHER hand over DEV_OTHER to NID_LOCAL |
| 194 | mgr.setRole(NID_LOCAL, DEV_OTHER, MASTER); |
| 195 | assertEquals("wrong master:", NID_LOCAL, mgr.getMasterFor(DEV_OTHER)); |
| 196 | } |
| 197 | |
| 198 | @Test |
| 199 | public void getDevicesOf() { |
| 200 | mgr.setRole(NID_LOCAL, DEV_MASTER, MASTER); |
| 201 | mgr.setRole(NID_LOCAL, DEV_OTHER, STANDBY); |
| 202 | assertEquals("should be one device:", 1, mgr.getDevicesOf(NID_LOCAL).size()); |
Ayaka Koshibe | 406d010 | 2014-09-24 16:08:12 -0700 | [diff] [blame] | 203 | //hand both devices to NID_LOCAL |
| 204 | mgr.setRole(NID_LOCAL, DEV_OTHER, MASTER); |
| 205 | assertEquals("should be two devices:", 2, mgr.getDevicesOf(NID_LOCAL).size()); |
| 206 | } |
| 207 | |
Ayaka Koshibe | 48239b0 | 2014-09-25 17:12:31 -0700 | [diff] [blame] | 208 | @Test |
| 209 | public void termService() { |
Yuta HIGUCHI | bcac499 | 2014-11-22 19:27:57 -0800 | [diff] [blame] | 210 | MastershipTermService ts = mgr; |
Ayaka Koshibe | 48239b0 | 2014-09-25 17:12:31 -0700 | [diff] [blame] | 211 | |
Yuta HIGUCHI | dfe6e3b | 2014-10-30 11:31:51 -0700 | [diff] [blame] | 212 | //term = 1 for both |
Ayaka Koshibe | 48239b0 | 2014-09-25 17:12:31 -0700 | [diff] [blame] | 213 | mgr.setRole(NID_LOCAL, DEV_MASTER, MASTER); |
Yuta HIGUCHI | dfe6e3b | 2014-10-30 11:31:51 -0700 | [diff] [blame] | 214 | assertEquals("inconsistent term: ", 1, ts.getMastershipTerm(DEV_MASTER).termNumber()); |
Ayaka Koshibe | 48239b0 | 2014-09-25 17:12:31 -0700 | [diff] [blame] | 215 | |
Yuta HIGUCHI | dfe6e3b | 2014-10-30 11:31:51 -0700 | [diff] [blame] | 216 | //hand devices to NID_LOCAL and back: term = 1 + 2 |
Ayaka Koshibe | 48239b0 | 2014-09-25 17:12:31 -0700 | [diff] [blame] | 217 | mgr.setRole(NID_OTHER, DEV_MASTER, MASTER); |
| 218 | mgr.setRole(NID_LOCAL, DEV_MASTER, MASTER); |
Yuta HIGUCHI | dfe6e3b | 2014-10-30 11:31:51 -0700 | [diff] [blame] | 219 | assertEquals("inconsistent terms: ", 3, ts.getMastershipTerm(DEV_MASTER).termNumber()); |
Ayaka Koshibe | 48239b0 | 2014-09-25 17:12:31 -0700 | [diff] [blame] | 220 | } |
| 221 | |
Claudine Chiu | dce0815 | 2016-03-09 18:19:28 +0000 | [diff] [blame] | 222 | @Test |
| 223 | public void balanceWithRegion1() { |
| 224 | //set up region - 2 sets of masters with 1 node in each |
| 225 | Set<NodeId> masterSet1 = ImmutableSet.of(NID1); |
| 226 | Set<NodeId> masterSet2 = ImmutableSet.of(NID2); |
| 227 | List<Set<NodeId>> masters = ImmutableList.of(masterSet1, masterSet2); |
| 228 | Region r = regionManager.createRegion(RID1, "R1", METRO, masters); |
| 229 | regionManager.addDevices(RID1, ImmutableSet.of(DID1, DID2)); |
| 230 | Set<DeviceId> deviceIds = regionManager.getRegionDevices(RID1); |
| 231 | assertEquals("incorrect device count", 2, deviceIds.size()); |
| 232 | |
| 233 | testClusterService.put(CNODE1, ControllerNode.State.ACTIVE); |
| 234 | testClusterService.put(CNODE2, ControllerNode.State.ACTIVE); |
| 235 | |
| 236 | //set master to non region nodes |
| 237 | mgr.setRole(NID_LOCAL, DID1, MASTER); |
| 238 | mgr.setRole(NID_LOCAL, DID2, MASTER); |
| 239 | assertEquals("wrong local role:", MASTER, mgr.getLocalRole(DID1)); |
| 240 | assertEquals("wrong local role:", MASTER, mgr.getLocalRole(DID2)); |
| 241 | assertEquals("wrong master:", NID_LOCAL, mgr.getMasterFor(DID1)); |
| 242 | assertEquals("wrong master:", NID_LOCAL, mgr.getMasterFor(DID2)); |
| 243 | |
| 244 | //do region balancing |
| 245 | mgr.useRegionForBalanceRoles = true; |
| 246 | mgr.balanceRoles(); |
| 247 | assertEquals("wrong master:", NID1, mgr.getMasterFor(DID1)); |
| 248 | assertEquals("wrong master:", NID1, mgr.getMasterFor(DID2)); |
| 249 | |
| 250 | // make N1 inactive |
| 251 | testClusterService.put(CNODE1, ControllerNode.State.INACTIVE); |
| 252 | mgr.balanceRoles(); |
| 253 | assertEquals("wrong master:", NID2, mgr.getMasterFor(DID1)); |
| 254 | assertEquals("wrong master:", NID2, mgr.getMasterFor(DID2)); |
| 255 | |
| 256 | } |
| 257 | |
| 258 | @Test |
| 259 | public void balanceWithRegion2() { |
| 260 | //set up region - 2 sets of masters with (3 nodes, 1 node) |
| 261 | Set<NodeId> masterSet1 = ImmutableSet.of(NID1, NID3, NID4); |
| 262 | Set<NodeId> masterSet2 = ImmutableSet.of(NID2); |
| 263 | List<Set<NodeId>> masters = ImmutableList.of(masterSet1, masterSet2); |
| 264 | Region r = regionManager.createRegion(RID1, "R1", METRO, masters); |
| 265 | Set<DeviceId> deviceIdsOrig = ImmutableSet.of(DID1, DID2, DID3, DEV_OTHER); |
| 266 | regionManager.addDevices(RID1, deviceIdsOrig); |
| 267 | Set<DeviceId> deviceIds = regionManager.getRegionDevices(RID1); |
| 268 | assertEquals("incorrect device count", deviceIdsOrig.size(), deviceIds.size()); |
| 269 | assertEquals("incorrect devices in region", deviceIdsOrig, deviceIds); |
| 270 | |
| 271 | testClusterService.put(CNODE1, ControllerNode.State.ACTIVE); |
| 272 | testClusterService.put(CNODE2, ControllerNode.State.ACTIVE); |
| 273 | testClusterService.put(CNODE3, ControllerNode.State.ACTIVE); |
| 274 | testClusterService.put(CNODE4, ControllerNode.State.ACTIVE); |
| 275 | |
| 276 | //set master to non region nodes |
| 277 | deviceIdsOrig.forEach(deviceId1 -> mgr.setRole(NID_LOCAL, deviceId1, MASTER)); |
| 278 | checkDeviceMasters(deviceIds, Sets.newHashSet(NID_LOCAL), deviceId -> |
| 279 | assertEquals("wrong local role:", MASTER, mgr.getLocalRole(deviceId))); |
| 280 | |
| 281 | //do region balancing |
| 282 | mgr.useRegionForBalanceRoles = true; |
| 283 | mgr.balanceRoles(); |
| 284 | Set<NodeId> expectedMasters = Sets.newHashSet(NID1, NID3, NID4); |
| 285 | checkDeviceMasters(deviceIds, expectedMasters); |
| 286 | |
| 287 | // make N1 inactive |
| 288 | testClusterService.put(CNODE1, ControllerNode.State.INACTIVE); |
| 289 | expectedMasters.remove(NID1); |
| 290 | mgr.balanceRoles(); |
| 291 | checkDeviceMasters(deviceIds, expectedMasters); |
| 292 | |
| 293 | // make N4 inactive |
| 294 | testClusterService.put(CNODE4, ControllerNode.State.INACTIVE); |
| 295 | expectedMasters.remove(NID4); |
| 296 | mgr.balanceRoles(); |
| 297 | checkDeviceMasters(deviceIds, expectedMasters); |
| 298 | |
| 299 | // make N3 inactive |
| 300 | testClusterService.put(CNODE3, ControllerNode.State.INACTIVE); |
| 301 | expectedMasters = Sets.newHashSet(NID2); |
| 302 | mgr.balanceRoles(); |
| 303 | checkDeviceMasters(deviceIds, expectedMasters); |
| 304 | |
| 305 | // make N3 active |
| 306 | testClusterService.put(CNODE3, ControllerNode.State.ACTIVE); |
| 307 | expectedMasters = Sets.newHashSet(NID3); |
| 308 | mgr.balanceRoles(); |
| 309 | checkDeviceMasters(deviceIds, expectedMasters); |
| 310 | |
| 311 | // make N4 active |
| 312 | testClusterService.put(CNODE4, ControllerNode.State.ACTIVE); |
| 313 | expectedMasters.add(NID4); |
| 314 | mgr.balanceRoles(); |
| 315 | checkDeviceMasters(deviceIds, expectedMasters); |
| 316 | |
| 317 | // make N1 active |
| 318 | testClusterService.put(CNODE1, ControllerNode.State.ACTIVE); |
| 319 | expectedMasters.add(NID1); |
| 320 | mgr.balanceRoles(); |
| 321 | checkDeviceMasters(deviceIds, expectedMasters); |
| 322 | } |
| 323 | |
| 324 | private void checkDeviceMasters(Set<DeviceId> deviceIds, Set<NodeId> expectedMasters) { |
| 325 | checkDeviceMasters(deviceIds, expectedMasters, null); |
| 326 | } |
| 327 | |
| 328 | private void checkDeviceMasters(Set<DeviceId> deviceIds, Set<NodeId> expectedMasters, |
| 329 | Consumer<DeviceId> checkRole) { |
| 330 | // each device's master must be contained in the list of expectedMasters |
Sho SHIMIZU | a09e1bb | 2016-08-01 14:25:25 -0700 | [diff] [blame] | 331 | deviceIds.forEach(deviceId -> { |
Claudine Chiu | dce0815 | 2016-03-09 18:19:28 +0000 | [diff] [blame] | 332 | assertTrue("wrong master:", expectedMasters.contains(mgr.getMasterFor(deviceId))); |
| 333 | if (checkRole != null) { |
| 334 | checkRole.accept(deviceId); |
| 335 | } |
| 336 | }); |
| 337 | // each node in expectedMasters must have approximately the same number of devices |
| 338 | if (expectedMasters.size() > 1) { |
| 339 | int minValue = Integer.MAX_VALUE; |
| 340 | int maxDevices = -1; |
| 341 | for (NodeId nodeId: expectedMasters) { |
| 342 | int numDevicesManagedByNode = mgr.getDevicesOf(nodeId).size(); |
| 343 | if (numDevicesManagedByNode < minValue) { |
| 344 | minValue = numDevicesManagedByNode; |
| 345 | } |
| 346 | if (numDevicesManagedByNode > maxDevices) { |
| 347 | maxDevices = numDevicesManagedByNode; |
| 348 | } |
| 349 | assertTrue("not balanced:", maxDevices - minValue <= 1); |
| 350 | } |
| 351 | } |
| 352 | } |
| 353 | |
| 354 | private final class TestClusterService extends StaticClusterService { |
Ayaka Koshibe | 406d010 | 2014-09-24 16:08:12 -0700 | [diff] [blame] | 355 | |
| 356 | ControllerNode local = new DefaultControllerNode(NID_LOCAL, LOCALHOST); |
| 357 | |
| 358 | @Override |
| 359 | public ControllerNode getLocalNode() { |
| 360 | return local; |
| 361 | } |
| 362 | |
Claudine Chiu | dce0815 | 2016-03-09 18:19:28 +0000 | [diff] [blame] | 363 | public void put(ControllerNode cn, ControllerNode.State state) { |
| 364 | nodes.put(cn.id(), cn); |
| 365 | nodeStates.put(cn.id(), state); |
Ayaka Koshibe | 406d010 | 2014-09-24 16:08:12 -0700 | [diff] [blame] | 366 | } |
Ayaka Koshibe | 406d010 | 2014-09-24 16:08:12 -0700 | [diff] [blame] | 367 | } |
Yuta HIGUCHI | 0c6e184 | 2014-11-05 22:34:23 -0800 | [diff] [blame] | 368 | |
| 369 | private final class TestSimpleMastershipStore extends SimpleMastershipStore |
| 370 | implements MastershipStore { |
| 371 | |
| 372 | public TestSimpleMastershipStore(ClusterService clusterService) { |
| 373 | super.clusterService = clusterService; |
| 374 | } |
| 375 | } |
Claudine Chiu | dce0815 | 2016-03-09 18:19:28 +0000 | [diff] [blame] | 376 | |
| 377 | private class TestRegionManager extends RegionManager { |
| 378 | TestRegionManager() { |
| 379 | eventDispatcher = new TestEventDispatcher(); |
Simon Hunt | 5361221 | 2016-12-04 17:19:52 -0800 | [diff] [blame] | 380 | networkConfigService = new NetworkConfigServiceAdapter(); |
Claudine Chiu | dce0815 | 2016-03-09 18:19:28 +0000 | [diff] [blame] | 381 | } |
| 382 | } |
Ayaka Koshibe | 406d010 | 2014-09-24 16:08:12 -0700 | [diff] [blame] | 383 | } |