JOB DSL for webhook trigger plugin

vishnujanardhanan commented 3 years ago

When reporting a bug, please try to provide as much information as possible.

Plugin version used. - 1.72
Jenkins version used. - 2.249.2

I wanted to trigger builds only when there is push to master branch, so i have configured the below job DSL to handle it. However no nuilds seems to get triggerred after i added them. If i just have tokens in the jobdsl it triggers the build for push on every branch. Can you please help?

triggers {
    genericTrigger {
        genericVariables {
            genericVariable {
                key("ref")
                value("\$.ref")
                expressionType("JSONPath") //Optional, defaults to JSONPath
                regexpFilter("") //Optional, defaults to empty string
                defaultValue("") //Optional, defaults to empty string
            }
        }
        token('apiinfra')
        printPostContent(true)
        regexpFilterText("\$ref")
        regexpFilterExpression("^(refs\\/heads\\/master)\$")
    }
}

post content logged in the jenkins build log was empty. I can see the post content from webhook in jenkins though.

tomasbjerre commented 3 years ago

How do you trigger the plugin? And what is the response of that?

vishnujanardhanan commented 3 years ago

@tomasbjerre A webhook is configured from git which triggers on every push. response is a 200 on git. post parameters seems to be empty on jenkins workspace logs if I enable the flag in webhook trigger plugin. So not sure whether i;m missing something here

tomasbjerre commented 3 years ago

I need to know exactly what that webhook contains. And exactly what the response is, not only http code.

vishnujanardhanan commented 3 years ago

ok, can i private msg you? Not able to post them here due to org policies

tomasbjerre commented 3 years ago

Post it here and remove sensitive parts. Or change its content.

Den tis 15 dec. 2020 15:30Vishnu Janardhanan notifications@github.com skrev:

ok, can i private msg you? Not able to post them here due to org policies

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/jenkinsci/generic-webhook-trigger-plugin/issues/195#issuecomment-745327837, or unsubscribe https://github.com/notifications/unsubscribe-auth/AADLKE22YSNHB6ZTFECYDRTSU5XJFANCNFSM4UTIGSMA .

vishnujanardhanan commented 3 years ago

  ],
  "removed": [

  ],
  "modified": [
    "infrastructure/01_mgmt/env/dev/master/config/config.xml"
  ]
},
{
  "id": "2683b259689c50c80a2eaadeb228b9968014b2bc",
  "tree_id": "4dc30553e7504093818412b706dc7d902600251e",
  "distinct": true,
  "message": "Merge pull request #422",
  "timestamp": "2020-12-15T15:26:46+05:30",
  "url": "https://github.xxx.org/xorg/xrepo/commit/2683b259689c50c80a2eaadeb228b9968014b2bc",
  "author": {
    "name": "xuser",
    "email": "xuser@in.xxx.com",
    "username": "xuser"
  },
  "committer": {
    "name": "GitHub Enterprise",
    "email": "noreply@github.xxx.com"
  },
  "added": [

  ],
  "removed": [

  ],
  "modified": [
    "infrastructure/01_mgmt/env/dev/master/config/config.xml"
  ]
}

], "head_commit": { "id": "2683b259689c50c80a2eaadeb228b9968014b2bc", "tree_id": "4dc30553e7504093818412b706dc7d902600251e", "distinct": true, "message": "Merge pull request #422", "timestamp": "2020-12-15T15:26:46+05:30", "url": "https://github.xxx.org/xorg/xrepo/commit/2683b259689c50c80a2eaadeb228b9968014b2bc", "author": { "name": "xuser", "email": "xuser@in.xxx.com", "username": "xuser" }, "committer": { "name": "GitHub Enterprise", "email": "noreply@github.xxx.com" }, "added": [

],
"removed": [

],
"modified": [
  "infrastructure/01_mgmt/env/dev/master/config/config.xml"
]

} }

tomasbjerre commented 3 years ago

Did you try without \\? like:

regexpFilterExpression("^(refs/heads/master)\$")

As described here: https://github.com/jenkinsci/generic-webhook-trigger-plugin/blob/master/src/test/resources/org/jenkinsci/plugins/gwt/bdd/github/github-push-specific-branches.feature#L10

vishnujanardhanan commented 3 years ago

@tomasbjerre thanks.. this filter worked -> regexpFilterExpression("^(refs/heads/master)\$")

jenkinsci / generic-webhook-trigger-plugin

JOB DSL for webhook trigger plugin #195