Support yaml config files when run via CLI.
AbandonedPublic
Actions

Authored by vlorentz on Dec 20 2018, 6:33 PM.

Details

Reviewers

None

Group Reviewers

Reviewers

Summary

Untuitively, config.read_raw_config is higher level than config.read;
the latter only supports INI.

Diff Detail

Repository

rDCIDX Metadata indexer

Branch

cli-yaml

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 3227
Build 4140: tox-on-jenkins	Jenkins
Build 4139: arc lint + arc unit

Event Timeline

vlorentz created this revision.Dec 20 2018, 6:33 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptDec 20 2018, 6:33 PM

vlorentz added a child revision: D875: docker-dev: Make the mimetype indexer work..Dec 20 2018, 6:37 PM

Build is green
See https://jenkins.softwareheritage.org/job/DCIDX/job/tox/162/ for more details.

Harbormaster completed remote builds in B3227: Diff 2775.Dec 20 2018, 6:37 PM

something is amiss.

config.read calls config.read_raw_config if a config_path is provided (it is here).
both supports the yaml due to that.

So i'm not understanding what you are saying.

nvm

Revision Contents
Changeset List

Path

Size

swh/

indexer/

storage/

api/

server.py

2 lines

Diff 2775

View Options

swh/indexer/storage/api/server.py

Support yaml config files when run via CLI.AbandonedPublicActions