landreev
landreev
When/if Jenkins comes back to life, if the tests complete w/out failing 3 times in a row with this extra second added, I'll vote for merging it quickly; if it...
I started a build by hand just now.
@stevenwinship Could you please sync the branch up with develop, appears to be the reason the last Jenkins run bombed.
one successful run earlier today (build 14). build 15: a really weird failure - the other harvesting test, `testHarvestingClientRun_AllowHarvestingMissingCVV_True`, 4 datasets harvested instead of 8. (??)
Don has fixed the payara distribution issue. But it's still Jenkins roulette, with the harvesting tests failing half of the times; somehow it's been made worse so far. So no,...
ok, one mystery solved - I just left a comment in #10464, the PR I merged yesterday; the new errors are the result of the extra check added, which is...
also, just saw that you have fixed it already.
> also, just saw that you have fixed it already. OK, you have fixed the cleanup part - but the search check in line 278 may still bomb for the...
Anyway, I just want to see it pass a few times in a row and merge it without further ado.
There were more annoying unrelated Jenkins failures. But the 2 harvesting tests have objectively passed three times in a row. Merging.