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 A6793459CC; Wed, 18 Sep 2024 16:16:49 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 96C814025E; Wed, 18 Sep 2024 16:16:49 +0200 (CEST) Received: from mail-pg1-f179.google.com (mail-pg1-f179.google.com [209.85.215.179]) by mails.dpdk.org (Postfix) with ESMTP id 8A98D4003C for ; Wed, 18 Sep 2024 16:16:47 +0200 (CEST) Received: by mail-pg1-f179.google.com with SMTP id 41be03b00d2f7-7db1f13b14aso5971835a12.1 for ; Wed, 18 Sep 2024 07:16:47 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=iol.unh.edu; s=unh-iol; t=1726669007; x=1727273807; darn=dpdk.org; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=02tlEenEV9+iJiX2k0qlvFSGbbvBad9frDm+rtmgIC4=; b=FD7ZGl6alJPz/RC5xYT1RdmUzk7e7gtNebNbYJXDukZvSN1p07QJ0RWnT4YoKnVyrF BRV/gH0FId0xcTR/pj+wnYLLHZdQVaQeURlfdg0T0GVKBICcmLLuJhoNR48bDhnAP8yd WURrX7ZV4b3k1CfOLIVmBljGaFfcYxIA9taAY= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1726669007; x=1727273807; h=content-transfer-encoding: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=02tlEenEV9+iJiX2k0qlvFSGbbvBad9frDm+rtmgIC4=; b=dUq0HLd4kVO2ZgSqV+dEytnse29R1IbmNbxBG8sNGDK9sful/mwQYv+VuqxgzTTFkp OqHtLzmFZN6YN+1LKJSgHjkqx2XHi+/M/vKI4VdFvvBtQ5XO1KBZVvOW1u0+EOSCRAki GHxZNDuwvj+tjJvKz+cIWOqTg6TNRj4RLDUk8kb5TgIpi5Jsqzrm4ZFoaLTw1NTgJagp /kBk1KqMkacSXtbFwJ7EoLk2+HKj45zcXhLKijGR7DM5E9sGMzCLKNWbCgzGyEk5M7yF elG/ApEXFh92M1i8VsPYakc3n212SA6BkkAir3iSg6GVt5WgiVmImhvAwMMNygXViMHy JWKg== X-Forwarded-Encrypted: i=1; AJvYcCWXR48NMPGPKcI7p8wOD3GgXF3GH6xZOKFYIENrV4t8/FZOX4HeYjkdrR7AV4rIxw4hvTw=@dpdk.org X-Gm-Message-State: AOJu0YwyTwkk3/2S8r1nrkgrShwkK+PfQKdii3xQOShRtIV7lnzb1UIH BAWUtTqDEfERQvQCtxov9V2s8dQmCZgcmlP3klkKlyra7jBwmxTl2atFrVI9efd1TMWrsyoQSzr MpBm0HqvMMF8vCK/whCyIhV/g+K5xrk7e/SNeOQ== X-Google-Smtp-Source: AGHT+IGqbASzn9KuQLihiP1049vS26hIyjeJqXBOf2XZcQKoxHmrzrZTXWjQ1MEW2hEYaQtNXFMSCQ8+nmwbT/68ZGg= X-Received: by 2002:a17:90a:2d84:b0:2d8:ee39:465 with SMTP id 98e67ed59e1d1-2db9ff7401amr29067986a91.1.1726669006629; Wed, 18 Sep 2024 07:16:46 -0700 (PDT) MIME-Version: 1.0 References: <20240911155058.11321-1-jspewock@iol.unh.edu> <20240916181445.13997-1-jspewock@iol.unh.edu> <80e007b5-f7a6-4316-ad90-b737ff0f009d@pantheon.tech> In-Reply-To: <80e007b5-f7a6-4316-ad90-b737ff0f009d@pantheon.tech> From: Jeremy Spewock Date: Wed, 18 Sep 2024 10:16:35 -0400 Message-ID: Subject: Re: [PATCH v2] dts: fix runner target in the Dockerfile To: =?UTF-8?Q?Juraj_Linke=C5=A1?= Cc: npratte@iol.unh.edu, probb@iol.unh.edu, wathsala.vithanage@arm.com, yoan.picchi@foss.arm.com, paul.szczepanek@arm.com, alex.chapman@arm.com, Luca.Vizzarro@arm.com, thomas@monjalon.net, Honnappa.Nagarahalli@arm.com, dev@dpdk.org Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable 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 On Wed, Sep 18, 2024 at 3:57=E2=80=AFAM Juraj Linke=C5=A1 wrote: > > > > diff --git a/dts/Dockerfile b/dts/Dockerfile > > > @@ -24,9 +27,12 @@ FROM base AS runner > > > +# Adds ~/.local/bin to PATH so that packages installed with pipx are c= allable. `pipx ensurepath` > > +# fixes this issue, but requires the shell to be re-opened which isn't= an option for this target. > > Let's explain this a bit more, I don't really know why this isn't an opti= on. The main reason it isn't an option is it is all happening in the same `docker build` process and it seems like however Docker is deciding to create the layers isn't refreshing the terminal. I don't think there is a way we could make it do so, but I can swap the "isn't an option" part of the comment for something more like "and the build process does not refresh the terminal in the required way before creating the next layer." > > > +ENV PATH=3D"$PATH:/root/.local/bin" > > +RUN poetry install --only main --no-root > > > > -CMD ["poetry", "run", "python", "main.py"] > > +ENTRYPOINT ["poetry", "run", "python", "main.py"] > >