fix: duplicate comments by github label commenter bot#6240
fix: duplicate comments by github label commenter bot#6240M-DEV-1 wants to merge 2 commits intolayer5io:masterfrom
Conversation
Signed-off-by: M-DEV-1 <mahadevankizhakkedathu@gmail.com>
|
🚀 Preview for commit 265bb7a at: https://67b2d8efad6302e85c8f455a--layer5.netlify.app |
|
I looked into meshery/meshery repository to compare workflows as I believe that the other repo does not face these issues. The workflows were more or less the same, so I added the conditionals in the .yml files to ensure that the bot does not comment twice. I have tested this locally using the If there are other ways to test out the changes, please do let me know as I do not have access to the secret token. |
|
I have discussed the same with @vishalvivekm in today's websites meeting. |
sudhanshutech
left a comment
There was a problem hiding this comment.
@M-DEV-1 why the bot comment 2 times, we should fix the root cause instead checking condition.
I'll investigate the root cause and make the changes |
@M-DEV-1 let's be consistent between our repos. What is causing the duplicative commenting? How often does this occur, @sudhanshutech? |
|
Yes @leecalcote, I'll make sure of that and look into what exactly is causing this issue. |
|
This issue appears to date back to April-May 2024, and I couldn't find any instances of it occurring prior to that. The only difference between the Meshery and Layer5 workflows is that label-trigger exists in the Layer5 repository. I think that the label-commenter and the label-trigger workflows might be clashing with each other. |
This makes an API call as the labeler workflow in present in |
|
Good. This investigation helps. Knowing the root cause will change your approach to a solution, @M-DEV-1. |
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
|
This issue is being automatically closed due to inactivity. However, you may choose to reopen this issue. |


Description
This PR fixes #5989
Notes for Reviewers
Added conditional statements to workflow files to check if the comment already exists.
Signed commits