RE: Inconsistent way to store host HV version in db

2013-01-17 Thread Koushik Das
Posted the review for this https://reviews.apache.org/r/8993/ > -Original Message- > From: Koushik Das [mailto:koushik@citrix.com] > Sent: Wednesday, January 16, 2013 11:15 AM > To: cloudstack-dev@incubator.apache.org > Subject: RE: Inconsistent way to store host H

RE: Inconsistent way to store host HV version in db

2013-01-15 Thread Koushik Das
Created CLOUDSTACK-988 > -Original Message- > From: Alex Huang [mailto:alex.hu...@citrix.com] > Sent: Wednesday, January 16, 2013 4:58 AM > To: cloudstack-dev@incubator.apache.org > Subject: RE: Inconsistent way to store host HV version in db > > Just because it&#

RE: Inconsistent way to store host HV version in db

2013-01-15 Thread Alex Huang
itrix.com] > Sent: Tuesday, January 15, 2013 1:40 AM > To: cloudstack-dev@incubator.apache.org > Subject: RE: Inconsistent way to store host HV version in db > > Yes it can be stored in host_details table but then given that all hosts in a > cluster have same version the data woul

RE: Inconsistent way to store host HV version in db

2013-01-15 Thread Koushik Das
some validation based on HV version. > -Original Message- > From: Devdeep Singh [mailto:devdeep.si...@citrix.com] > Sent: Tuesday, January 15, 2013 2:51 PM > To: cloudstack-dev@incubator.apache.org > Subject: RE: Inconsistent way to store host HV version in db > > Storin

RE: Inconsistent way to store host HV version in db

2013-01-15 Thread Devdeep Singh
version details? Regards, Devdeep > -Original Message- > From: Koushik Das [mailto:koushik@citrix.com] > Sent: Tuesday, January 15, 2013 2:23 PM > To: cloudstack-dev@incubator.apache.org > Subject: Inconsistent way to store host HV version in db > > The 'hos

Inconsistent way to store host HV version in db

2013-01-15 Thread Koushik Das
The 'host' table has a column 'hypervisor_version' but that doesn't get filled (tried with XS and Vmware). For Vmware the column 'version' is populated with HV version whereas in case of XS it has CS version appended with some timestamp. There is another table 'host_details'. For XS saw that ther