Closed
Bug 883681
Opened 12 years ago
Closed 12 years ago
puppetAgain should not force ifcfg management
Categories
(Infrastructure & Operations Graveyard :: CIDuty, task)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Callek, Assigned: Callek)
Details
Attachments
(1 file)
2.20 KB,
patch
|
dustin
:
review+
|
Details | Diff | Splinter Review |
So today in final stages of setting up seamonkey's puppetmaster I did a reboot, which then brought the host down.
:fox2mike noticed it had a modified ifcfg-eth0 which was forcing it to try to use DHCP which of course, community doesn't have available.
This patch allows us to mark it to not be managed (so we don't blow away the proper ifcfg)
Attachment #763289 -
Flags: review?(dustin)
Assignee | ||
Comment 1•12 years ago
|
||
Comment on attachment 763289 [details] [diff] [review]
[puppet320] v1
Review of attachment 763289 [details] [diff] [review]:
-----------------------------------------------------------------
::: modules/network/manifests/init.pp
@@ +2,4 @@
> # License, v. 2.0. If a copy of the MPL was not distributed with this
> # file, You can obtain one at http://mozilla.org/MPL/2.0/.
> class network {
> + import config
err include ::config
Comment 2•12 years ago
|
||
Comment on attachment 763289 [details] [diff] [review]
[puppet320] v1
Review of attachment 763289 [details] [diff] [review]:
-----------------------------------------------------------------
This looks fine. Don't forget to document the new config var.
Attachment #763289 -
Flags: review?(dustin) → review+
Assignee | ||
Comment 3•12 years ago
|
||
http://hg.mozilla.org/users/dmitchell_mozilla.com/puppet320/rev/4fb76299ca9d
https://wiki.mozilla.org/index.php?title=ReleaseEngineering%2FPuppetAgain%2FModules%2Fconfig&action=historysubmit&diff=667380&oldid=666847
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
Updated•7 years ago
|
Product: Release Engineering → Infrastructure & Operations
Updated•5 years ago
|
Product: Infrastructure & Operations → Infrastructure & Operations Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•