search

tarantool / tt

Command-line utility to manage Tarantool applications
Other
101 stars 12 forks source link

`tt cluster publish` usage is confusing and dangerous #877

Open psergee opened 4 months ago

psergee commented 4 months ago

It is unclear from cluster publish <app> <new.yaml> command name that it replaces a config.yaml in the application directory. Need more details in the help. This command does not ask for confirmation, and a user may accidentally overwrite existing configuration (expecting ETCD pubishing, for example) This command replaces existing configuration, but only if it is valid app configuation. Or even if the config valid, but unavailable: etcd is stopped, for example. This command should not require valid application to replace its config.

0x501D commented 4 months ago

This command, unexpectedly for me, erased the replicaset config, and for a long time I could not understand what was happening. I assumed that it read etcd parameters from the main config and sent remote.yml according to them.

psergee commented 4 months ago

This command, unexpectedly for me, erased the replicaset config for me, and for a long time I could not understand what was happening. I assumed that it read etcd parameters from the main config and sent remote.yml according to them.

Yeah... It silently and unexpectedly replaced my local configuration too. No log. No confirmation. Nothing.