Преглед изворни кода

Return proper error types on sandbox creation

Signed-off-by: Alessandro Boch <aboch@docker.com>
Alessandro Boch пре 9 година
родитељ
комит
2e88dfa406
1 измењених фајлова са 2 додато и 2 уклоњено
  1. 2 2
      libnetwork/controller.go

+ 2 - 2
libnetwork/controller.go

@@ -801,7 +801,7 @@ func (c *controller) NewSandbox(containerID string, options ...SandboxOption) (s
 			// If not a stub, then we already have a complete sandbox.
 			// If not a stub, then we already have a complete sandbox.
 			if !s.isStub {
 			if !s.isStub {
 				c.Unlock()
 				c.Unlock()
-				return nil, types.BadRequestErrorf("container %s is already present: %v", containerID, s)
+				return nil, types.ForbiddenErrorf("container %s is already present: %v", containerID, s)
 			}
 			}
 
 
 			// We already have a stub sandbox from the
 			// We already have a stub sandbox from the
@@ -836,7 +836,7 @@ func (c *controller) NewSandbox(containerID string, options ...SandboxOption) (s
 	c.Lock()
 	c.Lock()
 	if sb.ingress && c.ingressSandbox != nil {
 	if sb.ingress && c.ingressSandbox != nil {
 		c.Unlock()
 		c.Unlock()
-		return nil, fmt.Errorf("ingress sandbox already present")
+		return nil, types.ForbiddenErrorf("ingress sandbox already present")
 	}
 	}
 
 
 	if sb.ingress {
 	if sb.ingress {