← Back to branch summary

~launchpad-pqm/launchpad/devel

« back to all changes in this revision

Viewing changes to lib/lp/soyuz/model/queue.py

  • Committer: Julian Edwards
  • Date: 2011-07-07 10:31:38 UTC
  • mto: This revision was merged to the branch mainline in revision 13385.
  • Revision ID: julian.edwards@canonical.com-20110707103138-urvwpih6m5h8mvgb
Make the PackageCopyJob runner responsible for setting PackageUpload's 
status to done instead of the process-accepted script.

Show diffs side-by-side

added added

removed removed

Lines of Context:
lib/lp/soyuz/model/queue.py
720
720
 
721
721
    def realiseUpload(self, logger=None):
722
722
        """See `IPackageUpload`."""
 
723
        if self.package_copy_job is not None:
 
724
            # PCJs are "realised" in the job runner,
 
725
            # which creates publishing records using the packagecopier.
 
726
            # Because the process-accepted script calls realiseUpload for
 
727
            # any outstanding uploads in the ACCEPTED state we need to skip
 
728
            # them here.  The runner is also responsible for calling
 
729
            # setDone().
 
730
            return
723
731
        # Circular imports.
724
732
        from lp.soyuz.scripts.packagecopier import update_files_privacy
725
733
        assert self.status == PackageUploadStatus.ACCEPTED, (