Differential D4280

storage.api.server: Align configuration structure
ClosedPublic
Actions

Authored by ardumont on Oct 16 2020, 10:27 AM.

Details

Reviewers

None

Group Reviewers

Reviewers

Commits

rDCIDXda7e2ecee0cc: api.server: Align configuration structure with clients configuration

Summary

This should fix the swh-docker-dev ci failure [1]

Related to D4272

[1] https://jenkins.softwareheritage.org/job/swh-docker-dev/lastBuild/console

Test Plan

tox

doco up swh-idx-storage starts without failure [1]

[1]

$ grep swh-idx-storage -A3 $SWH_ENVIRONMENT_HOME/docker/docker-compose.override.yml
  swh-idx-storage:
    volumes:
      - "$SWH_ENVIRONMENT_HOME/swh-indexer:/src/swh-indexer"
$ doco up swh-idx-storage
...
swh-idx-storage_1               | [2020-10-16 08:30:27 +0000] [1] [INFO] Starting gunicorn 20.0.4
swh-idx-storage_1               | [2020-10-16 08:30:27 +0000] [1] [DEBUG] Arbiter booted
swh-idx-storage_1               | [2020-10-16 08:30:27 +0000] [1] [INFO] Listening at: http://0.0.0.0:5007 (1)
...

Diff Detail

Repository

rDCIDX Metadata indexer

Branch

master

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 16365
Build 25202: Phabricator diff pipeline on jenkins	Jenkins console · Jenkins
Build 25201: arc lint + arc unit

Event Timeline

ardumont created this revision.Oct 16 2020, 10:27 AM

Herald added a reviewer: Reviewers. · View Herald TranscriptOct 16 2020, 10:27 AM

Build is green

Patch application report for D4280 (id=15136)

Rebasing onto 934592a141...

Current branch diff-target is up to date.

Changes applied before test

commit 355cd274aa3523fd445e87e299f7845ce3d50bc6
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Fri Oct 16 10:27:53 2020 +0200

    storage.api.server: Align configuration structure

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

Harbormaster completed remote builds in B16363: Diff 15136.Oct 16 2020, 10:29 AM

ardumont edited the test plan for this revision. (Show Details)Oct 16 2020, 10:31 AM

ardumont edited the test plan for this revision. (Show Details)

ardumont edited the summary of this revision. (Show Details)

Update commit message

Build is green

Patch application report for D4280 (id=15138)

Rebasing onto 934592a141...

Current branch diff-target is up to date.

Changes applied before test

commit da7e2ecee0cc6d7dfc2b757e1f9da5e593d969d7
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Fri Oct 16 10:27:53 2020 +0200

    api.server: Align configuration structure with clients configuration

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

Harbormaster completed remote builds in B16365: Diff 15138.Oct 16 2020, 10:40 AM

This revision was not accepted when it landed; it landed in state Needs Review.Oct 16 2020, 10:47 AM

Closed by commit rDCIDXda7e2ecee0cc: api.server: Align configuration structure with clients configuration (authored by ardumont). · Explain Why

This revision was automatically updated to reflect the committed changes.

ardumont added a commit: rDCIDXda7e2ecee0cc: api.server: Align configuration structure with clients configuration.