如何向 Git 社区提交补丁代码

发表于 2年以前  | 总阅读数:588 次

关于 Git

Git 是目前世界上最为广泛使用的软件版本控制系统(Version Control System),同时也是一个成熟及活跃的开源项目。

Git 最初是由 Linux 之父 Linus Torvalds 在 2005 年创建,至今已经迭代了17年。但任何程序都会有Bug,对于 Git 这样一个成熟的开源项目也不例外。引用 Linus 的话来说:

“Bugs will happen, if they don’t happen in hardware, they will happen in software and if they don’t happen in your software and they will happen in somebody else’s software.” Torvalds said.

“错误总会发生,如果它们不发生在硬件中,它们将发生在软件中;如果它们不发生在你的软件中,也会发生在其他人的软件中。”

接下来,我们将从一个由 Git 引发的故障,一起来看看如何排查问题,并逐步进行 Git 社区贡献。

Git 问题排查

现场还原

这个问题最初在 2022.6.8 被发现,代码平台服务收到报警,收到了大量的 Git 的代码下载请求。在一番定位之后,发现在部分代码下载的服务器上,发现了下图的一连串 Git 进程:

从上图我们可以看到一个关键点,PID 与 PPID 首尾相连,这是一个典型的嵌套循环问题。

在这个循环当中,产生了大量的 Git 进程,不但对代码平台产生了压力,也对用户本地产生了极大的困扰(本地的进程资源被大量占用)。

所幸这个问题只要满足触发条件,就可以稳定复现。在获取了异常的仓库副本之后,我们就开始了问题剖析之旅。

问题剖析

首先,让我们一起分析下异常仓库的特点:

  1. .git/objects/info 目录下,存在 commit-graph 图文件。
  2. 仓库使用了 .git/objects/info/alternates 来引入一些外部的仓库。
  3. 使用了局部克隆(Partial Clone):在 .git/config 当中发现了一些痕迹。
[remote "origin"]
        promisor = true
        partialclonefilter = blob:none

为了不对代码平台产生异常调用,我们尝试在本地电脑上还原这个问题:

  1. 我们将异常仓库副本,解压到 work 目录下。
  2. 通过 git clone --bare {url} remote.git 我们下载了一个远程仓库。
  3. 通过 git -C work remote set-url origin "$(pwd)remote.git"work 的远程仓库设置成本地。

接下来,我们就可以通过执行异常命令来复现问题。

为了更好地跟踪,我们可以通过 GIT_TRACE=1 或者 GIT_TRACE2_EVENT="$(pwd)/event.trace" 来开启过程追踪。

为了更好地定位问题,我们也可以考虑修改 Git 源码,通过 trace_print 增加一些额外的调用日志。此外,我们还可以借助 LLDB / GDB 帮助我们进行本地观察与调试。

最终,在一番努力之后,一个可怕的死循环调用链逐渐浮出水面:

 git fetch
     -> do_fetch_pack_v2() 
         -> deref_without_lazy_fetch() 
             -> lookup_commit_in_graph() 
                 -> repo_has_object_file() 
                     -> promisor_remote_get_direct() 
             -> fetch_objects() 
                 -> git fetch (,我们又回到了最初相遇的地点,开始新的轮回)

原因分析

让我们就着调用链路,再来回顾一下原因:

  1. 由于本地是一个的局部克隆的仓库,存在一些对象缺失的情况,而访问这些缺失的对象,就可能产生懒加载拉取。
  2. 而在懒加载的过程中,引用查找 deref_without_lazy_fetch() 方法会尝试先去从 commit-graph 提交图当中寻找对象来进行一些加速。
  3. 而 lookup_commit_in_graph() 当中,使用了 repo_has_object_file() 来判断对象是否存在于 Git 仓库当中,这个方法,可能引发懒加载拉取本地缺失的对象,开启一轮新的git fetch
  4. 而新一轮的 git fetch ,由于本地存在 commit-graph,在引用查找中,再次进入了步骤2~3的循环当中。

这里有一个关键点,问题是由于提交记录存在于 commit-graph,而在 Git 对象仓库中缺失导致的。这种情况通常并不会发生。结合仓库使用了 objects/info/alternates 来引入外部仓库,我们来猜想一下问题是如何产生的。

猜想验证

如下图当中,通过 objects/info/alternates ,我们引入了一个外部的共享仓库,来减少当前仓库的对象存储。同时,我们创建了一个 commit-graph,来加速本地的提交记录访问。

由于共享的仓库与本地仓库维护的引用列表(分支、标签等统称为引用)并不相同,在触发 git gc或者自动 gc 之后,由于提交记录 B 是一个不可达的对象(不存在于任何引用当中),导致了存在于 commit-graph 当中的共享提交记录 B 被清理了。

这时候,只要在当前仓库触发了懒加载,就会引发前面的死循环调用链,产生大量的 Git 进程,最终拖垮用户本地及远端服务。

虽然从原因来看,这个问题的产生,某种意义上是由于不合理使用 alternates 及 commit-graph 导致的,但我们的确很难限制用户该如何使用。于是,我们决定寻求 Git 社区的帮助。

寻求 Git 社区帮助

我们决定,派你出发,开始这次的 Git 社区之旅。

芝麻开门

提到开源社区,大家都会不约而同想到Github[1],比较这可是全球最大的“程序员交友社区”。

可当你兴冲冲地跑到 https://github.com/git/git 准备提交一个 Issue (问题)时,嗯?说好的Issues 入口去哪了?

再看右边的描述:

虽然,你可以通过 Github 上的 GitGitGadget 机器人帮助我们进行补丁提交,可是你现在只是想请教社区一个问题,该怎么办呢?

曲径通幽

在 git-scm.com 上,通过阅读 《MyFirstContribution》[2],你找到了寻求帮助的三个路径:

1、git@vger.kernel.org这是主要的 Git 项目邮件列表,用于进行代码审查、版本公告、设计讨论等。

这也是目前 Git 社区最主要的交流方式,Github 上的 GitGitGadget 也是帮助你做邮件转换的工作。

任何有兴趣贡献的人,都在这里发布问题。

但需要注意的是,Git 列表需要纯文本电子邮件,并且在回复邮件时更喜欢内联和底部发布。

2、git-mentoring@googlegroups.com此邮件列表面向新的贡献者,并被创建为在主列表的公众视线之外发布问题和接收答案的地方。对帮助指导新人特别感兴趣的资深贡献者出现在列表中。为了避免搜索索引器,查看消息需要组成员身份;任何人都可以加入,无需批准。

3、#git-devel[3] 在 Libera Chat[4] 上这个 IRC 频道用于 Git 贡献者之间的对话。

如果有人当前在线并且知道你的问题的答案,你可以实时获得帮助;或者你也可以阅读 scrollback[5] 以查看是否有人回答了你。但IRC 不允许离线私信,因此如果你尝试私信某人然后退出 IRC,他们将无法回复你。

你随意翻开如下链接中看到了其中一天的讨论:

https://colabti.org/irclogger/irclogger_log/git-devel?date=2022-06-20

新手上路

接下来你选择使用邮件的方式,请求社区的帮助,于是,你奋笔疾书,写了一封邮件。

当你通过 git@vger.kernel.org 发送邮件到社区之后,就可以通过 https://lore.kernel.org/git 这个公共的收件箱列表,找到刚才发送的邮件。

除此之外,你惊喜地发现,新世界的大门正式向你敞开:

由于“Git 列表需要纯文本电子邮件,并且在回复邮件时更喜欢内联和底部发布”,因此在使用邮件服务时请特别注意:

  • 例如使用Gmail时,请选择“Plain text mode”:

  • 你也可以选择使用git send-email来发送/回复邮件:

https://kernel.org/pub/software/scm/git/docs/git-send-email.html

你可以通过点击 permalink 找到如下的命令行:

  git send-email \
    --in-reply-to={Message-ID} \
    --to=someone@email.com
    --cc=git@vger.kernel.org \
    --subject='Re: [PATCH v2 0/1] scalar: move to the top-level, test, CI and "install" support' \
    /path/to/YOUR_REPLY

请特别注意携带 in-reply-to,只有这样你的邮件才会被很好地组织在一起,避免丢失一些上下文。

小试牛刀

在与社区交流的过程中,你不断深入,找到了可行的修复方案。我想,此刻的你一定跃跃欲试,想尽快将自己的修复方案发给社区。但如果你不想石沉大海,或是迎来一些不必要的批评,请你先保持冷静,除了前面提到的《MyFirstContribution》,你还需要再做一些准备工作。

1、 阅读 Git 的《补丁提交》[6]> https://github.com/git/git/blob/master/Documentation/SubmittingPatches

《补丁提交》将指导你如何建立自己的工作,比如该基于 maint(稳定版本)还是该基于master ,又或者是指导你该如何清晰地表达这个补丁的价值。

在这个文档当中,你将学到关于补丁制作与提交的方方面面。

2、 阅读 Git 的《编码指南》[7]> https://github.com/git/git/blob/master/Documentation/CodingGuidelines

《编码指南》将帮助你了解一些Git非常较真的编码规范,从下面的一个例子你就可以看出他们对空格有多么的较真:

        (incorrect)
        cat hello > world < universe
        echo hello >$world

        (correct)
        cat hello >world <universe
        echo hello >"$world"

3、学习如何编写 Git 的测试用例> https://github.com/git/git/blob/master/t/README

Git 当中,使用一套以 shell 脚本为主,名为Sharness的测试框架。这套测试框架在2011年创建,并从 Git 的 test-lib.sh 当中派生出来,成为了一个独立的项目:https://github.com/chriscool/sharness

4、勇敢发送你的补丁当你一切准备就绪,确认Github CI也亮起了绿灯,勇敢地发送你的补丁吧,开始属于你的Git贡献之旅。

小结

我们从一个 Git 引发的故障开始,了解了 Git 问题排查的过程,在找到并复现问题之后,尝试寻求 Git 社区的帮助,并小试牛刀,向 Git 社区提交了我们的补丁。

Git 自2005年开始发展至今,正是由于其广泛的使用,以及一大批优秀的社区贡献者,让 Git 变得越来越好。使用 Git,发现问题并改进它,在这个过程中你一定可以得到别样的收获。

附录

  1. Github:https://github.com
  2. 《MyFirstContribution》:https://git-scm.com/docs/MyFirstContribution / https://github.com/git/git/blob/main/Documentation/MyFirstContribution.txt
  3. #git-devel:https://web.libera.chat/#git-devel
  4. Libera Chat:https://libera.chat
  5. scrollback:https://colabti.org/irclogger/irclogger_logs/git-devel
  6. CodingGuidelines:https://github.com/git/git/blob/master/Documentation/CodingGuidelines
  7. SubmmitingPatches:https://github.com/git/git/blob/master/Documentation/SubmittingPatches

本文由哈喽比特于2年以前收录,如有侵权请联系我们。
文章来源:https://mp.weixin.qq.com/s/1y6ifg0V4m4YybDfjdwPWA

 相关推荐

刘强东夫妇:“移民美国”传言被驳斥

京东创始人刘强东和其妻子章泽天最近成为了互联网舆论关注的焦点。有关他们“移民美国”和在美国购买豪宅的传言在互联网上广泛传播。然而,京东官方通过微博发言人发布的消息澄清了这些传言,称这些言论纯属虚假信息和蓄意捏造。

发布于:1年以前  |  808次阅读  |  详细内容 »

博主曝三大运营商,将集体采购百万台华为Mate60系列

日前,据博主“@超能数码君老周”爆料,国内三大运营商中国移动、中国电信和中国联通预计将集体采购百万台规模的华为Mate60系列手机。

发布于:1年以前  |  770次阅读  |  详细内容 »

ASML CEO警告:出口管制不是可行做法,不要“逼迫中国大陆创新”

据报道,荷兰半导体设备公司ASML正看到美国对华遏制政策的负面影响。阿斯麦(ASML)CEO彼得·温宁克在一档电视节目中分享了他对中国大陆问题以及该公司面临的出口管制和保护主义的看法。彼得曾在多个场合表达了他对出口管制以及中荷经济关系的担忧。

发布于:1年以前  |  756次阅读  |  详细内容 »

抖音中长视频App青桃更名抖音精选,字节再发力对抗B站

今年早些时候,抖音悄然上线了一款名为“青桃”的 App,Slogan 为“看见你的热爱”,根据应用介绍可知,“青桃”是一个属于年轻人的兴趣知识视频平台,由抖音官方出品的中长视频关联版本,整体风格有些类似B站。

发布于:1年以前  |  648次阅读  |  详细内容 »

威马CDO:中国每百户家庭仅17户有车

日前,威马汽车首席数据官梅松林转发了一份“世界各国地区拥车率排行榜”,同时,他发文表示:中国汽车普及率低于非洲国家尼日利亚,每百户家庭仅17户有车。意大利世界排名第一,每十户中九户有车。

发布于:1年以前  |  589次阅读  |  详细内容 »

研究发现维生素 C 等抗氧化剂会刺激癌症生长和转移

近日,一项新的研究发现,维生素 C 和 E 等抗氧化剂会激活一种机制,刺激癌症肿瘤中新血管的生长,帮助它们生长和扩散。

发布于:1年以前  |  449次阅读  |  详细内容 »

苹果据称正引入3D打印技术,用以生产智能手表的钢质底盘

据媒体援引消息人士报道,苹果公司正在测试使用3D打印技术来生产其智能手表的钢质底盘。消息传出后,3D系统一度大涨超10%,不过截至周三收盘,该股涨幅回落至2%以内。

发布于:1年以前  |  446次阅读  |  详细内容 »

千万级抖音网红秀才账号被封禁

9月2日,坐拥千万粉丝的网红主播“秀才”账号被封禁,在社交媒体平台上引发热议。平台相关负责人表示,“秀才”账号违反平台相关规定,已封禁。据知情人士透露,秀才近期被举报存在违法行为,这可能是他被封禁的部分原因。据悉,“秀才”年龄39岁,是安徽省亳州市蒙城县人,抖音网红,粉丝数量超1200万。他曾被称为“中老年...

发布于:1年以前  |  445次阅读  |  详细内容 »

亚马逊股东起诉公司和贝索斯,称其在购买卫星发射服务时忽视了 SpaceX

9月3日消息,亚马逊的一些股东,包括持有该公司股票的一家养老基金,日前对亚马逊、其创始人贝索斯和其董事会提起诉讼,指控他们在为 Project Kuiper 卫星星座项目购买发射服务时“违反了信义义务”。

发布于:1年以前  |  444次阅读  |  详细内容 »

苹果上线AppsbyApple网站,以推广自家应用程序

据消息,为推广自家应用,苹果现推出了一个名为“Apps by Apple”的网站,展示了苹果为旗下产品(如 iPhone、iPad、Apple Watch、Mac 和 Apple TV)开发的各种应用程序。

发布于:1年以前  |  442次阅读  |  详细内容 »

特斯拉美国降价引发投资者不满:“这是短期麻醉剂”

特斯拉本周在美国大幅下调Model S和X售价,引发了该公司一些最坚定支持者的不满。知名特斯拉多头、未来基金(Future Fund)管理合伙人加里·布莱克发帖称,降价是一种“短期麻醉剂”,会让潜在客户等待进一步降价。

发布于:1年以前  |  441次阅读  |  详细内容 »

光刻机巨头阿斯麦:拿到许可,继续对华出口

据外媒9月2日报道,荷兰半导体设备制造商阿斯麦称,尽管荷兰政府颁布的半导体设备出口管制新规9月正式生效,但该公司已获得在2023年底以前向中国运送受限制芯片制造机器的许可。

发布于:1年以前  |  437次阅读  |  详细内容 »

马斯克与库克首次隔空合作:为苹果提供卫星服务

近日,根据美国证券交易委员会的文件显示,苹果卫星服务提供商 Globalstar 近期向马斯克旗下的 SpaceX 支付 6400 万美元(约 4.65 亿元人民币)。用于在 2023-2025 年期间,发射卫星,进一步扩展苹果 iPhone 系列的 SOS 卫星服务。

发布于:1年以前  |  430次阅读  |  详细内容 »

𝕏(推特)调整隐私政策,可拿用户发布的信息训练 AI 模型

据报道,马斯克旗下社交平台𝕏(推特)日前调整了隐私政策,允许 𝕏 使用用户发布的信息来训练其人工智能(AI)模型。新的隐私政策将于 9 月 29 日生效。新政策规定,𝕏可能会使用所收集到的平台信息和公开可用的信息,来帮助训练 𝕏 的机器学习或人工智能模型。

发布于:1年以前  |  428次阅读  |  详细内容 »

荣耀CEO谈华为手机回归:替老同事们高兴,对行业也是好事

9月2日,荣耀CEO赵明在采访中谈及华为手机回归时表示,替老同事们高兴,觉得手机行业,由于华为的回归,让竞争充满了更多的可能性和更多的魅力,对行业来说也是件好事。

发布于:1年以前  |  423次阅读  |  详细内容 »

AI操控无人机能力超越人类冠军

《自然》30日发表的一篇论文报道了一个名为Swift的人工智能(AI)系统,该系统驾驶无人机的能力可在真实世界中一对一冠军赛里战胜人类对手。

发布于:1年以前  |  423次阅读  |  详细内容 »

AI生成的蘑菇科普书存在可致命错误

近日,非营利组织纽约真菌学会(NYMS)发出警告,表示亚马逊为代表的电商平台上,充斥着各种AI生成的蘑菇觅食科普书籍,其中存在诸多错误。

发布于:1年以前  |  420次阅读  |  详细内容 »

社交媒体平台𝕏计划收集用户生物识别数据与工作教育经历

社交媒体平台𝕏(原推特)新隐私政策提到:“在您同意的情况下,我们可能出于安全、安保和身份识别目的收集和使用您的生物识别信息。”

发布于:1年以前  |  411次阅读  |  详细内容 »

国产扫地机器人热销欧洲,国产割草机器人抢占欧洲草坪

2023年德国柏林消费电子展上,各大企业都带来了最新的理念和产品,而高端化、本土化的中国产品正在不断吸引欧洲等国际市场的目光。

发布于:1年以前  |  406次阅读  |  详细内容 »

罗永浩吐槽iPhone15和14不会有区别,除了序列号变了

罗永浩日前在直播中吐槽苹果即将推出的 iPhone 新品,具体内容为:“以我对我‘子公司’的了解,我认为 iPhone 15 跟 iPhone 14 不会有什么区别的,除了序(列)号变了,这个‘不要脸’的东西,这个‘臭厨子’。

发布于:1年以前  |  398次阅读  |  详细内容 »
 相关文章
Android插件化方案 5年以前  |  237227次阅读
vscode超好用的代码书签插件Bookmarks 2年以前  |  8063次阅读
 目录