Paths

Table of Contentst

Differential D5573

Update the counters' journal clients configuration to count the persons
ClosedPublic
Actions

Authored by vsellier on Apr 22 2021, 12:08 PM.

Details

Reviewers

ardumont

Group Reviewers

Reviewers

Maniphest Tasks

T3251: Count authors from revisions and releases

Commits

rDENVd2dac157b76b: Update the counters' journal clients configuration to count the persons

Summary

Related to T3251

Test Plan

docker-compose -f docker-compose.yml -f docker-compose-counters.yml -f docker-compose.override.yml up

Diff Detail

Repository

rDENV Development environment

Branch

master

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 20964
Build 32535: arc lint + arc unit

Event Timeline

vsellier created this revision.Apr 22 2021, 12:08 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptApr 22 2021, 12:08 PM

vsellier requested review of this revision.Apr 22 2021, 12:08 PM

Harbormaster completed remote builds in B20925: Diff 19920.Apr 22 2021, 12:08 PM

great

docker/docker-compose-counters.yml
40 ↗	(On Diff #19920)	Could we drop the wait-for-it stanza endpoint.sh if we use this?

This revision is now accepted and ready to land.Apr 22 2021, 12:39 PM

beware the typo in the diff and commit message.

ardumont mentioned this in T3251: Count authors from revisions and releases.Apr 23 2021, 11:22 AM

just keep the topic configuration as the journal split is not needed anymore
fix the type in the commit message

Harbormaster completed remote builds in B20964: Diff 19962.Apr 23 2021, 12:25 PM

vsellier retitled this revision from Update th counters' journal clients configuration to count the persons to Update the counters' journal clients configuration to count the persons.Apr 23 2021, 12:26 PM

Closed by commit rDENVd2dac157b76b: Update the counters' journal clients configuration to count the persons (authored by vsellier). · Explain WhyApr 23 2021, 12:29 PM

This revision was automatically updated to reflect the committed changes.

vsellier added a commit: rDENVd2dac157b76b: Update the counters' journal clients configuration to count the persons.

Revision Contents
Changeset List

Path

Size

docker/

conf/

counters_journal_client.yml

1 line

docker-compose.yml

1 line

Diff 19962

View Options

docker/conf/counters_journal_client.yml