gitlab ci 如何在提交到 master 分支前进行验证，通过验证才能 merge 或 push
css3 · 2020-08-27 11:09:02 +08:00 · 6987 次点击这是一个创建于 1553 天前的主题，其中的信息可能已经有所发展或是发生改变。
请教下各位大佬：
- 写了个验证脚本，放到代码工程中，并且将执行任务加到 ci stage 里边了
- 但现有的 ci 执行顺序是大家先 push 或 merge 然后才进入 stage 自动流程，然后才走的脚本验证流程，所以这个验证功能并没有什么用，验证未通过时，也还是合到 master 上了
- 怎么才能在 merge 或 push 前验证出来结果呢？验证不过，就拒绝提交 master
第 1 条附言 · 2020-08-27 11:50:33 +08:00
😂😂突发奇想，再加一个 git 回滚脚本，stage 里边的验证流程失败后，直接执行 git 回滚 stage 任务对本次提交进行回滚。不知道是否可行
 |
1
virusdefender 2020-08-27 11:10:58 +08:00
ci 中加一项跑你的验证，依赖 staging 那一步，跑完了整个 ci 才结束
|
 |
2
css3 OP @virusdefender
加了啊，问题在于，现有顺序是先提交后才触发执行 ci 。应该要先执行 ci,验证结果通过后才能 push 或 merge |
 |
3
Chingim 2020-08-27 11:18:47 +08:00
Pipelines for Merge Requests
https://docs.gitlab.com/ee/ci/merge_request_pipelines/ |
 |
4
maichael 2020-08-27 11:19:51 +08:00
锁分支，不给直接推送，然后手动在界面上提交合并请求
|
 |
5
AngryPanda 2020-08-27 11:21:27 +08:00
走 MR，不要给权限直接客户端操作
|
|
6
virusdefender 2020-08-27 11:31:20 +08:00
@css3 你们直接 push master ? 应该用 merge request
|
 |
7
NauxLiu 2020-08-27 11:34:01 +08:00
1. 将 master 设置为 protect branch，不允许直接 Push
2. 打开项目的 Merge Request 设置里的 Pipelines must succeed |
|
8
css3 OP @virusdefender 有 MR，但大家不遵守规范，一股脑提过来，要人工检查，索性就写了个脚本自动验证，这不是没按预计想的那样运行么😂😂
|
 |
9
wxsm 2020-08-27 11:42:02 +08:00 via iPhone
我刚好也在搞这个。目前在尝试 gitlab 前面再对接一个 gerrit
|
|
10
wxsm 2020-08-27 11:45:46 +08:00 via iPhone
但是不太顺利，怎么说呢，gerrit 能 push 的，人也能，人不能 push 的，gerrit 也不能。ce 没有评审功能很蛋疼。
|
 |
14
taogen 2020-08-27 12:04:20 +08:00
The branch you are building and testing on is only for building and testing.
On your dev branch if any tests fail you just fix it and push it again. |
 |
15
skyadmin 2020-08-27 12:09:24 +08:00
master 分支设为保护分支，不允许任何人，直接推送代码至 master 分支。就可以让大家必须都在 merge request 了。
|
|
16
skyadmin 2020-08-27 12:10:09 +08:00
项目的 settings - repository - Protected Branches 里设
|
|
17
skyadmin 2020-08-27 12:14:42 +08:00
然后勾选 settings - General - Merge request，勾选 Only allow merge requests to be merged if the pipeline succeeds 。
和#7 说的一样，只是给了下具体操作路径。 |
 |
18
1109599636 2020-08-27 12:31:20 +08:00
我的理解是个人提交到个人分支，然后个人分支跑 ci 各种流程，ci 通过了才可以 merge 到公共分支吧。如果在 commit 之前就做检查可以用 pre-commit hooks
|
 |
19
StarUDream 2020-08-27 12:34:54 +08:00
目前用的就是 15-17 楼所说的方法，目测可以解决楼主问题。
|
 |
20
594duck 2020-08-27 12:41:58 +08:00
|
 |
21
nightwitch 2020-08-27 12:57:37 +08:00
禁止提交到 master，要提交到 master 发 pull request 。发 pull request 的时候自动触发 CI 测试，CI 测试不过不允许合并
|
 |
22
zmxnv123 2020-08-27 13:04:15 +08:00
git hook?
|
 |
23
Rwing 2020-08-27 13:09:47 +08:00
@nightwitch 这个是正解
|
 |
24
Sapp 2020-08-27 13:43:38 +08:00
coommit 可以添加钩子啊，前端就有个叫哈士奇的，不通过 commit 都没办法提交
|
 |
25
phy25 2020-08-27 14:21:43 +08:00 via Android
比较优雅的办法，自动提交 MR：
https://docs.gitlab.com/ee/user/project/merge_requests/merge_when_pipeline_succeeds.html https://docs.gitlab.com/ee/user/project/push_options.html git push -o merge_request.create -o merge_request.target=my-target-branch -o merge_request.merge_when_pipeline_succeeds |
 |
26
janxin 2020-08-27 15:21:35 +08:00
禁止 master 分支直接提交
设置 Protected Branches 合并 MR 需要在 pipelines 执行成功 设置 Only allow merge requests to be merged if the pipeline succeeds |
 |
27
maltoze 2020-08-27 15:26:40 +08:00
|
 |
28
gen900 2020-08-27 15:33:51 +08:00 via iPhone
gitlab 里 没有通过 pipeline 的 merge request 不让合并的。就是你 push 后就在做 ci 了如果失败 mr 不让合并
|
 |
29
wshcdr 2020-08-27 15:38:06 +08:00
同意 MR 的做法
|
 |
30
otakustay 2020-08-27 20:12:44 +08:00
按照楼主的要求，其实你一开始就应该用 gerrit 而不是 gitlab
|
 |
31
dddd1919 2020-08-27 23:10:20 +08:00
提 git 的人不按规范执行，总有方法绕过你的设置，不统一提交规范，其他的就别扯了
用 git 靠的是约定，不是靠各种限制，如果大家都不遵守约定，那就找人专门来干这事，或者换个工具吧。 统一 git flow，再用 ci 才能达到比较好的效果，比如配置 ci 在 feature 上执行，禁用 remote push master，合并 mr 前要 ci 通过，ci 同时配置 build 和 run test 等等 |
 |
32
icylogic 2020-08-28 06:55:19 +08:00 via iPhone
我们 master/develop 任何人都不能推的，所有更改都要通过其他分支提 mr，mr 只能由 reviewer 点合并，如果 ci pipeline 挂掉或者有 conflict，合并按钮都是灰的没法点（可以选项里改）
|
|
33
wxsm 2020-09-23 15:27:45 +08:00
我通过 gitlab 的 webhook 和 api 自己实现了一套评审系统，详见： https://wxsm.space/posts/2020-09-23-gitlab-ce-code-review-bot.html
|
 |
34
dengxuejiu 2022-01-12 16:07:41 +08:00
亲测可用，在 merge 之前做代码规范检查等工作还是挺好用的
|
Select Language