Details

      Description

      We would like to get review board setup for Kafka.
      repo: http://git-wip-us.apache.org/repos/asf/kafka.git
      mailing list: dev@kafka.apache.org

      Thanks!

        Activity

        Hide
        jfarrell Jake Farrell added a comment -
        done
        Show
        jfarrell Jake Farrell added a comment - done
        Hide
        jkreps Jay Kreps added a comment - Reporter
        Thanks for setting this up. Unfortunately it doesn't quite work for me.

        If I try to upload a patch I get
        "Something broke! (Error 500)"
        I am using this patch: https://issues.apache.org/jira/secure/attachment/12599841/KAFKA-925-post-commit-v1.patch
        I am setting the repository to "kafka" and the base directory to "/".

        Trying another way if I install post-review and try to use that:

        jkreps$ git config reviewboard.url https://reviews.apache.org
        jkreps$ post-review --tracking-branch=origin/trunk
        ==> HTTP Authentication Required
        Enter authorization information for "Web API" at reviews.apache.org
        Username: jkreps
        Password:

        There was an error creating this review request.

        The repository path "https://git-wip-us.apache.org/repos/asf/kafka.git" is not in the
        list of known repositories on the server.

        Ask the administrator to add this repository to the Review Board server.
        For information on adding repositories, please read
        http://www.reviewboard.org/docs/manual/dev/admin/configuration/repositories/
        Show
        jkreps Jay Kreps added a comment - Reporter Thanks for setting this up. Unfortunately it doesn't quite work for me. If I try to upload a patch I get "Something broke! (Error 500)" I am using this patch: https://issues.apache.org/jira/secure/attachment/12599841/KAFKA-925-post-commit-v1.patch I am setting the repository to "kafka" and the base directory to "/". Trying another way if I install post-review and try to use that: jkreps$ git config reviewboard.url https://reviews.apache.org jkreps$ post-review --tracking-branch=origin/trunk ==> HTTP Authentication Required Enter authorization information for "Web API" at reviews.apache.org Username: jkreps Password: There was an error creating this review request. The repository path " https://git-wip-us.apache.org/repos/asf/kafka.git " is not in the list of known repositories on the server. Ask the administrator to add this repository to the Review Board server. For information on adding repositories, please read http://www.reviewboard.org/docs/manual/dev/admin/configuration/repositories/
        Hide
        jfarrell Jake Farrell added a comment -
        reviewboard uses git.apache.org as its repository source, not git-wip

        you will want to use: git://git.apache.org/kafka.git
        Show
        jfarrell Jake Farrell added a comment - reviewboard uses git.apache.org as its repository source, not git-wip you will want to use: git://git.apache.org/kafka.git
        Hide
        jkreps Jay Kreps added a comment - Reporter
        The second issues was just my mistake using the other git url. However the first issue remains. I tried a few things. Here is the problem: there are two "kafka" groups in the group selector, the first one will always result in the 500 error the second one works fine. Both look exactly the same in the UI. Can we remove whatever this bogus group is? It is confusing people trying to file reviews.
        Show
        jkreps Jay Kreps added a comment - Reporter The second issues was just my mistake using the other git url. However the first issue remains. I tried a few things. Here is the problem: there are two "kafka" groups in the group selector, the first one will always result in the 500 error the second one works fine. Both look exactly the same in the UI. Can we remove whatever this bogus group is? It is confusing people trying to file reviews.
        Hide
        jfarrell Jake Farrell added a comment -
        duplicate group was the incubator group, deleted
        Show
        jfarrell Jake Farrell added a comment - duplicate group was the incubator group, deleted

          People

          • Assignee:
            jfarrell Jake Farrell
            Reporter:
            jkreps Jay Kreps
            Request participants:
            None
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: