Github user swill commented on the pull request:

    https://github.com/apache/cloudstack/pull/1417#issuecomment-215184791
  
    @nitt10prashant I have to admit, I am confused on this one.  I have the 
latest Marvin installed that is generated when building this PR.
    
    ```
    Installing Marvin
    DEPRECATION: --allow-external has been deprecated and will be removed in 
the future. Due to changes in the repository protocol, it no longer has any 
effect.
    Processing ./tools/marvin/dist/Marvin-4.9.0-SNAPSHOT.tar.gz
    Requirement already up-to-date: mysql-connector-python>=1.1.6 in 
/usr/lib/python2.7/site-packages (from Marvin===4.9.0-SNAPSHOT)
    Collecting requests>=2.2.1 (from Marvin===4.9.0-SNAPSHOT)
      Downloading requests-2.9.1-py2.py3-none-any.whl (501kB)
        100% 
|████████████████████████████████|
 501kB 1.4MB/s 
    Collecting paramiko>=1.13.0 (from Marvin===4.9.0-SNAPSHOT)
      Downloading paramiko-1.16.0-py2.py3-none-any.whl (169kB)
        100% 
|████████████████████████████████|
 174kB 3.1MB/s 
    Requirement already up-to-date: nose>=1.3.3 in 
/usr/lib/python2.7/site-packages/nose-1.3.7-py2.7.egg (from 
Marvin===4.9.0-SNAPSHOT)
    Collecting ddt>=0.4.0 (from Marvin===4.9.0-SNAPSHOT)
      Downloading ddt-1.0.1-py2.py3-none-any.whl
    Collecting pyvmomi>=5.5.0 (from Marvin===4.9.0-SNAPSHOT)
      Downloading pyvmomi-6.0.0.2016.4.tar.gz (216kB)
        100% 
|████████████████████████████████|
 225kB 2.7MB/s 
    Collecting netaddr>=0.7.14 (from Marvin===4.9.0-SNAPSHOT)
      Downloading netaddr-0.7.18-py2.py3-none-any.whl (1.5MB)
        100% 
|████████████████████████████████|
 1.5MB 614kB/s 
    Collecting ecdsa>=0.11 (from paramiko>=1.13.0->Marvin===4.9.0-SNAPSHOT)
      Downloading ecdsa-0.13-py2.py3-none-any.whl (86kB)
        100% 
|████████████████████████████████|
 92kB 2.5MB/s 
    Requirement already up-to-date: pycrypto!=2.4,>=2.1 in 
/usr/lib64/python2.7/site-packages (from 
paramiko>=1.13.0->Marvin===4.9.0-SNAPSHOT)
    Collecting six>=1.7.3 (from pyvmomi>=5.5.0->Marvin===4.9.0-SNAPSHOT)
      Downloading six-1.10.0-py2.py3-none-any.whl
    Building wheels for collected packages: Marvin, pyvmomi
      Running setup.py bdist_wheel for Marvin ... done
      Stored in directory: 
/root/.cache/pip/wheels/7f/5a/37/7f723796f0a05b8f4a5eb63ebe90f381663e638e7556f46916
      Running setup.py bdist_wheel for pyvmomi ... done
      Stored in directory: 
/root/.cache/pip/wheels/b8/4a/84/b57aa91d338fdb54c766ef9d3c4b8d7c7a9f6b0da53e7b112d
    Successfully built Marvin pyvmomi
    Installing collected packages: requests, ecdsa, paramiko, ddt, six, 
pyvmomi, netaddr, Marvin
      Found existing installation: requests 2.6.0
        DEPRECATION: Uninstalling a distutils installed project (requests) has 
been deprecated and will be removed in a future version. This is due to the 
fact that uninstalling a distutils project will only partially uninstall the 
project.
        Uninstalling requests-2.6.0:
          Successfully uninstalled requests-2.6.0
      Found existing installation: ecdsa 0.11
        DEPRECATION: Uninstalling a distutils installed project (ecdsa) has 
been deprecated and will be removed in a future version. This is due to the 
fact that uninstalling a distutils project will only partially uninstall the 
project.
        Uninstalling ecdsa-0.11:
          Successfully uninstalled ecdsa-0.11
      Found existing installation: paramiko 1.12.4
        Uninstalling paramiko-1.12.4:
          Successfully uninstalled paramiko-1.12.4
      Found existing installation: six 1.9.0
        Uninstalling six-1.9.0:
          Successfully uninstalled six-1.9.0
    Successfully installed Marvin-4.9.0-SNAPSHOT ddt-1.0.1 ecdsa-0.13 
netaddr-0.7.18 paramiko-1.16.0 pyvmomi-6.0.0.2016.4 requests-2.9.1 six-1.10.0
    ```
    
    I get the same result I got before, so I tried to install it manually to 
make sure everything was setup.
    
    ```
    # pip install --upgrade 
/data/git/cs1/cloudstack/tools/marvin/dist/Marvin-*.tar.gz
    Processing 
/data/git/cs1/cloudstack/tools/marvin/dist/Marvin-4.9.0-SNAPSHOT.tar.gz
    Requirement already up-to-date: mysql-connector-python>=1.1.6 in 
/usr/lib/python2.7/site-packages (from Marvin===4.9.0-SNAPSHOT)
    Requirement already up-to-date: requests>=2.2.1 in 
/usr/lib/python2.7/site-packages (from Marvin===4.9.0-SNAPSHOT)
    Requirement already up-to-date: paramiko>=1.13.0 in 
/usr/lib/python2.7/site-packages (from Marvin===4.9.0-SNAPSHOT)
    Requirement already up-to-date: nose>=1.3.3 in 
/usr/lib/python2.7/site-packages/nose-1.3.7-py2.7.egg (from 
Marvin===4.9.0-SNAPSHOT)
    Requirement already up-to-date: ddt>=0.4.0 in 
/usr/lib/python2.7/site-packages (from Marvin===4.9.0-SNAPSHOT)
    Requirement already up-to-date: pyvmomi>=5.5.0 in 
/usr/lib/python2.7/site-packages (from Marvin===4.9.0-SNAPSHOT)
    Requirement already up-to-date: netaddr>=0.7.14 in 
/usr/lib/python2.7/site-packages (from Marvin===4.9.0-SNAPSHOT)
    Requirement already up-to-date: ecdsa>=0.11 in 
/usr/lib/python2.7/site-packages (from 
paramiko>=1.13.0->Marvin===4.9.0-SNAPSHOT)
    Requirement already up-to-date: pycrypto!=2.4,>=2.1 in 
/usr/lib64/python2.7/site-packages (from 
paramiko>=1.13.0->Marvin===4.9.0-SNAPSHOT)
    Requirement already up-to-date: six>=1.7.3 in 
/usr/lib/python2.7/site-packages (from pyvmomi>=5.5.0->Marvin===4.9.0-SNAPSHOT)
    Building wheels for collected packages: Marvin
      Running setup.py bdist_wheel for Marvin ... done
      Stored in directory: 
/root/.cache/pip/wheels/7f/5a/37/7f723796f0a05b8f4a5eb63ebe90f381663e638e7556f46916
    Successfully built Marvin
    Installing collected packages: Marvin
      Found existing installation: Marvin 4.9.0-SNAPSHOT
        Uninstalling Marvin-4.9.0-SNAPSHOT:
          Successfully uninstalled Marvin-4.9.0-SNAPSHOT
    Successfully installed Marvin-4.9.0-SNAPSHOT
    ```
    
    I ran it again with the same result.  
    
    For the record, I don't see this command being added in this PR, just the 
tests to test it.  Were there other files that were supposed to be added in 
this PR that didn't get in?


---
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