Always set (empty) UpdateStatus

When deploying a service the `UpdateStatus.State` field is not present during `docker service create`. This complicates creating scripts that check for `UpdateStatus.State` to reach a certain state, as they have to take into account that the `UpdateStatus` as a whole is missing. This patch always sets the `UpdateStatus.State` to make it easier to consume. Signed-off-by: Sebastiaan van Stijn <[email protected]>
moby · Feb 9, 2019 · 4ecb32b · 4ecb32b
1 parent cc6aa78
commit 4ecb32b
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/manager/controlapi/service.go b/manager/controlapi/service.go
@@ -655,9 +655,10 @@ func (s *Server) CreateService(ctx context.Context, request *api.CreateServiceRe
 	// TODO(aluzzardi): Consider using `Name` as a primary key to handle
 	// duplicate creations. See #65
 	service := &api.Service{
-		ID:          identity.NewID(),
-		Spec:        *request.Spec,
-		SpecVersion: &api.Version{},
+		ID:           identity.NewID(),
+		Spec:         *request.Spec,
+		SpecVersion:  &api.Version{},
+		UpdateStatus: &api.UpdateStatus{},
 	}
 
 	if allocator.IsIngressNetworkNeeded(service) {