From f8cec23a9bb1f2dd68c761f1598c4bfff99c632d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Viktor=20S=C3=B6derqvist?= <viktor.soderqvist@est.tech>
Date: Wed, 10 Apr 2024 18:14:58 +0200
Subject: [PATCH] Delete old deprecated cli program redis-trib (#281)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Actually, this script doesn't do anything except printing that it is
replaced by redis-cli.

Signed-off-by: Viktor Söderqvist <viktor.soderqvist@est.tech>
---
 src/redis-trib.rb                        | 129 -----------------------
 tests/cluster/tests/helpers/onlydots.tcl |   4 +-
 2 files changed, 2 insertions(+), 131 deletions(-)
 delete mode 100755 src/redis-trib.rb

diff --git a/src/redis-trib.rb b/src/redis-trib.rb
deleted file mode 100755
index b1af83069..000000000
--- a/src/redis-trib.rb
+++ /dev/null
@@ -1,129 +0,0 @@
-#!/usr/bin/env ruby
-
-def colorized(str, color)
-    return str if !(ENV['TERM'] || '')["xterm"]
-    color_code = {
-        white: 29,
-        bold: '29;1',
-        black: 30,
-        red: 31,
-        green: 32,
-        yellow: 33,
-        blue: 34,
-        magenta: 35,
-        cyan: 36,
-        gray: 37
-    }[color]
-    return str if !color_code
-    "\033[#{color_code}m#{str}\033[0m"
-end
-
-class String
-
-    %w(white bold black red green yellow blue magenta cyan gray).each{|color|
-        color = :"#{color}"
-        define_method(color){
-            colorized(self, color)
-        }
-    }
-
-end
-
-COMMANDS = %w(create check info fix reshard rebalance add-node 
-              del-node set-timeout call import help)
-
-ALLOWED_OPTIONS={
-    "create" => {"replicas" => true},
-    "add-node" => {"slave" => false, "master-id" => true},
-    "import" => {"from" => :required, "copy" => false, "replace" => false},
-    "reshard" => {"from" => true, "to" => true, "slots" => true, "yes" => false, "timeout" => true, "pipeline" => true},
-    "rebalance" => {"weight" => [], "auto-weights" => false, "use-empty-masters" => false, "timeout" => true, "simulate" => false, "pipeline" => true, "threshold" => true},
-    "fix" => {"timeout" => 0},
-}
-
-def parse_options(cmd)
-    cmd = cmd.downcase
-    idx = 0
-    options = {}
-    args = []
-    while (arg = ARGV.shift)
-        if arg[0..1] == "--"
-            option = arg[2..-1]
-
-            # --verbose is a global option
-            if option == "--verbose"
-                options['verbose'] = true
-                next
-            end
-            if ALLOWED_OPTIONS[cmd] == nil || 
-               ALLOWED_OPTIONS[cmd][option] == nil
-                next
-            end
-            if ALLOWED_OPTIONS[cmd][option] != false
-                value = ARGV.shift
-                next if !value
-            else
-                value = true
-            end
-
-            # If the option is set to [], it's a multiple arguments
-            # option. We just queue every new value into an array.
-            if ALLOWED_OPTIONS[cmd][option] == []
-                options[option] = [] if !options[option]
-                options[option] << value
-            else
-                options[option] = value
-            end
-        else
-            next if arg[0,1] == '-'
-            args << arg
-        end
-    end
-
-    return options,args
-end
-
-def command_example(cmd, args, opts)
-    cmd = "redis-cli --cluster #{cmd}"
-    args.each{|a| 
-        a = a.to_s
-        a = a.inspect if a[' ']
-        cmd << " #{a}"
-    }
-    opts.each{|opt, val|
-        opt = " --cluster-#{opt.downcase}"
-        if val != true
-            val = val.join(' ') if val.is_a? Array
-            opt << " #{val}"
-        end
-        cmd << opt
-    }
-    cmd
-end
-
-$command = ARGV.shift
-$opts, $args = parse_options($command) if $command
-
-puts "WARNING: redis-trib.rb is not longer available!".yellow
-puts "You should use #{'redis-cli'.bold} instead."
-puts ''
-puts "All commands and features belonging to redis-trib.rb "+
-     "have been moved\nto redis-cli."
-puts "In order to use them you should call redis-cli with the #{'--cluster'.bold}"
-puts "option followed by the subcommand name, arguments and options."
-puts ''
-puts "Use the following syntax:"
-puts "redis-cli --cluster SUBCOMMAND [ARGUMENTS] [OPTIONS]".bold
-puts ''
-puts "Example:"
-if $command
-    example = command_example $command, $args, $opts
-else
-    example = "redis-cli --cluster info 127.0.0.1:7000"
-end
-puts example.bold
-puts ''
-puts "To get help about all subcommands, type:"
-puts "redis-cli --cluster help".bold
-puts ''
-exit 1
diff --git a/tests/cluster/tests/helpers/onlydots.tcl b/tests/cluster/tests/helpers/onlydots.tcl
index 4a6d1aee0..0d69ef1b4 100644
--- a/tests/cluster/tests/helpers/onlydots.tcl
+++ b/tests/cluster/tests/helpers/onlydots.tcl
@@ -1,7 +1,7 @@
 # Read the standard input and only shows dots in the output, filtering out
 # all the other characters. Designed to avoid bufferization so that when
-# we get the output of redis-trib and want to show just the dots, we'll see
-# the dots as soon as redis-trib will output them.
+# we get the output of a program and want to show just the dots, we'll see
+# the dots as soon as the program outputs them.
 
 fconfigure stdin -buffering none