WaitForCAASServerOperationAction

Complexity

Total Complexity

4

Size/Duplication

Total Lines	16
Duplicated Lines	0 %

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getNode()	0	4	1
A	run()	0	8	3

from lib import actions
from time import sleep

__all__ = [
    'WaitForCAASServerOperationAction',
]


class WaitForCAASServerOperationAction(actions.BaseAction):

    def run(self, region, id):

This seems to re-define the built-in id.

        node = self.getNode(region, id)        

Trailing whitespace

        if node is not None:
            while(node.extra['status'].action == 'None'):

There is an unnecessary parenthesis after while.

                sleep(5)
                node = self.getNode(region, id)
        else:
            raise "VM with the name doesn't exist"
            

Trailing whitespace

Trailing whitespace

    def getNode(self, region, id):

This seems to re-define the built-in id.

        driver = self._get_compute_driver(region)        

Trailing whitespace

        node = driver.ex_get_node_by_id(id)        

Trailing whitespace

        return node