Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 1 | /* |
| 2 | * Copyright 2018-present Open Networking Foundation |
| 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 | |
| 17 | /* |
| 18 | * Two voltha cores receive the same request; each tries to acquire ownership of the request |
| 19 | * by writing its identifier (e.g. container name or pod name) to the transaction key named |
Richard Jankowski | e4d7766 | 2018-10-17 13:53:21 -0400 | [diff] [blame] | 20 | * after the serial number of the request. The core that loses the race for acquisition |
| 21 | * monitors the progress of the core actually serving the request by watching for changes |
| 22 | * in the value of the transaction key. Once the request is complete, the |
| 23 | * serving core closes the transaction by invoking the KVTransaction's Close method, which |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 24 | * replaces the value of the transaction (i.e. serial number) key with the string |
| 25 | * TRANSACTION_COMPLETE. The standby core observes this update, stops watching the transaction, |
| 26 | * and then deletes the transaction key. |
| 27 | * |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 28 | */ |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 29 | |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 30 | package core |
| 31 | |
| 32 | import ( |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 33 | "context" |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 34 | "time" |
| 35 | |
serkant.uluderya | 2ae470f | 2020-01-21 11:13:09 -0800 | [diff] [blame] | 36 | "github.com/opencord/voltha-lib-go/v3/pkg/db/kvstore" |
| 37 | "github.com/opencord/voltha-lib-go/v3/pkg/log" |
Kent Hagerman | 46dcd9d | 2019-09-18 16:42:59 -0400 | [diff] [blame] | 38 | "google.golang.org/grpc/codes" |
| 39 | "google.golang.org/grpc/status" |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 40 | ) |
| 41 | |
| 42 | // Transaction acquisition results |
| 43 | const ( |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 44 | UNKNOWN = iota |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 45 | SeizedBySelf |
| 46 | CompletedByOther |
| 47 | AbandonedByOther |
| 48 | AbandonedWatchBySelf |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 49 | ) |
| 50 | |
Kent Hagerman | 46dcd9d | 2019-09-18 16:42:59 -0400 | [diff] [blame] | 51 | var errorTransactionNotAcquired = status.Error(codes.Canceled, "transaction-not-acquired") |
| 52 | |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 53 | // Transaction constant |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 54 | const ( |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 55 | TransactionComplete = "TRANSACTION-COMPLETE" |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 56 | ) |
| 57 | |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 58 | // Transaction constants used to guarantee the Core processing a request hold on to the transaction until |
khenaidoo | f684e1b | 2019-10-28 19:00:37 -0400 | [diff] [blame] | 59 | // it either completes it (either successfully or times out) or the Core itself crashes ( |
| 60 | // e.g. a server failure). |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 61 | // If a request has a timeout of x seconds then the Core processing the request will renew the transaction lease |
| 62 | // every x/NUM_TXN_RENEWAL_PER_REQUEST seconds. After the Core completes the request it stops renewing the |
| 63 | // transaction and sets the transaction value to TRANSACTION_COMPLETE. If the processing Core crashes then it |
| 64 | // will not renew the transaction causing the KV store to delete the transaction after its renewal period. The |
| 65 | // Core watching the transaction will then take over. |
| 66 | // Since the MIN_TXN_RENEWAL_INTERVAL_IN_SEC is 3 seconds then for any transaction that completes within 3 seconds |
| 67 | // there won't be a transaction renewal done. |
| 68 | const ( |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 69 | NumTxnRenewalPerRequest = 2 |
| 70 | MinTxnRenewalIntervalInSec = 3 |
| 71 | MinTxnReservationDurationInSec = 5 |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 72 | ) |
| 73 | |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 74 | // TransactionContext represent transaction context attributes |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 75 | type TransactionContext struct { |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 76 | kvClient kvstore.Client |
| 77 | kvOperationTimeout int |
| 78 | owner string |
| 79 | txnPrefix string |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 80 | } |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 81 | |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 82 | var ctx *TransactionContext |
| 83 | |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 84 | var txnState = []string{ |
| 85 | "UNKNOWN", |
| 86 | "SEIZED-BY-SELF", |
| 87 | "COMPLETED-BY-OTHER", |
| 88 | "ABANDONED-BY-OTHER", |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 89 | "ABANDONED_WATCH_BY_SELF"} |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 90 | |
| 91 | func init() { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 92 | _, err := log.AddPackage(log.JSON, log.DebugLevel, nil) |
| 93 | if err != nil { |
| 94 | log.Errorw("unable-to-register-package-to-the-log-map", log.Fields{"error": err}) |
| 95 | } |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 96 | } |
| 97 | |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 98 | // NewTransactionContext creates transaction context instance |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 99 | func NewTransactionContext( |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 100 | owner string, |
| 101 | txnPrefix string, |
| 102 | kvClient kvstore.Client, |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 103 | kvOpTimeout int) *TransactionContext { |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 104 | |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 105 | return &TransactionContext{ |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 106 | owner: owner, |
| 107 | txnPrefix: txnPrefix, |
| 108 | kvClient: kvClient, |
| 109 | kvOperationTimeout: kvOpTimeout} |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 110 | } |
| 111 | |
| 112 | /* |
| 113 | * Before instantiating a KVTransaction, a TransactionContext must be created. |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 114 | * The parameters stored in the context govern the behavior of all KVTransaction |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 115 | * instances. |
| 116 | * |
| 117 | * :param owner: The owner (i.e. voltha core name) of a transaction |
| 118 | * :param txnPrefix: The key prefix under which all transaction IDs, or serial numbers, |
| 119 | * will be created (e.g. "service/voltha/transactions") |
| 120 | * :param kvClient: The client API used for all interactions with the KV store. Currently |
| 121 | * only the etcd client is supported. |
Richard Jankowski | 199fd86 | 2019-03-18 14:49:51 -0400 | [diff] [blame] | 122 | * :param: kvOpTimeout: The maximum time, in seconds, to be taken by any KV operation |
| 123 | * used by this package |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 124 | */ |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 125 | |
| 126 | // SetTransactionContext creates new transaction context |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 127 | func SetTransactionContext(owner string, |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 128 | txnPrefix string, |
| 129 | kvClient kvstore.Client, |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 130 | kvOpTimeout int) error { |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 131 | |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 132 | ctx = NewTransactionContext(owner, txnPrefix, kvClient, kvOpTimeout) |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 133 | return nil |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 134 | } |
| 135 | |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 136 | // KVTransaction represent KV transaction attributes |
Richard Jankowski | e4d7766 | 2018-10-17 13:53:21 -0400 | [diff] [blame] | 137 | type KVTransaction struct { |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 138 | monitorCh chan int |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 139 | txnID string |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 140 | txnKey string |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 141 | } |
| 142 | |
| 143 | /* |
| 144 | * A KVTransaction constructor |
| 145 | * |
| 146 | * :param txnId: The serial number of a voltha request. |
Richard Jankowski | e4d7766 | 2018-10-17 13:53:21 -0400 | [diff] [blame] | 147 | * :return: A KVTransaction instance |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 148 | */ |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 149 | |
| 150 | // NewKVTransaction creates KV transaction instance |
| 151 | func NewKVTransaction(txnID string) *KVTransaction { |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 152 | return &KVTransaction{ |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 153 | txnID: txnID, |
| 154 | txnKey: ctx.txnPrefix + txnID} |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 155 | } |
| 156 | |
| 157 | /* |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 158 | * Acquired is invoked by a Core, upon reception of a request, to reserve the transaction key in the KV store. The |
| 159 | * request may be resource specific (i.e will include an ID for that resource) or may be generic (i.e. list a set of |
| 160 | * resources). If the request is resource specific then this function should be invoked with the ownedByMe flag to |
| 161 | * indicate whether this Core owns this resource. In the case where this Core owns this resource or it is a generic |
| 162 | * request then we will proceed to reserve the transaction key in the KV store for a minimum time specified by the |
| 163 | * minDuration param. If the reservation request fails (i.e. the other Core got the reservation before this one - this |
| 164 | * can happen only for generic request) then the Core will start to watch for changes to the key to determine |
| 165 | * whether the other Core completed the transaction successfully or the Core just died. If the Core does not own the |
| 166 | * resource then we will proceed to watch the transaction key. |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 167 | * |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 168 | * :param minDuration: minimum time to reserve the transaction key in the KV store |
| 169 | * :param ownedByMe: specify whether the request is about a resource owned or not. If it's absent then this is a |
| 170 | * generic request that has no specific resource ID (e.g. list) |
| 171 | * |
| 172 | * :return: A boolean specifying whether the resource was acquired. An error is return in case this function is invoked |
| 173 | * for a resource that is nonexistent. |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 174 | */ |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 175 | |
| 176 | // Acquired aquires transaction status |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 177 | func (c *KVTransaction) Acquired(ctx context.Context, minDuration int64, ownedByMe ...bool) (bool, error) { |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 178 | var acquired bool |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 179 | var currOwner string |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 180 | var res int |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 181 | |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 182 | // Convert milliseconds to seconds, rounding up |
| 183 | // The reservation TTL is specified in seconds |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 184 | durationInSecs := minDuration / 1000 |
| 185 | if remainder := minDuration % 1000; remainder > 0 { |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 186 | durationInSecs++ |
| 187 | } |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 188 | if durationInSecs < int64(MinTxnReservationDurationInSec) { |
| 189 | durationInSecs = int64(MinTxnReservationDurationInSec) |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 190 | } |
| 191 | genericRequest := true |
| 192 | resourceOwned := false |
| 193 | if len(ownedByMe) > 0 { |
| 194 | genericRequest = false |
| 195 | resourceOwned = ownedByMe[0] |
| 196 | } |
| 197 | if resourceOwned || genericRequest { |
| 198 | // Keep the reservation longer that the minDuration (which is really the request timeout) to ensure the |
| 199 | // transaction key stays in the KV store until after the Core finalize a request timeout condition (which is |
| 200 | // a success from a request completion perspective). |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 201 | if err := c.tryToReserveTxn(ctx, durationInSecs*2); err == nil { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 202 | res = SeizedBySelf |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 203 | } else { |
| 204 | log.Debugw("watch-other-server", |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 205 | log.Fields{"transactionId": c.txnID, "owner": currOwner, "timeout": durationInSecs}) |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 206 | res = c.Watch(ctx, durationInSecs) |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 207 | } |
| 208 | } else { |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 209 | res = c.Watch(ctx, durationInSecs) |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 210 | } |
| 211 | switch res { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 212 | case SeizedBySelf, AbandonedByOther: |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 213 | acquired = true |
| 214 | default: |
| 215 | acquired = false |
| 216 | } |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 217 | log.Debugw("acquire-transaction-status", log.Fields{"transactionId": c.txnID, "acquired": acquired, "result": txnState[res]}) |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 218 | return acquired, nil |
| 219 | } |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 220 | |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 221 | func (c *KVTransaction) tryToReserveTxn(ctxt context.Context, durationInSecs int64) error { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 222 | var currOwner string |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 223 | var res int |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 224 | var err error |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 225 | value, _ := ctx.kvClient.Reserve(ctxt, c.txnKey, ctx.owner, durationInSecs) |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 226 | if value != nil { |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 227 | if currOwner, err = kvstore.ToString(value); err != nil { // This should never happen |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 228 | log.Errorw("unexpected-owner-type", log.Fields{"transactionId": c.txnID, "error": err}) |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 229 | return err |
| 230 | } |
| 231 | if currOwner == ctx.owner { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 232 | log.Debugw("acquired-transaction", log.Fields{"transactionId": c.txnID, "result": txnState[res]}) |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 233 | // Setup the monitoring channel |
| 234 | c.monitorCh = make(chan int) |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 235 | go c.holdOnToTxnUntilProcessingCompleted(ctxt, c.txnKey, ctx.owner, durationInSecs) |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 236 | return nil |
khenaidoo | 89b0e94 | 2018-10-21 21:11:33 -0400 | [diff] [blame] | 237 | } |
| 238 | } |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 239 | return status.Error(codes.PermissionDenied, "reservation-denied") |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 240 | } |
| 241 | |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 242 | // Watch watches transaction |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 243 | func (c *KVTransaction) Watch(ctxt context.Context, durationInSecs int64) int { |
Richard Jankowski | 199fd86 | 2019-03-18 14:49:51 -0400 | [diff] [blame] | 244 | var res int |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 245 | events := ctx.kvClient.Watch(ctxt, c.txnKey) |
A R Karthick | 43ba1fb | 2019-10-03 16:24:21 +0000 | [diff] [blame] | 246 | defer ctx.kvClient.CloseWatch(c.txnKey, events) |
Richard Jankowski | 199fd86 | 2019-03-18 14:49:51 -0400 | [diff] [blame] | 247 | |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 248 | transactionWasAcquiredByOther := false |
| 249 | |
| 250 | //Check whether the transaction was already completed by the other Core before we got here. |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 251 | if kvp, _ := ctx.kvClient.Get(ctxt, c.txnKey); kvp != nil { |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 252 | transactionWasAcquiredByOther = true |
| 253 | if val, err := kvstore.ToString(kvp.Value); err == nil { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 254 | if val == TransactionComplete { |
| 255 | res = CompletedByOther |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 256 | // Do an immediate delete of the transaction in the KV Store to free up KV Storage faster |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 257 | err = c.Delete(ctxt) |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 258 | if err != nil { |
| 259 | log.Errorw("unable-to-delete-the-transaction", log.Fields{"error": err}) |
| 260 | } |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 261 | return res |
| 262 | } |
| 263 | } else { |
| 264 | // An unexpected value - let's get out of here as something did not go according to plan |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 265 | res = AbandonedWatchBySelf |
| 266 | log.Debugw("cannot-read-transaction-value", log.Fields{"txn": c.txnID, "error": err}) |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 267 | return res |
| 268 | } |
| 269 | } |
| 270 | |
A R Karthick | 919f6db | 2019-08-29 18:14:56 +0000 | [diff] [blame] | 271 | for { |
| 272 | select { |
A R Karthick | 919f6db | 2019-08-29 18:14:56 +0000 | [diff] [blame] | 273 | case event := <-events: |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 274 | transactionWasAcquiredByOther = true |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 275 | log.Debugw("received-event", log.Fields{"txn": c.txnID, "type": event.EventType}) |
A R Karthick | 919f6db | 2019-08-29 18:14:56 +0000 | [diff] [blame] | 276 | if event.EventType == kvstore.DELETE { |
| 277 | // The other core failed to process the request |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 278 | res = AbandonedByOther |
A R Karthick | 919f6db | 2019-08-29 18:14:56 +0000 | [diff] [blame] | 279 | } else if event.EventType == kvstore.PUT { |
| 280 | key, e1 := kvstore.ToString(event.Key) |
| 281 | val, e2 := kvstore.ToString(event.Value) |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 282 | if e1 == nil && e2 == nil && key == c.txnKey { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 283 | if val == TransactionComplete { |
| 284 | res = CompletedByOther |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 285 | // Successful request completion has been detected. Remove the transaction key |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 286 | err := c.Delete(ctxt) |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 287 | if err != nil { |
| 288 | log.Errorw("unable-to-delete-the-transaction", log.Fields{"error": err}) |
| 289 | } |
A R Karthick | 919f6db | 2019-08-29 18:14:56 +0000 | [diff] [blame] | 290 | } else { |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 291 | log.Debugw("Ignoring-PUT-event", log.Fields{"val": val, "key": key}) |
A R Karthick | 919f6db | 2019-08-29 18:14:56 +0000 | [diff] [blame] | 292 | continue |
| 293 | } |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 294 | } else { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 295 | log.Warnw("received-unexpected-PUT-event", log.Fields{"txn": c.txnID, "key": key, "ctxKey": c.txnKey}) |
A R Karthick | 919f6db | 2019-08-29 18:14:56 +0000 | [diff] [blame] | 296 | } |
Richard Jankowski | 199fd86 | 2019-03-18 14:49:51 -0400 | [diff] [blame] | 297 | } |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 298 | case <-time.After(time.Duration(durationInSecs) * time.Second): |
| 299 | // Corner case: In the case where the Core owning the device dies and before this Core takes ownership of |
| 300 | // this device there is a window where new requests will end up being watched instead of being processed. |
| 301 | // Grab the request if the other Core did not create the transaction in the KV store. |
| 302 | // TODO: Use a peer-monitoring probe to switch over (still relies on the probe frequency). This will |
| 303 | // guarantee that the peer is actually gone instead of limiting the time the peer can get hold of a |
| 304 | // request. |
| 305 | if !transactionWasAcquiredByOther { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 306 | log.Debugw("timeout-no-peer", log.Fields{"txId": c.txnID}) |
| 307 | res = AbandonedByOther |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 308 | } else { |
| 309 | continue |
| 310 | } |
Richard Jankowski | 199fd86 | 2019-03-18 14:49:51 -0400 | [diff] [blame] | 311 | } |
A R Karthick | 919f6db | 2019-08-29 18:14:56 +0000 | [diff] [blame] | 312 | break |
Richard Jankowski | 199fd86 | 2019-03-18 14:49:51 -0400 | [diff] [blame] | 313 | } |
| 314 | return res |
| 315 | } |
| 316 | |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 317 | // Close closes transaction |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 318 | func (c *KVTransaction) Close(ctxt context.Context) error { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 319 | log.Debugw("close", log.Fields{"txn": c.txnID}) |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 320 | // Stop monitoring the key (applies only when there has been no transaction switch over) |
| 321 | if c.monitorCh != nil { |
| 322 | close(c.monitorCh) |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 323 | err := ctx.kvClient.Put(ctxt, c.txnKey, TransactionComplete) |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 324 | |
| 325 | if err != nil { |
| 326 | log.Errorw("unable-to-write-a-key-value-pair-to-the-KV-store", log.Fields{"error": err}) |
| 327 | } |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 328 | } |
| 329 | return nil |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 330 | } |
| 331 | |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 332 | // Delete deletes transaction |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 333 | func (c *KVTransaction) Delete(ctxt context.Context) error { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 334 | log.Debugw("delete", log.Fields{"txn": c.txnID}) |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 335 | return ctx.kvClient.Delete(ctxt, c.txnKey) |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 336 | } |
| 337 | |
| 338 | // holdOnToTxnUntilProcessingCompleted renews the transaction lease until the transaction is complete. durationInSecs |
| 339 | // is used to calculate the frequency at which the Core processing the transaction renews the lease. This function |
| 340 | // exits only when the transaction is Closed, i.e completed. |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 341 | func (c *KVTransaction) holdOnToTxnUntilProcessingCompleted(ctxt context.Context, key string, owner string, durationInSecs int64) { |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 342 | log.Debugw("holdOnToTxnUntilProcessingCompleted", log.Fields{"txn": c.txnID}) |
| 343 | renewInterval := durationInSecs / NumTxnRenewalPerRequest |
| 344 | if renewInterval < MinTxnRenewalIntervalInSec { |
| 345 | renewInterval = MinTxnRenewalIntervalInSec |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 346 | } |
| 347 | forLoop: |
| 348 | for { |
| 349 | select { |
| 350 | case <-c.monitorCh: |
npujar | 1d86a52 | 2019-11-14 17:11:16 +0530 | [diff] [blame] | 351 | log.Debugw("transaction-renewal-exits", log.Fields{"txn": c.txnID}) |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 352 | break forLoop |
| 353 | case <-time.After(time.Duration(renewInterval) * time.Second): |
npujar | 467fe75 | 2020-01-16 20:17:45 +0530 | [diff] [blame] | 354 | if err := ctx.kvClient.RenewReservation(ctxt, c.txnKey); err != nil { |
khenaidoo | 09771ef | 2019-10-11 14:25:02 -0400 | [diff] [blame] | 355 | // Log and continue. |
| 356 | log.Warnw("transaction-renewal-failed", log.Fields{"txnId": c.txnKey, "error": err}) |
| 357 | } |
| 358 | } |
| 359 | } |
Richard Jankowski | 215a3e2 | 2018-10-04 13:56:11 -0400 | [diff] [blame] | 360 | } |