Differential D8301

staging: Unify vlan declaration
ClosedPublic
Actions

Authored by ardumont on Aug 24 2022, 10:39 AM.

Details

Reviewers

vsellier

Group Reviewers

System administrators

Commits

rSPRE2a7e754dbaf3: staging: Unify vlan declaration

Summary

It was a problem once when we misnamed it and the error was unclear. Might as well
centralize it within the default configuration to avoid that problem again.

Test Plan

terraform plan does not change since it's the same as before.

Diff Detail

Repository

rSPRE sysadm-provisioning

Branch

master

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 31018
Build 48519: arc lint + arc unit

Event Timeline

ardumont created this revision.Aug 24 2022, 10:39 AM

Herald added a reviewer: System administrators. · View Herald TranscriptAug 24 2022, 10:39 AM

ardumont requested review of this revision.Aug 24 2022, 10:39 AM

Harbormaster completed remote builds in B31018: Diff 29967.Aug 24 2022, 10:39 AM

vsellier accepted this revision.Aug 24 2022, 10:42 AM

This revision is now accepted and ready to land.Aug 24 2022, 10:42 AM

Closed by commit rSPRE2a7e754dbaf3: staging: Unify vlan declaration (authored by ardumont). · Explain WhyAug 24 2022, 10:43 AM

This revision was automatically updated to reflect the committed changes.

ardumont added a commit: rSPRE2a7e754dbaf3: staging: Unify vlan declaration.

Revision Contents
Changeset List

Path

Size

proxmox/

terraform/

staging/

init.tf

18 lines

staging.tf

57 lines

Diff 29967

View Options

proxmox/terraform/staging/init.tf