Skip to content
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

Support start redis received signal #59

Merged
merged 1 commit into from
Oct 7, 2023

Conversation

drivebyer
Copy link
Collaborator

@drivebyer drivebyer commented Oct 7, 2023

#26

➜  ~ docker ps
CONTAINER ID   IMAGE                           COMMAND                   CREATED         STATUS         PORTS      NAMES
8d83f21e8e1d   quay.io/opstree/redis:v7.0.12   "/usr/bin/entrypoint…"   8 seconds ago   Up 7 seconds   6379/tcp   cranky_clarke
➜  ~ docker kill -s SIGTERM cranky_clarke
cranky_clarke

redis log:

2023-10-07 22:41:46 Redis is running without password which is not recommended
2023-10-07 22:41:46 Setting up redis in standalone mode
2023-10-07 22:41:46 Running without persistence mode
2023-10-07 22:41:46 Running without TLS mode
2023-10-07 22:41:46 ACL_MODE is not true, skipping ACL file modification
2023-10-07 22:41:46 Starting redis service in standalone mode.....
2023-10-07 22:41:46 1:C 07 Oct 2023 14:41:46.166 * oO0OoO0OoO0Oo Redis is starting oO0OoO0OoO0Oo
2023-10-07 22:41:46 1:C 07 Oct 2023 14:41:46.166 * Redis version=7.2.1, bits=64, commit=00000000, modified=0, pid=1, just started
2023-10-07 22:41:46 1:C 07 Oct 2023 14:41:46.166 * Configuration loaded
2023-10-07 22:41:46 1:M 07 Oct 2023 14:41:46.166 * monotonic clock: POSIX clock_gettime
2023-10-07 22:41:46 1:M 07 Oct 2023 14:41:46.167 # Failed to write PID file: Permission denied
2023-10-07 22:41:46 1:M 07 Oct 2023 14:41:46.167 * Running mode=standalone, port=6379.
2023-10-07 22:41:46 1:M 07 Oct 2023 14:41:46.167 * Server initialized
2023-10-07 22:41:46 1:M 07 Oct 2023 14:41:46.167 * Ready to accept connections tcp
2023-10-07 22:42:09 1:signal-handler (1696689729) Received SIGTERM scheduling shutdown...
2023-10-07 22:42:09 1:M 07 Oct 2023 14:42:09.419 * User requested shutdown...
2023-10-07 22:42:09 1:M 07 Oct 2023 14:42:09.419 * Saving the final RDB snapshot before exiting.
2023-10-07 22:42:09 1:M 07 Oct 2023 14:42:09.424 * DB saved on disk
2023-10-07 22:42:09 1:M 07 Oct 2023 14:42:09.424 * Removing the pid file.
2023-10-07 22:42:09 1:M 07 Oct 2023 14:42:09.424 # Redis is now ready to exit, bye bye...

@drivebyer drivebyer added the enhancement New feature or request label Oct 7, 2023
@drivebyer drivebyer requested a review from shubham-cmyk October 7, 2023 14:47
Copy link
Member

@shubham-cmyk shubham-cmyk left a comment

Choose a reason for hiding this comment

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

LGTM

@shubham-cmyk shubham-cmyk merged commit 90aecd3 into OT-CONTAINER-KIT:master Oct 7, 2023
@drivebyer drivebyer deleted the receive-signal branch October 8, 2023 01:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants