Page MenuHomeSoftware Heritage

common/origin_save: Fix elasticsearch request
ClosedPublic

Authored by anlambert on Jan 13 2022, 12:44 PM.

Details

Summary

The elasticsearch request used to get detailed info about a save code
now request was not returning any result anymore as some fields in
the swh-workers-* index got renamed.

Diff Detail

Repository
rDWAPPS Web applications
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 26002
Build 40638: Phabricator diff pipeline on jenkinsJenkins console · Jenkins
Build 40637: arc lint + arc unit

Event Timeline

Build is green

Patch application report for D6941 (id=25131)

Rebasing onto 45429e8d5c...

First, rewinding head to replay your work on top of it...
Applying: common/origin_save: Fix elasticsearch request
Changes applied before test
commit e341f636ee7f0cd022ede60a7faf718c41b170d2
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Thu Jan 13 12:43:06 2022 +0100

    common/origin_save: Fix elasticsearch request
    
    The elasticsearch request used to get detailed info about a save code
    now request was not returning any result anymore as some fields in
    the swh-workers-* index got renamed.

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

Build is green

Patch application report for D6941 (id=25132)

Rebasing onto 45429e8d5c...

Current branch diff-target is up to date.
Changes applied before test
commit 491576531af7330014af3408e3cb4d585c3c05da
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Thu Jan 13 12:43:06 2022 +0100

    common/origin_save: Fix elasticsearch request
    
    The elasticsearch request used to get detailed info about a save code
    now request was not returning any result anymore as some fields in
    the swh-workers-* index got renamed.

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

vsellier added a subscriber: vsellier.

LGTM Thanks for fixing this.

This revision is now accepted and ready to land.Jan 13 2022, 2:17 PM