Explorar o código

Merge pull request #46846 from akerouanton/refactor/container-rename-move-log-args

daemon: ContainerRename: move log args to log fields
Albin Kerouanton hai 1 ano
pai
achega
5ce8eee0a4
Modificáronse 1 ficheiros con 5 adicións e 2 borrados
  1. 5 2
      daemon/rename.go

+ 5 - 2
daemon/rename.go

@@ -102,8 +102,11 @@ func (daemon *Daemon) ContainerRename(oldName, newName string) (retErr error) {
 		if retErr != nil {
 			container.Name = oldName
 			container.NetworkSettings.IsAnonymousEndpoint = oldIsAnonymousEndpoint
-			if e := container.CheckpointTo(daemon.containersReplica); e != nil {
-				log.G(context.TODO()).Errorf("%s: Failed in writing to Disk on rename failure: %v", container.ID, e)
+			if err := container.CheckpointTo(daemon.containersReplica); err != nil {
+				log.G(context.TODO()).WithFields(log.Fields{
+					"containerID": container.ID,
+					"error":       err,
+				}).Error("failed to write container state to disk during rename")
 			}
 		}
 	}()