chore: add define in message bus (#200)

IceWhaleTech · Jun 12, 2024 · 8d0082a · 8d0082a
1 parent 2d41560
commit 8d0082a
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 2 deletions.
diff --git a/common/message.go b/common/message.go
@@ -38,6 +38,18 @@ var (
 		Description: utils.Ptr("progress of the app"),
 		Example:     utils.Ptr("64"),
 	}
+
+	PropertyTypeCheckPortConflict = message_bus.PropertyType{
+		Name:        "check_port_conflict",
+		Description: utils.Ptr("todo: the field should be remove in future. it is not need for most message"),
+		Example:     utils.Ptr("true"),
+	}
+
+	PropertyTypeDryRun = message_bus.PropertyType{
+		Name:        "dry_run",
+		Description: utils.Ptr("todo: the field should be remove in future. it is not need for most message"),
+		Example:     utils.Ptr("false"),
+	}
 )
 
 // container properties
@@ -132,7 +144,11 @@ var (
 		Name:     "app:install-progress",
 		PropertyTypeList: []message_bus.PropertyType{
 			PropertyTypeAppName,
+			PropertyTypeAppIcon,
 			PropertyTypeAppProgress,
+			PropertyTypeAppTitle,
+			PropertyTypeCheckPortConflict,
+			PropertyTypeDryRun,
 		},
 	}
 

diff --git a/service/image.go b/service/image.go
@@ -215,10 +215,10 @@ func pullImageProgress(ctx context.Context, out io.ReadCloser, notificationType
 		switch message.Status {
 		// pull a new layer
 		case string(Pull):
-			layerNum += 1
+			layerNum++
 		// pull a layer complete
 		case string(PullComplete):
-			completedLayerNum += 1
+			completedLayerNum++
 		}
 
 		// layer progress