Page MenuHomeSoftware Heritage

deposit_update_status: Add support for updating status_detail
ClosedPublic

Authored by vlorentz on Aug 12 2021, 11:28 AM.

Details

Summary

This will be used by the deposit loader in case of failure,
to tell users what the actual error is.

Dependency of D6079

Diff Detail

Repository
rDDEP Push deposit
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Build is green

Patch application report for D6080 (id=22014)

Rebasing onto 573df1cc3d...

Current branch diff-target is up to date.
Changes applied before test
commit 90607721fcfecd17b4cd91e3f3c293a98d056f90
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Aug 12 11:27:51 2021 +0200

    deposit_update_status: Add support for updating status_detail
    
    This will be used by the deposit loader in case of failure,
    to tell users what the actual error is.

See https://jenkins.softwareheritage.org/job/DDEP/job/tests-on-diff/672/ for more details.

Build is green

Patch application report for D6080 (id=22017)

Rebasing onto 573df1cc3d...

Current branch diff-target is up to date.
Changes applied before test
commit 737838e772c8b1aa3e8766b2927d0391337695a9
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Aug 12 11:27:51 2021 +0200

    deposit_update_status: Add support for updating status_detail
    
    This will be used by the deposit loader in case of failure,
    to tell users what the actual error is.

See https://jenkins.softwareheritage.org/job/DDEP/job/tests-on-diff/674/ for more details.

Great.

And it algo got tested in docker with success ;)

This revision is now accepted and ready to land.Aug 12 2021, 12:20 PM