Remove ContainerCrate in favor of ObsPackage

Author: dcermak

src/bci_build/containercrate.py

@@ -21,7 +21,6 @@
 from bci_build.container_attributes import PackageType
 from bci_build.container_attributes import ReleaseStage
 from bci_build.container_attributes import SupportLevel
-from bci_build.containercrate import ContainerCrate
 from bci_build.os_version import ALL_OS_LTSS_VERSIONS
 from bci_build.os_version import RELEASED_OS_VERSIONS
 from bci_build.os_version import OsVersion
@@ -203,9 +202,6 @@ class BaseContainerImage(ObsPackage):
     #: build flavors to produce for this container variant
     build_flavor: str | None = None
 
-    #: create that this container is part of
-    crate: ContainerCrate = None
-
     #: Add any replacements via `obs-service-replace_using_package_version
     #: <https://github.com/openSUSE/obs-service-replace_using_package_version>`_
     #: that are used in this image into this list.
@@ -1104,15 +1100,6 @@ async def write_file_to_dest(fname: str, contents: str | bytes) -> None:
                 False
             ), f"got an unexpected build_recipe_type: '{self.build_recipe_type}'"
 
-        if self.build_flavor:
-            dfile = "Dockerfile"
-            tasks.append(write_file_to_dest(dfile, self.crate.default_dockerfile()))
-            files.append(dfile)
-
-            mname = "_multibuild"
-            tasks.append(write_file_to_dest(mname, self.crate.multibuild(self)))
-            files.append(mname)
-
         if with_service_file:
             tasks.append(self._write_service_file(dest))
 

src/bci_build/package/__init__.py

@@ -3,7 +3,6 @@
 import datetime
 
 from bci_build.container_attributes import PackageType
-from bci_build.containercrate import ContainerCrate
 from bci_build.os_version import CAN_BE_LATEST_OS_VERSION
 from bci_build.os_version import OsVersion
 from bci_build.package import DOCKERFILE_RUN
@@ -12,6 +11,7 @@
 from bci_build.package import Package
 from bci_build.package import Replacement
 from bci_build.package import _build_tag_prefix
+from bci_build.package.obs_package import MultiBuildObsPackage
 from bci_build.registry import publish_registry
 
 # last version needs to be the newest
@@ -50,14 +50,11 @@ def _get_sac_supported_until(
     )
 
 
-TOMCAT_CONTAINERS = [
-    ApplicationStackContainer(
+def _create_tomcat_container(
+    os_version: OsVersion, tomcat_ver: str, jre_version: int
+) -> ApplicationStackContainer:
+    return ApplicationStackContainer(
         name="apache-tomcat",
-        package_name=(
-            f"apache-tomcat-{tomcat_ver.partition('.')[0]}-image"
-            if os_version.is_tumbleweed
-            else f"sac-apache-tomcat-{tomcat_ver.partition('.')[0]}-image"
-        ),
         _publish_registry=publish_registry(os_version, app_collection=True),
         pretty_name="Apache Tomcat",
         custom_description=(
@@ -125,15 +122,29 @@ def _get_sac_supported_until(
         entrypoint_user="tomcat",
         logo_url="https://tomcat.apache.org/res/images/tomcat.png",
     )
-    for tomcat_ver, os_version, jre_version in (
-        ("10.1", OsVersion.TUMBLEWEED, 22),
-        ("10.1", OsVersion.TUMBLEWEED, 21),
-        ("10.1", OsVersion.TUMBLEWEED, 17),
-        ("9", OsVersion.TUMBLEWEED, 17),
-        ("10.1", OsVersion.SP6, 21),
-        ("10.1", OsVersion.SP6, 17),
-        # (10.1, OsVersion.SP7, 21),
-    )
-]
 
-TOMCAT_CRATE = ContainerCrate(TOMCAT_CONTAINERS)
+
+TOMCAT_CONTAINERS: list[MultiBuildObsPackage | ApplicationStackContainer] = [
+    MultiBuildObsPackage.from_bcis(
+        bcis=[
+            _create_tomcat_container(os_version, tomcat_ver, jre_version)
+            for tomcat_ver, os_version, jre_version in (
+                ("10.1", OsVersion.TUMBLEWEED, 22),
+                ("10.1", OsVersion.TUMBLEWEED, 21),
+                ("10.1", OsVersion.TUMBLEWEED, 17),
+            )
+        ],
+        package_name="apache-tomcat-10-image",
+    ),
+    _create_tomcat_container(OsVersion.TUMBLEWEED, "9", 17),
+    MultiBuildObsPackage.from_bcis(
+        package_name="sac-apache-tomcat-image",
+        bcis=[
+            _create_tomcat_container(os_version, tomcat_ver, jre_version)
+            for tomcat_ver, os_version, jre_version in (
+                ("10.1", OsVersion.SP6, 21),
+                ("10.1", OsVersion.SP6, 17),
+            )
+        ],
+    ),
+]

src/bci_build/package/apache_tomcat.py

@@ -1,7 +1,7 @@
 from bci_build.container_attributes import BuildType
-from bci_build.containercrate import ContainerCrate
 from bci_build.os_version import OsVersion
 from bci_build.package import DevelopmentContainer
+from bci_build.package.obs_package import MultiBuildObsPackage
 
 _BASE_KWARGS = {
     "name": "test",
@@ -14,16 +14,20 @@
 
 
 def test_multibuild_with_multi_flavor_docker():
-    containers = [
-        DevelopmentContainer(
-            **_BASE_KWARGS,
-            build_recipe_type=BuildType.DOCKER,
-            build_flavor=flavor,
-        )
-        for flavor in ("flavor1", "flavor2")
-    ]
+    pkg = MultiBuildObsPackage(
+        package_name="test",
+        bcis=[
+            DevelopmentContainer(
+                **_BASE_KWARGS,
+                build_recipe_type=BuildType.DOCKER,
+                build_flavor=flavor,
+            )
+            for flavor in ("flavor1", "flavor2")
+        ],
+        os_version=_BASE_KWARGS["os_version"],
+    )
     assert (
-        ContainerCrate(containers).multibuild(containers[0])
+        pkg.multibuild
         == """<multibuild>
     <package>flavor1</package>
     <package>flavor2</package>