[Identity] Make the identity loading more robust

sky/backends/backend_utils.py

@@ -1638,6 +1638,9 @@ def check_owner_identity(cluster_name: str) -> None:
         for i, (owner,
                 current) in enumerate(zip(owner_identity,
                                           current_user_identity)):
+            # Clean up the owner identiy for the backslash and newlines, caused
+            # by the cloud CLI output, e.g. gcloud.
+            owner = owner.replace('\n', '').replace('\\', '')
             if owner == current:
                 if i != 0:
                     logger.warning(

sky/clouds/gcp.py

@@ -693,6 +693,7 @@ def get_current_user_identity(cls) -> Optional[List[str]]:
         try:
             account = _run_output('gcloud auth list --filter=status:ACTIVE '
                                   '--format="value(account)"')
+            account = account.strip()
         except subprocess.CalledProcessError as e:
             with ux_utils.print_exception_no_traceback():
                 raise exceptions.CloudUserIdentityError(

sky/global_user_state.py

@@ -495,9 +495,10 @@ def _load_owner(record_owner: Optional[str]) -> Optional[List[str]]:
         if result is not None and not isinstance(result, list):
             # Backwards compatibility for old records, which were stored as
             # a string instead of a list. It is possible that json.loads
-            # will parse the string with all numbers as an int, so we need
-            # to convert it back to a list of strings.
-            return [str(result)]
+            # will parse the string with all numbers as an int or escape
+            # some characters, such as \n, so we need to use the original
+            # record_owner.
+            return [record_owner]
         return result
     except json.JSONDecodeError:
         # Backwards compatibility for old records, which were stored as