Skip to content

test: fix flaky test_restart_simple_app_auto_yes#1294

Open
elhimov wants to merge 1 commit into
masterfrom
tntp-7654-flaky-test_restart_simple_app_auto_yes
Open

test: fix flaky test_restart_simple_app_auto_yes#1294
elhimov wants to merge 1 commit into
masterfrom
tntp-7654-flaky-test_restart_simple_app_auto_yes

Conversation

@elhimov
Copy link
Copy Markdown
Contributor

@elhimov elhimov commented May 8, 2026

The patch add proper check that tarantool is started.
Also logging was improved to get more information in case of failure.

I didn't forget about (remove if it is not applicable):

  • Well-written commit messages (see documentation how to write a commit message)

Related issues:

Closes TNTP-7654

@elhimov elhimov added the full-ci Enables full ci tests label May 12, 2026
@elhimov elhimov marked this pull request as ready for review May 14, 2026 09:06
@elhimov elhimov requested review from bigbes and sssciel May 14, 2026 09:07
Copy link
Copy Markdown
Member

@sssciel sssciel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, can you write a commit message with explanation?

@elhimov elhimov force-pushed the tntp-7654-flaky-test_restart_simple_app_auto_yes branch 3 times, most recently from 4ed8389 to e1cf6a4 Compare May 15, 2026 10:12
@elhimov
Copy link
Copy Markdown
Contributor Author

elhimov commented May 15, 2026

Please, can you write a commit message with explanation?

Added brief description.

@elhimov
Copy link
Copy Markdown
Contributor Author

elhimov commented May 15, 2026

I found the possible reason of the problem and fixed it.

The patch add proper check that tarantool is started.
Also logging was improved to get more information in case of failure.

Closes TNTP-7654
@elhimov elhimov force-pushed the tntp-7654-flaky-test_restart_simple_app_auto_yes branch from e1cf6a4 to 257bc1f Compare May 15, 2026 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

full-ci Enables full ci tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants