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]




@@ -788,20 +788,24 @@
 trollers
- += 
+.append(
 devops_e
@@ -822,32 +822,33 @@
 .node%5Bnode_name%5D
+)
 %0A        for nod
@@ -908,20 +908,24 @@
 computes
- += 
+.append(
 devops_e
@@ -942,32 +942,33 @@
 .node%5Bnode_name%5D
+)
 %0A        for nod
@@ -1028,20 +1028,24 @@
 storages
- += 
+.append(
 devops_e
@@ -1062,32 +1062,33 @@
 .node%5Bnode_name%5D
+)
 %0A        for nod
@@ -1145,20 +1145,24 @@
 .proxies
- += 
+.append(
 devops_e
@@ -1187,16 +1187,17 @@
 de_name%5D
+)
 %0A       
@@ -1271,12 +1271,16 @@
 ones
- += 
+.append(
 devo
@@ -1309,12 +1309,13 @@
 de_name%5D
+)
 %0A%0A%0A%0A

Fix typo