Browse Source

Updating header name and executor

Signed-off-by: Nishant Totla <nishanttotla@gmail.com>
(cherry picked from commit af5df117a805c8df661c7efa58c57680019b5752)
Nishant Totla 9 years ago
parent
commit
d199f78d18

+ 1 - 1
api/client/service/create.go

@@ -47,7 +47,7 @@ func runCreate(dockerCli *client.DockerCli, opts *serviceOptions) error {
 	}
 	}
 
 
 	headers := map[string][]string{
 	headers := map[string][]string{
-		"x-registry-auth": {encodedAuth},
+		"X-Registry-Auth": {encodedAuth},
 	}
 	}
 
 
 	response, err := apiClient.ServiceCreate(ctx, service, headers)
 	response, err := apiClient.ServiceCreate(ctx, service, headers)

+ 1 - 3
api/client/service/scale.go

@@ -76,9 +76,7 @@ func runServiceScale(dockerCli *client.DockerCli, serviceID string, scale string
 		if err != nil {
 		if err != nil {
 			return err
 			return err
 		}
 		}
-		headers = map[string][]string{
-			"x-registry-auth": {encodedAuth},
-		}
+		headers["X-Registry-Auth"] = []string{encodedAuth}
 	}
 	}
 
 
 	serviceMode := &service.Spec.Mode
 	serviceMode := &service.Spec.Mode

+ 1 - 3
api/client/service/update.go

@@ -53,9 +53,7 @@ func runUpdate(dockerCli *client.DockerCli, flags *pflag.FlagSet, serviceID stri
 		if err != nil {
 		if err != nil {
 			return err
 			return err
 		}
 		}
-		headers = map[string][]string{
-			"x-registry-auth": {encodedAuth},
-		}
+		headers["X-Registry-Auth"] = []string{encodedAuth}
 	}
 	}
 
 
 	service, _, err := apiClient.ServiceInspectWithRaw(ctx, serviceID)
 	service, _, err := apiClient.ServiceInspectWithRaw(ctx, serviceID)

+ 4 - 1
api/client/stack/deploy.go

@@ -184,18 +184,21 @@ func deployServices(
 		if service, exists := existingServiceMap[name]; exists {
 		if service, exists := existingServiceMap[name]; exists {
 			fmt.Fprintf(out, "Updating service %s (id: %s)\n", name, service.ID)
 			fmt.Fprintf(out, "Updating service %s (id: %s)\n", name, service.ID)
 
 
+			// TODO(nishanttotla): Pass headers with X-Registry-Auth
 			if err := apiClient.ServiceUpdate(
 			if err := apiClient.ServiceUpdate(
 				ctx,
 				ctx,
 				service.ID,
 				service.ID,
 				service.Version,
 				service.Version,
 				serviceSpec,
 				serviceSpec,
+				nil,
 			); err != nil {
 			); err != nil {
 				return err
 				return err
 			}
 			}
 		} else {
 		} else {
 			fmt.Fprintf(out, "Creating service %s\n", name)
 			fmt.Fprintf(out, "Creating service %s\n", name)
 
 
-			if _, err := apiClient.ServiceCreate(ctx, serviceSpec); err != nil {
+			// TODO(nishanttotla): Pass headers with X-Registry-Auth
+			if _, err := apiClient.ServiceCreate(ctx, serviceSpec, nil); err != nil {
 				return err
 				return err
 			}
 			}
 		}
 		}

+ 2 - 8
api/server/router/swarm/cluster_routes.go

@@ -107,10 +107,7 @@ func (sr *swarmRouter) createService(ctx context.Context, w http.ResponseWriter,
 		return err
 		return err
 	}
 	}
 
 
-	encodedAuth := ""
-	if auth, ok := r.Header["x-registry-auth"]; ok {
-		encodedAuth = auth[0]
-	}
+	encodedAuth := r.Header.Get("X-Registry-Auth")
 
 
 	id, err := sr.backend.CreateService(service, encodedAuth)
 	id, err := sr.backend.CreateService(service, encodedAuth)
 	if err != nil {
 	if err != nil {
@@ -135,10 +132,7 @@ func (sr *swarmRouter) updateService(ctx context.Context, w http.ResponseWriter,
 		return fmt.Errorf("Invalid service version '%s': %s", rawVersion, err.Error())
 		return fmt.Errorf("Invalid service version '%s': %s", rawVersion, err.Error())
 	}
 	}
 
 
-	encodedAuth := ""
-	if auth, ok := r.Header["x-registry-auth"]; ok {
-		encodedAuth = auth[0]
-	}
+	encodedAuth := r.Header.Get("X-Registry-Auth")
 
 
 	if err := sr.backend.UpdateService(vars["id"], version, service, encodedAuth); err != nil {
 	if err := sr.backend.UpdateService(vars["id"], version, service, encodedAuth); err != nil {
 		logrus.Errorf("Error updating service %s: %v", vars["id"], err)
 		logrus.Errorf("Error updating service %s: %v", vars["id"], err)

+ 0 - 1
daemon/cluster/executor/container/adapter.go

@@ -44,7 +44,6 @@ func (c *containerAdapter) pullImage(ctx context.Context) error {
 	var encodedAuthConfig string
 	var encodedAuthConfig string
 	if spec.PullOptions != nil {
 	if spec.PullOptions != nil {
 		encodedAuthConfig = spec.PullOptions.RegistryAuth
 		encodedAuthConfig = spec.PullOptions.RegistryAuth
-
 	}
 	}
 
 
 	authConfig := &types.AuthConfig{}
 	authConfig := &types.AuthConfig{}

+ 1 - 1
hack/vendor.sh

@@ -60,7 +60,7 @@ clone git golang.org/x/net 2beffdc2e92c8a3027590f898fe88f69af48a3f8 https://gith
 clone git golang.org/x/sys eb2c74142fd19a79b3f237334c7384d5167b1b46 https://github.com/golang/sys.git
 clone git golang.org/x/sys eb2c74142fd19a79b3f237334c7384d5167b1b46 https://github.com/golang/sys.git
 clone git github.com/docker/go-units 651fc226e7441360384da338d0fd37f2440ffbe3
 clone git github.com/docker/go-units 651fc226e7441360384da338d0fd37f2440ffbe3
 clone git github.com/docker/go-connections fa2850ff103453a9ad190da0df0af134f0314b3d
 clone git github.com/docker/go-connections fa2850ff103453a9ad190da0df0af134f0314b3d
-clone git github.com/docker/engine-api c57d0447ea1ae71f6dad83c8d8a1215a89869a0c
+clone git github.com/docker/engine-api 19b4fb48a86c3318e610e156ec06b684f79ac31d
 clone git github.com/RackSec/srslog 259aed10dfa74ea2961eddd1d9847619f6e98837
 clone git github.com/RackSec/srslog 259aed10dfa74ea2961eddd1d9847619f6e98837
 clone git github.com/imdario/mergo 0.2.1
 clone git github.com/imdario/mergo 0.2.1