incubator-cloudstack-dev mailing list archives

Thanks for filing this bug.
I am working on it recently, because new baremetal which is as a single plugin haven't been
checked in when javelin refactoring. So there are some old code which is not working still
stay in source, I am cleaning up/testing in my local source,
will close these bug when fixes are checked in
From: rohityadav89@gmail.com [mailto:rohityadav89@gmail.com] On Behalf Of Rohit Yadav
Sent: Wednesday, February 13, 2013 2:01 AM
To: Frank Zhang
Cc: cloudstack
Subject: Duplicate BareMetalTemplateAdapter classes
Hi Frank,
Can you check why we have two BareMetalTemplateAdapter;
./plugins/hypervisors/baremetal/src/com/cloud/baremetal/manager/BareMetalTemplateAdapter.java
(Found no usage across codebase, removable?)
./server/src/com/cloud/baremetal/BareMetalTemplateAdapter.java (only this one is used, as
spring would instantiate only this one with @Component annotation)
And remove one which is redundant code? Found this while fixing CLOUDSTACK-1237.
Regards.
On Wed, Feb 13, 2013 at 3:26 PM, Rohit Yadav <bhaisaab@apache.org<mailto:bhaisaab@apache.org>>
wrote:
This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9420/
This fix would have worked for Hypervisor but would have failed for baremetal... if we fix
like this, there may be other template adapters whose class (simple) names. So, it was better
to impl getName() for all implementing template adapters.
Hongfu thank you for your patch, I was in middle of working and testing the patch and went
ahead to commit the fix.
- Rohit
On February 13th, 2013, 5:07 a.m., Hongtu Zang wrote:
Review request for cloudstack, mice xia, anthony xu, and SrikanteswaraRao Talluri.
By Hongtu Zang.
Updated Feb. 13, 2013, 5:07 a.m.
Description
In TemplateManagerImpl.java, function getAdapter(), TemplateAdapterType.Hypervisor.getName()
returns "HyervisorAdapter", while it should returns "HyervisorTemplateAdapter". So, in AdapterBase.java
function getAdapterByName() returns null.
Testing
register a template and start a vm.
success.
Bugs: CLOUDSTACK-1237, CLOUDSTACK-1240
Diffs
* server/src/com/cloud/template/TemplateAdapter.java (19cfef0)
View Diff<https://reviews.apache.org/r/9420/diff/>