Skip to content

Commit 8859590

Browse files
authored
[ESM] Remove warning when worker shutdown is graceful (#12307)
1 parent e509f9d commit 8859590

File tree

1 file changed

+7
-1
lines changed
  • localstack-core/localstack/services/lambda_/event_source_mapping

1 file changed

+7
-1
lines changed

localstack-core/localstack/services/lambda_/event_source_mapping/esm_worker.py

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,9 @@ def __init__(
7575
function_version = get_function_version_from_arn(self.esm_config["FunctionArn"])
7676
self._state = lambda_stores[function_version.id.account][function_version.id.region]
7777

78+
# HACK: Flag used to check if a graceful shutdown was triggered.
79+
self._graceful_shutdown_triggered = False
80+
7881
@property
7982
def uuid(self) -> str:
8083
return self.esm_config["UUID"]
@@ -83,6 +86,7 @@ def stop_for_shutdown(self):
8386
# Signal the worker's poller_loop thread to gracefully shutdown
8487
# TODO: Once ESM state is de-coupled from lambda store, re-think this approach.
8588
self._shutdown_event.set()
89+
self._graceful_shutdown_triggered = True
8690

8791
def create(self):
8892
if self.enabled:
@@ -166,7 +170,9 @@ def poller_loop(self, *args, **kwargs):
166170
self.current_state = EsmState.DISABLED
167171
self.state_transition_reason = self.user_state_reason
168172
self.update_esm_state_in_store(EsmState.DISABLED)
169-
else:
173+
elif not self._graceful_shutdown_triggered:
174+
# HACK: If we reach this state and a graceful shutdown was not triggered, log a warning to indicate
175+
# an unexpected state.
170176
LOG.warning(
171177
"Invalid state %s for event source mapping %s.",
172178
self.current_state,

0 commit comments

Comments
 (0)