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

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

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

191 lines

Diff 22959

docs/infrastructure/index.rst

Loading...

docs/infrastructure/network.rst

Loading...

docs/infrastructure/service-urls.rst

Loading...