make introspection.RootDiskType int64 rather than int (#1634)#1988
make introspection.RootDiskType int64 rather than int (#1634)#1988jtopjian merged 1 commit intogophercloud:masterfrom
Conversation
|
Build failed.
|
go.mod
Outdated
| gopkg.in/yaml.v2 v2.2.7 | ||
| ) | ||
|
|
||
| go 1.13 |
There was a problem hiding this comment.
Can this be removed? It's causing the Travis failure. I'll clean this up in another PR.
There was a problem hiding this comment.
Oh yes, I meant to remove that before pushing, cleaned up now.
|
@mwhudson Thank you for looking into this - it's really appreciated. From your comments in #1634:
Breaking changes are OK as long as they are justified.
Yes, from the error message given in #1634, I agree. I'd also be inclined to change other |
Fixes tests (and functionality I suspect!) on 32-bit systems.
No worries, thanks for the quick review!
Fair enough.
All of them? I can do that easily enough, of course. I did have a quick scan and didn't think any of the others would be problematic but it might be best to be consistent. |
Sorry - I may have been too broad in my last message 🙂 I should have said that we should look at changing them where it makes sense, and only in this introspection package. For example, the Memory fields might work better as an int64. I know it's a long shot to have that much memory, but who knows 2-3 years from now. Things like that. |
|
Build succeeded.
|
|
Well OK. Want to merge this, or fix it yourself how ever you like? :) |
Fixes tests (and functionality I suspect!) on 32-bit systems.