erp5: restore ZODB using the --with-verify option of "repozo --recover"
"repozo --verify" is not working as this code expects it to: it simply prints errors in stdout, and doesn't return an error code in case of error. Thus, running it had absolutely no effect, except wasting IO and CPU time. This commit introduces the use of "repozo --recover --with-verify", which runs the verify and the recover in a same step, and has the advantage to raise (it doesn't exit with 0) in case of error. Also, as it does the verification and the recovery at the same time, it uses half the IO for the read. On a production server using SSDs, with a ZODB of 1Tb, runner-import-restore now takes 14h instead of 26h, iow a performance increase of 46%.
Status | Job ID | Name | Coverage | ||||||
---|---|---|---|---|---|---|---|---|---|
External | |||||||||
passed |
#651617
external
|
Cloudooo.UnitTest-Master |
00:09:31
|
||||||
failed |
#651671
external
|
ERP5.StandaloneDeploymentScriptTest-Debian.Buster |
15:01:46
|
||||||
passed |
#651666
external
|
ERP5.StandaloneDeploymentScriptTest-Debian.Buster.Master |
08:35:46
|
||||||
passed |
#651632
external
|
SlapOS.Eggs.UnitTest-Master.Python2 |
00:14:01
|
||||||
passed |
#651646
external
|
SlapOS.Eggs.UnitTest-Master.Python3 |
00:15:05
|
||||||
passed |
#651628
external
|
SlapOS.Eggs.UnitTest-Master.Python3Next |
00:15:15
|
||||||
passed |
#651629
external
|
SlapOS.Theia.ResilienceTest-Master-ERP5 |
06:48:17
|
||||||
passed |
#651621
external
|
Wendelin.StandaloneDeploymentScriptTest-Debian.Buster |
00:30:24
|
||||||