Add consumer seek failure log

[shubhi1092]

Description

• Added an error log when consumer.seek() fails.
• Added a check to ensure that the circuit breaker metric is not completed before completing it.

[Copilot]

Pull Request Overview

This PR enhances error handling and metric safety by adding detailed logging for failed Kafka seek operations and guarding against double-completing a circuit breaker metric.

• Add a Lumberjack error log when consumer.seek() fails
• Check isCompleted() before completing the circuit breaker metric

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
services-ordering-rdkafka/src/rdkafkaConsumer.ts	Log detailed context on consumer.seek() failures
lambdas/src/utils/circuitBreaker.ts	Prevent completing an already completed circuit breaker metric

Comments suppressed due to low confidence (2)

server/routerlicious/packages/services-ordering-rdkafka/src/rdkafkaConsumer.ts:585

• Add a unit or integration test that simulates a consumer.seek() error and verifies that Lumberjack.error is called with the expected properties.

Lumberjack.error(

server/routerlicious/packages/lambdas/src/utils/circuitBreaker.ts:112

• Add tests covering the scenario where circuitBreakerMetric is already completed to ensure the else branch doesn’t call success again and behaves as expected.

if (this.circuitBreakerMetric && !this.circuitBreakerMetric.isCompleted()) {