#8111 More exception handling for ftbfs
Merged 4 years ago by kevin. Opened 5 years ago by mohanboddu.
mohanboddu/releng ftbfs-exception  into  master

@@ -96,11 +96,17 @@ 

          attach_logs(bug, logs)

      except Fault as ex:

          print(ex)

-         username = raw_input('Bugzilla username: ')

-         BZCLIENT.login(user=username,

-                        password=getpass.getpass())

-         return report_failure(massrebuild, component, task_id, logs, summary,

-                               comment)

+         #sometimes there wont be BZ component, for ex: Fedora-KDE-Live

+         #skip those packages/components

+         if ex.faultCode == 51:

+             print(ex.faultString)

+             return None

+         else:

+             username = raw_input('Bugzilla username: ')

+             BZCLIENT.login(user=username,

+                            password=getpass.getpass())

+             return report_failure(massrebuild, component, task_id, logs, summary,

+                                   comment)

      return bug

  

  def attach_logs(bug, logs):
@@ -110,7 +116,16 @@ 

  

      for log in logs:

          name = log.rsplit('/', 1)[-1]

-         response = urllib2.urlopen(log)

+         try:

+             response = urllib2.urlopen(log)

+         except urllib2.HTTPError, e:

+             #sometimes there wont be any logs attached to the task.

+             #skip attaching logs for those tasks

+             if e.code == 404:

+                 print("Failed to attach {} log".format(name))

+                 continue

+             else:

+                 break

          fp = tempfile.TemporaryFile()

  

          CHUNK = 2 ** 20

rebased onto 39a5d8b

5 years ago

Pull-Request has been merged by kevin

4 years ago
Metadata