hubcio opened a new pull request, #3260: URL: https://github.com/apache/iggy/pull/3260
A transient 502 on a label DELETE aborted the whole command: removeLabelIfPresent only swallowed 404, so the rethrow skipped the paired addLabels and left the PR in a state the lifecycle hooks never re-enter. A /author placed in a review body never reached the bot at all -- only issue_comment and pull_request_target were triggers. And commands gave no visible feedback, so a rejected one looked identical to one that worked. Wrap every GitHub API call in withRetry (bounded exponential backoff over 5xx/429/network errors; 4xx still surfaces immediately). Add the pull_request_review trigger so review bodies are parsed like comments, and treat a changes_requested review by a committer as an implicit /author. React on the comment (+1 applied, confused denied) and post a one-line reply when /request-review is rejected. setLabels rollback now only re-adds the removed label when the remove actually deleted one -- re-adding unconditionally would fabricate a label on a PR that had neither S-* label. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
