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 |
| 23 | from twisted.internet.defer import inlineCallbacks, DeferredList |
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)([^/]+)$' |
| 45 | ASSIGNMENT_EXTRACTOR = '^%s(?P<member_id>[^/]+)/(?P<work_id>[^/]+)$' |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 46 | CORE_STORE_KEY_EXTRACTOR = '^%s(?P<core_store_id>[^/]+)/root$' |
khenaidoo | 82ce00d | 2017-08-15 12:01:46 -0400 | [diff] [blame^] | 47 | CORE_NUMBER_EXTRACTOR = '^.*\.([0-9]+)\..*$' |
| 48 | START_TIMESTAMP_EXTRACTOR = '^.*\..*\..*_([0-9]+)$' |
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 | |
Zsolt Haraszti | ac9310d | 2016-09-20 12:56:35 -0700 | [diff] [blame] | 75 | self.assignment_match = re.compile( |
Rouzbahan Rashidi-Tabrizi | 1c3eba8 | 2016-10-27 21:47:18 -0400 | [diff] [blame] | 76 | self.ASSIGNMENT_EXTRACTOR % self.coord.assignment_prefix).match |
Zsolt Haraszti | ac9310d | 2016-09-20 12:56:35 -0700 | [diff] [blame] | 77 | |
khenaidoo | 82ce00d | 2017-08-15 12:01:46 -0400 | [diff] [blame^] | 78 | self.core_match = re.compile(self.CORE_NUMBER_EXTRACTOR).match |
| 79 | self.timestamp_match = re.compile(self.START_TIMESTAMP_EXTRACTOR ).match |
| 80 | |
| 81 | |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 82 | @inlineCallbacks |
| 83 | def start(self): |
Zsolt Haraszti | 2bdb6b3 | 2016-11-03 16:56:17 -0700 | [diff] [blame] | 84 | log.debug('starting') |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 85 | # yield self._validate_workload() |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 86 | yield self._start_tracking_assignments() |
Zsolt Haraszti | 2bdb6b3 | 2016-11-03 16:56:17 -0700 | [diff] [blame] | 87 | log.info('started') |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 88 | |
Zsolt Haraszti | 2bdb6b3 | 2016-11-03 16:56:17 -0700 | [diff] [blame] | 89 | def stop(self): |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 90 | """Suspend leadership duties immediately""" |
Zsolt Haraszti | 2bdb6b3 | 2016-11-03 16:56:17 -0700 | [diff] [blame] | 91 | log.debug('stopping') |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 92 | self.halted = True |
| 93 | |
| 94 | # any active cancellations, releases, etc., should happen here |
| 95 | if isinstance(self.reassignment_soak_timer, DelayedCall): |
Zsolt Haraszti | ac9310d | 2016-09-20 12:56:35 -0700 | [diff] [blame] | 96 | if not self.reassignment_soak_timer.called: |
| 97 | self.reassignment_soak_timer.cancel() |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 98 | |
khenaidoo | b1602a3 | 2017-07-27 16:59:52 -0400 | [diff] [blame] | 99 | if isinstance(self.core_store_reassignment_soak_timer, DelayedCall): |
| 100 | if not self.core_store_reassignment_soak_timer.called: |
| 101 | self.core_store_reassignment_soak_timer.cancel() |
| 102 | |
Zsolt Haraszti | 2bdb6b3 | 2016-11-03 16:56:17 -0700 | [diff] [blame] | 103 | log.info('stopped') |
| 104 | |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 105 | # Private methods: |
| 106 | |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 107 | |
| 108 | def _start_tracking_assignments(self): |
| 109 | """ |
| 110 | We must track both the cluster member list as well as the workload |
| 111 | list. Upon change in either, we must rerun our sharding algorithm |
| 112 | and reassign work as/if needed. |
| 113 | """ |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 114 | reactor.callLater(0, self._track_members, 0) |
| 115 | |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 116 | @inlineCallbacks |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 117 | def _get_core_store_mappings(self): |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 118 | try: |
| 119 | # Get the mapping record |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 120 | (_, mappings) = yield self.coord.kv_get( |
khenaidoo | 08d48d2 | 2017-06-29 19:42:49 -0400 | [diff] [blame] | 121 | self.coord.core_store_assignment_key, recurse=True) |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 122 | if mappings: |
| 123 | self.core_store_assignment = loads(mappings[0]['Value']) |
| 124 | return |
| 125 | else: # Key has not been created yet |
| 126 | # Create the key with an empty dictionary value |
| 127 | value = dict() |
khenaidoo | 08d48d2 | 2017-06-29 19:42:49 -0400 | [diff] [blame] | 128 | result = yield self.coord.kv_put( |
| 129 | self.coord.core_store_assignment_key, |
| 130 | dumps(value)) |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 131 | if not result: |
| 132 | raise ConfigMappingException(self.instance_id) |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 133 | |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 134 | # Ensure the record was created |
| 135 | (_, mappings) = yield self.coord.kv_get( |
khenaidoo | 08d48d2 | 2017-06-29 19:42:49 -0400 | [diff] [blame] | 136 | self.coord.core_store_assignment_key, recurse=True) |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 137 | |
| 138 | self.core_store_assignment = loads(mappings[0]['Value']) |
| 139 | |
| 140 | except Exception, e: |
| 141 | log.exception('error', e=e) |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 142 | |
| 143 | @inlineCallbacks |
| 144 | def _update_core_store_references(self): |
| 145 | try: |
| 146 | # Get the current set of configs keys |
| 147 | (_, results) = yield self.coord.kv_get( |
| 148 | self.coord.core_store_prefix, recurse=False, keys=True) |
| 149 | |
| 150 | matches = (self.core_data_id_match(e) for e in results or []) |
| 151 | core_ids = [m.group(1) for m in matches if m is not None] |
| 152 | |
| 153 | self.core_store_ids = core_ids |
| 154 | |
| 155 | # Update the config mapping |
| 156 | self._get_core_store_mappings() |
| 157 | |
| 158 | log.debug('core-data', core_ids=core_ids, |
| 159 | assignment=self.core_store_assignment) |
| 160 | |
| 161 | except Exception, e: |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 162 | log.exception('error-update-store', e=e) |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 163 | |
khenaidoo | 82ce00d | 2017-08-15 12:01:46 -0400 | [diff] [blame^] | 164 | def _sanitize_member_list(self, members): |
| 165 | # This method removes any duplicates from the member list using the |
| 166 | # voltha number from the member id and the time that voltha instance |
| 167 | # started, again from the member id. This method is meaningful only |
| 168 | # in the swarm cluster. In a non-cluster environment the member id |
| 169 | # is formatted differently. In such a case, the method below will |
| 170 | # create an exception and return the member list as is. |
| 171 | |
| 172 | try: |
| 173 | unique_members = {} |
| 174 | update_occurred = False |
| 175 | log.info('members', members=members) |
| 176 | for member in members: |
| 177 | log.info('member', member=member) |
| 178 | # Extract the swarm assigned number of the voltha instance |
| 179 | voltha_number = self.core_match(member['id']).group(1) |
| 180 | timestamp = self.timestamp_match(member['id']).group(1) |
| 181 | if voltha_number not in unique_members: |
| 182 | unique_members[voltha_number] = {'id': member['id'], |
| 183 | 'timestamp': timestamp, |
| 184 | 'host': member['host']} |
| 185 | else: |
| 186 | # Verify whether if this member has the latest timestamp. If |
| 187 | # yes, overwrite the previous one |
| 188 | if unique_members[voltha_number]['timestamp'] < timestamp: |
| 189 | unique_members[voltha_number] = {'id': member['id'], |
| 190 | 'timestamp': timestamp, |
| 191 | 'host': member['host']} |
| 192 | update_occurred = True |
| 193 | |
| 194 | if update_occurred: |
| 195 | updated_members = [] |
| 196 | for _, unique_member in unique_members.iteritems(): |
| 197 | updated_members.append({'host': unique_member['host'], |
| 198 | 'id': unique_member['id']}) |
| 199 | return updated_members |
| 200 | else: |
| 201 | return members |
| 202 | except: |
| 203 | return members |
| 204 | |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 205 | @inlineCallbacks |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 206 | def _track_members(self, index): |
khenaidoo | b1602a3 | 2017-07-27 16:59:52 -0400 | [diff] [blame] | 207 | previous_index = index |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 208 | try: |
khenaidoo | b1602a3 | 2017-07-27 16:59:52 -0400 | [diff] [blame] | 209 | # Put a wait of 5 seconds to wait for a change of membership, |
| 210 | # if any. Without it, if all consul nodes go down then we will |
| 211 | # never get out of this watch. |
Zsolt Haraszti | ac9310d | 2016-09-20 12:56:35 -0700 | [diff] [blame] | 212 | (index, results) = yield self.coord.kv_get( |
khenaidoo | b1602a3 | 2017-07-27 16:59:52 -0400 | [diff] [blame] | 213 | self.coord.membership_prefix, wait='5s', index=index, recurse=True) |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 214 | |
khenaidoo | b1602a3 | 2017-07-27 16:59:52 -0400 | [diff] [blame] | 215 | # This can happen if consul went down and came back with no data |
| 216 | if not results: |
| 217 | log.error('no-active-members') |
| 218 | # Bail out of leadership and go for an early election |
| 219 | self.coord._just_lost_leadership() |
| 220 | return |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 221 | |
khenaidoo | b1602a3 | 2017-07-27 16:59:52 -0400 | [diff] [blame] | 222 | if previous_index != index: |
| 223 | log.info('membership-updated', |
| 224 | previous_index=previous_index, index=index) |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 225 | |
khenaidoo | b1602a3 | 2017-07-27 16:59:52 -0400 | [diff] [blame] | 226 | # Rebuild the membership, if any |
| 227 | |
| 228 | # Only members with valid session are considered active |
| 229 | members = [{'id': self.member_id_match(e['Key']).group(2), |
| 230 | 'host': loads(e['Value'])['host_address']} |
| 231 | for e in results if 'Session' in e] |
| 232 | |
khenaidoo | 82ce00d | 2017-08-15 12:01:46 -0400 | [diff] [blame^] | 233 | if members: |
| 234 | updated_members = self._sanitize_member_list(members) |
| 235 | else: |
| 236 | updated_members = None |
| 237 | |
| 238 | log.info('active-members', active_members=members, |
| 239 | sanitized_members=updated_members) |
khenaidoo | b1602a3 | 2017-07-27 16:59:52 -0400 | [diff] [blame] | 240 | |
| 241 | # Check if the two sets are the same |
khenaidoo | 82ce00d | 2017-08-15 12:01:46 -0400 | [diff] [blame^] | 242 | if updated_members != self.members: |
khenaidoo | b1602a3 | 2017-07-27 16:59:52 -0400 | [diff] [blame] | 243 | # update the current set of config |
| 244 | yield self._update_core_store_references() |
| 245 | log.info('membership-changed', |
| 246 | prev_members=self.members, |
khenaidoo | 82ce00d | 2017-08-15 12:01:46 -0400 | [diff] [blame^] | 247 | curr_members=updated_members, |
khenaidoo | b1602a3 | 2017-07-27 16:59:52 -0400 | [diff] [blame] | 248 | core_store_mapping=self.core_store_assignment) |
khenaidoo | 82ce00d | 2017-08-15 12:01:46 -0400 | [diff] [blame^] | 249 | self.members = updated_members |
khenaidoo | b1602a3 | 2017-07-27 16:59:52 -0400 | [diff] [blame] | 250 | self._restart_core_store_reassignment_soak_timer() |
| 251 | else: |
| 252 | log.debug('no-membership-change', index=index) |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 253 | |
| 254 | except Exception, e: |
Zsolt Haraszti | 2bdb6b3 | 2016-11-03 16:56:17 -0700 | [diff] [blame] | 255 | log.exception('members-track-error', e=e) |
Rouzbahan Rashidi-Tabrizi | 1c3eba8 | 2016-10-27 21:47:18 -0400 | [diff] [blame] | 256 | yield asleep( |
| 257 | self.coord.leader_config.get( |
| 258 | self.coord.leader_config[ |
| 259 | 'members_track_error_to_prevent_flood']), 1) |
| 260 | # to prevent flood |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 261 | finally: |
| 262 | if not self.halted: |
| 263 | reactor.callLater(0, self._track_members, index) |
| 264 | |
| 265 | def _restart_reassignment_soak_timer(self): |
| 266 | |
| 267 | if self.reassignment_soak_timer is not None: |
| 268 | assert isinstance(self.reassignment_soak_timer, DelayedCall) |
Zsolt Haraszti | 8dc1f5e | 2016-09-18 23:35:39 -0700 | [diff] [blame] | 269 | if not self.reassignment_soak_timer.called: |
| 270 | self.reassignment_soak_timer.cancel() |
Zsolt Haraszti | a341031 | 2016-09-18 23:29:04 -0700 | [diff] [blame] | 271 | |
| 272 | self.reassignment_soak_timer = reactor.callLater( |
| 273 | self.soak_time, self._reassign_work) |
| 274 | |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 275 | def _restart_core_store_reassignment_soak_timer(self): |
| 276 | |
khenaidoo | b1602a3 | 2017-07-27 16:59:52 -0400 | [diff] [blame] | 277 | if self.core_store_reassignment_soak_timer is not None: |
| 278 | assert isinstance(self.core_store_reassignment_soak_timer, DelayedCall) |
| 279 | if not self.core_store_reassignment_soak_timer.called: |
| 280 | self.core_store_reassignment_soak_timer.cancel() |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 281 | |
khenaidoo | b1602a3 | 2017-07-27 16:59:52 -0400 | [diff] [blame] | 282 | self.core_store_reassignment_soak_timer = reactor.callLater( |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 283 | self.soak_time, self._reassign_core_stores) |
| 284 | |
| 285 | @inlineCallbacks |
| 286 | def _reassign_core_stores(self): |
| 287 | |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 288 | def _get_core_data_id_from_instance(instance_name): |
| 289 | for id, instance in self.core_store_assignment.iteritems(): |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 290 | if instance and instance['id'] == instance_name: |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 291 | return id |
| 292 | |
| 293 | try: |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 294 | log.info('core-members', curr_members=self.members, |
| 295 | prev_members=self.core_store_assignment) |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 296 | |
| 297 | # 1. clear the mapping for instances that are no longer running |
| 298 | updated_mapping = dict() |
| 299 | existing_active_config_members = set() |
| 300 | cleared_config_ids = set() |
| 301 | inactive_members = set() |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 302 | if self.core_store_assignment: |
| 303 | for id, instance in self.core_store_assignment.iteritems(): |
| 304 | if instance not in self.members: |
| 305 | updated_mapping[id] = None |
| 306 | cleared_config_ids.add(id) |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 307 | if instance: |
| 308 | inactive_members.add(instance['id']) |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 309 | else: |
| 310 | updated_mapping[id] = instance |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 311 | existing_active_config_members.add(instance['id']) |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 312 | |
| 313 | # 2. Update the mapping with the new set |
| 314 | current_id = max(self.core_store_assignment) \ |
khenaidoo | 08d48d2 | 2017-06-29 19:42:49 -0400 | [diff] [blame] | 315 | if self.core_store_assignment else '0000' |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 316 | for instance in self.members: |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 317 | if instance['id'] not in existing_active_config_members: |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 318 | # Add the member to the config map |
| 319 | if cleared_config_ids: |
| 320 | # There is an empty slot |
| 321 | next_id = cleared_config_ids.pop() |
| 322 | updated_mapping[next_id] = instance |
| 323 | else: |
| 324 | # There are no empty slot, create new ids |
khenaidoo | 08d48d2 | 2017-06-29 19:42:49 -0400 | [diff] [blame] | 325 | current_id = get_next_core_id(current_id) |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 326 | updated_mapping[current_id] = instance |
| 327 | |
| 328 | self.core_store_assignment = updated_mapping |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 329 | log.info('updated-assignment', |
khenaidoo | 08d48d2 | 2017-06-29 19:42:49 -0400 | [diff] [blame] | 330 | core_store_assignment=self.core_store_assignment, |
| 331 | inactive_members=inactive_members) |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 332 | |
| 333 | # 3. save the mapping into consul |
khenaidoo | 08d48d2 | 2017-06-29 19:42:49 -0400 | [diff] [blame] | 334 | yield self.coord.kv_put(self.coord.core_store_assignment_key, |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 335 | dumps(self.core_store_assignment)) |
| 336 | |
| 337 | # 4. Assign the new workload to the newly created members |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 338 | curr_members_set = set([m['id'] for m in self.members]) |
khenaidoo | 08d48d2 | 2017-06-29 19:42:49 -0400 | [diff] [blame] | 339 | new_members = curr_members_set.difference( |
| 340 | existing_active_config_members) |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 341 | for new_member_id in new_members: |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 342 | yield self.coord.kv_put( |
| 343 | self.coord.assignment_prefix |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 344 | + new_member_id + '/' + |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 345 | self.coord.core_storage_suffix, |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 346 | _get_core_data_id_from_instance(new_member_id)) |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 347 | |
| 348 | # 5. Remove non-existent members |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 349 | for member_id in inactive_members: |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 350 | yield self.coord.kv_delete( |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 351 | self.coord.assignment_prefix + member_id, recurse=True) |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 352 | yield self.coord.kv_delete( |
khenaidoo | a8588f2 | 2017-06-16 12:13:34 -0400 | [diff] [blame] | 353 | self.coord.membership_prefix + member_id, |
khenaidoo | 032d330 | 2017-06-09 14:50:04 -0400 | [diff] [blame] | 354 | recurse=True) |
| 355 | |
| 356 | except Exception as e: |
| 357 | log.exception('config-reassignment-failure', e=e) |
| 358 | self._restart_core_store_reassignment_soak_timer() |