Skip to content

Commit b7eafbd

Browse files
authored
fix minor bug in lock handler (#1486)
* fix minor bug in lock handler Signed-off-by: Subhobrata Dey <[email protected]> * dummy commit Signed-off-by: Subhobrata Dey <[email protected]> --------- Signed-off-by: Subhobrata Dey <[email protected]>
1 parent a5bdc23 commit b7eafbd

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

.github/workflows/test-workflow.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ jobs:
5252
cp ./alerting/build/distributions/*.zip alerting-artifacts
5353
# This step uses the codecov-action Github action: https://github.com/codecov/codecov-action
5454
- name: Upload Coverage Report
55-
uses: codecov/codecov-action@v1
55+
uses: codecov/codecov-action@v3
5656
with:
5757
token: ${{ secrets.CODECOV_TOKEN }}
5858
# This step uses the upload-artifact Github action: https://github.com/actions/upload-artifact

alerting/src/main/kotlin/org/opensearch/alerting/MonitorRunnerService.kt

+2-2
Original file line numberDiff line numberDiff line change
@@ -353,7 +353,7 @@ object MonitorRunnerService : JobRunner, CoroutineScope, AbstractLifecycleCompon
353353
}
354354
} finally {
355355
monitorCtx.client!!.suspendUntil<Client, Boolean> { monitorCtx.lockService!!.release(lock, it) }
356-
logger.debug("lock ${lock!!.lockId} released")
356+
logger.debug("lock ${lock?.lockId} released")
357357
}
358358
}
359359
}
@@ -385,7 +385,7 @@ object MonitorRunnerService : JobRunner, CoroutineScope, AbstractLifecycleCompon
385385
}
386386
} finally {
387387
monitorCtx.client!!.suspendUntil<Client, Boolean> { monitorCtx.lockService!!.release(lock, it) }
388-
logger.debug("lock ${lock!!.lockId} released")
388+
logger.debug("lock ${lock?.lockId} released")
389389
}
390390
}
391391
}

0 commit comments

Comments
 (0)