赞
踩
Access controls in Gerrit are group based. Every user account is a member of one or more groups, and access and privileges are granted to those groups. Access rights cannot be granted to individual users.
在gerrit中权限控制是基于群组的. 每个用户有一个或者多个群组, 访问权限被赋予这些群组.访问权限不能赋予个人用户.
在Gerrit系统
自带下面的群组
所有用户都是匿名用户
成员, 所有用户都能继承Anonymous Users
所有访问权限.
当前只有Read access
权限值得赋予给Anonymous Users
群组, 因为其他权限都需要认证.
Project Owners
的访问权限在Project
范围内有效
Change Owner
的访问权限在Change
范围内有效
所有在页面上登录成功的用户都会自动注册为gerrit用户,属于Registered Users
群组
Registered Users
群组通常被赋予Code-Review -1..+1
权限, 允许给需要审查代码投票, 但不会引起审查被批准和拒绝
system groups
在Gerrit系统内部就定义好了, 而普通群组信息被保存在ACCOUNT_GROUPS
表中,Predefined groups
群组信息也保存在ACCOUNT_GROUPS
表中. Predefined groups
群组在Gerrit初始化时创建并且拥有唯一的UUID
值
All-Projects -> meta/config -> groups
文件内容
# UUID Group Name
#
5210215f92225c45a5ad123016c8706336f55a7d Administrators
df9b717d413c614cb51e39525619b311f077ec15 Non-Interactive Users
global:Anonymous-Users Anonymous Users
global:Project-Owners Project Owners
global:Registered-Users Registered Users
Gerrit
自带两个predefined groups
:
Administrators
是Gerrit root
角色, 在Gerrit初始化时Administrate Server
权限被赋予给这个Predefined Groups
群组.
在Administrators
组的成员可以管理所有项目, 但是不意味着任何其他权限. Administrators
组不会自动获得代码审查批准和提交权限.
Interactive Users
比如在web页面上审查代码, 在提交/获取代码的用户
Non-Interactive Users
是可以通过Gerrit接口进行操作的组, 在Gerrit初始化时Priority BATCH
和Stream Events
权限被赋予给这个Predefined Groups
组.
Non-Interactive Users
和Interactive Users
使用不同的线程池, 防止交互式用户抢占线程. 当系统资源紧张时确保了交互式的用户可以继续工作.
在All Projects
项目中的访问权限会自动被其他项目继承, 只有Administrate Server capability
能够编辑All-Projects
权限.
先计算子项目的访问权限, 再计算All Projects
的访问权限, 允许一些权限可以被覆盖.
对一个群组赋予DENY
限制时, 通常只对READ
权限有效.
refs/heads/*
和refs/tags/*
是Git常用的引用命名空间, 一个用来存储分支一个用来标签
在refs/*
命名空间下的引用都是有效的,Gerrit在refs/*
有一些特殊用处的命名空间和引用
这些特殊的引用的内容由Gerrit生成或者包含重要的项目配置信息
refs/changes/* #用于存储审查的补丁
#获取某个补丁集需要审查序号和补丁集序号
#'refs/changes/'<last two digits of change number>/ <change number>/ <patch set number>
refs/meta/config #项目配置的分支
refs/meta/dashboards/* #
refs/notes/review #保存代码审查信息的分支
refs/for/<branch ref> #进行代码审查时需要提交代码到这个命名空间
refs/publish/* #和refs/for/*命名空间作用一样
refs/drafts/* #用于草案代码审查,和refs/for/*的区别在于只有部分人可见
#审查者对代码进行修改, 审查的代码是修改后的补丁
git fetch ssh://admin@localhost:29418/gerrit_ci refs/changes/03/3/1
git branch fix_xxx FETCH_HEAD && git checkout fix_xxx
vi README
git add README
git commit --amend
git push origin HEAD:refs/for/master
git ls-remote ssh://admin@localhost:29418/gerrit_ci
61fd289472707d79f73289216a4c5f0ca4cee4e1 HEAD
eeaef9da4ea27d7c23bfb5f9a2ed1b5357ebbba8 refs/changes/01/1/1
5f8ed98b0f88787c22e705595e2818db62874f56 refs/changes/02/2/1
bfdb700f4aab3afc32ec79a29b0e25f8be758f8f refs/changes/03/3/1
effa7b004eec0b85e722fe10be6468e4ed9b78d3 refs/changes/03/3/2
61fd289472707d79f73289216a4c5f0ca4cee4e1 refs/heads/master
9f282c08d5108c6817dd1504e8bec0e94ba59d47 refs/meta/config
405030285eed7406b1ac7cfa6a5211331165b8e2 refs/notes/review
git clone ssh://admin@localhost:29418/All-Projects && scp -p -P 29418 admin@localhost:hooks/commit-msg All-Projects/.git/hooks/
#refs/meta/config 是All-Projects的引用
git fetch origin refs/meta/config:refs/remotes/origin/meta/config
git checkout meta/config
#现在目录下有groups project.config两个文件
#提交修改
git add .
git commit -m "modify config"
git push origin meta/config:meta/config
代码审查时允许用户丢弃这个审查。如果对change
有push
权限,同时具有push
,abandon
,restore
权限
用户可以创建新的references
, branches or tags
, 创建时引用必须不存在,不能删除已经创建的引用
如果仅仅推送标签,给refs/tags/*
赋予Create Reference
权限
这个权限通常用在创建某个命名空间下的分支, 如:某个部门自由创建分支权限refs/heads/hello/*
给某用户自由创建分支权限, 给refs/heads/sandbox/${username}/*
赋予Create Reference
权限
如果你这样赋予Create Reference
权限,记得同时赋予push force
权限, 这样拥有清理
查看提交中Author
和Committer
git log --format=full
commit 2dfae738781a3ba641ee06c913fd51162335a941
Author: admin <c2290910211@163.com>
Commit: gerrit_test <c2290910211@aliyun.com>
admin gerrit_test
Change-Id: I0830cf061306101e977f9adf55270c9b3a3f59c4
Author
一般表示谁创建了这个提交,也可以用git commit --amend --reset-author
等命令修改
Committer
一般表示谁修改了这个提交,在使用git commit --amend
等命令时修改
通常Gerrit需要在Author
和提交的Committer
认证信息中至少一个,与uploading user
注册过的邮箱地址匹配,Forge Author
和Forge Committer
允许用户绕过提交时的身份验证
Forge Author
允许提交中Author
信息不经过验证, 这个权限在下面场景非常有效,通过邮件接收第三方补丁,cherry-pick
其他人的分支提交,审查合并前修改其他人的一些次要问题.
默认在All-Projects
赋予Registered Users
组Forge Author
权限.
Forge Committer
允许提交中Committer
信息不经过验证和不验证匿名标签对象,通常在需要其他服务器自动提交时有用
Forge Server
允许在提交中Committer
信息使用Server
的用户名和邮箱. 这个权限在强制推送git filter-branch
修改过信息的提交和由这个Gerrit Code Review server
创建的合并提交时有用.
在etc/gerrit.config
可以配置Server
的用户名和邮箱,user.name
默认值Gerrit Code Review
, user.email
默认值在启动时生成gerrit@hostname
允许修改项目以下配置:
Owner
权限子命名空间的所有权可以通过命名空间格式来委派. 要委派以qa/
开始的所有分支权限给QA群组,给refs/heads/qa/*
添加Owner
权限。 QA小组的成员可以进一步细分访问权限,但只能对refs/heads/qa/
开始的分支有效.
这类分支控制用户如何上传提交到Gerrit. 根据命名空间赋予的Push
权限, 可以允许绕过代码审查直接推送到分支, 也可以允许创建新的change
进行代码审查.
在Direct Push
权限下,任何已经存在的分支都接收fast-forward
提交,创建新的分支需要Create Reference
权限. 删除已经存在的分支会被拒绝,因为在这个最安全的模式下, 提交不会被丢弃.
允许分支被删除. 由于force push
实际上删除分支后会创建这个分支,但是这是个原子操作并且会被记录,也允许force push
更新分支. 带有force
选项会导致项目历史中的提交被丢弃.
force
选项对只想使用Gerrit访问权限功能而不需要代码审查的项目有效, 对于需要进行代码审查的项目不应该分配这个权限.
Upload To Code Review
权限授权在refs/for/refs/heads/BRANCH
命名空间上,允许用户上传non-merge
提交到refs/for/BRANCH
命名空间,创建新的change
进行代码审查.
用户在自己环境下提交代码需要clone/fetch
项目代码,所以必须赋予Read
权限
对于开源公开的Gerrit安装方式,All-Projects
中refs/for/refs/heads/*
通常给Registered Users
赋予Read
和Push
权限. 很多私有安装, 通常refs/for/refs/heads/*
通常给all users
简单的赋予Read
和Push
权限.
force
选项赋予refs/for/refs/heads/*
命名空间没有作用.
Push Merge Commits
权限允许用户上传merge commits
.这是Push
附加的访问权限,所以只赋予Push Merge Commits
权限是不够的. 一些项目希望只能由Gerrit自动合并提交, 可以通过只赋予Push
权限而不赋予Push Merge Commit
权限.
赋予Push Merge Commit
权限通常必须以refs/for/
为前缀,例如refs/for/refs/heads/BRANCH
. refs/heads/BRANCH
作为补充, 赋予Read
权限后允许直接推送non-merge
提交,赋予Push Merge Commit
权限后也允许直接提交一个merge
提交.
允许推送带注释的标签
到远程仓库, 标签必须带有注释
.
#创建带注释的标签
git tag -a <tagname> -m <comment>
#提交标签到远程仓库
git push origin --tags
#获取标签
git fetch --tags
标签的email
一定会与提交者的邮箱进行验证,如果推送其他人的标签需要同时赋予Push Annotated Tag
和Forge Committer Identity
权限.
git show v1.0
tag v1.0
Tagger: username <email>
如果需要推送轻量级标签
(不带注释
), 给refs/tags/*
命名空间赋予Create Reference
权限, 轻量级标签就像Git中的分支一样.
如果需要删除或者重写标签, 给refs/tags/*
命名空间赋予带force选项的Push
权限,删除标签需要和删除分支一样的权限.
允许推送PGP签证
的标签到远程仓库
git tag -u "gpg-key-id" -m "tag comment" <tagname>
Read
权限控制查看项目的change
,comment
,code diff
和通过SSH或者HTTP协议访问仓库的权限
这个权限非常特殊, 先计算项目中的Read
权限, 再计算all-projects
的Read
权限.
如果项目中赋予DENY Read
权限,all-projects
项目不管是否赋予ALLOW READ
都将无效.这个权限对于隐藏一些项目非常有用.
允许用户在web页面上进行rebase changes
, change
作者和提交者可以通过页面进行rebase changes
(尽管没有赋予Rebase
权限)
允许用户从审查者列表中移除其他用户.
Change owners
能够移除那些审查分数为0或者负数的审查者.(尽管没有赋予Remove Reviewer
权限)
Project owners
和site administrators
能够移除任何审查者(尽管没有赋予Remove Reviewer
权限)
其他用户只能将他们自己从审查者列表中移除.
在项目中配置label My-Name
,label My-Name
和定义好的范围分数相关联. 同时也关联labelAs-My-Name
权限, 可以允许编辑用户定义的label
.
Gerrit带有配置好的Code-Review
标签
[access "refs/heads/*"]
label-Code-Review = -2..+2 group Administrators
label-Code-Review = -2..+2 group Project Owners
label-Code-Review = -1..+1 group Registered Users
[label "Code-Review"]
function = MaxWithBlock
defaultValue = 0
copyMinScore = true
copyAllScoresOnTrivialRebase = true
value = -2 This shall not be merged
value = -1 I would prefer this is not merged as is
value = 0 No score
value = +1 Looks good to me, but someone else must approve
value = +2 Looks good to me, approved
允许用户提交审查通过的提交. 提交审查代码后会合并到对应的分支上.
The label that the reviewer selects determines what can happen next. The +1 and -1 level are just an opinion where as the +2 and -2 levels are allowing or blocking the change. In order for a change to be accepted it must have at least one +2 and no -2 votes. Although these are numeric values, they in no way accumulate; two +1s do not equate to a +2.
Code-Review
标签+2
是通过,-2
是否定, -1~+1
只是代表意见并不会影响投票, 审查被通过至少需要一个+2
投票并且没有-2
投票. 两个+1
并不会等于+2
[access "refs/heads/*"]
submit = group Administrators
submit = group Project Owners
允许用户查看其他人提交的draft changes
.
draft changes
作者和添加为审查者都能看见draft changes
(尽管没有赋予View Drafts
权限)
允许用户公开其他人提交的draft changes
.
draft changes
作者可以公开draft changes
(尽管没有赋予Publish Drafts
权限)
允许用户删除其他人提交的draft changes
.
draft changes
作者可以删除draft changes
(尽管没有赋予Delete Drafts
权限)
允许修改change
的主题topic
change owner, branch owners, project owners, and site administrators
可以修改主题(尽管没有赋予Edit Topic Name
权限)
通过赋予一个群组Owner
访问权限在refs/*
命名空间, Gerrit管理员可以委派这个项目的访问控制权限给这个群组.
如果需要没有一个人能更新或者删除标签, 连项目owners
都没有权限. ALLOW
和DENY
规则并不能达到这样的目的, 因为项目owners
可以赋予任何他们自己想要的访问权限. 覆盖任何从All-Projects
或者其他父项目继承的权限是非常有效的方法.
在父项目中BLOCK
权限, 使得就算是子项目owners
也不能设置block
权限为ALLOW
. 尽管这样, 也应该保留owners
所有non-blocked
权限.
site wide
策略并且提供有意义的默认访问权限.site wide
策略的情况下, 项目owners
可以管理他们自己项目的访问权限.BLOCK
规则是全局范围的权限. 子项目不能重载继承的BLOCK
规则. 从父项目链表中搜索BLOCK
规则, 忽略在访问区域中的独占(Exclusive
)标志.
push
权限赋予BLOCK
规则, push
和force push
等推送都将被阻塞. force push
权限赋予BLOCK
规则只有force push
被阻塞, 但是如果push
权限具有ALLOW
规则的话可以进行non-forced
提交.
BLOCK
也可以赋予label
投票范围. 下面的配置可以阻塞group X
投+2
和-2
票,仍然可以投-1 ~ +1
的票.
[access "refs/heads/*"]
label-Code-Review = block -2..+2 group X
在这个阻塞规则min..max
范围的目的是: 阻塞-INFINITE..min
和max..INFINITE
投票,也意味着-1..+1
投票范围不受阻塞影响.
当相同访问区域同时包含BLOCK
和ALLOW
规则, ALLOW
规则会重载BLOCK
规则.
[access "refs/heads/*"]
push = block group X
push = group Y
如果群组X
和群组Y
都包含了同一个用户, 这个用户依然能够push
到refs/heads/*
命名空间.
在同一个项目的同一个访问区域,ALLOW规则才会重载
BLOCK规则.在同一个项目不同访问区域和子项目的同一个访问区域, ALLOW
规则不会重载BLOCK
规则.
BLOCK
规则示例在All-Projects
中阻塞Anonymous Users
组的push
权限
[access "refs/tags/*"]
push = block group Anonymous Users
由于所有人都是Anonymous Users
组成员, 所以可以阻塞所有人.
可能项目owner
需要创建tag
权限
[access "refs/tags/*"]
push = block group Anonymous Users
pushTag = group Project Owners
pushSignedTag = group Project Owners
假设提交到发布分支Release-Process
需要更为严格的过程, 假设我们需要确信只有Release Engineers
群组可以投票,就算是项目owner
权限也不可以投票. 在All-Projects
我们定义下面的规则.
[access "refs/heads/stable*"]
label-Release-Process = block -1..+1 group Anonymous Users
label-Release-Process = -1..+1 group Release Engineers
允许用户通过gsql
命令访问数据库, 以及包含代码审查信息的分支refs/notes/review
影响Gerrit中owner
和administrator
角色. 赋予administrateServer
能力能够赋予任何群组访问权限.
Gerrit中有两类线程池用来给INTERACTIVE Users
和Non-Interactive Users
使用,防止抢占线程.
允许其他用户使用Non-Interactive Users
的保留线程池.
INTERACTIVE
线程池Non-Interactive
线程池BATCH
选项,否则使用INTERACTIVE
线程池获取All-Projects
中refs/meta/config
分支并修改权限配置
git clone ssh://admin@localhost:29418/All-Projects && scp -p -P 29418 admin@localhost:hooks/commit-msg All-Projects/.git/hooks/
#refs/meta/config 是All-Projects的引用
git fetch origin refs/meta/config:refs/remotes/origin/meta/config
git checkout meta/config
#现在目录下有groups project.config两个文件
#groups文件包含project.config中需要的用户组和对应的UUID
#提交修改
git add .
git commit -m "modify config"
git push origin meta/config:meta/config
code review的目的是团队成员在一起共同学习,而不是相互”挑错”.将code review称为
代码回顾
好一些, 如果大家放弃”挑错”来共同学习,那么代码回顾中学习什么呢?
代码回顾的学习重点是团队成员共同识别模式,这里的模式是指程序员编写代码的习惯,包括”好模式”和”反模式”. 像富有表达力的命名, 单一职责的方法, 良好的格式缩进等,都是”好模式”. 团队成员通过阅读最近编写的测试代码和生产代码来识别”好模式”和”反模式”.既是团队成员之间相互学习的过程, 也是团队整体达成整洁代码共识的过程.
Copyright © 2003-2013 www.wpsshop.cn 版权所有,并保留所有权利。