scripts: Expose options for prebuilts DLCs to enable powerwash safety
.. this is to allow even prebuilt DLCs to toggle the powerwash safety
feature on CrOS. Prebuilt (artifacts meta) DLC builders/owners are
responsible for abiding by the build time restrictions set on powerwash
safe DLCs prior to using this feature.
BUG=b:222344877
TEST=./run_tests
Change-Id: I37fc4610af5cc652c0989ca76601c66fd930a695
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/4878284
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Commit-Queue: Jae Hoon Kim <kimjae@chromium.org>
Tested-by: Jae Hoon Kim <kimjae@chromium.org>
Reviewed-by: Yuanpeng Ni <yuanpengni@chromium.org>
diff --git a/scripts/cros_generate_dlc_artifacts_unittest.py b/scripts/cros_generate_dlc_artifacts_unittest.py
index ba8a2f1..b7ad7dd 100644
--- a/scripts/cros_generate_dlc_artifacts_unittest.py
+++ b/scripts/cros_generate_dlc_artifacts_unittest.py
@@ -27,6 +27,7 @@
@pytest.mark.parametrize("name", ((""), ("<some-name>")))
@pytest.mark.parametrize("description", ((""), ("<some-description>")))
@pytest.mark.parametrize("version", ("<some-version>",))
+@pytest.mark.parametrize("powerwash_safety", (True, False))
@mock.patch.object(dlc_lib, "EbuildParams")
def test_generate_dlc_params(
mock_ebuild_params,
@@ -35,6 +36,7 @@
name,
description,
version,
+ powerwash_safety,
tmp_path,
):
"""Tests out GenerateDlcParams"""
@@ -54,6 +56,7 @@
*(["--name", f"{name}"] if name else []),
*(["--description", f"{description}"] if description else []),
*(["--version", f"{version}"] if version else []),
+ "--powerwash-safety" if powerwash_safety else "--no-powerwash-safety",
]
opts = cros_generate_dlc_artifacts.ParseArguments(argv)
cros_generate_dlc_artifacts.GenerateDlcParams(opts)
@@ -71,4 +74,5 @@
fullnamerev="",
scaled=True,
loadpin_verity_digest=False,
+ powerwash_safe=powerwash_safety,
)