From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 4DEDB42E62; Sat, 15 Jul 2023 01:25:33 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 0B84F40EF1; Sat, 15 Jul 2023 01:25:33 +0200 (CEST) Received: from mail-pg1-f173.google.com (mail-pg1-f173.google.com [209.85.215.173]) by mails.dpdk.org (Postfix) with ESMTP id 9D4D540DFB for ; Sat, 15 Jul 2023 01:25:31 +0200 (CEST) Received: by mail-pg1-f173.google.com with SMTP id 41be03b00d2f7-54290603887so1437399a12.1 for ; Fri, 14 Jul 2023 16:25:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=iol.unh.edu; s=unh-iol; t=1689377131; x=1691969131; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=fwhVIiwmVL341/uhZpoFWp7CAjfVylxt7eB3hydOclM=; b=iyyTXglqYWKmRwNlcmuPouRDJzAFH8hQCrW4BUMfwtLGZGhHYcCJ15bA1yUi6WHL0R Bo6yGEtToiK8xeSm7+2fRfi5r/nGGLxaJtOs7cr/eP4wkTAEBGPj175zats5TT9v0SPp z6uCmoEDvSVtGokCSF8Oy8bunXyM91oZUtR+0= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1689377131; x=1691969131; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=fwhVIiwmVL341/uhZpoFWp7CAjfVylxt7eB3hydOclM=; b=D8w3zgehmHXgy6d5vg9ek3Gp+YGXfxfGA2ngKc31flvvaAF2Fz93WiRWNZFFgB9660 cZq9zSoPwu/39KuwpaMzbgOKyHiPP+JuGnu8NgXMVQrR2F8oE12Bbckc4Y5RXQEPiKIw Ubn60iJ8IK9kKv/WAvpTdgEaELWZWwqmFgl0gs+TPaVp0PuBTpG8VnIiHnG901HRRMiN QW3PQCnt4CPmwGi1zN3KG1Ysu82IxhLz5yL25ZFejbEfqRH1gGjIqhzIKqjTWaI7NHdQ dWpXSZOidV2BPVMRMAcmMTWLzPMkKerE3JFBFSlNdkmpukdwjRgDD+IP3cz96Tfpr94N Ec9w== X-Gm-Message-State: ABy/qLZNX5goQZ+W/tus1US65KYPiP24VNZkNNKRlWKJCznrQXL9+NcG 9+bTnsY8Cr0u1sDKMT+WrrWd729D8lmL64pQieP1mw== X-Google-Smtp-Source: APBJJlHj0/lwSCrE2q0dZ88pW9hfpnQkSLHaDXUO7jjJ+dYnavr2GtKz4IEz51dIInmWXPbR+gHihiG1oKO/SuZzlzQ= X-Received: by 2002:a17:90a:de08:b0:263:856:fcdf with SMTP id m8-20020a17090ade0800b002630856fcdfmr4385412pjv.12.1689377130700; Fri, 14 Jul 2023 16:25:30 -0700 (PDT) MIME-Version: 1.0 References: <20230713165347.21997-2-jspewock@iol.unh.edu> <20230713165347.21997-3-jspewock@iol.unh.edu> In-Reply-To: From: Jeremy Spewock Date: Fri, 14 Jul 2023 19:25:19 -0400 Message-ID: Subject: Re: [PATCH v7 1/1] dts: add smoke tests To: =?UTF-8?Q?Juraj_Linke=C5=A1?= Cc: Patrick Robb , Honnappa.Nagarahalli@arm.com, thomas@monjalon.net, lijuan.tu@intel.com, wathsala.vithanage@arm.com, dev@dpdk.org Content-Type: multipart/alternative; boundary="000000000000c479ca06007ac3fe" X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org --000000000000c479ca06007ac3fe Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Fri, Jul 14, 2023 at 11:30=E2=80=AFAM Juraj Linke=C5=A1 wrote: > On Fri, Jul 14, 2023 at 4:47=E2=80=AFPM Patrick Robb = wrote: > > > > > > Tested-by: Patrick Robb > > Have you tested with a non-root user? The unit tests are failing for me. > We have been running this as root in testing and there is an issue with running this as non-root users because the unit tests don't escalate privileges. This is a trivial fix of course, but in further testing I think it also makes sense to extend the timeouts for these unit tests (as we do in regular CI testing) and I will wrap this into an updated patch on Monday morning. --000000000000c479ca06007ac3fe Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable


<= div dir=3D"ltr" class=3D"gmail_attr">On Fri, Jul 14, 2023 at 11:30=E2=80=AF= AM Juraj Linke=C5=A1 <juraj.linkes@pantheon.tech> wrote:
On Fri, Jul 14, 2023 at 4:47= =E2=80=AFPM Patrick Robb <probb@iol.unh.edu> wrote:
>
>
> Tested-by: Patrick Robb <probb@iol.unh.edu>

Have you tested with a non-root user? The unit tests are failing for me.

We have been running this as root in testing and ther= e is an issue with running this as non-root users because the unit tests do= n't escalate privileges. This is a trivial fix of course, but in furthe= r testing I think it also makes sense to extend the timeouts for these unit= tests (as we do in regular CI testing) and I will wrap this into an update= d patch on Monday morning.
--000000000000c479ca06007ac3fe--