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

Aperta
  • Qualcosa non va?
#603 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

@ThomasK33 ci sta già lavorando.

Dal 5/11/2026.

  • #694 di @ThomasK33 — aperta
  • #699 di @ThomasK33 — integrata

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
25/100
Tipo di issue
Funzionalità
Chiarezza
Specificata chiaramente
Stato di attività
Ferma
Stack tecnologico
aws, go, postgresql, sqlite
Ambito
backend, database

Direzione di ricerca

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.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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

Lingua principale
Go
Stelle
0
Fork
0
Merge medio
6h 16m
PR unite (30g)
243

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di coder/openstorage

Tutte le issue di coder/openstorage

Issue simili

Altre issue su Go

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.