Hi will,
You need to install latest marvin. Changes to it requires for users to install
it again. Using mvn, it just builds and packages, it wont install.
under marvin folder, run python setup.py install, or can use pip install
marvin..tar.gz
Thanks!
Santhosh
________________________________________
From: Will Stevens [nore...@reviews.apache.org] on behalf of Will Stevens
[wstev...@cloudops.com]
Sent: Friday, December 06, 2013 4:14 PM
To: Girish Shilamkar
Cc: Santhosh Edukulla; Will Stevens; cloudstack
Subject: Re: Review Request 15922: Enhanced few features under Marvin
> On Dec. 6, 2013, 6:35 a.m., Girish Shilamkar wrote:
> > Committed to 4.2, 4.3 and master.
I am unable to run the following command on the 4.3 branch now:
mvn -P developer -pl tools/devcloud -Ddeploysvr -X
I get one of the following error:
[DEBUG] Executing command line: python ../marvin/marvin/deployDataCenter.py -i
devcloud.cfg
Traceback (most recent call last):
File "../marvin/marvin/deployDataCenter.py", line 567, in <module>
from marvin.marvinLog import MarvinLog
ImportError: No module named marvinLog
or
[DEBUG] Executing command line: python ../marvin/marvin/deployDataCenter.py -i
devcloud.cfg
Traceback (most recent call last):
File "../marvin/marvin/deployDataCenter.py", line 19, in <module>
import configGenerator
File "/mnt/hgfs/palo_alto/cloudstack/tools/marvin/marvin/configGenerator.py",
line 22, in <module>
from marvin.codes import *
ImportError: No module named codes
I have rebuilt my system many times now trying to get this to work.
Partially related to this process. In order to use devcloud I had to make the
following changes to the root pom.xml
$ git diff pom.xml
diff --git a/pom.xml b/pom.xml
index 31946d8..a49927e 100644
--- a/pom.xml
+++ b/pom.xml
@@ -747,9 +747,7 @@
</properties>
<modules>
<module>developer</module>
- <!--
<module>tools</module>
--->
</modules>
</profile>
<profile>
@@ -759,7 +757,7 @@
</properties>
<modules>
<module>developer</module>
- <!--module>tools/devcloud</module -->
+ <module>tools/devcloud</module>
</modules>
</profile>
<profile>
Also, because my code is in a shared directory, I have to also make this change:
$ git diff tools/marvin/pom.xml
diff --git a/tools/marvin/pom.xml b/tools/marvin/pom.xml
index 0869248..f33fedf 100644
--- a/tools/marvin/pom.xml
+++ b/tools/marvin/pom.xml
@@ -73,7 +73,7 @@
<executable>python</executable>
<arguments>
<argument>setup.py</argument>
- <argument>sdist</argument>
+ <argument>install</argument>
</arguments>
</configuration>
</execution>
@@ -126,7 +126,7 @@
<executable>python</executable>
<arguments>
<argument>setup.py</argument>
- <argument>sdist</argument>
+ <argument>install</argument>
</arguments>
</configuration>
</execution>
Here are my steps for trying to rebuild my environment:
# completely blow away and rebuild my DB
$ /etc/init.d/mysqld stop
$ rm -rf /var/lib/mysql/
$ /etc/init.d/mysqld start
# clean install (I have to skip tests because 'Apache CloudStack Console Proxy
- RDP Client' fails tests)
$ mvn -P developer,systemvm clean install -Dmaven.test.skip=true
# deploy the db
$ mvn -P developer -pl developer,tools/devcloud -Ddeploydb
# start the server
$ chmod 755
/mnt/hgfs/palo_alto/cloudstack/client/target/cloud-client-ui-4.3.0-SNAPSHOT/WEB-INF/classes/scripts/vm/systemvm/injectkeys.sh
$ export MAVEN_OPTS="-Xmx1024m -XX:MaxPermSize=512m -Xdebug
-Xrunjdwp:transport=dt_socket,address=8787,server=y,suspend=n"
$ mvn -pl :cloud-client-ui jetty:run
# once the server is started I resync marvin
$ cd tools/marvin/dist/
$ curl -o Marvin-0.1.0.tar.gz
https://builds.apache.org/job/cloudstack-marvin/lastSuccessfulBuild/artifact/tools/marvin/dist/Marvin-0.1.0-1555.tar.gz
$ export MAVEN_OPTS=""
$ mvn -Pdeveloper,marvin.sync -Dendpoint=localhost -pl :cloud-marvin
# now I try to deploy the server
$ export MAVEN_OPTS=""
$ mvn -P developer -pl tools/devcloud -Ddeploysvr -X
# I have tried running 'python setup.py install' inside 'tools/marvin' as well
as many other combinations of steps.
I have spent a solid half day trying to troubleshoot this...
- Will
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15922/#review29875
-----------------------------------------------------------
On Nov. 30, 2013, 6:27 p.m., Santhosh Edukulla wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/15922/
> -----------------------------------------------------------
>
> (Updated Nov. 30, 2013, 6:27 p.m.)
>
>
> Review request for cloudstack and Girish Shilamkar.
>
>
> Repository: cloudstack-git
>
>
> Description
> -------
>
> Added few enhancements to marvin.
> Added new module for Logging Facility to marvin.
> Added new Init facility to marvin.
> Currently, there are multiple ways we are doing logging
> Removed few unwanted logging cases.
> Removed few command line switch options for logging.
> The new way of logging now provides consolidated logging
> under one single folder timestamped under the configured
> folder path.
> Removed parsing configuration from deploydata center
> Added parsing,start logging and deploy as part of init
> Added new error handling facility to catch unknown exception from
> test cases. Currently, lot of scripts are throwing unknown
> exceptions, add a handler to plugin to dump them to a file
>
> ToDO:
> Will do clean up in phase2 for this patch.
> Separate deployDatacenter from creating test client.
> Clean up configGenerator
>
>
> Diffs
> -----
>
> setup/dev/advanced.cfg 216314f
> tools/marvin/marvin/cloudstackConnection.py 23f81fb
> tools/marvin/marvin/codes.py f409c7c
> tools/marvin/marvin/configGenerator.py 631e40f
> tools/marvin/marvin/deployDataCenter.py 3f7eebb
> tools/marvin/marvin/marvinInit.py PRE-CREATION
> tools/marvin/marvin/marvinLog.py PRE-CREATION
> tools/marvin/marvin/marvinPlugin.py 0e52bab
>
> Diff: https://reviews.apache.org/r/15922/diff/
>
>
> Testing
> -------
>
> Tested locally. Ran smoke tests. As well, run deploydatacenter separately to
> check deployement is working fine.
>
>
> Thanks,
>
> Santhosh Edukulla
>
>
