代码百分百评审部署方案_-_网络分享.pptVIP

代码百分百评审部署方案 Wang Dongliang 2012-11-2 目录 工具支撑 制度规范 监管审计 风险评估 实行效果 工具支撑（一） Review Board在审查补丁（Patch）方面表现完美。一个叫做“提交审查”的工具被用来和SCM系统进行连接（当前支持SVN、CVS、Perforce、Git和Mercurial等），可以允许你请求一个将被提交的修改的审查。用户基础页面将这个过程描述如下： 你在本地检出的代码上做了些可怕的修改。 你通过公布diff、编写描述和选择一些审查者来创建一个审查请求。 你在审查请求中点击“发布”并等待你的审查者看到它。 其他人看了你的审查请求，说道“这太可怕了，除非什么东西坏掉了。” 你根据他们的评论更新了你的代码。 你公布了更新后的diff，以及对他们评论的解答以指明你修改了什么（或者你要说明为什么你不打算按照他们的建议修改代码）。 大家看了你更新后的代码，并请你继续。 你将修改提交到仓库中。 你在审查请求中点击“设置为已提交”来从其他人的面板中移除你的审查请求。 工具支撑（二）  工具支撑（三） ReviewBoard部署（仅授权评审专家组有SUBMITTED权限） 允许远程访问数据库MYSQL vim /etc/mysql/my.cnf 修改bind-address = 0.0.0.0(或指定IP) 允许使用某账号远程访问数据库 #mysql -uuser -ppasswd mysqlGRANT ALL PRIVILEGES ON *.* TO ‘user’@‘%(或指定IP)?IDENTIFIED BY ‘passwd WITH GRANT OPTION; mysqlFLUSH RIVILEGES; 工具支撑（四） SVN勾子部署 sudo aptitude install mysql-client 启动cd xxx/xxx/hooks mv pre-commit.tml pre-commit 增加日志格式要求vim pre-commit；如下： REPOS=$1 TXN=$2 # Make sure that the log message contains some text. SVNLOOK=/usr/bin/svnlook $SVNLOOK log -t $TXN $REPOS | \ grep [a-zA-Z0-9] | grep -E MODIFY|ADD|DEL|modify|add|del|Modify|Add|Del | grep -E REVIEWID|reviewid|Reviewid /dev/null || exit 1 工具支撑（五） SVN勾子部署 验证review状态vim pre-commit；如下： # Make review,reviewid is not submiited,that error REVIEWID=`$SVNLOOK log -t $TXN $REPOS | awk -Freviewid: {print $2}` REVIEWIP=“xxx.xx.xx.xxx REVIEWUSER=“xxx REVIEWPASS=“xxx REVIEWDB=“xxx REVIEWTAB=reviews_reviewrequest COUNT=`mysql -h$REVIEWIP -u$REVIEWUSER -p$REVIEWPASS -D$REVIEWDB -e select * from $REVIEWTAB where id=$REVIEWID and status=S; | wc -l` [ $COUNT != 0 ] || exit 1 注释#$REPOS/hooks/commit-access-control.pl $REPOS $TXN \ 注释# $REPOS/hooks/commit-access-control.cfg 制度规范 代码编写规范（定期考核） 危险函数列表及常用代码使用说明 代码上传日志格式（规范上传日志） 评审专家组（多级别） 评审级别划分（按模块） 代码评审责任奖惩制度（增加评审质量） 监管审计 设立监管审计职责 按周、月通报，按季兑现责任奖惩 年底KPI考核 定期审计项 风险评估 一个REVIEWID重复使用 ReviewBoard搜索功能较差，增加责任奖励统计工作量 未及时评审，影响上传代码发布版本 实行效果 实现检入SVN库代码100%被评审 检入SVN库的代码质量提高 成立专家组进行评审拉齐技术水平差距 代码事故责任可追溯 降低致命和低级等错误的发生 评审过程可分享经验 谢谢！