On Fri, Mar 22, 2013 at 09:50:38AM -0700, Animesh Chaturvedi wrote: > > > Thanks > Animesh > > On Mar 22, 2013, at 9:27 AM, "Chip Childers" <chip.child...@sungard.com> > wrote: > > > On Thu, Mar 21, 2013 at 06:03:49PM -0700, Animesh Chaturvedi wrote: > >> 2. Add Cluster > >> - Baremetal show up in list of hypervisors. This is coming from > >> listHypervisors command > >> > >> How to fix: This comes from the file server/src/com/cloud/configuration/ > >> Config.java in property " hypervisor.list" and is pulled from Database > >> table "Configuration". > >> > >> How to fix. : removing the BareMetal from hypervisors.list property in > >> Config.java will fix it for fresh installs. This will fix the fresh > >> install but to fix it for upgrade we need to fix the upgrade sql data > >> files. I am not sure if we should fix this I am not clear on upgrade > >> concerns? Alex/Chip any comments. > > > > Help me understand this one. Is there data in a table somewhere for > > pre-4.1 released versions? > > Yes the Configuration database table has an attribute hypervisors.list and > the value for that attribute is comma separated list of supported > hypervisors, and includes 'BareMetal'. No Schema change is needed. Fix is to > remove 'BareMetal' from attribute value. > > I do not have enough history to make a call how important is to fix it. The > fix is trivial.
OK, so it was in that table as part of the previous baremetal "experimental" feature? In that case, can I also assume that there's no impact to the data being in the table?