From: Ricardo Ribalda Delgado <ricardo@ribalda.com>
To: openembedded-core@lists.openembedded.org, jpewhacker@gmail.com,
pbarker@konsulko.com
Cc: Ricardo Ribalda Delgado <ricardo@ribalda.com>
Subject: [PATCH v2 2/2] wic: Add --embed-rootfs argument
Date: Wed, 4 Mar 2020 15:49:36 +0100 [thread overview]
Message-ID: <20200304144936.2559106-2-ricardo@ribalda.com> (raw)
In-Reply-To: <20200304144936.2559106-1-ricardo@ribalda.com>
This option adds the content of a rootfs on a specific location on the
rootfs.
It is very useful for making a partition that contains the rootfs for a
host and a target Eg:
/ -> Roofs for the host
/export/ -> Rootfs for the target (which will netboot)
Although today we support making a partition for "/export" this might
not be compatible with some upgrade systems, or we might be limited by
the number of partitions.
With this patch we can use something like:
part / --source rootfs --embed-rootfs target-image /export --embed-rootfs target-image2 /export2
on the .wks file.
Signed-off-by: Ricardo Ribalda Delgado <ricardo@ribalda.com>
---
scripts/lib/wic/help.py | 8 ++++++++
scripts/lib/wic/ksparser.py | 1 +
scripts/lib/wic/partition.py | 1 +
scripts/lib/wic/plugins/source/rootfs.py | 22 +++++++++++++++++++++-
4 files changed, 31 insertions(+), 1 deletion(-)
diff --git a/scripts/lib/wic/help.py b/scripts/lib/wic/help.py
index 4d342fcf05..140dc504cd 100644
--- a/scripts/lib/wic/help.py
+++ b/scripts/lib/wic/help.py
@@ -979,6 +979,14 @@ DESCRIPTION
copies. This option only has an effect with the rootfs
source plugin.
+ --embed-rootfs: This option is specific to wic. It embeds a rootfs into
+ the given path to the resulting image. The option
+ contains two fields, the roofs and the path, separated
+ by a space. The rootfs follows the same logic as the
+ rootfs-dir argument. Multiple options can be provided
+ in order to embed multiple rootfs. This option only has
+ an effect with the rootfs source plugin.
+
--extra-space: This option is specific to wic. It adds extra
space after the space filled by the content
of the partition. The final size can go
diff --git a/scripts/lib/wic/ksparser.py b/scripts/lib/wic/ksparser.py
index 650b976223..64c8c1175e 100644
--- a/scripts/lib/wic/ksparser.py
+++ b/scripts/lib/wic/ksparser.py
@@ -138,6 +138,7 @@ class KickStart():
part.add_argument('--align', type=int)
part.add_argument('--exclude-path', nargs='+')
part.add_argument('--include-path', nargs='+')
+ part.add_argument('--embed-rootfs', nargs=2, action='append')
part.add_argument("--extra-space", type=sizetype)
part.add_argument('--fsoptions', dest='fsopts')
part.add_argument('--fstype', default='vfat',
diff --git a/scripts/lib/wic/partition.py b/scripts/lib/wic/partition.py
index 2d95f78439..13857df82f 100644
--- a/scripts/lib/wic/partition.py
+++ b/scripts/lib/wic/partition.py
@@ -31,6 +31,7 @@ class Partition():
self.extra_space = args.extra_space
self.exclude_path = args.exclude_path
self.include_path = args.include_path
+ self.embed_rootfs = args.embed_rootfs
self.fsopts = args.fsopts
self.fstype = args.fstype
self.label = args.label
diff --git a/scripts/lib/wic/plugins/source/rootfs.py b/scripts/lib/wic/plugins/source/rootfs.py
index 40419a64b3..089aaea477 100644
--- a/scripts/lib/wic/plugins/source/rootfs.py
+++ b/scripts/lib/wic/plugins/source/rootfs.py
@@ -17,6 +17,7 @@ import shutil
import sys
from oe.path import copyhardlinktree, copytree
+from pathlib import Path
from wic import WicError
from wic.pluginbase import SourcePlugin
@@ -80,7 +81,7 @@ class RootfsPlugin(SourcePlugin):
new_rootfs = None
# Handle excluded paths.
- if part.exclude_path or part.include_path:
+ if part.exclude_path or part.include_path or part.embed_rootfs:
# We need a new rootfs directory we can delete files from. Copy to
# workdir.
new_rootfs = os.path.realpath(os.path.join(cr_workdir, "rootfs%d" % part.lineno))
@@ -100,6 +101,25 @@ class RootfsPlugin(SourcePlugin):
for path in part.include_path or []:
copyhardlinktree(path, new_rootfs)
+ for embed in part.embed_rootfs or []:
+ [embed_rootfs, path] = embed
+ #we need to remove the initial / for os.path.join to work
+ if os.path.isabs(path):
+ path = path[1:]
+ if embed_rootfs in krootfs_dir:
+ embed_rootfs = krootfs_dir[embed_rootfs]
+ embed_rootfs = cls.__get_rootfs_dir(embed_rootfs)
+ tar_file = os.path.realpath(os.path.join(cr_workdir, "aux.tar"))
+ tar_cmd = "%s tar cpf %s -C %s ." % (cls.__get_pseudo(native_sysroot,
+ embed_rootfs), tar_file, embed_rootfs)
+ exec_native_cmd(tar_cmd, native_sysroot)
+ untar_cmd = "%s tar xf %s -C %s ." % (cls.__get_pseudo(native_sysroot, new_rootfs),
+ tar_file, os.path.join(new_rootfs, path))
+ Path(os.path.join(new_rootfs, path)).mkdir(parents=True, exist_ok=True)
+ exec_native_cmd(untar_cmd, native_sysroot,
+ cls.__get_pseudo(native_sysroot, new_rootfs))
+ os.remove(tar_file)
+
for orig_path in part.exclude_path or []:
path = orig_path
if os.path.isabs(path):
--
2.25.1
next prev parent reply other threads:[~2020-03-04 14:49 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-03-04 14:49 [PATCH v2 1/2] wic: Fix permissions when using exclude or include path Ricardo Ribalda Delgado
2020-03-04 14:49 ` Ricardo Ribalda Delgado [this message]
2020-03-05 9:33 ` [PATCH v2 2/2] wic: Add --embed-rootfs argument Paul Barker
2020-03-05 12:26 ` Ricardo Ribalda Delgado
2020-04-03 19:52 ` Ricardo Ribalda
2020-04-07 17:47 ` Paul Barker
2020-04-07 18:19 ` Ricardo Ribalda
[not found] ` <16039C80250EF791.10037@lists.openembedded.org>
2020-04-08 10:28 ` [OE-core] " Ricardo Ribalda
2020-04-05 7:01 ` [OE-core] [PATCH v2 1/2] wic: Fix permissions when using exclude or include path Richard Purdie
2020-04-05 7:21 ` Ricardo Ribalda
[not found] ` <1602DB74B6717A3F.2403@lists.openembedded.org>
2020-04-05 9:56 ` Ricardo Ribalda
2020-04-05 11:04 ` Richard Purdie
2020-04-05 11:53 ` Ricardo Ribalda
2020-04-05 12:01 ` Ricardo Ribalda
2020-04-05 15:29 ` Richard Purdie
2020-04-05 15:44 ` Ricardo Ribalda
2020-04-06 9:00 ` Richard Purdie
2020-04-06 9:03 ` Ricardo Ribalda
[not found] ` <16032F9D180FD5D7.1684@lists.openembedded.org>
2020-04-06 9:21 ` Ricardo Ribalda
2020-04-06 13:48 ` Richard Purdie
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=20200304144936.2559106-2-ricardo@ribalda.com \
--to=ricardo@ribalda.com \
--cc=jpewhacker@gmail.com \
--cc=openembedded-core@lists.openembedded.org \
--cc=pbarker@konsulko.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