Skip to content

Commit

Permalink
Log removal error
Browse files Browse the repository at this point in the history
  • Loading branch information
dragomirp committed Nov 29, 2024
1 parent 98fcae3 commit 19bfc88
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
2 changes: 2 additions & 0 deletions src/cluster.py
Original file line number Diff line number Diff line change
Expand Up @@ -789,6 +789,8 @@ def remove_raft_member(self, member_ip: str) -> None:
raise RemoveRaftMemberFailedError() from None

if not result.startswith("SUCCESS"):
logger.error(f"!!!!!!!!!!!!!{result}")
logger.debug("Remove raft member: Remove call not successful")
raise RemoveRaftMemberFailedError()

@retry(stop=stop_after_attempt(10), wait=wait_exponential(multiplier=1, min=2, max=10))
Expand Down
4 changes: 0 additions & 4 deletions tests/integration/ha_tests/test_replication.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,6 @@
# Copyright 2023 Canonical Ltd.
# See LICENSE file for licensing details.

import time

import pytest
from pytest_operator.plugin import OpsTest
from tenacity import Retrying, stop_after_delay, wait_fixed
Expand Down Expand Up @@ -71,8 +69,6 @@ async def test_reelection(ops_test: OpsTest, continuous_writes, primary_start_ti
async with ops_test.fast_forward():
await ops_test.model.wait_for_idle(apps=[app], status="active")

time.sleep(30)

await are_writes_increasing(ops_test, primary_name)

# Verify that a new primary gets elected (ie old primary is secondary).
Expand Down

0 comments on commit 19bfc88

Please sign in to comment.