From mboxrd@z Thu Jan 1 00:00:00 1970 X-GM-THRID: 6989998000320806912 X-Received: by 2002:a05:6602:2f01:: with SMTP id q1mr1982140iow.130.1627563021171; Thu, 29 Jul 2021 05:50:21 -0700 (PDT) X-BeenThere: isar-users@googlegroups.com Received: by 2002:a6b:8e09:: with SMTP id q9ls1011044iod.7.gmail; Thu, 29 Jul 2021 05:50:20 -0700 (PDT) X-Google-Smtp-Source: ABdhPJx4WK9Pu1Xu36X+uy4KIsJdftvHQFzMLKCU7wF4HHcvu93UrnPVHHTsmXOfE63EGt4eZLGU X-Received: by 2002:a6b:3e8b:: with SMTP id l133mr3991687ioa.137.1627563020709; Thu, 29 Jul 2021 05:50:20 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1627563020; cv=pass; d=google.com; s=arc-20160816; b=jtjlPLLkQeVYjzlnADiRUQGlGGPD/Pvm+2chaMfbQZ0oMUNxj8C2M/hRpahwbGzsWD a6TPKiUHNOp3LSo74t9YEWu0dTQYX9Ag+GBL3w/vSHR5TdJ7hl1LvIj2nYylmL7LmsqX KRBynnti3FSYT9Anz+T86Av3yRtBtgn0ZpMZXPAeG53WZhk3l+HT5JttPeOUVeim1oz2 /FTFfIuVfOu43fB3o56FN4yc2svFOAduKuhHjRpaFDayKIsHCH+pmk9icV0SaiH4GmOC r+24nQvb2KFFzPe7Sk0KniOf2SZ037A3q0s4ShGMPzTgzr1JORmlnHrFiwU01ZW6RXFE vSHw== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=mime-version:content-transfer-encoding:content-language:in-reply-to :user-agent:date:message-id:from:references:to:subject :dkim-signature; bh=2igccb+gqFfOiReU7c1FrL8B3KWLEEBuxUUole+PqyA=; b=AnHKIG1CkhqWyQBfP95VYUZOf2sg+3rwukN8o6E5022pVP6Ca9Y3hKjyQIl+qdWBG3 DMdvRdFOZGx06Q4gs7jwhC6RJcKWxIzRqc3x2ZVJIR0/upPKSGe1ckpFd/NjXa/nn5zB LJDy/qdjnZlOr6qcSBR+NFjRT1Wh3EVB/BUGo58xO4viXtZaoCddLbbwNOC4fIijA21S KW1YMJb5KpziaLuajM9IcQvEwKnQqucDGzZZJ8zxcSyDaPvB9SO13RIv9Rh3JNhsG+Op 08zil+3Botf8dWGe6d1W8PT8AJOpwJtHON/QbrdPgzuEWnd9rgVcprLHSBSKGGbi865H 97lg== ARC-Authentication-Results: i=2; gmr-mx.google.com; dkim=pass header.i=@siemens.onmicrosoft.com header.s=selector1-siemens-onmicrosoft-com header.b=E3VqDCUf; arc=pass (i=1 spf=pass spfdomain=siemens.com dkim=pass dkdomain=siemens.com dmarc=pass fromdomain=siemens.com); spf=pass (google.com: domain of silvano.cirujano-cuesta@siemens.com designates 40.107.8.41 as permitted sender) smtp.mailfrom=silvano.cirujano-cuesta@siemens.com; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=siemens.com Return-Path: Received: from EUR04-VI1-obe.outbound.protection.outlook.com (mail-eopbgr80041.outbound.protection.outlook.com. [40.107.8.41]) by gmr-mx.google.com with ESMTPS id e12si168697ile.4.2021.07.29.05.50.20 for (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Thu, 29 Jul 2021 05:50:20 -0700 (PDT) Received-SPF: pass (google.com: domain of silvano.cirujano-cuesta@siemens.com designates 40.107.8.41 as permitted sender) client-ip=40.107.8.41; Authentication-Results: gmr-mx.google.com; dkim=pass header.i=@siemens.onmicrosoft.com header.s=selector1-siemens-onmicrosoft-com header.b=E3VqDCUf; arc=pass (i=1 spf=pass spfdomain=siemens.com dkim=pass dkdomain=siemens.com dmarc=pass fromdomain=siemens.com); spf=pass (google.com: domain of silvano.cirujano-cuesta@siemens.com designates 40.107.8.41 as permitted sender) smtp.mailfrom=silvano.cirujano-cuesta@siemens.com; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=siemens.com ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=N0rTWSDj01CU6WgdEQ8VKFYAreNSNGtjF4dD4Jh7AvlfuCMpNEp6PtnZqfvscxQ3ku1KwOEzvmsMlm7m7ETBtKJ7P7mmB2Uro0oYz2J/EQk6QpVLRUh/NzdL4dWObghmO5A6/jKFA40902SAQF8i/5+RHCph0G6zgZoMP8cou6HOnUMh6LUafcg3ZYXOuA5p4Lf1sSd/V3s7u5L+B5KC9fOqPbFBws7n2gl90jcp35Iorc6iz5ir7A38p2zCB+jUb3wPSKCd92ObK5yotyEI5hhxcM5Qqezfn/uIHNEgbHJdw7I0YBUcfHsWJ6jJML08pQMb7m/6itb+Ye30inazoQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=2igccb+gqFfOiReU7c1FrL8B3KWLEEBuxUUole+PqyA=; b=M96to47tlFVyxx73Wr//C/HwxQmwzpvfB82BYnWy5kY6x71Y8vDvoiPllSfuNVGOzDqVRtYJO5R/e/Zl2Dq14zrX/NNiqUeXKiKd7Dtqdh3JyxLbXeY0UEXQ+zeAdnqyRLwIM6J9h3xtnNcCpoeuZdanLyV/lt+b9XHOi0kpodBRsP+JEYFwJqwb5teJ7MdmITXs3P9w/ZG0eEEcaX/J/wHm0yy3ZOFhacqXQMiINKyzABugKgnaDPJzNS9Uxp2nCvFnWkJobfbX7vRjcIPOUQPkz9V9itJbSrbh3D1E3mBjgfzBNUTqNDMi4A5HigQrd/4hbbiTJ9SscsSbiZxRPQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=siemens.com; dmarc=pass action=none header.from=siemens.com; dkim=pass header.d=siemens.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=siemens.onmicrosoft.com; s=selector1-siemens-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=2igccb+gqFfOiReU7c1FrL8B3KWLEEBuxUUole+PqyA=; b=E3VqDCUfyr6XhOX6tWhnltH+mzwIBZ+8WbRGdXQpScsgpSoF/RSq0QIebghtH7WYg+QK6M307qM6M/53f2H1EvVOFBEkEv738j3+TVMWc4zi6vOTPRWRo80tSvxkg3CnEq7OZi4swlSm0tJ1VDkW052m9Uuw73+ESe5NzP1/Zfo= Authentication-Results: googlegroups.com; dkim=none (message not signed) header.d=none;googlegroups.com; dmarc=none action=none header.from=siemens.com; Received: from PR3PR10MB4254.EURPRD10.PROD.OUTLOOK.COM (2603:10a6:102:ab::7) by PR3PR10MB4158.EURPRD10.PROD.OUTLOOK.COM (2603:10a6:102:a7::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4373.18; Thu, 29 Jul 2021 12:50:18 +0000 Received: from PR3PR10MB4254.EURPRD10.PROD.OUTLOOK.COM ([fe80::8d6e:ae99:c95c:83bd]) by PR3PR10MB4254.EURPRD10.PROD.OUTLOOK.COM ([fe80::8d6e:ae99:c95c:83bd%9]) with mapi id 15.20.4373.018; Thu, 29 Jul 2021 12:50:17 +0000 Subject: Re: [PATCH 3/4] classes: allow more control over container image name and tag To: Henning Schild , isar-users@googlegroups.com References: <20210728152210.7089-1-henning.schild@siemens.com> <20210728152210.7089-4-henning.schild@siemens.com> From: Silvano Cirujano Cuesta Message-ID: <8ce8e348-890e-08f3-b087-c3a604e4b931@siemens.com> Date: Thu, 29 Jul 2021 14:50:12 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.12.0 In-Reply-To: <20210728152210.7089-4-henning.schild@siemens.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit X-ClientProxiedBy: MN2PR15CA0038.namprd15.prod.outlook.com (2603:10b6:208:237::7) To PR3PR10MB4254.EURPRD10.PROD.OUTLOOK.COM (2603:10a6:102:ab::7) Return-Path: silvano.cirujano-cuesta@siemens.com MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from [139.23.72.45] (195.145.170.185) by MN2PR15CA0038.namprd15.prod.outlook.com (2603:10b6:208:237::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4373.18 via Frontend Transport; Thu, 29 Jul 2021 12:50:16 +0000 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 7860af83-5114-4ef6-de63-08d9528f6ae5 X-MS-TrafficTypeDiagnostic: PR3PR10MB4158: X-LD-Processed: 38ae3bcd-9579-4fd4-adda-b42e1495d55a,ExtAddr X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:8273; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: Gso25PHXZQ86PwsdE5MftCYJzbU9kXEoZim3GQqlygRh1Aa7ureYaGteGdmlOC3OkB5bjNoz/QnthGpQs+j5w4T7HsfN4FsDkZWVSXaDIBNm8aG+eboGJkToRAS76qSrUwXKK298Cpv2s128jo0IkkYx2rwALvtKkiBeNafhY4D6iXVVnT4kptWQh61wv+ibnnuRRjhMA1mwsawtjVdVn3KDftzTXKvsGMgeZ2PpwRB/bnDTzVaHON0vw4MSwn57wW/+ECbGIMZSWjNA7xiBwRf2c7IxA7UbFuhL9lrvStPh86OjTZtB6ADiq52MuGmLjJ2FlsEkssLe192HyjdEylqUpYDkntGUHgcElFoLGbB1d9jA4q4JjA1VvUVB1Org0byK0VU6i3GAwdrnn0RPMeB3mUmf6FPKF+vlGH+D45OCIqKIiHvjPmQfhzAUmQsD5dqzG42CwxWTFS2kkd4xsaanZ5jYH2ibl9o7RD9vYA/F3RIpGlsoWuvmfjVSkAqcvlaTsc7rYcsXDsgN7oCxMC0upNn8uIpgIHv+sY12mk18xsF+fZIZL5xRLVLHbtcspmz3qxUntWNtCm5CsT5RYf4rar7Oyo0GvfWwY/feQq6Xxav2Yng2UyqYafD9LN/4a01BkTGUsfGskzNbLR1OazT8Jud3vcxqs9OiOhJWSJMh2sVVISED0PaWpNoA1R3YUwgP4evKcJWZowfrJK5YAB+bLlMH1yMV5yuVmntZYOGdPr+TstP/1NDQewawwomg X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:PR3PR10MB4254.EURPRD10.PROD.OUTLOOK.COM;PTR:;CAT:NONE;SFS:(4636009)(39860400002)(376002)(346002)(136003)(366004)(396003)(83380400001)(8676002)(36756003)(6666004)(53546011)(86362001)(6486002)(5660300002)(8936002)(186003)(38100700002)(16576012)(316002)(2906002)(478600001)(31696002)(66946007)(66556008)(26005)(6706004)(31686004)(66476007)(2616005)(956004)(78286007)(45980500001)(43740500002);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?bTE2VktKS1laQmJza015THpNTG9FWXpramxvUmNxdzFPeGFYQUgzbW00UVVF?= =?utf-8?B?RUFVZEdQSllHa0pPU3dKZm5zcWM2YmpnUGROZzU3QU9haERpT1lYNnAxWGhS?= =?utf-8?B?Zko0TmlZdWpTYXBsSHluaWpkc0RBbTFxc1ZyOVJJQVc4RzJrYWw0OG00RjJh?= =?utf-8?B?Mkk5dHU0c21ocmQ2bWVVZHNhTC9WSmxKQ0NTeWZoazB5SUp1UzE4WXB2bWMy?= =?utf-8?B?ZWlIL0M3WkRHbEZ4ZHVpRzJQWG5yMlhxTXNRMUhvcUdER1ZYUExlaitmcHNC?= =?utf-8?B?VzN0aXM2WVlENkwzSFlRWVRFM0NYUTdtbzNTd2lCL2RBdmRWV3lrY2pyOExM?= =?utf-8?B?d1pwRlp1THFWclhiU01qYVNxMERmdXpnVGtPN1JUdFl1Ri80N0xsS1A5b2c2?= =?utf-8?B?VkxLb2JYTk5rWmU4bnBOOW9qb1pGK3dlcktRbE92L0JXVVVESCtRUDRoOTFs?= =?utf-8?B?SExPcHlMYTc3M3dHT2tHV015MHJZZTRTVXlYSlRGYlZtOWM0ckxWNmk1SGE4?= =?utf-8?B?NzM1QmJ5MzZyd1ZtTFpsWkNGYmRCSmlsejVHODhQd2R2aG0yM3dsOWZIRWNO?= =?utf-8?B?SGdpam5sNTdiUEZIUDArL2lXVDRNTFdnYlRHcXVIT3FMcUxXTjVybUw3OStk?= =?utf-8?B?VGdBcjcxMndzMDFJaTdpRmpyQSs0UTZqdkxDeDhyWXhBZUM3TXJ5NVRObXlV?= =?utf-8?B?TWtqRng3UC9RbUl1a2p5elphdXUvckNkMkExSTdFVXBqRmNIZnVNRmlsZHY3?= =?utf-8?B?dDQ5SSt1cEs2QUMwL3A2NDJ4KzlSU2pMVE11eWtST3ZJaTZubm9sYTl4ck9T?= =?utf-8?B?Z1lhalZtVnE4WWxrMHp2R2d2UGloRWhCazV3YUs0bzJvanJOdVo4Y3cwRTRm?= =?utf-8?B?ZGs5VjJ4OWE3aVZ5RFRrNEExQlpKZm9XRUpubkxxRTFQOWIxVStSQWErRWc5?= =?utf-8?B?Z1lRYTJidTE5SmtSOGQrQ1VnMEViN1lwcVc3eDN0a2dvbUJvVFRrakM4aUlF?= =?utf-8?B?NmR5Y3JFRnV6RTlRY1Z6Ykg0d3lmVGFwbnFhUE1vTy9DTVFFcWhqdFZ2a3Va?= =?utf-8?B?T3VsK0I0VUxkODVGYVZWS1N2alFleU9DU0xoNlVWTHFzNG1hSFhXYkZaQ1Q1?= =?utf-8?B?ejlQNHkxdVVnZUdyVXU4S2JWd0ZxSFh0RnQ4K0t6Qll5Sm9rTlNlK0lLT1NZ?= =?utf-8?B?QVlKMmtoWmdnRmxwYjRIc2E3TVdCb1duZS9YZjYrNWdML2RTSWk3VnVndndG?= =?utf-8?B?MFprbDA4R2t0WitjeHdVNjhVSHRCSGtlRDB4d2MxWkF4TnhhMTYwZzZ3RnVj?= =?utf-8?B?Y0JTelY2SGNIRTgvZWpIZ1pNeWRGZ29ZRlRkKy9WeEF3U0Y1bTlzNTVPdHNX?= =?utf-8?B?KzVtWmQzYmMxa1B5c2xiUWRqYTV1cVNUOE9GWm53VVB4Q1FwaUp1eVRrM1pB?= =?utf-8?B?VmZHZGV4aENqUHhWS2FKMGV1UlQ4bks1WDcrUTBJOWVJSFVKMEM1UXgrWHQr?= =?utf-8?B?azJQSFdwTWl1WTVhZEh6ZC9IZ3grKzZQeVR5b3JBS3R4R1RMNHNUUVQ4blBQ?= =?utf-8?B?SzFKYUtaeE9UUzBvd2Q5TFB6YVh6TWFhUGhzQzNiK1NUWEdKMGFDVVMra1V0?= =?utf-8?B?K2oyMS9rcUZBRW1IUzJ2UmJNVWEwKzQrb3FtWW04VzFWMmZzSmw1N3RYcnMw?= =?utf-8?B?MFJsMkJLZnNmODQxZDdiRXl6ekJ3WGpzbnJEUjNOVXFNa2ljK1dicWFSb3B4?= =?utf-8?Q?ITWgIlk8Nic9ANtaQQpzz6/rMBIpILa/LWjnkvE?= X-OriginatorOrg: siemens.com X-MS-Exchange-CrossTenant-Network-Message-Id: 7860af83-5114-4ef6-de63-08d9528f6ae5 X-MS-Exchange-CrossTenant-AuthSource: PR3PR10MB4254.EURPRD10.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 Jul 2021 12:50:17.8588 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 38ae3bcd-9579-4fd4-adda-b42e1495d55a X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: GUveRParHTMNRbatJ9mzt9PCUEg7s6c/gSqFqqIV5xHHT+VHqdTTWMLyMLbnwvOt5bOKsXrcLJ4qDUendEZjxJinwCzKEeNxs3bQ0vfSE2A5ShYUc/4ZXDALf8bEf6h8 X-MS-Exchange-Transport-CrossTenantHeadersStamped: PR3PR10MB4158 X-TUID: cD0Udvk3n/pf On 28/07/2021 17:22, Henning Schild wrote: > This patch allows more fine-grained control over how the resulting > container will be tagged. Where the default name will be PN together > with DISTRO and ARCH, and tag will be derived from PV and PR This is a completely meaningful change. My implementation didn't foresee the need for free selection of container name and tag. The default container name and tag that is patch is providing is also meaningful, but it's breaking the interface (resulting container image has a fixed, predefined name). I don't have the impression that there are many users of this feature. Therefore we might afford it. But I don't follow ISAR development close enough to be able to assess what's the impact of this change on the user community. The alternative is breaking this transition in two steps: 1. Make the container image name and tag configurable, but keep the default backwards compatible (even if yours is more meaningful). And announce the upcoming breaking change. 2. Change the default effectively breaking the interface. > > Signed-off-by: Henning Schild > --- > meta/classes/container-img.bbclass | 4 +--- > meta/classes/image-container-extension.bbclass | 11 ++++++----- > meta/classes/image-sdk-extension.bbclass | 2 +- > 3 files changed, 8 insertions(+), 9 deletions(-) > > diff --git a/meta/classes/container-img.bbclass b/meta/classes/container-img.bbclass > index 79ef3e8d756b..9928a58ef53d 100644 > --- a/meta/classes/container-img.bbclass > +++ b/meta/classes/container-img.bbclass > @@ -9,10 +9,8 @@ > do_container_image[stamp-extra-info] = "${DISTRO}-${MACHINE}" > do_container_image[vardeps] += "CONTAINER_FORMATS" > do_container_image(){ > - rootfs_id="${DISTRO}-${DISTRO_ARCH}" > - > bbdebug 1 "Generate container image in these formats: ${CONTAINER_FORMATS}" > - containerize_rootfs "${IMAGE_ROOTFS}" "${rootfs_id}" "${CONTAINER_FORMATS}" > + containerize_rootfs "${IMAGE_ROOTFS}" "${CONTAINER_FORMATS}" > } > > addtask container_image before do_image after do_image_tools > diff --git a/meta/classes/image-container-extension.bbclass b/meta/classes/image-container-extension.bbclass > index 0e70ba9c1405..43b67f9d864d 100644 > --- a/meta/classes/image-container-extension.bbclass > +++ b/meta/classes/image-container-extension.bbclass > @@ -6,15 +6,16 @@ > # This class extends the image.bbclass for containerizing the root filesystem. > > CONTAINER_FORMATS ?= "docker-archive" > +CONTAINER_IMAGE_NAME ?= "${PN}-${DISTRO}-${DISTRO_ARCH}" > +CONTAINER_TAG ?= "${PV}-${PR}" > > containerize_rootfs() { > local cmd="/bin/dash" > local empty_tag="empty" > - local tag="latest" > + local tag="${CONTAINER_TAG}" > local oci_img_dir="${WORKDIR}/oci-image" > local rootfs="$1" > - local rootfs_id="$2" > - local container_formats="$3" > + local container_formats="$2" > > # prepare OCI container image skeleton > bbdebug 1 "prepare OCI container image skeleton" > @@ -42,9 +43,9 @@ containerize_rootfs() { > sudo chown --recursive $(id -u):$(id -g) "${oci_img_dir}" > > # convert the OCI container image to the desired format > - image_name="isar-${rootfs_id}" > + image_name="${CONTAINER_IMAGE_NAME}" > for image_type in ${CONTAINER_FORMATS} ; do > - image_archive="${DEPLOY_DIR_IMAGE}/${rootfs_id}-${image_type}.tar" > + image_archive="${DEPLOY_DIR_IMAGE}/${image_name}-${tag}-${image_type}.tar" > bbdebug 1 "Creating container image type: ${image_type}" > case "${image_type}" in > "docker-archive" | "oci-archive") > diff --git a/meta/classes/image-sdk-extension.bbclass b/meta/classes/image-sdk-extension.bbclass > index 426b92595554..fa15b588068c 100644 > --- a/meta/classes/image-sdk-extension.bbclass > +++ b/meta/classes/image-sdk-extension.bbclass > @@ -80,7 +80,7 @@ do_populate_sdk() { > # generate the SDK in all the desired container formats > if [ -n "${sdk_container_formats}" ] ; then > bbnote "Generating SDK container in ${sdk_container_formats} format" > - containerize_rootfs "${SDKCHROOT_DIR}" "sdk-${DISTRO}-${DISTRO_ARCH}" "${sdk_container_formats}" > + containerize_rootfs "${SDKCHROOT_DIR}" "${sdk_container_formats}" > fi > } > > -- Silvano Cirujano Cuesta -- Siemens AG, T RDA IOT SES-DE Corporate Competence Center Embedded Linux