nizhikov commented on a change in pull request #8891:
URL: https://github.com/apache/ignite/pull/8891#discussion_r596577716
##########
File path: modules/ducktests/tests/ignitetest/tests/rebalance/__init__.py
##########
@@ -61,14 +62,11 @@ def await_rebalance_start(ignite, timeout=1):
"""
for node in ignite.nodes:
try:
- rebalance_start_time = get_event_time(
- ignite, node,
- "Starting rebalance routine \\[test-cache-",
- timeout=timeout)
- except TimeoutError:
- continue
+ rebalance_start_time = get_event_time(node, "Starting rebalance
routine \\[test-cache-", timeout=timeout)
Review comment:
Why this change?
##########
File path: modules/ducktests/tests/ignitetest/tests/discovery_test.py
##########
@@ -243,7 +244,7 @@ def _simulate_and_detect_failure(self, servers,
failed_nodes, event_timeout_sec,
for survivor in [n for n in servers.nodes if n not in failed_nodes]:
for failed_id in ids_to_wait:
- logged_timestamps.append(get_event_time(servers, survivor,
node_failed_event_pattern(failed_id),
+ logged_timestamps.append(get_event_time(survivor,
node_failed_event_pattern(failed_id),
Review comment:
Why this change?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
