Differential D8407

loaders in development
ClosedPublic
Actions

Authored by anlambert on Sep 7 2022, 11:25 AM.

Details

Reviewers

ardumont

Group Reviewers

Reviewers

Commits

rDENV2aab3ffd23ce: docker/conf: Declare celery tasks for listers/loaders in development

Summary

It enables to test those in the docker environment.

Diff Detail

Repository

rDENV Development environment

Branch

master

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 31367
Build 49068: arc lint + arc unit

Event Timeline

anlambert created this revision.Sep 7 2022, 11:25 AM

Herald added a reviewer: Reviewers. · View Herald TranscriptSep 7 2022, 11:25 AM

anlambert requested review of this revision.Sep 7 2022, 11:25 AM

Harbormaster completed remote builds in B31367: Diff 30340.Sep 7 2022, 11:25 AM

Thanks.

fwiw, i think this should be added with diffs by the contributors of those new listers/loaders.

@bchauvet ^ I expected this to be done already.

@franckbret for your information, i must have forgotten to be explicit about this before so ;)

This revision is now accepted and ready to land.Sep 7 2022, 5:43 PM

Rebase

Harbormaster completed remote builds in B31382: Diff 30356.Sep 7 2022, 7:04 PM

In D8407#218601, @ardumont wrote:

Thanks.

fwiw, i think this should be added with diffs by the contributors of those new listers/loaders.

@bchauvet ^ I expected this to be done already.

@franckbret for your information, i must have forgotten to be explicit about this before so ;)

We should add this explicit step in lister/loader development process I guess, something like :"Declare the celery tasks in docker configuration files for lister/loader."

In D8407#218601, @ardumont wrote:

Thanks.

fwiw, i think this should be added with diffs by the contributors of those new listers/loaders.

@bchauvet ^ I expected this to be done already.

@franckbret for your information, i must have forgotten to be explicit about this before so ;)

We should add this explicit step in lister/loader development process I guess, something like :"Declare the celery tasks in docker configuration files for lister/loader."

/me *nods*

Closed by commit rDENV2aab3ffd23ce: docker/conf: Declare celery tasks for listers/loaders in development (authored by anlambert). · Explain WhySep 8 2022, 2:09 PM

This revision was automatically updated to reflect the committed changes.

anlambert added a commit: rDENV2aab3ffd23ce: docker/conf: Declare celery tasks for listers/loaders in development.