diff --git a/distributedcloud/dcmanager/manager/subcloud_manager.py b/distributedcloud/dcmanager/manager/subcloud_manager.py index f720fbd41..6e1e8240a 100644 --- a/distributedcloud/dcmanager/manager/subcloud_manager.py +++ b/distributedcloud/dcmanager/manager/subcloud_manager.py @@ -532,11 +532,12 @@ class SubcloudManager(manager.Manager): # Check if it's already in the correct state if is_unmanaged and is_rehome_pending: - LOG.info("Remote subcloud %s from system peer %s is " - "already unmanaged and rehome-pending, " - "skipping unmanage attempt" - % (system_peer.peer_name, - remote_subcloud.get('name'))) + LOG.info( + f"Remote subcloud {remote_subcloud.get('name')} " + f"from system peer {system_peer.peer_name} is " + "already unmanaged and rehome-pending, " + "skipping unmanage attempt" + ) break try: diff --git a/distributedcloud/dcmanager/manager/system_peer_manager.py b/distributedcloud/dcmanager/manager/system_peer_manager.py index b2a15dc34..2db735c35 100644 --- a/distributedcloud/dcmanager/manager/system_peer_manager.py +++ b/distributedcloud/dcmanager/manager/system_peer_manager.py @@ -808,8 +808,8 @@ class SystemPeerManager(manager.Manager): association_update['sync_status'] = \ consts.ASSOCIATION_SYNC_STATUS_FAILED association_update['sync_message'] = \ - (f"Failed to sync {error_msg.keys()} in the Subcloud " - f"Peer Group {peer_group_name}.") + (f"Failed to sync {list(error_msg.keys())} in the " + f"Subcloud Peer Group {peer_group_name}.") association = self._update_sync_status( context, association_id, **association_update) @@ -899,9 +899,9 @@ class SystemPeerManager(manager.Manager): LOG.error(f"Failed to delete subcloud(s) from " f"the Subcloud Peer Group {peer_group_name} " f"on peer site: {json.dumps(delete_error_msg)}") - sync_message = (f"Deletion of {delete_error_msg.keys()} from " - f"the Subcloud Peer Group {peer_group_name} " - f"on the peer site failed.") + sync_message = (f"Deletion of {list(delete_error_msg.keys())} " + "from the Subcloud Peer Group " + f"{peer_group_name} on the peer site failed.") self._update_sync_status_to_failed(context, association_id, sync_message) return