[Stable10] Add conditional delay to avoid same stime in int test

[SergioBertolinSG]

Should prevent the "Merging shares" test suites to fail randomly as the
results order is dependent on the share time.

Backport of #28310

[SergioBertolinSG]

Required for #28268

[davitol]

👍

[PVince81]

👍

[PVince81]

so it doesn't work :-(

sharing.Merging shares for recipient when shared from outside with group then user and recipient renames in between

[phil-davis]

I had this example in a PR in master today, that should have been at a point past when this stuff was merged to master:
#28318 (comment)
I was a bit surprised that it had happened. When tried to rebase, my PR was already up-to-date with master, so I am pretty confident that it ran with this code.

[PVince81]

This failure is bad luck. Some tests running too fast (yeah I know, surprising?). This PR was supposed to eliminate this failure by introducing a conditional delay.

[PVince81]

we could maybe add a log before the sleep just to see whether it's really going there when the failure happens...

or maybe this one tests uses another code path that doesn't go there... @SergioBertolinSG mind checking ?

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.