cloudstack-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Chandan Purushothama (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (CLOUDSTACK-7388) [Automation] Fix the script "test_redundant_router.py" - Script is hardcoded to use "password" as Host's password
Date Thu, 21 Aug 2014 22:30:14 GMT

     [ https://issues.apache.org/jira/browse/CLOUDSTACK-7388?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Chandan Purushothama updated CLOUDSTACK-7388:
---------------------------------------------

    Description: 
Script is at: component/maint/.

Observe the below code in test_redundant_router.py script:

{code}
...
.
.
                        },
                        "host": {
                                 "username": "root",
                                 "password": "password",
                                 "publicport": 22,
                        },
                        "network": {
.....
.
.
.
    @attr(tags=["advanced", "advancedns", "ssh"])
    def test_redundantVR_internals(self):
        """Test redundant router internals
        """
.
.
.
        else:
            result = get_process_status(
                                backup_host.ipaddress,
                                self.services['host']["publicport"],
                                self.services['host']["username"],
                                self.services['host']["password"],
                                backup_router.linklocalip,
                                'ip addr show eth2',
                                )
        res = str(result)
.
.
.
{code}

The above code should be changed to use the host information from configuration file outside
this script.

  was:
Observe the below code in test_redundant_router.py script:

{code}
...
.
.
                        },
                        "host": {
                                 "username": "root",
                                 "password": "password",
                                 "publicport": 22,
                        },
                        "network": {
.....
.
.
.
    @attr(tags=["advanced", "advancedns", "ssh"])
    def test_redundantVR_internals(self):
        """Test redundant router internals
        """
.
.
.
        else:
            result = get_process_status(
                                backup_host.ipaddress,
                                self.services['host']["publicport"],
                                self.services['host']["username"],
                                self.services['host']["password"],
                                backup_router.linklocalip,
                                'ip addr show eth2',
                                )
        res = str(result)
.
.
.
{code}

The above code should be changed to use the host information from configuration file outside
this script.


> [Automation] Fix the script "test_redundant_router.py" - Script is hardcoded to use "password"
as Host's password
> -----------------------------------------------------------------------------------------------------------------
>
>                 Key: CLOUDSTACK-7388
>                 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-7388
>             Project: CloudStack
>          Issue Type: Bug
>      Security Level: Public(Anyone can view this level - this is the default.) 
>          Components: Automation, Test
>    Affects Versions: 4.5.0
>            Reporter: Chandan Purushothama
>            Priority: Critical
>             Fix For: 4.5.0
>
>
> Script is at: component/maint/.
> Observe the below code in test_redundant_router.py script:
> {code}
> ...
> .
> .
>                         },
>                         "host": {
>                                  "username": "root",
>                                  "password": "password",
>                                  "publicport": 22,
>                         },
>                         "network": {
> .....
> .
> .
> .
>     @attr(tags=["advanced", "advancedns", "ssh"])
>     def test_redundantVR_internals(self):
>         """Test redundant router internals
>         """
> .
> .
> .
>         else:
>             result = get_process_status(
>                                 backup_host.ipaddress,
>                                 self.services['host']["publicport"],
>                                 self.services['host']["username"],
>                                 self.services['host']["password"],
>                                 backup_router.linklocalip,
>                                 'ip addr show eth2',
>                                 )
>         res = str(result)
> .
> .
> .
> {code}
> The above code should be changed to use the host information from configuration file
outside this script.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Mime
View raw message