Ack for this one. I have encountered this issue for some PRs.

- Henry

On Tue, Oct 5, 2021 at 2:44 AM Enrico Olivelli <eolive...@gmail.com> wrote:

> Hello,
> The bk-merge-script has new problems, now it fails for me consistently, it
> looks like a problem with the GH Merge API, see the error message below.
>
> In Pulsar we are using the GH merge button
> the main limitations of using the Merge Button are:
> - you have to manually set labels and milestone
> - it does not handle cherry-picks
>
> I propose to switch to using the GH merge button, or at least to allow
> Committers to use it
> We can write instructions for committers
>
> Thoughts ?
> Enrico
>
>
> From https://github.com/apache/bookkeeper
>
>  * [new ref]             refs/pull/2808/head -> PR_TOOL_MERGE_PR_2808
>
> Enter reviewers [Prashant Kumar <None>, Enrico Olivelli <
> eolive...@gmail.com>,
> Andrey Yegorov <None>]:
>
> ['git', 'log', 'HEAD..PR_TOOL_MERGE_PR_2808', '--pretty=format:%h [%an]
> %s']
>
> List pull request commits in squashed commit message? (y/n): n
>
> git rev-parse PR_TOOL_MERGE_PR_2808
>
>
> Merge Pull Request (2808) with following details:
>
> {
>
>     "commit_title": "set response body for bookie is_ready api",
>
>     "commit_message": "API `/api/v1/bookie/is_ready` should set some
> response body, when code 200 returned.\r\nOr, this will confuse people,
> they don't know if this api  return successfully.\r\n\r\n\r\n![image](
>
> https://user-images.githubusercontent.com/9278488/134855390-f8c18ba2-3968-4352-82fd-f59883dcd2cb.png
> )\n\nReviewers:
> Prashant Kumar <None>, Enrico Olivelli <eolive...@gmail.com>, Andrey
> Yegorov <None>\n\nThis closes #2808 from
> gaozhangmin/api_bookie_is_ready_message_when_return_code_200",
>
>     "sha": "89f16843232848972bbd551b96ecc13428f3f36d",
>
>     "merge_method": "squash"
>
> } (y/n): y
>
> Unable to fetch URL, exiting:
> https://api.github.com/repos/apache/bookkeeper/pulls/2808/merge - HTTP
> Error 404: Not Found
>
> HTTP Error 404: Not Found
>

Reply via email to