From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:40463) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dcprw-0000xV-Gj for guix-patches@gnu.org; Wed, 02 Aug 2017 05:23:09 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dcprq-00038H-Ts for guix-patches@gnu.org; Wed, 02 Aug 2017 05:23:08 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:34983) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1dcprq-000381-Q5 for guix-patches@gnu.org; Wed, 02 Aug 2017 05:23:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1dcprq-0004gC-FU for guix-patches@gnu.org; Wed, 02 Aug 2017 05:23:02 -0400 Subject: [bug#27876] [PATCH] cuirass: add Hydra compatible HTTP API. Resent-Message-ID: From: ludo@gnu.org (Ludovic =?UTF-8?Q?Court=C3=A8s?=) References: <20170730100759.17734-1-m.othacehe@gmail.com> <87ini84ptn.fsf@gnu.org> <877eynnk7a.fsf@gmail.com> Date: Wed, 02 Aug 2017 11:22:21 +0200 In-Reply-To: <877eynnk7a.fsf@gmail.com> (Mathieu Othacehe's message of "Tue, 01 Aug 2017 21:48:09 +0200") Message-ID: <87zibitjci.fsf@gnu.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+kyle=kyleam.com@gnu.org Sender: "Guix-patches" To: Mathieu Othacehe Cc: 27876@debbugs.gnu.org Heya, Mathieu Othacehe skribis: >> That said, there=E2=80=99s only one place where we work, which is where = we spawn ^^^^ I meant =E2=80=9Cfork=E2=80=9D. :-) >> the =E2=80=98evaluate=E2=80=99 command. It may be that spawning it with= =E2=80=98open-pipe=E2=80=99 >> from (ice-9 popen) would sidestep the problem because =E2=80=98open-pipe= =E2=80=99 is >> specifically written to permit this. > > The open-pipe solution seems fine indeed ! Good. > Yes my planning is the following : > > 1. Fiberizing the whole thing to make it scalable. > 2. Add build start/stop detection : I think the ideal would be wip-ui > branch to get merged before, so that I can use the same mechanism to > parse build output :) > 3. Add authentification, and the other API you're describing. Sounds good! So it looks like you kind of like the =E2=80=98wip-ui=E2=80=99 approach. I= wasn=E2=80=99t sure but maybe that=E2=80=99s the way to go. After all, even if it=E2=80=99s a = bit of a hack, the API itself could still work if/when we have a better protocol with the daemon. We=E2=80=99ll see=E2=80=A6 Ludo=E2=80=99.