Page MenuHomeSoftware Heritage
Differential D1197

Deduplicate origin batch scheduling wrt the scheduler.
ClosedPublic
Actions

Authored by vlorentz on Feb 26 2019, 2:24 PM.

Details

Reviewers
douardda
Group Reviewers
Reviewers
Commits
rDCIDXf271e1dba463: Deduplicate origin batch scheduling wrt the scheduler.
Summary

Depends on D1196

Diff Detail

Repository
rDCIDX Metadata indexer
Branch
schedule_origin_batches
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 4707
Build 6270: tox-on-jenkinsJenkins
Build 6269: arc lint + arc unit

Event Timeline

vlorentz created this revision.Feb 26 2019, 2:24 PM
Herald added a reviewer: Reviewers. · View Herald TranscriptFeb 26 2019, 2:24 PM
swh-public-ci added a subscriber: swh-public-ci.Feb 26 2019, 2:27 PM

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DCIDX/job/tox/466/
See console output for more information: https://jenkins.softwareheritage.org/job/DCIDX/job/tox/466/console

Harbormaster failed remote builds in B4454: Diff 3762!Feb 26 2019, 2:27 PM
douardda requested changes to this revision.Mar 19 2019, 10:05 AM
douardda added a subscriber: douardda.

Just to clarify this options sorting pb. Otherwise ok.

swh/indexer/cli.py
125

It's a bit weird that the parameters are not the same order as the options of the cli command.

This revision now requires changes to proceed.Mar 19 2019, 10:05 AM
vlorentz updated this revision to Diff 3991.Mar 19 2019, 10:13 AM
  • fix argument order
Harbormaster failed remote builds in B4707: Diff 3991!Mar 19 2019, 10:14 AM
swh-public-ci added a comment.Mar 19 2019, 10:14 AM

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DCIDX/job/tox/507/
See console output for more information: https://jenkins.softwareheritage.org/job/DCIDX/job/tox/507/console

swh-public-ci added a comment.Mar 27 2019, 2:35 PM

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DCIDX/job/tox/511/
See console output for more information: https://jenkins.softwareheritage.org/job/DCIDX/job/tox/511/console

douardda accepted this revision.Mar 28 2019, 9:30 AM
This revision is now accepted and ready to land.Mar 28 2019, 9:30 AM
swh-public-ci added a comment.Mar 28 2019, 4:48 PM

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DCIDX/job/tox/513/
See console output for more information: https://jenkins.softwareheritage.org/job/DCIDX/job/tox/513/console

vlorentz updated this revision to Diff 4247.Apr 2 2019, 4:20 PM

rebase

swh-public-ci added a comment.Apr 2 2019, 4:24 PM

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

Harbormaster completed remote builds in B5015: Diff 4247.Apr 2 2019, 4:24 PM
Closed by commit rDCIDXf271e1dba463: Deduplicate origin batch scheduling wrt the scheduler. (authored by vlorentz). · Explain WhyApr 2 2019, 4:29 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
swh/
indexer/
43 lines
tests/
14 lines

Diff 3991

View Options

swh/indexer/cli.py

Loading...
View Options

swh/indexer/tests/test_cli.py

Loading...
About · Developer information · Legal