1.0.57

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@mojoio/docker",
-  "version": "1.0.55",
+  "version": "1.0.57",
   "lockfileVersion": 1,
   "requires": true,
   "dependencies": {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mojoio/docker",
-  "version": "1.0.55",
+  "version": "1.0.57",
   "description": "easy communication with docker remote api from node, TypeScript ready",
   "private": false,
   "main": "dist/index.js",

ts/docker.classes.service.ts

@@ -107,7 +107,7 @@ export class DockerService {
   public UpdatedAt: string;
   public Spec: {
     Name: string;
-    Labels: interfaces.TLabels; // ZBD
+    Labels: interfaces.TLabels;
     TaskTemplate: {
       ContainerSpec: {
         Image: string;
@@ -142,7 +142,7 @@ export class DockerService {
 
     const dockerData = await this.dockerHostRef.request(
       'POST',
-      `/services/${this.ID}/update?version=${this.Version.Index + 1}`,
+      `/services/${this.ID}/update?version=${this.Version.Index}`,
       {
         Name: this.Spec.Name,
         TaskTemplate: this.Spec.TaskTemplate,
@@ -162,7 +162,7 @@ export class DockerService {
     Object.assign(this, dockerData);
   }
 
-  public async updateFromRegistry() {
+  public async needsUpdate(): Promise<boolean> {
     // TODO: implement digest based update recognition
 
     await this.reReadFromDockerEngine();
@@ -173,7 +173,15 @@ export class DockerService {
     const imageVersion = new plugins.smartversion.SmartVersion(dockerImage.Labels.version);
     const serviceVersion = new plugins.smartversion.SmartVersion(this.Spec.Labels.version);
     if (imageVersion.greaterThan(serviceVersion)) {
-      console.log('service needs to be updated');
+      console.log(`service ${this.Spec.Name}  needs to be updated`);
+      return true;
+    } else {
+      console.log(`service ${this.Spec.Name} is up to date.`);
+    }
+  }
+
+  public async updateFromRegistry() {
+    if (await this.needsUpdate()) {
       this.update();
     }
   }