NAK/cmnt: [ACT][PATCH 1/4] UBUNTU: SAUCE: fix git clone failures and setups not being idempotent
Po-Hsu Lin
po-hsu.lin at canonical.com
Mon Jun 7 14:02:31 UTC 2021
I agree that the second run on the same SUT should not alter the test
result.
We don't have this failed-with-second-run issue before because we will
clean up the autotest directory by rsync'ing a clean tree from our
jenkins server no matter it's a freshly deployed system or a manually
provisioned one. The old autotest/client/tmp/<test-suite> directory
will be gone, the test will start fresh.
This is defintely an issue we need to deal with if we want to run tests
for multiple times manually on the same SUT (I just clean out the
aformentioned tmp directory manually).
As discussed on Mattermost, I think this fix is base on the assumption
that the change to autotest f6e444df45f45 ("UBUNTU: SAUCE: fix missing
setup on second run") is correct.
With a quick test on the ubuntu_ltp_syscalls test, this will make the
test clone the source and build it twice before running the actual
syscall tests from LTP. From this fact it looks like the setup() was
called twice with one execution.
On the other hand, this issue does not exist in libhugetlbfs test,
which will do a git clone for an external source as well, therefore
it doesn't look like a real fix to me.
More information about the kernel-team
mailing list