JenkinsIcon indicating the project type

Jenkins

Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-7277

Error in GitPublisher

    Details

    • Type: Bug
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Fixed
    • Component/s: git-plugin
    • Labels:
      None
    • Environment:
      Ubuntu Lucid, Hudson 1.373
    • Similar Issues:

      Description

      The same project as in JENKINS-7276 also has this problem.

      Pushing result hudson-ARCHITECTURE=indep,DISTRIBUTION=lucid-12-SUCCESS to master branch of origin repository
[lucid] $ git push git@github.com:simplegeo/sqlite3-ruby.git HEAD:master
GitAPI created
No repository found for target repo name 
GitAPI created
No repository found for target repo name 
Finished: FAILURE

      Full log: http://ci.pub.simplegeo.com:8080/job/sqlite3-ruby/12/ARCHITECTURE=indep,DISTRIBUTION=lucid/console

      I have no idea why, since the `git push` command works fine when I run it by han, but it's failing my build.

        Attachments

          Activity

          Ascending order - Click to sort in descending order
          ieure ieure created issue -
          Hide
          Permalink
          abayer Andrew Bayer added a comment -

          Hmm - I think this is something getting confused in the branch/tag pushing sections of GitPublisher. I'm assuming you have the "Merge Results" option checked. Do you have anything specified for the "Tags to push" and/or "Branches to push" sections? Any chance you could attach your job's config.xml?

          Show
          abayer Andrew Bayer added a comment - Hmm - I think this is something getting confused in the branch/tag pushing sections of GitPublisher. I'm assuming you have the "Merge Results" option checked. Do you have anything specified for the "Tags to push" and/or "Branches to push" sections? Any chance you could attach your job's config.xml?
          Hide
          Permalink
          abayer Andrew Bayer added a comment -

          That is - the push you're actually seeing there isn't the problem. It's working fine. The barfing is happening because it seems to think there are branches and/or tags it should be pushing as well, but you (I assume) don't have any configured.

          Show
          abayer Andrew Bayer added a comment - That is - the push you're actually seeing there isn't the problem. It's working fine. The barfing is happening because it seems to think there are branches and/or tags it should be pushing as well, but you (I assume) don't have any configured.
          Hide
          Permalink
          ieure ieure added a comment -

          Okay, I see what was going on. I had clicked the "add tag" and/or "add branch" buttons, then didn't fill anything out there. I guess it doesn't validate the config like other plugins, so I wasn't aware it was misconfigured. After deleting the empty sections, it seems to work fine.

          I guess it just needs to verify the settings and show some error text if you do that.

          Show
          ieure ieure added a comment - Okay, I see what was going on. I had clicked the "add tag" and/or "add branch" buttons, then didn't fill anything out there. I guess it doesn't validate the config like other plugins, so I wasn't aware it was misconfigured. After deleting the empty sections, it seems to work fine. I guess it just needs to verify the settings and show some error text if you do that.
          Hide
          Permalink
          abayer Andrew Bayer added a comment -

          Ah, good - I was afraid my logic was busted somehow. Yeah, I have a tough time getting form validation working properly with repeatable fields, but I'll figure it out and get it in the next release.

          Show
          abayer Andrew Bayer added a comment - Ah, good - I was afraid my logic was busted somehow. Yeah, I have a tough time getting form validation working properly with repeatable fields, but I'll figure it out and get it in the next release.
          Hide
          Permalink
          dogfood dogfood added a comment -

          Integrated in plugins_hudson-git-plugin #21
          JENKINS-7277 Added validation for tag and branch name fields in GitPublisher, and revised the remote validation to not allow blank repo names for tag/branch pushes

          Andrew Bayer :
          Files :

          • src/main/java/hudson/plugins/git/GitPublisher.java
          • src/main/resources/hudson/plugins/git/GitSCM/config.jelly
          • src/main/resources/hudson/plugins/git/GitPublisher/config.jelly
          • src/main/java/hudson/plugins/git/GitSCM.java
          Show
          dogfood dogfood added a comment - Integrated in plugins_hudson-git-plugin #21 JENKINS-7277 Added validation for tag and branch name fields in GitPublisher, and revised the remote validation to not allow blank repo names for tag/branch pushes Andrew Bayer : Files : src/main/java/hudson/plugins/git/GitPublisher.java src/main/resources/hudson/plugins/git/GitSCM/config.jelly src/main/resources/hudson/plugins/git/GitPublisher/config.jelly src/main/java/hudson/plugins/git/GitSCM.java
          abayer Andrew Bayer made changes -
          Field Original Value New Value
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          markewaite Mark Waite made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          rtyler R. Tyler Croy made changes -
          Workflow JNJira [ 137376 ] JNJira + In-Review [ 204478 ]

            People

            • Assignee:
              abayer Andrew Bayer
              Reporter:
              ieure ieure
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: