Repository navigation
Add ability to delete using ip address - #21
Merged
Merged
Conversation
alexellis
reviewed
Jan 5, 2020
alexellis
reviewed
Jan 5, 2020
alexellis
reviewed
Jan 5, 2020
alexellis
reviewed
Jan 5, 2020
Member
|
Exceptional work 👏 thank you for testing each provider too. I have a few review comments/questions, but this overall looks great. |
alexellis
requested changes
Jan 5, 2020
alexellis
left a comment
Member
There was a problem hiding this comment.
A few minor changes requested and questions
Member
|
I just merged in #16 which added the EC2 functionality. Would you be able to rebase this and add the delete by IP for EC2 please? |
alexellis
reviewed
Jan 5, 2020
Member
|
When rebasing / adding EC2, please could you squash the commits into one so that history doesn't contain the revisions, but just the final version? Thanks a lot for getting this feature into inletsctl 💯 |
Contributor
Author
Sure, will do! |
adamjohnson01
force-pushed
the
delete-by-ip
branch
2 times, most recently
from
January 6, 2020 12:41
10fba9f to
07637d2
Compare
adamjohnson01
force-pushed
the
delete-by-ip
branch
2 times, most recently
from
January 22, 2020 10:40
61a85c0 to
d0b5150
Compare
Added lookup functions to allow the use of the ip address instead of the id when deleting exit nodes Signed-off-by: Adam Johnson <adamjohnson01@gmail.com>
adamjohnson01
force-pushed
the
delete-by-ip
branch
from
January 22, 2020 10:55
d0b5150 to
585d0f3
Compare
Contributor
Author
|
@alexellis , rebased and added ec2 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
I added lookup functions for each provisioner that list nodes based on tag/label where the provisioner supports it and then uses the ip to get the id of the node which is used to then delete it.
Related to #2
How Has This Been Tested?
I created an exit node and then deleted it using every provider.
Packet
Create
Delete
Run delete again
DigitalOcean
Create
Delete
Run delete again
GCE
Create
Delete
Run delete again
Civo
Create
Delete
Run delete again
Scaleway
Create
Delete
Run delete again
AWS EC2
Create
Delete
Run delete again
How are existing users impacted? What migration steps/scripts do we need?
This functionality will only work on exit nodes created with a version that contains this pull request as it tags or labels the exit nodes.
Checklist:
I have:
git commit -s