-
Notifications
You must be signed in to change notification settings - Fork 727
Conversation
svenwltr
commented
Aug 13, 2017
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😍
README.md
Outdated
|
||
## Caution! | ||
|
||
Be aware that *aws-nuke* is a very descructive Tool, hence you have to be very |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tool
README.md
Outdated
|
||
To reduce the blast radius of accidents, there are some safety precautions: | ||
|
||
1. By default *aws-nuke* only lists all nukable resources. You need to add |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nukeable
README.md
Outdated
you want to nuke is part of this blacklist, *aws-nuke* will abort. It is | ||
recommended, that you add every production account to this blacklist. | ||
6. To ensure you just ignore the blacklisting feature, the blacklist must | ||
container at least one Account ID. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
contain
README.md
Outdated
and add it to a central repository. This way the account blacklist is way | ||
easier to manage and keep up to date. | ||
|
||
Feel free to create an issue, if you have any ideas to improve the safty |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
safety
I added a usage part. Is that maybe too much? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i like it very much
d5a5277
to
92605d2
Compare
I now added some more sections. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice!
how would you go about releasing this? new repo or just set this one to public?
Just making this one public. I don't see any benefit in a new repo. |