extras-buildsys/client buildclient.py,1.8,1.9

[Daniel Williams (dcbw)]

Author: dcbw

Update of /cvs/fedora/extras-buildsys/client
In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv23432/client

Modified Files:
	buildclient.py 
Log Message:
2005-06-14  Dan Williams <dcbw at redhat.com>

    * client/buildclient.py
        - Write mock output to a log on failure

    * server/buildmaster.py -> server/BuildMaster.py

    * server/BuildJob.py
      server/UserInterface.py
      server/buildserver.py
        - Fix for buildmaster.py -> BuildMaster.py

    * server/BuildJob.py
      server/BuildMaster.py
        - BuildMaster object now has a 'createrepo' method, which BuildJobs
            call when they need to update the repo.  BuildMaster.createrepo() is
            locked so that we never run two createrepos at the same time

    * server/CONFIG.py
        - Clarify some options

    * server/UserInterface.py
        - Remove list_waiting_jobs and list_building_jobs, we now have a more
            flexible "list_jobs" interface that will be extended to allow more
            search parameters
        - In update_clients(), return new clients that we've found

    * server/client_manager.py
        - Print out clients we find when we start up
        - In update_clients(), return new clients that we've found

    * utils/package-builder.py
        - Implement most of remaining functionality:
            - hook up enqueue and enqueue_srpm commands
            - Add a list_own_jobs command
            - grab user's email address from certificate or ~/.package-builder
        - Allow non-SSL connections to build server




Index: buildclient.py
===================================================================
RCS file: /cvs/fedora/extras-buildsys/client/buildclient.py,v
retrieving revision 1.8
retrieving revision 1.9
diff -u -r1.8 -r1.9
--- buildclient.py	14 Jun 2005 16:41:00 -0000	1.8
+++ buildclient.py	15 Jun 2005 04:10:02 -0000	1.9
@@ -142,11 +142,12 @@
                     else:
                         print "Bad status %s encountered!" % self._status
                 elif exit_status > 0:
-                    print "%s: job failed! mock exit status was %d\n" % (self._uniqid, exit_status)
-                    print "---------------------------------"
-                    for line in self._pobj.fromchild:
-                        print line
-                    print "---------------------------------"
+                    # Write out the failure log
+                    fail_log = os.path.join(self._result_dir, "mock_failure.log")
+                    f = open(fail_log, "w+")
+                    f.write("%s: job failed! mock exit status was %d\n\n" % (self._uniqid, exit_status))
+                    f.writelines(self._pobj.fromchild)
+                    f.close()
                     self._status = 'failed'
                     self._files = self._find_files()
                 else: