https://jira.whamcloud.com This file is an XML representation of an issue en-us 9.4.14 940014 05-12-2023 https://jira.whamcloud.com/browse/LU-940 Lustre <p>Many places in the test framework issue 3 syncs <span class="error">&#91;sync; sync; sync&#93;</span> do flush the filesystem before <span class="error">&#91;for example&#93;</span> failover.</p> <p>This 3 sync strategy should not be required, is not what a user would do and so is either unneccessary or hides unlying sync issues.</p> <p>The multiple calls to sync should be remove.</p> LU-940

Many places in the test framework issue 3 syncs [sync; sync; sync] do flush the filesystem before [for example] failover.

Bug Major Resolved Low Priority WC Triage Chris Gearing Mon, 19 Dec 2011 08:08:47 +0000 Wed, 28 Feb 2018 20:21:10 +0000 Wed, 28 Feb 2018 20:21:10 +0000 0 1 <p>I thought about the same when saw syncs before failover are being done implicitly. This is bad practice because it makes our tests working fine, but nobody will do sync on customer clusters before reboot, so we are getting green test results by moving problems on customer side which is not acceptable. Our green tests make just no sense if bugs are hidden.</p> <p>Meanwhile there is 3 sync in replay-barrier which is artificial construction just to make sure all data is on disk before we continue. It is used mostly in our unit tests to check various recovery cases with strong operation ordering and state, so it is required to have syncs there otherwise operation state is undefined - committed or not.</p> <p>There were a couple of places in test framework which used 3 syncs in a row.<br/> I have fixed those in this patch (<a href="http://review.whamcloud.com/#/c/7280" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/#/c/7280</a>)</p> <p>I somehow missed Mikhail's comment above.<br/> Since the 3 sync strategy in replay_barrier is an artificial construction, this patch is not required. <br/> Abandoned it.</p> Development Rank 1|hzvxlz: Rank (Obsolete) 9625 Severity