Save hypervisor version as string rather than an integer
The hypervisor_version reported by vmwareapi, powervm, hyperv and the fake driver all use strings as their version number. Currently, the hypervisor_version is being saved as an integer in the compute_nodes table. It would be consistent and more convenient to store it as a string, which would avoid the need to convert it to int.
Related discussion can be found on:
https:/
Related mailing list discussion:
http://
Blueprint information
- Status:
- Not started
- Approver:
- Russell Bryant
- Priority:
- Undefined
- Drafter:
- Aditi Raveesh
- Direction:
- Needs approval
- Assignee:
- Aditi Raveesh
- Definition:
- Review
- Series goal:
- None
- Implementation:
- Not started
- Milestone target:
- None
- Started by
- Completed by
Related branches
Related bugs
Sprints
Whiteboard
The big concern with this is how comparisons(
Currently, in case of xen hypervisor, the hypervisor version is used as an int while doing a version comparison. In other places, the version is converted from a string to an int, and then compared. An easier approach would be to keep this consistent: save the version as a string in the db (rather than as an int), and use string comparison, such as the method is_compatible from versionutils module to make any kind of version comparisons. This would eliminate the need of multiple conversion util methods such as convert_
Suggestions? --aditirav
deferred from icehouse-3 to "next": http://
Removed from next, as next is now reserved for near misses from the last milestone --johnthetubaguy