Apache Allura™ / Tickets / #5785 AssertionError in cloning git repos

#5785 AssertionError in cloning git repos

Milestone: v1.0.0

Status: closed

Owner: nobody

Labels: migration (63) support (446) p3 (267) 42cc (432)

Component: General

Reviewer: Cory Johns

Updated: 2015-08-20

Created: 2013-02-08

Creator: Dave Brondsema

Private: No

Looks like the clone probably finished, but failed during refresh. /p/encuestame/code 500s currently

Forking/cloning repo code in project encuestame from /nfs/classic/sf-git/e/en/encuestame/encuestame failed.

Forking/cloning repo code in project zynaddsubfx from /nfs/classic/sf-git/z/zy/zynaddsubfx/zynaddsubfx failed.

Traceback (most recent call last):
  File "/var/local/allura/Allura/allura/tasks/repo_tasks.py", line 26, in clone
    cloned_from_url)
  File "/var/local/allura/Allura/allura/model/repository.py", line 275, in init_as_clone
    self.refresh(notify=False)
  File "/var/local/allura/Allura/allura/model/repository.py", line 452, in refresh
    refresh_repo(self, all_commits, notify)
  File "/var/local/allura/Allura/allura/model/repo_refresh.py", line 107, in refresh_repo
    compute_lcds(ci, cache)
  File "/var/local/allura/Allura/allura/model/repo_refresh.py", line 499, in compute_lcds
    _compute_lcds(tree, cache)
  File "/var/local/allura/Allura/allura/model/repo_refresh.py", line 504, in _compute_lcds
    lcd = LastCommit.get(tree, create=True)  # auto-vivify LCD
  File "/var/local/allura/Allura/allura/model/repo.py", line 742, in get
    lcd = cls._build(commit.get_path(path))
  File "/var/local/allura/Allura/allura/model/repo.py", line 771, in _build
    entries = tree.commit.repo.last_commit_ids(tree.commit, unchanged)
  File "/var/local/allura/Allura/allura/model/repository.py", line 257, in last_commit_ids
    return self._impl.last_commit_ids(commit, paths)
  File "/var/local/env-allura/lib/python2.7/site-packages/TimerMiddleware-0.4.1-py2.7.egg/timermiddleware/__init__.py", line 112, in wrapper
    return self.run_and_log(func, inst, *args, **kwargs)
  File "/var/local/env-allura/lib/python2.7/site-packages/TimerMiddleware-0.4.1-py2.7.egg/timermiddleware/__init__.py", line 137, in run_and_log
    return func(*args, **kwargs)
  File "/var/local/allura/Allura/allura/model/repository.py", line 132, in last_commit_ids
    changed = set([path for path in paths if not parent.has_path(path)])
  File "/var/local/allura/Allura/allura/model/repo.py", line 352, in has_path
    self.get_path(path)
  File "/var/local/allura/Allura/allura/model/repo.py", line 347, in get_path
    cur = cur[part]
  File "/var/local/allura/Allura/allura/model/repo.py", line 426, in __getitem__
    obj.set_context(self, name)
  File "/var/local/allura/Allura/allura/model/repo.py", line 448, in set_context
    assert commit_or_tree is not self
AssertionError

summary: AssertionError in encuestame git repo --> AssertionError in cloning git repos
Description has changed:

Diff:

--- old
+++ new
@@ -2,6 +2,9 @@

 ~~~~
 Forking/cloning repo code in project encuestame from /nfs/classic/sf-git/e/en/encuestame/encuestame failed.
+
+Forking/cloning repo code in project zynaddsubfx from /nfs/classic/sf-git/z/zy/zynaddsubfx/zynaddsubfx failed.
+

 Traceback (most recent call last):
   File "/var/local/allura/Allura/allura/tasks/repo_tasks.py", line 26, in clone

Dave Brondsema - 2013-02-14

private: Yes --> No
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Chris Tsai - 2013-02-14

labels: migration --> migration, support, p3
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2013-02-19

labels: migration, support, p3 --> migration, support, p3, 42cc
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Igor Bondarenko - 2013-02-20

status: open --> in-progress
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Igor Bondarenko - 2013-02-20

Created #277: [#5785] AssertionError in cloning git repos (2cp)

Related

Tickets: ~~#5785~~

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Igor Bondarenko - 2013-02-25

status: in-progress --> code-review
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Igor Bondarenko - 2013-02-25

Closed #277. allura:je/42cc_5785

The error occurs on repos with submodules. Some repos just skip submodule directories, so they isn't accessible from code viewer, and some raise an exception like encuestame/code and zynaddsubfx/code.

I don't sure what is a proper way to handle submodules in allura code browser, so I assumed that skipping submodule dirs is ok (just like repos with submodules that do not raise an exception).

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Cory Johns - 2013-02-25

qa: Cory Johns
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Cory Johns - 2013-02-26

status: code-review --> validation
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2013-02-26

status: validation --> closed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Apache Allura™

Forge software for hosting software projects

Milestone

Searches

Help

#5785 AssertionError in cloning git repos

Related

Discussion

Related