[v2] test: Have test_fs work with non-functional guestmount tools

Tom Rini trini at konsulko.com
Wed Jul 15 21:51:21 CEST 2020


On Thu, Jul 09, 2020 at 09:42:25AM -0400, Tom Rini wrote:

> Since 2011 Ubuntu has intentionally broken support for guestmount[1] by
> default and requires sysadmin intervention to re-enable support.  This
> in turn exposed that in our tests if guestmount is available but fails
> we do not fall back to trying to use sudo.  Restructure our code to try
> sudo if guestmount fails rather than only when it is not in our path.
> Further, only note that we are using fuse on success of the call.
> 
> [1]: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/759725
> 
> Cc: Heinrich Schuchardt <xypron.glpk at gmx.de>
> Cc: Simon Glass <sjg at chromium.org>
> Cc: Stephen Warren <swarren at nvidia.com>
> Signed-off-by: Tom Rini <trini at konsulko.com>
> Reviewed-by: Stephen Warren <swarren at nvidia.com>
> Reviewed-by: Simon Glass <sjg at chromium.org>

So, I'm rejecting this patch.  While it fixes the tests running under
GitLab, when the runner is configured correctly:
https://gitlab.denx.de/u-boot/u-boot/-/jobs/124997
It causes Travis to fail the tests now:
https://travis-ci.org/github/trini/u-boot/jobs/708328027
whereas the usually are run and pass:
https://travis-ci.org/github/trini/u-boot/jobs/708035304

And in neither case are these all run on Azure:
https://dev.azure.com/u-boot/u-boot/_build/results?buildId=909&view=logs&j=50449d1b-398e-53ae-48fa-6bf338edeb51&t=97605dd2-f5a5-5dd7-2118-315ffdc8bcd6

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200715/a40dd75b/attachment.sig>


More information about the U-Boot mailing list