openstack/zun
Code Issues Proposed changes

Merge "Deal with security groups with duplicated name"

Browse Source
This commit is contained in:
Zuul 2018-01-23 05:39:03 +00:00 committed by Gerrit Code Review
commit 67b38b7d51
1 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
zun/api/controllers/v1/containers.py
View File

@ -188,10 +188,15 @@ class ContainersController(base.Controller):
policy.enforce(context, "container:create", policy.enforce(context, "container:create",
action="container:create") action="container:create")
# remove duplicate security_groups from list
if container_dict.get('security_groups'): if container_dict.get('security_groups'):
container_dict['security_groups'] = list( # remove duplicate security_groups from list
set(container_dict.get('security_groups'))) container_dict['security_groups'] = list(set(
container_dict.get('security_groups')))
for index, sg in enumerate(container_dict['security_groups']):
security_group_id = self._check_security_group(context,
{'name': sg})
container_dict['security_groups'][index] = security_group_id
try: try:
run = strutils.bool_from_string(run, strict=True) run = strutils.bool_from_string(run, strict=True)
container_dict['interactive'] = strutils.bool_from_string( container_dict['interactive'] = strutils.bool_from_string(