S3-WAL: a GitHub sync create without default_branch keeps main (#599 follow-up)

未关闭
#603 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 1 天内回复

@ThomasK33 已经在做这个了。

开始于 2026年11月5日。

  • #694 来自 @ThomasK33 —— 未关闭
  • #699 来自 @ThomasK33 —— 已合并

评估

难度
4/5
预计耗时
3-5 天
新手友好度
25/100
Issue 类型
功能
描述清晰度
描述清楚
活跃度
停滞
技术栈
aws, go, postgresql, sqlite
领域
backend, database

调研方向

Read docs/explanation/storage-engine.md and docs/explanation/wire-compatibility.md for the S3-WAL format stability policy and recorded deviation. The change is explicitly deferred until the next metadata format change (v3 or later); then inspect internal/meta/s3walmeta*.go and the TestS3WALDropsDefaultBranchOmitted test. Done means the v3+ format handles create, adoption, PATCH clear, and conversion, with the deviation removed and the test moved into TestAdoptUpstreamDefaultBranch.

由索引模型根据 Issue 内容生成。

描述

Problem

On S3-WAL, a GitHub sync create that omits default_branch keeps main. On PostgreSQL and SQLite, the first successful sync adopts the GitHub repository's default branch (#599). Upstream spec 40e43e00 (2026-10-28) documents the GitHub default for this case.

Why it is deferred

S3-WAL metadata v2 is frozen at "v1 plus four fields" (docs/explanation/storage-engine.md, S3WAL format stability policy). An older v2 writer drops unknown fields on rewrite, so a new default_branch_omitted field needs a format change. The #599 desk decision keeps v2 unchanged and records a deviation row in docs/explanation/wire-compatibility.md.

Trigger

Act on this issue at the next S3-WAL metadata format change (v3 or later). Add upstreams[].default_branch_omitted to that format, set it in S3WAL.CreateLinkedRepo, implement S3WAL.AdoptUpstreamDefaultBranch and the PATCH clear (as PostgreSQL and SQLite do), and remove the deviation row.

Scope

  • internal/meta/s3walmeta*.go: the field, the create write, the adopt write, the PATCH clear, and the convert path.
  • TestS3WALDropsDefaultBranchOmitted becomes the S3-WAL lane of TestAdoptUpstreamDefaultBranch.

Refs #599


Generated with xum • Model: anthropic:claude-opus-5-5 • Thinking: high

主要语言
Go
星标
0
派生
0
平均合并
6 小时 16 分钟
30 天内合并 PR
243

环境准备

  • 提供 Dockerfile 或 Docker Compose 文件
  • 没有 Pull Request 模板
  • 阅读贡献指南

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

coder/openstorage 的其他 Issue

查看 coder/openstorage 的全部 Issue

相似的 Issue

更多 Go Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。