Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update AMI ID for Linux slaves #183

Merged
merged 1 commit into from Dec 30, 2015
Merged

Update AMI ID for Linux slaves #183

merged 1 commit into from Dec 30, 2015

Conversation

@edunham
Copy link
Contributor

edunham commented Dec 30, 2015

Workflow for switching from Daala AWS account to Servo AWS account:

  • Create Buildbot user in Servo account, download its creds, and set up its security groups to match Buildbot user in Daala account
  • Copy the buildslave AMI (originally private to the Daala account) to the Servo account (though this changes its ID)
  • PR to change all instances of Daala account's copy of the buildslave AMI ID to the Servo account's copy (that's this)
  • Land PR, update /srv/pillar/buildbot.sls aws-access-key and aws-secret-key to correspond to Servo AWS account's Buildbot user (carefully not running Salt in between these steps)
  • Run Salt on servo-master to deploy changes
  • Fix whatever all this broke
  • Update the wiki

In other words, I'll merge this one by hand at the right time after r+ :)

Review on Reviewable

@larsbergstrom
Copy link
Contributor

larsbergstrom commented Dec 30, 2015

r+

The AMI of the old ID was private to the Daala AWS account. To copy it to one
owned by the Servo account, I followed the instructions at
http://docs.aws.amazon.com/AWSEC2/latest/UserGuide/CopyingAMIs.html : gave the
Servo account permissions on the Daala account's AMI, created a copy in the
Servo account, noted the new ID.
@edunham edunham force-pushed the edunham:accountswitch branch from ad34884 to e85612e Dec 30, 2015
@edunham
Copy link
Contributor Author

edunham commented Dec 30, 2015

Old ID was for an instance in US-West-1. Everything else is in US-West-2. Changed ID to a copy in us-west-2 and will figure out why we're in this region later.

@larsbergstrom
Copy link
Contributor

larsbergstrom commented Dec 30, 2015

Sounds good to me :-)

edunham added a commit that referenced this pull request Dec 30, 2015
Update AMI ID for Linux slaves
@edunham edunham merged commit a72201a into servo:master Dec 30, 2015
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.