fix: addressed CI problems with git submodules (#783)

* fix: addressed CI problems with git submodules

* fix: pin to git2.x

* pin to git 3.2

Author: jsam

conftest.py

@@ -159,7 +159,7 @@ def project(repository):
     """Create a test project."""
     from git import Repo
 
-    repo = Repo(repository)
+    repo = Repo(repository, search_parent_directories=True)
     commit = repo.head.commit
 
     os.chdir(repository)
@@ -306,7 +306,10 @@ def old_repository(tmpdir_factory, old_bare_repository):
     repo_path = tmpdir_factory.mktemp('repo')
     yield {
         'repo':
-            Repo(old_bare_repository['path'].strpath).clone(repo_path.strpath),
+            Repo(
+                old_bare_repository['path'].strpath,
+                search_parent_directories=True
+            ).clone(repo_path.strpath),
         'exit_code': old_bare_repository['exit_code']
     }
     shutil.rmtree(repo_path.strpath)

renku/cli/exception_handler.py

@@ -136,7 +136,8 @@ def _handle_sentry(self):
                 from renku.core.commands import get_git_home
                 from renku.core.models.datasets import Creator
 
-                user = Creator.from_git(Repo(get_git_home()))
+                repo = Repo(get_git_home())
+                user = Creator.from_git(repo)
 
                 scope.user = {'name': user.name, 'email': user.email}
 

renku/core/commands/graph.py

@@ -284,7 +284,6 @@ def process_dependencies(self, dependencies, visited=None):
             visited.add(processing.commit)
 
             activity = processing.client.process_commit(processing.commit)
-
             if activity is None:
                 continue
 

renku/core/management/repository.py

@@ -180,7 +180,6 @@ def process_commit(self, commit=None, path=None):
         from renku.core.models.provenance.activities import Activity
 
         commit = commit or self.repo.head.commit
-
         if len(commit.parents) > 1:
             return Activity(commit=commit, client=self)
 

renku/core/models/provenance/activities.py

@@ -183,6 +183,7 @@ def paths(self):
 
         for file_ in self.commit.diff(self.commit.parents or NULL_TREE):
             path_ = Path(file_.a_path)
+
             is_dataset = self.client.DATASETS in str(path_)
             not_refs = LinkReference.REFS not in str(path_)
             does_not_exists = not path_.exists()

setup.py

@@ -80,7 +80,7 @@
     'cwltool==1.0.20181012180214',
     'environ_config>=18.2.0',
     'filelock>=3.0.0',
-    'gitpython>=2.1.8',
+    'gitpython==3.0.3',
     'patool>=1.12',
     'psutil>=5.4.7',
     'pyasn1>=0.4.5',