#3059 Update controllers & templates to use new SCM data

v1.0.0
closed
sf-8 (45)
General
2015-08-20
2011-10-21
No

[#2020] set up the new data structures, we should start using them. A total repo refresh is in progress to backfill the data needed.

Related

Tickets: #2020
Tickets: #2138
Tickets: #3060
Tickets: #3759

Discussion

1 2 > >> (Page 1 of 2)
  • Rick Copeland - 2012-02-07
    • status: open --> in-progress
    • assigned_to: Rick Copeland ☕
    • size: --> 4
    • milestone: forge-backlog --> forge-feb-17
     
  • Rick Copeland - 2012-02-09
    • status: in-progress --> code-review
    • qa: Tim Van Steenburgh
     
  • Tim Van Steenburgh

    Fresh sandbox; got this error running Allura tests:

    Traceback (most recent call last):
      File "/home/tvansteenburgh/tvansteenburgh-6014/forge/Allura/allura/tests/model/test_repo.py", line 274, in test_set_last_commit_nodate
        obj, isnew = M.RepoObject.upsert('foo1')
    AttributeError: 'module' object has no attribute 'RepoObject'
    
     
  • Tim Van Steenburgh

    • status: code-review --> in-progress
     
  • Rick Copeland - 2012-02-10
    • status: in-progress --> code-review
     
  • Tim Van Steenburgh

    • qa: Tim Van Steenburgh --> Dave Brondsema
     
  • Tim Van Steenburgh

    forge:tv/3059 (I merged the original rc/3059 into this.)

    My commits contain some cleanup based on my code review. I removed a lot of code that is no longer used, added a couple tests, and fixed some things that were broken. Just give it a quick code-review to make sure I didn't do anything dumb. Tests all pass. There is still an issue with the CommitRunBuilder which breaks the commit browser - that's being handled under [#3759].

     

    Related

    Tickets: #3759

  • Dave Brondsema

    Dave Brondsema - 2012-03-07
    • assigned_to: Rick Copeland ☕ --> Tim Van Steenburgh
     
  • Dave Brondsema

    Dave Brondsema - 2012-03-21

    For SVN commits, the view of that commit does not list which files changed. E.g. /p/test3059/code-0/4/ However, the specific diff on the file like /p/test3059/code-0/4/tree/trunk/x?diff=4f69ebd79c1040786f000037:3 does work.

     
  • Dave Brondsema

    Dave Brondsema - 2012-03-22
    • status: code-review --> open
     
  • Dave Brondsema

    Dave Brondsema - 2012-03-22

    Forking doesn't work right either. I made a git repo with just a few commits and pushed them up (separately, if that matters). Then I went and used the fork link to fork the repo. The forked repo's tree view had unknown and None fields. History and commit views seem to work fine. This also happens if you manually fork a repo and push it into a different tool (not using the builtin fork form).

    Lastly, we need to figure how we want to handle this in production. Some pages don't work well until they are reindexed. But reindexing will take quite some time. Would be nice to push out the reindexing code first, but it isn't clearly separated from the view changes :(

     
  • Tim Van Steenburgh

    • status: open --> in-progress
     
  • Dave Brondsema

    Dave Brondsema - 2012-04-04
    • size: 4 --> 8
     
  • Tim Van Steenburgh

    To ssh://engr.geek.net/forge
    + 1ca6c12...9f665a5 tv/3759 -> tv/3759 (forced update)

    Latest development is continuing on tv/3759. Rick committed fixes for SVN diffs and git/hg forking. I tested these, rebased the branch against dev, and pushed it back.

    In the process of testing I found two new bugs:

    • On a forked git or hg repo, the Commit links in the tree view of the forked repo point to the upstream instead of the fork.
    • On an svn repo, click Browse Commits. Click on a commit. Click on a file in the commit detail view that loads at the bottom of the page. Get a 500.
     
  • Tim Van Steenburgh

    Tested manually and these two problems are no longer occurring. I did get a test failure though:

    Running tests in module ForgeHg
    ...E............................
    ======================================================================
    ERROR: forgehg.tests.functional.test_controllers.TestRootController.test_diff
    ----------------------------------------------------------------------
    Traceback (most recent call last):
      File "/var/local/env-allura/lib/python2.7/site-packages/nose-1.1.2-py2.7.egg/nose/case.py", line 197, in runTest
        self.test(*self.arg)
      File "/home/tvansteenburgh/tvansteenburgh-6034/forge/ForgeHg/forgehg/tests/functional/test_controllers.py", line 141, in test_diff
        resp = resp.click(description='diff')
      File "/var/local/env-allura/lib/python2.7/site-packages/WebTest-1.3.1-py2.7.egg/webtest/app.py", line 224, in click
        index=index, verbose=verbose)
      File "/var/local/env-allura/lib/python2.7/site-packages/WebTest-1.3.1-py2.7.egg/webtest/app.py", line 320, in _find_element
        % total_links)
    IndexError: No matching elements found (from 15 possible)
    
    • assigned_to: Tim Van Steenburgh --> Rick Copeland ☕

    • qa: ⸘Dave Brondsema‽ --> Tim Van Steenburgh

     
  • Tim Van Steenburgh

    • assigned_to: Rick Copeland ☕ --> Tim Van Steenburgh
    • qa: Tim Van Steenburgh --> nobody
     
  • Tim Van Steenburgh

    Gonna take a crack at this test failure since Rick can't get to it any time soon.

     
  • Tim Van Steenburgh

    • status: in-progress --> code-review
    • qa: Dave␣Brondsema
     
  • Tim Van Steenburgh

    To ssh://engr.geek.net/forge
    + 24857f1...88f9384 tv/3759 -> tv/3759 (forced update)

    Test was out-of-date after Cory's diff refactor, which was recently rebased into this branch.

    QA: Tests all pass. Find more broken stuff?

     
  • Dave Brondsema

    Dave Brondsema - 2012-04-23
     
  • Dave Brondsema

    Dave Brondsema - 2012-04-23

    Easy testing worked fine, so I went with a big test. I think the bug can probably be replicated on a smaller scale too. Here's what I did.

    • made git repo with a few test commits (hopefully not related)
    • cd /var/local/allura
    • git remote add sb /git/p/testgit/code.git
    • git push sb
    • wait for indexing, watch logs, refresh webpage a few times
    • go to /p/testgit/code/?branch=ref%2Ftv%2F3759
    • problem: the repo's lastest commit is shown for all folders. files properly show their last relevant commit
     
  • Dave Brondsema

    Dave Brondsema - 2012-04-25
    • status: code-review --> in-progress
     
  • Tim Van Steenburgh

    To ssh://engr.geek.net/forge
    + 2fb0961...54073ed tv/3759 -> tv/3759 (forced update)

    QA: Same test - push the allura repo into a new Code tool, and confirm that the folder commits are correct now.

     
  • Tim Van Steenburgh

    • status: in-progress --> code-review
     
  • Dave Brondsema

    Dave Brondsema - 2012-05-01
    • status: code-review --> in-progress
     
1 2 > >> (Page 1 of 2)

Log in to post a comment.