Github user rhtyd commented on the pull request:

    https://github.com/apache/cloudstack/pull/1502#issuecomment-214858085
  
    @swill @pdion891 thanks, I don't know of any format or wiki/document where 
we keep this thing. Each feature/marvin-test may require test specific 
environment so I'm not sure if we can have a general solution.
    
    In general, for writing marvin tests new dependencies maybe added to marvin 
itself (like for this PR, `ipmisim` dependency is added to marvin, so anytime 
you install it you automatically get the dependency necessary by your Python 
based test). Along with this, you may need any external dependencies added to 
packaging requirements/dependency (for example, `ipmitool` added to debian 
control, while for rpms `ipmitool` was already in the list of dependencies for 
cloudstack-management package). For running tests, we should expect the test to 
work out of the box provided the dependencies are installed. For this PR, I've 
got `ipmisim` added to marvin's setup.py, and for Travis specific there is a 
apt-get install ipmitool. After this the oobm marvin test runs like any other 
test and internally is responsible for setting up (fake) ipmi server etc.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to