cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From nvazquez <...@git.apache.org>
Subject [GitHub] cloudstack pull request: CLOUDSTACK-9252: Support configurable NFS...
Date Tue, 02 Feb 2016 03:32:13 GMT
Github user nvazquez commented on the pull request:

    https://github.com/apache/cloudstack/pull/1361#issuecomment-178346694
  
    @rafaelweingartner thanks for your reply!
    
    I'm familiar to Spring, I've worked with it, I know how it works but I wouldn't say I
know it in depth. If you don't mind I would find your detailed explanation really helpful
for this feature and future tasks.
    
    About bean injection in <code>TemplateServiceImpl</code>, I first tryied <code>@Inject</code>
annotation, but ACS couldn't start because it couldn't find a bean of type <code>ImageStoreDetailsUtil</code>.
I had the same issue with <code>@Autowired</code> annotation. The only solution
I could find was adding bean in xml context file.
    
    I agree with you about not extending <code>Manager</code> interface, I saw
it was always done like that and just followed the pattern. I will remove that from <code>ImageStoreDetailsUtil</code>
    
    I could start ACS but didn't fully test <code>Vmware*</code> classes. I would
try to get the context and then the bean from it as you suggested.
    
    Thanks a lot for your help!


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message