Received-SPF: pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Fri, 29 Apr 2022 09:48:38 +0200
From:   Petr Vorel <pvorel@suse.cz>
To:     ltp@lists.linux.it
Cc:     Steve Dickson <steved@redhat.com>,
        libtirpc-devel@lists.sourceforge.net, linux-nfs@vger.kernel.org,
        Cyril Hrubis <chrubis@suse.cz>,
        automated-testing@lists.yoctoproject.org,
        Li Wang <liwang@redhat.com>, Jan Stancek <jstancek@redhat.com>,
        Yang Xu <xuyang2018.jy@fujitsu.com>,
        Anders Roxell <anders.roxell@linaro.org>,
        Naresh Kamboju <naresh.kamboju@linaro.org>,
        Daniel =?iso-8859-2?Q?D=EDaz?= <daniel.diaz@linaro.org>,
        Chuck Lever <chuck.lever@oracle.com>,
        "J . Bruce Fields" <bfields@fieldses.org>,
        Trond Myklebust <trondmy@hammerspace.com>,
        Veronika Kabatova <vkabatov@redhat.com>,
        Alexey Kodanev <aleksei.kodanev@bell-sw.com>
Subject: Re: [RFC PATCH 0/3] Remove RPC rup and rusers tests
Message-ID: <YmuYVtvPWxYarLEF@pevik>
Reply-To: Petr Vorel <pvorel@suse.cz>
References: <20220428144308.32639-1-pvorel@suse.cz>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20220428144308.32639-1-pvorel@suse.cz>
Precedence: bulk

Hi all,

I'm sorry, I again used the old mail address of automated-testing ML.
Please have look at this patchset:

https://lore.kernel.org/ltp/20220428144308.32639-1-pvorel@suse.cz/
https://patchwork.ozlabs.org/project/ltp/list/?series=297407&state=*

Kind regards,
Petr

> Hi all,

> IMHO safe to remove these two tests, but sending to broad audience in
> case anybody really want to have these 2 kept (they'd be rewritten to
> new LTP shell API).

> BTW in long term I'd prefer to remove all RPC tests
> (testcases/network/rpc/ directory). IMHO they should be part of libtirpc
> (which has no tests), but these tests are old, messy and I'm not sure
> how relevant they are nowadays.

> Kind regards,
> Petr

> Petr Vorel (3):
>   rpc: Remove rup01.sh test
>   rpc: Remove rusers01.sh test
>   rpc: Move rest of RPC tests to runtest/net.rpc_tests

>  runtest/net.rpc                               |  8 ---
>  runtest/net.rpc_tests                         |  3 ++
>  scenario_groups/network                       |  1 -
>  .../network/rpc/basic_tests/rup/Makefile      | 29 -----------
>  .../network/rpc/basic_tests/rup/rup01.sh      | 50 -------------------
>  .../network/rpc/basic_tests/rusers/Makefile   | 29 -----------
>  .../rpc/basic_tests/rusers/rusers01.sh        | 50 -------------------
>  testscripts/network.sh                        |  4 +-
>  8 files changed, 4 insertions(+), 170 deletions(-)
>  delete mode 100644 runtest/net.rpc
>  delete mode 100644 testcases/network/rpc/basic_tests/rup/Makefile
>  delete mode 100755 testcases/network/rpc/basic_tests/rup/rup01.sh
>  delete mode 100644 testcases/network/rpc/basic_tests/rusers/Makefile
>  delete mode 100755 testcases/network/rpc/basic_tests/rusers/rusers01.sh