Skip to content

Commit 8410227

Browse files
committed
remove kafka exception block
1 parent aba8ca1 commit 8410227

File tree

2 files changed

+0
-7
lines changed

2 files changed

+0
-7
lines changed

ros/processor/system_eraser.py

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@
1010
GROUP_ID_SUGGESTIONS_ENGINE,
1111
POLL_TIMEOUT_SECS
1212
)
13-
from confluent_kafka import KafkaException
1413
from ros.lib import consume
1514
from ros.lib.app import app
1615
from ros.extensions import db
@@ -69,10 +68,6 @@ def run(self):
6968
if message is None:
7069
continue
7170

72-
if message.error():
73-
logging.error(f"{self.service} - Consumer error: {message.error()}")
74-
raise KafkaException(message.error())
75-
7671
try:
7772
payload = json.loads(message.value().decode('utf-8'))
7873
event_type = payload.get('type')

tests/test_system_eraser.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,6 @@ def test_run_processes_delete_message(mock_app_context, mock_db_session, mock_co
7777
payload = {"type": "delete", "id": "host-999"}
7878
message = MagicMock()
7979
message.value.return_value = json.dumps(payload).encode("utf-8")
80-
message.error.return_value = None # No error
8180

8281
# Consumer should return the message once then None
8382
mock_consumer.poll.side_effect = [message, None]
@@ -94,7 +93,6 @@ def test_run_ignores_non_delete_message(mock_app_context, mock_db_session, mock_
9493
payload = {"type": "update", "id": "host-777"}
9594
message = MagicMock()
9695
message.value.return_value = json.dumps(payload).encode("utf-8")
97-
message.error.return_value = None # No error
9896

9997
mock_consumer.poll.side_effect = [message, None]
10098

0 commit comments

Comments
 (0)