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

tikv cannot restart twice after enable raftstore.periodic-full-compact-start-times #16299

Closed
tonyxuqqi opened this issue Jan 5, 2024 · 0 comments · Fixed by #16300
Closed
Labels
severity/critical type/bug The issue is confirmed as a bug.

Comments

@tonyxuqqi
Copy link
Contributor

Bug Report

What version of TiKV are you using?

master

What operating system and CPU are you using?

Steps to reproduce

1 tiup deploy a cluster
2 tiup cluster edit-config to set raftstore.periodic-full-compact-start-times: ["23:00 +0000", "03:00 +0700"]
3 tiup cluster reload {cluster_name} -R tikv
4 after finish, run tiup cluster reload again.

What did you expect?

Tikv can restart

What did happened?

Tikv cannot restart

Error: init config failed: 192.168.1.232:33735: executor.ssh.execute_failed: Failed to execute command over SSH for 'tidb@192.168.1.232:22' {ssh_stderr: thread 'main' panicked at src/config/mod.rs:4278:17:
invalid auto generated configuration file /data_fast/tikv3/data/last_tikv.toml, err trailing input for key raftstore.periodic-full-compact-start-times at line 201 column 38
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
, ssh_stdout: , ssh_command: export LANG=C; PATH=$PATH:/bin:/sbin:/usr/bin:/usr/sbin /data_fast/tikv3/bin/tikv-server --config-check --config=/data_fast/tikv3/conf/tikv.toml --pd "" --data-dir "/data_fast/tikv3/data"}, cause: Process exited with status 101: check config failed

@tonyxuqqi tonyxuqqi added the type/bug The issue is confirmed as a bug. label Jan 5, 2024
ti-chi-bot bot pushed a commit that referenced this issue Jan 5, 2024
close #16299

Use "%H:%M" to display time for `ReadableOffsetTime`

Signed-off-by: Alex Feinberg <alex@strlen.net>
pingandb pushed a commit to pingandb/tikv that referenced this issue Jan 18, 2024
close tikv#16299

Use "%H:%M" to display time for `ReadableOffsetTime`

Signed-off-by: Alex Feinberg <alex@strlen.net>
Signed-off-by: pingandb <songge102@pingan.com.cn>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
severity/critical type/bug The issue is confirmed as a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant