Differential D4753

Add clearly-defined vm role to access the staging clearly defined db instance
ClosedPublic
Actions

Authored by ardumont on Dec 16 2020, 12:52 PM.

Tags

None

Subscribers

Details

Reviewers

Group Reviewers

System administrators

Maniphest Tasks

T2865: Prepare an environment to test the ClearlyDefined integration

Commits

rSPSITE2fbf746bab01: Add clearly-defined vm role to access the staging clearly defined db instance

Summary

This installs the necessary cogs to access the particular clearly_defined db instance.

$ psql service=staging-clearly-defined              # guest user, read-only access
$ psql service=admin-staging-clearly-defined  # clearly-define user access read/write access to the db schema

inventory [1] updated

Related to T2865

[1] https://inventory.internal.softwareheritage.org/virtualization/virtual-machines/90/

Test Plan

vagrant up staging-clearly-defined

.pg_service.conf and .pgpass configured:

$ tg1999@clearly-defined:~$ cat .pg_service.conf
# This file is managed by puppet. Local modifications will be overwritten.

[clearly-defined]
dbname=clearly_defined
host=db1.internal.staging.swh.network
port=5432
user=guest
[admin-clearly-defined]
dbname=clearly_defined
host=db1.internal.staging.swh.network
port=5432
user=guest
tg1999@clearly-defined:~$ ls -lah .pgpass
-r-------- 1 tg1999 tg1999 288 Dec 16 12:31 .pgpass

Diff Detail

Repository

rSPSITE puppet-swh-site

Lint

Automatic diff as part of commit; lint not applicable.

Unit

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

Event Timeline

ardumont created this revision.Dec 16 2020, 12:52 PM

Harbormaster completed remote builds in B17977: Diff 16821.Dec 16 2020, 12:52 PM

ardumont edited the test plan for this revision. (Show Details)Dec 16 2020, 1:33 PM

ardumont edited the summary of this revision. (Show Details)Dec 16 2020, 1:35 PM

Adapt to allow the per node configuration override.

Harbormaster completed remote builds in B17978: Diff 16822.Dec 16 2020, 1:36 PM

ardumont mentioned this in T2865: Prepare an environment to test the ClearlyDefined integration.Dec 16 2020, 1:37 PM

Add access to root user as well

Harbormaster completed remote builds in B17979: Diff 16823.Dec 16 2020, 1:38 PM

LGTM
as tested together, removing the deep option of lookup seems to result in a more predictable behavior :)

This revision is now accepted and ready to land.Dec 16 2020, 3:03 PM

Rebase

Harbormaster completed remote builds in B17983: Diff 16827.Dec 16 2020, 3:10 PM

Closed by commit rSPSITE2fbf746bab01: Add clearly-defined vm role to access the staging clearly defined db instance (authored by ardumont). · Explain WhyDec 16 2020, 3:10 PM

This revision was automatically updated to reflect the committed changes.

ardumont added a commit: rSPSITE2fbf746bab01: Add clearly-defined vm role to access the staging clearly defined db instance.

Revision Contents
Changeset List

Path

Size

data/

hostname/

clearly-defined.internal.staging.swh.network.yaml

13 lines

subnets/

2 lines

manifests/

4 lines

site-modules/

profile/

manifests/

postgresql/

4 lines

role/

manifests/

swh_db_client.pp

3 lines

Diff 16828

data/hostname/clearly-defined.internal.staging.swh.network.yaml

Loading...

data/subnets/vagrant.yaml

Loading...

manifests/site.pp

Loading...

site-modules/profile/manifests/postgresql/client.pp

Loading...

site-modules/role/manifests/swh_db_client.pp

Loading...