Feedback on the Issue of Not Verifying During `tiup uninstall xx:vxx` Uninstallation

Translated
1

Note:
This topic has been translated from a Chinese forum by GPT and might contain errors.

Original topic: tiup uninstall xx:vxx uninstall不校验问题反馈

| username: bert

Bug Report
Clearly and accurately describe the issue you have found, and provide any possible steps to reproduce the issue to help the development team address it promptly.
[TiDB Version]
v7.1.0
[Bug Impact]
Uninstall does not verify; uninstalling a non-existent version also reports success. Not sure if this counts as a bug, but it is suggested to improve this.
[Possible Steps to Reproduce]

[root@vlan99 ~]# tiup list --verbose
Available components:
Name            Owner      Installed                                              Platforms                                          Description
----            -----      ---------                                              ---------                                          -----------
PCC             community  1.0.1                                                  darwin/amd64,linux/amd64                           A tool used to capture plan changes among different versions of TiDB
bench           pingcap    v1.12.0                                                darwin/amd64,darwin/arm64,linux/amd64,linux/arm64  Benchmark database with different workloads
br              pingcap    v7.1.0                                                 darwin/amd64,darwin/arm64,linux/amd64,linux/arm64  TiDB/TiKV cluster backup restore tool
cdc             pingcap    v7.1.0                                                 darwin/amd64,darwin/arm64,linux/amd64,linux/arm64  CDC is a change data capture tool for TiDB
chaosd          community  v1.1.1                                                 linux/amd64,linux/arm64                            An easy-to-use Chaos Engineering tool used to inject failures to a physical node
client          pingcap    v1.12.2                                                darwin/amd64,darwin/arm64,linux/amd64,linux/arm64  Client to connect playground
cloud           pingcap                                                           darwin/amd64,darwin/arm64,linux/amd64,linux/arm64  CLI tool to manage TiDB Cloud
cluster         pingcap    v1.10.2,v1.10.3,v1.11.0,v1.11.1,v1.12.2,v1.9.2         linux/arm64,darwin/amd64,darwin/arm64,linux/amd64  Deploy a TiDB cluster for production
ctl             pingcap    v6.5.0,v7.1.0                                          darwin/amd64,linux/amd64,linux/arm64               TiDB controller suite
dm              pingcap    v1.12.2                                                darwin/amd64,darwin/arm64,linux/amd64,linux/arm64  Data Migration Platform manager
dmctl           pingcap    v7.1.0                                                 linux/amd64,linux/arm64                            dmctl component of Data Migration Platform
errdoc          pingcap    v4.0.7                                                 linux/amd64,linux/arm64,darwin/amd64               Document about TiDB errors
pd-recover      pingcap    v7.1.0                                                 darwin/amd64,darwin/arm64,linux/amd64,linux/arm64  PD Recover is a disaster recovery tool of PD, used to recover the PD cluster which cannot start or provide services normally
playground      pingcap    v1.10.2,v1.10.3,v1.11.0,v1.11.1,v1.12.2,v1.9.1,v1.9.2  darwin/amd64,darwin/arm64,linux/amd64,linux/arm64  Bootstrap a local TiDB cluster for fun
tidb            pingcap    v6.5.0,v7.1.0                                          darwin/amd64,darwin/arm64,linux/amd64,linux/arm64  TiDB is an open source distributed HTAP database compatible with the MySQL protocol
tidb-dashboard  pingcap                                                           linux/arm64,darwin/amd64,darwin/arm64,linux/amd64  TiDB Dashboard is a Web UI for monitoring, diagnosing, and managing the TiDB cluster
tidb-lightning  pingcap    v6.1.0,v6.2.0,v6.4.0,v7.1.0                            darwin/amd64,darwin/arm64,linux/amd64,linux/arm64  TiDB Lightning is a tool used for fast full import of large amounts of data into a TiDB cluster
tikv-br         pingcap                                                           linux/amd64                                        TiKV cluster backup restore tool
tikv-cdc        pingcap    v1.1.1                                                 linux/amd64                                        TiKV-CDC is a change data capture tool for TiKV
tiup            pingcap                                                           linux/amd64,linux/arm64,darwin/amd64,darwin/arm64  TiUP is a command-line component management tool that can help to download and install TiDB platform components to the local system
[root@vlan99 ~]# tiup uninstall playground:v1.10.XXX
Uninstalled component `playground:v1.10.XXX` successfully!
[root@vlan99 ~]#
2
| username: bert | Original post link

Just posting for fun :grinning:

3
| username: srstack | Original post link

Sure, can you go to GitHub - pingcap/tiup: A component manager for TiDB and create an issue?

4
| username: redgame | Original post link

Looks like…

5
| username: Billmay表妹 | Original post link

There is indeed this issue, but it does not affect the business. It has already been reported to the relevant R&D team.

They will determine whether it needs to be fixed. For now, you can just ignore it.

6
| username: Billmay表妹 | Original post link

The related issue has been fixed, and the specific progress can be seen at: https://github.com/pingcap/tiup/issues/2208

7
| username: bert | Original post link

The TiDB community is well-managed, and the feedback speed is really fast.

8
| username: 秋季番Oo | Original post link

The non-existent version indicates that it is impossible for a service of this version to be running, which has no impact on business operations. However, it is still a bug, probably with a low priority.

Additionally, the community’s feedback is indeed very quick. Thumbs up for the cousin :heart:

9
| username: srstack | Original post link

fix PR: fix: tiup uninstall check component exists by srstack · Pull Request #2209 · pingcap/tiup · GitHub

10
| username: system | Original post link

This topic was automatically closed 60 days after the last reply. New replies are no longer allowed.