add logs in osbs build log to indicate platform specific

build has started * CLOUDBLD-10692 Signed-off-by: Harsh Modi <hmodi@redhat.com>
containerbuildsystem · Aug 22, 2022 · b0515e8 · b0515e8
1 parent 1c69519
commit b0515e8
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/koji_containerbuild/plugins/builder_containerbuild.py b/koji_containerbuild/plugins/builder_containerbuild.py
@@ -435,6 +435,10 @@ def _write_logs(self, build_id, logs_dir, platforms: list = None):
                 if (task_platform != 'noarch') and (task_platform not in final_platforms):
                     continue
 
+                if task_platform != 'noarch':
+                    logfiles['noarch'].write(bytearray(f'{task_platform} build has started. '
+                                                       'Check platform specific logs\n', 'utf-8'))
+
                 log_filename = f'{task_platform}.log'
                 logfiles[task_platform] = open(os.path.join(logs_dir, log_filename),
                                                'wb')

diff --git a/tests/test_builder_containerbuild.py b/tests/test_builder_containerbuild.py
@@ -133,6 +133,11 @@ def check_meta_entry(filename):
             else:
                 if task_platform != 'noarch':
                     log_filename = f'{task_platform}'
+                    if log_filename not in log_contents:
+                        log_contents[log_name] = '{old}{new}\n'.format(
+                            old=log_contents.get(log_name, ''),
+                            new=f'{task_platform} build has started. '
+                                'Check platform specific logs')
                 else:
                     log_filename = log_name
                 log_contents[log_filename] = '{old}{new}\n'.format(
@@ -203,7 +208,12 @@ def test_write_logs(self, tmpdir, get_logs_exc, build_not_finished, final_platfo
 
         koji_tmpdir = tmpdir.mkdir('koji')
         koji_tmpdir.join('x.log').write('line 1\n'
-                                        'line 2\n')
+                                        'line 2\n'
+                                        'x86_64 build has started. Check platform specific logs\n'
+                                        's390x build has started. Check platform specific logs\n'
+                                        'ppc64le build has started. Check platform specific logs\n'
+                                        'aarch64 build has started. Check platform specific logs\n'
+                                        )
 
         (flexmock(koji.pathinfo)
             .should_receive('work')