X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=csit%2Flibraries%2FSxp.py;h=0c116134acaedf357ce882ad962ea46f061fdca3;hb=827fea54d7459ea79f2acdd321c091f3a03cfb5a;hp=59a8339d6519c0f48766584079261728bc1f7deb;hpb=ff3688d31959c081dc25fd7a0db4176e50646072;p=integration%2Ftest.git diff --git a/csit/libraries/Sxp.py b/csit/libraries/Sxp.py index 59a8339d65..0c116134ac 100644 --- a/csit/libraries/Sxp.py +++ b/csit/libraries/Sxp.py @@ -12,12 +12,12 @@ def get_active_controller_from_json(resp, service): :type service: str :return: Index of controller """ - entities = json.loads(resp)['entity-owners']['entity-type'] + entities = json.loads(resp)["entity-owners"]["entity-type"] for entity in entities: - if entity['type'] == "org.opendaylight.mdsal.ServiceEntityType": - for instance in entity['entity']: - if service in instance['id']: - return int(instance['owner'][-1:]) + if entity["type"] == "org.opendaylight.mdsal.ServiceEntityType": + for instance in entity["entity"]: + if service in instance["id"]: + return int(instance["owner"][-1:]) return 0 @@ -47,16 +47,16 @@ def get_average_of_items(items): def get_opposing_mode(mode): """Generate string representing opposing SXP peer mode - :param mode: SXP peer mode - :type mode: str - :returns: String with opposing SXP peer mode. + :param mode: SXP peer mode + :type mode: str + :returns: String with opposing SXP peer mode. - """ - if 'speaker' == mode: - return 'listener' - elif 'listener' == mode: - return 'speaker' - return 'both' + """ + if "speaker" == mode: + return "listener" + elif "listener" == mode: + return "speaker" + return "both" def get_ip_from_number(n, base=2130706432): @@ -105,7 +105,9 @@ def lower_version(ver1, ver2): return ver2 -def get_filter_entry(seq, entry_type, sgt="", esgt="", acl="", eacl="", pl="", epl="", ps=""): +def get_filter_entry( + seq, entry_type, sgt="", esgt="", acl="", eacl="", pl="", epl="", ps="" +): """Generate xml containing FilterEntry data :param seq: Sequence of entry @@ -132,24 +134,24 @@ def get_filter_entry(seq, entry_type, sgt="", esgt="", acl="", eacl="", pl="", e entries = "" # Generate XML request containing combination of Matches of different types if sgt: - args = sgt.split(',') + args = sgt.split(",") entries += add_sgt_matches_xml(args) elif esgt: - args = esgt.split(',') + args = esgt.split(",") entries += add_sgt_range_xml(args[0], args[1]) if pl: entries += add_pl_entry_xml(pl) elif epl: - args = epl.split(',') + args = epl.split(",") entries += add_epl_entry_xml(args[0], args[1], args[2]) if acl: - args = acl.split(',') + args = acl.split(",") entries += add_acl_entry_xml(args[0], args[1]) elif eacl: - args = eacl.split(',') + args = eacl.split(",") entries += add_eacl_entry_xml(args[0], args[1], args[2], args[3]) if ps: - args = ps.split(',') + args = ps.split(",") entries += add_ps_entry_xml(args[0], args[1]) # Wrap entries in ACL/PrefixList according to specified values if pl or epl: @@ -167,13 +169,15 @@ def add_peers(*args): :returns: String containing xml data for request """ - templ = Template(''' + templ = Template( + """ $ip - ''') + """ + ) peers = "" for count, value in enumerate(args): - peers += templ.substitute({'ip': value}) + peers += templ.substitute({"ip": value}) return peers @@ -185,13 +189,15 @@ def add_domains(*args): :returns: String containing xml data for request """ - templ = Template(''' + templ = Template( + """ $name - ''') + """ + ) peers = "" for count, value in enumerate(args): - peers += templ.substitute({'name': value}) + peers += templ.substitute({"name": value}) return peers @@ -203,11 +209,13 @@ def add_sgt_matches_xml(sgt_entries): :returns: String containing xml data for request """ - templ = Template(''' - $sgt''') + templ = Template( + """ + $sgt""" + ) matches = "" for sgt in sgt_entries: - matches += templ.substitute({'sgt': sgt}) + matches += templ.substitute({"sgt": sgt}) return matches @@ -221,10 +229,12 @@ def add_sgt_range_xml(start, end): :returns: String containing xml data for request """ - templ = Template(''' + templ = Template( + """ $start - $end''') - match = templ.substitute({'start': start, 'end': end}) + $end""" + ) + match = templ.substitute({"start": start, "end": end}) return match @@ -240,13 +250,16 @@ def add_acl_entry_default_xml(seq, entry_type, acl_entries): :returns: String containing xml data for request """ - templ = Template(''' + templ = Template( + """ $entry_type $seq$acl_entries - ''') + """ + ) matches = templ.substitute( - {'seq': seq, 'entry_type': entry_type, 'acl_entries': acl_entries}) + {"seq": seq, "entry_type": entry_type, "acl_entries": acl_entries} + ) return matches @@ -260,12 +273,14 @@ def add_acl_entry_xml(ip, mask): :returns: String containing xml data for request """ - templ = Template(''' + templ = Template( + """ $ip $mask - ''') - return templ.substitute({'ip': ip, 'mask': mask}) + """ + ) + return templ.substitute({"ip": ip, "mask": mask}) def add_eacl_entry_xml(ip, mask, amask, wmask): @@ -282,7 +297,8 @@ def add_eacl_entry_xml(ip, mask, amask, wmask): :returns: String containing xml data for request """ - templ = Template(''' + templ = Template( + """ $ip $mask @@ -290,8 +306,9 @@ def add_eacl_entry_xml(ip, mask, amask, wmask): $amask $wmask - ''') - return templ.substitute({'ip': ip, 'mask': mask, 'amask': amask, 'wmask': wmask}) + """ + ) + return templ.substitute({"ip": ip, "mask": mask, "amask": amask, "wmask": wmask}) def add_ps_entry_default_xml(seq, entry_type, ps_entries): @@ -306,12 +323,16 @@ def add_ps_entry_default_xml(seq, entry_type, ps_entries): :returns: String containing xml data for request """ - templ = Template(''' + templ = Template( + """ $entry_type $seq$ps_entries - ''') - return templ.substitute({'seq': seq, 'entry_type': entry_type, 'ps_entries': ps_entries}) + """ + ) + return templ.substitute( + {"seq": seq, "entry_type": entry_type, "ps_entries": ps_entries} + ) def add_pl_entry_default_xml(seq, entry_type, pl_entries): @@ -326,12 +347,16 @@ def add_pl_entry_default_xml(seq, entry_type, pl_entries): :returns: String containing xml data for request """ - templ = Template(''' + templ = Template( + """ $entry_type $seq$pl_entries - ''') - return templ.substitute({'seq': seq, 'entry_type': entry_type, 'pl_entries': pl_entries}) + """ + ) + return templ.substitute( + {"seq": seq, "entry_type": entry_type, "pl_entries": pl_entries} + ) def add_pl_entry_xml(prefix): @@ -342,11 +367,13 @@ def add_pl_entry_xml(prefix): :returns: String containing xml data for request """ - templ = Template(''' + templ = Template( + """ $prefix - ''') - return templ.substitute({'prefix': prefix}) + """ + ) + return templ.substitute({"prefix": prefix}) def add_epl_entry_xml(prefix, op, mask): @@ -361,15 +388,17 @@ def add_epl_entry_xml(prefix, op, mask): :returns: String containing xml data for request """ - templ = Template(''' + templ = Template( + """ $prefix $op $mask - ''') - return templ.substitute({'prefix': prefix, 'mask': mask, 'op': op}) + """ + ) + return templ.substitute({"prefix": prefix, "mask": mask, "op": op}) def add_ps_entry_xml(op, length): @@ -382,11 +411,13 @@ def add_ps_entry_xml(op, length): :returns: String containing xml data for request """ - templ = Template(''' + templ = Template( + """ $length $op - ''') - return templ.substitute({'length': length, 'op': op}) + """ + ) + return templ.substitute({"length": length, "op": op}) def parse_peer_groups(groups_json): @@ -398,7 +429,7 @@ def parse_peer_groups(groups_json): """ data = json.loads(groups_json) - groups = data['output'] + groups = data["output"] output = [] for group in groups.values(): output += group @@ -414,11 +445,13 @@ def parse_connections(connections_json): """ data = json.loads(connections_json) - connections = data['output']['connections'] - output = [] - for connection in connections.values(): - output += connection - return output + output = data["output"] + result = [] + if output: + connections = output["connections"] + for connection in connections.values(): + result += connection + return result def find_connection(connections_json, version, mode, ip, port, state): @@ -440,11 +473,15 @@ def find_connection(connections_json, version, mode, ip, port, state): """ for connection in parse_connections(connections_json): - if (connection['peer-address'] == ip and connection['tcp-port'] == int(port) and ( - mode.strip() == 'any' or connection['mode'] == mode) and connection['version'] == version): - if state == 'none': + if ( + connection["peer-address"] == ip + and connection["tcp-port"] == int(port) + and (mode.strip() == "any" or connection["mode"] == mode) + and connection["version"] == version + ): + if state == "none": return True - elif connection['state'] == state: + elif connection["state"] == state: return True return False @@ -459,7 +496,7 @@ def parse_bindings(bindings_json): """ data = json.loads(bindings_json) output = [] - for bindings_json in data['output'].values(): + for bindings_json in data["output"].values(): for binding in bindings_json: output.append(binding) return output @@ -478,8 +515,8 @@ def find_binding(bindings, sgt, prefix): """ for binding in parse_bindings(bindings): - if binding['sgt'] == int(sgt): - for ip_prefix in binding['ip-prefix']: + if binding["sgt"] == int(sgt): + for ip_prefix in binding["ip-prefix"]: if ip_prefix == prefix: return True return False @@ -496,12 +533,12 @@ def parse_prefix_groups(prefix_groups_json, source_): """ data = json.loads(prefix_groups_json) - bindings = data['sxp-node:master-database'] + bindings = data["sxp-node:master-database"] output = [] for binding in bindings.values(): for binding_source in binding: - if source_ == "any" or binding_source['binding-source'] == source_: - for prefix_group in binding_source['prefix-group']: + if source_ == "any" or binding_source["binding-source"] == source_: + for prefix_group in binding_source["prefix-group"]: output.append(prefix_group) return output @@ -524,14 +561,24 @@ def find_binding_legacy(prefix_groups_json, sgt, prefix, source_, action): """ found = False for prefixgroup in parse_prefix_groups(prefix_groups_json, source_): - if prefixgroup['sgt'] == int(sgt): - for binding in prefixgroup['binding']: - if binding['ip-prefix'] == prefix and binding['action'] == action: + if prefixgroup["sgt"] == int(sgt): + for binding in prefixgroup["binding"]: + if binding["ip-prefix"] == prefix and binding["action"] == action: found = True return found -def add_connection_xml(version, mode, ip, port, node, password_, domain_name, bindings_timeout=0, security_mode=''): +def add_connection_xml( + version, + mode, + ip, + port, + node, + password_, + domain_name, + bindings_timeout=0, + security_mode="", +): """Generate xml for Add Connection request :param version: Version of SXP protocol (version1/2/3/4) @@ -555,7 +602,8 @@ def add_connection_xml(version, mode, ip, port, node, password_, domain_name, bi :returns: String containing xml data for request """ - templ = Template(''' + templ = Template( + """ $node $domain @@ -572,15 +620,30 @@ def add_connection_xml(version, mode, ip, port, node, password_, domain_name, bi 30 $timeout $timeout + 90 + 180 + 90 -''') +""" + ) data = templ.substitute( - {'ip': ip, 'port': port, 'mode': mode, 'version': version, 'node': node, - 'password_': password_, 'domain': get_domain_name(domain_name), 'timeout': bindings_timeout, - 'security_type': '' + security_mode + '' if security_mode else ''}) + { + "ip": ip, + "port": port, + "mode": mode, + "version": version, + "node": node, + "password_": password_, + "domain": get_domain_name(domain_name), + "timeout": bindings_timeout, + "security_type": "" + security_mode + "" + if security_mode + else "", + } + ) return data @@ -598,13 +661,22 @@ def delete_connections_xml(address, port, node, domain_name): :returns: String containing xml data for request """ - templ = Template(''' + templ = Template( + """ $node $domain $address $port -''') - data = templ.substitute({'address': address, 'port': port, 'node': node, 'domain': get_domain_name(domain_name)}) +""" + ) + data = templ.substitute( + { + "address": address, + "port": port, + "node": node, + "domain": get_domain_name(domain_name), + } + ) return data @@ -620,14 +692,16 @@ def add_peer_group_xml(name, peers, ip): :returns: String containing xml data for request """ - templ = Template(''' + templ = Template( + """ $ip $name $peers -''') - data = templ.substitute({'name': name, 'peers': peers, 'ip': ip}) +""" + ) + data = templ.substitute({"name": name, "peers": peers, "ip": ip}) return data @@ -641,11 +715,13 @@ def delete_peer_group_xml(name, ip): :returns: String containing xml data for request """ - templ = Template(''' + templ = Template( + """ $ip $name -''') - data = templ.substitute({'name': name, 'ip': ip}) +""" + ) + data = templ.substitute({"name": name, "ip": ip}) return data @@ -657,10 +733,12 @@ def get_peer_groups_from_node_xml(ip): :returns: String containing xml data for request """ - templ = Template(''' + templ = Template( + """ $ip -''') - data = templ.substitute({'ip': ip}) +""" + ) + data = templ.substitute({"ip": ip}) return data @@ -684,16 +762,25 @@ def add_filter_xml(group, filter_type, entries, ip, policy=None): policy = "" + policy + "" else: policy = "" - templ = Template(''' + templ = Template( + """ $ip $group $filter_policy $filter_type$entries -''') +""" + ) data = templ.substitute( - {'group': group, 'filter_type': filter_type, 'ip': ip, 'entries': entries, 'filter_policy': policy}) + { + "group": group, + "filter_type": filter_type, + "ip": ip, + "entries": entries, + "filter_policy": policy, + } + ) return data @@ -715,7 +802,8 @@ def add_domain_filter_xml(domain, domains, entries, ip, filter_name=None): """ if filter_name: filter_name = "" + filter_name + "" - templ = Template(''' + templ = Template( + """ $ip $domain @@ -723,9 +811,17 @@ def add_domain_filter_xml(domain, domains, entries, ip, filter_name=None): $domains $entries -''') +""" + ) data = templ.substitute( - {'domain': domain, 'domains': domains, 'ip': ip, 'entries': entries, 'filter_name': filter_name}) + { + "domain": domain, + "domains": domains, + "ip": ip, + "entries": entries, + "filter_name": filter_name, + } + ) return data @@ -741,13 +837,14 @@ def delete_filter_xml(group, filter_type, ip): :returns: String containing xml data for request """ - templ = Template(''' + templ = Template( + """ $ip $group $filter_type -''') - data = templ.substitute( - {'group': group, 'filter_type': filter_type, 'ip': ip}) +""" + ) + data = templ.substitute({"group": group, "filter_type": filter_type, "ip": ip}) return data @@ -764,14 +861,19 @@ def delete_domain_filter_xml(domain, ip, filter_name=None): """ if filter_name: - filter_name = '' + filter_name + "" - templ = Template(''' + filter_name = ( + '' + + filter_name + + "" + ) + templ = Template( + """ $ip $domain $filter_name -''') - data = templ.substitute( - {'domain': domain, 'ip': ip, 'filter_name': filter_name}) +""" + ) + data = templ.substitute({"domain": domain, "ip": ip, "filter_name": filter_name}) return data @@ -785,11 +887,13 @@ def get_connections_from_node_xml(ip, domain_name): :returns: String containing xml data for request """ - templ = Template(''' + templ = Template( + """ $ip $domain -''') - data = templ.substitute({'ip': ip, 'domain': get_domain_name(domain_name)}) +""" + ) + data = templ.substitute({"ip": ip, "domain": get_domain_name(domain_name)}) return data @@ -805,17 +909,30 @@ def get_bindings_from_node_xml(ip, binding_range, domain_name): :returns: String containing xml data for request """ - templ = Template(''' + templ = Template( + """ $ip $range $domain -''') - data = templ.substitute({'ip': ip, 'range': binding_range, 'domain': get_domain_name(domain_name)}) +""" + ) + data = templ.substitute( + {"ip": ip, "range": binding_range, "domain": get_domain_name(domain_name)} + ) return data -def add_node_xml(node_id, port, password, version, node_ip=None, expansion=0, bindings_timeout=0, keystores=None, - retry_open_timer=1): +def add_node_xml( + node_id, + port, + password, + version, + node_ip=None, + expansion=0, + bindings_timeout=0, + keystores=None, + retry_open_timer=1, +): """Generate xml for Add Node request :param node_id: Ipv4 address formatted node id @@ -837,9 +954,10 @@ def add_node_xml(node_id, port, password, version, node_ip=None, expansion=0, bi :returns: String containing xml data for request """ - tls = '' + tls = "" if keystores: - tls = Template(''' + tls = Template( + """ $keystore @@ -855,10 +973,17 @@ def add_node_xml(node_id, port, password, version, node_ip=None, expansion=0, bi $passwd - ''').substitute( - {'keystore': keystores['keystore'], 'truststore': keystores['truststore'], 'passwd': keystores['password']}) - - templ = Template(''' + """ + ).substitute( + { + "keystore": keystores["keystore"], + "truststore": keystores["truststore"], + "passwd": keystores["password"], + } + ) + + templ = Template( + """ $id $retry_open_timer @@ -879,11 +1004,21 @@ def add_node_xml(node_id, port, password, version, node_ip=None, expansion=0, bi $version ODL SXP Controller $ip -''') +""" + ) data = templ.substitute( - {'ip': node_ip or node_id, 'id': node_id, 'port': port, 'password': password, - 'version': version, 'expansion': expansion, 'timeout': bindings_timeout, 'tls': tls, - 'retry_open_timer': retry_open_timer}) + { + "ip": node_ip or node_id, + "id": node_id, + "port": port, + "password": password, + "version": version, + "expansion": expansion, + "timeout": bindings_timeout, + "tls": tls, + "retry_open_timer": retry_open_timer, + } + ) return data @@ -895,28 +1030,108 @@ def delete_node_xml(node_id): :returns: String containing xml data for request """ - templ = Template(''' + templ = Template( + """ $id -''') - data = templ.substitute({'id': node_id}) +""" + ) + data = templ.substitute({"id": node_id}) + return data + + +def add_domain_xml_fluorine(node_id, name, sgt, prefixes, origin): + """Generate xml for Add Domain request (Fluorine version: bindings with origin) + + :param node_id: Id of node + :type node_id: str + :param name: Name of Domain + :type name: str + :param sgt: Security group + :type sgt: int + :param prefixes: List of ip-prefixes + :type prefixes: str + :param origin: Origin of added bindings + :type origin: str + :returns: String containing xml data for request + + """ + master_database = "" + if prefixes != "None": + xml_prefixes = "" + for prefix in prefixes.split(","): + xml_prefixes += "\n" + "" + prefix + "" + if xml_prefixes: + master_database += """ + + $sgt + $xml_prefixes + + """ + master_database = Template(master_database).substitute( + ({"sgt": sgt, "xml_prefixes": xml_prefixes}) + ) + + templ = Template( + """ + $id + $name + $origin + $master_database +""" + ) + + data = templ.substitute( + { + "name": name, + "id": node_id, + "origin": origin, + "master_database": master_database, + } + ) return data -def add_domain_xml(node_id, name): - """Generate xml for Add Domain request +def add_domain_xml_oxygen(node_id, name, sgt, prefixes): + """Generate xml for Add Domain request (Oxygen version: bindings without origin) :param node_id: Id of node :type node_id: str :param name: Name of Domain :type name: str + :param sgt: Security group + :type sgt: int + :param prefixes: List of ip-prefixes + :type prefixes: str :returns: String containing xml data for request """ - templ = Template(''' - $id - $name -''') - data = templ.substitute({'name': name, 'id': node_id}) + master_database = "" + if prefixes != "None": + xml_prefixes = "" + for prefix in prefixes.split(","): + xml_prefixes += "\n" + "" + prefix + "" + if xml_prefixes: + master_database += """ + + $sgt + $xml_prefixes + + """ + master_database = Template(master_database).substitute( + ({"sgt": sgt, "xml_prefixes": xml_prefixes}) + ) + + templ = Template( + """ + $id + $name + $master_database +""" + ) + + data = templ.substitute( + {"name": name, "id": node_id, "master_database": master_database} + ) return data @@ -930,7 +1145,15 @@ def delete_domain_xml(node_id, name): :returns: String containing xml data for request """ - return add_domain_xml(node_id, name) + templ = Template( + """ + $node_id + $name +""" + ) + + data = templ.substitute({"node_id": node_id, "name": name}) + return data def get_domain_name(domain_name): @@ -941,14 +1164,18 @@ def get_domain_name(domain_name): :returns: String containing xml data for request """ - if domain_name == 'global': - return '' + if domain_name == "global": + return "" else: - return '' + domain_name + '' + return ( + '' + + domain_name + + "" + ) -def add_bindings_xml(node_id, domain, sgt, prefixes): - """Generate xml for Add Bindings request +def add_bindings_xml_fluorine(node_id, domain, sgt, prefixes, origin): + """Generate xml for Add Bindings request (Fluorine version with origin type) :param node_id: Id of node :type node_id: str @@ -958,21 +1185,69 @@ def add_bindings_xml(node_id, domain, sgt, prefixes): :type sgt: int :param prefixes: List of ip-prefixes :type prefixes: str + :param origin: Origin of added bindings + :type origin: str :returns: String containing xml data for request """ - bindings = '' - for prefix in prefixes.split(','): - bindings += '\n' + '' + prefix + '' - templ = Template(''' - $id - $name - - $sgt - $bindings - -''') - data = templ.substitute({'name': domain, 'id': node_id, 'sgt': sgt, 'bindings': bindings}) + xml_prefixes = "" + for prefix in prefixes.split(","): + xml_prefixes += "\n" + "" + prefix + "" + templ = Template( + """ + $id + $name + $origin + + + $sgt + $xml_prefixes + + +""" + ) + data = templ.substitute( + { + "name": domain, + "id": node_id, + "sgt": sgt, + "xml_prefixes": xml_prefixes, + "origin": origin, + } + ) + return data + + +def add_bindings_xml_oxygen(node_id, domain, sgt, prefixes): + """Generate xml for Add Bindings request (Oxygen version without origin type) + + :param node_id: Id of node + :type node_id: str + :param domain: Name of Domain + :type domain: str + :param sgt: Security group + :type sgt: int + :param prefixes: List of ip-prefixes + :type prefixes: str + :returns: String containing xml data for request + + """ + xml_prefixes = "" + for prefix in prefixes.split(","): + xml_prefixes += "\n" + "" + prefix + "" + templ = Template( + """ + $id + $name + + $sgt + $xml_prefixes + +""" + ) + data = templ.substitute( + {"name": domain, "id": node_id, "sgt": sgt, "xml_prefixes": xml_prefixes} + ) return data @@ -985,12 +1260,28 @@ def delete_bindings_xml(node_id, domain, sgt, prefixes): :type domain: str :param sgt: Security group :type sgt: int - :param prefixes: List of ip-prefixes + :param prefixes: Comma separated list of ip-prefixes :type prefixes: str :returns: String containing xml data for request """ - return add_bindings_xml(node_id, domain, sgt, prefixes) + xml_prefixes = "" + for prefix in prefixes.split(","): + xml_prefixes += "\n" + "" + prefix + "" + templ = Template( + """ + $id + $name + + $sgt + $xml_prefixes + +""" + ) + data = templ.substitute( + {"name": domain, "id": node_id, "sgt": sgt, "xml_prefixes": xml_prefixes} + ) + return data def prefix_range(start, end): @@ -1006,12 +1297,12 @@ def prefix_range(start, end): start = int(start) end = int(end) index = 0 - prefixes = '' + prefixes = "" while index < end: - prefixes += get_ip_from_number(index + start) + '/32' + prefixes += get_ip_from_number(index + start) + "/32" index += 1 if index < end: - prefixes += ',' + prefixes += "," return prefixes @@ -1027,14 +1318,18 @@ def route_definition_xml(virtual_ip, net_mask, interface): :returns: String containing xml data for request """ - templ = Template(''' + templ = Template( + """ $vip $interface $mask - ''') - data = templ.substitute({'mask': net_mask, 'vip': virtual_ip, 'interface': interface}) + """ + ) + data = templ.substitute( + {"mask": net_mask, "vip": virtual_ip, "interface": interface} + ) return data @@ -1049,13 +1344,17 @@ def route_definitions_xml(routes, old_routes=None): """ if old_routes and "" in old_routes: - templ = Template(old_routes.replace("", "$routes")) + templ = Template( + old_routes.replace("", "$routes") + ) else: - templ = Template(''' + templ = Template( + """ $routes - ''') - data = templ.substitute({'routes': routes}) + """ + ) + data = templ.substitute({"routes": routes}) return data @@ -1069,11 +1368,13 @@ def add_binding_origin_xml(origin, priority): :returns: String containing xml data for request """ - templ = Template(''' + templ = Template( + """ $origin $priority -''') - data = templ.substitute({'origin': origin, 'priority': priority}) +""" + ) + data = templ.substitute({"origin": origin, "priority": priority}) return data @@ -1087,11 +1388,13 @@ def update_binding_origin_xml(origin, priority): :returns: String containing xml data for request """ - templ = Template(''' + templ = Template( + """ $origin $priority -''') - data = templ.substitute({'origin': origin, 'priority': priority}) +""" + ) + data = templ.substitute({"origin": origin, "priority": priority}) return data @@ -1103,10 +1406,12 @@ def delete_binding_origin_xml(origin): :returns: String containing xml data for request """ - templ = Template(''' + templ = Template( + """ $origin -''') - data = templ.substitute({'origin': origin}) +""" + ) + data = templ.substitute({"origin": origin}) return data @@ -1121,7 +1426,7 @@ def find_binding_origin(origins_json, origin): """ for json_origin in parse_binding_origins(origins_json): - if json_origin['origin'] == origin: + if json_origin["origin"] == origin: return True return False @@ -1139,8 +1444,8 @@ def find_binding_origin_with_priority(origins_json, origin, priority): """ for json_origin in parse_binding_origins(origins_json): - if json_origin['origin'] == origin: - if json_origin['priority'] == int(priority): + if json_origin["origin"] == origin: + if json_origin["priority"] == int(priority): return True return False @@ -1154,7 +1459,7 @@ def parse_binding_origins(origins_json): """ output = [] - for origins in origins_json['binding-origins'].values(): + for origins in origins_json["binding-origins"].values(): for origin in origins: output.append(origin) return output