From 41082930f33fa43391986ac05394dbfff3b5dfd7 Mon Sep 17 00:00:00 2001
From: Kotresh HR <khiremat@redhat.com>
Date: Sun, 19 Apr 2020 00:03:37 +0530
Subject: [PATCH] bash_completion: Do not auto complete obsolete and hidden
 cmds

This patch fixes two things.

1. Do not auto complete obsolete and hidden cmds.
2. sub command completions often failed due to the
   use of associative arrays which does not keep the
   order. Hence used non-associative arrays.

Fixes: https://tracker.ceph.com/issues/45141
Signed-off-by: Kotresh HR <khiremat@redhat.com>
---
 src/bash_completion/ceph | 2 +-
 src/ceph.in              | 3 +++
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/bash_completion/ceph b/src/bash_completion/ceph
index beec700e854..d28c8a4fa76 100644
--- a/src/bash_completion/ceph
+++ b/src/bash_completion/ceph
@@ -27,7 +27,7 @@ _ceph()
 	COMPREPLY=( $(compgen -W "${options_noarg} ${options_arg}" -- $cur) )
 	return 0
     fi
-    declare -A hint_args
+    declare -a hint_args
     for (( i=1 ; i<cnt ; i++ ))
     do
 	#skip this word if it is option
diff --git a/src/ceph.in b/src/ceph.in
index 369b9696540..13929a4cca8 100755
--- a/src/ceph.in
+++ b/src/ceph.in
@@ -679,6 +679,9 @@ def complete(sigdict, args, target):
     match_count = 0
     comps = []
     for cmdtag, cmd in sigdict.items():
+        flags = cmd.get('flags', 0)
+        if flags & (Flag.OBSOLETE | Flag.HIDDEN):
+            continue
         sig = cmd['sig']
         j = 0
         # iterate over all arguments, except last one
-- 
2.39.5