Differential D6308

Add a documentation page to list the services urls
ClosedPublic
Actions

Authored by vsellier on Sep 20 2021, 3:57 PM.

Details

Reviewers

douardda
ardumont

Group Reviewers

Maniphest Tasks

T3591: Create a list/inventory of the hostname/port of all services
T3408: Provide read-only access to production servers

Commits

rDDOC3a935af020e7: add a documentation page to list the services urls

Summary

Related to T3408

Diff Detail

Repository

rDDOC Development documentation

Branch

master

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 23824
Build 37148: arc lint + arc unit

Event Timeline

vsellier created this revision.Sep 20 2021, 3:57 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptSep 20 2021, 3:57 PM

vsellier requested review of this revision.Sep 20 2021, 3:57 PM

Harbormaster completed remote builds in B23824: Diff 22898.Sep 20 2021, 3:57 PM

vsellier planned changes to this revision.Sep 20 2021, 3:57 PM

ok, nice start.
Do you reference only the staging part?

docs/infrastructure/service-urls.rst
28	typo ^

fix typos

Harbormaster completed remote builds in B23836: Diff 22908.Sep 20 2021, 6:20 PM

Do you reference only the staging part?

Nope the diff is a draft to validate the format.
Production urls are in progress, (will be in the section line 84)

I put the staging urls first as it would be better to use them instead of the production ones.

docs/infrastructure/service-urls.rst
28	lol thanks. it's fixed

fix a couple of staging urls
add the production urls

Harbormaster completed remote builds in B23837: Diff 22909.Sep 20 2021, 7:42 PM

complete other tools section

Harbormaster completed remote builds in B23838: Diff 22910.Sep 20 2021, 9:45 PM

vsellier retitled this revision from WIP - add a documentation page to list the services urls to Add a documentation page to list the services urls.Sep 21 2021, 7:52 AM

vsellier added a project: System administration.

vsellier edited the summary of this revision. (Show Details)

vlorentz added a task: T3591: Create a list/inventory of the hostname/port of all services.Sep 21 2021, 11:30 AM

olasd added a subscriber: olasd.Sep 21 2021, 11:34 AM

olasd added inline comments.

docs/infrastructure/service-urls.rst
67	RabbitMQ

RabbitMq GUI -> RabbitMQ

Harbormaster completed remote builds in B23860: Diff 22931.Sep 21 2021, 11:49 AM

vlorentz added a subscriber: vlorentz.Sep 21 2021, 5:48 PM

vlorentz added inline comments.

docs/infrastructure/service-urls.rst
61–62	can we call them "RPC servers" instead? "backend" isn't very specific

vsellier added inline comments.Sep 22 2021, 10:41 AM

docs/infrastructure/service-urls.rst
61–62	The RPC servers are listed in the previous section, in this one, it's more something like "databases". I'm not really convinced too by "backend" but I have no better idea for now...

douardda added a subscriber: douardda.Sep 22 2021, 10:43 AM

douardda added inline comments.

docs/infrastructure/service-urls.rst
25	nitpick: is there reason for keeping the trailing / on this one only?
60	Not read the whole doc, so maybe it's not a valid remqrk, but are "private" and "VPN" defined somewhere?

vsellier added inline comments.Sep 22 2021, 10:56 AM

docs/infrastructure/service-urls.rst
25	no reason, It's the remaining probably, of a copy/paste I will clean that thanks
60	Nope good catch

Remove useless ending '/'
define VPN / private meanings

Harbormaster completed remote builds in B23885: Diff 22954.Sep 22 2021, 11:25 AM

LGTM (not checked everything is accurate nor there are obvious missing services, but it's a huge improvement as is, thx)

This revision is now accepted and ready to land.Sep 22 2021, 11:41 AM

ardumont accepted this revision.Sep 22 2021, 11:52 AM

Closed by commit rDDOC3a935af020e7: add a documentation page to list the services urls (authored by vsellier). · Explain WhySep 22 2021, 12:35 PM

This revision was automatically updated to reflect the committed changes.

vsellier added a commit: rDDOC3a935af020e7: add a documentation page to list the services urls.

Revision Contents
Changeset List

Path

Size

docs/

infrastructure/

3 lines

2 lines

service-urls.rst

96 lines

Diff 22898

docs/infrastructure/index.rst

Loading...

docs/infrastructure/network.rst

Loading...

docs/infrastructure/service-urls.rst

Loading...