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

Github webhook ignore "Github project" field (I.E. projectUrl)

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Incomplete
    • Icon: Minor Minor
    • github-plugin
    • Jenkins 1.623 from deb (Ubuntu Server 15.04)
      Github plugin 1.12.0

      When setting up a Project with "Build when a change is pushed to GitHub" enabled, the webhook handler use the source code repository url to match the github project is applies on and ignore the dedicated field "Github project" (I.E. projectUrl).

      I think this is not a wanted behavior as this field seems to be dedicated to this purpose.

      Additionally this prevent from creating jobs that does not checkout the repository but are still triggered by the github webhook

          [JENKINS-29827] Github webhook ignore "Github project" field (I.E. projectUrl)

          Félicien François created issue -

          Kanstantsin Shautsou added a comment -

          Do you mean to use GH project property?

          Kanstantsin Shautsou added a comment - Do you mean to use GH project property?
          Félicien François made changes -
          Environment New: Jenkins 1.623 from deb (Ubuntu Server 15.04)
          Github plugin 1.12.0

          Félicien François added a comment -

          Yes this is what I mean

          Félicien François added a comment - Yes this is what I mean
          Félicien François made changes -
          Summary Original: Github webhook ignore projectUrl field New: Github webhook ignore "Github project" field (I.E. projectUrl)
          Félicien François made changes -
          Description Original: When setting up a Project with "Build when a change is pushed to GitHub" enabled, the webhook handler use the source code repository url to match the github project is applies on and ignore the dedicated field "Github project" (I.E. projectUrl).

          I think this is not a wanted behavior as this field seems to be dedicated to this purpose.

          Additionally this prevent creating jobs that does not checkout the repository (or checkout another repository) but are triggered by the github webhook           		New: When setting up a Project with "Build when a change is pushed to GitHub" enabled, the webhook handler use the source code repository url to match the github project is applies on and ignore the dedicated field "Github project" (I.E. projectUrl).

          I think this is not a wanted behavior as this field seems to be dedicated to this purpose.

          Additionally this prevent from creating jobs that does not checkout the repository but are still triggered by the github webhook

          Kirill Merkushev added a comment -

          think we should add repo name contributor which extracts gh url from property

          Kirill Merkushev added a comment - think we should add repo name contributor which extracts gh url from property
          Kirill Merkushev made changes -
          Assignee Original: Kanstantsin Shautsou [ integer ] New: Kirill Merkushev [ lanwen ]

          Kanstantsin Shautsou added a comment -

          I thought about replacing SCM parsing usage, but not sure then how git polling/scmNotify should work. I can suggest firstly migrate to commitNotify.

          Kanstantsin Shautsou added a comment - I thought about replacing SCM parsing usage, but not sure then how git polling/scmNotify should work. I can suggest firstly migrate to commitNotify.

          Kirill Merkushev added a comment - - edited

          Félicien François, by default build is triggered only if scm change detected. So it never worked before with property only.

          Kirill Merkushev added a comment - - edited Félicien François, by default build is triggered only if scm change detected. So it never worked before with property only.

            lanwen Kirill Merkushev
            felicienfrancois Félicien François
            Votes:
            2 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: