github-integration-plugin icon indicating copy to clipboard operation
github-integration-plugin copied to clipboard

Published 20 hours ago • verified publisher icon KostyaSha

PR not building for comment at times

Open prawen opened this issue 4 years ago • 5 comments

Hello,

I'm new to this plugin and really appreciate your help here.

I have configured the plugin as per the instructions provided. Client cache is set to 20. My trigger event is "Comment matched to Pattern" My webhook settings on gitlab.com sends below events a. Issue Comments b. Pull requests

Expected flow:

  1. User raises and submits PR
  2. User makes a comment as per the pattern and the build should be triggered

Actual result: When a comment matches my pattern, build is getting triggered and the build status is being reported before and after the build. This is working as per my expectation.

Actual issue I'm facing: The comment based trigger of build is not happening at times. Below is the flow

  1. User raises and submits PR
  2. Let us consider this PR number is 5
  3. Github PR polling log says "no comment matched for PR 5"
  4. User comments as per the pattern
  5. Github PR polling log says "PR [#5 User story] not changed" and it continues to say same message
  6. For debug purpose, I added a label and removed
  7. Github PR polling log says "No matching comments found for 5"
  8. For debug purpose, I changed assignee and approvers
  9. Github PR polling log says "No matching comments found for 5"
  10. User comments again with same pattern
  11. Github PR polling log says "Comment matched to pattern: state has changed (new comment found - 'TEST-MERGE') Comment matched to pattern: matching comment TEST-MERGE"

I'm not exactly able to explain the sequence when this issue occurs. But this has been observed frequently. I waited for atleast 10-15 minutes before retrying with the same comment.

prawen avatar Jul 28 '20 08:07 prawen

I have the same issue. In my case, Jenkins just needs some delay between comments to detect the next one.

maxvasylets avatar Aug 20 '20 09:08 maxvasylets

Just disable cache. okhttp3's cache looks broken

KostyaSha avatar Aug 30 '20 22:08 KostyaSha

I have the same issue. In my case, Jenkins just needs some delay between comments to detect the next one.

Having the same issue here, leaving some delay between trigger events (i.e. comments) works.

Just disable cache. okhttp3's cache looks broken

What do you mean by cache? Is this in the plugin options?

Edit: found it!

ryuwd avatar Sep 17 '20 16:09 ryuwd

I found what was causing comments to be ignored on my end. I naively used the regex .*? to trigger builds, since I wanted the build to trigger for all comments. That regex does not match newlines, so comments that had newlines at the end were ignored. I'm now using [\S\s]* instead.

ryuwd avatar Sep 21 '20 21:09 ryuwd

Yes, mattern matching is plain. You need add multiline setting in pattern according to java regexp docs.

KostyaSha avatar Sep 21 '20 21:09 KostyaSha