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

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.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 29968

View Options

proxmox/terraform/staging/init.tf