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

Aberta
  • Algo errado?
#603 0 comentários 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 1 dia

@ThomasK33 já está trabalhando nisso.

Desde 5/11/2026.

  • #694 de @ThomasK33 — aberto
  • #699 de @ThomasK33 — integrado

Avaliação

Dificuldade
4/5
Tempo estimado
3-5 dias
Facilidade para iniciantes
25/100
Tipo de issue
Funcionalidade
Clareza
Claramente especificada
Status de atividade
Estagnada
Stack de tecnologia
aws, go, postgresql, sqlite
Domínio
backend, database

Direção de pesquisa

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.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

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

Linguagem predominante
Go
Estrelas
0
Forks
0
Merge médio
6h 16min
PRs com merge (30d)
243

Preparar o ambiente

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de coder/openstorage

Todas as issues de coder/openstorage

Issues semelhantes

Mais issues de Go

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.