Remove casper module from puppet

RESOLVED FIXED

Status

Infrastructure & Operations
RelOps: Puppet
RESOLVED FIXED
11 months ago
11 months ago

People

(Reporter: dragrom, Assigned: dragrom)

Tracking

Details

Attachments

(1 attachment)

(Assignee)

Description

11 months ago
The entire casper module can be ripped out.  It isn't actually being used by anything and there are no plans to ever use casper.
(Assignee)

Updated

11 months ago
Assignee: relops → dcrisan
Status: NEW → ASSIGNED
(Assignee)

Comment 1

11 months ago
Created attachment 8877138 [details] [diff] [review]
Bug_1372494_Remove_casper_module_from_puppet.patch

Remove casper puppet module
Attachment #8877138 - Flags: review?(jwatkins)
Comment on attachment 8877138 [details] [diff] [review]
Bug_1372494_Remove_casper_module_from_puppet.patch

Review of attachment 8877138 [details] [diff] [review]:
-----------------------------------------------------------------

r+ but you need to remove the include in the node defs
https://dxr.mozilla.org/build-central/source/puppet/manifests/moco-nodes.pp#279

In fact, go ahead and remove all three casper node definitions.  Those hosts don't exist anymore.
Attachment #8877138 - Flags: review?(jwatkins) → review+
(Assignee)

Comment 3

11 months ago
Comment on attachment 8877138 [details] [diff] [review]
Bug_1372494_Remove_casper_module_from_puppet.patch

 https://hg.mozilla.org/build/puppet/rev/99510b144d9a322164fce6e0d3dee8224af358b9
Attachment #8877138 - Flags: checked-in+
(Assignee)

Updated

11 months ago
Status: ASSIGNED → RESOLVED
Last Resolved: 11 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.