From: Henning Schild <henning.schild@siemens.com>
To: <isar-users@googlegroups.com>
Cc: Silvano Cirujano Cuesta <silvano.cirujano-cuesta@siemens.com>,
"Henning Schild" <henning.schild@siemens.com>
Subject: [PATCH v6 4/5] classes: rename CONTAINER_FORMATS to CONTAINER_IMAGE_FORMATS
Date: Wed, 15 Dec 2021 18:25:49 +0100 [thread overview]
Message-ID: <20211215172550.27153-5-henning.schild@siemens.com> (raw)
In-Reply-To: <20211215172550.27153-1-henning.schild@siemens.com>
This brings some consistency with other image-classes, that image class
variables are prefixed with that class.
Also fix the docs (where the trailing S was missing), and mention the
interface break in our changelog.
Signed-off-by: Henning Schild <henning.schild@siemens.com>
---
RECIPE-API-CHANGELOG.md | 4 ++++
doc/user_manual.md | 8 ++++----
meta/classes/container-img.bbclass | 6 +++---
meta/classes/image-container-extension.bbclass | 4 ++--
4 files changed, 13 insertions(+), 9 deletions(-)
diff --git a/RECIPE-API-CHANGELOG.md b/RECIPE-API-CHANGELOG.md
index b3e6a2af199d..3bbb42a9a701 100644
--- a/RECIPE-API-CHANGELOG.md
+++ b/RECIPE-API-CHANGELOG.md
@@ -310,3 +310,7 @@ will still also work, but with deprecation warning shown.
### Change default "NAME:TAG" when building container images
The "NAME" used to be rather static and the TAG was always "latest", now the values are derived from recipe variables PN, PV, PR.
+
+### Renamed variable CONTAINER_FORMATS to CONTAINER_IMAGE_FORMATS
+
+The meaning remains the same, just the name changed.
diff --git a/doc/user_manual.md b/doc/user_manual.md
index eaafd6a9d708..caaab8ccfc02 100644
--- a/doc/user_manual.md
+++ b/doc/user_manual.md
@@ -260,7 +260,7 @@ requirements (e.g. libraries) can be easily resolved in a containerized
environment.
Container images can be generated in different formats, selected with the
-variable `CONTAINER_FORMAT`. One or more (whitespace separated) of following
+variable `CONTAINER_IMAGE_FORMATS`. One or more (whitespace separated) of following
options can be given:
- `docker-archive`: (default) an archive containing a Docker image that can
be imported with [`docker load`](https://docs.docker.com/engine/reference/commandline/load)
@@ -282,7 +282,7 @@ The resulting container image archives (only for `docker-archive` and
`oci-archive`) are made available as
`tmp/deploy/images/${MACHINE}/${DISTRO}-${DISTRO_ARCH}-${container_format}.tar.xz`
(being `container_format` each one of the formats specified in
-`CONTAINER_FORMAT`).
+`CONTAINER_IMAGE_FORMATS`).
### Example
@@ -291,9 +291,9 @@ The resulting container image archives (only for `docker-archive` and
For one-shot builds (use `local.conf` otherwise):
```
-export BB_ENV_EXTRAWHITE="$BB_ENV_EXTRAWHITE IMAGE_FSTYPES CONTAINER_FORMAT"
+export BB_ENV_EXTRAWHITE="$BB_ENV_EXTRAWHITE IMAGE_TYPE CONTAINER_IMAGE_FORMATS"
export IMAGE_FSTYPES="container-img"
-export CONTAINER_FORMAT="docker-archive"
+export CONTAINER_IMAGE_FORMATS="docker-archive"
```
- Trigger creation of container image from root filesystem
diff --git a/meta/classes/container-img.bbclass b/meta/classes/container-img.bbclass
index 4e0fe0efb484..e26895a92b1e 100644
--- a/meta/classes/container-img.bbclass
+++ b/meta/classes/container-img.bbclass
@@ -8,10 +8,10 @@
do_container_image[dirs] = "${DEPLOY_DIR_IMAGE}"
do_container_image[stamp-extra-info] = "${DISTRO}-${MACHINE}"
-do_container_image[vardeps] += "CONTAINER_FORMATS"
+do_container_image[vardeps] += "CONTAINER_IMAGE_FORMATS"
do_container_image(){
- bbdebug 1 "Generate container image in these formats: ${CONTAINER_FORMATS}"
- containerize_rootfs "${IMAGE_ROOTFS}" "${CONTAINER_FORMATS}"
+ bbdebug 1 "Generate container image in these formats: ${CONTAINER_IMAGE_FORMATS}"
+ containerize_rootfs "${IMAGE_ROOTFS}" "${CONTAINER_IMAGE_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 b8cf85a5c256..cdec46334370 100644
--- a/meta/classes/image-container-extension.bbclass
+++ b/meta/classes/image-container-extension.bbclass
@@ -5,7 +5,7 @@
#
# This class extends the image.bbclass for containerizing the root filesystem.
-CONTAINER_FORMATS ?= "docker-archive"
+CONTAINER_IMAGE_FORMATS ?= "docker-archive"
CONTAINER_IMAGE_NAME ?= "${PN}-${DISTRO}-${DISTRO_ARCH}"
CONTAINER_IMAGE_TAG ?= "${PV}-${PR}"
@@ -45,7 +45,7 @@ containerize_rootfs() {
# convert the OCI container image to the desired format
image_name="${container_name_prefix}${CONTAINER_IMAGE_NAME}"
- for image_type in ${CONTAINER_FORMATS} ; do
+ for image_type in ${CONTAINER_IMAGE_FORMATS} ; do
image_archive="${DEPLOY_DIR_IMAGE}/${image_name}-${tag}-${image_type}.tar"
bbdebug 1 "Creating container image type: ${image_type}"
case "${image_type}" in
--
2.32.0
next prev parent reply other threads:[~2021-12-15 17:26 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-12-15 17:25 [PATCH v6 0/5] Allow better control over container tags Henning Schild
2021-12-15 17:25 ` [PATCH v6 1/5] classes: make sure container extension can run multiple times Henning Schild
2021-12-15 17:25 ` [PATCH v6 2/5] classes: simplify tag handling in container class Henning Schild
2021-12-15 17:25 ` [PATCH v6 3/5] classes: allow more control over container image name and tag Henning Schild
2022-01-04 17:53 ` Henning Schild
2021-12-15 17:25 ` Henning Schild [this message]
2021-12-15 17:25 ` [PATCH v6 5/5] classes: fix comment in container-img class Henning Schild
2021-12-22 15:27 ` [PATCH v6 0/5] Allow better control over container tags Anton Mikanovich
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20211215172550.27153-5-henning.schild@siemens.com \
--to=henning.schild@siemens.com \
--cc=isar-users@googlegroups.com \
--cc=silvano.cirujano-cuesta@siemens.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox