-
Notifications
You must be signed in to change notification settings - Fork 276
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
claudia destroy ignores --profile #100
Comments
I am having the same problem. |
the most likely cause is this line claudia/src/util/destroy-role.js Line 2 in c3b837d
|
You might need consider destroy a lambda which is created with an existing role (using --role option). I got this error while I destroyed a lambda with an existing role.
|
@linshu that's my bad. The actual working workaround is |
👍 |
this is now on NPM as 2.8.0 |
I would assume that
claudia destroy --profile private
will cleanup all changes introduced by Claudia. This in my case includes removing Lambda function, all installed notification handlers, role and policies.Unfortunately it's failing, because the
--profile
part seems to be ignored on https://github.com/claudiajs/claudia/blob/master/src/commands/destroy.js#L30 level anddefault
profile is used. Which leads to situation where Lambda is properly removed, but roles and policies remain intact. I believe that there is also no code for notification handler/s removal.AWS_PROFILE=private claudia destroy
seems to be temporary workaround for that issue.Any project where different AWS profile than
default
is in use. I'm trying to use Claudia here: https://github.com/secretescapes/aws-lambda-image/tree/claudia-to-rule-them-all.Clone that branch and replace
private
profile (in thepackage.json
) with one that should be used. Be sure thatdefault
profile is invalid or to restrictive to allow Claudia to work properly.Then just run
npm run create
.npm destroy
The text was updated successfully, but these errors were encountered: