fix(meta): Disable device notification by default

Set Notifications.PostDeviceChanges to false by default in core-metadata, because support-notifications service is not always up. Modify the logging level Warn when sending notifications failed. Close #3768 Signed-off-by: Cloud Tsai <[email protected]>
edgexfoundry · Oct 31, 2021 · 0eb2b22 · 0eb2b22
1 parent adab524
commit 0eb2b22
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/cmd/core-metadata/res/configuration.toml b/cmd/core-metadata/res/configuration.toml
@@ -52,7 +52,7 @@ Type = "consul"
   Type = "redisdb"
 
 [Notifications]
-PostDeviceChanges = true
+PostDeviceChanges = false
 Content = "Meatadata notice: "
 Sender = "core-metadata"
 Description = "Metadata change notice"

diff --git a/internal/core/metadata/application/notify.go b/internal/core/metadata/application/notify.go
@@ -236,7 +236,7 @@ func sendNotification(ctx context.Context, dic *di.Container, name string, actio
 	req := requests.NewAddNotificationRequest(dto)
 	res, err := client.SendNotification(ctx, []requests.AddNotificationRequest{req})
 	if err != nil {
-		lc.Errorf("fail to send the notification for %s, err: %v", name, err)
+		lc.Warnf("fail to send the notification for %s, err: %v", name, err)
 		return
 	}
 	if len(res) > 0 && res[0].StatusCode > http.StatusMultiStatus {