-
Notifications
You must be signed in to change notification settings - Fork 325
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
add ec2 keypairs ingestion #124
Conversation
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.
Second pass - some minor changes. Looking good!
Just an update: we currently have this running in our internal testing setup and will followup when our runs finish |
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.
Test run is looking good so far. I caught one more minor thing while we wait for it to finish.
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.
Last change!
Testing worked great on our end.
* use ARN as Id for EC2KeyPair * add load_ec2_key_pairs test * remove LOGIN_KEY relationship in cleanup jobs * change LOGIN_KEY to SSH_LOGIN_TO * delete proper relationships in cleanup * remove commented dead code * update schema * node merge ARN on both keypair.id and keypair.arn * create index on :EC2KeyPair(id) * remove create index on :EC2KeyPair(arn)
Thank you again for your contribution! 😁 |
My pleasure :) |
I closed #120, addressed comments from @sachafaust and @achantavy in this PR.
add ec2 keypairs ingestion
use ARN as Id for EC2KeyPair
Ingest EC2 Key Pairs and associate them with EC2 Instances.
Fixes issue #121.