blob: a353ef218e18cfda2b2f4c560122108e06f1a197 [file] [log] [blame]
Richard Jankowski215a3e22018-10-04 13:56:11 -04001/*
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 Jankowskie4d77662018-10-17 13:53:21 -040020 * 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 Jankowski215a3e22018-10-04 13:56:11 -040024 * 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 Jankowski215a3e22018-10-04 13:56:11 -040028 */
npujar1d86a522019-11-14 17:11:16 +053029
Richard Jankowski215a3e22018-10-04 13:56:11 -040030package core
31
32import (
npujar467fe752020-01-16 20:17:45 +053033 "context"
npujar1d86a522019-11-14 17:11:16 +053034 "time"
35
serkant.uluderya2ae470f2020-01-21 11:13:09 -080036 "github.com/opencord/voltha-lib-go/v3/pkg/db/kvstore"
37 "github.com/opencord/voltha-lib-go/v3/pkg/log"
Kent Hagerman46dcd9d2019-09-18 16:42:59 -040038 "google.golang.org/grpc/codes"
39 "google.golang.org/grpc/status"
Richard Jankowski215a3e22018-10-04 13:56:11 -040040)
41
42// Transaction acquisition results
43const (
khenaidoo89b0e942018-10-21 21:11:33 -040044 UNKNOWN = iota
npujar1d86a522019-11-14 17:11:16 +053045 SeizedBySelf
46 CompletedByOther
47 AbandonedByOther
48 AbandonedWatchBySelf
Richard Jankowski215a3e22018-10-04 13:56:11 -040049)
50
Kent Hagerman46dcd9d2019-09-18 16:42:59 -040051var errorTransactionNotAcquired = status.Error(codes.Canceled, "transaction-not-acquired")
52
npujar1d86a522019-11-14 17:11:16 +053053// Transaction constant
Richard Jankowski215a3e22018-10-04 13:56:11 -040054const (
npujar1d86a522019-11-14 17:11:16 +053055 TransactionComplete = "TRANSACTION-COMPLETE"
Richard Jankowski215a3e22018-10-04 13:56:11 -040056)
57
khenaidoo09771ef2019-10-11 14:25:02 -040058// Transaction constants used to guarantee the Core processing a request hold on to the transaction until
khenaidoof684e1b2019-10-28 19:00:37 -040059// it either completes it (either successfully or times out) or the Core itself crashes (
60// e.g. a server failure).
khenaidoo09771ef2019-10-11 14:25:02 -040061// 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.
68const (
npujar1d86a522019-11-14 17:11:16 +053069 NumTxnRenewalPerRequest = 2
70 MinTxnRenewalIntervalInSec = 3
71 MinTxnReservationDurationInSec = 5
khenaidoo09771ef2019-10-11 14:25:02 -040072)
73
npujar1d86a522019-11-14 17:11:16 +053074// TransactionContext represent transaction context attributes
Richard Jankowski215a3e22018-10-04 13:56:11 -040075type TransactionContext struct {
khenaidoo09771ef2019-10-11 14:25:02 -040076 kvClient kvstore.Client
77 kvOperationTimeout int
78 owner string
79 txnPrefix string
Richard Jankowski215a3e22018-10-04 13:56:11 -040080}
khenaidoo89b0e942018-10-21 21:11:33 -040081
Richard Jankowski215a3e22018-10-04 13:56:11 -040082var ctx *TransactionContext
83
khenaidoo89b0e942018-10-21 21:11:33 -040084var txnState = []string{
85 "UNKNOWN",
86 "SEIZED-BY-SELF",
87 "COMPLETED-BY-OTHER",
88 "ABANDONED-BY-OTHER",
khenaidoo09771ef2019-10-11 14:25:02 -040089 "ABANDONED_WATCH_BY_SELF"}
Richard Jankowski215a3e22018-10-04 13:56:11 -040090
npujar1d86a522019-11-14 17:11:16 +053091// NewTransactionContext creates transaction context instance
Richard Jankowski215a3e22018-10-04 13:56:11 -040092func NewTransactionContext(
khenaidoo89b0e942018-10-21 21:11:33 -040093 owner string,
94 txnPrefix string,
95 kvClient kvstore.Client,
khenaidoo09771ef2019-10-11 14:25:02 -040096 kvOpTimeout int) *TransactionContext {
Richard Jankowski215a3e22018-10-04 13:56:11 -040097
khenaidoo89b0e942018-10-21 21:11:33 -040098 return &TransactionContext{
khenaidoo09771ef2019-10-11 14:25:02 -040099 owner: owner,
100 txnPrefix: txnPrefix,
101 kvClient: kvClient,
102 kvOperationTimeout: kvOpTimeout}
Richard Jankowski215a3e22018-10-04 13:56:11 -0400103}
104
105/*
106 * Before instantiating a KVTransaction, a TransactionContext must be created.
npujar1d86a522019-11-14 17:11:16 +0530107 * The parameters stored in the context govern the behavior of all KVTransaction
Richard Jankowski215a3e22018-10-04 13:56:11 -0400108 * instances.
109 *
110 * :param owner: The owner (i.e. voltha core name) of a transaction
111 * :param txnPrefix: The key prefix under which all transaction IDs, or serial numbers,
112 * will be created (e.g. "service/voltha/transactions")
113 * :param kvClient: The client API used for all interactions with the KV store. Currently
114 * only the etcd client is supported.
Richard Jankowski199fd862019-03-18 14:49:51 -0400115 * :param: kvOpTimeout: The maximum time, in seconds, to be taken by any KV operation
116 * used by this package
Richard Jankowski215a3e22018-10-04 13:56:11 -0400117 */
npujar1d86a522019-11-14 17:11:16 +0530118
119// SetTransactionContext creates new transaction context
Richard Jankowski215a3e22018-10-04 13:56:11 -0400120func SetTransactionContext(owner string,
khenaidoo89b0e942018-10-21 21:11:33 -0400121 txnPrefix string,
122 kvClient kvstore.Client,
khenaidoo09771ef2019-10-11 14:25:02 -0400123 kvOpTimeout int) error {
Richard Jankowski215a3e22018-10-04 13:56:11 -0400124
khenaidoo09771ef2019-10-11 14:25:02 -0400125 ctx = NewTransactionContext(owner, txnPrefix, kvClient, kvOpTimeout)
khenaidoo89b0e942018-10-21 21:11:33 -0400126 return nil
Richard Jankowski215a3e22018-10-04 13:56:11 -0400127}
128
npujar1d86a522019-11-14 17:11:16 +0530129// KVTransaction represent KV transaction attributes
Richard Jankowskie4d77662018-10-17 13:53:21 -0400130type KVTransaction struct {
khenaidoo09771ef2019-10-11 14:25:02 -0400131 monitorCh chan int
npujar1d86a522019-11-14 17:11:16 +0530132 txnID string
khenaidoo09771ef2019-10-11 14:25:02 -0400133 txnKey string
Richard Jankowski215a3e22018-10-04 13:56:11 -0400134}
135
136/*
137 * A KVTransaction constructor
138 *
139 * :param txnId: The serial number of a voltha request.
Richard Jankowskie4d77662018-10-17 13:53:21 -0400140 * :return: A KVTransaction instance
Richard Jankowski215a3e22018-10-04 13:56:11 -0400141 */
npujar1d86a522019-11-14 17:11:16 +0530142
143// NewKVTransaction creates KV transaction instance
144func NewKVTransaction(txnID string) *KVTransaction {
khenaidoo89b0e942018-10-21 21:11:33 -0400145 return &KVTransaction{
npujar1d86a522019-11-14 17:11:16 +0530146 txnID: txnID,
147 txnKey: ctx.txnPrefix + txnID}
Richard Jankowski215a3e22018-10-04 13:56:11 -0400148}
149
150/*
khenaidoo09771ef2019-10-11 14:25:02 -0400151 * Acquired is invoked by a Core, upon reception of a request, to reserve the transaction key in the KV store. The
152 * request may be resource specific (i.e will include an ID for that resource) or may be generic (i.e. list a set of
153 * resources). If the request is resource specific then this function should be invoked with the ownedByMe flag to
154 * indicate whether this Core owns this resource. In the case where this Core owns this resource or it is a generic
155 * request then we will proceed to reserve the transaction key in the KV store for a minimum time specified by the
156 * minDuration param. If the reservation request fails (i.e. the other Core got the reservation before this one - this
157 * can happen only for generic request) then the Core will start to watch for changes to the key to determine
158 * whether the other Core completed the transaction successfully or the Core just died. If the Core does not own the
159 * resource then we will proceed to watch the transaction key.
Richard Jankowski215a3e22018-10-04 13:56:11 -0400160 *
khenaidoo09771ef2019-10-11 14:25:02 -0400161 * :param minDuration: minimum time to reserve the transaction key in the KV store
162 * :param ownedByMe: specify whether the request is about a resource owned or not. If it's absent then this is a
163 * generic request that has no specific resource ID (e.g. list)
164 *
165 * :return: A boolean specifying whether the resource was acquired. An error is return in case this function is invoked
166 * for a resource that is nonexistent.
Richard Jankowski215a3e22018-10-04 13:56:11 -0400167 */
npujar1d86a522019-11-14 17:11:16 +0530168
169// Acquired aquires transaction status
npujar467fe752020-01-16 20:17:45 +0530170func (c *KVTransaction) Acquired(ctx context.Context, minDuration int64, ownedByMe ...bool) (bool, error) {
khenaidoo89b0e942018-10-21 21:11:33 -0400171 var acquired bool
npujar1d86a522019-11-14 17:11:16 +0530172 var currOwner string
khenaidoo89b0e942018-10-21 21:11:33 -0400173 var res int
Richard Jankowski215a3e22018-10-04 13:56:11 -0400174
khenaidoo89b0e942018-10-21 21:11:33 -0400175 // Convert milliseconds to seconds, rounding up
176 // The reservation TTL is specified in seconds
khenaidoo09771ef2019-10-11 14:25:02 -0400177 durationInSecs := minDuration / 1000
178 if remainder := minDuration % 1000; remainder > 0 {
khenaidoo89b0e942018-10-21 21:11:33 -0400179 durationInSecs++
180 }
npujar1d86a522019-11-14 17:11:16 +0530181 if durationInSecs < int64(MinTxnReservationDurationInSec) {
182 durationInSecs = int64(MinTxnReservationDurationInSec)
khenaidoo09771ef2019-10-11 14:25:02 -0400183 }
184 genericRequest := true
185 resourceOwned := false
186 if len(ownedByMe) > 0 {
187 genericRequest = false
188 resourceOwned = ownedByMe[0]
189 }
190 if resourceOwned || genericRequest {
191 // Keep the reservation longer that the minDuration (which is really the request timeout) to ensure the
192 // transaction key stays in the KV store until after the Core finalize a request timeout condition (which is
193 // a success from a request completion perspective).
npujar467fe752020-01-16 20:17:45 +0530194 if err := c.tryToReserveTxn(ctx, durationInSecs*2); err == nil {
npujar1d86a522019-11-14 17:11:16 +0530195 res = SeizedBySelf
khenaidoo09771ef2019-10-11 14:25:02 -0400196 } else {
Girish Kumarf56a4682020-03-20 20:07:46 +0000197 logger.Debugw("watch-other-server",
npujar1d86a522019-11-14 17:11:16 +0530198 log.Fields{"transactionId": c.txnID, "owner": currOwner, "timeout": durationInSecs})
npujar467fe752020-01-16 20:17:45 +0530199 res = c.Watch(ctx, durationInSecs)
khenaidoo09771ef2019-10-11 14:25:02 -0400200 }
201 } else {
npujar467fe752020-01-16 20:17:45 +0530202 res = c.Watch(ctx, durationInSecs)
khenaidoo09771ef2019-10-11 14:25:02 -0400203 }
204 switch res {
npujar1d86a522019-11-14 17:11:16 +0530205 case SeizedBySelf, AbandonedByOther:
khenaidoo09771ef2019-10-11 14:25:02 -0400206 acquired = true
207 default:
208 acquired = false
209 }
Girish Kumarf56a4682020-03-20 20:07:46 +0000210 logger.Debugw("acquire-transaction-status", log.Fields{"transactionId": c.txnID, "acquired": acquired, "result": txnState[res]})
khenaidoo09771ef2019-10-11 14:25:02 -0400211 return acquired, nil
212}
Richard Jankowski215a3e22018-10-04 13:56:11 -0400213
npujar467fe752020-01-16 20:17:45 +0530214func (c *KVTransaction) tryToReserveTxn(ctxt context.Context, durationInSecs int64) error {
npujar1d86a522019-11-14 17:11:16 +0530215 var currOwner string
khenaidoo09771ef2019-10-11 14:25:02 -0400216 var res int
npujar1d86a522019-11-14 17:11:16 +0530217 var err error
npujar467fe752020-01-16 20:17:45 +0530218 value, _ := ctx.kvClient.Reserve(ctxt, c.txnKey, ctx.owner, durationInSecs)
khenaidoo89b0e942018-10-21 21:11:33 -0400219 if value != nil {
khenaidoo09771ef2019-10-11 14:25:02 -0400220 if currOwner, err = kvstore.ToString(value); err != nil { // This should never happen
Girish Kumarf56a4682020-03-20 20:07:46 +0000221 logger.Errorw("unexpected-owner-type", log.Fields{"transactionId": c.txnID, "error": err})
khenaidoo09771ef2019-10-11 14:25:02 -0400222 return err
223 }
224 if currOwner == ctx.owner {
Girish Kumarf56a4682020-03-20 20:07:46 +0000225 logger.Debugw("acquired-transaction", log.Fields{"transactionId": c.txnID, "result": txnState[res]})
khenaidoo09771ef2019-10-11 14:25:02 -0400226 // Setup the monitoring channel
227 c.monitorCh = make(chan int)
npujar467fe752020-01-16 20:17:45 +0530228 go c.holdOnToTxnUntilProcessingCompleted(ctxt, c.txnKey, ctx.owner, durationInSecs)
khenaidoo09771ef2019-10-11 14:25:02 -0400229 return nil
khenaidoo89b0e942018-10-21 21:11:33 -0400230 }
231 }
khenaidoo09771ef2019-10-11 14:25:02 -0400232 return status.Error(codes.PermissionDenied, "reservation-denied")
Richard Jankowski215a3e22018-10-04 13:56:11 -0400233}
234
npujar1d86a522019-11-14 17:11:16 +0530235// Watch watches transaction
npujar467fe752020-01-16 20:17:45 +0530236func (c *KVTransaction) Watch(ctxt context.Context, durationInSecs int64) int {
Richard Jankowski199fd862019-03-18 14:49:51 -0400237 var res int
Scott Baker0e78ba22020-02-24 17:58:47 -0800238 events := ctx.kvClient.Watch(ctxt, c.txnKey, false)
A R Karthick43ba1fb2019-10-03 16:24:21 +0000239 defer ctx.kvClient.CloseWatch(c.txnKey, events)
Richard Jankowski199fd862019-03-18 14:49:51 -0400240
khenaidoo09771ef2019-10-11 14:25:02 -0400241 transactionWasAcquiredByOther := false
242
243 //Check whether the transaction was already completed by the other Core before we got here.
npujar467fe752020-01-16 20:17:45 +0530244 if kvp, _ := ctx.kvClient.Get(ctxt, c.txnKey); kvp != nil {
khenaidoo09771ef2019-10-11 14:25:02 -0400245 transactionWasAcquiredByOther = true
246 if val, err := kvstore.ToString(kvp.Value); err == nil {
npujar1d86a522019-11-14 17:11:16 +0530247 if val == TransactionComplete {
248 res = CompletedByOther
khenaidoo09771ef2019-10-11 14:25:02 -0400249 // Do an immediate delete of the transaction in the KV Store to free up KV Storage faster
npujar467fe752020-01-16 20:17:45 +0530250 err = c.Delete(ctxt)
npujar1d86a522019-11-14 17:11:16 +0530251 if err != nil {
Girish Kumarf56a4682020-03-20 20:07:46 +0000252 logger.Errorw("unable-to-delete-the-transaction", log.Fields{"error": err})
npujar1d86a522019-11-14 17:11:16 +0530253 }
khenaidoo09771ef2019-10-11 14:25:02 -0400254 return res
255 }
256 } else {
257 // An unexpected value - let's get out of here as something did not go according to plan
npujar1d86a522019-11-14 17:11:16 +0530258 res = AbandonedWatchBySelf
Girish Kumarf56a4682020-03-20 20:07:46 +0000259 logger.Debugw("cannot-read-transaction-value", log.Fields{"txn": c.txnID, "error": err})
khenaidoo09771ef2019-10-11 14:25:02 -0400260 return res
261 }
262 }
263
A R Karthick919f6db2019-08-29 18:14:56 +0000264 for {
265 select {
A R Karthick919f6db2019-08-29 18:14:56 +0000266 case event := <-events:
khenaidoo09771ef2019-10-11 14:25:02 -0400267 transactionWasAcquiredByOther = true
Girish Kumarf56a4682020-03-20 20:07:46 +0000268 logger.Debugw("received-event", log.Fields{"txn": c.txnID, "type": event.EventType})
A R Karthick919f6db2019-08-29 18:14:56 +0000269 if event.EventType == kvstore.DELETE {
270 // The other core failed to process the request
npujar1d86a522019-11-14 17:11:16 +0530271 res = AbandonedByOther
A R Karthick919f6db2019-08-29 18:14:56 +0000272 } else if event.EventType == kvstore.PUT {
273 key, e1 := kvstore.ToString(event.Key)
274 val, e2 := kvstore.ToString(event.Value)
khenaidoo09771ef2019-10-11 14:25:02 -0400275 if e1 == nil && e2 == nil && key == c.txnKey {
npujar1d86a522019-11-14 17:11:16 +0530276 if val == TransactionComplete {
277 res = CompletedByOther
khenaidoo09771ef2019-10-11 14:25:02 -0400278 // Successful request completion has been detected. Remove the transaction key
npujar467fe752020-01-16 20:17:45 +0530279 err := c.Delete(ctxt)
npujar1d86a522019-11-14 17:11:16 +0530280 if err != nil {
Girish Kumarf56a4682020-03-20 20:07:46 +0000281 logger.Errorw("unable-to-delete-the-transaction", log.Fields{"error": err})
npujar1d86a522019-11-14 17:11:16 +0530282 }
A R Karthick919f6db2019-08-29 18:14:56 +0000283 } else {
Girish Kumarf56a4682020-03-20 20:07:46 +0000284 logger.Debugw("Ignoring-PUT-event", log.Fields{"val": val, "key": key})
A R Karthick919f6db2019-08-29 18:14:56 +0000285 continue
286 }
khenaidoo09771ef2019-10-11 14:25:02 -0400287 } else {
Girish Kumarf56a4682020-03-20 20:07:46 +0000288 logger.Warnw("received-unexpected-PUT-event", log.Fields{"txn": c.txnID, "key": key, "ctxKey": c.txnKey})
A R Karthick919f6db2019-08-29 18:14:56 +0000289 }
Richard Jankowski199fd862019-03-18 14:49:51 -0400290 }
khenaidoo09771ef2019-10-11 14:25:02 -0400291 case <-time.After(time.Duration(durationInSecs) * time.Second):
292 // Corner case: In the case where the Core owning the device dies and before this Core takes ownership of
293 // this device there is a window where new requests will end up being watched instead of being processed.
294 // Grab the request if the other Core did not create the transaction in the KV store.
295 // TODO: Use a peer-monitoring probe to switch over (still relies on the probe frequency). This will
296 // guarantee that the peer is actually gone instead of limiting the time the peer can get hold of a
297 // request.
298 if !transactionWasAcquiredByOther {
Girish Kumarf56a4682020-03-20 20:07:46 +0000299 logger.Debugw("timeout-no-peer", log.Fields{"txId": c.txnID})
npujar1d86a522019-11-14 17:11:16 +0530300 res = AbandonedByOther
khenaidoo09771ef2019-10-11 14:25:02 -0400301 } else {
302 continue
303 }
Richard Jankowski199fd862019-03-18 14:49:51 -0400304 }
A R Karthick919f6db2019-08-29 18:14:56 +0000305 break
Richard Jankowski199fd862019-03-18 14:49:51 -0400306 }
307 return res
308}
309
npujar1d86a522019-11-14 17:11:16 +0530310// Close closes transaction
npujar467fe752020-01-16 20:17:45 +0530311func (c *KVTransaction) Close(ctxt context.Context) error {
Girish Kumarf56a4682020-03-20 20:07:46 +0000312 logger.Debugw("close", log.Fields{"txn": c.txnID})
khenaidoo09771ef2019-10-11 14:25:02 -0400313 // Stop monitoring the key (applies only when there has been no transaction switch over)
314 if c.monitorCh != nil {
315 close(c.monitorCh)
npujar467fe752020-01-16 20:17:45 +0530316 err := ctx.kvClient.Put(ctxt, c.txnKey, TransactionComplete)
npujar1d86a522019-11-14 17:11:16 +0530317
318 if err != nil {
Girish Kumarf56a4682020-03-20 20:07:46 +0000319 logger.Errorw("unable-to-write-a-key-value-pair-to-the-KV-store", log.Fields{"error": err})
npujar1d86a522019-11-14 17:11:16 +0530320 }
khenaidoo09771ef2019-10-11 14:25:02 -0400321 }
322 return nil
Richard Jankowski215a3e22018-10-04 13:56:11 -0400323}
324
npujar1d86a522019-11-14 17:11:16 +0530325// Delete deletes transaction
npujar467fe752020-01-16 20:17:45 +0530326func (c *KVTransaction) Delete(ctxt context.Context) error {
Girish Kumarf56a4682020-03-20 20:07:46 +0000327 logger.Debugw("delete", log.Fields{"txn": c.txnID})
npujar467fe752020-01-16 20:17:45 +0530328 return ctx.kvClient.Delete(ctxt, c.txnKey)
khenaidoo09771ef2019-10-11 14:25:02 -0400329}
330
331// holdOnToTxnUntilProcessingCompleted renews the transaction lease until the transaction is complete. durationInSecs
332// is used to calculate the frequency at which the Core processing the transaction renews the lease. This function
333// exits only when the transaction is Closed, i.e completed.
npujar467fe752020-01-16 20:17:45 +0530334func (c *KVTransaction) holdOnToTxnUntilProcessingCompleted(ctxt context.Context, key string, owner string, durationInSecs int64) {
Girish Kumarf56a4682020-03-20 20:07:46 +0000335 logger.Debugw("holdOnToTxnUntilProcessingCompleted", log.Fields{"txn": c.txnID})
npujar1d86a522019-11-14 17:11:16 +0530336 renewInterval := durationInSecs / NumTxnRenewalPerRequest
337 if renewInterval < MinTxnRenewalIntervalInSec {
338 renewInterval = MinTxnRenewalIntervalInSec
khenaidoo09771ef2019-10-11 14:25:02 -0400339 }
340forLoop:
341 for {
342 select {
343 case <-c.monitorCh:
Girish Kumarf56a4682020-03-20 20:07:46 +0000344 logger.Debugw("transaction-renewal-exits", log.Fields{"txn": c.txnID})
khenaidoo09771ef2019-10-11 14:25:02 -0400345 break forLoop
346 case <-time.After(time.Duration(renewInterval) * time.Second):
npujar467fe752020-01-16 20:17:45 +0530347 if err := ctx.kvClient.RenewReservation(ctxt, c.txnKey); err != nil {
khenaidoo09771ef2019-10-11 14:25:02 -0400348 // Log and continue.
Girish Kumarf56a4682020-03-20 20:07:46 +0000349 logger.Warnw("transaction-renewal-failed", log.Fields{"txnId": c.txnKey, "error": err})
khenaidoo09771ef2019-10-11 14:25:02 -0400350 }
351 }
352 }
Richard Jankowski215a3e22018-10-04 13:56:11 -0400353}