Closed
Bug 1136404
Opened 10 years ago
Closed 10 years ago
Remove Puppet at boot for Windows EC2 instances
Categories
(Infrastructure & Operations :: RelOps: Puppet, task)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: markco, Assigned: markco)
References
Details
Attachments
(2 files)
1.68 KB,
patch
|
Details | Diff | Splinter Review | |
1.68 KB,
patch
|
dustin
:
review+
markco
:
checked-in+
|
Details | Diff | Splinter Review |
A mechanism will need to be in place to remove the Puppet at boot schedule tasks for Windows EC2 instances that are based off of the golden image.
Assignee | ||
Comment 1•10 years ago
|
||
Attachment #8584161 -
Flags: review?(dustin)
Assignee | ||
Comment 2•10 years ago
|
||
Attachment #8584163 -
Flags: review?(dustin)
Assignee | ||
Updated•10 years ago
|
Attachment #8584161 -
Flags: review?(dustin)
Assignee | ||
Comment 3•10 years ago
|
||
This 2 part. First the aws user data will have this after the Puppet run:
:RM_SCH_TASK
schtasks /delete /TN RUNPUPPET
Then the attached patch removes the checks that would not allow buildbot to start.
Comment 4•10 years ago
|
||
Comment on attachment 8584163 [details] [diff] [review]
BUG1136404-2.patch
Review of attachment 8584163 [details] [diff] [review]:
-----------------------------------------------------------------
This will make things a little tricky when we *also* want to configure onsite hosts that do keep running Puppet.
Is it possible to replace the puppet run script, or add a conditional to it, so that it just pretends to run puppet (and sets the semaphore)?
Assignee | ||
Comment 5•10 years ago
|
||
We have a separate scripts based on if the machine is in AWS or not:
http://hg.mozilla.org/build/puppet/file/1c285f744720/modules/buildslave/manifests/startup/windows.pp
11 # Batch file to start buildbot
12 # Batch files varies slightly between AWS and datacenter machines
13 case $::fqdn {
14 /.*\.releng\.(use1|usw2)\.mozilla\.com$/: {
15 include instance_metadata
16 file {
17 'c:/programdata/puppetagain/start-buildbot.bat':
18 content => template("${module_name}/EC2_start-buildbot.bat.erb"),
19 require => Class[instance_metadata];
20 }
21 }
22 default: {
23 file {
24 'c:/programdata/puppetagain/start-buildbot.bat':
25 content => template("${module_name}/start-buildbot.bat.erb");
26 }
27 }
28 }
Comment 6•10 years ago
|
||
Comment on attachment 8584163 [details] [diff] [review]
BUG1136404-2.patch
Ah, perfect, thanks for the reminder!
Attachment #8584163 -
Flags: review?(dustin) → review+
Assignee | ||
Comment 7•10 years ago
|
||
Comment on attachment 8584163 [details] [diff] [review]
BUG1136404-2.patch
Review of attachment 8584163 [details] [diff] [review]:
-----------------------------------------------------------------
https://hg.mozilla.org/build/puppet/rev/7eb37cc224db
Attachment #8584163 -
Flags: checked-in+
Assignee | ||
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•