koji: Log exception on missing remote branches

FTR, I introduced the check in a previous commit, and removed the
tuple containing the results because it is useless and not working
when there is no result, this change is an updated version of the test
to trigger and exception which will be ignored and attempt to look for
commit in other branches, a log may be printed to help trouble shoot.

Fix redefined-builtin mistake
(BTW I noticed similar mistakes elsewhere to be fixed in later PR)

Fix indantation mistake from previous commit

Signed-off-by: Philippe Coval <philippe.coval@vates.tech>

Author: rzr

scripts/koji/koji_build.py

@@ -56,12 +56,15 @@ def check_commit_is_available_remotely(dirpath, hash, target, warn):
             supported_branches = PROTECTED_TARGETS.get(target)
             if supported_branches:
                 for branch in supported_branches:
-                    if is_remote_branch_commit(dirpath, hash, branch):
-                        logging.debug(f"commit {hash} is on top of branch {branch} (target: {target})")
-                        break
+                    try:
+                        if is_remote_branch_commit(dirpath, hash, branch):
+                            logging.debug(f"commit {hash} is on top of branch {branch} (target: {target})")
+                            break
+                    except Exception as e:
+                        logging.debug(f"{e} Ignoring")
                 else:
                     raise Exception(f"The current commit is not the last commit of any remote branches {supported_branches}.\n"
-                                f"This is required when using the protected target {target}.")
+                                    f"This is required when using the protected target {target}.")
             else:
                 raise Exception(f"Protected target {target} has no allowed branches.")
         except Exception as e:
@@ -157,8 +160,10 @@ def push_bumped_release(git_repo, target, test_build_id, pre_build_id):
 
 def is_remote_branch_commit(git_repo, sha, branch):
     with cd(git_repo):
-        list = subprocess.check_output(['git', 'ls-remote', 'origin', f'refs/heads/{branch}']).decode().strip().split()
-        remote_sha = list[0] if list else None
+        references = subprocess.check_output(['git', 'ls-remote', 'origin', f'refs/heads/{branch}']).decode().strip().split()
+        if not references:
+            raise Exception(f'Missing remote branch "{branch}"')
+        remote_sha = references[0]
     return sha == remote_sha
 
 def build_id_of(name, candidate):