如何让审阅者在功能测试完成时自动批准拉取请求?

Posted

技术标签:

【中文标题】如何让审阅者在功能测试完成时自动批准拉取请求?【英文标题】:How do I have a reviewer automatically approve a pull request on functional test completion? 【发布时间】:2019-11-07 10:07:17 【问题描述】:

我有一个 Azure DevOps Git 存储库,它位于云中的 Azure DevOps 中。 我们使用 Pull Requests 来管理代码合并到我们的发布分支。

功能分支分配了一个自动审核者,其唯一目的是在我们的持续部署环境中成功完成功能测试时添加批准投票。我们的想法是,这将为我们的发布分支的自动合并提供更强大的检查。

我有一个部分 Powershell API,它访问 Azure DevOps REST API 将使我在拉取请求中获得对审阅者的引用,但我尝试通过 API 为审阅者投票或批准时失败了.

文档建议我需要将审阅者 ID 传递给以下 URI

https://dev.azure.com/$organization/$projectId/_apis/git/repositories/$repositoryId/pullrequests/$pullRequestId/reviewers/$reviewerId?api-version=5.0

当我使用从拉取请求中获得的审阅者 ID 时

https://dev.azure.com/$organization/$projectId/_apis/git/repositories/$repositoryId/pullrequests

reviewerUrl : https://dev.azure.com/my-organization/be283e5e-0466-41ef-aeb7-24264e12f6d6/_apis/git/repositories/3c4b0093-30fc-4652-a39e-08bb442b1879/pullRequests/2/reviewers/a72ce17b-22de-41a0-b6a5-49e5ba189826
vote        : 10
isRequired  : True
displayName : testaccount@xxxxxxx.com
url         : https://spsprodcca1.vssps.visualstudio.com/A41d8d933-620f-4153-952c-4ee19c0e4c0b/_apis/Identities/a72ce17b-22de-41a0-b6a5-49e5ba189826
_links      : @avatar=
id          : a72ce17b-22de-41a0-b6a5-49e5ba189826
uniqueName  : testaccount@xxxxxxx.com
imageUrl    : https://dev.azure.com/my-organization/_api/_common/identityImage?id=a72ce17b-22de-41a0-b6a5-49e5ba189826

我收到以下错误

"参数值无效。\r\n参数名称: 必须有一个有效的审阅者 ID 提供。”

我不确定我是否以正确的方式进行此操作,但我希望我的 Octopus 部署在我们的目标环境中启动功能测试,然后在成功完成测试后,为 自动测试审阅者通过 REST API。

我能找到的所有文档最终都会让我回到这个文档,但我一辈子都看不到如何为我的审阅者申请批准。

https://docs.microsoft.com/en-us/rest/api/azure/devops/git/pull%20request%20reviewers/create%20pull%20request%20reviewer?view=azure-devops-rest-5.0

【问题讨论】:

我无法重现该问题,我可以运行带有审阅者 ID 的 api 没有任何问题。 您使用的审阅者 ID 是您从附加到拉取请求的审阅者那里拉取的 ID 吗?如果没有,你是从哪里得到的? 是的,来自拉取请求(但我将 PR id 添加到 api 以仅获取一个 PR 详细信息) 在我的例子中 - 我使用的 id 是 a72ce17b-22de-41a0-b6a5-49e5ba189826? 没错,是的... 【参考方案1】:

我不认为使用虚假的拉取请求审阅者是要走的路。相反,我建议考虑使用Pull Request status 扩展拉取请求工作流程。

您使用Status API 向您的拉取请求添加自定义状态。您在开始测试(处于待处理状态)之前以及测试完成(成功或失败)后调用它。

【讨论】:

【参考方案2】:

需要注意的几点:

    只有审阅者可以发布状态更新,因此 API 调用必须为希望批准拉取请求的审阅者使用 PAT。

    批准拉取请求的审阅者必须具有对存储库的贡献者访问权限才能批准拉取请求。否则,您将收到错误消息。

    确保将审阅者添加到拉取请求中。例如,将其添加为文件夹的自动审阅者,以便所有拉取请求都自动添加它。

    在拉取请求 ($pullRequest.reviewers) 上找到的审阅者似乎与使用 API 调用直接获取拉取请求审阅者有所不同。

    用于更新拉取请求审阅者的 PATCH 版本在尝试更新现有审阅者时似乎会导致错误。您必须使用 PUT 更新现有审阅者的拉取请求批准。

这是我为让它工作所做的:

行政事项:

确保审阅者在 Azure DevOps 中拥有帐户并具有 BASIC 访问级别

确保审阅者是项目团队的一员,因此具有对项目的贡献者访问权限,最重要的是可以批准拉取请求。

确保审阅者拥有可用于 API 访问的 PAT 令牌。

API 资料

使用分配给审阅者的 PAT 获取已知拉取请求的拉取请求审阅者:

将审稿人票数更新为 10

确保审阅者数据正确转换为 JSON 格式。

确保您通过 PUT API 调用传递的标头具有正确的授权和内容类型。 "Basic <Base64String>""application/json"

function ConvertTo-Base64 

    [CmdletBinding()]
    param([Parameter(Mandatory=$true)][System.String]$input)

    $inputBytes = [System.Text.Encoding]::ASCII.GetBytes($input);
    $inputBase64 = [System.Convert]::ToBase64String($inputBytes);
    return $inputBase64


function Get-AzureDevOpsPullRequestReviewer 

    [CmdletBinding()]
    param([Parameter(Mandatory=$true)][System.String]$token,
          [Parameter(Mandatory=$true)][System.String]$organization,
          [Parameter(Mandatory=$true)][System.String]$projectId,
          [Parameter(Mandatory=$true)][System.String]$repositoryId,
          [Parameter(Mandatory=$true)][System.String]$pullRequestId,
          [Parameter(Mandatory=$false)][System.String]$reviewerId=$null)

    $uri = "https://dev.azure.com/$organization/$projectId/_apis/git/repositories/$repositoryId/pullrequests/$pullrequestId/reviewers";

    if ($reviewerId)  $uri = "$uri/$reviewerId" 

    $uri = "$uri`?api-version=5.0";

    $tokenBase64 = ConvertTo-Base64(":$token");

    $headers = @
        "Accept" = "application/json";
        "Authorization" = "Basic $tokenBase64";
    

    $response = Invoke-WebRequest -Uri $uri -headers $headers;

    $content = ConvertFrom-Json($response.Content);
    if ($reviewerId)  return $content  else  return $content.value 


function Approve-AzureDevOpsPullRequest 

    [CmdletBinding()]
    param([Parameter(Mandatory=$true)][System.String]$token,
          [Parameter(Mandatory=$true)][System.String]$organization,
          [Parameter(Mandatory=$true)][System.String]$projectId,
          [Parameter(Mandatory=$true)][System.String]$repositoryId,
          [Parameter(Mandatory=$true)][System.String]$pullRequestId,
          [Parameter(Mandatory=$true)]$reviewer)

    $uri = "https://dev.azure.com/$organization/$projectId/_apis/git/repositories/$repositoryId/pullrequests/$pullRequestId/reviewers/$($reviewer.id)`?api-version=5.0"

    $tokenBase64 = ConvertTo-Base64(":$token");

    $headers = @
        "Accept" = "application/json";
        "Authorization" = "Basic $tokenBase64";
        "Content-Type" = "application/json";
    

    $body = ConvertTo-Json($reviewer);

    $response = Invoke-WebRequest -Uri $uri -headers $headers -body $body -Method Put

    return $response;


$myOrganization  = "benco-devops";
$myProjectId     = "47b0a6fc-a58f-4bbf-9950-2d5e33ae0587";
$myRepositoryId  = "1582ab2b-ae01-41e1-9695-f9966fdd7985";
$myReviewerId    = "ben-project-pipeline@benco-devops.com";
$myReviewerPAT   = "1z3xq9zmw1syznfhqzoeaoppbx2xsowvqscgnowuin7xkxk5fy7c";
$myPullRequestId = 2;

$reviewer = Get-AzureDevopsPullRequestReviewer -token $myReviewerPAT -organization $myOrganization -projectId $myProjectId -repositoryId $myRepositoryId -pullRequestId $myPullRequestId | ? uniqueId -eq $myReviewerId;

$reviewer.votes = 10;

Approve-AzureDevOpsPullRequest -token $myReviewerPAT -organization $myOrganization -projectId $myProjectId -repositoryId $myRepositoryId -pullRequestId $myPullRequestId -reviewer $reviewer;

【讨论】:

太棒了!感谢您在这里分享您的解决方案,您可以接受它作为答案!

以上是关于如何让审阅者在功能测试完成时自动批准拉取请求?的主要内容,如果未能解决你的问题,请参考以下文章

如何在BitBucket上限制pull请求的作者的合并权限

Azure Devops - 当我需要审阅者时批准自己的 PR

Git-TFS - 如何在拉取请求被批准之前确保分支是最新的?

GitLab 12.0发布,大力加强安全功能,包括可视审阅和依赖列表

批准合并请求时如何压缩提交,gitlab

如何使用“合并前需要拉取请求审查”来保护分支