Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 1 | # |
Zsolt Haraszti | 3eb27a5 | 2017-01-03 21:56:48 -0800 | [diff] [blame] | 2 | # Copyright 2017 the original author or authors. |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -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 | # |
| 16 | |
Zsolt Haraszti | ac9310d | 2016-09-20 12:56:35 -0700 | [diff] [blame] | 17 | import re |
Zsolt Haraszti | 023ea7c | 2016-10-16 19:30:34 -0700 | [diff] [blame] | 18 | |
Zsolt Haraszti | ac9310d | 2016-09-20 12:56:35 -0700 | [diff] [blame] | 19 | from hash_ring import HashRing |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 20 | from structlog import get_logger |
| 21 | from twisted.internet import reactor |
| 22 | from twisted.internet.base import DelayedCall |
khenaidoo | 5431e4c | 2017-08-17 15:05:40 -0400 | [diff] [blame] | 23 | from twisted.internet.defer import inlineCallbacks, DeferredList, returnValue |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 24 | from simplejson import dumps, loads |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 25 | |
Zsolt Haraszti | 023ea7c | 2016-10-16 19:30:34 -0700 | [diff] [blame] | 26 | from common.utils.asleep import asleep |
khenaidoo | 08d48d2 | 2017-06-29 19:42:49 -0400 | [diff] [blame] | 27 | from common.utils.id_generation import get_next_core_id |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 28 | |
Zsolt Haraszti | 2bdb6b3 | 2016-11-03 16:56:17 -0700 | [diff] [blame] | 29 | log = get_logger() |
| 30 | |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 31 | |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 32 | class ConfigMappingException(Exception): |
| 33 | pass |
| 34 | |
| 35 | |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 36 | class Leader(object): |
| 37 | """ |
| 38 | A single instance of this object shall exist across the whole cluster. |
| 39 | This is guaranteed by the coordinator which instantiates this class |
| 40 | only when it secured the leadership lock, as well as calling the halt() |
| 41 | method in cases it looses the leadership lock. |
| 42 | """ |
| 43 | |
Zsolt Haraszti | ac9310d | 2016-09-20 12:56:35 -0700 | [diff] [blame] | 44 | ID_EXTRACTOR = '^(%s)([^/]+)$' |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 45 | CORE_STORE_KEY_EXTRACTOR = '^%s(?P<core_store_id>[^/]+)/root$' |
khenaidoo | 82ce00d | 2017-08-15 12:01:46 -0400 | [diff] [blame] | 46 | CORE_NUMBER_EXTRACTOR = '^.*\.([0-9]+)\..*$' |
| 47 | START_TIMESTAMP_EXTRACTOR = '^.*\..*\..*_([0-9]+)$' |
khenaidoo | 5431e4c | 2017-08-17 15:05:40 -0400 | [diff] [blame] | 48 | ASSIGNMENT_ID_EXTRACTOR = '^(%s)([^/]+)/core_store$' |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 49 | |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 50 | # Public methods: |
| 51 | |
| 52 | def __init__(self, coordinator): |
Zsolt Haraszti | ac9310d | 2016-09-20 12:56:35 -0700 | [diff] [blame] | 53 | |
| 54 | self.coord = coordinator |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 55 | self.halted = False |
Zsolt Haraszti | ac9310d | 2016-09-20 12:56:35 -0700 | [diff] [blame] | 56 | self.soak_time = 3 # soak till membership/workload changes settle |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 57 | |
| 58 | self.workload = [] |
| 59 | self.members = [] |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 60 | self.core_store_ids = [] |
| 61 | self.core_store_assignment = None |
| 62 | |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 63 | self.reassignment_soak_timer = None |
khenaidoo | b1602a3 | 2017-07-27 16:59:52 -0400 | [diff] [blame] | 64 | self.core_store_reassignment_soak_timer = None |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 65 | |
Zsolt Haraszti | ac9310d | 2016-09-20 12:56:35 -0700 | [diff] [blame] | 66 | self.workload_id_match = re.compile( |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 67 | self.ID_EXTRACTOR % self.coord.workload_prefix).match |
Zsolt Haraszti | ac9310d | 2016-09-20 12:56:35 -0700 | [diff] [blame] | 68 | |
| 69 | self.member_id_match = re.compile( |
Rouzbahan Rashidi-Tabrizi | 1c3eba8 | 2016-10-27 21:47:18 -0400 | [diff] [blame] | 70 | self.ID_EXTRACTOR % self.coord.membership_prefix).match |
Zsolt Haraszti | ac9310d | 2016-09-20 12:56:35 -0700 | [diff] [blame] | 71 | |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 72 | self.core_data_id_match = re.compile( |
| 73 | self.CORE_STORE_KEY_EXTRACTOR % self.coord.core_store_prefix).match |
| 74 | |
khenaidoo | 82ce00d | 2017-08-15 12:01:46 -0400 | [diff] [blame] | 75 | self.core_match = re.compile(self.CORE_NUMBER_EXTRACTOR).match |
| 76 | self.timestamp_match = re.compile(self.START_TIMESTAMP_EXTRACTOR ).match |
| 77 | |
khenaidoo | 5431e4c | 2017-08-17 15:05:40 -0400 | [diff] [blame] | 78 | self.assignment_id_match = re.compile( |
| 79 | self.ASSIGNMENT_ID_EXTRACTOR % self.coord.assignment_prefix).match |
khenaidoo | 82ce00d | 2017-08-15 12:01:46 -0400 | [diff] [blame] | 80 | |
khenaidoo | d6e0e80 | 2017-08-29 19:55:44 -0400 | [diff] [blame] | 81 | self.members_tracking_sleep_to_prevent_flood = \ |
| 82 | self.coord.leader_config.get((self.coord.leader_config[ |
| 83 | 'members_track_error_to_prevent_flood']), 1) |
| 84 | |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 85 | @inlineCallbacks |
| 86 | def start(self): |
Zsolt Haraszti | 2bdb6b3 | 2016-11-03 16:56:17 -0700 | [diff] [blame] | 87 | log.debug('starting') |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 88 | # yield self._validate_workload() |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 89 | yield self._start_tracking_assignments() |
Zsolt Haraszti | 2bdb6b3 | 2016-11-03 16:56:17 -0700 | [diff] [blame] | 90 | log.info('started') |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 91 | |
Zsolt Haraszti | 2bdb6b3 | 2016-11-03 16:56:17 -0700 | [diff] [blame] | 92 | def stop(self): |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 93 | """Suspend leadership duties immediately""" |
Zsolt Haraszti | 2bdb6b3 | 2016-11-03 16:56:17 -0700 | [diff] [blame] | 94 | log.debug('stopping') |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 95 | self.halted = True |
| 96 | |
| 97 | # any active cancellations, releases, etc., should happen here |
| 98 | if isinstance(self.reassignment_soak_timer, DelayedCall): |
Zsolt Haraszti | ac9310d | 2016-09-20 12:56:35 -0700 | [diff] [blame] | 99 | if not self.reassignment_soak_timer.called: |
| 100 | self.reassignment_soak_timer.cancel() |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 101 | |
khenaidoo | b1602a3 | 2017-07-27 16:59:52 -0400 | [diff] [blame] | 102 | if isinstance(self.core_store_reassignment_soak_timer, DelayedCall): |
| 103 | if not self.core_store_reassignment_soak_timer.called: |
| 104 | self.core_store_reassignment_soak_timer.cancel() |
| 105 | |
Zsolt Haraszti | 2bdb6b3 | 2016-11-03 16:56:17 -0700 | [diff] [blame] | 106 | log.info('stopped') |
| 107 | |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 108 | # Private methods: |
| 109 | |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 110 | |
| 111 | def _start_tracking_assignments(self): |
| 112 | """ |
| 113 | We must track both the cluster member list as well as the workload |
| 114 | list. Upon change in either, we must rerun our sharding algorithm |
| 115 | and reassign work as/if needed. |
| 116 | """ |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 117 | reactor.callLater(0, self._track_members, 0) |
| 118 | |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 119 | @inlineCallbacks |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 120 | def _get_core_store_mappings(self): |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 121 | try: |
| 122 | # Get the mapping record |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 123 | (_, mappings) = yield self.coord.kv_get( |
khenaidoo | 08d48d2 | 2017-06-29 19:42:49 -0400 | [diff] [blame] | 124 | self.coord.core_store_assignment_key, recurse=True) |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 125 | if mappings: |
| 126 | self.core_store_assignment = loads(mappings[0]['Value']) |
| 127 | return |
| 128 | else: # Key has not been created yet |
| 129 | # Create the key with an empty dictionary value |
| 130 | value = dict() |
khenaidoo | 08d48d2 | 2017-06-29 19:42:49 -0400 | [diff] [blame] | 131 | result = yield self.coord.kv_put( |
| 132 | self.coord.core_store_assignment_key, |
| 133 | dumps(value)) |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 134 | if not result: |
| 135 | raise ConfigMappingException(self.instance_id) |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 136 | |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 137 | # Ensure the record was created |
| 138 | (_, mappings) = yield self.coord.kv_get( |
khenaidoo | 08d48d2 | 2017-06-29 19:42:49 -0400 | [diff] [blame] | 139 | self.coord.core_store_assignment_key, recurse=True) |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 140 | |
| 141 | self.core_store_assignment = loads(mappings[0]['Value']) |
| 142 | |
| 143 | except Exception, e: |
| 144 | log.exception('error', e=e) |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 145 | |
| 146 | @inlineCallbacks |
| 147 | def _update_core_store_references(self): |
| 148 | try: |
| 149 | # Get the current set of configs keys |
| 150 | (_, results) = yield self.coord.kv_get( |
| 151 | self.coord.core_store_prefix, recurse=False, keys=True) |
| 152 | |
| 153 | matches = (self.core_data_id_match(e) for e in results or []) |
| 154 | core_ids = [m.group(1) for m in matches if m is not None] |
| 155 | |
| 156 | self.core_store_ids = core_ids |
| 157 | |
| 158 | # Update the config mapping |
| 159 | self._get_core_store_mappings() |
| 160 | |
| 161 | log.debug('core-data', core_ids=core_ids, |
| 162 | assignment=self.core_store_assignment) |
| 163 | |
| 164 | except Exception, e: |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 165 | log.exception('error-update-store', e=e) |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 166 | |
khenaidoo | 82ce00d | 2017-08-15 12:01:46 -0400 | [diff] [blame] | 167 | def _sanitize_member_list(self, members): |
| 168 | # This method removes any duplicates from the member list using the |
| 169 | # voltha number from the member id and the time that voltha instance |
| 170 | # started, again from the member id. This method is meaningful only |
| 171 | # in the swarm cluster. In a non-cluster environment the member id |
| 172 | # is formatted differently. In such a case, the method below will |
| 173 | # create an exception and return the member list as is. |
| 174 | |
| 175 | try: |
| 176 | unique_members = {} |
| 177 | update_occurred = False |
| 178 | log.info('members', members=members) |
| 179 | for member in members: |
| 180 | log.info('member', member=member) |
| 181 | # Extract the swarm assigned number of the voltha instance |
| 182 | voltha_number = self.core_match(member['id']).group(1) |
| 183 | timestamp = self.timestamp_match(member['id']).group(1) |
| 184 | if voltha_number not in unique_members: |
| 185 | unique_members[voltha_number] = {'id': member['id'], |
| 186 | 'timestamp': timestamp, |
| 187 | 'host': member['host']} |
| 188 | else: |
| 189 | # Verify whether if this member has the latest timestamp. If |
| 190 | # yes, overwrite the previous one |
| 191 | if unique_members[voltha_number]['timestamp'] < timestamp: |
| 192 | unique_members[voltha_number] = {'id': member['id'], |
| 193 | 'timestamp': timestamp, |
| 194 | 'host': member['host']} |
khenaidoo | 5431e4c | 2017-08-17 15:05:40 -0400 | [diff] [blame] | 195 | update_occurred = True |
khenaidoo | 82ce00d | 2017-08-15 12:01:46 -0400 | [diff] [blame] | 196 | |
| 197 | if update_occurred: |
| 198 | updated_members = [] |
| 199 | for _, unique_member in unique_members.iteritems(): |
| 200 | updated_members.append({'host': unique_member['host'], |
| 201 | 'id': unique_member['id']}) |
| 202 | return updated_members |
| 203 | else: |
| 204 | return members |
| 205 | except: |
| 206 | return members |
| 207 | |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 208 | @inlineCallbacks |
khenaidoo | 5431e4c | 2017-08-17 15:05:40 -0400 | [diff] [blame] | 209 | def _is_temporal_state(self, members): |
| 210 | try: |
| 211 | # First get the current core assignments |
| 212 | (_, results) = yield self.coord.kv_get( |
| 213 | self.coord.assignment_prefix, |
| 214 | recurse=True) |
| 215 | |
| 216 | log.debug('core-assignments', assignment=results) |
| 217 | if results: |
| 218 | old_assignment = [ |
| 219 | {'id': self.assignment_id_match(e['Key']).group(2), |
| 220 | 'core': e['Value']} |
| 221 | for e in results] |
| 222 | |
| 223 | # If there are no curr_assignments then we are starting the |
| 224 | # system. In this case we should keep processing |
| 225 | if len(old_assignment) == 0: |
| 226 | returnValue(False) |
| 227 | |
| 228 | # Tackle the simplest scenario - #members >= #old_assignment |
| 229 | if len(members) >= len(old_assignment): |
| 230 | returnValue(False) |
| 231 | |
| 232 | # Everything else is a temporal state |
| 233 | log.info('temporal-state-detected', members=members, |
| 234 | old_assignments=old_assignment) |
| 235 | |
| 236 | returnValue(True) |
| 237 | else: |
| 238 | returnValue(False) |
| 239 | except Exception as e: |
| 240 | log.exception('temporal-state-error', e=e) |
| 241 | returnValue(True) |
| 242 | |
| 243 | @inlineCallbacks |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 244 | def _track_members(self, index): |
khenaidoo | b1602a3 | 2017-07-27 16:59:52 -0400 | [diff] [blame] | 245 | previous_index = index |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 246 | try: |
khenaidoo | d6e0e80 | 2017-08-29 19:55:44 -0400 | [diff] [blame] | 247 | log.info('member-tracking-before') |
| 248 | is_timeout, (tmp_index, results) = yield \ |
| 249 | self.coord.consul_get_with_timeout( |
| 250 | key=self.coord.membership_prefix, |
| 251 | recurse=True, |
| 252 | index=index, |
| 253 | timeout=10) |
khenaidoo | 5431e4c | 2017-08-17 15:05:40 -0400 | [diff] [blame] | 254 | # Check whether we are still the leader - a new regime may be in |
| 255 | # place by the time we see a membership update |
| 256 | if self.halted: |
| 257 | log.info('I am no longer the leader') |
| 258 | return |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 259 | |
khenaidoo | d6e0e80 | 2017-08-29 19:55:44 -0400 | [diff] [blame] | 260 | if is_timeout: |
| 261 | log.debug('timeout-or-no-membership-changed') |
| 262 | return |
| 263 | |
khenaidoo | b1602a3 | 2017-07-27 16:59:52 -0400 | [diff] [blame] | 264 | # This can happen if consul went down and came back with no data |
| 265 | if not results: |
| 266 | log.error('no-active-members') |
| 267 | # Bail out of leadership and go for an early election |
| 268 | self.coord._just_lost_leadership() |
| 269 | return |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 270 | |
khenaidoo | d6e0e80 | 2017-08-29 19:55:44 -0400 | [diff] [blame] | 271 | # After timeout event the index returned from |
| 272 | # consul_get_with_timeout is None. If we are here it's not a |
| 273 | # timeout, therefore the index is a valid one. |
| 274 | index=tmp_index |
| 275 | |
| 276 | log.info('membership-tracking-data', index=index, results=results) |
| 277 | |
khenaidoo | b1602a3 | 2017-07-27 16:59:52 -0400 | [diff] [blame] | 278 | if previous_index != index: |
| 279 | log.info('membership-updated', |
| 280 | previous_index=previous_index, index=index) |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 281 | |
khenaidoo | b1602a3 | 2017-07-27 16:59:52 -0400 | [diff] [blame] | 282 | # Rebuild the membership, if any |
| 283 | |
| 284 | # Only members with valid session are considered active |
| 285 | members = [{'id': self.member_id_match(e['Key']).group(2), |
| 286 | 'host': loads(e['Value'])['host_address']} |
| 287 | for e in results if 'Session' in e] |
| 288 | |
khenaidoo | 82ce00d | 2017-08-15 12:01:46 -0400 | [diff] [blame] | 289 | if members: |
| 290 | updated_members = self._sanitize_member_list(members) |
| 291 | else: |
| 292 | updated_members = None |
| 293 | |
| 294 | log.info('active-members', active_members=members, |
| 295 | sanitized_members=updated_members) |
khenaidoo | b1602a3 | 2017-07-27 16:59:52 -0400 | [diff] [blame] | 296 | |
khenaidoo | 5431e4c | 2017-08-17 15:05:40 -0400 | [diff] [blame] | 297 | # Check if we are in a temporal state. If true wait for the |
| 298 | # next membership changes |
| 299 | temporal_state = yield self._is_temporal_state(updated_members) |
| 300 | if temporal_state: |
| 301 | log.info('temporal-state-detected') |
| 302 | pass # Wait for next member list change |
| 303 | elif updated_members != self.members: |
| 304 | # if the two sets are the same |
khenaidoo | b1602a3 | 2017-07-27 16:59:52 -0400 | [diff] [blame] | 305 | # update the current set of config |
| 306 | yield self._update_core_store_references() |
| 307 | log.info('membership-changed', |
| 308 | prev_members=self.members, |
khenaidoo | 82ce00d | 2017-08-15 12:01:46 -0400 | [diff] [blame] | 309 | curr_members=updated_members, |
khenaidoo | b1602a3 | 2017-07-27 16:59:52 -0400 | [diff] [blame] | 310 | core_store_mapping=self.core_store_assignment) |
khenaidoo | 82ce00d | 2017-08-15 12:01:46 -0400 | [diff] [blame] | 311 | self.members = updated_members |
khenaidoo | b1602a3 | 2017-07-27 16:59:52 -0400 | [diff] [blame] | 312 | self._restart_core_store_reassignment_soak_timer() |
| 313 | else: |
| 314 | log.debug('no-membership-change', index=index) |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 315 | |
| 316 | except Exception, e: |
Zsolt Haraszti | 2bdb6b3 | 2016-11-03 16:56:17 -0700 | [diff] [blame] | 317 | log.exception('members-track-error', e=e) |
Rouzbahan Rashidi-Tabrizi | 1c3eba8 | 2016-10-27 21:47:18 -0400 | [diff] [blame] | 318 | # to prevent flood |
khenaidoo | d6e0e80 | 2017-08-29 19:55:44 -0400 | [diff] [blame] | 319 | yield asleep(self.members_tracking_sleep_to_prevent_flood) |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 320 | finally: |
| 321 | if not self.halted: |
khenaidoo | 5431e4c | 2017-08-17 15:05:40 -0400 | [diff] [blame] | 322 | reactor.callLater(1, self._track_members, index) |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 323 | |
| 324 | def _restart_reassignment_soak_timer(self): |
| 325 | |
| 326 | if self.reassignment_soak_timer is not None: |
| 327 | assert isinstance(self.reassignment_soak_timer, DelayedCall) |
Zsolt Haraszti | 8dc1f5e | 2016-09-18 23:35:39 -0700 | [diff] [blame] | 328 | if not self.reassignment_soak_timer.called: |
| 329 | self.reassignment_soak_timer.cancel() |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 330 | |
| 331 | self.reassignment_soak_timer = reactor.callLater( |
| 332 | self.soak_time, self._reassign_work) |
| 333 | |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 334 | def _restart_core_store_reassignment_soak_timer(self): |
| 335 | |
khenaidoo | b1602a3 | 2017-07-27 16:59:52 -0400 | [diff] [blame] | 336 | if self.core_store_reassignment_soak_timer is not None: |
| 337 | assert isinstance(self.core_store_reassignment_soak_timer, DelayedCall) |
| 338 | if not self.core_store_reassignment_soak_timer.called: |
| 339 | self.core_store_reassignment_soak_timer.cancel() |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 340 | |
khenaidoo | b1602a3 | 2017-07-27 16:59:52 -0400 | [diff] [blame] | 341 | self.core_store_reassignment_soak_timer = reactor.callLater( |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 342 | self.soak_time, self._reassign_core_stores) |
| 343 | |
| 344 | @inlineCallbacks |
| 345 | def _reassign_core_stores(self): |
| 346 | |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 347 | def _get_core_data_id_from_instance(instance_name): |
| 348 | for id, instance in self.core_store_assignment.iteritems(): |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 349 | if instance and instance['id'] == instance_name: |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 350 | return id |
| 351 | |
| 352 | try: |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 353 | log.info('core-members', curr_members=self.members, |
| 354 | prev_members=self.core_store_assignment) |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 355 | |
| 356 | # 1. clear the mapping for instances that are no longer running |
| 357 | updated_mapping = dict() |
| 358 | existing_active_config_members = set() |
| 359 | cleared_config_ids = set() |
| 360 | inactive_members = set() |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 361 | if self.core_store_assignment: |
| 362 | for id, instance in self.core_store_assignment.iteritems(): |
| 363 | if instance not in self.members: |
| 364 | updated_mapping[id] = None |
| 365 | cleared_config_ids.add(id) |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 366 | if instance: |
| 367 | inactive_members.add(instance['id']) |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 368 | else: |
| 369 | updated_mapping[id] = instance |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 370 | existing_active_config_members.add(instance['id']) |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 371 | |
| 372 | # 2. Update the mapping with the new set |
| 373 | current_id = max(self.core_store_assignment) \ |
khenaidoo | 08d48d2 | 2017-06-29 19:42:49 -0400 | [diff] [blame] | 374 | if self.core_store_assignment else '0000' |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 375 | for instance in self.members: |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 376 | if instance['id'] not in existing_active_config_members: |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 377 | # Add the member to the config map |
| 378 | if cleared_config_ids: |
| 379 | # There is an empty slot |
| 380 | next_id = cleared_config_ids.pop() |
| 381 | updated_mapping[next_id] = instance |
| 382 | else: |
| 383 | # There are no empty slot, create new ids |
khenaidoo | 08d48d2 | 2017-06-29 19:42:49 -0400 | [diff] [blame] | 384 | current_id = get_next_core_id(current_id) |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 385 | updated_mapping[current_id] = instance |
| 386 | |
| 387 | self.core_store_assignment = updated_mapping |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 388 | log.info('updated-assignment', |
khenaidoo | 08d48d2 | 2017-06-29 19:42:49 -0400 | [diff] [blame] | 389 | core_store_assignment=self.core_store_assignment, |
| 390 | inactive_members=inactive_members) |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 391 | |
| 392 | # 3. save the mapping into consul |
khenaidoo | 08d48d2 | 2017-06-29 19:42:49 -0400 | [diff] [blame] | 393 | yield self.coord.kv_put(self.coord.core_store_assignment_key, |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 394 | dumps(self.core_store_assignment)) |
| 395 | |
| 396 | # 4. Assign the new workload to the newly created members |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 397 | curr_members_set = set([m['id'] for m in self.members]) |
khenaidoo | 08d48d2 | 2017-06-29 19:42:49 -0400 | [diff] [blame] | 398 | new_members = curr_members_set.difference( |
| 399 | existing_active_config_members) |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 400 | for new_member_id in new_members: |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 401 | yield self.coord.kv_put( |
| 402 | self.coord.assignment_prefix |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 403 | + new_member_id + '/' + |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 404 | self.coord.core_storage_suffix, |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 405 | _get_core_data_id_from_instance(new_member_id)) |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 406 | |
| 407 | # 5. Remove non-existent members |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 408 | for member_id in inactive_members: |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 409 | yield self.coord.kv_delete( |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 410 | self.coord.assignment_prefix + member_id, recurse=True) |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 411 | yield self.coord.kv_delete( |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 412 | self.coord.membership_prefix + member_id, |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 413 | recurse=True) |
| 414 | |
| 415 | except Exception as e: |
| 416 | log.exception('config-reassignment-failure', e=e) |
| 417 | self._restart_core_store_reassignment_soak_timer() |