tvix-store protos: add CA fields

#315
Opened by flokli at 2023-10-05T14·35+00

While not part of anything signed, these are still necessary to re-synthesize .narinfo files fully, and keep all metadata around.

Some notes after talking with @edef:

At some point we could also update the protos and create a StorePath representation in the protos, so we don't store nixbase32-encoded strings, but just the raw bytes, but that's probably for a subsequent CL.

Same applies to CA.

  1. While a System field would be a nice addition nix doesn't have one in either NarInfo or ValidPathInfo.

    griff at 2023-10-05T15·02+00

  2. flokli changed the subject of this issue from "tvix-store protos: add Deriver, CA, System fields" to "tvix-store protos: add Deriver, CA fields" at 2023-10-05T20·49+00
  3. Ugh, I have no idea where the brainfart of System came from. Sorry, updated the issue description :-D

    flokli at 2023-10-05T20·49+00

  4. cl/9647 does add Deriver fields, and with all the cleanups around nixhash (cl/9722 and context), parsing CA fields should have gotten less annoying.

    We'd need to probably figure out which formats Nix can produce, and add them as an example to the documentation.

    flokli at 2023-10-16T08·32+00

  5. flokli changed the subject of this issue from "tvix-store protos: add Deriver, CA fields" to "tvix-store protos: add CA fields" at 2023-10-16T08·36+00
  6. cl/9795 adds the CA field to the proto definitions, it still needs to be integrated.

    flokli at 2023-10-18T00·04+00

  7. cl/9795 was merged, this can be closed now.

    flokli at 2023-10-24T12·44+00

  8. flokli closed this issue at 2023-10-24T12·44+00