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
Equinix update #1548
Equinix update #1548
Conversation
Codecov Report
@@ Coverage Diff @@
## trunk #1548 +/- ##
==========================================
- Coverage 83.08% 83.08% -0.01%
==========================================
Files 394 394
Lines 84795 84808 +13
Branches 9004 9006 +2
==========================================
+ Hits 70456 70460 +4
- Misses 11274 11282 +8
- Partials 3065 3066 +1
Continue to review full report at Codecov.
|
d85509a
to
93cf485
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.
LGTM.
disk = None | ||
if data['specs'].get('drives', ''): | ||
disk = 0 | ||
for disks in data['specs']['drives']: |
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.
Eventually it would be nice to split this formatting functionality in separate functions to make (unit) testing easier, but not a blocker for now.
Various updates on EquinixMetal compute driver
Description
Handle exceptions where necessary, pass explicitly
disk
anddisk_size
instead of kwargs on create_node, addsupported_sizes
field on images andregions
field on sizes.Status
Checklist (tick everything that applies)