From ca943c6864199f9fdd8a92b9b9a95630e1c0e017 Mon Sep 17 00:00:00 2001
From: Muzahid <muzahid.h@cisinlabs.com>
Date: Tue, 22 Feb 2022 21:15:15 +0530
Subject: [PATCH] chenges for dockerfile and oscodename

---
 lib/worker_multibuild.py | 13 +++++--------
 multibuild.py            | 20 ++++++++++----------
 2 files changed, 15 insertions(+), 18 deletions(-)

diff --git a/lib/worker_multibuild.py b/lib/worker_multibuild.py
index 9f48875..7fd7b14 100644
--- a/lib/worker_multibuild.py
+++ b/lib/worker_multibuild.py
@@ -15,8 +15,8 @@ request_headers = {
 
 ty = "/change_hook/base"
 path =".buildbot"
-
-dockerfile_extra_contents_focal = """
+dockerfile_extra_contents = {}
+dockerfile_extra_contents['focal'] = """
 
 # Buildbot
 RUN apt-get install -yq --no-install-suggests --no-install-recommends \
@@ -35,7 +35,7 @@ ENTRYPOINT /usr/local/bin/buildbot_entrypoint.sh "$BUILDMASTER" "$WORKERNAME" "$
 
 """
 
-dockerfile_extra_contents_bionic = """
+dockerfile_extra_contents['bionic'] = """
 
 # Buildbot
 RUN apt-get install -yq --no-install-suggests --no-install-recommends \
@@ -69,7 +69,7 @@ def list_jobs(directory=".buildbot"):
     return results
 
 
-def _get_dockerfile_contents(jobname, os_codename):
+def _get_dockerfile_contents(jobname, os_codename='bionic'):
     """
     Read contents of a Dockerfile and add extra contents for the given os_codename
     """
@@ -78,10 +78,7 @@ def _get_dockerfile_contents(jobname, os_codename):
         # remove any line containing FROM or RUN keywords
         re.match(r"(?m)^(FROM|RUN).*$", "", contents)
 
-    return contents + {
-        "focal": dockerfile_extra_contents_focal,
-        "bionic": dockerfile_extra_contents_bionic,
-    }[os_codename]
+    return contents + dockerfile_extra_contents[os_codename]
 
 
 def trigger_child_hooks(buildbotUrl: str, os_codename: str, repository, branch, jobname, directory=".buildbot", is_build_script_available, is_test_script_available):
diff --git a/multibuild.py b/multibuild.py
index c1ea83e..7687624 100644
--- a/multibuild.py
+++ b/multibuild.py
@@ -29,9 +29,9 @@ def find_artifacts(directory="out"):
         return join(directory, _)
 
 
-@util.renderer
-def get_dockerfile_contents(props):
-    return _get_dockerfile_contents(props.getProperty('jobname', default=None), util.Interpolate("%(prop:os_codename)s"))
+# @util.renderer
+# def get_dockerfile_contents(props):
+#     return _get_dockerfile_contents(props.getProperty('jobname', default=None), util.Interpolate("%(prop:os_codename)s"))
 
 
 def add_parent_step(build_factory):
@@ -44,13 +44,13 @@ def add_parent_step(build_factory):
         command=["sudo", "wget", "-O", "https://git.bitmessage.org/Bitmessage/buildbot_multibuild/raw/branch/master/lib/worker_multibuild.py", "worker_multibuild.py"]
     ))
 
-    build_factory.addStep(
-        steps.SetPropertyFromCommand(
-            name="Get OS codename",
-            command="grep ^VERSION_CODENAME= /etc/os-release | cut -d= -f2",
-            property="os_codename",
-        )
-    )
+    # build_factory.addStep(
+    #     steps.SetPropertyFromCommand(
+    #         name="Get OS codename",
+    #         command="grep ^VERSION_CODENAME= /etc/os-release | cut -d= -f2",
+    #         property="os_codename",
+    #     )
+    # )
     
     build_factory.addStep(
         steps.ShellCommand(