swh-indexer: Fix configuration to add the tools configuration entry
ClosedPublic
Actions

Authored by ardumont on Dec 9 2020, 7:46 PM.

Details

Reviewers

vsellier

Group Reviewers

Reviewers

Maniphest Tasks

T2854: docker: Service Startup Error

Commits

rDENV81f3b0e8eee4: swh-indexer: Fix configuration to add the tools configuration entry

Summary

Without this, the indexer fails on KeyError tools (which blocks further
inquiry in swh-search).

I forgot to mention I encountered this when I replied [1]

It's not the same issue as mentioned there (and it's still not blocking for
it). It's worth acknowledging there is an issue nonetheless.

It's blocking for swh-search within docker though.

Related to T2854

Test Plan

doco up -f docker-compose.yml -f docker-compose.search.yml up

Diff Detail

Repository

rDENV Development environment

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Event Timeline

ardumont created this revision.Dec 9 2020, 7:46 PM

Harbormaster completed remote builds in B17849: Diff 16676.Dec 9 2020, 7:46 PM

Rebase

Harbormaster completed remote builds in B17853: Diff 16680.Dec 9 2020, 10:05 PM

LGTM

vsellier accepted this revision.Dec 10 2020, 9:45 AM

This revision is now accepted and ready to land.Dec 10 2020, 9:45 AM

Closed by commit rDENV81f3b0e8eee4: swh-indexer: Fix configuration to add the tools configuration entry (authored by ardumont). · Explain WhyDec 10 2020, 9:50 AM

This revision was automatically updated to reflect the committed changes.

ardumont added a commit: rDENV81f3b0e8eee4: swh-indexer: Fix configuration to add the tools configuration entry.

Revision Contents
Changeset List

Path

Size

docker/

conf/

indexer.yml

5 lines

Diff 16688

View Options

swh-indexer: Fix configuration to add the tools configuration entryClosedPublicActions