Remove redundant config checks in load_and_check_config.
ClosedPublic
Actions

Authored by vlorentz on Jan 22 2020, 12:25 PM.

Details

Reviewers

ardumont

Group Reviewers

Reviewers

Commits

rDSTOcfee7b519bb0: Remove redundant config checks in load_and_check_config.

Summary

There is no reason to force the server to serve only the 'local' backend anymore
Missing arguments will error when instantiating the backend class.

Diff Detail

Repository

rDSTO Storage manager

Branch

master

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 10228
Build 15170: tox-on-jenkins	Jenkins
Build 15169: arc lint + arc unit

Event Timeline

vlorentz created this revision.Jan 22 2020, 12:25 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptJan 22 2020, 12:25 PM

vlorentz mentioned this in D2566: Add Cassandra backend..Jan 22 2020, 12:28 PM

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

Harbormaster completed remote builds in B10228: Diff 9165.Jan 22 2020, 12:28 PM

vlorentz added a child revision: D2566: Add Cassandra backend..Jan 22 2020, 12:29 PM

looks good to me

This revision is now accepted and ready to land.Jan 22 2020, 2:47 PM

Closed by commit rDSTOcfee7b519bb0: Remove redundant config checks in load_and_check_config. (authored by vlorentz). · Explain WhyJan 22 2020, 2:49 PM

This revision was automatically updated to reflect the committed changes.

vlorentz added a commit: rDSTOcfee7b519bb0: Remove redundant config checks in load_and_check_config..

Revision Contents
Changeset List

Path

Size

swh/

storage/

api/

server.py

14 lines

tests/

test_server.py

44 lines

Diff 9165

View Options

swh/storage/api/server.py