use two-tuples rather than dicts to allow multiple aliases for the same target

pull/25014/head
bengerman 8 years ago committed by Toshio Kuratomi
parent 3610a2457f
commit 5b39368108

@ -1094,13 +1094,13 @@ class TaskParameters(DockerBaseClass):
if links is None: if links is None:
return None return None
result = {} result = []
for link in links: for link in links:
parsed_link = link.split(':', 1) parsed_link = link.split(':', 1)
if len(parsed_link) == 2: if len(parsed_link) == 2:
result[parsed_link[0]] = parsed_link[1] result.append((parsed_link[0], parsed_link[1]))
else: else:
result[parsed_link[0]] = parsed_link[0] result.append((parsed_link[0], parsed_link[0]))
return result return result
def _parse_ulimits(self): def _parse_ulimits(self):
@ -1451,7 +1451,7 @@ class Container(DockerBaseClass):
diff = True diff = True
if network.get('links') and connected_networks[network['name']].get('Links'): if network.get('links') and connected_networks[network['name']].get('Links'):
expected_links = [] expected_links = []
for link, alias in network['links'].items(): for link, alias in network['links']:
expected_links.append("%s:%s" % (link, alias)) expected_links.append("%s:%s" % (link, alias))
for link in expected_links: for link in expected_links:
if link not in connected_networks[network['name']].get('Links', []): if link not in connected_networks[network['name']].get('Links', []):
@ -1552,7 +1552,7 @@ class Container(DockerBaseClass):
self.log('parameter links:') self.log('parameter links:')
self.log(self.parameters.links, pretty_print=True) self.log(self.parameters.links, pretty_print=True)
exp_links = [] exp_links = []
for link, alias in self.parameters.links.items(): for link, alias in self.parameters.links:
exp_links.append("/%s:%s/%s" % (link, ('/' + self.parameters.name), alias)) exp_links.append("/%s:%s/%s" % (link, ('/' + self.parameters.name), alias))
return exp_links return exp_links

Loading…
Cancel
Save