Skip to content

Commit

Permalink
fix update failure caused by recent AWS changes
Browse files Browse the repository at this point in the history
(Copied changes from claudiajs#230)
  • Loading branch information
Jim Ng committed Feb 28, 2022
1 parent ce63ec1 commit c709067
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 4 deletions.
6 changes: 5 additions & 1 deletion src/commands/update.js
Original file line number Diff line number Diff line change
Expand Up @@ -145,7 +145,11 @@ module.exports = function update(options, optionalLogger) {
},
() => logger.logStage('waiting for IAM role propagation'),
Promise
);
).then(result => {
logger.logStage('waiting for lambda resource allocation');
return waitUntilNotPending(lambda, lambdaConfig.name, awsDelay, awsRetries)
.then(() => result);
});
}
},
cleanup = function () {
Expand Down
6 changes: 3 additions & 3 deletions src/tasks/wait-until-not-pending.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
const retry = require('oh-no-i-insist');
module.exports = function waitUntilNotPending(lambda, functionName, timeout, retries) {
module.exports = async function waitUntilNotPending(lambda, functionName, timeout, retries) {
'use strict';
await new Promise(resolve => setTimeout(resolve, timeout));

return retry(
() => {
return lambda.getFunctionConfiguration({FunctionName: functionName}).promise()
Expand All @@ -20,5 +22,3 @@ module.exports = function waitUntilNotPending(lambda, functionName, timeout, ret
Promise
);
};


0 comments on commit c709067

Please sign in to comment.