From users-return-7086-apmail-trafficserver-users-archive=trafficserver.apache.org@trafficserver.apache.org Mon May 7 15:14:48 2018 Return-Path: X-Original-To: apmail-trafficserver-users-archive@www.apache.org Delivered-To: apmail-trafficserver-users-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 69B8918D30 for ; Mon, 7 May 2018 15:14:48 +0000 (UTC) Received: (qmail 82165 invoked by uid 500); 7 May 2018 15:14:47 -0000 Delivered-To: apmail-trafficserver-users-archive@trafficserver.apache.org Received: (qmail 82063 invoked by uid 500); 7 May 2018 15:14:47 -0000 Mailing-List: contact users-help@trafficserver.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: users@trafficserver.apache.org Delivered-To: mailing list users@trafficserver.apache.org Received: (qmail 82046 invoked by uid 99); 7 May 2018 15:14:47 -0000 Received: from mail-relay.apache.org (HELO mailrelay1-lw-us.apache.org) (207.244.88.152) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 07 May 2018 15:14:47 +0000 Received: from mail-it0-f45.google.com (mail-it0-f45.google.com [209.85.214.45]) by mailrelay1-lw-us.apache.org (ASF Mail Server at mailrelay1-lw-us.apache.org) with ESMTPSA id A0DAD7C8; Mon, 7 May 2018 15:14:46 +0000 (UTC) Received: by mail-it0-f45.google.com with SMTP id t7-v6so11399943itf.0; Mon, 07 May 2018 08:14:46 -0700 (PDT) X-Gm-Message-State: ALKqPwcLVQcsWSn2+cyp2aWMuNTWjVOBUK6DyPVSmwfdviTMLigkKH46 6BrGcZ4ANkfJB3g1i7LIq+1Bt7yis1x66Rq6D+s= X-Google-Smtp-Source: AB8JxZpDNWyT0KdDKxPAOGn2ikn0ufEqxs3xgb6FxuO6BsWTiM5ELrvfJpFKA0rJ1ewBInlqoADbO/pbeQQrGmFXS4s= X-Received: by 2002:a24:c301:: with SMTP id s1-v6mr1693978itg.100.1525706086028; Mon, 07 May 2018 08:14:46 -0700 (PDT) MIME-Version: 1.0 References: <547E2D8D-0EF6-4273-84AD-B75947E9207C@apache.org> In-Reply-To: <547E2D8D-0EF6-4273-84AD-B75947E9207C@apache.org> From: Phil Sorber Date: Mon, 07 May 2018 15:14:35 +0000 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PROPOSAL] Replace LuaJIT configurations with YAML To: users@trafficserver.apache.org Cc: dev Content-Type: multipart/alternative; boundary="0000000000004a1cef056b9f21a4" --0000000000004a1cef056b9f21a4 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Mon, May 7, 2018 at 5:47 AM Leif Hedstrom wrote: > Hi, > > I=E2=80=99d like to propose that we eliminate the existing LuaJIT configu= rations > with a simple YAML format. This would be the first step towards a unified > configuration format, and I think we have to admit defeat on Lua, and do > something simpler and more normal for both developers and users. > +1. Keep it for plugins, (remap especially) but for config it seems overkill. > > There are only 2 (or 3) configurations using the LuaJIT configurations: > > logging.config > sni.config > > > But, also see my other proposal of killing metrics.config, which is > currently in LuaJIT as well. > > If we agree to this, we=E2=80=99ll remove LuaJIT as a first class citizen= from ATS > v8.0.0. For the ts-lua plugin, we=E2=80=99ll continue to support this (of= course), > but it will require an externally provided LuaJIT distribution. We alread= y > have the =E2=80=94with-luajit configure option, and we=E2=80=99ll expand = on that to > auto-detect system level LuaJIT availability. > > If there are no strong arguments against this proposal, I=E2=80=99d like = to get > this change done for ATS v8.0.0. > > Thanks, > > =E2=80=94 Leif > > --0000000000004a1cef056b9f21a4 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
On Mon, May 7,= 2018 at 5:47 AM Leif Hedstrom <zwoo= p@apache.org> wrote:
Hi,

I=E2=80=99d like to propose that we eliminate the existing LuaJIT configura= tions with a simple YAML format. This would be the first step towards a uni= fied configuration format, and I think we have to admit defeat on Lua, and = do something simpler and more normal for both developers and users.

+1. Keep it for plugins, (remap especially) bu= t for config it seems overkill.
=C2=A0

There are only 2 (or 3) configurations using the LuaJIT configurations:

=C2=A0 =C2=A0 =C2=A0 =C2=A0 logging.config
=C2=A0 =C2=A0 =C2=A0 =C2=A0 sni.config


But, also see my other proposal of killing metrics.config, which is current= ly in LuaJIT as well.

If we agree to this, we=E2=80=99ll remove LuaJIT as a first class citizen f= rom ATS v8.0.0. For the ts-lua plugin, we=E2=80=99ll continue to support th= is (of course), but it will require an externally provided LuaJIT distribut= ion. We already have the =E2=80=94with-luajit configure option, and we=E2= =80=99ll expand on that to auto-detect system level LuaJIT availability.
If there are no strong arguments against this proposal, I=E2=80=99d like to= get this change done for ATS v8.0.0.

Thanks,

=E2=80=94 Leif

--0000000000004a1cef056b9f21a4--