Skip to content

Commit

Permalink
Added a note as to why we do this
Browse files Browse the repository at this point in the history
  • Loading branch information
bkrodgers committed Apr 12, 2016
1 parent eccf23e commit f83ada2
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions assets/custom-types/remote-route53/remote_route53.js
Expand Up @@ -150,6 +150,11 @@ var changeResourceRecordSetsHelper = function(route53, params, callback, iterati
if (!iteration) {
iteration = 0;
}
/* If the CF template has multiple records in a single zone, it may try to fire them all off at once.
* For whatever reason, route 53 can't handle simultaneous updates to a given zone.
* This will catch the PriorRequestNotComplete error that is thrown in that situation and retry.
* All other error handling is delegated back to the callback.
*/
route53.changeResourceRecordSets(params, function (err, data) {
if (err && err.code == "PriorRequestNotComplete") {
if (iteration < iterationLimit) {
Expand Down

0 comments on commit f83ada2

Please sign in to comment.