cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From wilderrodrigues <...@git.apache.org>
Subject [GitHub] cloudstack pull request: CLOUDSTACK-8957 - VR password server feat...
Date Wed, 28 Oct 2015 16:15:33 GMT
GitHub user wilderrodrigues opened a pull request:

    https://github.com/apache/cloudstack/pull/998

    CLOUDSTACK-8957 - VR password server feature is broken

    This PR implements the update of the password as it use to be done with the bash file,
but using the Python code for it.
    
    A new integration test was added in order to cover the fix. The test can be found at component/test_password_server.py
and does the following:
    
    * Creates an Account
    * Creates a Service Offering
    * Creates a Network Offering
    * Creates a Network
    * Creates two Virtual Machines
    * Creates two FW rules
    * Creates two PF rules
    * SSH into each VM and checks the default routes (by pinging 8.8.8.8)
    * SSH to the host and executes a command in the router to check the password-[gateway]
file
      - The file is under /var/cache/cloud/passwords-[gateway]
    
    The [gateway] is replaced by the gateway from each virtual machine NIC.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/ekholabs/cloudstack fix/vr_passwd_server-CLOUDSTACK-8957

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cloudstack/pull/998.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #998
    
----
commit ea132cc4789e3c2161a0618ac02e7a68a8d67aed
Author: Wilder Rodrigues <wrodrigues@schubergphilis.com>
Date:   2015-10-27T12:31:31Z

    CLOUDSTACK-8957 - Implement password server in configure.py

commit 92ccd45db5dd0aa175d7702e2723af4d01528b8b
Author: Wilder Rodrigues <wrodrigues@schubergphilis.com>
Date:   2015-10-28T07:43:26Z

    CLOUDSTACK-8957 - Make some improvements in the way the process is checked
    
      - Fix import of CsProcess module

commit ac3e352125261fc34527a56028998b644371e2d7
Author: Wilder Rodrigues <wrodrigues@schubergphilis.com>
Date:   2015-10-28T15:31:04Z

    CLOUDSTACK-8957 - Add integration test that cheks if the password file is present and
not empty

----


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