class NodeRoles(object):
    def __init__(self,
                 controller_names=None,
                 compute_names=None,
                 storage_names=None,
                 proxy_names=None,
                 keystone_names=None):
        self.controller_names = controller_names,
        self.compute_names = compute_names,
        self.storage_names = storage_names,
        self.proxy_names = proxy_names,
        self.keystone_names = keystone_names


class Nodes(object):
    def __init__(self, devops_environment, node_roles):
        self.controllers = []
        self.computes = []
        self.storages = []
        self.proxies = []
        self.keystones = []
        for node_name in node_roles.controller_names:
            print(node_name)
            self.controllers += devops_environment.node[node_name]
        for node_name in node_roles.compute_names:
            self.computes += devops_environment.node[node_name]
        for node_name in node_roles.storage_names:
            self.storages += devops_environment.node[node_name]
        for node_name in node_roles.proxy_names:
            self.proxies += devops_environment.node[node_name]
        for node_name in node_roles.keystone_names:
            self.keystones += devops_environment.node[node_name]




class NodeRoles(object):
    def __init__(self,
                 controller_names=None,
                 compute_names=None,
                 storage_names=None,
                 proxy_names=None,
                 keystone_names=None):
        self.controller_names = controller_names
        self.compute_names = compute_names
        self.storage_names = storage_names
        self.proxy_names = proxy_names
        self.keystone_names = keystone_names


class Nodes(object):
    def __init__(self, devops_environment, node_roles):
        self.controllers = []
        self.computes = []
        self.storages = []
        self.proxies = []
        self.keystones = []
        for node_name in node_roles.controller_names:
            print('asdf %s' % node_name)
            self.controllers += devops_environment.node[node_name]
        for node_name in node_roles.compute_names:
            self.computes += devops_environment.node[node_name]
        for node_name in node_roles.storage_names:
            self.storages += devops_environment.node[node_name]
        for node_name in node_roles.proxy_names:
            self.proxies += devops_environment.node[node_name]
        for node_name in node_roles.keystone_names:
            self.keystones += devops_environment.node[node_name]




Fix typo