Differential D868

Do only register tasks that are not "abstract" and defined in the very module
AbandonedPublic
Actions

Authored by douardda on Dec 20 2018, 4:15 PM.

Tags

None

Subscribers

Details

Reviewers

Group Reviewers

Summary

This is especially required for the gitlab lister to prevent things like
RangeListerTask or _GitLabListerTaskBase (after it has been renamed from
GitLabListerTask) from being registered (which fails and prevent tasks
and queues from being properly set up at worker startup time).

Diff Detail

Repository

rDSCH Scheduling utilities

Branch

master

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 3216
Build 4124: tox-on-jenkins	Jenkins
Build 4123: arc lint + arc unit

Event Timeline

douardda created this revision.Dec 20 2018, 4:15 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptDec 20 2018, 4:16 PM

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

Harbormaster completed remote builds in B3216: Diff 2768.Dec 20 2018, 4:18 PM

douardda mentioned this in D869: Rename GitLabListerTask in _GitLabListerTaskBase.Dec 20 2018, 4:20 PM

vlorentz accepted this revision.Dec 20 2018, 4:20 PM

This revision is now accepted and ready to land.Dec 20 2018, 4:20 PM

(probably) not required any more after D889

Revision Contents
Changeset List

Path

Size

swh/

scheduler/

celery_backend/

4 lines

Diff 2768

swh/scheduler/celery_backend/config.py

Loading...