-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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 instance id to build properties #2762
Conversation
@dinatale2, thanks for your PR! By analyzing the history of the files in this pull request, we identified @stell-aelsabbahy, @davebx and @tardyp to be potential reviewers. |
If we do this for EC2, then I think we should do this for the rest of the latent workers. |
looks good, but please watch out the contributor checklist.
|
9f2c964
to
377e14e
Compare
Codecov Report@@ Coverage Diff @@
## master #2762 +/- ##
==========================================
+ Coverage 87.32% 87.32% +<.01%
==========================================
Files 305 305
Lines 32582 32583 +1
==========================================
+ Hits 28451 28452 +1
Misses 4131 4131
Continue to review full report at Codecov.
|
Add instance id to a workers properties. This will enable use of the AWS toolkit. Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
377e14e
to
b0da4c9
Compare
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 dont think there is an easy way to make this generic. so looks good
Contributor Checklist:
Note sure if this is a good idea or not. But, having the instance id is handy when attempting to run various ec2 related tools. Any thoughts?