谈谈代码分支管理

前言

从2019年上半年云音乐的客户端团队开始迁移到双周迭代后,随之而来的是我们需要重新调整代码分支的管理方法,来应对开发流程的变更。

双周迭代顾名思义一周开发一周测试,目的就是为了快速交付。纵观整个开发流程,我们需要在两周内完成:需求交互评审-技术方案设计-开发测试-CodeReview-持续集成-灰度全量发布等工作,并且把每个节点的时间都固定下来,节奏感对于开发来说特别重要,让大家明确每个时间点应该做什么,对于简化管理和提高效率尤为重要。那新的代码分支管理如何适应这种节奏呢?

先谈谈现状

业内有很多的分支管理方法,包括著名的GitFlow,TBD以及从他们衍生出来的版本。

GitFlow

GitFlow设计非常全面,考虑到了实际开发过程中的各种问题,都能稳定应对,是非常适合大型项目的代码管理。他的主要理念包括2个主干分支:Master和Develop,1个发布分支:Release,若干个开发分支:Feature,以及HotFix分支。整体实施流程略微复杂,对持续集成不太友好,下面是典型的GitFlow模型:

GitFlow有几个问题:

因为大家都在自己的Feature分支上面开发,特别是对于长期的Feature分支会可能存在严重的合并冲突问题,需要花费大量的时间解决

同样是基于第一条,导致Feature分支之间相互隔离,以至于CI/CD困难,单个功能可能没有问题,但合并以后是否也没问题谁也无法保证

TBD

Trunk Based Development早在svn时代就已经流行,是种比较简单的分支管理模型,他只有一个主干分支,每个人写完代码自测通过后就往主干上面Push,要发布的时候就拉个Release分支,对持续集成友好。他主要的几个问题:

如果临上线前发现有问题,剔除代码比较困难

即使没有需求变更要下掉代码,有些提交如果未能达到上线标准,需要加Feature Toggle来控制打开关闭。这是一个需要平衡的事情,往往会增加一定开发成本和风险。

Aone Flow

这是一篇阿里技术的博客里面提到的他们内部采用的代码管理方式,区别于GitFlow,核心点在于没有固定Develop分支,要发布的时候把要上线的Feature分支一起合并到一个Release分支,这样做的好处是需求变更哪个Feature不要了,只需要把其他Feature分支重新合并到新的Release分支即可

云音乐代码分支管理

那云音乐需要怎么样的代码分支管理方法?我们适应目前双周迭代的分支管理模型应该要满足这几点:

够敏捷但又管理可控

云音乐迭代了这些年后版本相对趋于稳定,对于一个千万级体量的中大型APP来说,稳定,风险可控是一个基本诉求。而在这前提下面如何保持一定的灵活度是需要一直探索和寻找的平衡点

双周迭代的重要特性通俗来讲就是“上下车”制度,或者叫赶班车,也即能灵活应对突发情况随时将一个需求挪到下个版本上线,或者将一个需求挪到这个版本上线,这就意味着Release分支的合入也需要非常灵活,这一点跟Aone Flow非常像。

流程稳定,认知清晰

所有开发同学都非常清晰双周迭代的节奏,也要非常清晰代码管理的方法,每个迭代周期往哪个上线分支合并,什么节点合入都是非常明确的。避免出现忘记合入不知道往哪里合入的问题,同时我们也希望把这部分合并工作能够分摊到每个开发,做到自己分支自己负责。

CodeReview是必需的

相信大家都能认可CodeReview的意义,最佳做CodeReview的时机往往是代码合入点,通过PR或者MR来发起

4. 方便持续集成流水线

传统方式因为大家都提交在自己分支上面,代码分散同时存在合并后产生额外风险,我们期望尽可能尽快把Feature分支集合起来走流水线

那为了解决这几个问题,我们需要几个分支:

稳定的发布分支,统一大家认知确保在固定合入时机点前发起MR进行合入,我们固定叫release-xxxx分支,在下个迭代开始时从前一个Release分支拉出

稳定的主干分支(Master),只是作为HotFix分支拉取,当然其实HotFix也可以从已发布的Release分支或者Tag里面拉出,Master的意义在于确保一条可以随时发布的干净的版本记录

若干开发分支,这个Feature分支和GitFlow,AoneFlow没有差别,在合入Release分支的时候进行CodeReview,我们固定叫feature-xxx

HotFix分支,这个没有差别

所以整体来看我们的最佳实践会是GitFlow和AoneFlow的结合体:

1. 首先是把GitFlow的Develop和Release分支进行合并,减少复杂度。固定每个迭代的Release分支名字方便认知,对于客户端来说Release没有环境之分,不需要有多个Release分支

2.  不允许在Release上面提交代码,所有功能开发都走Feature分支,即便合并后有bugfix,仍然走Feature修改再次合入,这样确保所有代码合入都是测试完成且CodeReview完成。同时也具备AoneFlow的优点,一旦需求有要下车,则直接删掉Release分支进行重建,把其他Feature分支合入即可。

3.  因为固定合入时间节点,所以持续集成的时间也是固定的,我们一般是定在第二周的周二,虽然没有第一时间像TBD一样进行,但已经是我们想要找到的一个合理平衡点。当然Feature分支上面也可以有流水线,跑的东西略微有些差别(Feature分支跑的东西偏测试环境)

4. Master分支随时处于Release状态,确保HotFix能够及时进行

其他

除了约定,云音乐内部还有个叫《全链路研发平台》的工具来辅助代码分支管理,这个平台涵盖了从需求产生到交付的全生命周期管理,包括需求排期,提测,打包,持续集成,卡点,发布等环节。和代码分支管理相关的主要有这些:

每个需求都会要求关联一个Feature分支,否则无法提测

每个Feature分支是否合入当前迭代固定的Release分支状态可查询,方便管理版本上线前卡点

最后,没有一个代码分支管理方法是直接适用自己的项目场景的,还是要根据实际情况进行一些取舍,找到一个最佳的平衡点。

更多文章请关注微信公众号:安卓之美

©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 206,968评论 6 482
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 88,601评论 2 382
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 153,220评论 0 344
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 55,416评论 1 279
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 64,425评论 5 374
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 49,144评论 1 285
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 38,432评论 3 401
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 37,088评论 0 261
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 43,586评论 1 300
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 36,028评论 2 325
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 38,137评论 1 334
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 33,783评论 4 324
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 39,343评论 3 307
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 30,333评论 0 19
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 31,559评论 1 262
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 45,595评论 2 355
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 42,901评论 2 345

推荐阅读更多精彩内容