[LTP] [PATCH 1/1] runltp: Remove RHOST setup

* [LTP] [PATCH 1/1] runltp: Remove RHOST setup
@ 2018-09-03 10:50 Petr Vorel
  2018-09-03 11:30 ` Cyril Hrubis
  2018-09-10 22:42 ` Petr Vorel
  0 siblings, 2 replies; 5+ messages in thread
From: Petr Vorel @ 2018-09-03 10:50 UTC (permalink / raw)
  To: ltp

as RSH is deprecated. The default management link of network tests are
network namespaces. For remote management is better to setup SSH via:
TST_USE_SSH=1

Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
To be honest, I'd prefer removing -N option from runltp or replace it
with info "Run network tests via network.sh script" (+ remove
scenario_groups/network).

I'm looking forward the day this crazy script is replaced with something
more convenient.


Kind regards,
Petr
---
 runltp | 24 ------------------------
 1 file changed, 24 deletions(-)

diff --git a/runltp b/runltp
index bbbb58771..5e2f5668d 100755
--- a/runltp
+++ b/runltp
@@ -570,30 +570,6 @@ main()
       INSTANCES="$INSTANCES -O ${TMP}"
     }
 
-    [ "$RUN_NETEST" -eq 1 ] && \
-    {
-        [ -z "$RHOST" ] || [ -z "$PASSWD" ] && \
-        {
-            [ -z "$RHOST" ] && \
-            {
-                printf "INFO: Enter RHOST = 'name of the remote host machine'"
-                printf "\n-> "
-                read RHOST
-            }
-
-            [ -z "$PASSWD" ] && \
-            {
-                printf "\nINFO: "
-                printf "Enter PASSWD = 'root passwd of the remote host machine'"
-                printf "\n-> "
-                read PASSWD
-            }
-            export RHOST=$RHOST
-            export PASSWD=$PASSWD
-            echo "WARNING: security of $RHOST may be compromised"
-        }
-    }
-
     # If user does not provide a command file select a default set of testcases
     # to execute.
     if [ -z "$CMDFILES" ] && [ -z "$CMDFILEADDR" ]; then
-- 
2.18.0


^ permalink raw reply related	[flat|nested] 5+ messages in thread