Skip to content

TCM: Add tt tcm status/stop #1152

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
May 6, 2025
Merged

TCM: Add tt tcm status/stop #1152

merged 2 commits into from
May 6, 2025

Conversation

AlexandrLitkevich
Copy link
Contributor

@AlexandrLitkevich AlexandrLitkevich commented Apr 15, 2025

What has been done? Why? What problem is being solved?

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

  • Well-written commit messages (see documentation how to write a commit message)
  • Don't forget about TarantoolBot in a commit message (see example)
  • Tests (see documentation for a testing package)
  • Changelog (see documentation for changelog format)
  • Documentation (see documentation for documentation style guide)

Related issues:

Closes #TNTP-1992
Closes #TNTP-1993

@AlexandrLitkevich AlexandrLitkevich force-pushed the TNTP-1992 branch 4 times, most recently from 68dc463 to 8f4ff5f Compare April 15, 2025 12:26
@AlexandrLitkevich AlexandrLitkevich added the full-ci Enables full ci tests label Apr 15, 2025
@AlexandrLitkevich AlexandrLitkevich changed the title Tntp 1992 TCM: Add tt tcm status/stop Apr 15, 2025
@AlexandrLitkevich AlexandrLitkevich force-pushed the TNTP-1992 branch 4 times, most recently from 61141f5 to 3923916 Compare April 16, 2025 05:41
@AlexandrLitkevich AlexandrLitkevich force-pushed the TNTP-1992 branch 2 times, most recently from 6a1d18d to 79fc7fb Compare April 16, 2025 11:30
oleg-jukovec

This comment was marked as duplicate.

Copy link
Contributor

@oleg-jukovec oleg-jukovec left a comment

Choose a reason for hiding this comment

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

Tests are red.

@AlexandrLitkevich AlexandrLitkevich force-pushed the TNTP-1992 branch 2 times, most recently from 9f9fbfe to 72c6d0e Compare May 5, 2025 11:59
@TarantoolBot document
Title: add the tt tcm status command.

Add `tt tcm status` command that displays whether TCM is running in:
- watchdog mode (with auto-recovery)
- interactive mode

Closes #TNTP-1992
@TarantoolBot document
Title: add the tt tcm stop command.

Add command tt tcm stop to gracefully terminate TCM:
- Works in both modes(watchdog and interactive)
- Stops all subprocesses

Closes #TNTP-1993
@dmyger dmyger merged commit 8c81e5e into master May 6, 2025
24 checks passed
@dmyger dmyger deleted the TNTP-1992 branch May 6, 2025 12: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.

4 participants