Re: [PATCH 1/2] Support publishing build events

[Ludovic Courtès]

Hi Chris,

Christopher Baines <address@hidden> skribis:

> ---
>  Makefile.am                 |   8 +-
>  bin/cuirass-send-events.in  |  90 +++++++++++++++++++++++
>  src/cuirass/base.scm        |   9 ++-
>  src/cuirass/database.scm    | 142 +++++++++++++++++++++++++++++++++---
>  src/cuirass/http.scm        |  24 ++++++
>  src/cuirass/send-events.scm |  69 ++++++++++++++++++
>  src/schema.sql              |  12 +++
>  src/sql/upgrade-5.sql       |  15 ++++
>  8 files changed, 356 insertions(+), 13 deletions(-)
>  create mode 100644 bin/cuirass-send-events.in
>  create mode 100644 src/cuirass/send-events.scm
>  create mode 100644 src/sql/upgrade-5.sql

Sorry for the delay.  It LGTM, thank you!

Please add a commit log :-), and let us know when you’d like to test
with the instance on berlin or that on bayfront.

IIUC, the ‘send-events’ program is provided as a debugging aid, right?
I mean it’s not used anywhere in the code.

If you can think of ways to tests parts of this, that’d also be great,
but it shouldn’t block it.

Thanks,
Ludo’.