Warning if git is not installed (#1414)

* warning if git is not installed Signed-off-by: Kevin Su <[email protected]> * lint Signed-off-by: Kevin Su <[email protected]> Signed-off-by: Kevin Su <[email protected]>
flyteorg · Jan 18, 2023 · 905cabc · 905cabc
1 parent 4b1675f
commit 905cabc
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/flytekit/remote/remote.py b/flytekit/remote/remote.py
@@ -19,7 +19,6 @@
 
 from flyteidl.admin.signal_pb2 import Signal, SignalListRequest, SignalSetRequest
 from flyteidl.core import literals_pb2 as literals_pb2
-from git import Repo
 
 from flytekit import Literal
 from flytekit.clients.friendly import SynchronousFlyteClient
@@ -127,9 +126,14 @@ def _get_git_repo_url(source_path):
     Get git repo URL from remote.origin.url
     """
     try:
+        from git import Repo
+
         return "github.com/" + Repo(source_path).remotes.origin.url.split(".git")[0].split(":")[-1]
+    except ImportError:
+        remote_logger.warning("Could not import git. is the git executable installed?")
     except Exception:
         # If the file isn't in the git repo, we can't get the url from git config
+        remote_logger.debug(f"{source_path} is not a git repo.")
         return ""