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

Support creation of cross hostedzone Route53 alias records. #575

Open
wants to merge 41 commits into
base: master
Choose a base branch
from

Conversation

mampcs
Copy link

@mampcs mampcs commented May 22, 2014

Enhances the api to allow creation of alias records where the target
could be in a different hosted zone and also supports setting the target
health check flag on the alias record.
The different hosted zone is quite common if we are creating an alias to
an Amazon Load Balancer.
Also, without the health check flag being set, the AWS api will throw an
exception when creating an alias to an ELB.
Finally, when creating an alias record, the api was failing because an
empty ResourceRecord was being passed to the create request.

Enhances the api to allow creation of alias records where the target
could be in a different hosted zone and also supports setting the target
health check flag on the alias record.
The different hosted zone is quite common if we are creating an alias to
an Amazon Load Balancer.
Also, without the health check flag being set, the AWS api will throw an
exception when creating an alias to an ELB.
Finally, when creating an alias record, the api was failing because an
empty ResourceRecord was being passed to the create request.
@cloudbees-pull-request-builder

asgard-pull-requests #424 FAILURE
Looks like there's a problem with this pull request

Melvin Cardozo added 2 commits May 21, 2014 20:50
The hosted zone id of the alias target is different from that of the
hosted zone from which the record is being removed. Using the hosted
zone id of the zone from which the record is being deleted is not
correct.
@cloudbees-pull-request-builder

asgard-pull-requests #425 SUCCESS
This pull request looks good

claymccoy and others added 25 commits June 9, 2014 15:56
Apparently there is a limit of 400 ELBs per request, and we need to start using the "optional" marker to get the rest.
This allows Asgard for certain AWS accounts to use properties file formatted user data for all windows AMI deployments, without changing the behavior of Asgard managing other AWS accounts.
…lication from a single source account.

The old code assumed that there could only be a single target account for image replication.
@cloudbees-pull-request-builder

asgard-pull-requests #452 SUCCESS
This pull request looks good

claymccoy and others added 13 commits June 24, 2014 20:47
because last() throws an exception for an empty Collection
In order to support automatically taking any block device mapping of an AMI into an ASG they can be taken from the image itself. Any default EBS volumes on the AMI will be mounted as usual. 

This solves the issue we've been having with asgard not mounting the ephemeral disks in the AMI
Gives presedence for those in the configuration
This reverts commit ff6e5e1, reversing
changes made to d527213.
…us ASG.

We do not want to disable the whole ASG.
@cloudbees-pull-request-builder

asgard-pull-requests #459 SUCCESS
This pull request looks good

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants