Skip to content
代码片段 群组 项目

Let the central pipeline participate in merge judgment

已合并 路志远 requested to merge feat-mono-mvc2-pipeline into main-jh

What does this MR do and why?

Close #4484 (closed)

Mono 功能开启后,如果项目设置了 流水线必须成功,GitLab 只会查找当前项目的普通流水线,而不会考虑 主流水线

当前 MR 优化上述逻辑:在检查 MR 能否合并时,把 主流水线 也当做自己的流水线来参与判断。

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before After

How to set up and validate locally

  1. 全局开启 Mono 的 FF:ff_monorepo
  2. 在项目中设置 流水线必须成功
  3. 创建一个没有流水线的 MR,可以看到 MR 不允许被合并
  4. 为 MR 添加 monorepo 的 Label,并且创建一个 主流水线
  5. 可以看到 MR 允许合并了。
路志远 编辑于

合并请求报告

Merged results pipeline #3376112 已通过但有警告

Merged results pipeline 已通过但有警告 使用提交 616868d1

测试覆盖率 62.59% 来自1个作业

合并者: 路志远路志远 5个月前 (9月 26, 2024 7:57上午 UTC)

Loading

流水线 #3378792 已通过但有警告

流水线 已通过但有警告 使用提交 046cd0f9main-jh

4 环境受到影响。

动态

过滤活动
  • 批准
  • 指派人和审核者
  • 评论(来自机器人)
  • 评论(来自用户)
  • 提交和分支
  • 编辑
  • 标记
  • 锁定状态
  • 提及
  • 合并请求状态
  • 跟踪
  • :wave: @luzhiyuan.deer,请添加类型标记:

    • typebug:已发布或上线的产品中所出现的缺陷。这种类型的 issue 需在 QA 验证过后才可以关闭,而下面其它类型的 issue 在相关的 merge request (MR) 合并后即可关闭。
    • typefeature:对于用户可见的新功能或对现有功能的改进。
    • typemaintenance:非用户可见的产品改进,如:解决之前的技术债。

    或者你也可以直接添加子类型标记(例如 featureenhancement),我们会自动补全上述标记。要详细了解标记,请查阅文档:极狐 Handbook

  • 路志远 marked this merge request as draft

    marked this merge request as draft

  • 5 Warnings
    :warning: 88451a7c: 此提交的 主题 必须包含 3 个单词. 更多信息请参见提交信息指南
    :warning: 请为此合并请求添加一个合适的子类型标记
    :warning: 请为此合并请求添加一个合并请求类型
    :warning: 此合并请求需要有 backend 标记。请联系审核者或维护者添加这些标记。
    :warning: 此合并请求没有对照到任何里程碑。
    1 Message
    :book: 更新日志缺失:

    若您希望为极狐Gitlab 开源版本创建更新日志,请在提交信息添加 Changelog 结尾来将其添加到更新日志。

    若您希望为极狐Gitlab 商业版本创建更新日志,请在上述规则的基础上额外添加 EE: true 到提交信息末尾。

    若此合并请求不需要更新日志,请忽略此条消息。

    审核者推荐 (JH-Beta)

    已检测到需要审核的改动!

    请参考以下 Danger 建议的审核者和维护者:

    改动分类 审核者 维护者
    backend xiaofeng (@xfyuan) Mathew (@chaomao)

    代码审核步骤:

    1. 在合并请求准备好(非 Draft 状态)后,设置一位或多位审核者(参考上方表格的"审核者")。
    2. 第一次得到审核者批准时会进入安全审核流程,请等待安全审核通过。
    3. 在得到所有审核者批准后,设置一位项目维护者做最终审核并完成代码合并(参考上方表格的"维护者")。

    更多有关代码审核的流程请参见代码审核流程

    为了更平均地分配审核请求到所有可用的审查者,Danger 会基于审核者的状态和其他信息为每个席位挑选审核者。 如果有更好的审核者人选,请放心地忽略此算法推荐的人选。

    决定好审核者的人选之后,请将他们指派为审核者或者@提醒他们! Danger Bot 不会帮您做这件事。

    如有需要,您可以重试生成这条评论的:repeat: danger-review 作业

    Generated by :no_entry_sign: Danger

    **** 已编辑 于
  • 路志远 added 1 commit

    added 1 commit

    Compare with previous version

  • 路志远 changed the description

    changed the description

  • 路志远 added 2 commits

    added 2 commits

    • 1b5b0ebb - Rename param "filter_by_precise_time_range"
    • a3a9c0e2 - Add spec for mergeable_ci_state

    Compare with previous version

  • 路志远 changed title from Draft: Override MergeRequest#mergeable_ci_state? to Draft: Let the central pipeline participate in merge judgment

    changed title from Draft: Override MergeRequest#mergeable_ci_state? to Draft: Let the central pipeline participate in merge judgment

  • 路志远 changed the description

    changed the description

  • 路志远 marked this merge request as ready

    marked this merge request as ready

  • 路志远 requested review from @xfyuan

    requested review from @xfyuan

  • 路志远 enabled an automatic merge when all merge checks for a3a9c0e2 pass

    enabled an automatic merge when all merge checks for a3a9c0e2 pass

  • xfyuan approved this merge request

    approved this merge request

  • merged

  • 路志远 mentioned in commit 046cd0f9

    mentioned in commit 046cd0f9

  • 此合并请求已被合并,该合并请求的

    • 时间跨度为:2 天 3 小时 2 分钟
    • 流水线相关信息为: 在此合并请求中,一共有 3 条相关的流水线。其中运行成功的流水线数量为 2, 运行失败的流水线数量为 1。相关的 Job 数量为 225,其中成功的 Job 数量为 212,失败的 Job 数量为 4,跳过的 Job 数量为 0。

    以下是失败的 Job 的统计情况:

    失败的 Job 的名称 失败次数
    lint : code_quality 2
    lint : static-verification-with-database 1
    lint : static-analysis 2/2 1

    此合并请求没有重试的 Job!

  • :wave: @luzhiyuan.deer 我已自动根据此合并请求的发布分支 (GitLab 项目的合并请求) 或合并时间 (非 GitLab 项目的合并请求) 为此合并请求自动设置了里程碑 17.5,请您手动检查该合并请求对应议题的里程碑是否需要修改,谢谢!若您不希望我自动为您修改该合并请求的里程碑,请为此合并请求加上 milestone fixed 标签。

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
注册登录再回复
加载中