From 49c4de4aebe0ecc3005e56a9ab06b5e43f5b312c Mon Sep 17 00:00:00 2001
From: Hu Keping <hukeping@huawei.com>
Date: Fri, 10 Apr 2015 01:52:55 +0800
Subject: [PATCH] Remove Job from rename

A part of ISSUE#12151-Remove engine.Job mechanism

Signed-off-by: Hu Keping <hukeping@huawei.com>
---
 api/server/server.go |  8 ++++----
 daemon/daemon.go     |  1 -
 daemon/rename.go     | 10 +++-------
 3 files changed, 7 insertions(+), 12 deletions(-)

diff --git a/api/server/server.go b/api/server/server.go
index e59aec5a79..c3e2fd0a60 100644
--- a/api/server/server.go
+++ b/api/server/server.go
@@ -859,10 +859,10 @@ func postContainerRename(eng *engine.Engine, version version.Version, w http.Res
 		return fmt.Errorf("Missing parameter")
 	}
 
-	newName := r.URL.Query().Get("name")
-	job := eng.Job("container_rename", vars["name"], newName)
-	job.Setenv("t", r.Form.Get("t"))
-	if err := job.Run(); err != nil {
+	d := getDaemon(eng)
+	name := vars["name"]
+	newName := r.Form.Get("name")
+	if err := d.ContainerRename(name, newName); err != nil {
 		return err
 	}
 	w.WriteHeader(http.StatusNoContent)
diff --git a/daemon/daemon.go b/daemon/daemon.go
index 5aa6250a88..44a59eac39 100644
--- a/daemon/daemon.go
+++ b/daemon/daemon.go
@@ -119,7 +119,6 @@ func (daemon *Daemon) Install(eng *engine.Engine) error {
 	for name, method := range map[string]engine.Handler{
 		"commit":            daemon.ContainerCommit,
 		"container_copy":    daemon.ContainerCopy,
-		"container_rename":  daemon.ContainerRename,
 		"container_inspect": daemon.ContainerInspect,
 		"container_stats":   daemon.ContainerStats,
 		"create":            daemon.ContainerCreate,
diff --git a/daemon/rename.go b/daemon/rename.go
index 66e7ac080f..72e14a1af1 100644
--- a/daemon/rename.go
+++ b/daemon/rename.go
@@ -2,16 +2,12 @@ package daemon
 
 import (
 	"fmt"
-
-	"github.com/docker/docker/engine"
 )
 
-func (daemon *Daemon) ContainerRename(job *engine.Job) error {
-	if len(job.Args) != 2 {
-		return fmt.Errorf("usage: %s OLD_NAME NEW_NAME", job.Name)
+func (daemon *Daemon) ContainerRename(oldName, newName string) error {
+	if oldName == "" || newName == "" {
+		return fmt.Errorf("usage: docker rename OLD_NAME NEW_NAME")
 	}
-	oldName := job.Args[0]
-	newName := job.Args[1]
 
 	container, err := daemon.Get(oldName)
 	if err != nil {