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

listers-loaders-conf-update

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 31382
Build 49091: 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.