universe: improve logging for single asset sync attempt

When syncing a single asset (for example for creating an address) fails,
the cause isn't easily visible in the logs. First, because the actual
remote error wasn't logged and second because it was logged as debug.
This commit fixes both.

Author: guggero

universe/auto_syncer.go

@@ -846,18 +846,31 @@ func (f *FederationEnvoy) SyncAssetInfo(ctx context.Context,
 		// Sync failures are expected from Universe servers that do not
 		// have a relevant universe root.
 		if err != nil {
-			log.Debugf("asset lookup for %v failed with remote"+
-				"server: %v", assetID.String(), addr.HostStr())
-			//lint:ignore nilerr failure is expected and logged
+			log.Warnf("Asset lookup failed: asset_id=%v, "+
+				"remote_server=%v: %v", assetID.String(),
+				addr.HostStr(), err)
+
+			// We don't want to abort syncing here, as this might
+			// just be one server in our list and returning an error
+			// would cause us to stop trying the other servers.
+			// lint:ignore nilerr failure is expected and logged.
 			return nil
 		}
 
 		// There should only be one sync diff since we're only syncing
 		// one universe root.
 		if syncDiff != nil {
 			if len(syncDiff) != 1 {
-				log.Debugf("unexpected number of sync diffs: "+
-					"%v", len(syncDiff))
+				log.Warnf("Unexpected number of sync diffs "+
+					"when looking up asset: num_diffs=%d, "+
+					"asset_id=%v, remote_server=%v",
+					len(syncDiff), assetID.String(),
+					addr.HostStr())
+
+				// We don't want to abort syncing here, as this
+				// might just be one server in our list and
+				// returning an error would cause us to stop
+				// trying the other servers.
 				return nil
 			}