I need to read HV version as part of CLOUDSTACK-672. It not just for consistency, the HV version was not getting updated in the correct place.
From: Nitin Mehta [mailto:nore...@reviews.apache.org] On Behalf Of Nitin Mehta Sent: Friday, January 18, 2013 5:21 PM To: Alex Huang; Nitin Mehta Cc: cloudstack; Koushik Das Subject: Re: Review Request: CLOUDSTACK-988: HV version must be updated in hypervisor_version column of host table This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8993/ Is this just for the sake of consistency ? Was something affected because of this ? If yes, we need to take care of migration as well. - Nitin On January 17th, 2013, 12:46 p.m., Koushik Das wrote: Review request for cloudstack, Nitin Mehta and Alex Huang. By Koushik Das. Updated Jan. 17, 2013, 12:46 p.m. Description HV version is stored in 'hypervisor_version' column in host table wherever present. Also correctly populating 'version' column in host table with resource file version. Testing Verified that HV version is correctly updated in host table for XS and Vmware Bugs: CLOUDSTACK-988 Diffs * core/src/com/cloud/network/resource/TrafficSentinelResource.java (a13e080) * plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java (a444cfe) * plugins/network-elements/f5/src/com/cloud/network/resource/F5BigIpResource.java (c9c3711) * plugins/network-elements/juniper-srx/src/com/cloud/network/resource/JuniperSrxResource.java (f823ab8) * plugins/network-elements/netscaler/src/com/cloud/network/resource/NetscalerResource.java (afb0135) * plugins/network-elements/nicira-nvp/src/com/cloud/network/resource/NiciraNvpResource.java (12ed72a) * server/src/com/cloud/baremetal/ExternalDhcpResourceBase.java (cbfcfd1) * server/src/com/cloud/baremetal/PxeServerResourceBase.java (c307ccf) * server/src/com/cloud/resource/DummyHostServerResource.java (5144a91) * server/src/com/cloud/resource/ResourceManagerImpl.java (ffc09f0) View Diff<https://reviews.apache.org/r/8993/diff/>
