Closed
Bug 1480108
Opened 7 years ago
Closed 7 years ago
OCC deploy of gecko-(1,2,3)-b-win2012 broken
Categories
(Infrastructure & Operations :: RelOps: OpenCloudConfig, task)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: pmoore, Assigned: grenade)
Details
Attachments
(1 file)
58 bytes,
patch
|
pmoore
:
review+
|
Details | Diff | Splinter Review |
Am 27.07.2018 um 16:26 schrieb Rob Thijssen <rthijssen@mozilla.com>:
Hi everyone,
There's a problem with deployments from OCC of the gecko-(1,2,3)-b-win2012
worker types. Only deployments to these workers are affected. Not testers.
I'll fix it shortly but for the time being, if you are in the position of
having to deploy any of these when I'm not around, there are a couple steps
to take before and after in order for the deployment to work correctly. The
problem is caused because OCC doesn't yet know how to modify worker type
definitions that contain multiple instance types (eg: c4.4xlarge,
c5.4xlarge) instead of only one instance type. Notes below are for
gecko-1-b-win2012, adjust links for the other two.
- before kicking off a deployment, visit the worker type aws provisioner
config at:
https://tools.taskcluster.net/aws-provisioner/gecko-1-b-win2012/edit .
Delete one of the two definitions inside the "instanceTypes" array
property. It doesn't matter which one you delete (c4.4xlarge, c5.4xlarge)
as long as there's only one left when you save the edited workertype
definition.
- kick off the deployment normally with a git commit and push containing
deploy syntax.
- revisit the aws provisioner config (taking care to refresh the page
from before, to pick up the new config from the occ deploy. occ will change
the config but your old browser tab will still have old config with old ami
and deployment ids). add an instanceType definition for c4.4xlarge.
reference definitions are available here:
https://github.com/mozilla-releng/OpenCloudConfig/tree/python-ci-migration/userdata/Configuration/provisioner
wake me up if there's a problem or this isn't clear and a deployment is
urgent. *************
Have a great weekend!
/rob
_______________________________________________
tools-taskcluster mailing list
tools-taskcluster@lists.mozilla.org
https://lists.mozilla.org/listinfo/tools-taskcluster
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → rthijssen
Assignee | ||
Comment 1•7 years ago
|
||
this pr moves the instance type definitions into the worker type manifest.
instead of hard coding the instance type and block device mappings into the update-workertpe.sh ci script, we move those definitions into the json manifest which gives us support for multiple instance types within each worker type definition.
Attachment #8997861 -
Flags: review?(pmoore)
Reporter | ||
Comment 2•7 years ago
|
||
Comment on attachment 8997861 [details] [diff] [review]
https://github.com/mozilla-releng/OpenCloudConfig/pull/167
Review of attachment 8997861 [details] [diff] [review]:
-----------------------------------------------------------------
Totally awesome!!
Attachment #8997861 -
Attachment is patch: true
Attachment #8997861 -
Attachment mime type: text/x-github-pull-request → text/plain
Attachment #8997861 -
Flags: review?(pmoore) → review+
Assignee | ||
Comment 3•7 years ago
|
||
this is now resolved with merge: https://github.com/mozilla-releng/OpenCloudConfig/commit/331e1ff7bbfd2f309d7520be7833792af7db2eee
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•