----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8993/#review15487 -----------------------------------------------------------
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 Mehta On Jan. 17, 2013, 12:46 p.m., Koushik Das wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/8993/ > ----------------------------------------------------------- > > (Updated Jan. 17, 2013, 12:46 p.m.) > > > Review request for cloudstack, Nitin Mehta and Alex Huang. > > > 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. > > > This addresses bug 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 > > Diff: https://reviews.apache.org/r/8993/diff/ > > > Testing > ------- > > Verified that HV version is correctly updated in host table for XS and Vmware > > > Thanks, > > Koushik Das > >