1940608 - Git PushLog: Clarify outstanding questions

Status: ASSIGNED

(Conduit :: Lando, task)

Description

[Olivier Mehani [:shtrom]]

Questions

• Where do we want to store the documentation?
  • https://github.com/mozilla-conduit/conduit-docs -> https://moz-conduit.readthedocs.io/en/latest/
• Who else (apart from stakeholders) uses the PushLog?
  • BugHerder, MozCi, Libmozdata, …
• SQLite data: What does node text contain? Is this just the changeset ID?
  • Mercurial Changeset SHA
• What is in the tags array in the pushlog?
• Message format: can we just match the pushlog data, or the Hg pulse message?
  • Backward compat with HG changeset ids, or just replace with git commit ids?
  • Git2hg expectations
    • Message: https://github.com/mozilla-conduit/git-hg-sync/blob/main/git_hg_sync/events.py#L5
    • Exchange: https://github.com/mozilla-conduit/git-hg-sync/blob/main/config.toml.example#L1-L9
• Do we need pulse exchanges in local / suite?
  • Unlikely, maybe just a simple text log file
• Impact of choice of branching repo model Worker handling of Git branches https://docs.google.com/document/d/1iVvKBWLMsc-oTMbhhgY2C8nl9kMvTTXyJdJaCBPT4JM/edit?tab=t.0 https://docs.google.com/document/d/1NnwxxhywYH4e6mYqqiG_76Gyayh3vxcbl45TUww3A0Y/edit?tab=t.0#heading=h.4q6tydqcyxaa
  • Likely to impact scoping of pushID
• Need to migrate existing data? -> Git PushLog IDs synchronisation
  • Do we want to backfill DB data with old pushes?
    • Yes
  • Do we want continuous push IDs? Can we have gaps?
    • Can have gaps
    • How are IDs scoped (per repo / branch)?

Outcome: Everyone impacted has been identified and was able to refine approach, and provide clarity on how to progress

Comment 1

[Olivier Mehani [:shtrom]]

@aryx, @ahal, @ryanvm (I may be overshooting on who can answer these questions, if so please ignore).

Question on whether to migrate the existing pushlog data in https://docs.google.com/document/d/1YrDtdTOE44BP53XFTpPr1clg9Gg26mAAoIVLPXNijpw/edit?tab=t.gxa0r4ut47iq

Comment 2

[Liz Henry (:lizzard)]

We have answered what we needed for now and so this no longer blocks the migration even though there are still questions.