openstack/zun
Code Issues Proposed changes

Merge "check the security group has been added to the container"

Browse Source
This commit is contained in:
Zuul 2018-01-17 03:52:04 +00:00 committed by Gerrit Code Review
commit 8727c1a0b3
1 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -427,6 +427,10 @@ class ContainersController(base.Controller):
context = pecan.request.context context = pecan.request.context
compute_api = pecan.request.compute_api compute_api = pecan.request.compute_api
security_group_id = self._check_security_group(context, security_group) security_group_id = self._check_security_group(context, security_group)
if security_group_id in container.security_groups:
msg = _("Security group %(id)s has been added to container.") % {
'id': security_group_id}
raise exception.InvalidValue(msg)
compute_api.add_security_group(context, container, compute_api.add_security_group(context, container,
security_group_id) security_group_id)
pecan.response.status = 202 pecan.response.status = 202