Merge pull request #251 from laraPPr/bot_status

fix for `bot: status` was calling wrong values which caused the `bot: status` to fail
EESSI · Feb 13, 2024 · eb5eb70 · eb5eb70
2 parents 23dd5fe + 0dc3650
commit eb5eb70
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/tasks/build.py b/tasks/build.py
@@ -794,7 +794,7 @@ def request_bot_build_issue_comments(repo_name, pr_number):
                 first_line = comment['body'].split('\n')[0]
                 arch_map = get_architecture_targets(cfg)
                 for arch in arch_map.keys():
-                    target_arch = '/'.join(arch.split('/')[1:])
+                    target_arch = '/'.join(arch.split('/')[-1])
                     if target_arch in first_line:
                         status_table['arch'].append(target_arch)
 
@@ -826,7 +826,7 @@ def request_bot_build_issue_comments(repo_name, pr_number):
                         status_table['url'].append(comment['html_url'])
                         if 'FAILURE' in row['comment']:
                             status_table['result'].append(':cry: FAILURE')
-                        elif 'SUCCESS' in value['comment']:
+                        elif 'SUCCESS' in row['comment']:
                             status_table['result'].append(':grin: SUCCESS')
                         elif 'UNKNOWN' in row['comment']:
                             status_table['result'].append(':shrug: UNKNOWN')