Page MenuHomeSoftware Heritage
Differential D863

journal.publisher: Fix multiple issues and add logging
ClosedPublic
Actions

Authored by ardumont on Dec 20 2018, 2:21 PM.

Details

Reviewers
vlorentz
Group Reviewers
Reviewers
Commits
rDJNL11fb891781df: publisher: Be compliant with other endpoints
rDJNL9c90d92a15ea: publisher: Add logging
rDJNL1e4106b1800e: journal.publisher: Filter on existing revisions
rDJNL66dbf6549a82: journal.publisher: Allow to run in debug mode
Summary

Revealed by actually running kafka with docker.

Test Plan

tox

Diff Detail

Repository
rDJNL Journal infrastructure
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 3197
Build 4099: tox-on-jenkinsJenkins
Build 4098: arc lint + arc unit

Event Timeline

ardumont created this revision.Dec 20 2018, 2:21 PM
Herald added a reviewer: Reviewers. · View Herald TranscriptDec 20 2018, 2:21 PM
ardumont added a child revision: D860: docker-compose: Add journal stack.Dec 20 2018, 2:22 PM
Harbormaster completed remote builds in B3197: Diff 2753.Dec 20 2018, 2:23 PM
swh-public-ci added a subscriber: swh-public-ci.Dec 20 2018, 2:23 PM

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

vlorentz accepted this revision.Dec 20 2018, 2:30 PM
vlorentz added a subscriber: vlorentz.
vlorentz added inline comments.
swh/journal/publisher.py
57–58

why ######?

176

same question here and below

This revision is now accepted and ready to land.Dec 20 2018, 2:30 PM
ardumont marked an inline comment as done.Dec 20 2018, 2:34 PM
ardumont added inline comments.
swh/journal/publisher.py
57–58

arf to see them, i forgot to remove those.

ardumont updated this revision to Diff 2758.Dec 20 2018, 2:35 PM

Remove wrong log formats

swh-public-ci added a comment.Dec 20 2018, 2:37 PM

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

Harbormaster completed remote builds in B3200: Diff 2758.Dec 20 2018, 2:37 PM
Closed by commit rDJNL66dbf6549a82: journal.publisher: Allow to run in debug mode (authored by ardumont). · Explain WhyDec 20 2018, 2:41 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
swh/
journal/
53 lines
tests/
18 lines

Diff 2753

View Options

swh/journal/publisher.py

Loading...
View Options

swh/journal/tests/test_publisher.py

Loading...
About · Developer information · Legal