Closed Bug 1074277 Opened 10 years ago Closed 10 years ago

manage check_ami.py by puppet

Categories

(Release Engineering :: General, defect)

x86_64
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: rail, Assigned: rail)

References

Details

(Whiteboard: [kanban:engops:https://mozilla.kanbanize.com/ctrl_board/6/1973] )

Attachments

(1 file, 2 obsolete files)

Since we deployed runner on CentOS/Ubuntu we can get rid of the hack in
http://hg.mozilla.org/build/cloud-tools/file/da847f307007/cloudtools/aws/ami.py#l26
Attached patch check_ami.diff (obsolete) — Splinter Review
* Imported https://raw.githubusercontent.com/catlee/runner/master/example-tasks.d/0-check_ami.py
* need to install pyyaml
Attachment #8496960 - Flags: review?(catlee)
Attachment #8496960 - Flags: review?(catlee) → review+
Attached patch check_ami-1.diff (obsolete) — Splinter Review
exit 1 on on-demand or in-house machines makes runner unhappy
Attachment #8497187 - Flags: review?
Comment on attachment 8497187 [details] [diff] [review]
check_ami-1.diff

This shouldn't be installed on non-spot instances.
Attachment #8497187 - Flags: review?
Comment on attachment 8497187 [details] [diff] [review]
check_ami-1.diff

Backed out:
remote:   https://hg.mozilla.org/build/puppet/rev/eb361af56027
remote:   https://hg.mozilla.org/build/puppet/rev/4cd7e2eaef3e

On-demand instances started failing because they don't have user-data.
Attachment #8497187 - Flags: checked-in-
Whiteboard: [kanban:engops:https://mozilla.kanbanize.com/ctrl_board/6/1973]
Blocks: 1103123
We can land this now, spot and on-demand instances are equal now!
Attachment #8496960 - Attachment is obsolete: true
Attachment #8497187 - Attachment is obsolete: true
Attachment #8528571 - Flags: review?(catlee)
Attachment #8528571 - Flags: review?(catlee) → review+
https://hg.mozilla.org/build/cloud-tools/rev/aace5bbd0bcf
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: