Page MenuHomeSoftware Heritage
Differential D8038

Provision scrubber1 for checker services
ClosedPublic
Actions

Authored by ardumont on Jun 27 2022, 2:59 PM.

Details

Reviewers
vsellier
Group Reviewers
System administrators
Maniphest Tasks
T4324: production: Deploy swh-scrubber database and checkers
Commits
rSPREc1b2b15b2cad: Provision scrubber1 for checker services
Summary

Related to T4324

Test Plan

terraform applied [1]

[1] P1392

Diff Detail

Repository
rSPRE sysadm-provisioning
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.Jun 27 2022, 2:59 PM
Herald added a reviewer: System administrators. · View Herald TranscriptJun 27 2022, 2:59 PM
ardumont requested review of this revision.Jun 27 2022, 2:59 PM
Harbormaster completed remote builds in B30033: Diff 28942.Jun 27 2022, 2:59 PM
ardumont mentioned this in T4324: production: Deploy swh-scrubber database and checkers.Jun 27 2022, 3:06 PM
vsellier requested changes to this revision.Jun 27 2022, 3:24 PM
vsellier added a subscriber: vsellier.
vsellier added inline comments.
proxmox/terraform/production/production.tf
162

(it seems the first available id id 153)

179
This revision now requires changes to proceed.Jun 27 2022, 3:24 PM
ardumont edited the test plan for this revision. (Show Details)Jun 27 2022, 3:28 PM
ardumont updated this revision to Diff 28948.Jun 27 2022, 3:28 PM

Adapt according to review

Harbormaster completed remote builds in B30038: Diff 28948.Jun 27 2022, 3:28 PM
ardumont added a comment.EditedJun 27 2022, 3:32 PM

Aadapt according to review ^

proxmox/terraform/production/production.tf
179

nice catch and thx btw.

ardumont edited the test plan for this revision. (Show Details)Jun 27 2022, 4:38 PM
ardumont updated this revision to Diff 28951.Jun 27 2022, 4:39 PM

Update with actual appliance (which updated the state)

Harbormaster completed remote builds in B30041: Diff 28951.Jun 27 2022, 4:39 PM
ardumont added inline comments.Jun 27 2022, 4:41 PM
proxmox/terraform/production/production.tf
179

so actually it needs to be "vmbr0" in our terraform manifest.
This is the actual mapping for the vlan440.

ardumont added inline comments.Jun 27 2022, 4:42 PM
proxmox/terraform/versions.tf
7

In my side-tracking quest of finding out why it refused to apply initially (due to the vlan being badly defined in the end), this got upgraded as well (and stuff got applied so i kept it).

vsellier accepted this revision.Jun 27 2022, 4:48 PM
This revision is now accepted and ready to land.Jun 27 2022, 4:48 PM
Closed by commit rSPREc1b2b15b2cad: Provision scrubber1 for checker services (authored by ardumont). · Explain WhyJun 27 2022, 4:58 PM
This revision was automatically updated to reflect the committed changes.
ardumont added a commit: rSPREc1b2b15b2cad: Provision scrubber1 for checker services.

Revision Contents
Changeset List

PathSize
proxmox/
terraform/
production/
28 lines
204 lines
2 lines

Diff 28953

View Options

proxmox/terraform/production/production.tf

Loading...
View Options

proxmox/terraform/production/terraform.tfstate

Loading...
View Options

proxmox/terraform/versions.tf

Loading...
About · Developer information · Legal